connery in openGPTs

I config the connery with gmail , but when I access it from openGPTs, the output is error, the log is "runner:start: {"type":"all-exceptions-filter","message":"Not Found"}"
runner:start: [4:01:34 AM] Found 0 errors.
I'm most interested in:

Environment (Do you set up and run both OpenGPTs and Connery locally?)
Steps to set up (Which versions/branches did you set up the OpenGPTs/Connery/Gmail plugin from?)

Hi, could this issue be reopened ? I’m experiencing the same problem while trying to communicate between the runner and my OpenGPTs client :
{"type":"all-exceptions-filter","message":"Not Found"}"
The Connery runner is hosted on Codespaces with the Gmail plugin. I used the CONNERY_RUNNER_URL without a trailing slash in my local OpenGPTs setup, along with the CONNERY_RUNNER_API_KEY provided by the runner.”
curl -X GET CONNERY_RUNNER_URL -H "x-api-key: CONNERY_RUNNER_API_KEY" works fine
{"status":"success","data":{"message":"Welcome to the Connery Runner API 👋"}}%
Thanks

Sorry, I need to provide additional information. I am currently using an older version of Connery v0.0.7 after testing v0.0.8 and v0.3.5. The Gmail plugin is<EMAIL_ADDRESS>OpenGPTs is running locally using Docker Compose. It has worked before (I successfully sent a webpage summary to an email address). I must say, it’s frustrating now!

Hi @drzagor, thank you for reaching out. I’m sorry to hear about your frustration with the SDK.
We recently made significant changes to align the SDK with our new strategy, resulting in breaking changes. The previous approach using the Runner is no longer supported. Instead, the latest SDK architecture wraps each plugin as a standalone server (Plugin Server), which can be hosted anywhere, including serverless environments like AWS Lambda. Each Plugin Server exposes a REST API, allowing interaction and action execution.
At the moment, OpenGPTs are not supported in the new SDK, as we haven’t observed sufficient demand in that area.
However, you can still use the plugins packaged with the latest SDK version alongside OpenGPTs by creating a custom LangChain wrapper and calling the actions via the Plugin Server REST API.
Please note that the SDK is still in beta and has no major version yet. As such, there may be occasional small breaking changes until we release the major version.
Thank you for your understanding.

Thank you for the detailed response and for clarifying the recent changes to the SDK. I appreciate your advice on how to continue using the plugins with OpenGPTs through a custom LangChain wrapper. I understand the challenges during the beta phase and look forward to future updates.


Can not login after changing password

After i update new password, i can not login with new password until i restart the application.
My code:
var result = await _userManager.ChangePasswordAsync(user, userModel.CurrentPassword, userModel.Password);
Do you have any suggest?

This issue was moved to aspnet/Identity#1068


egressgateway: Remove unnecessary check on entry deletions

When reconciling the egress gateway map with the corresponding k8s policies, we first (1) add/update any missing/outdated entries then (2) remove all stale entries. Currently, in that second part, we remove any entry that has either a different key or a different value. It is however not necessary to check the value because any outdated value will already be updated as part of (1).
If (1) was buggy and left some outdated values, then this commit would cause a behavior change: instead of removing any (k,v) entry that has an incorrect value v, we would leave it there. In both cases, the resulting map won't match the k8s policies anyway.

/test


Add package information to output string

If you have lots of these tests in a solution, this makes it a little easier to see which ones are failing.

True, that is more aesthetically pleasing.


Error with custom database

Hi,
I am trying to run Emu with a custom Marjaam database. I could successfully built the custom database and then while running I am getting the below error. Not sure is something worng with the database or something else. Can you please help?
Processing /zfs/omics/personal/rjain/AML1_2/filtered/barcode95_filtered.fastq
[M::mm_idx_gen::0.0061.83] collected minimizers
[M::mm_idx_gen::0.0093.89] sorted minimizers
[M::main::0.0093.87] loaded/built the index for 383 target sequence(s)
[M::mm_mapopt_update::0.0103.70] mid_occ = 357
[M::mm_idx_stat] kmer size: 15; skip: 10; is_hpc: 0; #seq: 383
[M::mm_idx_stat::0.0103.63] distinct minimizers: 5960 (67.67% are singletons); average occurrences: 5.438; average spacing: 5.787; total length: 187553
[M::worker_pipeline::0.09714.26] mapped 71 sequences
[M::main] Version: 2.26-r1175
[M::main] CMD: minimap2 -ax map-ont -t 24 -N 50 -p .9 -K 500000000 -o ./Results-emu/barcode95_filtered_emu_alignments.sam /zfs/omics/personal/rjain/emu_maarjam/marjaam/species_taxid.fasta /zfs/omics/personal/rjain/AML1_2/filtered/barcode95_filtered.fastq
[M::main] Real time: 0.099 sec; CPU: 1.381 sec; Peak RSS: 0.053 GB
Traceback (most recent call last):
File "/home/rjain/miniconda3/envs/py37/bin/emu", line 804, in
SAM_FILE, log_prob_cigar_op, longest_align_dict, locs_p_cigar_zero)
File "/home/rjain/miniconda3/envs/py37/bin/emu", line 195, in log_prob_rgs_dict
dict_longest_align, align_len)
File "/home/rjain/miniconda3/envs/py37/bin/emu", line 167, in compute_log_prob_rgs
species_tid = int(ref_name.split(":")[0])
ValueError: invalid literal for int() with base 10: 'Y17635'
Processing /zfs/omics/personal/rjain/AML1_2/filtered/barcode96_filtered.fastq
[M::mm_idx_gen::0.0071.90] collected minimizers
[M::mm_idx_gen::0.0093.76] sorted minimizers
[M::main::0.0093.75] loaded/built the index for 383 target sequence(s)
[M::mm_mapopt_update::0.0103.58] mid_occ = 357
[M::mm_idx_stat] kmer size: 15; skip: 10; is_hpc: 0; #seq: 383
[M::mm_idx_stat::0.0103.52] distinct minimizers: 5960 (67.67% are singletons); average occurrences: 5.438; average spacing: 5.787; total length: 187553
[M::worker_pipeline::0.80521.47] mapped 1113 sequences
[M::main] Version: 2.26-r1175
[M::main] CMD: minimap2 -ax map-ont -t 24 -N 50 -p .9 -K 500000000 -o ./Results-emu/barcode96_filtered_emu_alignments.sam /zfs/omics/personal/rjain/emu_maarjam/marjaam/species_taxid.fasta /zfs/omics/personal/rjain/AML1_2/filtered/barcode96_filtered.fastq
[M::main] Real time: 0.808 sec; CPU: 17.295 sec; Peak RSS: 0.070 GB
Traceback (most recent call last):
File "/home/rjain/miniconda3/envs/py37/bin/emu", line 804, in
SAM_FILE, log_prob_cigar_op, longest_align_dict, locs_p_cigar_zero)
File "/home/rjain/miniconda3/envs/py37/bin/emu", line 195, in log_prob_rgs_dict
dict_longest_align, align_len)
File "/home/rjain/miniconda3/envs/py37/bin/emu", line 167, in compute_log_prob_rgs
species_tid = int(ref_name.split(":")[0])
ValueError: invalid literal for int() with base 10: 'AM295493'
Thanks tons!

Y17635; AM295493... these seems to be the tax_id in my database.


helm: add option to enable automatic etcd name resolution

This feature is useful if users want to deploy an k8s service
that translates to an etcd cluster. As already implemented in Cilium, it
will perform the service translation internally without having to reach
a DNS to perform this translation. This will work by relying on the
watching mechanism that we have for k8s for which it is required that
disabled-k8s-services is set to false
Signed-off-by: André Martins<EMAIL_ADDRESS>

test-me-please

Coverage decreased (-0.01%) to 46.674% when pulling 1e31276c396c99d2ee9bb29aed3205deb46cb8a1 on pr/add-etcd-operator-fallback-docs into 8804f86828f21f0af31b3e01efe97f375564cc23 on master.


[Docs] Add all mesheryctl guides to bulleted collection

Current State
This page category for mesheryctl guides (https://docs.meshery.io/guides/mesheryctl) is missing a number of mesheryctl guides.

Desired State
All of the mesheryctl guides should be listed on the page.

Contributor Guides and Resources

📚 Instructions for contributing to documentation

Meshery documentation site and source

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

Hey @leecalcote , can i work on this one or should this be left for a newcomer in the community?
Anything works 🙂

Hi @leecalcote & @verma-kunal,
If this is for newcomers only,  I can take it up. Thanks.

Hi @leecalcote & @verma-kunal, If this is for newcomers only, I can take it up. Thanks.

absolutely @Vidyaa123 ! Go for it.
Do ask any doubts you would have! Would be happy to help 🙂

Hi @leecalcote & @verma-kunal, If this is for newcomers only, I can take it up. Thanks.

absolutely @Vidyaa123 ! Go for it. Do ask any doubts you would have! Would be happy to help 🙂

👍

@verma-kunal & @leecalcote please review.


'nn' requirement missing

The package 'nn' is a dependency for 'cunn' but it does not appear on the list in the README. Should probably be added to the list.
Thanks

thanks! i've updated the readme to reflect this


Map imagery out of date

The base map imagery is severely out of date, making my OSC tracks not follow the correct roadways. Can we get a map update, please?

duplicate of https://github.com/openstreetcam/openstreetview.org/issues/16

Per the developer's response in #16:

The web tiles have been updated with fresh (-er) maps, from April 2018. The team's plan is to keep them updated, I'm closing the ticket until proven otherwise.


Atom 1.25.0b3 does not stop writing on the hard drive

Hello I would not know if it is a mistake in itself, but it happens a lot from the previous beta, I notice that Atom 1.25.0b3 freezes a bit the pc, and when I do some tests my memory consumption is normal, but the writing on hard disk it is always above 90%, and when closing the atom lowers to its normal level when I tried the previous version (1.25.0b2) this improved a lot, even consumed but much less, I did a test more, and opened a project with nodemon and use version 1.25.0b3 to verify the files and folders and nodemon does not stop rebooting (photo 1) to stop atom, everything runs perfectly, I did this same test in 1.25.0b2 and all good (photo 2)

Can you explain how we can reproduce this issue? If you need help writing detailed repro steps, please see the How Can I Contribute? section of the Atom CONTRTIBUTING guide.
Since you didn't fill out the template there are several bits of information missing here preventing us from being able to reproduce this. Can you make sure you have followed all of these steps, especially the first one about Safe Mode?

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom

Thanks for reaching out!

Thank you for your issue!
We haven't gotten a response to our questions in our comment https://github.com/atom/atom/issues/16927#issuecomment-372147816. With only the information that is currently in the issue, we don't have enough information to take action. I'm going to close this but don't hesitate to reach out if you have or find the answers we need, we'll be happy to reopen the issue.

fix on release 1.26.0b0


fix: Mutability in the comptime interpreter

Description
Problem*
Resolves https://github.com/noir-lang/noir/issues/5504
Summary*
Mutable variables weren't wrapped in a mutable reference before like they are in SSA - they are now.
Additional Context
Documentation*
Check one:

[x] No documentation needed.
[ ] Documentation included in this PR.
[ ] [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Closing in favor of https://github.com/noir-lang/noir/pull/5517


Create tests for videolink logic from EditProfile.tsx

Create some tests in different situations for videolink logic in EditProfile.tsx.

Link the issue:


Deprecate some long overdue functions

Describe changes
I removed some deprecated functionality
Pre-requisites
Please ensure you have done the following:

[ ] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table and the corresponding website section.
[ ] I have added tests to cover my changes.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

Sorry it wasnt an approval yet! :D ! Also @stefannica can comment on the secret managers stuff

Thank you for doing it , i hope @stefannica knows we are throwing an error now though

A much needed measure !


Should the intro page define reflectometry?

Hi there - I'm reading this guide as someone with a strong mathematics & optimisation background, but little (no higher than A-level) theoretical physics background. This guide is well-worded and kept simple, but after reading the first section I stopped and thought "wait, what actually is reflectometry?", and had to dig around Wikipedia for a bit. The guide jumps into analysis using the field terminology, without explaining what a reflectometry profile is, what scattering length density is, why the scattering length density profile of a bilayer would be a Heaviside function, etc.
 I found I had to simultaneously have Chapter 3 of Sivia's book open in order to understand what we were actually analysing, and to define symbols which are not defined in the guide - what is $q$? What is $q_x$? What is $q_y$?
Even though analysis is the main focus of the guide, I think it would be worth having a short page defining the actual concepts of what we are doing and what we are measuring, e.g. something like

What is reflectometry?
Neutron or X-ray reflectometry is a technique for analysing the properties of materials. It involves directing a beam of neutrons or X-rays from a flat surface, and measuring the way in which these neutrons or X-rays reflect from the surface, known as reflectivity.
Reflectivity can be measured using scattering length density, which is...

I think it may also be worth having the slab model page moved to this introduction - as soon as I saw Fig. 4, the scattering length density profile being a Heaviside function made immediate intuitive sense, and it would be instructive to explain scattering length density using this example.

Hey Alex, thanks for this. There is actually a hackathon event to work on this material in a few weeks (as part of the SXNS conference) so I will make this a priority for this event. That said, if you would like to make this change and open a pull request, you are more than welcome to!


Fix draw line

I have fixed the draw line (mistake in the shader) and added a condition to avoid an assert on the asset editor @DorianPinaud if you could see that, it seems like the bug comes from the vertice manager.

Cool ! Super ! Je test ca ASAP.
N'oublie pas de rajouter les Close #NbrDeLissue
:+1:

Tested ! Good !!! Zuper ! :+1:


Changes to homepage

Is your feature request related to a problem? Please describe.

The text "become a volunteer" is overflowing out of the button.
There is no image on the left of contact form.
The hamburger button is too small on mobile.
Adding a scrolling effect to about us will also create a more pleasing effect for the website.

Describe the solution you'd like

Would like to set the width to auto on mobile for button.
Would like to add an image for the contact us form.
Would like to add a scroll effect on about us

Describe alternatives you've considered

Could make become volunteer appear as a drop-down on mobile.

Screenshots

I have read the ReadMe and I am eager to make contributions to this project. Kindly assign me this @Stiffpixels

@akshitagupta15june  please assign me this issue.

Also assign  necessary labels to them

I want to work on this please assign me this under GSSOC 2024 I'll give my 100% .


Extensible Resource Deployment Observability Improvements

Recreation of #3.
Includes changes from my feedback on that PR.

I'm going to close in this in favor of including this design in https://github.com/Azure/bicep-reps/pull/6. Recent iterations of that change the fields added in this REP to handle various stacks deployment edge cases in tracing extensible resources.


Aborting apply when creating a stack puts it in a bad state

After aborting an apply command (with Ctrl+C), on an initial stack creation, I got the following error:
/usr/local/bundle/gems/stack_master-0.16.0/lib/stack_master/stack.rb:20:in `template_default_parameters': undefined method `fetch' for false:FalseClass (NoMethodError)

After deleting the stack this stopped happening.
I suspect this was introduced by https://github.com/envato/stack_master/pull/178/files#diff-cf22d74c8f612a3282d758a02c9d21e7R83, because we may create an empty stack that won't get cleaned up if stack_master is uncleanly terminated. We can optionally:

Hook the TERM signal and cleanup the stack
Make StackMaster properly deal with an existing, empty, stack

+1
Happened when my stack was in REVIEW_IN_PROGRESS. Deleting the stack manually sorted it out
At the very least a nicer error message would be useful.

Stacks are also stuck at REVIEW_IN_PROGRESS if user answers no when prompted to apply stack and the user/role stack_master is running under does not have delete stack permission. It might be worth adding a switch to avoid calculating the changeset even if that means we cannot preview the resources that will be affected.

This was fixed in #202

Seeing this in stack_master 1.4.0 when stack is in REVIEW_IN_PROGRESS state.

@greenu can you please open a new issue and describe the steps required to reproduce this error?


Make getExplorerInfo return values per network

In a discussion related to how to deal with getExplorerInfo (it's a function that returns the explorer URL to inspect an address or transaction). Pablo and I suggested that we should have a separate file in order to define all the functions that should handle a particular case depending on the current network.
The intention was that at the moment of creating a new network, a developer should go to src/config folder and replicate all the existing files with the new network. Giving it a second thought I think it is an overkill, at the moment getExplorerInfo it's the only function that needs a special treatment depending on the network.
Another alternative is to create another file utils.ts (?) where we can handle all these particular cases, but we will end having an index.ts and a utils.ts where won't be crystal what's the purpose of each one.
What I suggest to do is to make a little refactor in the current  getExplorerInfo.  That's what this PR does.

Travis automatic deployment:
https://pr1470--safereact.review.gnosisdev.com/app


fix: incorrect alacritty bundle identifier

Resolves #91

https://github.com/Strong-Foundation/Karabiner-Elements-Remapping
@elijaholmos
Use this brother.
I already added your fix and fixed it for all my users already.


Incorrect "Relative Time Format" in demo for years before 100 AD

OpenUI5 version: 1.40.8
Browser/version (+device/version): Not browser dependent - Replicated on Chrome and Firefox
URL (minimal example if possible):
https://openui5.hana.ondemand.com/explored.html#/sample/sap.ui.core.sample.TypeDateAsDate/preview
Steps to reproduce the problem:

From the date picker select a date before 1 Jan 0100 (e.g. Apr 19, 0098)

What is the expected result?
"Relative time" should be calculated to something like "1915 years ago"
What happens instead?
"Relative Time: 18 years ago" (year is treated as 1998)

Hi @naskog,
Thanks for the ticket.
Internal incident<PHONE_NUMBER> has been opened for this issue.


[charts] drop support for Helm 2 for all charts

Support for Helm 2 will be discontinued at the end of 2020 September by all Bank-Vaults charts.
This means that they might work with Helm 2 as well, but we don't test and offer support installing/upgrading them with that Helm 2 version line.
See https://helm.sh/blog/2019-10-22-helm-2150-released/ for more details about the topic.

12 months after Helm 3's public release, support for Helm 2 will formally end.
Tue, Oct 22, 2019

fixes: #1059

Is this happening? 🙂
If you're looking for help for this, I'd be happy to contribute. We want to use the charts but the v1/v2 situation breaks helm lint which is required in our CI/CD setup.

Hi @patoarvizu I got distracted a bit by other projects, so if you have a working solution I would be happy to review that PR! 🙂


Cannot view resources correctly after creating APIs from swagger definitions

Description:
Cannot view resources correctly after creating APIs from swagger definitions
Steps to reproduce:
Issue1

Create an API by using swagger content with parameters that referring to the parameters component.

openapi: 3.0.0

info:
  version: "1.0.0"
  title: TEST API
  description: >-
    Test Description

servers:
  - url: https://localhost:8080
security:
  - default: []

paths:
  "/test/something":
    get:
      security:
        - default: []
      x-auth-type: Application
      summary: sample summary
      description: >-
        sample description
      operationId: operation1
      parameters:
        - $ref: '#/components/parameters/param1'
        - $ref: '#/components/parameters/param2'
        - $ref: '#/components/parameters/param3'
      responses:
        '200':
          description: OK
components:
  parameters:
    param1:
      in: query
      name: param1
      description: "description"
      schema:
        type: integer
        default: 0
        minimum: 0
    param2:
      in: query
      name: param2
      description: "description"
      schema:
        type: integer
        default: 0
        minimum: 0
    param3:
      in: query
      name: param3
      description: "description"
      schema:
        type: integer
        default: 0
        minimum: 0

  securitySchemes:
    default:
      type: oauth2
      flows:
        implicit:
          authorizationUrl: 'https://www.test.com'
          scopes: {}

After creating the API, Go to the Resources section and could not able to see parameter names as below image.

Issue 2

Create an API by using swagger content with global request parameters.

openapi: 3.0.0

info:
  version: "1.0.0"
  title: TEST API
  description: >-
    Test Description

servers:
  - url: https://localhost:8080
security:
  - default: []

paths:
  "/test/something":
    parameters:
      - $ref: '#/components/parameters/xRequestId'
    get:
      security:
        - default: []
      x-auth-type: Application
      summary: sample summary
      description: >-
        sample description
      operationId: operation1
      parameters:
        - $ref: '#/components/parameters/offset'
      responses:
        '200':
          description: OK
components:
  parameters:
    xRequestId:
      in: header
      name: X-Request-ID
      description: The correlation Id.
      schema:
        $ref: "#/components/schemas/X-Request-Id"
      required: true
    offset:
      in: query
      name: offset
      description: "description"
      schema:
        type: integer
        default: 0
        minimum: 0
  schemas:
    X-Request-Id:
      type: string
      format: uuid
      description: The correlation id, as received.
      example:
        "123e4567-e89b-12d3-a456-426614174000"

  securitySchemes:
    default:
      type: oauth2
      flows:
        implicit:
          authorizationUrl: 'https://www.test.com'
          scopes: {}

After creating the API, navigated to the Resources section and able to observe the below UI error.

Affected Product Version:
3.0.0

This has been fixed in the master branch and it has been ported to 3.0.0.
https://github.com/wso2/carbon-apimgt/pull/8272/commits/3ce2c37fde2c4fb717c71ce172d61a19e868cd2d)


Implement doc checkers

Reference: https://github.com/kubernetes/test-infra/issues/24821#issuecomment-1057524854
As per the design doc, we should implement automated checkers to make sure we uphold minimum quality standard for both existing documentation as well as newly proposed documentation.
There have been some discussions in the design doc already, but feel free to comment on here about which checker(s) to implement first.

/reopen
/lifecycle frozen
Here are tools I saw:

K8s doc's Internal link checking tool

According to the doc, the tool

Builds the site and generates output HTML in the /public directory of your local kubernetes/website repository
Pulls the wdjp/htmltest Docker image
Mounts your local kubernetes/website repository to the Docker image
Scans the files generated in the /public directory and provides command line output when it encounters broken internal links

and its workflow seems similar to the current link checker in this Prow doc repo.
https://github.com/kubernetes-sigs/prow/blob/0d8946d8dccdcbbb1dc00c3c20641b202b0494a8/site/Makefile#L26-L27
https://github.com/kubernetes-sigs/prow/blob/0d8946d8dccdcbbb1dc00c3c20641b202b0494a8/site/check-broken-links.sh#L22-L30

K8s doc's linkchecker.py

While the above linkcheckers analyze whether the links in the Hugo-generated HTML files are working or not (404, ...),
this linkchecker performs sort of static analysis on markdown files.

And yes, it would be great to implement a new linkchecking Prow workflow

for every PR, and/or
executed periodically

/reopen
Does not feel like an issue that should rot - either we think this is useful, or we should explicitly decide we don't want that.

I think the first option (https://github.com/kubernetes/website/tree/main/content/en/docs/doc-contributor-tools/linkchecker) could be adopted. It is simpler as it already uses the same underlying tool (htmltest) that we currently use.
The first step is to add it as a CI job (either the same link checker as used in kubernetes/website or the existing Make rule we have). I don't think it makes sense to add it as a presubmit as links can break sporadically. Probably running the job 2x a day would be good so that we can get some signal that all links are OK.


ReactLayout rendering on top of BlazeLayout and vice versa

I have routes which use both BlazeLayout and ReactLayout. This is because I am using meteor-useraccounts/flow-routing which only supports Blaze at the time.
The Problem is when a browser calls a route which uses ReactLayout and the follows to a page with BlazeLayout or vice versa. The result is two layouts displayed one underneath the other.
FlowRouter.route('/', {
  name : "start",
  action: function(params, queryParams) {
    BlazeLayout.render("TopAppLayoutBlaze", { main: "HomePage" });
  }
});

FlowRouter.route("/tests/test01", {
  action: function() {
    ReactLayout.render(X.TopAppLayout, {
      content: <SimpleComponent name="Test" />
    })
  }
});

For this case, don't mix layout engines.
Render ReactComponents inside blaze.
See: http://react-in-meteor.readthedocs.org/en/latest/react-template-helper/

Actually I was hoping to combine both as well, to show how Blaze and React can coexist in the same app while you're migrating from one to the other. But I case if it doesn't work I'll just find another way.

It's not possible to combine but it's possible to co-exists. But there is no use in that.
My suggested solution is the best idea.

Yeah I meant coexist, sorry. I actually did try your solution but it's a bit tricky in my case as I wanted to use a React layout. Anyway I'll make a video about it soon, maybe you can tell me if there's a better way to do it then.

sure.
On Sun, Dec 27, 2015 at 11:04 AM Sacha Greif<EMAIL_ADDRESS>wrote:

Yeah I meant coexist, sorry. I actually did try your solution but it's a
bit tricky in my case as I wanted to use a React layout. Anyway I'll make a
video about it soon, maybe you can tell me if there's a better way to do it
then.
—
Reply to this email directly or view it on GitHub
https://github.com/kadirahq/meteor-react-layout/issues/11#issuecomment-167382195
.

for coexisting:
BlazeLayout.reset()
ReactLayout.render(SomeReactComponent)

I have a blaze app and use flow-router-ssr with react for some SEO content pages.
To clear the React Container, I render a empty span before the blaze rendering:
FlowRouter.route('/login', {
  action: function(params, queryParams) {
    ReactLayout.render(React.createClass({ render() {return } }))
    BlazeLayout.render('skeleton', { content: "login" })
  }
})

The other way round (blaze route => react route) works without hack.

I'm curious how you would do this with React Mounter

mount(() => ());


Clean dependant select contents on change. Now only sets value to null.

Feature Request or enhancement or bug?
I don't know how to classify this, as it only allows unexpected behaviour from users :smile:
What I did
I was testing to have dependant selects, so when you select a value in one, the dependant one list of options gets refreshed with appropriate values.
What I found out
Since my computer is not the fastest in the galaxy it happens that when I update the first select, the dependant one is set to null (correct), but the list of options is not cleared. So when I navigate to the dependant one I can select an "old" value while new contents is loaded. Here is a small demo:

After changing the first select to 'Venganza' I can still select old options while correct ones are loaded. This should not be possible.
Bug or enhancement?
I know select2 has a method to clear contents (https://select2.org/programmatic-control/add-select-clear-items#clearing-selections). It also sets to null the value. Sorry, I ignore where this "set to null" is done in backpack....
I'm running this:
### PHP VERSION:
PHP 7.4.9 (cli) (built: Aug  4 2020 11:52:41) ( ZTS Visual C++ 2017 x64 )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies

### LARAVEL VERSION:
v7.24.0@71bfb1f2f89e4e31fba539d43fe287143e635e68

### BACKPACK VERSION:
4.1.16@4c2264f8dd52381aa891136edb61e8d4083a033d

Hope it helps!
Regards!

Hello @aula2030
Thanks for the bug report. I think you are right here, there is no sense in having the options laying around while they should be cleared when dependency changes.
I'v just submitted the PR #3301 that addresses this issue by removing the options from the select while setting the value to null.
I am going to close this, let's keep any further discussion regarding this matter in the PR thread.
Thanks again, best Pedro

Great! Thank you!! :+1:

Merged! Will be available with a composer update starting tomorrow, most likely. Thanks everyone!


Add mock outputs for destroy to all layers

Resolves #186
Adds mock outputs for all terragrunt layers to, hopefully fix destroys failing when layers have not been applied

/test-all

:robot: pr-bot :robot:
:runner: Running tests: https://github.com/UCLH-Foundry/FlowEHR/actions/runs/4510542119 (with refid 0804a808)
(in response to this comment from @t-young31)

:robot: pr-bot :robot:
Destroying environment (with refid 0804a808)
(in response to this comment from @t-young31)

/test-all

:robot: pr-bot :robot:
:runner: Running tests: https://github.com/UCLH-Foundry/FlowEHR/actions/runs/4531903961 (with refid 0804a808)
(in response to this comment from @t-young31)

/destroy

:robot: pr-bot :robot:
Destroying environment (with refid 0804a808)
(in response to this comment from @t-young31)

/test-all

:robot: pr-bot :robot:
:runner: Running tests: https://github.com/UCLH-Foundry/FlowEHR/actions/runs/4532894083 (with refid 0804a808)
(in response to this comment from @t-young31)

CI is still having problems with databricks. @jjgriff93 is on it

A potential fix isn't far away!

/test

:robot: pr-bot :robot:
:runner: Running tests: https://github.com/UCLH-Foundry/FlowEHR/actions/runs/4536349928 (with refid 0804a808)
(in response to this comment from @t-young31)

/test

:robot: pr-bot :robot:
:runner: Running tests: https://github.com/UCLH-Foundry/FlowEHR/actions/runs/4540494373 (with refid 0804a808)
(in response to this comment from @t-young31)

/test

/test-all

:robot: pr-bot :robot:
:runner: Running tests: https://github.com/UCLH-Foundry/FlowEHR/actions/runs/4540937610 (with refid 0804a808)
(in response to this comment from @t-young31)

:robot: pr-bot :robot:
:runner: Running tests: https://github.com/UCLH-Foundry/FlowEHR/actions/runs/4540938993 (with refid 0804a808)
(in response to this comment from @t-young31)

/test-force-approve
CI cleanup failing with
Run az acr repository delete \
  az acr repository delete \
    --name acrmgmtflowehrci \
    --image flowehr/devcontainer:flwr-080[4](https://github.com/UCLH-Foundry/FlowEHR/actions/runs/4540938993/jobs/8006398906#step:7:4)a808
  az storage blob directory delete \
    -c tfstate \
    -d flwr-0804a808 \
    --account-name stgmgmtflowehrci
  shell: /usr/bin/bash -e {0}
  env:
    DEVCONTAINER_NAME: flowehr/devcontainer
    AZURE_HTTP_USER_AGENT:
    AZUREPS_HOST_ENVIRONMENT:
ERROR: Unable to prompt for confirmation as no tty available. Use --yes.
Error: Process completed with exit code 1.

bug: #216

:robot: pr-bot :robot:
:white_check_mark: Marking tests as complete (for commit 86206f5bc798bad5e1e984776ca3aa1d1f05bb40)
(in response to this comment from @t-young31)


Placing labels on a page

Nowhere I can find information that and how labels can be placed on pages, and what separates them from fields. You can find an example in the request page of report 6699 "Move Negative Sales Lines".
When you currently search for "Label", the only thing you will find is the label data type.

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

ID: 0363ee62-5ad8-5e9a-82f5-27b3fa351506
Version Independent ID: cb7dcfb3-3766-35a9-dade-dc5d450f6e25
Content: Pages Overview - Business Central
Content Source: dev-itpro/developer/devenv-pages-overview.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Hello, thanks for your feedback and your patience. With the latest doc refresh we've added a new topic on labels, please find it here Working with Labels.


Entity tamed event broken

Seems the 'on entity tamed' event is broken.
I'm using build 196 development version of denizen and spigot 1.12.
I thought it was only me but someone else in Discord also mentioned it and also tested all the different variants of the event (specific entities...) but with no results.
    on entity tamed:         - announce "1tamed <pl.name>"     on player tames wolf:         - announce "2tamed <pl.name>"     on player tames entity:         - announce "3tamed <pl.name>"     on wolf tamed:         - announce "4tamed <pl.name>"
None of them trigger, not even in the debug.

Wrong repo? lol this is the Sentinel repo...


Fix dragging problem

Fix #390.
Proposed
Change the pop-up permission selection list for a regular selection list. This fixes the issue #390 by removing the ability to drag the contributor when clicking on the select. The regular select takes nearly any type of click, sliding, so on.
Repercussions
Doesn't look as cool as the pop-up. However, used the css for the pop-up to inform the format of the list and changing to bold when changed.
Somehow an issue with switching to Read from the other two permissions was fixed, unknown what was fixed.

@Ghalko There is a lot of noise in the commit history for this PR. Can you send a new PR to develop with just your changes?

Close in favor of #2571


Install by country

Hi Michael.
Thanks for the project, again.
I'm facing a problem when I run the install command with a filter by country.
Happens that it's inserting all countries on the database, and just not Australia.
I'm using this:
php artisan geonames:install --country=AU --language=en
Thanks,

Sorry for the delay.
I've made quite a few changes to the library.
Try again, and let me know if the problem persists.


if a symlink is used instead of a folder  in "custom subdirectory" the addon reports the file already exists instead of offering to download it

I tried to use a symlink to point ~/Downloads/FurAffinity to /data/FurAffinity but this does not work at all. instead, the addon is showing that the file "already exists"
this is in 3.2.0.

Can you open the Browser Console and get the error message it gives when you try to save a file? Unfortunately I have basically zero control over this so I don't know if I can do anything about it, but I'd like what it's doing.

I've been able to test it again in FF56 on fedora using version 3.2.1.
I'm pleased to say that this issue can no longer be reproduced with those versions of firefox and the extension - the symlink is used, the subfolder for the artist used and the file created.

I've been able to test it again in FF56 on fedora using version 3.2.1.
I'm pleased to say that this issue can no longer be reproduced with those versions of firefox and the extension - the symlink is used, the subfolder for the artist used and the file created.
it'd be nice to have auto download back in a new update :)


Coverity 1534638: Resource leak in object for stale_response plugin

[approve ci autest]

Cherry-picked to v10.0.x


Implementing livereload

Yet another feature I suggest to use is koa-livereload. I've tested it with gulp-livereload and it works, performing page reload when some of watched files changes.
It would be nice to pass into optionsnot only Boolean flag, but also config for custom livereload port (as it can be passed), although this is not commonly used, as we usually have only one livereload instance running at the same time.
And another option, excludes, is important too in order not to inject livereload script into some partials where it might be undesirable. I'm currently trying to find out a way to deal with such troubles.

hi, thanks again.. personally, i use Chrome Devtools as an IDE for web development - it's possible to edit html, css and JS live in Chrome..
but i respect that other might need a livereload feature so i will look at this soon..

👍

This is resolved by local-web-server v2 and the lws-live-reload module.
Install the v2 preview
$ npm install -g local-web-server@next

Install the middleware module into your project
$ npm install lws-live-reload --save-dev

Launch ws with the desired middleware stack, for example:
$ ws --stack log live-reload spa static index
Serving at http://mbp.local:8000, http://<IP_ADDRESS>:8000, http://<IP_ADDRESS>:8000

I'm still working on the v2 documentation, you can see work in progress on the next branch.
Any feedback, please let me know.

@75lb it looks like lws-live-reload is not compatible with the current local web server (v 4.2.1) - is that right? Are you aware of any alternative solutions? When I run npx ws --stack log live-reload spa static index (as shown above) I get this error:
TypeError: Cannot read property 'middleware' of undefined
    at /<redacted>/node_modules/.pnpm/lws@3.1.0/node_modules/lws/lib/middleware-stack.js:49:68
    at Array.map (<anonymous>)
    at Function.from (/<redacted>/node_modules/.pnpm/lws@3.1.0/node_modules/lws/lib/middleware-stack.js:42:39)
    at WsCli.getConfig (/<redacted>/node_modules/.pnpm/lws@3.1.0/node_modules/lws/lib/cli-app.js:130:35)
    at WsCli.start (/<redacted>/node_modules/.pnpm/lws@3.1.0/node_modules/lws/lib/cli-app.js:9:27)
    at Object.<anonymous> (/<redacted>/node_modules/.pnpm/local-web-server@4.2.1/node_modules/local-web-server/bin/cli.js:7:7)
    at Module._compile (internal/modules/cjs/loader.js:1063:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1092:10)
    at Module.load (internal/modules/cjs/loader.js:928:32)
    at Function.Module._load (internal/modules/cjs/loader.js:769:14)

@davidclark87 hi.. the lws-live-reload module is not maintained anymore since adequate live-reloading functionality is built into Firefox and Chrome devtools, and has been for a number of years..

@75lb thanks for getting back to me! I was basically looking to have the browser automatically reload when the directory being served by LWS changes (say, because tsc --watch just output an updated JS file to it). Does Chrome dev tools offer this, or something similar? I wasn't able to find the feature you were referring to by Googling.

@davidclark87 the lws-live-reload module wraps koa-live-reload, which has not been maintained since 2016.. i'm not sure live-reloading is a "thing" anymore.. do you want the browser to reload automatically? What if you had unsaved state or CSS edits?
Might not be what you need, but nodemon can restart ws automatically on changes to the underlying filesystem..
https://www.npmjs.com/package/nodemon

@75lb thanks again. I've ended up using a different stack for my project so don't have any skin in the game here, but if it helps explain where I was coming from, live-reloading is provided by webpack/webpack-dev-server so it is indeed a thing for many web developers. I agree there are times where it's not desirable, but for me at least those feel like the exception. Most codebases I work in have build steps which run in watch mode during development (e.g. TypeScript compile, or Rollup bundle) so without live-reloading I have to manually track when those processes have written to disk before reloading the browser. With live reloading, I can simply wait for the browser to automatically reload for me. I suspect in a build-less dev environment it'd be less valuable as I know it's safe to immediately refresh after a code change.

ok, thanks -i'll look into it at some point.. i have a WIP tool that streams an event stream into the browser via SSE.. i use it in dashboard applications like performance monitoring (pipe the output of a command-line monitoring tool directly into a real-time browser dashboard).. it could be also used to stream filesystem change events into the browser for display and/or auto-reloading.. SSE is more rebust and lightweight than the websocket solution webpack uses (websockets require you to manually manage the connection, SSE does not).

moved into a new discussion thread


Fix high/low updates and "Invalid state message" error from MQTT

Fixes #117

Thanks. I will merge, test and then push a new reæease if everything works.


Converted Address, Email and Phone to optional fields in a Company

Updated AddCompanyCommandParser by removing the requirement of prefixes for phone, email and address fields

Created a new function getOptionalValue in the ArgumentMultimap to obtain values for fields which are non-compulsory

Updated isValid functions for Phone, Address and Email fields such that empty strings are also recognised as valid strings

Updated tests for the altered classes

Looks good for me.


Bug - selection on mobile Samsung Browser 5 with loaded remote data

Hi,
this bug is really strange, I have it only on Samsung Browser. http://whatsmybrowser.org/b/JMA4Q7C
When I click on the 5th element of a select2 option loaded in ajax, just like your exemple (Loading remote data / Select2 ) it select the previous element.
Even if I put this on my css, it select the previous element :
.select2-dropdown{
  position:fixed;
  top:0;
  width: 100% !important;
}
.select2-results__option{
  padding-top: rem-calc(100);
  padding-bottom: rem-calc(100);
  background: #FFF;
  z-index: 100;
}

When I type a word, then a space, then I click on the 5th elements, it select the 5th element. no bug here ! Oô
Why is this working when I add a space ?
Samsung browser is the default brower on samsung phone, do you see what is going on ?
thank you

I don't have access to an Android device so I am unable to confirm this.  Could someone with an Android device please see if they can replicate this behavior?
Also, for the future, please try to use Markdown to format code snippets.  Thanks!

This appears to be a duplicate of https://github.com/select2/select2/issues/5457.


Bug in checkbox edit mode

There's a typo when you add a checkbox to the form. See attached photo:

I created a pull request for the fix @kevinchappell  :)

Closed By: #46


Documentation of report_mem from commondocs

The report_mem utility is discussed in the wiki materials, but doesn't appear in the now docs pages. I'm adding the material to the Policies and Best Practices section. If there's a better location specified I can move this material.

resolved with #307


Enhance programmer methods on Select result

Have made some programming around select.
It's a little bit complicated and column names is missing I think.
In order to make glusql competitive to JS, sqlite, PHP and more
I think something like the following is required
Some kind of Displayselected instead od Selected
The outermost SELECT could get this result.
But of course  not for an INSERT .. SELECT ...

Column names
Simplified enums on cell values

A: Usage is:  iter over colnames in order to display them in order
B: Usage on whole row is: 'simple' item for all column values
C: Usage on columns in a row
let cell = arow[colname.iter().position(|&x| x == "THECOLNAME")
Below is some pseudo Rust for data definitions
All is immutable
enum DisplayCell
{
Str {value:&str},
Int {value:i64},
Float {value:f64},
Empty, // null
Bool {value:&str} // 'true', 'false' or 1,0 for language independancy
}
struct DisplayRow
{
arow:Vec
}
struct Displayselected
{
headers:Vec<&str>,
rows:Vec
}

Column names
This is right thing GlueSQL should have.
But at the same time, it is not related feature with execution layer.
Fetching column names can be simply done by handling parsed sql AST.
It would be good to add a new module in src/ to handle this kind of work.

Simplified enums on cell values
I'm not sure whether this simplified enums are required in this repo or not.

I think this issue is related to gluesql-js.
It has similar function convert.rs.

So, I think it's good to add "1. Column names" as a new issue, but not sure "2. Simplified enums on cell values" is the feature this gluesql core repository needs to have.
How do you think?

I had a gluesql-rust in mind also.
That is a good interface for an application programmer (like me...)
Look into both gluesql-js and convert.rs
And column names.. Look into that more and make an issue.
Nice coding by the way in convert.rs. And other
Will try make some experiments around a gluesql-rust.rs !!


Fix - Report fields and values don't get updated in Report if admin deletes them from workspace

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/48827
PROPOSAL: https://github.com/Expensify/App/issues/48827#issuecomment-2339210493
Tests

Enable field for a workspace and create a list type field
create an expense report
Check the list field created in (1) in the report and list options it has
Update the list field by deleting or disabling list options
Now open the field on the expense report and verify that the list options is updated according to changes you made in (4)

[ x] Verify that no errors appear in the JS console

Offline tests

QA Steps

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/user-attachments/assets/aecf60a5-eec6-430f-802b-b4d0755820a0

Android: mWeb Chrome
https://github.com/user-attachments/assets/519bd7ac-7fae-4dda-a40f-2c6ad01dd776

iOS: Native

https://github.com/user-attachments/assets/907c33b2-7ade-4005-b10d-7a2ab6c1cef8

iOS: mWeb Safari

https://github.com/user-attachments/assets/aa47558b-42c9-4d36-b4a8-873c5eec2285

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/ef64aaa4-2b61-4867-b6ef-63ddfaba2587

MacOS: Desktop

https://github.com/user-attachments/assets/2335237d-9a41-4680-b4f8-cc641836d78c

🚀 Deployed to staging by https://github.com/puneetlath in version: 9.0.37-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/grgia in version: 9.0.38-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Support for CRAM files

Now that SAMtools 1.0 is released and supports CRAM files, I was wondering if there were any plans to add CRAM support to Freebayes?
Reference: http://www.ebi.ac.uk/ena/about/cram_toolkit
(I understand you probably want to add it, but time to do these major things is scarce!)

@ekg Now that you are migrat(ed/ing) to htslib, will native CRAM be possible?

@ekg does freebayes 1.1 support CRAM now?

According to a recent paper by Wala et al. in Bioinformatics Freebayes supports CRAM files when it is compiled with the SeqLib package. Based on the Freebayes documentation, it is compiled with SeqLib by default. The latest release should therefore support CRAM files, although none of the documentation states this at the time of me writing this!


feat: support for hashing algorithms

Support for sha3-256, sha2-256, and blake2b-256

Nice work! The SAIDs all check out.


bug: DaisyUI doesn't detect classes when using template litterals

What version of daisyUI are you using?
4.7.2
Which browsers are you seeing the problem on?
Chrome
Reproduction URL
https://stackblitz.com/edit/daisyui-angular-bys9bg?file=src%2Fapp%2Fbutton.component.ts
Describe your issue
Classes like btn-primary, btn-secondary... are not working when using template litterals in TypeScript.
I need to do:
export type ButtonColor =
  | 'neutral'
  | 'primary'
  | 'secondary'
  | 'accent'
  | 'info'
  | 'success'
  | 'warning'
  | 'error'
  | 'ghost'
  | 'link';

type ButtonColorClass =

    | 'btn-neutral'
    | 'btn-primary'
    | 'btn-secondary'
    | 'btn-accent'
    | 'btn-info'
    | 'btn-success'
    | 'btn-warning'
    | 'btn-error'
    | 'btn-ghost'
    | 'btn-link';

Instead of
type ButtonColorClass = `btn-${ButtonColor}`;

to have my button with a working btn-primary class

It's not related to daisyUI. It's how Tailwind CSS works. All class names should exist in the file as a string:
https://tailwindcss.com/docs/content-configuration#dynamic-class-names


Region routing broken for forwarded requests and cars that don't support Vehicle Command Protocol

If you send a Vehicle Command Protocol request to an access token with "ou_code": "EU" but with a user region of NA, the proxy sends the command to EU, receives a 421, then reroutes to NA, then learns the car doesn't support Vehicle Command Protocol, then forwards the request on to the (wrong) EU region and gives up. Net result: all commands fail on the vehicle.
The root issue is that forwarded requests do not automatically retry the correct region.
2024-01-03T14:53:16Z [info ] Received POST request for /api/1/vehicles/VIN/command/door_lock
2024-01-03T14:53:16Z [debug] Executing door_lock on VIN
2024-01-03T14:53:16Z [info ] Starting dispatcher service...
2024-01-03T14:53:16Z [info ] Requesting session info from DOMAIN_INFOTAINMENT
2024-01-03T14:53:16Z [info ] Requesting session info from DOMAIN_VEHICLE_SECURITY
2024-01-03T14:53:16Z [debug] Sending request to https://fleet-api.prd.eu.vn.cloud.tesla.com/api/1/vehicles/VIN/signed_command: {"routable_message":"..."}
2024-01-03T14:53:16Z [debug] Sending request to https://fleet-api.prd.eu.vn.cloud.tesla.com/api/1/vehicles/VIN/signed_command: {"routable_message":"..."}
2024-01-03T14:53:16Z [debug] Server returned 421: Misdirected Request: {"response":null,"error":"user out of region, use base URL: https://fleet-api.prd.na.vn.cloud.tesla.com, see https://developer.tesla.com/docs/fleet-api#regional-requirements","error_description":""}
2024-01-03T14:53:16Z [debug] Server returned 421: Misdirected Request: {"response":null,"error":"user out of region, use base URL: https://fleet-api.prd.na.vn.cloud.tesla.com, see https://developer.tesla.com/docs/fleet-api#regional-requirements","error_description":""}
2024-01-03T14:53:16Z [debug] Received HTTP Status 421. Updating server URL.
2024-01-03T14:53:16Z [debug] Received HTTP Status 421. Updating server URL.
2024-01-03T14:53:16Z [debug] [20e25b4af188a5043a55432e99ba4c52] Retrying transmission after error: {"response":null,"error":"user out of region, use base URL: https://fleet-api.prd.na.vn.cloud.tesla.com, see https://developer.tesla.com/docs/fleet-api#regional-requirements","error_description":""}
2024-01-03T14:53:16Z [debug] [f85719b1f9a8ac1135d89aea21861d40] Retrying transmission after error: {"response":null,"error":"user out of region, use base URL: https://fleet-api.prd.na.vn.cloud.tesla.com, see https://developer.tesla.com/docs/fleet-api#regional-requirements","error_description":""}
2024-01-03T14:53:17Z [debug] Sending request to https://fleet-api.prd.na.vn.cloud.tesla.com/api/1/vehicles/VIN/signed_command: {"routable_message":"..."}
2024-01-03T14:53:17Z [debug] Sending request to https://fleet-api.prd.na.vn.cloud.tesla.com/api/1/vehicles/VIN/signed_command: {"routable_message":"..."}
2024-01-03T14:53:18Z [debug] Server returned 422: Unprocessable Entity: {"response":null,"error":"vehicle does not support the Tesla Vehicle Command Protocol, please refer to the documentation here: https://developer.tesla.com/docs/fleet-api#2023-10-09-rest-api-vehicle-commands-endpoint-deprecation-warning","error_description":""}
2024-01-03T14:53:18Z [warn ] [f85719b1f9a8ac1135d89aea21861d40] Terminal transmission error: vehicle does not support protocol -- use REST API
2024-01-03T14:53:18Z [debug] Forwarding request to https://fleet-api.prd.eu.vn.cloud.tesla.com/api/1/vehicles/VIN/command/door_lock
2024-01-03T14:53:18Z [debug] [20e25b4af188a5043a55432e99ba4c52] Retrying transmission after error: Post "https://fleet-api.prd.na.vn.cloud.tesla.com/api/1/vehicles/VIN/signed_command": context canceled

@sethterashima FYI since you were working on this.

Is token with ou_code = EU and user region = NA a valid configuration? In what situation this is expected? Isn't it safe to just use the region and ignore the ou_code in the token?

This SDK doesn't appear to use region at all. It relies on ou_codes and redirects (or lack of, in this case) so it is broken for this Tesla account configuration. I suspect it relates to importing cars from NA.

I think I encountered another account like this. My workaround was to toggle between NA and EU when it fails. It's ugly, but I hope Tesla fixes it.

We're seing HTTP 421 with evcc.io. Setup is EU developer account with self-hosted users all over the world. We're so far using the EU api endpoint. Due to lack of https://github.com/teslamotors/vehicle-command/issues/29 we're handcrafting the requests (no commands, just reading).
It's unclear if EU api for non-EU vehicle  is a valid scenario?


code comment added

Appreciate the PR @redoyrahman02, but I don't think these comments are really necessary.


ImportError in utils.py

"from model_utils import Choices" raise ImportError, but if it is changed to "from django.db.models import Choices" program works.

Please check v0.1.14


Compression for lance files

I'm reviving issue #682: any update on compression for lance files? I have a parquet file with roughly 25 million rows. There are 388 columns with the following dtypes:

dtype
count

int32
365

categorical (large_string -> int32)
11

double
7

bool
2

int64
1

date32[day]
1

large_string
1

With LZ4 compression the parquet file is about 5.6 GB, but with the default settings in lance I get 26 .lance files adding up to 40.3 GB.
Love the project and appreciate your work on it!

That's approximately 4 bytes per value which tracks pretty well with what we'd expect today.
Compression is something we're actively working on (https://github.com/lancedb/lance/issues/1929).  However, we are doing this in the form of more sophisticated encodings and not generalized compression.
This is both the way the industry is going (e.g. procella) and important for us because it allows us to compress the data while still maintaining fast row-based lookups.
I don't know if we'll get exactly 5.6 GiB but I hope we will be quite a bit closer.

Cool - good to know! I'll close this issue then 😄


treewide: use finalAttrs.finalPackage in testers.testVersion

Description of changes
candidates were located using rg -F 'testers.testVersion { package =' | grep -v finalAttrs | cut -d: -f1 | xargs grep -E '(buildRustPackage|buildGoModule)' -L | xargs grep mkDerivation -l
There are surprisingly many examples of finalAttrs migrations that don't change testers.testVersion
should be 0 rebuilds

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

I wonder how we can handle cases like these where we have to use this recursive technique.
https://github.com/NixOS/nixpkgs/blob/a2cad4db0b6d352c0bc484f82e48c445f8092778/pkgs/by-name/ze/zed-editor/package.nix#L248

There's also versionCheckHook which you can simply add to nativeBuildInputs.
I think we could consider deprecating testers.testVersion in favour of that.

versionCheckHook turns a sanity/sniff test into a build failure, a tradeoff i believe the package maintainer should get to decide. It is likely break if a package switches to the 0-unstable-yyyy-mm-dd version schema.

Will merge (or you can self-merge) after the merge conflict is fixed, Peder.


Timeout on refreshSession

I'm calling await Supabase.instance.client.auth.refreshSession().timeout(const Duration(seconds: 30)); when my app switches from not having connection to having connection. This is timing out if the "Access token (JWT) expiry time" has passed. (It defaults to 1 hour, but can reproduce this issue easily if I change it to 2 minutes)

From debugging, looks like it gets stuck returning the existing future on the internal completer.
https://github.com/ConProgramming/supabase-flutter/blob/main/packages/gotrue/lib/src/gotrue_client.dart#L1071

Yeah, I'm working on a fix around the session refresh flow, which should fix this issue too. Hopefully I can ship the fix soon.

Worked on a PR. Testing it some, seems to work well https://github.com/supabase/supabase-flutter/pull/873
Obviously would have to remove logs to merge, but kept for readability. Let me know how this looks.

What is the purpose of calling refresh session when coming back online in the first place? You shouldn't need to explicitly call refresh session on your own as the SDK will handle refreshing the session whenever needed.

I was using it with a .timeout, then signing out if timed out. Because if I didn't then sdk calls were just hanging infinitely due to the internal refreshSession completer never completing.

What call was hanging on what occasion? Rather than trying to change the refreshSessoin method, we should fix the root issue. You shouldn't have to be calling refreshSession in the first place.

Mentioned at the top of this issue, calling refreshSession hands when my app switches from not having connection to having connection if the "Access token (JWT) expiry time" has passed (It defaults to 1 hour, but can reproduce this issue easily if I change it to 2 minutes)

@ConProgramming
Yes, but why are you calling refreshSession in the first place?

If I dont call refreshSession, all sdk calls still hang either way
They are all hanging on the same completer
Calling refreshSession manually confirms that is the error

If I dont call refreshSession, all sdk calls still hang either way

Hangs when you call what? The API methods, correct? Then we should not mess with the refreshSession method, but rather work on a fix so that it doesn't hang, which I'm doing right now, so I ask you to wait patiently. The users should also never have to call refreshSession implicitly unless they have a very specific reason.

The API methods are hanging because refreshSession is hanging, yes?
The problem is that the completer never completes

refreshSession isn't just hanging for no reason. It's waiting for the retries to be completed.

The problem is that the completer never completes

If you are referring to when the device is offline, then that is the expected behavior as the retries should be in progress, although after a few minutes or so it should fail.
I have a rework in progress that will make the API methods fail right away if there is no network connection while retries will be in progress in the background. Again, I ask that you wait patiently. No need to reply to this message.


Added Task.Cower

Simple to begin with, needed to get a grip.

Doesn't GET_SCREEN_RESOLUTION always return 1280x720 no matter what? So do we really need it?

Question aside: has anybody ever tested GET_SCREEN_RESOLUTION on a screen
with size lower than 1280x720? Does the return value depend on the
resolution set ingame?
Am 07.05.2015 09:22 schrieb "crosire"<EMAIL_ADDRESS>
Doesn't GET_SCREEN_RESOLUTION always return 1280x720 no matter what? So do
we really need it?
—
Reply to this email directly or view it on GitHub
https://github.com/crosire/scripthookvdotnet/pull/28#issuecomment-99753376
.

Just tested it on 1920x1080 and 800x600, returned 1280x720 in both cases.

Yeah, I hadn't tested, I was up all night trying to figure out git. I remove that then.

You can just do that in a following commit, Github will append it to this.pull request automatically =)

Alright I'll get to it when home :)

Yea it's kinda strange, he also used some unnamed Native-Call for calculating the line height/width. Still don't know what it does :joy:

Exactly, how does he know what they do?
On skype he said 'see how they do it in the scripts' for various questions I asked. So I'm guessing he also has no idea what it does and just grabs the entire code block and changes the parameters and just trusts it does something.
I was asking about why ADD_EXPLOSION stops rendering the camera shake and explosion particle effects after using it it a few times. I never got it to work again, even after restart/reload.
Have you guys have this problem, with ADD_EXPLOSION?

He takes his information from the decompiled GTA V scripts. Just do a search for whatever native you are interested in check how the game uses it to figure out which parameter does what.

Yes, I was talking about the hashed ones like...
UI::0x67S7H89(4, rptr(a_01)), 1, 0, 0,0, 1)
That is an example, not a real function.

Trial and error I guess.

Ok, I'm home, got to find out about this git branch thing and remove screen_get_resolution (or replace with custom method, like screen_get_dimentions)

I'd start by removing the screen resolution property and pushing that to master, so I can merge things into the codebase with this pull request. The branch stuff is interesting for any new patch you may come up in the future.

Ok, done, can I put anything in a branch? Be it new lines in game.cpp/game.hpp or ped.cpp/ped.hpp or is it topic-specific?

I suggest reading this: https://help.github.com/articles/github-flow-in-the-browser/


Native support for StridedSlice in 6D and Transpose in 7D

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 20.04
TensorFlow installed from (source or binary): source
TensorFlow version (or github SHA if from source): 804ef7223ef08fd14c274b4a4044cc4aeee68863

Provide the text output from tflite_convert
2022-01-08 12:47:32.812069: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcuda.so.1'; dlerror: libcuda.so.1: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /opt/intel/openvino_2021/data_processing/dl_streamer/lib:/opt/intel/openvino_2021/data_processing/gstreamer/lib:/opt/intel/openvino_2021/opencv/lib:/opt/intel/openvino_2021/deployment_tools/ngraph/lib:/opt/intel/openvino_2021/deployment_tools/inference_engine/external/tbb/lib::/opt/intel/openvino_2021/deployment_tools/inference_engine/external/hddl/lib:/opt/intel/openvino_2021/deployment_tools/inference_engine/external/omp/lib:/opt/intel/openvino_2021/deployment_tools/inference_engine/external/gna/lib:/opt/intel/openvino_2021/deployment_tools/inference_engine/external/mkltiny_lnx/lib:/opt/intel/openvino_2021/deployment_tools/inference_engine/lib/intel64:/usr/local/nvidia/lib:/usr/local/nvidia/lib64
2022-01-08 12:47:32.812092: W tensorflow/stream_executor/cuda/cuda_driver.cc:269] failed call to cuInit: UNKNOWN ERROR (303)
2022-01-08 12:47:32.812107: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:163] no NVIDIA GPU device is present: /dev/nvidia0 does not exist
2022-01-08 12:47:32.812367: I tensorflow/core/platform/cpu_feature_guard.cc:151] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  SSE3 SSE4.1 SSE4.2 AVX AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
WARNING:absl:Please consider providing the trackable_obj argument in the from_concrete_functions. Providing without the trackable_obj argument is deprecated and it will use the deprecated conversion path.
2022-01-08 12:47:34.806218: I tensorflow/core/grappler/devices.cc:66] Number of eligible GPUs (core count >= 8, compute capability >= 0.0): 0
2022-01-08 12:47:34.806383: I tensorflow/core/grappler/clusters/single_machine.cc:358] Starting new session
2022-01-08 12:47:34.860258: I tensorflow/core/grappler/optimizers/meta_optimizer.cc:1164] Optimization results for grappler item: graph_to_optimize
  function_optimizer: function_optimizer did nothing. time = 0.008ms.
  function_optimizer: function_optimizer did nothing. time = 0ms.

2022-01-08 12:47:37.790651: W tensorflow/compiler/mlir/lite/python/tf_tfl_flatbuffer_helpers.cc:357] Ignored output_format.
2022-01-08 12:47:37.790685: W tensorflow/compiler/mlir/lite/python/tf_tfl_flatbuffer_helpers.cc:360] Ignored drop_control_dependency.
2022-01-08 12:47:38.104138: W tensorflow/compiler/mlir/lite/flatbuffer_export.cc:1892] TFLite interpreter needs to link Flex delegate in order to run the model since it contains the following Select TFop(s):
Flex ops: FlexStridedSlice, FlexTranspose
Details:
	tf.StridedSlice(tensor<1x120x160x3x1x32xf32>, tensor<6xi32>, tensor<6xi32>, tensor<6xi32>) -> (tensor<1x120x160x1x32xf32>) : {begin_mask = 55 : i64, device = "", ellipsis_mask = 0 : i64, end_mask = 55 : i64, new_axis_mask = 0 : i64, shrink_axis_mask = 8 : i64}
	tf.Transpose(tensor<1x30x4x40x4x1x4xf32>, tensor<7xi32>) -> (tensor<1x30x40x1x4x4x4xf32>) : {device = ""}
See instructions: https://www.tensorflow.org/lite/guide/ops_select
2022-01-08 12:47:38.104385: I tensorflow/compiler/mlir/lite/flatbuffer_export.cc:1963] Estimated count of arithmetic ops: 53.877 G  ops, equivalently 26.938 G  MACs

Estimated count of arithmetic ops: 53.877 G  ops, equivalently 26.938 G  MACs
WARNING:absl:Buffer deduplication procedure will be skipped when flatbuffer library is not properly loaded

Standalone code to reproduce the issue
TFLite Native support for StridedSlice for 6D, and Transpose for 7D would be very beneficial. It would be nice if we could avoid Flex operations if possible.

saved_model and converted tflite files
saved_model_and_float32tflite.zip

Conversion Script

import tensorflow as tf

input_shapes = [[1,120,160,6]]

model = tf.saved_model.load(
    'flyingthings_finalpass_xl/saved_model_120x160'
)
concrete_func = model.signatures[
    tf.saved_model.DEFAULT_SERVING_SIGNATURE_DEF_KEY
]
concrete_func_input_tensors = [
    tensor for tensor in concrete_func.inputs \
        if tensor.dtype != tf.resource and not 'unknown' in tensor.name
]
for conc_input, def_input in zip(concrete_func_input_tensors, input_shapes):
    conc_input.set_shape(def_input)
converter = tf.lite.TFLiteConverter.from_concrete_functions([concrete_func])
converter.target_spec.supported_ops = [
    tf.lite.OpsSet.TFLITE_BUILTINS,
    tf.lite.OpsSet.SELECT_TF_OPS,
]
tflite_model = converter.convert()
with open('model_float32.tflite', 'wb') as w:
    w.write(tflite_model)

Any other info / logs

Model Citation Repository - HITNet: Hierarchical Iterative Tile Refinement Network for Real-time Stereo Matching
https://github.com/google-research/google-research/tree/master/hitnet

FlexStridedSlice (6D)

FlexTranspose (7D)

z这是个什么玩意，我只是评论一下，重合肉丝就是就

@PINTO0309 ,
Please take a look at this SO link and issue with the similar error.It helps.Thanks!

@tilakrayal Thank you.
CUDA warnings have nothing to do with this issue. I have transcribed all the logs without omitting them, but you can ignore the warnings.
This issue is an OP request for TensorFlow Lite. A feature request has been made to implement FlexStridedSlice and FlexTranspose to replace the standard operators. StridedSlice, Transpose

is there any  update?
well, i got the same issues with tensorflow-lite version 2.8.1.  I found an option TFLITE_ENABLE_FLEX in tensorflow-lite 2.5.3 but the option was gone from 2.6.1
I took the part of TFLITE_ENABLE_FLEX in tensorflow 2.5.3 and applied them to 2.8.1. It still doesn't work.
anyone share your tips if you got progress?

I solved the problem on my own.

@PINTO0309 can you please share how you solved this problem ?

For exactly one year this issue got no response, so I addressed the Transpose only ∞ dimension transposition myself. StridedSlice should also succeed with an implementation similar to Transpose.
I have implemented two patterns of dimensional compression algorithms.

Pattern that actively compresses a dimension of size 1, then transposes it, and finally re-expands the compressed dimension
Pattern of Split and Transpose of a dimension with a small size when there are few dimensions with a size of 1

I can't explain it in words, so please look at the test implementation.
https://github.com/PINTO0309/onnx2tf/issues/93


fix(ci): @tauri-apps/cli publish workflow

a few regressions from #10607

I gotta leave, hoping https://github.com/tauri-apps/tauri/actions/runs/10429823972 finally works
otherwise i'll check again when i wake up


NullPointerException while requesting 'restrictions' from Livecoin

LivecoinAdapters.java, method adaptToExchangeMetaData(...), row 94:
BigDecimal minSize = product.getMinLimitQuantity().setScale(product.getPriceScale(), BigDecimal.ROUND_UNNECESSARY);
Generates NullPointerException, when trying to set scale on MinLimitQuantity.
Products are created after API request to https://api.livecoin.net/exchange/restrictions , if I right.
And in the returning JSON can be null for minLimitQuantity key.
So, Livecoin can't init.
I think there is need to check product.getMinLimitQuantity() on null before setting the scale.

I have the same error too.

I temporarily fixed it like that:
      BigDecimal minSize;       if (product.getMinLimitQuantity() == null) {         minSize = BigDecimal.ZERO;       } else {         minSize = product.getMinLimitQuantity().setScale(product.getPriceScale(), BigDecimal.ROUND_UNNECESSARY);       }
Not sure if we should put 0 or null though.

Can be closed when https://github.com/timmolter/XChange/pull/2344 PR is accepted.


Support catching the atom in a subscribe hook

We could do something where the action is the thing that gets passed to the with ActionContext rather than self.f
(on line 71 in the new world - line 54 in the old world)
that might be nicer
But since we already passed the function I didn't change that
feel free to tag a commit onto this one to make that happen
and or make it work - this pr is untested

Now tested - turns out you can add attributes to functions but not methods.
functions have their own __dict__ but methods don't - who knew.
>>> class A:
...    def foo(): pass
...
>>> a = A()
>>> a.foo.__dict__ is A.foo.__dict__
True
>>> A.foo.bar = 1
>>> a.foo.eggs = 2
AttributeError

Every day is a school day!


moving system packages to aspnetcore50 and fixing tests for logging

This pull request is now managed with CodeReviewHub. Refresh.


How to disable mod apns notification to IOS using mod_apns.erl file

Hello @mrDoctorWho  @zinid
How to disable notification in IOS using mod_apns.erl file and I am using mysql database.
We are using below mod_apns.erl file.
mod_apns.erl - https://github.com/joanlopez/ejabberd-mod-apns/blob/master/src/mod_apns.erl
there is always add notification_enabled = 1
How can I prevent notification if I will change notification_enabled = 0?
I am not receiving group chat notification using mod_apns.erl even notification group flag enabled.
notification_group_enabled = 1
Please help me for 1 and 2 questions.

Hi @nileshbacancy,
Please do not create duplicate issues.
Your question should be asked in the repository corresponding the code you use. Probably @joanlopez can answer your question.

Hello @mrDoctorWho
Sorry, but I can not able to see "New Issue" button option on @joanlopez repository.


Don't Merge! PR real time test for source branch full build

Docs Build status updates of commit 6990d9c:
:clock10: Full build: the average full build time is 5 min(s) 59 sec(s), this is based on the last 580 build(s) for this repository.

Docs Build status updates of commit 6990d9c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

docfx.json
:white_check_mark:Succeeded

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


Check existing usage of deploy button on GH pages for correct URL

CRA project linked to signalnerve domain instead of deploy.workers.cloudflare.com

https://docs.google.com/spreadsheets/d/1Fa5e8OniZXCPy3RdMR7B4i5zQpIOzcjhIMh9hfnJ_ZQ/edit#gid=0


Removed afterburner usage from Cosmos Exception

Removed afterburner usage from Cosmos Exception
Closes https://github.com/Azure/azure-sdk-for-java/issues/19847
On investigating this more, the issue started happening on CosmosException after we converted the cosmos exception message into Json format in this PR : https://github.com/Azure/azure-sdk-for-java/pull/18092 after release 4.11.0 and above.
Currently just logging CosmosDiagnostics doesn't throw this warning, but when logging CosmosException, the warning is thrown, because CosmosException uses Utils.getSimpleObjectMapper() - which registers module AfterBurnerModule()

Utils.simpleObjectMapper.registerModule(new AfterburnerModule());

There is no need to use this same ObjectMapper for CosmosException, it can use its own ObjectMapper as used by CosmosDiagnostics, ClientSideRequestStatistics, ClientTelemetry, RntbdObjectMapper and so on.

As discussed offline - I would prefer also renaming Utils.getSimpleObjectMapper to something like Utils.getObjectMapperWithAfterBurner to avoid unintentional regressions in the future.
Otherwise LGTM

can we maybe call it Utils.getObjectMapperForPayloadData() or something like that as it is intended to be used for serializing/deserializing data payload and afterburner is intended to optimize that code path only.

@FabianMeiswinkel @moderakh  - I have updated the name to getObjectMapperForPayLoadData.
The reason I didn't choose objectMapperWithAfterBurner is because in near future if we register more modules to this objectmapper, then having afterBurner in the name might not justify the different modules this objectmapper refers to.
Let me know if this makes sense or not.


Ecies hex

Hex encoding for the encrypted part of registeraddress transactions.
This is needed because the electrum wallet requires all transactions to be fully hex encoded (even push only transactions).

Yes, both are used. Base64 is used for the kyc file. Base 16 is used on the
blockchain.
On Wed, 8 May 2019, 21:03 Nikos Kostoulas<EMAIL_ADDRESS>wrote:

@nkostoulas commented on this pull request.
how come there's two implementations for Encode/Decode? Is both ECIES and
ECIES_HEX used?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/commerceblock/ocean/pull/116#pullrequestreview-235263713,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABAPHPGLNLVSK2G3OBD74SDPUMW2PANCNFSM4HLTCZCQ
.


separate requirements

Seems useful for test/dev but we'll never use it for prod. Separate requirements and make prod as slim possible.
https://github.com/uisautomation/sms-webapp/blob/d63400151b0d22bf552d7c1e795e1acff4238d38/requirements.txt#L6
https://github.com/uisautomation/sms-webapp/blob/d63400151b0d22bf552d7c1e795e1acff4238d38/requirements.txt#L9

So, I use IPython in production a lot when running ./manage.py shell. Fair point about tox.

solution proposed on #9


Decimal places for Percentile & Drycalc#?

Would it be possible to get 1 or 2 decimal places?

Sorry, I somehow didn't see this. Where exactly do you want the decimal places? I tried to make the number of decimals dynamic based on when exactly you need it. For example, the calculation is only accurate to ~1% anyway, so 67% vs. 67.1% is not different enough to warrant including the decimal.
However, 99.9% is very different than 99.999%, so the number of decimals will increase automatically already.
Putting decimals on the already-cluttered collection log page overlay is also questionable, since there just isn't any more room...
Thoughts?


Update file paths so it works out of the box

Think we decided not to merge this change. Main reason I didn't set up the otp/router/graph file structure in the repository is that the tutorial creates more than one router: 'current' and 'new'. These  share some files but not all, and to provide the structure for both would require duplicating files in the repository.

Agreed - thanks for clarification. How about, for reproducibility, there are some mv or cp bash functions for getting the right structure? I can verify that those folder names are needed for the rest of the tutorial to work.
Up for giving that a 'bash' if you give the green light. Fantastic tutorial today btw.

Yeah, sounds a good plan.  Thanks.


[Bug]: Filezilla returns 403 Forbidden

What happened?
When trying to Invoke-WebRequest on the URI supplied for FileZilla I receive 403 Forbidden
Version
2307.810
What PowerShell edition/s are you running Evergreen on?
Windows PowerShell
Which operating system/s are you running Evergreen on?
Windows 10
Have you reviewed the documentation?

[ ] Troubleshooting at: https://stealthpuppy.com/evergreen/troubleshoot/
[ ] Known issues at: https://stealthpuppy.com/evergreen/issues/

Verbose output
Invoke-WebRequest :
403 Forbidden
Forbidden
You don't have permission to access this resource.
At line:1 char:2
+ (Invoke-WebRequest -UseBasicParsing -URI $URI)
+  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (System.Net.HttpWebRequest:HttpWebRequest) [Invoke-WebRequest], WebException
    + FullyQualifiedErrorId : WebCmdletWebResponseException,Microsoft.PowerShell.Commands.InvokeWebRequestCommand

That is all the output I get after I receive the URI to download

This happens all the time with various download URLs, so it’s not really an Evergreen bug. Try adding -UserAgent to the command followed by the user agent string for a typical browser (you can look them up) and that solves it most of the time.
I’m thinking of putting together a module for downloading files, since Invoke-WebRequest fails often, doesn’t auto detect the filename or preserve the modified date, etc!

This will have been an issue with the source site (i.e. Filezilla). It's working now, so Filezilla may have made a temporary change on their end.
Please ensure you've followed the troubleshooting article here: https://stealthpuppy.com/evergreen/troubleshoot/.

 I copied the browser request to Powershell via browser dev tools and trimmed the headers down until I found what it needed - adding the ‘accept’ header in conjunction with the user agent let me download it.
I’ll add support for this in my PsDownload module.
$session = New-Object Microsoft.PowerShell.Commands.WebRequestSession
$session.UserAgent = "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/<IP_ADDRESS>"
Invoke-WebRequest -UseBasicParsing -Uri "https://download.filezilla-project.org/client/FileZilla_3.66.4_win64-setup.exe" `
-WebSession $session `
-Headers @{ "accept"="text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.7"
}


Item holding for tiny pumpkins

This now has some slight merge conflicts due to a recently merged PR.


Fix syntax error in the ETL doctest

The title says it all: this fixes the error in the doctest.

Thanks.


[ART-4761] mirror builder image streams of ocp-private namespace

if the upstream_image exists in stream config, mirror it to ocp private by config upstream_image_mirror

@Ximinhan - this looks good. But it will need to be opened against openshift-eng/art-tools now since we migrated today.

ok, replaced by https://github.com/openshift-eng/art-tools/pull/11


Plugins load external libraries from hardcoded paths

freeboard.loadWidgetPlugin() uses a parameter external_scripts which specifies file paths such as plugins/thirdparty/jquery.sparkline.min.js. This causes issues when trying to use freeboard as part of a system or framework that doesn't use this exact URL structure.
Ideally there would be a way to load dependencies externally to these scripts, or at least a way to define the asset root used by plugins.

Oh yeah, bumped into this as well, totally dumb. :package:


Unable to include the package in a Flutter app

import 'package:timezone/standalone.dart';
import 'package:timezone/timezone.dart';
class MyHomePageState extends State {
@override
void initState() {
initializeTimeZone().then(() {
final detroit = getLocation('America/Detroit');
final now = new TZDateTime.now(detroit);
});
}
}
Compile time warning is as follows:
compiler message: file:///Users/craigjarman/.pub-cache/hosted/pub.dartlang.org/timezone-0.4.3/lib/src/date_time.dart:321:34: Error: The parameter 'other' of the method 'TZDateTime::difference' has type timezone.src.date_time::TZDateTime, which does not match the corresponding type in the overridden method (dart.core::DateTime).
compiler message: Change to a supertype of dart.core::DateTime (or, for a covariant parameter, a subtype).
compiler message:   Duration difference(TZDateTime other) {
compiler message:                                  ^
compiler message: file:///b/build/slave/Linux_Engine/build/src/third_party/dart/runtime/lib/date_patch.dart: Context: This is the overriden method ('difference').
compiler message: file:///Users/craigjarman/.pub-cache/hosted/pub.dartlang.org/timezone-0.4.3/lib/src/date_time.dart:386:36: Error: The parameter 'other' of the method 'TZDateTime::isAtSameMomentAs' has type timezone.src.date_time::TZDateTime, which does not match the corresponding type in the overridden method (dart.core::DateTime).
compiler message: Change to a supertype of dart.core::DateTime (or, for a covariant parameter, a subtype).
compiler message:   bool isAtSameMomentAs(TZDateTime other) {
compiler message:                                    ^
compiler message: file:///b/build/slave/Linux_Engine/build/src/third_party/dart/sdk/lib/core/date_time.dart: Context: This is the overriden method ('isAtSameMomentAs').
compiler message: file:///Users/craigjarman/.pub-cache/hosted/pub.dartlang.org/timezone-0.4.3/lib/src/date_time.dart:396:28: Error: The parameter 'other' of the method 'TZDateTime::compareTo' has type timezone.src.date_time::TZDateTime, which does not match the corresponding type in the overridden method (dart.core::DateTime).
compiler message: Change to a supertype of dart.core::DateTime (or, for a covariant parameter, a subtype).
compiler message:   int compareTo(TZDateTime other) => millisecondsSinceEpoch.compareTo(other.millisecondsSinceEpoch);
compiler message:                            ^
compiler message: file:///b/build/slave/Linux_Engine/build/src/third_party/dart/sdk/lib/core/date_time.dart: Context: This is the overriden method ('compareTo').
compiler message: file:///Users/craigjarman/.pub-cache/hosted/pub.dartlang.org/timezone-0.4.3/lib/src/date_time.dart:11:7: Error: The non-abstract class 'TZDateTime' is missing implementations for these members:
compiler message:   'microsecondsSinceEpoch', 'microsecond'.
compiler message: Try to either
compiler message:  - provide an implementation,
compiler message:  - inherit an implementation from a superclass or mixin,
compiler message:  - mark the class as abstract, or
compiler message:  - provide a 'noSuchMethod' implementation.
compiler message:
compiler message: class TZDateTime implements DateTime {
compiler message:       ^^^^^^^^^^
compiler message: file:///b/build/slave/Linux_Engine/build/src/third_party/dart/runtime/lib/date_patch.dart: Context: 'microsecondsSinceEpoch' is defined here.
compiler message: file:///b/build/slave/Linux_Engine/build/src/third_party/dart/runtime/lib/date_patch.dart: Context: 'microsecond' is defined here.
Compiler failed on lib/main.dart

Hi, I've stopped maintaining this library long time ago, maintained repo is here: https://github.com/srawlins/timezone


Add new cypress spec to test all vulnerabilities

Needs to be controlled by env var setting whether the tests all need to pass. For Travis CI they need to all pass (meaning all expected vulnerabilities are present) for the build to succeed.
People learning to fix the vulnerabilities in their own forks would want tests that confirm the vulnerability is fixed instead. For that case it would be nice to run the whole spec rather than skipping the later tests as soon as one fails.
Note: move the existing A9 test into the new spec

Note: The CI tests need to confirm the vulnerability is exposed all the way through to the frontend, so should operate at the DOM level.
The inverse tests should make server requests directly to ensure that the issue has been fully fixed, not just hidden behind client-side verification.


feat: update credentials and details

SUMMARY

Update headers and details for add/edit database modal.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Edit

Create

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@hughhhh if this is ready, can you add the description and screenshots? thanks!

/testenv up

/testenv up

/testenv up


Use pnpjs to make requests

Currently requests are prepared manually for each command. @waldekmastykarz, is there any reason for not using pnpjs to do that? It would simplify code a lot.

We've discussed this in the past already. It would simplify requests as you mentioned, but:

only for SharePoint, we're supporting other services as well, so it would require us to use two different approaches for requests and tests, making it harder to reuse code across the CLI
it would require contributors to learn PnPjs before they can help, not everyone might know it already
it would require us to align the development of our commands with PnPjs in case we want to add a command that isn't exposed through PnPjs or fallback to calling SharePoint APIs directly

Given we're not using any of the advanced/complex features like batching, sorting, filtering, where PnPjs really shines, and we typically issue a single request, using PnPjs would have limited added value.


Possibility have No headers

Hi @AllenFang
It's possible have a table without header and just the body of table? I try with header formatter with  return null but have always the header without content inside.
any ideas?
Thanks

Sorry @AllenFang
I Post in wrong repo of react-bootstrap-table


Add Spring Cloud Alibaba Samples Project

Spring Cloud Alibaba Samples Project

Nacos Discovery
Nacos Config
Dubbo RPC
Open Feign REST
Seata
Druid

@slievrly  please review and merge


Bounty #1 for Homayoun

We need to solve several issues at once:

https://github.com/ARK-Builders/arklib-android/issues/20
Ensure that everywhere ResourceID is taken from arklib, right now it is CRC32 checksum + filesize.
When we need to print a string with both values, we can just delimit them using dash (-).

[ ] Update ARK-Shelf
[ ] Update ARK-Shelf-Desktop
[ ] Update ARK-Navigator
We should also implement migration in ARK Navigator, i.e. check some value (or index version) in app data and if there is none (or version is too low) then perform storages upgrade. Index and previews could be just dropped. Storages must be backed up before performing upgrade and old resource ids must be replaced by new resource ids. E.g. 5839051: english, greek must be replaced with 88363-5839051: english, greek in .ark/tags, etc.

https://github.com/ARK-Builders/arklib-android/issues/21
Basically, we just need to take code from Navigator and move it into arklib-android, ensuring any app could just import couple of modules and use metadata as well as index. We don't really need indexing in the apps right now (except Navigator). Could be useful later.

https://github.com/ARK-Builders/ARK-Memo/issues/8
We need to use new metadata storage and separate title into it. Also, we can introduce created-date metadata field.
This issue is kinda similar to https://github.com/ARK-Builders/ARK-Shelf/pull/21

Just noticed that Navigator writes empty metadata storages to disk:
[kirill@lenovo ~]$ for f in Pictures/.ark/meta/* ; do cat $f ; echo ; done
null
null
null
null
null
null

I think, we should not write empty storages. Not all resource kinds use metadata storage at all, so better to keep disk clean.

Interested! 😎

@hhio618 btw I haven't seen you applied for the bounty itself..
https://gitcoin.co/issue/29473

Hi. I would like to know if this bounty is still available?

This set of tasks was split into smaller bounties.