Adding quality, filtersFailed, and filtersPassed to Variant, fixes #219

Using changes proposed by @johntbates in pull request #220 and @fnothaft in comment
https://github.com/ga4gh/schemas/pull/220#issuecomment-91592681

Do we want our filters to just be strings, or do we want some complicated ontology of filter types and their parameters?

Sorry for the noise, rebased and squashed second commit to fix non-ascii doc characters.

Please see #656 to close this

Closing in favor of #656


Upgrade Docusaurus - Enable Docusaurus Faster on website

Docusaurus maintainer here 👋
This is a POC to ensure this plugin is compatible with Docusaurus Faster
I'm not sure how your setup works exactly, not super familiar with Yarn PnP but got some annoying errors due to it 😅
I'm able to build with Webpack, but Rspack fails with this error I can't solve when PnP is enabled:

[ERROR] Client bundle compiled with errors therefore further build is impossible.
× Module not found: Can't resolve '/Users/sebastienlorber/Desktop/projects/docusaurus-search-local-easyops-cn/.yarn/virtual/@docusaurus-core-virtual-aaf120fccf/0/cache/<EMAIL_ADDRESS>in '/Users/sebastienlorber/Desktop/projects/docusaurus-search-local-easyops-cn/website'

And idea?
Benchmark
Using node modules linker
Cold builds - much faster
hyperfine --prepare 'yarn workspace website clear' --runs 3 'DOCUSAURUS_FASTER=false yarn build:website' 'DOCUSAURUS_FASTER=true yarn build:website'

Benchmark 1: DOCUSAURUS_FASTER=false yarn build:website
  Time (mean ± σ):     11.864 s ±  0.251 s    [User: 19.999 s, System: 5.471 s]
  Range (min … max):   11.609 s … 12.110 s    3 runs

Benchmark 2: DOCUSAURUS_FASTER=true yarn build:website
  Time (mean ± σ):      3.139 s ±  0.050 s    [User: 4.666 s, System: 0.862 s]
  Range (min … max):    3.106 s …  3.196 s    3 runs

Summary
  DOCUSAURUS_FASTER=true yarn build:website ran
    3.78 ± 0.10 times faster than DOCUSAURUS_FASTER=false yarn build:website

Warm builds - same:
DOCUSAURUS_FASTER=false yarn build:website
hyperfine --runs 3 'DOCUSAURUS_FASTER=false yarn build:website' 'DOCUSAURUS_FASTER=true yarn build:website'

Benchmark 1: DOCUSAURUS_FASTER=false yarn build:website
  Time (mean ± σ):      3.031 s ±  0.066 s    [User: 3.602 s, System: 0.539 s]
  Range (min … max):    2.990 s …  3.107 s    3 runs

  Warning: The first benchmarking run for this command was significantly slower than the rest (3.107 s). This could be caused by (filesystem) caches that were not filled until after the first run. You should consider using the '--warmup' option to fill those caches before the actual benchmark. Alternatively, use the '--prepare' option to clear the caches before each timing run.

Benchmark 2: DOCUSAURUS_FASTER=true yarn build:website
  Time (mean ± σ):      3.120 s ±  0.084 s    [User: 4.671 s, System: 0.859 s]
  Range (min … max):    3.065 s …  3.217 s    3 runs

Summary
  DOCUSAURUS_FASTER=false yarn build:website ran
    1.03 ± 0.04 times faster than DOCUSAURUS_FASTER=true yarn build:website

👍 Great work!
I got no idea why it doesn't work with yarn PnP, either. Though using node modules linker works, right? Maybe related to dependencies with canary versions.

I'll try again once v3.6 is out and we'll see
For now it's already nice to have a POC proving that this plugin works with the new bundler :)

Docusaurus v3.6 is out: https://docusaurus.io/blog/releases/3.6
PR ready to review
Unfortunately, the Yarn PnP error remains, and it only builds with node linker. Maybe @arcanis any idea what could cause this error?
[ERROR] Client bundle compiled with errors therefore further build is impossible.
× Module not found: Can't resolve '/Users/sebastienlorber/Desktop/projects/docusaurus-search-local-easyops-cn/.yarn/__virtual__/@docusaurus-core-virtual-22c3351990/0/cache/<EMAIL_ADDRESS>in '/Users/sebastienlorber/Desktop/projects/docusaurus-search-local-easyops-cn/website'


Uncomment Tracking permission

This is a follow up of #200.
As iOS 14 has been released for several months, I think it's time to make SPTrackingPermission available.
This reverts commit e4b6ac6e5c2acb0ea9246f9cefe3b93f24aba0b9.
Closes #222

@ivanvorobei any chance on adding this in to a release? iOS 14.5 released on April 26th and requires this permission.

@dcarmo-contracts sorry, you are right. Merging.


integration test: use unique names for SmbShare resources

Depends on: #224
Fixes: #222
To help reduce various possible conflicts between the tests as the re-use the same YAML files over and over for test sources, it's better to have unique names for the resources we create. It can also help tie them back to the tests that
created them. This can also help avoid name conflicts between test cases and across test runs.
Because we want to have unique names we also have to remove some of the more "static" configuration input parameters. This is also a good cleanup as we should not assume the operator will name the server resource after the share. In the future, this should be derived from the serverGroup status field. However, we don't want to do too much in these patches.

Trying to make all the names unique was too difficult and in order to get some improvements rather than none, I just make the smb share names unique. The other resources are referred to by name in the YAML files and short of building or reusing a template system I thought it would still be too much of a code change. I also like that the YAML files in the tests/files directory are all usable as-is and thus can also serve as working examples outside the test case and I didn't want to lose that property.

Annoyingly, the centos CI is marked as failed but the actual tests passed:
/TestAvailModeUnchanged (0.11s)
        --- PASS: TestIntegration/reconciliation/scaleoutCluster (41.51s)
            --- PASS: TestIntegration/reconciliation/scaleoutCluster/TestScaleoutClusterSuite (0.08s)
PASS
ok  	github.com/samba-in-kubernetes/samba-operator/tests/integration	980.713s

VS
configmap "samba-operator-controller-cfg" deleted
service "samba-operator-controller-manager-metrics-service" deleted
deployment.apps "samba-operator-controller-manager" deleted
client_loop: send disconnect: Broken pipe
Build step 'Execute shell' marked build as failure
Performing Post build task...
Match found for :Building remotely : True
Logical operation result is TRUE
Running script  : # cico-node-done-from-ansible.sh
# A script that releases nodes from a SSID file written by
SSID_FILE=${SSID_FILE:-$WORKSPACE/cico-ssid}

for ssid in $(cat ${SSID_FILE})
do
    cico -q node done $ssid
done

[samba_sink-mini-k8s-1.23-clustered] $ /bin/sh -xe /tmp/jenkins2061305518328134222.sh
+ SSID_FILE=/tmp/workspace/samba_sink-mini-k8s-1.23-clustered/cico-ssid
++ cat /tmp/workspace/samba_sink-mini-k8s-1.23-clustered/cico-ssid
+ for ssid in $(cat ${SSID_FILE})
+ cico -q node done a3acc1ae

POST BUILD TASK : SUCCESS
END OF POST BUILD TASK : 0
Setting status of 64763d62d3aeec6c0d26ffab8151862339fa2907 to FAILURE with url https://jenkins-samba.apps.ocp.ci.centos.org/job/samba_sink-mini-k8s-1.23-clustered/444/ and message: 'Build finished. '
Using context: centos-ci/sink-clustered/mini-k8s-1.23
Email was triggered for: Failure - Any
Sending email for trigger: Failure - Any
Sending email to<EMAIL_ADDRESS>Finished: FAILURE

^ @anoopcs9

configmap "samba-operator-controller-cfg" deleted
service "samba-operator-controller-manager-metrics-service" deleted
deployment.apps "samba-operator-controller-manager" deleted
client_loop: send disconnect: Broken pipe
Build step 'Execute shell' marked build as failure

That's interesting. I'm seeing it for the first time and it looks like make delete-deploy was hung and never completed. Because the overall job took 8hr 53 min :-/ to complete(as failure). I'll keep an eye on it. If needed we can take the other route(similar to rook teardown) where we directly destroy the cluster without proper cleanup of resources.

It failed the same way again -  a successful run of the golang tests but a broken pipe during cleanup. I am sure that these changes make the golang part of the tests take longer to run. @anoopcs9 do you think that perhaps the longer runtime is causing the overall test run to timeout and fail?

It failed the same way again - a successful run of the golang tests but a broken pipe during cleanup. I am sure that these changes make the golang part of the tests take longer to run. @anoopcs9 do you think that perhaps the longer runtime is causing the overall test run to timeout and fail?

In the last 25 runs(including nightly runs) there were 2 occurrences where it failed after successful run of actual golang tests. First time it completed within 980.713s which is comparatively longer but within 20m limit. Second time it took around 1058.948s(still within 20m limit). In both cases make delete-deploy after test run was hung for very long and eventually ssh connection times out.
Thus, even with these proposed changes for increased cleanup wait time, golang tests are getting completed within 20m(so far). Let me try to reproduce the situation(including these changes) locally and will take a decision.

/test centos-ci/sink-clustered/mini-k8s-1.24

Maybe we should use the  --v=N option to get more info out of kubectl?

Maybe we should use the --v=N option to get more info out of kubectl?

I figured out the reason for hang during cleanup. I was wrong earlier as I could see smbshare resource still listed for kubectl api-resources. Further investigation led me to realize that a particular resource from TestMountPathPermissions was not getting cleaned via deleteFromFiles() because it was created using createFromFilesWithSuffix().
index d1cb8f5..66be684 100644
--- a/tests/integration/path_permissions.go
+++ b/tests/integration/path_permissions.go
@@ -79,7 +79,7 @@ func (s *MountPathPermissionsSuite) SetupSuite() {

 func (s *MountPathPermissionsSuite) TearDownSuite() {
        ctx := s.defaultContext()
-       deleteFromFiles(ctx, s.Require(), s.tc, s.smbShareSources)
+       deleteFromFilesWithSuffix(ctx, s.Require(), s.tc, s.smbShareSources, s.testID)
        deleteFromFiles(ctx, s.Require(), s.tc, s.commonSources)
 }

With the above additional change I could successfully complete the entire test procedure.

Yay.. CI passed !

:hattip:


infra/fix: try to fix macos/rust/beta build

The problem in rust infrastrature - rust-lang/rust#52686, so closing


Avoid page refresh/dealy when liked or disliked

There is a delay/page refresh once like or dislike is clicked. We want to avoid it to improve the experience.

@skrishn

Once like or dislike button is clicked we have multiple operations running

apply edits on the layer to update the data
update local storage
Refresh the feature details to show the updated like and dislike field values

These operations are async requests and we need to wait till all these actions are completed hence we have added the loading indicator.

@sumitzarkar, I understand why it is happening, can we just assume the that when liking or disliking it is a pretty simple operation and 99.9% of the time it will be successfully and just increment the count synchronously, so there is no page reload/refresh. If it fails after the async request we could do a refresh of the UI to set the state due to the error?

@chris-fox
The only issue with this is if in the popup details the like/dislike field is shown then to reflect the new updated values we have to refresh the entire details.
If we are not concerned about the updation of the like/dislike field values in feature details we can avoid the loading indicator and do other tasks in the background like you mentioned

yea, let's do that. I don't have a use case for wanting the like/dislike fields in the feature details. We can wait to see if we get customer requests for refreshing the details after a like/dislike.

@chris-fox
We have one more way to show loading indicator as we are using calcite-buttons, this will avoid user clicking same button multiple time when the operation is in progress

@sumitzarkar, that looks good.

I am seeing the new loading icon next to the like/dislike buttons but I still see the entire feature details panel reload after the like/dislike action completes.

Yes, I see the same. @sumitzarkar can we fix this feature panel loading?

@chris-fox @skrishn

I am seeing the new loading icon next to the like/dislike buttons but I still see the entire feature details panel reload after the like/dislike action completes.

As discussed we can reproduce this only with arcade elements in the popup details.
We have removed the logic to refresh the attributes in info card

Note: The only drawback would be if a like or dislike field is added in popup details those will not reflect the updates when like/dislike is updated.
We have fixed this and will deliver it in the next code drop

@sumitzarkar I see the numbers jumping after hitting like/dislike
https://localgovdev.mapsdevext.arcgis.com/apps/instant/reporter/index.html?appid=54fe3738157d4775b8ff9794c34f58ff
screen-recorder-tue-jun-11-2024-15-12-57.webm

@skrishn

I see the numbers jumping after hitting like/dislike

We don't have control over this. We see this behavior as we are showing the button's loading indicator and updating the button's lable

@sumitzarkar I'm still seeing this on QA too. As I mentioned in the call, I see this behavior after the recent code drop. Please look to see if there is a way to avoid the number sliding to the left as the loading indicator updates the value.

@skrishn, this is the default behavior of the calcite button when you set the loading property to true. This was to prevent the user from rapidly clicking the like or dislike button before we could apply the edits to the feature service.
https://developers.arcgis.com/calcite-design-system/components/button/
I guess an alternative would be to disable the buttons until the edit is applied. In this way they wouldn't show the loading icon, but would grey out until the edit was complete. This might be a weird user experience though.

@skrishn @chris-fox
Please confirm if any changes requried in this

@sumitzarkar No action is required at this point. We can consider the alternative from Chris based on user feedback.


Include compatibility for Debian 12

Add the installation commands for Debian 12, with the following modifications:

removing python (deprecated),
removing julia (it can not be installed on Debian only by that) packages,
including texlive-plain-generic (to solve the problem: ! LaTeX Error: File `ulem.sty' not found.)

Thanks!


BUILD.gn up to tests

PTAL at this new PR!

Looks good. I think we should land these as separate commits.

Land which parts as separate commits?

I meant to land all four together, but unsquashed. Is that a bad idea? The D3D12 thing seems unrelated to the rest, for example.

Oh, that's what I always do :P

Finally figured out you can click "requests" on Travis to see what requests it gets, and that it complained about .travis.yml not parsing.


poc: Add JSON output to deku-cli commands

Depends

[ ] #580

Problem

We are currently juggling awk etc when scripting to get the output we want which is error prone
Solution

Add a --output flag so that we can choose to get JSON output which is easier to parse using as an example jq

This needs to be rebased etc, but what do we want to do to silence other things than the json output?

@ulrikstrid, I'm not sure yet. I'm not super comfortable with the change to the tezos bridge - I want to understand why we're calling the error handler with null in the first place. I need to review it more thoroughly and test it. I'll look at this PR in more detail early next week.

@ulrikstrid, I modified this PR to be much smaller. Let me know if you don't like it and we'll roll back. If this minimal change doesn't address problems you're having in Grafana etc. we can address in a follow up.
@aguillon, no need to mention jq in readme since the only supported way to run the sandbox is either nix run .#sandbox or nix develop -c ./sandbox.sh either of which will include everything required.


The open graph is not showing for the first time

I am using your service and it works well ! Thank you for the free plan.
I have found a small problem while using your product.
First time for the request of the open graph is not working and then I works from second time.
I think that first time, the server cache the HTML and then It doesn’t response especially meta tag for the open graph.
Is there any way to solve the problem? Thank you!
( Is there any way to register all my URL automatically? I am using reactJS and the URL is created dynamically so It looks hard... )
I am using Meteor to generate server and client.
Versions:
dferber_prerender: 2.2.2_3
Meteor: <IP_ADDRESS>
Thank you!

I responded to your email, but it looks like this is due to Facebook not wanting to wait longer than ~5 seconds for a page to render...so if we render a page on the fly and it takes longer than that, then Facebook might not wait long enough for the right content.
If the page is already cached in Prerender.io, we would return it from our server in around 100ms and Facebook would get the right info, so you'll want to make sure your pages are cached as much as possible. You can upload a sitemap.xml or use our recache API to pre-cache those URLs so they're already cached when Facebook requests them.


Blank html files

The html files doesn't show anything when see it the browser

I just open the examples to see if it works, they was all blank.  then I try to do it myself and it was still blank

Please follow these instructions on how you use the element.


Use SqlSquared.Sql instead of String

Note: this is braking change

@safareli looks like one of the tests is failing due to invalid sql (there's a partial pattern error), so maybe Maxim's idea isn't so bad? :wink:
I'm fine with it either way for test purposes though, parsed strings or AST construction.

Looks good :shipit:


duplicate code in multi chunks or sooooo big common chunk file

duplicate code in multi chunks and sooooo big common chunk file
Suppose I have a SPA website. And I have a file main.js in which I config the route.
define(function(){
  // suppose I define route here
  route.when('#/aaa', function(){
    // a code split point
    require([
      'services/A',
      'style/a',
      'module/a'
    ])
  }).when('#/bbb', function(){
    // a code split point
    require([
      'services/A',
      'services/B',
      'style/b',
      'module/b'
    ])
  }).when('#/ccc', function(){
    // a code split point
    require([
      'services/B',
      'serivces/C',
      'style/c',
      'module/c'
    ])
  }).when('#/ddd', function(){
    // a code split point
    require([
      'services/A',
      'serivces/C',
      'style/d',
      'module/d'
    ])
  })// and has many route configs like this
});

I use webpack to bundle code. My webpack.config.js like this
module.exports = {
  entry: {
    main: path.resolve(__dirname, 'main.js')
  },
  output: {
    path: path.resolve(__dirname),
    filename: 'bundle.js'
  },
  plugins: [
    new webpack.optimize.CommonsChunkPlugin({
      name: 'main',
      async: true
    })
  ]
}

When I run webpack, various chunks have duplicate services/A services/B etc. code.
This is not I want.
I read webpack docs, and then I add minChunks: 2 option to CommonsChunkPlugin.
Then I run webpack again.
Now, every chunk only have its own code. But I also get a big common chunk file, which include
services/A, services/B, services/C and other shared code between various page file.
When I run these code, in page /aaa, the /aaa's files loaded, and also the big common chunk file.
Now the big problem is, page /aaa doesn't need services/B and services/C code at all, But the common chunk files contains all shared code between various page files. So the common chunk file is so big, and have many unused code.
I know I can set minChunks to bigger number, but in this way, every page file may agagin have duplicate code.
HELP!
Can I have other methods to only load necessary common chunks when route to different page?

@sokra Could you please take a look at my question? Thanks very much!!

There is a tradeoff here.
move more stuff into the commons chunk means less duplication, bigger commons chunk and more unneeded code loaded.
You can try multiple levels of CommonsChunks and/or manualy splitting of the chunks (minChunks can be a function).

@sokra sorry, could you please explain what's the meaning of multiple levels of commonChunks?
I think I may need something like this.
and I will see the example for seting minchunks to function to see if can solve my problem.
thanks for your reply and awesome work on webpack.

see two-explicit-vendor-chunks example in the examples folder

In your example, all webpackJsonp like functions will go into vendor.js
but in my question, that files are not vendor(all third party libraries and plugins are in vendor.js), just common chunks, and I want to async load them when required, insteadof loading them in html before app.js or main.js or something else as shown in your example.
Anyway, thanks for your reply.

Hello @sokra I follow your two-explicit-vendor-chunks example, and add one example Treri/webpack-example
but the common used code not only exist in separate file but also these files required them.
Could you please take a look at my demo project?
You may say I can remove app from commonChunks plugins' names list, but If I do that, app.js will not have webpack's initialize code, and throw webpackJsonp is not defined error.
Could you give me some advise please?

Hello again @sokra
I bring a good news. I solved my problem! New code is in my webpack-example then-loader branch https://github.com/Treri/webpack-example/tree/feature/then-loader
This is what I want. And I think it is a typical use case, that's only bunlde each page's own sources, and async load common chunks(like common service or something else). Happy to migrate my project to webpack~~

Awesome! I had the same issue for a few days, i am so happy to find a solution at here,now i am taking a closer look at your then-loader, thanks very very much @Treri

@jrs320 Thanks. I have updated code in mater branch. You can have a look the newest code in https://github.com/Treri/webpack-angular-example


Adding dark mode

Light mode doesn't seems to be pleasing
Changes Needed

Adding Dark-mode button. (so that webpage is both light mode and dark mode)
logo fades or hidden in bright mode , need some changes
Fonts need to be improved

Add a label of hactoberfest and assign it to me , I will get this done.

Sure, Go ahead!

@AyushSarode If you don't mind can I split this in 2 or 3 PRs , as I have to complete my Hactoberfest PR tasks as well 😅

@AyushSarode If you don't mind can I split this in 2 or 3 PRs , as I have to complete my Hactoberfest PR tasks as well 😅

Yes, also this is my another repo, you can contribute in this too, add dark mode here.
https://github.com/AyushSarode/Social-media-buttons.github.io

@AyushSarode If you don't mind can I split this in 2 or 3 PRs , as I have to complete my Hactoberfest PR tasks as well 😅

And also can you tell me how did you add PRO tag in your profile, how much did you give to github? or is it free?

@AyushSarode If you don't mind can I split this in 2 or 3 PRs , as I have to complete my Hactoberfest PR tasks as well 😅

And also can you tell me how did you add PRO tag in your profile, how much did you give to github? or is it free?

It's free you just need to be a university student , visit https://education.github.com/ and register yourself for student developer pack

@AyushSarode If you don't mind can I split this in 2 or 3 PRs , as I have to complete my Hactoberfest PR tasks as well 😅

And also can you tell me how did you add PRO tag in your profile, how much did you give to github? or is it free?

It's free you just need to be a university student , visit https://education.github.com/ and register yourself for student developer pack

I'm in 12th grade!

@AyushSarode Hey could you please add a new branch named dark-mode so I can generate PR into that , without affecting the main branch
Initial commits are ready :-

@AyushSarode Hey could you please add a new branch named dark-mode so I can generate PR into that , without affecting the main branch
Initial commits are ready :-

Sure, looks amazing

@AyushSarode Hey could you please add a new branch named dark-mode so I can generate PR into that , without affecting the main branch
Initial commits are ready :-

I created branch for darkmode, btw it looks amazing!

@AyushSarode thanks

Hello Sir,
Can You assign this issue to me?


How to check resume function for plugin

Dear all,
I have a question.
Currently I'm developing embulk. I implemented  resume function for my plugin but I can't confirm operation by test for resume function. How to test  resume function ? How to implement a transaction fail during run embulk to check resume function?
Thank you!

Hi, if you need to trigger resume, I think you can create its configuration mock (see https://github.com/embulk/embulk/blob/master/embulk-core/src/main/java/org/embulk/EmbulkRunner.java#L284)

thank you so much your comments,
Is there an easier method than that?

Hello, @toanvan.
#740 may help. Unfortunately, resume doesn't work for a long time.

@hiroyuki-sato
thank you,


$buttongroup-spacing implemented differently for $global-flexbox true and false

The spacing within a .buttongroup is implemented differently, depending weather the variable $global-flexbox is set false or true.
In a flexbox context, the spacing is implemented by a margin-right
In a non-flexbox context the spacing is implemented by a white border-right
The border-method causes funky results, when $buttongroup-spacing ramped up to values bigger, than the default button-border-width
How can we reproduce this bug?

Set the value of $buttongroup-spacing to a high value. e.g. 10px;
Change the value of $global-flexbox from false to true and vice versa.
Compare the results

What happened?
The Border-Method the color of the top border of the button to leak into the spacing:

The problem of the border-method becomes most visible, when placing a button-group on a colored element, like a top bar. The spacing is running in the body-color:

Source of this Problem
This difference is caused by this code within the button group. Starting at line 51:
     @if $global-flexbox {
        margin-right: $buttongroup-spacing;
      }
      @else {
        border-#{$global-right}: $buttongroup-spacing solid $body-background;
      }

** Suggestion **
If the border-method is not absolutely necessary for browser compatibility, i suggest to remove the conditional completely and use for both cases the margin-method.

This Issue is related to the discussion in issue https://github.com/zurb/foundation-sites/issues/8447

The conditinal had been intorduced in this commit https://github.com/zurb/foundation-sites/commit/5de0264d02d3ba7b1d24da93d6b86a5531109988 by @gakimball on on 13 Jan.
It seems to have collided with this commit https://github.com/zurb/foundation-sites/commit/3cda91d65f3b6a546ce64d08b17b2a211a40d79a by @andycochran on 20 Jan, where he already reworked button groups to not use borders.

Hmm… Yeah, I fixed this (not for flexbox tho). Button Groups should not use borders for spacing.

It actually works for flexbox. Only the non-flex version still contains the border.
Looks to me like you guys worked on the same code at the same time, and something went boing when you merged the commits.
If using margins is the expected behavior, i guess this one should fix it: https://github.com/g12n/foundation-sites/commit/c30a07cea546139c4f0e4bdf1c4f3938fde706d9

@g12n That PR should fix it.


Solve Daily Problem: 12 Agust 24 - Median of two sorted arrays

Median of two sorted arrays
link
Difficulty: MediumAccuracy: 47.98%Submissions: 65K+Points: 4
Given 2 sorted integer arrays arr1 and arr2. Find the median of two sorted arrays arr1 and arr2.
Examples:
Input: arr1 = [1, 2, 4, 6, 10], arr2 = [4, 5, 6, 9, 12]
Output: 11
Explanation: The merged array looks like [1, 2, 4, 4, 5, 6, 6, 9, 10, 12]. Sum of middle elements is 11 (5 + 6).
Input: arr1 = [1, 12, 15, 26, 38], arr2 = [2, 13, 17, 30, 45]
Output: 32
Explanation: The merged array looks like [1, 2, 12, 13, 15, 17, 26, 30, 38, 45]. Sum of middle elements is 32 (15 + 17).

Expected Time Complexity: O(log n)
Expected Auxiliary Space: O(1)
Constraints:

1 <= arr1.size() == arr2.size() <= 103
1 <= arr1[i] <= 106
1 <= arr2[i] <= 106

Solved:
class Solution:
    def sum_of_middle_elements(self, arr1, arr2):
        arr3 = arr1 + arr2
        arr3.sort()
        result = []
        n = len(arr3)

        if n % 2 == 0:
            result.append(arr3[n // 2-1])
            result.append(arr3[n // 2])
        else:
            result.append(arr3[n // 2])

        return result[0] + result[1]


How to correctly pass characters to ImGui?

Hi, I'm passing characters to ImGui using imguiBeginFrame(), as char inputChar parameter. The passed char comes from inputGetChar() function. It seem to work, but only alphanumeric values are working. No backspace, arrow keys, etc. What I'm doing wrong?

I have found a solution. It seems that io.KeyMap[] is not configured until USE_ENTRY define is not set (imgui.cpp). Setting this and recompiling BGFX resolve this issue.

I confirm. I just uncommented this line in examples/common/imgui/imgui.cpp:
#define USE_ENTRY 1
And now ImGui recognizes backspace, arrows, etc.
I can even move to other ImGui controls with arrow keys or tab and adjust values etc.
Still, I got an issue with utf8 characters, here is my code:
const uint8_t* ch = inputGetChar();
int iChar = !ch ? -1 :
	!ch[1] ? int(ch[0]) :
	!ch[2] ? (int(ch[0]) << 8) + int(ch[1]) :
	!ch[3] ? (int(ch[0]) << 16) + (int(ch[1]) << 8) + int(ch[2]) :
	(int(ch[0]) << 24) + (int(ch[1]) << 16) + (int(ch[2]) << 16) + int(ch[3]);

And then I'm passing iChar as last to imguiBeginFrame.
I get the right value, e.g. for ł character: C582, but in ImGui::InputText I see only ? character.
Same happens with this code:
static char str1[128] {u8"łaA"};
ImGui::InputTextWithHint("Search", "Search text", str1, IM_ARRAYSIZE(str1));

I see ?aA in that input.
Any idea what is wrong?


Add recognition for funders to daostar.org

We need to be able to recognize who has given grants to DAOstar.

See
https://www.notion.so/daostar/3253abe36c4a47b6855c88f8e045bf3c?v=dea7dfdcf45f4b20bbc249ff7c406895
On Thu, Jun 16, 2022 at 2:30 PM Jenny Fan @.***> wrote:

@thelastjosh https://github.com/thelastjosh Do you have the list of
granters to add?
—
Reply to this email directly, view it on GitHub
https://github.com/metagov/daostar/issues/12#issuecomment-1158005172,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACHA5PFJ3T4VXEQXXENY2J3VPNXDJANCNFSM5X5C63EQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
http://www.joshuatan.com/research/


Export to csv feature enabled as table property.

@lopenchi , yes its available in my repo.

When can we expect this to be merged and available?


Linux support.

See: https://github.com/krzyzanowskim/CryptoSwift/blob/master/Sources/CryptoSwift/Multiplatform.swift
And: http://blog.krzyzanowskim.com/2015/12/04/swift-package-manager-and-linux-compatible/

Current coverage is 100% (diff: 100%)

Merging #5 into master will not change coverage

@@           master     #5   diff @@
====================================
  Files          19     19
  Lines        1067   1072     +5
  Methods         0      0
  Messages        0      0
  Branches        0      0
====================================
+ Hits         1067   1072     +5
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update e41489b...238fdd2

Awesome!

I'm a little worried about how this shim seeds the Mersenne twister; I assume it won't just produce the same series after every startup, would it? arc4random plugs into the kernel random pool, so it provides relatively high quality entropy.
Also, it would be better to fill the buffer by generating four bytes at once (or eight if mt19937_64 is also available); there is no need to go byte by byte.


Failed to recover a recoverable Key Vault resource

After deleting a previous deployment (Resource Group) in Azure and changing the name of the APP_NAME variable to a different name, the next deployment attempts to recover the Key Vault that was previously deleted and it fails.
Assumption is that the Key Vault with the same name does exist as a softdeleted resource, but it used to live in a different resource group because the new RG has a new name due to the change in APP_NAME?

Close no repro


[WIP] Try Azure DevOoops

So, this is not too bad, but agonisingly slow due to absence of caching (see Microsoft/azure-pipelines-yaml#113).

@kim Maybe maintain an external cache in Microsoft OneDrive or whatever their s3/gcs equivalent is.

@xla Yeah potentially. Just need to figure out how to set up the Azure PowerShell or whatever. Note also that the Windows build times out, so perhaps we can conclude the experiment here.


Stuck At Please log in under Azure

I'm not sure where to go from here.

Same problem & no documentation

The issue is still legit, took time to figure out what I had to do. Ran it in terminal as well :)
There probably can be a bit better explanation or clickable link or something in there, though it seems it's already handled in the linked PR


Add integration test for dataset module

Also adding a scheduled build every week.
Closes #267

Hi @kami619 - here's now an integration tests which only uses Java please let me know what you think of this.

side note: with these tests in place, I wonder if we still need all the tests in provision-minikube.yml, or if we could skip some of them as we don't expect them to reveal many additional errors.

@ahus1 I will get to this today, sorry been occupied with other tasks

side note: with these tests in place, I wonder if we still need all the tests in provision-minikube.yml, or if we could skip some of them as we don't expect them to reveal many additional errors.

I would still keep those tests from provision-minikube.yml as they cover external databases such as PostgreSQL and CockroachDB, just in case we have any integration issues with them.


[Bug] inlay hint bug in sticky line

Reproducible in vscode.dev or in VS Code Desktop?

[X] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[ ] Not reproducible in the monaco editor playground

Monaco Editor Playground Link
https://microsoft.github.io/monaco-editor/playground.html?source=v0.51.0#XQAAAAKbAQAAAAAAAABBqQkHQ5NjdMjwa-jY7SIQ9S7DNlzs5W-mwj0fe1ZCDRFc9ws9XQE0SJE1jc2VKxhaLFIw9vEWSxW3yscwzt51RgmSaumWUF-TBOs7hSruXioSxyeSO7k_l1UIzWNddFqt-6khNLmgbO-hd7_R9DHrqg9S4UTF4tNbOQ6HWysoutPZajWg6a6p4gu_aijmUDOoVbql1zvXc--j_6BXxKfUIVOWBV9SRM_oPYAAYrQcgs4cuI-zuvU2UoPxlzIsSmj0jQAnUhWfyLjRI4l3nvAhiuyIF2IfXsiOdgKeb7Is2JnTgxZZFStTSaAvLaA3NgEtQlWZiPCXmcRie_aVz43uf7KgfKtbSUdLQRooKr943W_-09tDdvotyVJQx3akcatJmcaqAbBMuWvIp5VFoWEvI_w91R8
Monaco Editor Playground Code
const value = `
function foo(option) {}

export default foo({
${'  //\n'.repeat(80)}
  bar() {
    //
  },
})
`;

const editor = monaco.editor.create(document.getElementById("container"), {
  value,
  language: "javascript",
});

monaco.languages.typescript.javascriptDefaults.setInlayHintsOptions({
  includeInlayParameterNameHints: 'all',
})

Reproduction Steps

Open playground link
scroll down the preview code to about line number 88

Actual (Problematic) Behavior

Expected Behavior
no bug
Additional Context
No response

Can repro. @aiday-mar can you look into this in debt week?

Hi thank you for filing this issue. After some investigation I believe sticky scroll works as intended, it is reflecting the editor lines which themselves have a growing number of inlay hints appended to them. You can see this happen in the following screen recording where we briefly see the actual editor lines with all the extra inlay hints.
https://github.com/user-attachments/assets/460fc235-b5d9-491b-88b6-f7da27a7bcf5
This behavior is also mirrored in the following issue https://github.com/microsoft/monaco-editor/issues/4700. I will close this issue, as it is a consequence of the root cause of the issue.


[CO-325] API V1 Improvements - Part 1

Description
This PR contains some API V1 improvements. Another one will follow in the upcoming week with another set of changes. We are shipping this first part now to avoid too big PR and because some of its content is needed by the Daedalus team to move on.
This contains:

CO-324 Per-field endpoints
CO-327 Swagger schemas' examples don't rely on the Example type-class
CO-328 --cert cURL option is missing from API documentation
CO-342 Backward-compatible CBackupPhrase parser

Linked issue
[CO-325]
Type of change

[x] 🐞 Bug fix (non-breaking change which fixes an issue)
[x] 🛠 New feature (non-breaking change which adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)
[x] 🏭 Refactoring that does not change existing functionality but does improve things like code readability, structure etc
[x] 🔨 New or improved tests for existing code
[ ] ⛑ git-flow chore (backport, hotfix, etc)

Developer checklist

[x] I have read the style guide document, and my code follows the code style of this project.
[x] If my code deals with exceptions, it follows the guidelines.
[x] I have updated any documentation accordingly, if needed. Documentation changes can be reflected in opening a PR on cardanodocs.com, amending the inline Haddock comments, any relevant README file or one of the document listed in the docs directory.

Testing checklist

[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

QA Steps

Screenshots (if available)

I tested this together with #3367 and this solves the issues with creating and restoring wallets over api V0

Merging with failing AppVeyor (timeout). We ought to do something about it, almost impossible to get a PR green and it takes an unreasonable amount of time.


feat: dynamic import es or cjs module when build wallet-ts

Summary

Bug Fix

Fixed importing cjs module when build wallet-ts into esm

Summary by CodeRabbit

Chores

Refactored build scripts to separate CommonJS (CJS) and ECMAScript Module (ESM) builds into distinct commands.
Introduced a new build script to orchestrate both ESM and CJS builds along with other necessary steps.

nice, let me refactor this a bit so it's a generalized approach that we can reuse in case we need it and I'll republish the packages - I will send you the updated package versions in this PR in a bit

Thanks for making this, here is the commit.
Here are the updated package versions:
fix: esm replacement pairs (bangjelkoski)
@injectivelabs/bridge-ts: 1.14.11-beta.66
@injectivelabs/sdk-ts: 1.14.11-beta.65
@injectivelabs/sdk-ui-ts: 1.14.11-beta.87
@injectivelabs/wallet-ts: 1.14.11-beta.79


Find factories registered for namespaced models

Namespaced models like Invoice::Item should be able to be found by FactoryGirl.create(Invoice::Item) if registered as factory :invoice_item.
I added a test for this new behavior.
Fixes #740

@monkbroc I'm not sure accessing factories by the class itself is something we want to promote. While I understand some amount of ease with using described_class in RSpec, I think it's a way of causing confusion, as there's not a 1:1 mapping between the classes.
factory :invoice_item, class: Invoice::Item do
  # base attributes

  factory :awesome_invoice_item do
    # custom attributes to make it awesome
  end
end

Both of these will build an Invoice::Item instance, but when running build(:invoice_item), which takes precedence, the last-declared or the one with the class? The fact that it's not obvious is concerning.
The other aspect is that there's a tight coupling when using classes (not that there wasn't before, but this change further enables class usage). Given:
factory :item, class: Invoice::Item

build(:item) works correctly, while build(Invoice::Item) does not. Because of the conditional nature of this, I do wonder if the more appropriate action would be to deprecate providing a class (to be removed in FG 5.0), and instead require a string or symbol directly matching how the factory key is registered.
I do appreciate you taking the time to solve this, but I do wonder if it's something that should rather be disallowed rather than further enabled. Thoughts?

I agree that providing fewer variants of the FG API would be better. Deprecating finding a factory using a class is the way to go for FG 5.0. I can submit a PR for this if you'd like (don't know if FG 5 is on the horizon yet).

@monkbroc if you'd like to tackle this, I'd love to see a PR. In the past, we've wrapped this sort of functionality in feature flags and included deprecations to help the upgrade process. Let me see if I can find some older features we'd done this for as examples.


Enable OBU Header for AV1 v1.0 aomdec

Can be part of #103 but aiming to comply current master of libaom,
so that rav1e bitstream can be decodable by current version of aomdec.

Closed by #341.


Exported JSON cannot be imported

Fixes #586
This PR is an attempt to fix the export/import of JSON messages.
There's a whole slew of broken things with the current code base 😞

Serialization

❌ Incorrectly stored body - the body is stored as an object rather than raw JSON. To work properly, the body has to be saved as an escaped JSON string.
❌ Incorrectly stored properties/headers - headers are stored as an object where it should be an array of key/value objects

Deserialization

❌ Incorrectly deserialized body - deserialized property is called content rather than body
✖ Incorrect merging of deserialized properties/headers and custom provide via app properties/headers - properties provided via app are lost. Don't think it matters since a restored message can be tweaked via modifying the serialized text.

To make it worse, there are multiple paths of invocation and reuse for XML & JSON.
Frankly, I don't know if it's worth spending time fixing it; fixing one thing will break something else.
I'm going to attempt fix JSON as this is the most common message serialization type used. No guarantees it won't break XML or other formats the application claims to support.

[x] Fix body serialization
[x] Fix body deserialization
[x] Fix property serialization
[x] Works with a single exported message (queue)
[x] Works with multiple exported messages (queue)
[x] Works with DLQ-ed single message (queue)
[x] Works with DLQ-ed multiple messages (queue)
[x] Works with a single exported message (topic/subscription)
[x] Works with multiple exported messages (topic/subscription)
[x] Works with DLQ-ed single message (topic/subscription)
[x] Works with DLQ-ed multiple messages (topic/subscription)
[x] Investigate what "Template" stands for - no longer relevant
Serialization/deserializaton from to/files other than JSON is not working --> Raised as https://github.com/paolosalvatori/ServiceBusExplorer/issues/619
[x] Save application properties types and do not hardcode to String

Changes in action
Preserving types
https://user-images.githubusercontent.com/1309622/158310081-586550e7-970b-4b13-b172-a92f02f5ff6e.mp4
Single message export/import
https://user-images.githubusercontent.com/1309622/158314129-82c0d2b7-4bc3-471a-8455-5f094b4f48e3.mp4
Multiple messages export/import
https://user-images.githubusercontent.com/1309622/158316558-23f6f0e7-d186-4a21-8918-2a4558563eed.mp4

:x: Build ServiceBusExplorer 902 failed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/78288e22a5 by @SeanFeldman)

:x: Build ServiceBusExplorer 904 failed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/f04488ee52 by @SeanFeldman)

@benclerix-cegeka, if you'd like to pull down this branch and give it a try, I'd appreciate it.
Something to keep in mind - the saved JSON is no longer as readable as it was before, but it's de-serializable.
I haven't tested it on an array of messages, only a single message.

I built the sln and opened src\ServiceBusExplorer\bin\Debug\net462\ServiceBusExplorer.exe.
Whenever I open a queue I get the following exception:
<09:52:02> Exception: The type initializer for 'Azure.Core.ClientOptions' threw an exception.
<09:52:02> InnerException: Could not load file or assembly 'System.Threading.Tasks.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)
<09:52:22> Exception: The type initializer for 'Azure.Core.ClientOptions' threw an exception.
<09:52:22> InnerException: Could not load file or assembly 'System.Threading.Tasks.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)

Maybe I'm doing something wrong?

I built the sln and opened src\ServiceBusExplorer\bin\Debug\net462\ServiceBusExplorer.exe.

What IDE did you use?

:x: Build ServiceBusExplorer 906 failed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/481d80d043 by @SeanFeldman)

:x: Build ServiceBusExplorer 908 failed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/5810650166 by @SeanFeldman)

:x: Build ServiceBusExplorer 910 failed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/2df5ab12c7 by @SeanFeldman)

:x: Build ServiceBusExplorer 914 failed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/bcb3b6397f by @SeanFeldman)

:x: Build ServiceBusExplorer 916 failed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/9e9f67547c by @SeanFeldman)

@paolosalvatori @ErikMogensen, have a look at the change in action. I had to raise an additional bug issue that is outside of this work scope.
Next, I need to figure out why GitVersion is failing to unblock this PR.

:x: Build ServiceBusExplorer 917 failed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/3ce966ea64 by @SeanFeldman)

I built the sln and opened src\ServiceBusExplorer\bin\Debug\net462\ServiceBusExplorer.exe.

What IDE did you use? Try a different environment/machine. I've had no issues (used VS2019, VS2022, Rider2021)

Never mind: yesterday I tried to run the ServiceBusExplorer.exe directly from the bin folder which doesn't work.
When I run it from within VS2019 it does run as expected.
I tested the fix and it works perfectly.
Thanks for looking into this!

Last outstanding and unrelated to the change issue is GitVersion which decided to break.

Hi @SeanFeldman do you need me to approve the PR or are you still working on it?

@paolosalvatori, first, I need to figure out why GitVersion is failing.

:x: Build ServiceBusExplorer 921 failed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/3ce966ea64 by @SeanFeldman)


is there a way that I can change the opacity when trying to zoom in or out?

Expected Behavior

Actual Behavior
If you notice the image outside the viewport is a darker than the one in the viewport. I want to increase that opacity. is it possible?
Steps to Reproduce the Problem

Example Link
Please recreate your issue using JSbin, JSFiddle, or Codepen.

Link:

Specifications

Browser:
Version:

The shadow comes from our viewport styles: https://github.com/Foliotek/Croppie/blob/master/croppie.css#L34
Some of these styles are mandatory, but the box shadow is not.  If you want to decrease the opacity on the rgba of the box shadow, feel free.

Thanks for the help.


refactor: change database table name

:tada: This PR is included in version 0.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Set family in model builders

A small PR that calls set_family in model builders' build_model().

This is a refactoring change, and doesn't change functionality right?

No


issues/24: Remove struct tags from display

What(What have you changed/added/removed?)
Why(Why did you change/add/remove it?)

Fixes: https://github.com/komuw/kama/issues/24

Problem is that this struct field: https://github.com/komuw/kama/blob/406f831be8514c6b38f2027dfe5c88072800771e/e2e/e2e_test.go#L104
is now(on this branch) been displayed as;
FIELDS: [
	....

	SliceOfHttpRequest []http.Request
	OneHttpRequest http.Request

See; https://stackoverflow.com/questions/23507033/get-struct-field-tag-using-go-reflect-package


Add support for non URI-based sources (e.g., Collections)

@olegz I was looking for a feature like this to test rapidly. Is this issue open or a Work In Progress?
Thanks!

No, it already exists, although could possibly be made better. Its all about implementing SourceSupplier. I think SQL module uses it. Sorry a bit busy now to get more details but that is where I would look.
Thanks
Oleg


Fixes typos and best metric selection

Description

I changed typos (successs --> success)
I fixed the logic of 'best_metric,' which will be used to check whether to continue the Eureka loop.

best_metric should be the task_success value with the minimum distance to the target value.

Type of change

Bug fix (non-breaking change which fixes an issue)

Checklist

[x] I have run the pre-commit checks with ./isaaclab.sh --format
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

Thanks a lot for the fixes @csmile-1006!


Previous image migration

There needs to be functionality to migrate old images over

As there are many ways to transfer images to S3 and this is a one-time process, I don't think it's necessary to add this feature. For instance, you can use s3cmd' sync command to achieve that.


Missing license information

Hello,
I had a play with the farming simulator and it looks like a good baseline to build a nice game.
Can you let us know what the license is (MIT, GPL?)
I am looking to integrate some short simple games to host on a website that keeps scores and player states, so any changes I make I'd send back pull requests.
Thanks

I hadn't think this simulator had reusable value since it's so experimental, so I didn't feel the need to supply a license. But if you want to use it, I'm okay with the most liberal license as possible. Probably MIT license?
Also, if you intend to fork this repository, be aware that there are two main branches. The master branch depends on an external library, pixi.js, which is hardware-accelerated graphics library, but it's not suitable for a rather static game such as farmsim. So I branched no-pixi, which is an attempt to get rid of dependency of pixi.js. It turned out that simple HTML elements such as div and img worked quite well for such a game (and they are more mobile-friendly), so now I consider the master branch obsolete.
However, there are remnants of pixi.js dependencies which I haven't bother cleaning up, even in no-pixi branch.
Also, I am not satisfied with the simulation model. It doesn't have seasons, temperature, extreme weather, terrain, wild animals and such kind of things you would worry about if you are into gardening.  It is a difficult decision of game design, but I think a full-fledged game requires careful consideration on it.

Thanks for the information!


i cant add more than 1 sprite in stage editor

Describe your bug here.
so i went to stage editor and i wanted to make a bg, but when i try to add the second sprite, it just doesnt open the selecting sprite window. and if you delete the first sprite and try adding another, it still doesnt work.
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
i edited nothing except for adding sprites for bg in my modpack

For adding sprites into the stage, it must be outside of the /mods/images folder to work. There's a bug that makes certain buttons unresponsive when the asset is already in the images folder.
They SERIOUSLY need to make a hotfix for this...

For adding sprites into the stage, it must be outside of the /mods/images folder to work. There's a bug that makes certain buttons unresponsive when the asset is already in the images folder. They SERIOUSLY need to make a hotfix for this...

its not really that big problem if you just dont put it in images before

its not really that big problem if you just dont put it in images before
The problem is that it's impossible to save a stage if you add an animated object: Since the animations will only work if both the image and xml file are in the images folder.

Perfect! I  hope the fixed version comes out soon, I'm excited to try out the stage editor in its full potential :)


Program to Find GCD of two float Numbers in c++

This is not a bug. This is a c++ problem.
Hi myself Srishti Majumder, a fellow contributor in SWOC 3.0. Please assign me this issue.

Sorry this problem has been implemented.


Fix onnx.load_model after onnx 1.15.0

format argument in onnx.load_model is not used until version 1.15.0, and the format must be a string registered into supported format.

@LoSealL Merged. Thanks for the fix!


Allow invoking cmake from a specific shell

Brief Issue Summary
Sometimes it is necessary to run cmake from a different (non cmd) shell on windows. For example when using MSYS2 to install libraries (eg wxWidgets) using pacman it is necessary to run cmake from MSYS2's sh.exe or it will be unable to find the libraries installed with pacman. Would it be possible to add a setting to allow running cmake from a specific shell on different oses?
Possibly something like
cmake.shell.win = "C:\msys64\mingw64shell.bat"

Sorry for the delay.
Trouble is that CMake isn't really executed by any shell. It is started via Node's child_process, which I believe simply wraps CreateProcess(). It's more likely that it needs some set of environment variables from MSYS, and to run the correct cmake.exe installed in MSYS. What happens if you try to set cmake.cmakePath to the binary installed by pacman inside of C:\msys64?
I've not used MSYS or pacman on Windows, so I wouldn't know quite how to start looking into this one, nor how to write tests to validate it.

Ok, I have a solution. Putting MSYS2's sh.exe in the system's PATH allows the use of CMake's MSYS Makefiles generator which finds the libraries correctly. It seems like this is the easy way to deal with this as CMake will find and use sh.

@MB3hel Change PATH maybe a choice. However, if I do not want change PATH in the system, can we just change PATH in VS Code? Maybe create a bat file to launch VS Code can be a solution. But it is not that elegant. Is there any method such as changing settings.json to add something in PATH just for a specific project alone? Thanks!

You should be able to add something like this to your settings.json if you just need the path set for cmake @tan-wei
"cmake.environment": {
    "PATH": "C:/MSYS/bin;${env:PATH}"
},


One click reporting MVP

Description
On what platform have you tested the change?

[ ] Android - Emulator
[ ] Android - Physical device
[ ] iOS - Emulator
[ ] iOS - Physical device

Out of scope and potential follow-up
Screenshots
Checklist (for submission)

[ ] Analytics/tracking events have been added (if relevant)
[ ] Added new screen(s) to src/features/screens.ts and new Redux slice(s) to mocks/mockedInitialState.ts (if relevant)

Checklist (for reviewers)

[ ] Checked against Figma designs (if relevant)
[ ] Checked that data has been successfully saved to the backend (if relevant)

Checklist (for merger)

[ ] Make sure the right merge strategy is chosen, commit merge into master and squash merge into develop.

Closing as stagnant, but can revisit when we want the code. Nice work!


Automated cherry pick of #9458: Bugfix/yousong eip ipaddr

Cherry pick of #9458 on release/3.6.
#9458: Bugfix/yousong eip ipaddr

Codecov Report

Merging #9478 (bb0568d) into release/3.6 (b9d30c0) will decrease coverage by 1.89%.
The diff coverage is 3.39%.

@@               Coverage Diff               @@
##           release/3.6   #9478       +/-   ##
===============================================
- Coverage         5.86%   3.97%    -1.90%
===============================================
  Files              828     170      -658
  Lines           129133   25462   -103671
===============================================
- Hits              7577    1012     -6565
+ Misses          120876   24336    -96540
+ Partials           680     114      -566

Flag
Coverage Δ

aFlag
3.97% <3.39%> (-1.90%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/cloudcommon/db/apply_manager_method.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/archs.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/db_dispatcher.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/models.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/sharablebase.go
6.01% <0.00%> (-0.06%)
:arrow_down:

pkg/cloudcommon/db/virtualjointbase.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/virtualresource.go
0.00% <0.00%> (ø)

pkg/cloudcommon/policy/policy.go
1.20% <0.00%> (-0.01%)
:arrow_down:

pkg/cloudprovider/objectstore.go
2.06% <0.00%> (-0.51%)
:arrow_down:

pkg/cloudprovider/resources.go
0.00% <0.00%> (ø)

... and 607 more

Continue to review full report at Codecov.

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


ConnectionFactory.connect should return immediately

ConnectionFactory.connect should return immediately regardless of connected state.  Current behavior is that the call will block until the client can establish an initial connection to the server.  This can cause problems if the server is absent for long periods.  Make ConnectionFactory.connect non-blocking and return immediately so that the connection can be closed even if a connection to the server is never established.

Instead of having the call return immediately so that the connection object can be shutdown, change the implementation so that the calling thread can be interrupted by the user.  Also implement a timeout property.


Windows mws "leaking" files to %WINDIR% System32 / SYSWOW64

Gogs version (or commit ref): 10.0.8
Git version: 2.12
Operating system: Win Srv 2012R2
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[X] SQLite

Can you reproduce the bug at https://try.gogs.io:

[X] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist (usually found in log/gogs.log):

When started as a service, some files are created in C:\Windows\SysWow64\Data (for x86) or C:\Windows\System32\Data (amd64)
These are (if only set /data/gogs.db and not C:/GOGS/DATA/gogs.db) the gogs.db
the folders "session" and "temp" (with local-repo/*)
There should be a way to make them "permanent" in the app.ini so you wont wonder "WTF? I just deleted everything and all users and organisations are still there???" or strange behaviour during "changing file via web"
So my old request "database gets corrupted" just disolved in "now you can see me..." ;)
...

Hi, I think this is duplicated to https://github.com/gogits/gogs/issues/4260. You can comment on that.

it's partly #4260. I confirm the db --- if you just fill in data/gogs.db. If you enter (as shown) a full path like C:\gogs\data\gogs.db this is used.
BUT tmp and the subfolders still reside in system32 and/or syswow64.
There should be a "basedir" or something of the like so ALL gogs-folders are in one location (for backup and problem solving highly appreciated) ;)

Why DB setting has anything to do with session? I do not understand the logic and relation here.

the DB is stored correctly in "C:\GOGS\data\gogs.db" if the setting in NOT "data/gogs.db" but "C:/gogs/data/gogs.db".
If you set only "data/gogs.db" (as on linux), it's stored in SYSWOW64 or System32 - depending on the version. If you install i386 on an amd64, it's SYSWOW64. The same path gogs is using für session, temp, local-* and so on.

Maybe helpful: https://discuss.gogs.io/t/gogs-ignores-app-data-path-from-custom-conf-app-ini/982/6

https://gogs.io/docs/advanced/configuration_cheat_sheet#server-(server), APP_DATA_PATH.


String concatenation with string.Join

The How-To on string concatenation doesn't mention String.Join method. One might say that it is not pure concatenation as we use separator, but the last paragraph uses LINQ to join string with space separator. Performance-wise Join uses StringBuilder inside. @BillWagner if you think the section should be updated with Join mention and examples, I'm willing to update it.

That is a great idea, @pkulikov I'd be happy to have you update the How-to so that it includes string.Join.
I've assigned this to you, and set it in the backlog milestone. @- mention me when you have a PR, and I'll review it.


claw_db undefined error fixed

fatal: [default]: FAILED! => {"msg": "The conditional check 'crayfish_db == 'mysql'' failed. The error was: error while evaluating conditional (crayfish_db == 'mysql'): 'claw_db' is undefined\n\nThe error appears to have been in '/home/andr/CLAW-latest/claw-playbook/roles/external/Islandora-Devops.crayfish/tasks/db-mysql.yml': line 3, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: Create Gemini DB (mysql)\n  ^ here\n"}
When changed with crayfish_db_name installed on CentOS. Please check it on Debian.

My error when working with clayfish, exluded database.yml accidently.


Can't run pip install -r requirements.txt

Could you please provide a docker image, which has installed all the dependencies of this app, to run your sample code?
It always shows many types of dependencies error when we are preparing the run environments.

I’m sorry you are having issues. The requirements file should install the necessary packages, which issues are you encountering?

yes, I initial the env with pip install -r requirements. But it reports many errors, such as:
`Collecting matplotlib==3.0.3
Using cached https://mirrors.aliyun.com/pypi/packages/26/04/8b381d5b166508cc258632b225adbafec49bbe69aa9a4fa1f1b461428313/matplotlib-3.0.3.tar.gz (36.6 MB)
Preparing metadata (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [52 lines of output]
Traceback (most recent call last):
File "", line 2, in
File "", line 34, in
File "/tmp/pip-install-zanu5_as/matplotlib_2e4e6a9fbcbc467c99ec43ae9cb63e48/setup.py", line 225, in
msg = pkg.install_help_msg()
File "/tmp/pip-install-zanu5_as/matplotlib_2e4e6a9fbcbc467c99ec43ae9cb63e48/setupext.py", line 650, in install_help_msg
release = platform.linux_distribution()[0].lower()
AttributeError: module 'platform' has no attribute 'linux_distribution'
IMPORTANT WARNING:
pkg-config is not installed.
matplotlib may not be able to find some of its dependencies
============================================================================
Edit setup.cfg to change the build options
  BUILDING MATPLOTLIB
              matplotlib: yes [3.0.3]
                  python: yes [3.10.12 (main, Jun 11 2023, 05:26:28) [GCC
                          11.4.0]]
                platform: yes [linux]

  REQUIRED DEPENDENCIES AND EXTENSIONS
                   numpy: yes [version 1.25.1]
        install_requires: yes [handled by setuptools]
                  libagg: yes [pkg-config information for 'libagg' could not
                          be found. Using local copy.]
                freetype: no  [The C/C++ header for freetype2 (ft2build.h)
                          could not be found.  You may need to install the
                          development package.]
                     png: no  [pkg-config information for 'libpng' could not
                          be found.]
                   qhull: yes [pkg-config information for 'libqhull' could not
                          be found. Using local copy.]

  OPTIONAL SUBPACKAGES
             sample_data: yes [installing]
                toolkits: yes [installing]
                   tests: no  [skipping due to configuration]
          toolkits_tests: no  [skipping due to configuration]

  OPTIONAL BACKEND EXTENSIONS
                     agg: yes [installing]
                   tkagg: yes [installing; run-time loading from Python Tcl /
                          Tk]
                  macosx: no  [Mac OS-X only]
               windowing: no  [Microsoft Windows only]

  OPTIONAL PACKAGE DATA
                    dlls: no  [skipping due to configuration]

  ============================================================================
                          * The following required packages can not be built:
                          * freetype, png
  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed`

I see. Unfortunately I cannot provide a docker image at the moment but can help you debug in this thread. In this case it looks like the version of matplotlib that is requires is too old and relying on a deprecated package. You can finx this by unpinning the dependency and running pip install matplotlib.

I can't run the requirement.txt file either.
My error message:
error in pandas setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers; Expected end or semicolon (after version specifier)           pytz >= 2011k

@xingvoong this sounds like a different issue. The requirements.txt file does not include a pytz package, so I'm assuming this is an error in the pandas version. I'd try the same thing I recommended above, and run pip install pandas


Option to deselect all plugins

Plugins are enabled by default and there is no way to disable all from CLI.

@chfast I think you can use plugin: noop
@rohan-at-sentry do you mind documenting this?

Done --> https://docs.codecov.com/docs/cli-options#do-upload


Response of get_account_data takes 20 mins...

While calling get_account_data it takes total 20 mins to get the response from the node. Whats the issue?

Hi @rav1112 did the response contain lots of addresses?  The slowest part of get_account_data() is generating addresses — the more addresses you've used from a particular seed, the more time it will take to complete this request.
If there were only a few addresses, then the issue might just be that the node you connected to is really slow for some reason; try connecting to a different node and see if the response comes back more quickly.

Hey @rav1112 it's been a little while; I'm going to go ahead and close this issue.  If you are still having problems with get_account_data(), feel free to re-open this issue so that we can have a look 😺


05/28/2018 02:15 : Create Pull Request for branch master -> prTesting1_2018052802

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon1.md
: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.


[BUG]: Start Race button not centered

Description
When the window is resized to a width between 640 and 767 pixels, the "Start Race" button on the landing page is not properly centered. I think it could be fixed in hero-banner.tsx
(optional) What browsers are you seeing the problem on?
Chrome
Code of Conduct

[X] I agree to follow this project's Code of Conduct

ill work on this


Crashes when removing sfizz on Reaper

Enviroment: EndeavourOS (Arch Linux), REAPER 6.26, VST3
Using sfizz develop build from OBS, sfizz-1617848785.39f56f2d-0
[sfizz] new synth
[New Thread 0x7fffdd6a0640 (LWP 6651)]
[New Thread 0x7fffdce9f640 (LWP 6652)]
[New Thread 0x7fffdc69e640 (LWP 6653)]
[New Thread 0x7fffdbe9d640 (LWP 6654)]
[New Thread 0x7fffdb69c640 (LWP 6655)]
[New Thread 0x7fffdae9b640 (LWP 6656)]
[Thread 0x7fffdae9b640 (LWP 6656) exited]
[New Thread 0x7fffdae9b640 (LWP 6657)]
[sfizz] about to create view: editor
[sfizz] about to open view with parent 0x260058a
[New Thread 0x7fffd51a4640 (LWP 6658)]
[New Thread 0x7fffd49a3640 (LWP 6659)]

(reaper:6633): GLib-CRITICAL **: 10:02:02.444: Failed to set scheduler settings: Operation not permitted
[New Thread 0x7fffabfff640 (LWP 6660)]
[New Thread 0x7fffab7fe640 (LWP 6661)]
[New Thread 0x7fffaaffd640 (LWP 6662)]
[Thread 0x7fffaaffd640 (LWP 6662) exited]
[Thread 0x7fffabfff640 (LWP 6660) exited]
[Thread 0x7fffd49a3640 (LWP 6659) exited]
[Detaching after vfork from child process 6664]
[Thread 0x7fffab7fe640 (LWP 6661) exited]
[Thread 0x7fffdae9b640 (LWP 6657) exited]
[Thread 0x7fffdc69e640 (LWP 6653) exited]
[Thread 0x7fffdce9f640 (LWP 6652) exited]
[Thread 0x7fffdb69c640 (LWP 6655) exited]
[Thread 0x7fffdbe9d640 (LWP 6654) exited]
[Thread 0x7fffdd6a0640 (LWP 6651) exited]

Thread 1 "reaper" received signal SIGSEGV, Segmentation fault.
--Type <RET> for more, q to quit, c to continue without paging--<RET>
0x00000000008e1dd6 in ?? ()
(gdb) bt
#0  0x00000000008e1dd6 in  ()
#1  0x00007ffff76f4098 in  () at /opt/REAPER/libSwell.so
#2  0x0000000000415756 in  ()
#3  0x00007ffff79a7b25 in __libc_start_main () at /usr/lib/libc.so.6
#4  0x000000000041ae5a in _start ()

I supposed this is not enough information to gather, so let me know what should I do for a more meaningful log

It's not possible to reproduce it, can you indicate a list of steps that we can follow?
As of the backtrace, it doesn't tell much without debug info, surely it would be more useful if run on a debug-enabled build. (that is -DCMAKE_BUILD_TYPE=RelWithDebInfo)

Simply add sfizz to a FX chain and removing it causing crashes. I also tried to build from with -DCMAKE_BUILD_TYPE=Debug and attach Reaper to gdb using gdb reaper -ex run -ex bt, still got the same backtrace.
That said, I did a clean reinstall, getting sfizz-git from AUR and the crashes seems didn't happen anymore. I think I'll close the issue for the time being


Setup & Mobile for Porfolio Files

In this branch,

I created the index html file as the homepage of the portfolio website.
I added added the html header element which contains header meta tags.
I added the body element which includes, a second header element, the title, sub title, a paragraph and social media links.
I added an external CSS folder which contains a master stylesheet and an image folder to hold all graphics.
I added a linter that helps to diagnose and fix technical issues

I’m repeating the Module and I asked the students success who said it was fine to use previous work
Kind regards,
Kemboi Kiptanui

From: Francis Wayungi @.>
Sent: Saturday, July 8, 2023 9:45:08 AM
To: okemboi/portfolio @.>
Cc: Kemboi @.>; Mention @.>
Subject: Re: [okemboi/portfolio] Setup & Mobile for Porfolio Files (PR #1)
@wayungi commented on this pull request.
Hi @okemboihttps://github.com/okemboi ,
[image]https://user-images.githubusercontent.com/99656687/224543512-ddbefa59-dc68-47c2-b7fe-f55acdacd3eb.png
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request ❌ ❌

You have submitted a project pre-approved PR for review. Kindly submit a new PR for evry milestone.

Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.
—
Reply to this email directly, view it on GitHubhttps://github.com/okemboi/portfolio/pull/1#pullrequestreview-1520661809, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AS3MLRYYPQCNY6PDJZYNDBTXPD6XJANCNFSM6AAAAAAWNCCJI4.
You are receiving this because you were mentioned.Message ID: @.***>

Hey @okemboi , the mistake is mine. please let me fixed.

OK. Thank you
Kind regards,
Kemboi Kiptanui

From: Francis Wayungi @.>
Sent: Saturday, July 8, 2023 9:48:17 AM
To: okemboi/portfolio @.>
Cc: Kemboi @.>; Mention @.>
Subject: Re: [okemboi/portfolio] Setup & Mobile for Porfolio Files (PR #1)
Hey @okemboihttps://github.com/okemboi , the mistake is mine. please let me fixed.
—
Reply to this email directly, view it on GitHubhttps://github.com/okemboi/portfolio/pull/1#issuecomment-1626888505, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AS3MLRZOFOEBYWMUZI4KGADXPD7DDANCNFSM6AAAAAAWNCCJI4.
You are receiving this because you were mentioned.Message ID: @.***>

Please let me know if it shows approved on your dashboard

Hey @okemboi, is everything okay now?

Thank you
Best Regards
K. Kemboi

From: Souad El Mansouri @.>
Sent: Saturday, July 8, 2023 11:56:57 AM
To: okemboi/portfolio @.>
Cc: Kemboi @.>; Mention @.>
Subject: Re: [okemboi/portfolio] Setup & Mobile for Porfolio Files (PR #1)
@souad988 commented on this pull request.
Hi @okemboihttps://github.com/okemboi ,
Your project is complete! There is nothing else to say other than... it's time to merge it [:shipit:]
Congratulations! 🎉
Highlights
Good efforts
Design well implemented
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policyhttps://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this formhttps://airtable.com/shrQAqnBwek5a0O0s.
—
Reply to this email directly, view it on GitHubhttps://github.com/okemboi/portfolio/pull/1#pullrequestreview-1520675331, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AS3MLR7WSHNPYEGDMQQYPJTXPEOFTANCNFSM6AAAAAAWNCCJI4.
You are receiving this because you were mentioned.Message ID: @.***>


Allow import_dangerfile to accept full path

Currently import_dangerfile(path: '...') assumes the user provides the path to the directory containing a Dangerfile.
This is not the case when we are splitting large Dangerfiles into smaller ones. We have to place each Dangerfile in separate directories.
In this PR I modified the import_dangerfile_from_path method to check if the provided path is a file, if yes, then use that path to import. Otherwise, append Dangerfile to the path and import it.
Fixes #1379

Cool, thanks 👍🏻


Multi language after deploy

Hello,
I have a project which I added another language using the new multilanguage feature. In this case I already had created the bot in  english and added portuguese. I translated the bot and then I changed the deault language and the bot language to portuguese as you can see in the screenshot below.

Below is my local configuration
{   "feature": {     "UseShowTypingMiddleware": false,     "UseInspectionMiddleware": false,     "RemoveRecipientMention": false   },   "MicrosoftAppPassword": "",   "MicrosoftAppId": "",   "cosmosDb": {     "authKey": "",     "collectionId": "botstate-collection",     "cosmosDBEndpoint": "",     "databaseId": "botstate-db"   },   "applicationInsights": {     "InstrumentationKey": ""   },   "blobStorage": {     "connectionString": "",     "container": "transcripts"   },   "luis": {     "name": "edpV3",     "authoringKey": "xxxxxxxxxxx",     "endpointKey": "",     "authoringRegion": "westeurope",     "defaultLanguage": "pt-br",     "environment": "composer",     "endpoint": "xxxxxxxxxxx",     "authoringEndpoint": ""   },   "qna": {     "knowledgebaseid": "xxxxxxxxx",     "hostname": "xxxxxxxxx",     "endpointKey": "xxxxxxxxx",     "subscriptionKey": "xxxxxxxxx",     "qnaRegion": "westus"   },   "telemetry": {     "logPersonalInformation": false,     "logActivities": true   },   "runtime": {     "customRuntime": false,     "path": "",     "command": ""   },   "downsampling": {     "maxImbalanceRatio": 10,     "maxUtteranceAllowed": 15000   },   "defaultLanguage": "pt-br",   "languages": [     "en-us",     "pt-br"   ] }
So far so good. I tested on the emulator and it worked as intended. When I change the locale on the emulator the bot starts speaking in the respective language and understands what I tell him in the respective language also.
The problem is when I deploy the bot.
Below is my deploy configuration.
{   "accessToken": "xxxxxxxxxxxx",   "name": "edp-test-bot",   "environment": "dev",   "settings": {     "applicationInsights": {       "InstrumentationKey": "<Instrumentation Key>"     },     "cosmosDb": {       "cosmosDBEndpoint": "<endpoint url>",       "authKey": "<auth key>",       "databaseId": "botstate-db",       "collectionId": "botstate-collection",       "containerId": "botstate-container"     },     "blobStorage": {       "connectionString": "<connection string>",       "container": "<container>"     },     "luis": {       "authoringKey": "xxxxxxxxxxxxxxxxxxxx",       "authoringEndpoint": "xxxxxxxxxxxxxxxxxxxxx",       "endpointKey": "xxxxxxxxxxxxxxxxx",       "endpoint": "xxxxxxxxxxxxxxxxxxxxxxx",       "region": "westeurope",       "defaultLangage": "pt-br"     },     "MicrosoftAppId": "xxxxxxxxxxxx",     "MicrosoftAppPassword": "xxxxxxxxxxxxxxxxx"   },   "hostname": "edp-testbot",   "luisResource": "edp-testbot-luis",   "defaultLanguage": "pt-br" }
The problem is that the bot starts speaking in portuguese but it only understands when I talk in english, so it is checking on the en-us apps that are on LUIS instead of checking the pt-br ones.
I'm using the same LUIS resource for local and for the prooduction version.
I added the "defaultLanguage" properties with hope it would fix my problem but it had no effect.
Can you please check if there is something wrong with my configurations.
Also below is an image showing that both en-us and pt-br apps exist on LUIS.

An update
I tried to deploy the chatbot with only the portuguese language (eliminated the english) and it still does not work, it starts talking to me in portuguese but when I talk to him it seems it doesn't reach the LUIS service.
It continues to work ok locally in the emulator

@Bernardo08pt can you share the information coming from the LuisTrace log when testing in the Emulator?

Do you mean this

@Bernardo08pt , thanks for reporting this, there is a bug in our azure deployment logic, we take the language field from the publishing profile instead of defaultLanguage, you can mitigate this issue by just updating your profile, replace the defaultLanguage with language.
{
	"accessToken": "xxxxxxxxxxxx",
	"name": "edp-test-bot",
	"environment": "dev",
	"settings": {
		"applicationInsights": {
			"InstrumentationKey": "<Instrumentation Key>"
		},
		"cosmosDb": {
			"cosmosDBEndpoint": "<endpoint url>",
			"authKey": "<auth key>",
			"databaseId": "botstate-db",
			"collectionId": "botstate-collection",
			"containerId": "botstate-container"
		},
		"blobStorage": {
			"connectionString": "<connection string>",
			"container": "<container>"
		},
		"luis": {
			"authoringKey": "xxxxxxxxxxxxxxxxxxxx",
			"authoringEndpoint": "xxxxxxxxxxxxxxxxxxxxx",
			"endpointKey": "xxxxxxxxxxxxxxxxx",
			"endpoint": "xxxxxxxxxxxxxxxxxxxxxxx",
			"region": "westeurope",
			"defaultLangage": "pt-br"
		},
		"MicrosoftAppId": "xxxxxxxxxxxx",
		"MicrosoftAppPassword": "xxxxxxxxxxxxxxxxx"
	},
	"hostname": "edp-testbot",
	"luisResource": "edp-testbot-luis",
	"defaultLanguage": "pt-br"
}


Modules: "require is not defined in ES module scope"

I'm trying to port my CDK app to SST. I use ES Modules because I use @aws-cdk/aws-lambda-nodejs which uses esbuild, meaning tree-shaking only works in ESM projects.
(Note that the docs that say SST uses its own Lambda builder so it can use esbuild is very strange since NodejsFunction already does and you can customize how it builds)
> sst deploy "--stage" "prod" "--region" "us-east-1"

Using stage: prod
Preparing your SST app
Detected tsconfig.json
Transpiling source
Linting source
ReferenceError: require is not defined in ES module scope, you can use import instead
This file is being treated as an ES module because it has a '.js' file extension and '/Users/cyber/dev/platform/packages/infra/package.json' contains "type": "module". To treat it as a CommonJS script, rename it to use the '.cjs' file extension.
    at file:///Users/cyber/dev/platform/packages/infra/.build/eslint.js:10:20
    at ModuleJob.run (internal/modules/esm/module_job.js:170:25)
    at async Loader.import (internal/modules/esm/loader.js:178:24)
    at async Object.loadESM (internal/process/esm_loader.js:68:5)
There was a problem linting the source.

eslint.js does in fact call const { ESLint } = require("eslint"); which is not valid in ESM. Maybe use import or name the files with the .cjs extension?
If I disable linting then I run into the same problem in run.js.

I think the easiest way to fix this is to write out a package.json file in .build with the sole contents:
{"type": "commonjs"}
This works for me if I do it manually. I was trying to see if there was somewhere in the code that would be a good place to add that simple writeFileSync call but I wasn't sure where exactly. I can make a PR if you point to me where.
It would be great to use ES modules though. Esbuild isn't really that great of a tool for lambda development if you use commonjs because it can't tree shake, resulting in larger bundles, and larger cold start times.


SOLR-15965 Use proper signatures for SolrAuth

https://issues.apache.org/jira/browse/SOLR-15965
This is a draft of what the code could look like in 9.0 if we want to maintain backwards compatibility along the full 9x line. Alternatively, the majority of this code could go to 8.11.2 and only the "V2" portions of this code would make it into 9.0

I don't know why precommit failed on this building the ref guide with broken links, the link appears to work for me locally when I build it... @ctargett any ideas? Alternatively, we can wait until Antora comes in and then I'll redo the content anyway.


Problem to send messages to everyone in our organization

Hi,
I am facing a problem to send message to everyone in our organization. we are getting some errors:
These errors are related to:

TeamsConversationActivity
Exception while executing function: TeamsConversationActivity The remote server returned an error: (400) Bad Request

When the same message are sent to Team or a specified people, it works normally,
Anyone has an idea?

Hello @flpmoreno,
TeamsConversationActivity has a different logic when an author sends a message to members of specific Teams.
Could you try following:

Send a message to members of an M365 / Distribution / Security group
Send a message to everyone
Share the error message captured in the message status for step 1, step 2.
If possible, share the failed requests from Application Insights with errors/exception details. (Go to Application Insights and check for failed requests)

Hi @juna-gupta
Thank you for your guideline here. It was useful.
The error was solved by setting ProactiveInstallApp to TRUE and some changes directly in the TABLE.
I got MS support to help me.
Thanks in advanced and congrats for all support here,


Really each reward

Thanks for your torch-based training.
Can you provide the results together in another branch (e.g. torch-based)?


[ColdFix] Fix for interact_bls times when using TESS data

TESS data has a different time offset to Kepler data. TESS uses BTJD (JD - 2457000) where Kepler uses BKJD (JD - 2454833). I've made a quick fix to interact_bls to take these two time factors into account in the unfolded light curve plot panel. I've also change it so that the t0 values have the correct time factor appended.

Fixes #412

Codecov Report

Merging #420 into master will decrease coverage by 0.14%.
The diff coverage is 14.28%.

@@            Coverage Diff             @@
##           master     #420      +/-   ##
==========================================
- Coverage   88.53%   88.39%   -0.15%
==========================================
  Files          28       28
  Lines        4816     4825       +9
==========================================
+ Hits         4264     4265       +1
- Misses        552      560       +8

Impacted Files
Coverage Δ

lightkurve/interact_bls.py
37.64% <14.28%> (-0.98%)
:arrow_down:

Continue to review full report at Codecov.

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


CupertinoSegmentedControl in iOS 13 has been redesigned

Looks like we have some changes in iOS 13 for the CupertinoSegmentedControl.
OLD - (iOS 12 and older)

NEW - (iOS 13)

An issue has been opened to track this already - https://github.com/flutter/flutter/issues/33796
There is a larger-scale issue at https://github.com/flutter/flutter/issues/35026 that tracks all iOS13 changes. I will close this for now. If you find any other iOS 13 changes that have not already been added, please feel free to add those.
I will close this for now :) Thank you!

is the flutter support for ios 13 dark mode?
and how to add support for dark mode for ios only?


Current Internet Speed

#30
it will say about internet speed
"Download Speed is 10 MB/s"
"upload Speed is 10 MB/s"

@BAHETIHARSH Can you pull all the changes from upstream so that the merge conflicts will be resolved


Captioning figures, code snippets, et al.

Make this possible.

It already was possible via correct image sizing and image markdown.


Ability to assert arguments of specific function invocation

For the situation where a function is called 10 times, is there a way to assert what args the 6th invocation of the function was called with?
assert_called(MyModule.my_function(session_arg), 10)
In this example session_arg will bind to the last matching call, which allows me to do asserts against that. However, I want to assert the 6th call. Is this possible, and if not, could it be added as a feature?

For more advanced use cases like this you can make use of history/2 https://hexdocs.pm/patch/Patch.html#history/2 which provides all the calls that have been seen.  So for example you could do
{:my_function, [session_arg]} =
  MyModule
  |> history()
  |> Enum.filter(fn {func, args} -> func == :my_function end)
  |> Enum.at(5)

assert session_arg == expected_session_arg

@Arsenalist I'm going to close this out as answered, but if you still require assistance please feel free to open a new issue.


Error: Unable to update lock within the stale threshold

Which package is this bug report for? If unsure which one to select, leave blank
@crawlee/cheerio (CheerioCrawler)
Issue description
There is an exception running the crawler after some time.
I haven't detected yet the exact conditions, in some cases the crawler is running for several minutes (+10) and suddenly fails, in others it just happens after a few seconds.
/Users/ignacio/Workspaces/p2pbookstore/packages/crawler/node_modules/graceful-fs/polyfills.js:306
        if (cb) cb.apply(this, arguments)
                   ^
Error: Unable to update lock within the stale threshold
    at /Users/ignacio/Workspaces/p2pbookstore/packages/crawler/node_modules/proper-lockfile/lib/lockfile.js:136:25
    at callback (/Users/ignacio/Workspaces/p2pbookstore/packages/crawler/node_modules/graceful-fs/polyfills.js:306:20)
    at FSReqCallback.oncomplete (node:fs:211:5)
    at FSReqCallback.callbackTrampoline (node:internal/async_hooks:130:17) {
  code: 'ECOMPROMISED'
}

Code sample
I'm afraid I don't have it yet (trying - will update if I make it).

Only think I have apart from the defaults is: `sameDomainDelaySecs: 1`.
I also updated from 3.3 when started happening (precisely because of sameDomainDelaySecs).

Package version
<EMAIL_ADDRESS>Node.js version
v18.17.0
Operating system
MacOS
Apify platform

[ ] Tick me if you encountered this issue on the Apify platform

I have tested this on the next release
No response
Other context
No response

Just a quick hint, it seems the problem can be workaround with maxConcurrency: 1, (it might be obvious but just commenting)

Could you share some code sample that reproduces this? Or this is a very intermitent issue? 😅

I have the same issue. In my case, it has something to do with minConcurrency. With a large number of requests in the queue, desiredConcurrency dropped to 1. After installing minConcurrency, it began to receive such an error.


Fix gap callback for degenerate Hamiltonians

Fixes #324 by going to the next eigenenergy if the gap is zero (< EIGVAL_CUTOFF for precision reasons). If this happens a warning is raised. I also moved test_callbacks.py to the new tests and enabled testing with the numpy backends.

Fixes #324 by going to the next eigenenergy if the gap is zero (< EIGVAL_CUTOFF for precision reasons). If this happens a warning is raised. I also moved test_callbacks.py to the new tests and enabled testing with the numpy backends.

Can you show me how to plot out other Ground states when the system has two degenerations of ground states in linear.py? I'm struggling to find the other ground states when running linear.py ( it only plots out one ground state).
(Sorry, I'm a beginner in Python.)


arch/x86: move arch-specific thread_state flags to thread_arch struct

As the kernel has grown it has used more of the thread_state bits and now bits inadvertently overlap with those used by the x86 arch.  This has gone unnoticed because the overlap only occurs with the SMP scheduler enabled, which it never has been on 32-bit x86.  So, rather than grow the thread_state, which is arguably private to the scheduler anyway, the relevant flags bits have been moved to the arch-specific thread state.
Fixes: #17361

@andyross I'd actually like to go one step farther and revise the comments in kernel_structs.h to indicate that arches should not use any bits for themselves, but I thought I'd check with you first.  Looks like this was supposed to be OK long ago, but as far as I can tell, x86 is/was the only arch to exploit that, and we're running out of room in the byte.


Classname not printed

When I run php sami.phar parse config.php, the errors do not show the classname (also for update, of course). The output is:
ERROR: An error occurred while parsing "::__construct" line "37": "2" @param tags are expected but only "1" found

And I don't want to look through all my constructors :)
config.php:
<?php

return new Sami\Sami('../git-repository/src');

Does this happen only for constructor or other methods as well?
Any PR coming in?

All methods. What do you mean, if a PR is cimong in?

I could try and fix it tonight, but right now I am at work.

What do you mean, if a PR is coming in?

From my experience people reporting issues usually are willing to submit PR, because the issue being reported is one discovered at work and is usually a blocker to them.
Some people just report issues and don't send PR. By asking I've made visible so that an issue is open for any willing person to implement or we should wait for issue author to send PR as well.

I will look at it tonight and fix it if possible. Will get back.


Add attributes argument to recordException API

Which problem is this PR solving?
Adds the optional attributes argument to the recordException API.
Fixes #4070
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] packages/opentelemetry-sdk-trace-base/test/common/Span.test.ts

Checklist:

[x] Followed the style guidelines of this project
[x] Unit tests have been added
[ ] Documentation has been updated

For reference, https://github.com/open-telemetry/opentelemetry-js-api/pull/97 is the last attempt to implement the feature. @dyladan would you mind confirming if the condition is changed since then (about the comment https://github.com/open-telemetry/opentelemetry-js-api/pull/97#issuecomment-868480479)?

For reference, open-telemetry/opentelemetry-js-api#97 is the last attempt to implement the feature. @dyladan would you mind confirming if the condition is changed since then (about the comment open-telemetry/opentelemetry-js-api#97 (comment))?

That change indeed looks very similar. Not sure why it didn't fully go in. It looks like the addEvent API has been appropriately modified, but not recordException.

@dyladan would you mind confirming if the condition is changed since then (about the comment open-telemetry/opentelemetry-js-api#97 (comment))?

We've had several feature releases of the API since then. There is no need to backport fixes to previous minor versions. The backport requirement is only for major versions according to https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/versioning-and-stability.md#api-support

I updated the changelog to unblock the changelog workflow.

@legendecas @dyladan thank you for your reviews! How do I merge this PR now?

@MSNev I update the APIs as requested. RE:

Also need to add the param to the documentation

Could you point me to where I need to make this change?

Also need to add the param to the documentation

Could you point me to where I need to make this change?

I believe you have done this well in the latest commit.


Repo issues for rocprofiler-dev and rocr_debug_agent

This is the output
Err:1 http://repo.radeon.com/rocm/apt/debian xenial/main amd64 rocprofiler-dev amd64 1.0.0
  Hash Sum mismatch
Get:2 http://repo.radeon.com/rocm/apt/debian xenial/main amd64 rocr_debug_agent amd64 1.0.0 [890 kB]
Err:2 http://repo.radeon.com/rocm/apt/debian xenial/main amd64 rocr_debug_agent amd64 1.0.0
  Writing more data than expected (890180 > 890176)
Fetched 216 kB in 0s (2032 kB/s)
E: Failed to fetch http://repo.radeon.com/rocm/apt/debian/pool/main/r/rocprofiler-dev/rocprofiler-dev_1.0.0_amd64.deb  Hash Sum mismatch

E: Failed to fetch http://repo.radeon.com/rocm/apt/debian/pool/main/r/rocr_debug_agent/rocr_debug_agent_1.0.0_amd64.deb  Writing more data than expected (890180 > 890176)

The update of the deb-database is not done (correctly)?

I tested the repo again. Didn't see this issue. The current release is 2.10. I also tried 2.9 release which was the latest in October and no issue too.
Did you clear apt cache before install? Are you behind a firewall or using proxy servers? These are possible reasons from your side to see this error.

Missed your reply. Cache cleared, no proxy, all other repos work. I solved it by manually downloading rocprofiler-dev, removing rocprofiler-dev, manually installing rocprofiler-dev with dpkg -i, and then reinstalling "hipblas rocblas rocm-dev rocprofiler-dev rocrand".
Is the repo hosted on a server that has FAT or any other MS filesystem? It seems that the more data is sent than advertised. When I download the file on Windows, the file-size and size-on-disk are equal to what is printed. I think it would be solved by or updating the filesize to size-on-disk, or by not sending the trailing zeroes.

Please check latest ROCm Documentation and ROCm 5.7.1 to see if your issue has been resolved.  If resolved, please close the ticket.  Thanks.

Original ticket is more than a year old and the person that opened the ticket has not responded to the latest request.  If this is still an issue, please file a new ticket and we will investigate.  Thanks!


NameError: name 'auth_result' is not defined

Hi, i get the following error when i try to run the program:
NameError: name 'auth_result' is not defined
What am i doing wrong ?
Thx Ed.

Hard to tell when you're not writing what exactly you're doing...

I have made an account in Homeconnect. I don't really know what the redirect url should be. I think that is the problem ?
I am running your code on Windows btw. (python 3.8)

If you're just using this locally, it doesn't matter. Use http://localhost (but make sure to set it in the application settings at Home Connect). Your browser will redirect you to this address with some token append and it doesn't matter whether it exists - you just have to copy the address from the address bar in your browser.

Do you mean this:  (this shows up after logging in into Homeconnect)
http://localhost/?code=eyJ4LXJlZyI6IkVVIiwieCHJUgHz8jJHGGnsnajwlbiI6ImQyODVkMzRmLTlkZGQtNGQ2Zi04dfkjdfkjh321kjh213kjh213jhMmQ5ZWUwNTIzNCIsImNsdHkiOiJwcml2YXRlIn0%3D&state=QXbhEpRlQXhLAd2vhbIXPzsFjh1IMV&grant_type=authorization_code
What should be the next step ?

Got it working. Thank you.


Set record status via pac power-fx

We are currently using PowerShell scripts and the corresponding PS Modules to connect to Dataverse to set the status of specified records. e.g. SLA
When updating a SLA via Solution import you have to ensure it's activated afterwards again.
However, I now wanted to replace the PS usage with pac cli as it is available in Azure Pipelines without additional effort and Power-Fx is maybe easier to use.
When I want to update the status of record like SLA or process, I'm facing an error message:

When I use the exactly same command within a Canvas app it works fine, so it seems it is a limitation of pac power-fx.
Are there any plans to close this gap?

We are tracking this internally. There is no hard estimate but we are interested in fixing this soon.

For internal reference: https://github.com/microsoft/Power-Fx-Dataverse/issues/480


use containerd/project header test

Time to align our copyright / license header to containerd/project.

Adds containerd/project's use of github.com/kunalkushwaha/ltag to our CI / Project Checks Job.
Modifies all copyright headers to align with containerd headers.

Addresses a portion of #1448
Signed-off-by: Mike Brown<EMAIL_ADDRESS>

LGTM


citation

Hi! Great paper :) So the citation [20] should actually go to Ryan Murdock (https://twitter.com/advadnoun), who discovered how to put CLIP and BigGAN together in a colab notebook. I simply packaged it into a CLI tool.

Thanks! We'll change the reference accordingly.