Can't get the window.plugins.backgroundGeoLocation object

Hi Chris,
I am looking for a solution to my problem and I've been searching in issues, stackoverflow and more, without finding a solution :
I installed your plugin as it is explained here : https://github.com/christocracy/cordova-plugin-background-geolocation.
When deviceready event is received, I try to execute your code example in it but I encountered a few problems :

first, window.navigator.geolocation was not found, I had to replace it by navigator.geolocation to get the first geolocation call.
then, the window.plugins.backgroundGeoLocation object is not found too, it always returns null.

I tried with the solution you explain here : https://github.com/christocracy/cordova-plugin-background-geolocation/issues/202 (replacing
<clobbers target="plugins.backgroundGeoLocation" />

by
<clobbers target="window.backgroundGeoLocation" />

in the plugins/org.transistorsoft.cordova.background-geolocation/plugins.xml file) but without the result is still null.
When I try to console the window.plugins object, it only returns me { insomnia: {} }.
I am quite confused, I don't know how to get your plugin object in order to get the geolocation work in background (I had no problem using the gelocation plugin but I need it to run it in background).
My version of phonegap : 5.1.1-0.29.0
My version of cordova : 5.1.1
Thank you by advance for your answer.
Hopping to read you soon !

I am in the exact same situation.  Geolocation work fine but when I use this plugin I get the same error.
I am just using Cordova (not Phonegap) version 5.0.0 on Ubuntu 14.04 LTS.

I may be experiencing something similar in the Premium Version
Try this:  edit plugin.xml
Change that to:
<clobbers target="window.backgroundGeoLocation" />

You'll have to access the plugin like so, of course
onDeviceReady = function() {
    var bgGeo = window.backgroundGeoLocation;
    .
    .
    .
};

And make sure you destroy your platform then rebuild it when you try this
On Thursday, July 9, 2015, neotrode<EMAIL_ADDRESS>wrote:

I am in the exact same situation. Geolocation work fine but when I use
this plugin I get the same error.
I am just using Cordova (not Phonegap) version 5.0.0 on Ubuntu 14.04 LTS.
—
Reply to this email directly or view it on GitHub
https://github.com/christocracy/cordova-plugin-background-geolocation/issues/203#issuecomment-120212640
.

--
Snet from Gmail Mobile

Thanks for your answer.
I tried your solution :

I replaced

<clobbers target="plugins.backgroundGeoLocation" />

by
<clobbers target="window.backgroundGeoLocation" />

I destroyed my platforms with the command

phonegap platform remove <platform>

I added the ios platform again with a

phonegap platform add ios

I built it with

phonegap build ios

I got the following error :
Failed to install 'de.appplant.cordova.plugin.background-mode':Error: Expected plugin to have ID "org.apache.cordova.device" but got "cordova-plugin-device".
    at checkID (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/cordova-lib/src/plugman/fetch.js:164:15)
    at /Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/cordova-lib/src/plugman/fetch.js:148:9
    at _fulfilled (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:787:54)
    at self.promiseDispatch.done (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:816:30)
    at Promise.promise.promiseDispatch (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:749:13)
    at /Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:557:44
    at flush (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:108:17)
    at process._tickCallback (node.js:355:11)

Error: Expected plugin to have ID "org.apache.cordova.device" but got "cordova-plugin-device".
    at checkID (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/cordova-lib/src/plugman/fetch.js:164:15)
    at /Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/cordova-lib/src/plugman/fetch.js:148:9
    at _fulfilled (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:787:54)
    at self.promiseDispatch.done (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:816:30)
    at Promise.promise.promiseDispatch (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:749:13)
    at /Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:557:44
    at flush (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:108:17)
    at process._tickCallback (node.js:355:11)

According to what I saw here : https://github.com/katzer/cordova-plugin-background-mode/issues/90
that was not so important and my platform has been installed anyway, so I tried.
I serve my phonegap app and do a
console.log(window.backgroundGeoLocation);

This is still null...
Any idea of what I did wrong here ?

We need to isolate the problem away from your app.
Please install the SampleApp in the repo and debug with that
On Friday, July 10, 2015, Arthur LORENT<EMAIL_ADDRESS>wrote:

Thanks for your answer.
I tried your solution :

I replaced

by

I destroyed my platforms with the command

phonegap platform remove

I added the ios platform again with a

phonegap platform add ios

I built it with

phonegap build ios
I got the following error :
Failed to install 'de.appplant.cordova.plugin.background-mode':Error: Expected plugin to have ID "org.apache.cordova.device" but got "cordova-plugin-device".
at checkID (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/cordova-lib/src/plugman/fetch.js:164:15)
at /Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/cordova-lib/src/plugman/fetch.js:148:9
at _fulfilled (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:787:54)
at self.promiseDispatch.done (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:816:30)
at Promise.promise.promiseDispatch (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:749:13)
at /Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:557:44
at flush (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:108:17)
at process._tickCallback (node.js:355:11)
Error: Expected plugin to have ID "org.apache.cordova.device" but got "cordova-plugin-device".
at checkID (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/cordova-lib/src/plugman/fetch.js:164:15)
at /Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/cordova-lib/src/plugman/fetch.js:148:9
at _fulfilled (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:787:54)
at self.promiseDispatch.done (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:816:30)
at Promise.promise.promiseDispatch (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:749:13)
at /Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:557:44
at flush (/Users/arthurlorent/.nvm/versions/node/v0.12.2/lib/node_modules/phonegap/node_modules/cordova/node_modules/q/q.js:108:17)
at process._tickCallback (node.js:355:11)
According to what I saw here : katzer/cordova-plugin-background-mode#90
https://github.com/katzer/cordova-plugin-background-mode/issues/90
that was not so important and my platform has been installed anyway, so I
tried.
I serve my phonegap app and do a
console.log(window.backgroundGeoLocation);
This is still null...
Any idea of what I did wrong here ?
—
Reply to this email directly or view it on GitHub
https://github.com/christocracy/cordova-plugin-background-geolocation/issues/203#issuecomment-120301584
.

--
Snet from Gmail Mobile

I updated the clobbers tag and now get bgGeo undefined.  If I nest the scripts onDeviceReady, as suggested, then I can not call any functions nested within the onDeviceReady block via a button tag's onclick routine.
Also note, I followed your suggestion for a baseline and downloaded the SampleApp, added a platform, and proceeded with a build.  I get the same error as noted in the beginning of this thread, "window.plugins.backgroundGeoLocation object is not found"

Should I be using event trigger listeners in the javascript side rather than call a function from the html  button tags?

If you update the clobbers tag to window.backgroundGeoLocation, there is no more window.plugins.backgroundGeoLocation' -- it's window.backgroundGeoLocation`.
"html button tags", what??
You're not doing this <button onclick="someFunction()" /> ?

yep and according to that response, I am admitting that embarrassingly.  Should I stick to the js side and listen for events instead?

Yes
On Friday, July 10, 2015, Frank<EMAIL_ADDRESS>wrote:

yep and according to that response, I am admitting that embarrassingly.
Should I stick to the js side and listen for events instead?
—
Reply to this email directly or view it on GitHub
https://github.com/christocracy/cordova-plugin-background-geolocation/issues/203#issuecomment-120566072
.

--
Snet from Gmail Mobile

You rock!  That did it.  So in summary, your initial suggestion to change the cobble tag to:  took care of it once I re-arranged the code to nest within onDeviceReady.
My app is firing away and the background tracking icon is appearing. Now I just need to work on my code.
My apologies adding the monkey wrench with my logic of html triggering js.  I have some old habits.
Thanks again Chris.

The premium version is 100* better
On Saturday, July 11, 2015, Frank<EMAIL_ADDRESS>wrote:

You rock! That did it. So in summary, your initial suggestion to change
the cobble tag to: took care of it once I re-arranged the code to nest
within onDeviceReady.
My app is firing away and the background tracking icon is appearing. Now I
just need to work on my code.
My apologies adding the monkey wrench with my logic of html triggering js.
I have some old habits.
Thanks again Chris.
—
Reply to this email directly or view it on GitHub
https://github.com/christocracy/cordova-plugin-background-geolocation/issues/203#issuecomment-120572361
.

--
Snet from Gmail Mobile

I'm sure.  Once I have this going perfectly, I will gladly proceed to premium.

Try out edge branch.  Moved window.plugins.backgroundGeoLocation -> window.BackgroundGeolocation.
NOTE There's a change in came-casing as well:  backgroundGeoLocation vs BackgroundGeolocation.

After a few tests, here are the results :
I tried your sample example using only cordova on an android phone and surprise, it works without changing anything in the plugin.xml file.
I still have an error : google is not defined. It seems that the Google Map API call is not well handled, but I get the geolocation notifications.
Using phonegap seems to be quite different because you don't need to build the platform before rendering it on you mobile. I could use all the native cordova plugins (accessible via the navigator variable) but I couldn't make your plugin work (I can't access to your plugin).
This should be possible because I could use the insomnia plugin, which prevent the mobile from enter in sleeping mode.
Here is how the new Phonegap tool works if you want to know more about it : http://app.phonegap.com/
Let me now if you think that there is an existing solution for the phonegap use. If not, I will use cordova and will drop phonegap for my future developments.

What version of Cordova are you using?  perhaps this is a
cordova-plugin-whitelist issue?
https://github.com/apache/cordova-plugin-whitelist
PhoneGap is very limiting.  There are more flexible build-services out
there, such as Cloud Five
https://www.cloudfiveapp.com/
On Tue, Jul 14, 2015 at 12:57 PM, Arthur LORENT<EMAIL_ADDRESS>wrote:

After a few tests, here are the results :
I tried your sample example using only cordova on an android phone and
surprise, it works without changing anything in the plugin.xml file.
I still have an error : google is not defined. It seems that the Google
Map API call is not well handled, but I get the geolocation notifications.
Using phonegap seems to be quite different because you don't need to build
the platform before rendering it on you mobile. I could use all the native
cordova plugins (accessible via the navigator variable) but I couldn't make
your plugin work (I can't access to your plugin).
This should be possible because I could use the insomnia plugin, which
prevent the mobile from enter in sleeping mode.
Here is how the new Phonegap tool works if you want to know more about it
: http://app.phonegap.com/
Let me now if you think that there is an existing solution for the
phonegap use. If not, I will use cordova and will drop phonegap for my
future developments.
—
Reply to this email directly or view it on GitHub
https://github.com/christocracy/cordova-plugin-background-geolocation/issues/203#issuecomment-121307805
.

--
Chris Scott
Transistor Software http://www.transistorsoft.com


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
f8b40e2 -> d366e67

github.com/gruntwork-io/terratest
require
minor
v0.43.13 -> v0.44.0

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.21.8 -> v1.21.10

Release Notes

gruntwork-io/terratest (github.com/gruntwork-io/terratest)
v0.44.0
Compare Source
Description

Updated minimal Go version to 1.21

Related links

https://github.com/gruntwork-io/terratest/pull/1339

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.21.10
Compare Source
Bug Fixes

deps: update module github.com/ibm/platform-services-go-sdk to v0.50.1 (#651) (df3e66d)

v1.21.9
Compare Source
Bug Fixes

deps: update gomod (#650) (1835de4)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

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

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

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


Fix symfony 3.x require version strings

I have seen this on others projects, (however this project use the same notation than symfony), what is the difference with the current implementation ?

Versions - Composer

The one you have is locked to 3.0 exactly. But I'm running 3.0.1 and this will not install for me. The caret allows it to match later versions as well. The link provided by @hxpro explains all the specifics.

Thanks.


How to Delete Slack Message?

I read the doc and I don't see any example on how to delete slack message. How to delete slack message? or the feature is not available?

Not available, why do you want it?

I want create a slack real-time status for my build where slack will send a message to be stating the build has started and when it completed, it will update the original message with mention. However, slack doesn’t notify author mention with edit message. I can edit the message and mention  in a thread but I don’t want to create a thread. So I thought of deleting the old message and create a new message of build completed with author mention. To me it’s a cleaner but might not be the best approach.

thread is probably the best way to go at least if you want to mention people


Add README to VTL resolvers directory

This is hopefully a simple DX idea.
I didn't know you could over-write generated resolvers as described here: https://docs.amplify.aws/cli/graphql-transformer/resolvers
I thought the /amplify/backend/api/<MyApp>/resolvers folder was just where all resolvers lived. I didn't know that they actually all get generated into /amplify/backend/api/<MyApp>/build/resolvers and any that appear in /amplify/backend/api/<MyApp>/resolvers are over-rides or custom resolvers that will replace the generated one.
This is actually really clever but the convention was hard for me to find, I had to have my build broken for 3 weeks to learn this bit (See https://github.com/aws-amplify/amplify-cli/issues/6315 to follow along at home).
Might I suggest that folders with this over-rides could be renamed as customResolvers rather than resolvers or something similar? I know that on the face of it this is a minor change but it will have knock on effects and it could be the genie is out of the bottle and its too late to change it. But that to me would have helped with my mental model of what that folder actually is for.
If we went a step forward could it be worth dropping a readme into these folders explaining what they are, when you might use them and links to the online docs? I only found this gem of a page today (https://docs.amplify.aws/cli/reference/files) and if nothing else it would be nice to add something in there about where to put custom code that over-rides the generated stuff.

@PeteDuncanson thanks for the suggestion. I fear that renaming that folder would be a big headache, but I think adding a readme to the folder describing the override process would be a good improvement.

@PeteDuncanson thanks for the suggestion. I fear that renaming that folder would be a big headache, but I think adding a readme to the folder describing the override process would be a good improvement.

Hi @edwardfoyle, I'm trying to make a PR which closes this issue. Could you please clarify whether the generated README file should be added to .gitignore?
Thanks.


fix: Expose type for custom request parameter

Introduces and exposes a type for parameter accepted by addCustomRequestConfiguration with the format below as per the SDK documentation:
type CustomRequestConfig = Omit<
  HttpRequestConfig,
  "url" | "baseURL" | "data" | "headers" | "params"
>;

Using HttpRequestConfig instead of axiosRequestConfig suggested by the user ensures that we are not tightly coupled with Axios.
Closes SAP/cloud-sdk-backlog#1187

Also found an inconsistency in our SDK documentation in the topic Setting a Custom Request Configuration, it is mentioned that the SDK doesn't allow method option to be overwritten, when in fact it is allowed (this is also present in a test case). @jjtang1985

Looking at the git history, support for the method option was specifically added here therefore it would also make sense for you to open a docs PR.


2023 09 newsletter

Hi,
I added two links to the paragraph about CoFest 2023. One of them is still a placeholder for a link to the blog post I'm preparing.
Best.

Thanks so much @jejust!


Upgrade the "Table view" to a "Sandwich" view

This provides information about the caller & callees of individual functions selected in the table view.
TODO before landing:

[ ] Make the hover effect provide tooltips

Pull Request Test Coverage Report for Build 149

39 of 156 (25.0%)  changed or added relevant lines in 8 files are covered.
4 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.7%) to 35.97%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

flamechart-minimap-view.tsx
0
1
0.0%

style.ts
0
2
0.0%

profile-table-view.tsx
0
5
0.0%

flamechart-renderer.ts
0
8
0.0%

application.tsx
0
13
0.0%

flamechart-view.tsx
0
18
0.0%

sandwich-view.tsx
0
70
0.0%

Files with Coverage Reduction
New Missed Lines
%

profile-table-view.tsx
1
0.0%

application.tsx
1
0.0%

flamechart-renderer.ts
1
0.0%

flamechart-view.tsx
1
0.0%

Totals

Change from base Build 141:
-0.7%

Covered Lines:
1044

Relevant Lines:
2753

💛  - Coveralls


Try to disable or block the editing anymore

Hi, I have one more question while testing.
I want to disable editing and wanna see current work as final,
so tried to block it with disableEdit() like :
        mind.toCenter();
        mind.disableEdit();

But I still can add new child and set new color for my nodes.
Is there anyway to block to edit anymore ?

it's a bug, I will fix it 🤣

Oh I see ! Hope to be fixed soon :)

fixed ; )

Thanks alot for your help !
So I've tested it with 0.16.3, but I can still add new child after setting disableEdit().
I don't knwo what's wrong..
Downloaded it from :
https://cdn.jsdelivr.net/npm/mind-elixir/dist/MindElixir.js

Ah, does the disableEdit() valid for only onkeydown and dbclick ?
How about items of contextmenu ? I tested with clicking a menu item of menu, "Add child".

🤣you are right, this is another fault.

0.16.4 fixed

Thanks 👍 , I will try again !


XAA + QTGMC = Crash

Describe the bug or Issue
If you add the AVS Filters QTGMC and XAA together, StaxRip will forever go unresponding and eventually crash.
Step To Reproduce the Problem

Load file
Add Field -> QTGMC (Placebo -> Progressive Full Repair)
Add Line -> XAA
It will not respond forever and eventually crash.

Screenshots
Freeze
Desktop (please complete the following information):

OS: Windows 10 Build 1803
Version 1.9 & 2.0
FrameServer Being Used Avisynth
Version of External Application / Filter Being Used Not Sure
List of Filters Being Used QTGMC and XAA

Additional context
It seems to only occur with the specific options I mentioned. Its quite odd.
On <IP_ADDRESS> XAA doesnt work at all, it says "There is no function named 'TryFTurnRight'."
EDIT: It doesnt occur if you do XAA first but still crashes if you then move XAA after QTGMC

It's likely due to filters & vector files your using which are not supported by your CPU, Most filters are AVX2 enabled by default since some of the updated versions have no AVX version anymore.

Dither
eedi2
eedi3
FTurn
MaskTools2
nnedi3
nnedi3ocl
RgTools
SangNom2
TCannyMod
TEMmod
ResizeX
edi_rpow2

It doesn't crash, if wrong filters are loaded, it just shoots out the error, the vectors filter is likely your problem.
Plus it doesn't help that AVS+ made FTurn functions useless and I believe it was renamed to LTurn instead of LeftTurn. XAA is pretty much dead. Not sure if I can get it to run in AVS+ anymore or not.
It was pure luck I had it running in the first place since I had to alter the code, I would suggest using VapourSynth(Your going to see allot more of can't run because x is not support due to vectors or other reasons. Allot of Avs filters are just not updated anymore.). Most of the AA scripts have been ported over to VS Plus some better ones like TAAmbk(not sure if XAA was ported over or not). I've been starting to use VS more, Before I only used I only used AVS. Plus VS has QTGMC as well.
Many of the AVS filters even running have problems, creating issues within the video. SVPFlow is one example I noticed Works perfectly in VS, but AVS version creates artifacts & odd jitters around text.


Add debug logging for dropped packets due to "full request buffer" and configurable buffer size.

I suggest to add logging here (via Logger) https://github.com/getsentry/sentry-go/blob/v0.0.1-beta/transport.go#L97 - otherwise there is no info about dropped packets when using debug mode. Dropped packets are just logged as being sent in https://github.com/getsentry/sentry-go/blob/v0.0.1-beta/transport.go#L85.
Shouldn't a sentry.Flush() call fail if there were some dropped packets? Currently it does not. I know this is no common use case, since usually non-blocking calls would be used and Flush might not hit the buffer size.
Furthermore it would be great to configure the request worker buffer size https://github.com/getsentry/sentry-go/blob/v0.0.1-beta/transport.go#L16
Thanks for your work! We currently evaluate Sentry & this Go client.

Logging fixed: https://github.com/getsentry/sentry-go/commit/e97ead2229dc3754c87ef9f6175d159e43488cf4
Buffer configuration added: https://github.com/getsentry/sentry-go/commit/6eeb2a3826f67452c960533c8a66fc0e971e35bc
As for a Flush question, we decided in all the SDKs, that allowing it to timeout if there are too many events queued is just fine and shouldn't cause any troubles.
Cheers!

Thanks for your quick response, configurable buffer size and the info about Flush. Just noting that queuing up e.g. 500 events in a simple for loop and immediately flushing them afterwards will not yield a timeout. The flush will return early in case of a low buffer size and just drop packages. If it timeouts in such a case it would be ok (I know, not a common use-case ;))
Sth like this:
for i := 0; i < 500; i++ {
	sentry.CaptureMessage("Message" + strconv.Itoa(i))
}

if sentry.Flush(time.Second * 2) {
	// not triggered
} else {
	fmt.Println("timeout")
}

If events get delivered in <2sec, then sure, it won't timeout :) However, we try to aim for the most common scenario, and leave enough room for configuration so that we can cover the rest of the cases in one way or another.
In this specific case, it's possible to do:
type customTransport struct {
  interrupted chan bool
  flushing bool
}

func (t *customTransport) Configure(options ClientOptions) {}

func (t *customTransport) SendEvent(event *Event) {
  if t.flushing == true && reachedBufferLimit (
    t.interrupted <- true
  )

  // send, add to wait group etc. etc.
}

func (t *customTransport) Flush(timeout time.Duration) bool {
  t.flushing = true
  c := make(chan struct{})

  go func() {
    t.wg.Wait()
    t.flushing = false
    close(c)
  }()

  select {
  case <-c:
    return true
  case <-time.After(timeout):
  case <-t.interrupted:
    return false
  }
}

In our case they weren't, we received 30 packets in Sentry and 470 were dropped ;) Thanks, the customTransport will do.

@christian-ehmig if you get so many events, you may also want to give SampleRate a try :)


Will there be an IDE?

Will this be shipped with a IDE like XCode or a visual studio (not vscode) plugin since setting up the environment in a text editor is complicated (especially when offline)?

No, there’s no plan to integrate an IDE into the installer.  If you’re looking for that, I’d suggest that you take a look at the docker images (https://hub.docker.com/r/compnerd/swift). Improvements to that are welcome.

I see. I'm not sure if AppCode will support more platforms as Swift supports more platforms officially. So are there actually a IDE (not text editor) for Swift on non-macos systems?

Right, thats what makes the docker image so interesting :)  It is a Linux environment, but, we can inject the SDKs for different targets.  It is a modified version of VSCode that runs entirely in the docker image, you just need Chrome to connect to it.  You might want to take a look at https://github.com/compnerd/swift-build/blob/master/docs/GettingStartedDocker.md for some brief introductory material.  If you have suggestions to improve that, Im definitely open to that :)

Okay, thanks.


icpc command not found, Installation and ussage step

Dear Team,
I have been trying to use this repository from mu Ubuntu system, when I run this command "make CXX=icpc", i get the error "make icpc command not found \n makefile:25: recipe for target 'verse-library' failed". I have looked for this error and tried multiple ways to resolve this issue, but some how nothing seems to work. It would be great if yo can provide me a way to resolve this issue, or to reach the binary file which is generated by these step, so that I can use that binary file in the python wrapper class and execute the code.
Kindly provide any help feedback on this.
Regards,
Sidhant

Hi,
icpc is Intel's c++ compiler.
https://software.intel.com/content/www/us/en/develop/tools/compilers/c-compilers.html
You can also use gcc, which will work just fine.

Hello,
I already have the c++ compilers (gcc and g++ both) installed on my system. Also I tried running the file example.sh (instead of running the three command given under "Installation and Usage"). This example.sh file runs successfully(i.e does not give any error), but it does not generate the '.bin' file anywhere. As per my understanding(correct me if i am wrong), after running the example.sh file, it should generate the binary file, which we can use with the python wrapper for generating the embedding.

No, you are not correct. wrapper.py should work after you run make in the src folder. You need to input the correct cpath parameter to the wrapper.


Possible valgrind issue ?

Converting from the C API data structure for which nanoarrow is so useful seems to tickle something from valgrind which may make CRAN unhappy.  (Though I am unsure at this moment if they act us to change things on possibly lost.  In any event, working on a related issue on my side I notice the 'residual' reported here (see below) that can be tickled via a simple variant around one your README example, namely a conversion to arrow:
R -q -d valgrind -e 'arrow::as_arrow_table(nanoarrow::as_nanoarrow_array(data.frame(col1= c(1.1,2.2))))'

Full logs below. This on Ubuntu 22.10 with all packages at current CRAN versions.  (nanoarrow may be from repo, I could check.)
I think this may be a false positive: I have seen similar 'possibly lost' before.
Or could it be related to how arrow is instantiated?  Other uses (as e.g. an export into a tibble) do not exhibit this (cf second example below).

$ R -q -d valgrind -e 'as.data.frame(nanoarrow::as_nanoarrow_array(data.frame(col1= c(1.1,2.2))))'
==2812420== Memcheck, a memory error detector
==2812420== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==2812420== Using Valgrind-3.18.1 and LibVEX; rerun with -h for copyright info
==2812420== Command: /usr/lib/R/bin/exec/R -q -e as.data.frame(nanoarrow::as_nanoarrow_array(data.frame(col1=~+~c(1.1,2.2))))
==2812420==
> as.data.frame(nanoarrow::as_nanoarrow_array(data.frame(col1= c(1.1,2.2))))
  col1
1  1.1
2  2.2
>
>
==2812420==
==2812420== HEAP SUMMARY:
==2812420==     in use at exit: 55,176,410 bytes in 12,118 blocks
==2812420==   total heap usage: 28,209 allocs, 16,091 frees, 83,130,127 bytes allocated
==2812420==
==2812420== LEAK SUMMARY:
==2812420==    definitely lost: 0 bytes in 0 blocks
==2812420==    indirectly lost: 0 bytes in 0 blocks
==2812420==      possibly lost: 0 bytes in 0 blocks
==2812420==    still reachable: 55,176,410 bytes in 12,118 blocks
==2812420==                       of which reachable via heuristic:
==2812420==                         newarray           : 4,264 bytes in 1 blocks
==2812420==         suppressed: 0 bytes in 0 blocks
==2812420== Reachable blocks (those to which a pointer was found) are not shown.
==2812420== To see them, rerun with: --leak-check=full --show-leak-kinds=all
==2812420==
==2812420== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)
edd@rob:~/git/rcppnanoarrow(master)$ R -q -d valgrind -e 'arrow::as_arrow_table(nanoarrow::as_nanoarrow_array(data.frame(col1= c(1.1,2.2))))'
==2817635== Memcheck, a memory error detector
==2817635== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==2817635== Using Valgrind-3.18.1 and LibVEX; rerun with -h for copyright info
==2817635== Command: /usr/lib/R/bin/exec/R -q -e arrow::as_arrow_table(nanoarrow::as_nanoarrow_array(data.frame(col1=~+~c(1.1,2.2))))
==2817635==
> arrow::as_arrow_table(nanoarrow::as_nanoarrow_array(data.frame(col1= c(1.1,2.2))))
Table
2 rows x 1 columns
$col1 <double>
>
>
==2817635==
==2817635== HEAP SUMMARY:
==2817635==     in use at exit: 66,601,761 bytes in 15,000 blocks
==2817635==   total heap usage: 90,088 allocs, 75,088 frees, 153,000,130 bytes allocated
==2817635==
==2817635== 416 bytes in 1 blocks are possibly lost in loss record 170 of 1,580
==2817635==    at 0x4849A83: calloc (in /usr/libexec/valgrind/vgpreload_memcheck-amd64-linux.so)
==2817635==    by 0x4012949: calloc (rtld-malloc.h:44)
==2817635==    by 0x4012949: allocate_dtv (dl-tls.c:375)
==2817635==    by 0x4012949: _dl_allocate_tls (dl-tls.c:634)
==2817635==    by 0x4D4BF99: allocate_stack (allocatestack.c:423)
==2817635==    by 0x4D4BF99: pthread_create@@GLIBC_2.34 (pthread_create.c:650)
==2817635==    by 0x14899D93: ??? (in /usr/local/lib/R/site-library/arrow/libs/arrow.so)
==2817635==    by 0x1489B01C: ??? (in /usr/local/lib/R/site-library/arrow/libs/arrow.so)
==2817635==    by 0x4004FBD: call_init.part.0 (dl-init.c:70)
==2817635==    by 0x40050A7: call_init (dl-init.c:33)
==2817635==    by 0x40050A7: _dl_init (dl-init.c:117)
==2817635==    by 0x4E287F3: _dl_catch_exception (dl-error-skeleton.c:182)
==2817635==    by 0x400BE95: dl_open_worker (dl-open.c:808)
==2817635==    by 0x400BE95: dl_open_worker (dl-open.c:771)
==2817635==    by 0x4E28799: _dl_catch_exception (dl-error-skeleton.c:208)
==2817635==    by 0x400C20B: _dl_open (dl-open.c:886)
==2817635==    by 0x4D46E6B: dlopen_doit (dlopen.c:56)
==2817635==
==2817635== 2,920 bytes in 1 blocks are possibly lost in loss record 279 of 1,580
==2817635==    at 0x4844899: malloc (in /usr/libexec/valgrind/vgpreload_memcheck-amd64-linux.so)
==2817635==    by 0x401207D: malloc (rtld-malloc.h:56)
==2817635==    by 0x401207D: allocate_dtv_entry (dl-tls.c:684)
==2817635==    by 0x401207D: allocate_and_init (dl-tls.c:709)
==2817635==    by 0x401207D: tls_get_addr_tail (dl-tls.c:907)
==2817635==    by 0x4015D6B: __tls_get_addr (tls_get_addr.S:55)
==2817635==    by 0x14899F93: ??? (in /usr/local/lib/R/site-library/arrow/libs/arrow.so)
==2817635==    by 0x4D4B401: start_thread (pthread_create.c:442)
==2817635==    by 0x4DD9743: clone (clone.S:100)
==2817635==
==2817635==
==2817635== LEAK SUMMARY:
==2817635==    definitely lost: 0 bytes in 0 blocks
==2817635==    indirectly lost: 0 bytes in 0 blocks
==2817635==      possibly lost: 3,336 bytes in 2 blocks
==2817635==    still reachable: 66,598,425 bytes in 14,998 blocks
==2817635==                       of which reachable via heuristic:
==2817635==                         newarray           : 4,264 bytes in 1 blocks
==2817635==         suppressed: 0 bytes in 0 blocks
==2817635== Reachable blocks (those to which a pointer was found) are not shown.
==2817635== To see them, rerun with: --leak-check=full --show-leak-kinds=all
==2817635==
==2817635== ERROR SUMMARY: 2 errors from 2 contexts (suppressed: 0 from 0)
$

$ R -q -d valgrind -e 'tibble::as_tibble(nanoarrow::as_nanoarrow_array(data.frame(col1= c(1.1,2.2))))'
==2851061== Memcheck, a memory error detector
==2851061== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==2851061== Using Valgrind-3.18.1 and LibVEX; rerun with -h for copyright info
==2851061== Command: /usr/lib/R/bin/exec/R -q -e tibble::as_tibble(nanoarrow::as_nanoarrow_array(data.frame(col1=~+~c(1.1,2.2))))
==2851061==
> tibble::as_tibble(nanoarrow::as_nanoarrow_array(data.frame(col1= c(1.1,2.2))))
# A tibble: 2 × 1
   col1
  <dbl>
1   1.1
2   2.2
>
>
==2851061==
==2851061== HEAP SUMMARY:
==2851061==     in use at exit: 61,673,573 bytes in 13,974 blocks
==2851061==   total heap usage: 60,305 allocs, 46,331 frees, 134,526,038 bytes allocated
==2851061==
==2851061== LEAK SUMMARY:
==2851061==    definitely lost: 0 bytes in 0 blocks
==2851061==    indirectly lost: 0 bytes in 0 blocks
==2851061==      possibly lost: 0 bytes in 0 blocks
==2851061==    still reachable: 61,673,573 bytes in 13,974 blocks
==2851061==                       of which reachable via heuristic:
==2851061==                         newarray           : 4,264 bytes in 1 blocks
==2851061==         suppressed: 0 bytes in 0 blocks
==2851061== Reachable blocks (those to which a pointer was found) are not shown.
==2851061== To see them, rerun with: --leak-check=full --show-leak-kinds=all
==2851061==
==2851061== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)
$

Thanks for the very prompt reply!  I had seen the .supp on your repo, but never perused documentation.  I am mostly fearful of getting my wrist slapped by CRAN if I use the similar pattern on nanoarrow/arrow conversion.  Any thoughts?   Have you had any issues with your packages?
(I had a real issue where I wasn't relasing a small subcomponent of a larger data structure so I am remain grateful for what valrgrind does but I also had some headscrathers.  Again, thanks for the quick follow-up!)

Hm, with
    ARROW_DEFAULT_MEMORY_POOL=system R -q -d valgrind \
            -e 'arrow::as_arrow_table(nanoarrow::as_nanoarrow_array(data.frame(col1= c(1.1,2.2))))'

I get the same result as above.  Using it from my checkout of your repo and pointing to valgrind.supp one of the two disappears:
$ R -q -d valgrind --debugger-args="--suppressions=valgrind.supp" -e 'arrow::as_arrow_table(nanoarrow::as_nanoarrow_array(data.frame(col1= c(1.1,2.2))))'
==3525498== Memcheck, a memory error detector
==3525498== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==3525498== Using Valgrind-3.18.1 and LibVEX; rerun with -h for copyright info
==3525498== Command: /usr/lib/R/bin/exec/R -q -e arrow::as_arrow_table(nanoarrow::as_nanoarrow_array(data.frame(col1=~+~c(1.1,2.2))))
==3525498==
> arrow::as_arrow_table(nanoarrow::as_nanoarrow_array(data.frame(col1= c(1.1,2.2))))
Table
2 rows x 1 columns
$col1 <double>
>
>
==3525498==
==3525498== HEAP SUMMARY:
==3525498==     in use at exit: 66,601,762 bytes in 15,000 blocks
==3525498==   total heap usage: 90,088 allocs, 75,088 frees, 153,000,132 bytes allocated
==3525498==
==3525498== 2,920 bytes in 1 blocks are possibly lost in loss record 279 of 1,580
==3525498==    at 0x4844899: malloc (in /usr/libexec/valgrind/vgpreload_memcheck-amd64-linux.so)
==3525498==    by 0x401207D: malloc (rtld-malloc.h:56)
==3525498==    by 0x401207D: allocate_dtv_entry (dl-tls.c:684)
==3525498==    by 0x401207D: allocate_and_init (dl-tls.c:709)
==3525498==    by 0x401207D: tls_get_addr_tail (dl-tls.c:907)
==3525498==    by 0x4015D6B: __tls_get_addr (tls_get_addr.S:55)
==3525498==    by 0x14899F93: ??? (in /usr/local/lib/R/site-library/arrow/libs/arrow.so)
==3525498==    by 0x4D4B401: start_thread (pthread_create.c:442)
==3525498==    by 0x4DD9743: clone (clone.S:100)
==3525498==
==3525498== LEAK SUMMARY:
==3525498==    definitely lost: 0 bytes in 0 blocks
==3525498==    indirectly lost: 0 bytes in 0 blocks
==3525498==      possibly lost: 2,920 bytes in 1 blocks
==3525498==    still reachable: 66,598,426 bytes in 14,998 blocks
==3525498==                       of which reachable via heuristic:
==3525498==                         newarray           : 4,264 bytes in 1 blocks
==3525498==         suppressed: 416 bytes in 1 blocks
==3525498== Reachable blocks (those to which a pointer was found) are not shown.
==3525498== To see them, rerun with: --leak-check=full --show-leak-kinds=all
==3525498==
==3525498== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 1 from 1)
--3525498--
--3525498-- used_suppression:      1 <jemalloc>:Thread locals don't appear to be freed valgrind.supp:19 suppressed: 416 bytes in 1 blocks
==3525498==
==3525498== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 1 from 1)
$


New KS page

This mod got removed from KerbalStuff, causing a 404, and then got back added to KerbalStuff but with a new page.
This happened also withAircraftIVAHelmet and Area21SCraft, but didn't got back added to KerbalStuff.

Needs AircraftIVAHelmet but no alternative download found. Hopefully LuizChacon will reupload again

For documentation, the unavailability of AircraftIVAHelmet breaks A6Intruder A7Corsair2 AV8HarrierWITHTRUEVTOLFlight EF2000EuroFighter AircraftIVAHelmet F14CTomcat F15DCStrikeEagle F18ASuperHornet F22Lightning2OrRaptor F35BSTOLStealthMultiroleFighter F5ATalonFighterAggressor FairchildRepublicA10Warthog MIG29Fulcrum SukhoiSU27Flanker and F16FightingFalcon.
As this PR doesn't pass testing due to broken dependencies (and I know first hand how slow the author is to resolve problems) I would suggest creating an issue about it and not a PR. If you'd like, feel free to add it to  #1860 and edit the post for context.
@Dazpoet the LuizChacon saga continues.

Closing as I added the mods who require the dependancy to the post.


feat(interp,exts): implement stack trace

Summary
This implements an extension _brs_.getStackTrace, which, as you might expect, gets the stack trace.
Also, I cleaned up the README a little -- reorganized the extensions alphabetically.

Great work @lkipke ! This is a very clean implementation, and is well documented as well. Now that component libraries landed (#590), would you mind checking _brs_.getStackTrace() from inside one of those? Everything should be fine, but it seems worth checking before this merges 😃

Yep, it all looks good!
      Array [
        "[Foo:Bar::init]",
    +   "<Component: roArray> =
    + [
    +     /Users/levi.kipke/Projects/brs/test/e2e/resources/component-library/source/main.brs:2:4
    +     /var/folders/sr/j_zsg28s4tqgc3g0kw74hs1rbz38fw/T/brs-pkgcomponentslibrary.zip2bz65m/components/Bar.brs:3:4
    +     /var/folders/sr/j_zsg28s4tqgc3g0kw74hs1rbz38fw/T/brs-pkgcomponentslibrary.zip2bz65m/components/Bar.brs:3:10
    +     /var/folders/sr/j_zsg28s4tqgc3g0kw74hs1rbz38fw/T/brs-pkgcomponentslibrary.zip2bz65m/source/util.brs:2:4
    +     /var/folders/sr/j_zsg28s4tqgc3g0kw74hs1rbz38fw/T/brs-pkgcomponentslibrary.zip2bz65m/source/util.brs:2:10
    + ]",
        "[Foo:util::someUtil]",


Created Feature Card 27 white #1691

Closes #1691
Here is the component I have built.
Here is a screenshot of that.
 Screen Shot

Done Now.


[CQT-252] Integrate libQASM 0.6.8 into OpenSquirrel

Update libqasm version to 0.6.8.
Gate modifiers:

Take a Callable that returns a BlochSphereRotation.
Invoke the Callable, together with a list of arguments, to obtain a BlochSphereRotation.
Then return either a new BlochSphereRotation with a new angle (inv and pow), or a ControlledGate.
Notice the list of arguments *args received by call contains both the qubit operands and an optional parameter.

Fix tests to use the correct syntax for parametrized gates.
default_gate_modifiers.py:

Implement inv, pow, and ctrl gate modifiers.

ir.py:

Allow Float instances to be constructed from another Float. The same for Int.
Fix comment typos.
Make Float and Int classes look alike.
Add IR.add_statement.

instruction_library.py:

Fix error message.

parser.py:

Update to parse the new AST: GateInstruction's, NonGateInstruction's, gate modifiers, and parametrized gates.

test_libqasm.py:

Add test_gate_modifiers.

This PR fixes CQT-209 as well.


Installation instructions don't work

Hi,
Thanks for the great gem!
I'm following readme, trying to install the gem. As my project is fairly new, I didn't have activeadmin setup before. With that said, I also didn't have devise gem installed. That was the first failure after installing activeadmin-mongoid. The second issue now is from the readme, saying to require "active_resource/railtie", while I didn't install activeresource gem neither.
I think a better approach would be to have this mentioned in the readme, instead of gem users going back and forth between missing dependencies error messaged and installing those dependencies.
EDIT: It would also make it clearer whether gems like devise and/or activeresource are required or optional.
Thanks! ❤

I'm stuck on
/Users/vfonic/.gem/ruby/2.5.0/gems/bundler-1.16.4/lib/bundler/runtime.rb:84:in `rescue in block (2 levels) in require': There was an error while trying to load the gem 'activeadmin-mongoid'.
Gem Load Error is: uninitialized constant ActiveRecord

That points to:
/Users/vfonic/myapp/config/application.rb:11:in `<top (required)>'

Which is this line:
require_relative 'boot'

require 'action_controller/railtie'
require 'action_mailer/railtie'
require 'active_resource/railtie'
require 'sprockets/railtie'
require 'rails/test_unit/railtie'

# Require the gems listed in Gemfile, including any gems
# you've limited to :test, :development, or :production.
Bundler.require(*Rails.groups) ########################################## <= THIS LINE

module Civiclift
  class Application < Rails::Application
    # Initialize configuration defaults for originally generated Rails version.
    config.load_defaults 5.1

    # Settings in config/environments/* take precedence over those specified here.
    # Application configuration can go into files in config/initializers
    # -- all .rb files in that directory are automatically loaded after loading
    # the framework and any gems in your application.

    # Don't generate system test files.
    config.generators.system_tests = nil
  end
end

Here's my Gemfile:
source 'https://rubygems.org'

ruby '2.5.1'

git_source(:github) do |repo_name|
  repo_name = "#{repo_name}/#{repo_name}" unless repo_name.include?("/")
  "https://github.com/#{repo_name}.git"
end

gem 'rails'
gem 'puma'
gem 'sass-rails'
gem 'uglifier'
# gem 'bootsnap', require: false
gem 'jbuilder'

group :development, :test do
  gem 'byebug', platforms: [:mri, :mingw, :x64_mingw]
  gem 'rspec-rails'
end

group :development do
  gem 'web-console'
  gem 'listen'
  gem 'spring'
  gem 'spring-watcher-listen'
end

gem 'pry-rails'
gem 'awesome_print'

gem 'tzinfo-data', platforms: [:mingw, :mswin, :x64_mingw, :jruby]

gem 'mini_racer', platforms: :ruby
gem 'rack-timeout'

gem 'pg'

gem 'figaro'

# gem 'webpacker'
# gem 'react_on_rails', '11.0.7'

gem 'activeadmin-mongoid', '~> 0.5.0'
gem 'activeresource'
gem 'devise'

gem 'http'
gem 'addressable'
gem 'rollbar'
gem 'aws-sdk-s3'
gem 'cloudinary'

Am I doing something wrong? How can I ensure ActiveRecord is not being loaded?

@vfonic why do you need to use activeadmin-mongoid with postgres? activeadmin-mongoid is the hack to activeadmin, which helps with integration activeadmin and mongoid orm, but in your gemfile i see only pg and no mongoid

@vfonic and if you just switch to plain activeadmin and replace back
require "action_controller/railtie"
require "action_mailer/railtie"
require "active_resource/railtie"
require "sprockets/railtie"
require "rails/test_unit/railtie"

to
require 'rails/all',
you'll can fix this problem :-)

Ah, you're right!
Well, I was (about to start) using ActiveRecord in my app, but then I also needed mongo and if I could make it work with ActiveAdmin, that would be magical and I'd drop pg/ActiveRecord support. I'll remove 'pg' gem and add 'mongoid' and let you know if that fixes the issue.

@vfonic note that some features of activeadmin (such as built-in filters, you'll need to implement by yourself and with database indexing) not implemented here for mongoid due to features of mongodb, so if you are limited to choice between pg and mongo just in this, better pick pg and main gem for that needs

Thanks @xfynx! I have some data in mongo that I need to access (and would be ideal to be able to get to that data in an ActiveAdmin-like interface).
It still doesn't work. So I took a step back, and created a new rails app:
rails _5.1.6_ new active_admin_mongoid --skip-coffee --skip-turbolinks --skip-test --skip-system-test --skip-active-record
cd active_admin_mongoid
echo "gem 'activeadmin-mongoid', '~> 0.5.0'" >> Gemfile # mongoid 6.0.3 is installed as dependency
bundle update
rails console

/Users/vfonic/.gem/ruby/2.5.0/gems/bundler-1.16.4/lib/bundler/runtime.rb:84:in `rescue in block (2 levels) in require': There was an error while trying to load the gem 'activeadmin-mongoid'.
Gem Load Error is: uninitialized constant ActiveRecord
Did you mean?  ActiveModel

:/ Should I try with lower rails version?

@vfonic
oh, i got it.
you'll need to add gem 'ransack', '<=1.8.6' after line gem 'activeadmin-mongoid'.
i'll add it to readme later with PR. newer version of ransack causes your error. and probably you'll get error about devise - just add it too and app will start work.
There are still troubles around mongoid and activeadmin :-)

It worked! Thanks so much @xfynx! I'll try add actually add something to AA now and will let you know whether it works for actually loading data or just the rails server starts. Then I'll close the issue.

Hey @xfynx,
After some playing around, I figured I also need gem 'kaminari-mongoid' for the index pages.
Thanks for the help setting it all up! It works now.


support 1, 2 and 3 clusters for HA deployment

By default mkcloud required 9 nodes for an HA setup as it only supported 3 clusters (data min. 2 nodes, services min. 3 nodes, network min. 3 nodes) plus one compute node as minimum.
This feature now supports the deployment of an HA setup on 1, 2 or 3 clusters.
To configure the scenario use the variable "clusterconfig".
3 clusters: clusterconfig="data=2:network=3:services=3"
2 clusters: clusterconfig="services+data=2:network=3"
1 cluster:  clusterconfig="network+services+data=2"

The first one of each group (combined via +) defines the name of the cluster. The services from the grouped clusters are merged into the first one. A Group defines its node number via "=2" at the end. Groups can be separated via ":".

Woohoo!  Great - looking forward to reviewing this and getting the functionality merged.
BTW this is a good example of why shell is too limited a language for this complex CI code: we are forced to squash the multi-dimensional cluster config into a single scalar $clusterconfig string.  If we were coding in Ruby or Python we could easily use a Hash or dict or even YAML, which would be much nicer.  (Yes I know bash has associative arrays, but they are really horrible.)


The slave of the kvrocks cluster failed to synchronize data from the master

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
kvrocks 2.8 ，kvrocks-controller unstable，kvrocks编译是采用了-DPORTABLE=1
Minimal reproduce step
在运行了kvcontroller的情况下，使用api创建集群 /api/v1/namespaces/test-ns/clusters
{
"name":"demo-kvcluster",
"nodes":["<IP_ADDRESS>:16660","<IP_ADDRESS>:16661","<IP_ADDRESS>:16662","<IP_ADDRESS>:16663","<IP_ADDRESS>:16664","<IP_ADDRESS>:16665"],
"replicas":2,
"password":"xxx#$()D-"
}
What did you expect to see?
在master节点上面set ld<PHONE_NUMBER>111111，在slave上面可以get到
What did you see instead?

slave上未能get 到 ld
Anything Else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

@QQxiaoyuyu Could you provide the nodes.conf of <IP_ADDRESS>:16663?

The reson is master-use-repl-port = yes, cluster must config no


Not able to fetch server.jks file from other than classpath

Question
Hi
For Encryption, I have created Spring cloud config server keystore.jks file and I have stored it at ${user.home}/  location but when I give encrypt.keystore.location=${user.home}/keystore.jks it tells me an error saying :
class path resource [home//keystore.jks] cannot be opened because it does not exist.
Can't I store keystore at any location other than classpath, because I have to pass keystore file at runtime instead of packaging it in jar, because my jar will be public.

This sounds similar to #1176.  I am going to close it as a duplicate.  Please let me know if you think this is a different issue.

Hi @ryanjbaxter
My issue is different than the one you mentioned. The problem is that I dont want to package my Java key store with jar file, I want to pick it up from file system location other than classpath. My spring cloud config boot application will be one of my deliverable, but the jks keystore should be generated and used by the end customer.(intended to use the jar to encrypt files). I sthere any way to keep the jks file outside jar and give file system location. I hope I made myself clear. Let me know if you need any clarifications regarding this.


Allow a button to be associated with more than one group

I should be able to specify a button with more than one group.

Fixed in mainline


Unhandled promise rejection causes all test running concurrently to fail

Are you requesting a feature or reporting a bug?
Bug
What is the current behavior?
Running test with -c 5 with chained assertions causes all test that are currently running to fail if one test fails that has chain assertions.
   1) Unhandled promise rejection:

      [object Object]

What is the expected behavior?
Only the test that failed will fail and other tests that are concurrently running aren't impacted.  Probably shouldn't throw a Unhandled promise rejection either.
This causes a lot of false positives.
How would you reproduce the current behavior (if this is a bug)?
I will try to get you guys a repro as I can't share what I have right now.
Here are the steps where I see the issue.

Run the test concurrently on saucelabs or locally with -c 5
Test have assertions chained at the end of each test.

Example of where I see the failure:
         289 |    .typeText(PageModel.titleInput, title, { paste:
      true })
         290 |    .click(PageModel.buttonTextInput)
         291 |    .typeText(PageModel.accountingCodeInput, number)
         292 |    .click(PageModel.saveButton)
         293 |    .expect(PageModel.successBar.count)
       > 294 |    .eql(1)
         295 |    .expect(PageModel.accountingCodeInput.value)
         296 |    .eql(number);
         297 |});
         298 |

   2) Unhandled promise rejection:

      [object Object]

      Browser: Chrome 68.0.3440 / Windows 10.0.0

When one test fails with a Unhandled promise rejection all 5 test will fail with Unhandled promise rejection and the next test will start to execute.
NOTE: if each of the assertions are broken up this doesn't occur and works fine.
Example: instead of
 await t
.expect(someElement.value).eql(2)
.expect(someElement.value).eql(somevalue)
.expect(someElement.value).eql(somevalue)

change to
 await t.expect(someElement.value).eql(2)
 await t.expect(someElement.value).eql(somevalue)
 await t.expect(someElement.value).eql(somevalue)

No Unhandled promise rejection when this fails.
Specify your

operating system: Windows 10 and Mac on saucelabs.  Doesn't matter which browser either.
testcafe version:0.22.0
node.js version: v9.2.1

this is probably a dup of https://github.com/DevExpress/testcafe/issues/2852 i will close this for now and see if that change fixes the issue!


add breadcrumb line

https://user-images.githubusercontent.com/35755741/224184059-63ade663-4b9e-49f8-9b50-77cfb2df53de.mov

Frontend and Release Workflow Started here


Correct type imports

Now, when i import some type, code for it looks like import { TelegramUser } from 'nestjs-telegram/dist/interfaces/telegramTypes.interface';, seems direct link to dist folder is not correct approach.
May be should change it to something like import { TelegramUser } from 'nestjs-telegram'?

Fixed in v1.1.3


Update README.md

While executing the scripts in build.sh with just sudo -i, I having to reset the $OOS_SOURCE_DIR over and over. Seems like switching the user to root would fix the issue.

Declined. Reason: su -i requires root password. Most people won't have the root password and you usually don't have it for cloud machines.


Add Expander sample

Description

Add Expander control sample. More info about this feature can be found in the spec.

Motivation and Context

How Has This Been Tested?

Manually tested by me using mouse and keyboard.
Types of changes

[ ] 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)

The control template is gonna need the border hidden for the Expander header.  I think this is because of the changes made to the Button control, and the Expander's Lightweight Styles not affecting the border colours and thickness

The control template is gonna need the border hidden for the Expander header.  I think this is because of the changes made to the Button control, and the Expander's Lightweight Styles not affecting the border colours and thickness


Getting lint errors in a file I've deleted

Looks like esprint is caching something it shouldn't be.
Omers-MacBook-Pro:sigma-monorepo-2point0 omer$ npm run lint

><EMAIL_ADDRESS>lint /Users/omer/code/sigma-monorepo-2point0
> esprint

/Users/omer/code/sigma-monorepo-2point0/src/react/webapp/src/mutations/index.js
  7:1  error  More than 1 blank line not allowed  no-multiple-empty-lines

✖ 1 problem (1 error, 0 warnings)

npm ERR! Darwin 16.6.0
npm ERR! argv "/usr/local/Cellar/node/7.9.0/bin/node" "/usr/local/bin/npm" "run" "lint"
npm ERR! node v7.9.0
npm ERR! npm  v4.2.0
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>lint: `esprint`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>lint script 'esprint'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the sigma-monorepo package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     esprint
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs sigma-monorepo
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls sigma-monorepo
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/omer/.npm/_logs/2017-07-07T05_30_17_507Z-debug.log
Omers-MacBook-Pro:sigma-monorepo-2point0 omer$ cat /Users/omer/code/sigma-monorepo-2point0/src/react/webapp/src/mutations/index.js
cat: /Users/omer/code/sigma-monorepo-2point0/src/react/webapp/src/mutations/index.js: No such file or directory

Hey @omerzach ! Thanks so much for the bug report. I've gone ahead and put up a quick patch for this in #64 .


[SMALLFIX]Supply the name of variable to Preconditions.checkNotNull

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2749/
Test FAILed.

@161220023 please merge master to resolve merge conflicts


fix:Loading Spinner is taking whole screen and is also not suiting with the theme

Hi Aayushai
This pull request includes the following change:

Loading spinner is properly positioned
While loading, the spinner is centrally positioned, taking up only its own width and height. The background UI remains visible with a subtle blur effect

Please review the changes and provide feedback.

Hi @syamjir , This issue is assigned to me.
You have to understand , that's not how the open source community work! this is not the standard.

Hi @syamjir , the issue was assigned to @AnshMehta24 and it has been merged also, whenever you want to resolve an issue just ask to assign you for that issue.

Hi @aayushai ,
Thank you for the update! I appreciate the information regarding the assignment and merge of the issue. In the future, I’ll make sure to request assignments for any issues I want to resolve.
Best,
syamjir


Found it

Had lost track of this repository but I just found it.
Surprisingly helpful service.
Thanks!

@mamigot I’m glad this is working out for you 🧐
Sent with GitHawk

🐦


Integrate Firebase Firestore for Real-time Game Data Sync

Description:
Utilize Firebase Firestore to store and retrieve game data such as guest profiles, ingredients, and dishes in real-time, ensuring that the game's content is dynamically updated.
Acceptance Criteria:

Set up Firestore database with collections for guests, ingredients, and dishes.
Implement data retrieval in the game to fetch this information dynamically.
Ensure changes in the database reflect immediately in the game.

Update: created ingredients collection in Firebase. Game retrieves ingredients at the start of game (dynamically ?). Changes in database are reflected in game (need to check if immediately)

Update: Guests are displayed on a new page once player clicks 'person' guest icon. Removed original top row guest profile scrollable section for cleaner look. Dishes appear to be fetched. No icons at the moment for dishes. Considering if worth it to include or not


cigar string

Hi Arne and Markus,
Thank you for the tool and I really enjoy the speed of MA.
I have a pipeline which relays one the CIGAR string of the sam file. The cigar string of MA output is like "2=3X2=1X1=4D5=1X7=3X4=1X5=1X6=1X1=4D2=3D10=3D2=3I1=1D5=2X4=2I" and it's not compatible with my current pipeline. I wonder if there is a way or instruction to convert the CIGAR string to the standard format like minimap2?
Thank you,
Gorliver

Hello Gorliver,
I assume the problem is the ambiguity in the SAM-format with respect to the M-operation and the “=,X”-operations (which allows distinguishing between match and mismatch in contrast to the M-operation). By default, MA writes SAM-files using the “=,X”-operations. We added a new option Use_M_in_CIGAR to MA for forcing M-operation based SAM-output. Therefore, please clone the newest version of MA and add ‘--Use_M_in_CIGAR true’ to your command line. If you still have problems with your SAM-files, please let me know.
Best wishes,
Markus

Hi Markus,
Thanks for the option!


@jsx-email/preview is a dependency, but missing on npmjs

Expected Behavior
Being able to install the "all" package to check if it can be a drop-in replacement of @react-email/components.
Actual Behavior
$ npm i @jsx-email/all                                                                                 ✔  37s 
npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/@jsx-email%2fpreview - Not found
npm ERR! 404
npm ERR! 404  '<EMAIL_ADDRESS>is not in this registry.
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

Thanks for the heads up, should be resolved now.


ISPN-5956 Wait for the test cache to rebalance

https://issues.jboss.org/browse/ISPN-5956

unused import org.infinispan.test.TestingUtil; in almost all of the changed files

I removed the imports.

looks good. pushing...

integrated! thanks @danberindei !


Spread CMS Design System across app

We have two different USWDS implementations in the app right now, which is both silly and probably the cause of some quirky display bugs. We should migrate everything to the CMS Design System.
Pages to update
TBD
Components to remove or reimplement

[ ] Alert
[ ] Anchor
[ ] Button
[ ] Dropdown
[ ] Expire
[ ] Footer
[ ] Header
[ ] LabeledText
[ ] ProtectedRoute
[ ] RadioGroup
[ ] TextInput
[ ] USAGovBanner

Won't do: we reverted to custom components.


chore: update asyncpi bot github actions

This is chore to update the CODEOWNERS file as per the request https://github.com/asyncapi/community/issues/275

Please @anandsunderraman, open this PR against asyncapi/go-watermill-template instead of your fork. Also, check the changes because you introduced an extra @ character!

@smoya please look at https://github.com/asyncapi/go-watermill-template/pull/135


Failed to install python modules using pip install base on python:2.7.15-alpine3.8

Hello, I want to install some python modules base on python:2.7.15-alpine3.8 but always failed.
Here is my Dockerfile :
FROM python:2.7.15-alpine3.8
# install python modules
COPY requirements.txt /build/RUN apk add --update curl gcc g++ \
    && rm -rf /var/cache/apk/* \
&& pip install --upgrade pip \
&& pip install setuptools --upgrade \
&& pip install -r /build/requirements.txt

Here is my requirements.txt :
scipy
pathlib
python-docx
pandas
mysql-connector-python==8.0.12
xlrd

There is too much warning or error information, I will post some of them.
      File "/usr/local/lib/python2.7/site-packages/numpy/distutils/misc_util.py", line 943,
in _get_configuration_from_setup_py        config = setup_module.configuration(*args)
      File "scipy/linalg/setup.py", line 19, in configuration
        raise NotFoundError('no lapack/blas resources found')
    numpy.distutils.system_info.NotFoundError: no lapack/blas resources found

    ----------------------------------------
Command "/usr/local/bin/python -u -c "import setuptools, tokenize;__file__='/tmp/pip-install
-_nrjIx/scipy/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record /tmp/pip-record-nYcbQS/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /tmp/pip-install-_nrjIx/scipy/

   #include "libxml/xmlversion.h"
                                 ^  compilation terminated.
  Compile failed: command 'gcc' failed with exit status 1
  creating tmp
  cc -I/usr/include/libxml2 -c /tmp/xmlXPathInitKognPA.c -o tmp/xmlXPathInitKognPA.o
  /tmp/xmlXPathInitKognPA.c:1:26: fatal error: libxml/xpath.h: No such file or directory
   #include "libxml/xpath.h"
                            ^
  compilation terminated.
  *********************************************************************************
  Could not find function xmlCheckVersion in library libxml2. Is libxml2 installed?
  *********************************************************************************
  error: command 'gcc' failed with exit status 1

  ----------------------------------------
  Running setup.py clean for lxml
  Failed building wheel for lxml

  pandas/_libs/testing.c: In function 'inittesting':
    ^~~~~~~
  pandas/_libs/internals.c: In function '__pyx_gb_6pandas_5_libs_9internals_12generator':  pandas/_libs/internals.c:9336:36: warning: '__pyx_t_23' may be used uninitialized in this
function [-Wmaybe-uninitialized]           __pyx_cur_scope->__pyx_t_9 = __pyx_t_23;
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~  pandas/_libs/internals.c:9333:36: warning: '__pyx_t_11' may be used uninitialized in this
function [-Wmaybe-uninitialized]           __pyx_cur_scope->__pyx_t_5 = __pyx_t_11;
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~  pandas/_libs/internals.c:9330:36: warning: '__pyx_t_6' may be used uninitialized in this f
unction [-Wmaybe-uninitialized]           __pyx_cur_scope->__pyx_t_4 = __pyx_t_6;
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~  pandas/_libs/internals.c:9335:36: warning: '__pyx_t_15' may be used uninitialized in this
function [-Wmaybe-uninitialized]           __pyx_cur_scope->__pyx_t_8 = __pyx_t_15;
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~  pandas/_libs/internals.c:9334:36: warning: '__pyx_t_12' may be used uninitialized in this function [-Wmaybe-uninitialized]           __pyx_cur_scope->__pyx_t_7 = __pyx_t_12;           ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~

I hope somebody knows why and how to fix. Thanks.

see https://github.com/docker-library/python/issues/318 .

Alpine Linux doesn't support Python PEP513. So when you use Alpine Linux based image to install some packages, it needs to compile. (you must install gcc and other packages for this.)

Alpine Linux doesn't support Python PEP513. So when you use Alpine Linux based image to install some packages, it needs to compile. (you must install gcc and other packages for this.)

Thanks for you reply. I go see #318 and install gcc, still don't work. I want to know what's other packages, where can I know the list? I can pip install from python:2-stretch.

I can't figure out why Alpine won't at least install those packages from source. I would highly recommend in your use case to just use Debian. The packages you're installing probably eclipse the size benefit of using Alpine and it seems you have to install most if not all of those packages from source on Alpine which currently exhibit numerous issues preventing their installation, whereas Debian can use manylinux wheels that install in literally 10 seconds. I had thought that Alpine can use none wheels but they were not supported wheel on this platform when I specified them.
Also if that's your complete Dockerfile you'll probably want to remove gcc at the end of the RUN line as it's 76MB which is 17MB bigger than the image, also g++ as well which is 64MB.
If you want to continue troubleshooting this perhaps https://serverfault.com/questions/771211/docker-alpine-and-matplotlib is relevant to the compiling issues. I can't figure out why the packages won't build from source, perhaps it's dependencies? Regardless there's nothing we could change in the image to remedy this. You could try asking the Docker Community Forums, the Docker Community Slack, or Stack Overflow.
See also https://github.com/docker-library/docs/issues/904 and https://github.com/docker-library/python/issues/341#issuecomment-426044842 which is odd as I couldn't get the 2.7 none wheels to install when specifying their URL, and pip by default chose the .tar.gz


Event information

GitHubAction event information

I don't really see the use-case for adding a few dozen functions instead of accessing the dict directly. Could you explain what problem those solve?

Having functions for accessing properties is very common for those who don't know the actual architecture of the object. That is the case for CI environment. People don't have a playground access like how they do offline. It also makes it easier for people just to use a function instead of getting familiar with that architecture.

Having functions for accessing properties is very common for those who don't know the actual architecture of the object. That is the case for CI environment. People don't have a playground access like how they do offline.

Those are documented and examples are provided for every type of event: https://developer.github.com/webhooks/event-payloads/

Having functions for accessing properties is very common for those who don't know the actual architecture of the object. That is the case for CI environment. People don't have a playground access like how they do offline.

Those are documented and examples are provided for every type of event: https://developer.github.com/webhooks/event-payloads/

A lot of the properties given in the functions are missing. For example, see iscreated.
As I said it makes it possible to change the internal implementation. For example, do we want to have a Dict or a Struct. In Julia, that is important because different types need different accessing (. vs []).

I don't really see the use-case for adding a few dozen functions instead of accessing the dict directly

This is my general opinion too

I don't have strong opinions here. If you feel like that, it is OK with me. Hiding the data behind functions is a very common technique that is used both to make it easy to access data and also to stay backward-compatible. Breaking the package interface, just to change how we want to implement/handle, is not always the smoothest way. I want to prevent the breakage headache unlike how it happened for example in PkgTemplates.jl.

I've decided not to merge this, I think it's probably best for people looking for this kind of info to use the GitHub Actions documentation to find what they need for their particular use case, rather than us try to keep every use case up to date here.


enableTableSelectCallbacks

Thanks for this. Quick question as the didSelectRowAtIndexPath is not firing, where is the enableTableSelectCallbacks set that is mentioned?

ignore that I've just found reference to a previous issue regarding this same thing.
using the following works.
-(void)table:(WKInterfaceTable *)table didSelectRowAtIndex:(NSInteger)rowIndex {
NSIndexPath *indexPath = [table indexPathFromRowIndex:rowIndex];
[self table:table didSelectRowAtIndexPath:indexPath];
}


Disable fmt check for prod-quick

Description:  Disable fmt check for prod-quick
Changes Proposed in PR: remove verify target for prod-quick and add for prod
Fixes: -
General Checklist

[x] Updated Added functionality/ bug fix in release notes - NA
[x] Added examples for new feature - NA
[x] Updated the documentation- NA

CRD Checklist

[x] Updated required CR schema- NA

/azp run


fix: compare full GAV when looking for declarations that match coordinates.

This resolves an issue when two versions of a dependency are in the various dependency graphs
(e.g., main and test). Without this, we were getting erroneous advice to remove used dependencies,
and then correct advice to re-add used-transitive dependencies. Additionally, the 'reason' results
were confusing and incorrect for a similar reason. As the simplest solution, require full GAV when
requesting the 'reason for' some advice in this circumstance.
There is still one minor issue in the reason output, which is it'll print the shortest path to all
versions of a given dep, even when a specific version is requested.

@jjohannes I think this resolves https://github.com/autonomousapps/dependency-analysis-gradle-plugin/issues/947.


Please update octicons from 3.1.0 to 6.0.1

Unfortunately not so easy because some octicons have been removed and hence updating might likely break some extensions that rely on them

Is there a plan going forwards with this? I just tried to use an octicon and it came up blank. I couldn't work out why until I checked the versions being used, which led me here.
From the change log, it looks like some of the removed icons are just renamed. Would it be possible to add a simple 'redirect' to return the new icon if the old name is used? Similarly, if an icon has genuinely been removed, could a placeholder be returned? In fact, it would probably be nicer to return something if a bad icon is named, rather than leaving a blank space.

would it be possible to add a simple 'redirect' to return the new icon if the old name is used?

Hm... Do you have a pointer/release notes that describe these changes?

I was just looking through here:
https://github.com/primer/octicons/blob/master/CHANGELOG.md

Great - thanks!

So, this would be the compat table and the good news is that no extension seems to use those three removed icons. The bad news is that octicons are now shipped as npm module which we cannot consume at the level at which we define the OcticonLabel
const _compatMapping = {
	// https://github.com/primer/octicons/blob/master/CHANGELOG.md#320-november-6-2015
	'screen-normal': '',
	'screen-full': '',
	// https://github.com/primer/octicons/blob/master/CHANGELOG.md#340-january-22-2016
	'color-mode': '',
	// https://github.com/primer/octicons/blob/master/CHANGELOG.md#600
	'ellipses': 'ellipsis',
	//https://github.com/primer/octicons/blob/master/CHANGELOG.md#601
	'kebab-veritcal': 'kebab-vertical',
	// https://github.com/primer/octicons/blob/master/CHANGELOG.md#octicons_node-700
	'file-text': 'file',
	'mail-reply': 'reply'
};

@jrieken we already have all the octicon resources checked in to our sources, so why is the fact that Octicons now ship as an NPM module an issue for us moving forward? Can we not just do the same?

Maybe @misolori has an idea on how to do that. I wonder if we can convert the set back to a font so that it is easy for us to consume it as we did before.

@chryw and I talked about how to convert the new svgs back into an icon font. We'd need to setup a project (probably in node) that would have the right modules to convert svgs into icons. @chryw already has a similar project but we'd need to:

Format the octicons file names to include their unicode
Format the less/sass templates to match the old styles format
Audit the changes from the old version of Octicons (renames/deletions/additions)
Map all of the changes
Also create local ttf file for users to install

This would take some time to setup, but nonetheless it is very hacky. Github has already moved towards using svgs so this would be a patch that we'd need to maintain.
One last thing to consider is if in the future we decide to create our own icon set (related to #8017) then this would be deprecated. I'd be in favor of putting our efforts into creating our own icon set that can be consumed by everyone.


about Classifier

the validation.py proposed resnet18_224 as a pretrained test model, how about the classify acc of Classifier2 in model_x.py ?

The classifier2 is much smaller than the resnet18. For our dataset, classifier2 is not enough to handle the complicated problem with so many possibilities. The performance will drop accordingly. We do not make full comparisons but you can try if you are intereasted. Thanks.


Add test case in update operation

Signed-off-by: leonwanghui<EMAIL_ADDRESS>This patch is first part of #66 , focusing on update scenario.

@norshtein Please take a review, thanks!


how to change download path to download files in documents ?

i need the files to be download in documents folder should i move it after i download them or is there method in it ?, also is there way of getting speed ratio that would be great thanks 😀

1)As explained in Readme file, the default path is inside caches directory.( and this is preferable)  If you want the file at some other location, you can move it as you get the URL in completion block.
2)Can you elaborate on this? May be support it with a use case.

is there a way to get download speed for a specific task , or get the speed for all the tasks combined , get remaining time or file size ? these would be great features to add
PS: 👍  for your project , it's updated , clean and easy to use

@A-jay98 Remaining time / Remaining file size entirely depends on how the network behaves at that point of time. I don't think so that responsibility lies with the Download Manager, hence haven't included them. Maybe you can fork and add these for your use. Thanks!


Add checks to OpenSearch in auto-generated manifests.

Signed-off-by: dblock<EMAIL_ADDRESS>Description
In https://github.com/opensearch-project/opensearch-build/pull/514 the generated manifests lacked basic checks. This adds version and build checks that ensures that the OpenSearch core builds successfully.
I didn't add any automatic checks to the plugins because those aren't always the same (e.g. security doesn't have a version check because it uses maven that only builds one of the flavors of snapshot vs. non-snapshot and therefore we cannot check version without hacks), and because those would fail without first building OpenSearch.
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #519 (29cfb0e) into main (757a225) will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #519      +/-   ##
==========================================
+ Coverage   70.25%   70.32%   +0.07%
==========================================
  Files          59       59
  Lines        1553     1557       +4
==========================================
+ Hits         1091     1095       +4
  Misses        462      462

Impacted Files
Coverage Δ

...undle-workflow/src/manifests_workflow/component.py
100.00% <100.00%> (ø)

...src/manifests_workflow/component_opensearch_min.py
100.00% <100.00%> (ø)

...workflow/src/manifests_workflow/input_manifests.py
95.65% <100.00%> (ø)

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 d3519a6...29cfb0e. Read the comment docs.

I understand we'd like to start with OpenSearch but we should add support to all plugins.
As called out in the description we could leave out a few to start with but most of the plugins should be do-able.

We have support for all but 2 plugins (performance-analyzer-rca and security), see #437 for details.
This PR adds checks automatically. Because plugins require OpenSearch to first build and publish to maven local, the checks won't work unless OpenSearch is already present in the manifest. If a component increments a version, it will cause a new manifest to be generated without OpenSearch (it has not incremented its version yet). So the auto-PR will break 100% of the time, hence I didn't add checks to components.


Mongo cbv6

Summary

Implemented the entire bot in Mongo, just if it was ever needed. (To be added as a new branch)
Checklist

[x] If code changes were made then they have been tested.
[ ] This PR fixes an issue.

Please reopen PR at the new branch I just created, will be closing this


--skip vs. --offset. Another limit option

Hi Evan,
first of all - thank you for the great tool, it's very useful and does save a lot of time!
What I was missing, is another "limit" option, which could control, how many records should be fetched from the source. E.g. I needed to extract N random samples from one index a put them into another one. Currently, it is not possible to "tell" the tool to stop after getting the first N documents.
So, another option was to use --skip <total - N> parameter and fetch N last docs (which is OK for a random query). But the skip function does not work as expected:
node elasticdump --input="http://localhost:9200/logstash-*" --output="http://localhost:9200/random" --searchBody '{"query":{"function_score":{"query":{"match_all":{}},"random_score":{}}}}' --skip 2447399 --limit 10000
Wed, 18 May 2016 16:32:03 GMT | starting dump
Wed, 18 May 2016 16:32:03 GMT | Warning: skipping 2447399 rows.
Wed, 18 May 2016 16:32:03 GMT |   * skipping doesn't guarantee that the skipped rows have already been written, please refer to the README.
Wed, 18 May 2016 16:32:04 GMT | got 10000 objects from source elasticsearch (offset: 0)
Wed, 18 May 2016 16:32:18 GMT | sent 10000 objects to destination elasticsearch, wrote 9241
Wed, 18 May 2016 16:33:46 GMT | got 2601 objects from source elasticsearch (offset: 20000)
Wed, 18 May 2016 16:33:48 GMT | sent 2601 objects to destination elasticsearch, wrote 2246

As you can see, it does not skip rows: it starts from the beginning and uses the doubled limit value for each iteration.
Fortunately, there is a hidden --offset parameter, which works like a charm:
node elasticdump --input="http://localhost:9200/logstash-*" --output="http://localhost:9200/random" --searchBody '{"query":{"function_score":{"query":{"match_all":{}},"random_score":{}}}}' --offset 2457399 --limit 100
Wed, 18 May 2016 17:14:50 GMT | starting dump
Wed, 18 May 2016 17:14:50 GMT | got 100 objects from source elasticsearch (offset: 2457399)
Wed, 18 May 2016 17:14:50 GMT | sent 100 objects to destination elasticsearch, wrote 100
Wed, 18 May 2016 17:14:50 GMT | got 100 objects from source elasticsearch (offset: 2457499)
Wed, 18 May 2016 17:14:51 GMT | sent 100 objects to destination elasticsearch, wrote 100

So, two questions:

what is the purpose of the --skip parameter, if there is a good working --offset (which is not documented yet)?
do you think it makes sense to implement a new limit option (e.g. --stop-after N), in order to get only the first N records from the source?

Thank you!

it turned out, that neither of these parameters work correctly :(
--offset works with a random result set only.

Great catch!
will be fixed via https://github.com/taskrabbit/elasticsearch-dump/pull/223


JSX ref

Is JSX's 'ref' attribute supported? In the following example it does nothing:

// voby 0.16

const App = () => {
    let eInput;

    function onClick() {
        console.log("eInput: " + eInput); // outputs 'eInput: undefined'
    }

    return (
        <input type="text" ref={eInput} onClick={onClick}/>
    )
};

render(<App />, document.body);

It is supported, but it must be a function, which will be called with the node. So basically I'd recommend making that an observable.
The code you pasted can't possibly work, you are just passing ref={undefined} effectively, there's no way that variable could get assigned. It works in Solid only because its transform changes how the language works, you are not really writing plain JS anymore.


Value of items not correct?

Hi,
I've tried out the plugin, seems like a really good idea and I'm glad it's on RL.
I noticed picking up loot @ brutal black dragons the value seemed off. 118k of dragon dart tips only pulled in a few thousand GP for example. Is this intentional?

Hey, you mean at the animation or actually in the profit calculation?

The overlay box itself was showing 70kish profit (includes other loot). The gold "drops" that appear can't be read as the icon is over them, but that might be a font issue on my side?
It doesn't change when I use up bolts or potions either, I assumed thats just how it works

Ok so the bug is (apart from font issue):
bolts / arrows fired aren't taken into calculation, aswell as arrows picked up when you  have arrows equiped.

About the potions: It doesn't reproduce for me in strength potions - I drink and and profit goes down, how do you use potions?

Ah sorry, when I mentioned the dragon darts those are dart tips (I'm using diamond (e)bolts), basically unfinished stuff so I don't think it's related to "ammunition" as I can't equip those.
I'm using prayer pots, extended anti-fires and divine ranging pot. Didn't see any reduction when drinking those normally.

OK I see what you mean, it's range related I think

Yup. Melee works fine for me, ranged completely broken.

Hi,
I've tried out the plugin, seems like a really good idea and I'm glad it's on RL.
I noticed picking up loot @ brutal black dragons the value seemed off. 118k of dragon dart tips only pulled in a few thousand GP for example. Is this intentional?

Hey, I've uploaded a fix for ranged. I think it should fix this bug. The new version should have an update in the GUI, (showing elapsed time).

Hi,
I've tried out the plugin, seems like a really good idea and I'm glad it's on RL.
I noticed picking up loot @ brutal black dragons the value seemed off. 118k of dragon dart tips only pulled in a few thousand GP for example. Is this intentional?

Hey, I've uploaded a fix for ranged. I think it should fix this bug. The new version should have an update in the GUI, (showing elapsed time).

Great I'll try that out later today, thanks for sorting that!


afterReflex, reflexSuccess, reflexError don't trigger if the element which "caused" the reflex isn't present on the page after the reflex..

As the title suggets. I'm not sure if it's a bug, or if this is the intended functionality.

Yes that's expected > https://docs.stimulusreflex.com/appendices/troubleshooting#dont-modify-the-dom-in-the-middle-of-a-reflex
I should make sure that piece of docs is in sockpuppet as well.


Java Collection : LinkedList

Type
Collection
Directory
Collection => LinkedList => LinkedList.java
Description
Just declare the data structure, take the input dynamically, demonstrate some inbuilt methods.
Programming Language

[ ] c++
[ ] c
[ ] python
[ ] java
[ ] kotlin
[ ] javascript
[ ] typescript
[ ] c#
[ ] swift
[ ] ruby
[ ] scala
[ ] go
[ ] php
[ ] haskell
[ ] bash

I can implement the basic methods of LinkedList Collection in java. Can you assign this task to me?

Hey @ahampriyanshu  can you please assign this task to me for C++, Thanks!


Arcade Cabinet has no collision with entities.

You can just walk right through arcade cabinets, shoot arrows through it etc.
buildersaddition-1.18.2-20220308a.jar
forge - 40.1.60

Fixed


Module not found: Can't resolve '../aws-exports'

I just cloned the repo and run following commands:
yarn install
yarn dev
But getting attached issue

I'm getting the same problem

the same here. any progress??


Elite emote clue scroll - Top of Slayer Tower

current code:
		new EmoteClue("Headbang at the top of Slayer Tower. Equip a seercull, a combat bracelet and helm of Neitiznot.", new WorldPoint(3115, 3420, 0), HEADBANG, item(SEERCULL), item(COMBAT_BRACELET), item(HELM_OF_NEITIZNOT)),
1st bug:
Combat bracelet (4) is not recognized on both inv and equipment menu.

2nd bug:
Correct location:

I'll write a fix for these today or tomorrow.


Safe procedure restart kube-apiserver

Description

The need for a safe restart kube-apiserver

Solution

Add safe procedure in code

Checklist

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] Integration CI passed
[ ] Unit tests. If Yes list of new/changed tests with brief description
[ ] There is no merge conflicts

Unit tests
Indicate new or changed unit tests and what they do, if any.

Pls specify clearly in the details what the unsafety was and how we made it safe.


CVE-2021-23362

Hi,
A vulnerability has been reported on hosted-git-info
The package hosted-git-info before 3.0.8 are vulnerable to Regular Expression Denial of Service (ReDoS) via regular expression shortcutMatch in the fromUrl function in index.js. The affected regular expression exhibits polynomial worst-case time complexity.

Hi @dbashford
Did you get chance to look into it and have an update?

Hi @dbashford any update on this issue?

Hi @dbashford
Did you get chance to look into it and have an update?

Hi @dbashford
Any update on this issue?


remove unused environment var loading entire history into an env var

It seems that ATUIN_HISTORY environment variable isn't used anywhere, and defining it loads your entire history into an environment variable. If you history gets too big, you'll get an error that your argument list is larger than the OS limit. On macOS this is 1MB.
Fixes #241

I think this looks ok. I indeed can't find the use for that variable.
I copied and pasted it from the ZSH file and I must have also mis-interpreted it too because its supposed to be the string "atuin history list" and not the actual output of the command. Not sure what that would be for though

Yeah I used ripgrep to search the entire codebase and that was the only instance of the string "ATUIN_HISTORY" being used anywhere. It also seems like loading your entire history into an environment variable would be something you definitely do not want to do 😅

It's a relic from long ago: https://github.com/ellie/atuin/commit/41f072a8b443b5a404916598f0c9b7a52f5c05fb
Seems you used to be able to configure what the search prompt showed based on that env var. But no longer used anywhere

Yeah I used ripgrep to search the entire codebase

Our config lib processes env vars for us automatically, so they would show as config.get_str("history"), or as a field in our Settings struct. Seems it's not their either though so I'd be happy with accepting this. Thanks again for looking into it, we don't use fish so it's nice to have someone test out code 😅

Happy to help!


Jest breaks util.promisify(setTimeout)

🐛 Bug Report
Jest breaks util.promisify(setTimeout) functionality by tampering with the setTimeout object.
When running code with jest it doesn't have the field Symbol(util.promisify.custom). When running code with plain node, it does.
To Reproduce
Run this test suite with jest
const util = require('util');
test('Should pass', async () => {
    await util.promisify(setTimeout)(1000);

    expect(true).toBe(true);
});

Expected behavior
Test should pass in 1s.
envinfo
  System:
    OS: Linux 4.19 Netrunner 19.08 (Indigo)
    CPU: (12) x64 Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz
  Binaries:
    Node: 12.12.0 - /bin/node
    npm: 6.12.0 - /bin/npm
  npmPackages:
    jest: ^24.9.0 => 24.9.0

The provided reproduction passes for me

Can you set up repl.it, codesandbox or a repository showing the error?

Sure
https://codesandbox.io/s/determined-chandrasekhar-jfey6?fontsize=14
In the right bottom select terminal, click the "Plus" button and run npm run test

@SimenB

Confirmed that setTimeout[util.promisify.custom] is missing in jest-environment-jsdom. But it is there in jest-environment-node. I think this is correct? Most environments with a DOM (browsers) will not have this either, only Node will.
However, it should definitely be there in jest-environment-node when using jest.useFakeTimers() - we should make sure jest-fake-timers supports that.

Ah, I just tested with node. But yeah, util.promisify won't work for the timer in jsdom

@GeorgeBekh that should clarify for your case, see docs on how to use different test environments. We could change this issue to be about jest-fake-timers missing the property though :)

Thanks I've forgot, about the testEnvironment option.
Sure. I'm glad I've helped somehow :)

updated title/description :)

Is there some assign already? I would like to take a look at this.

Go for it! :)

Fixing it for Lolex at the same time would be lovely, if you're up for it 🙂 Jest will be using Lolex as fake timer implementation as default in v26
https://github.com/sinonjs/lolex/issues/223


Add dream-html package

This pull-request includes a derivation for yawaramin/dream-html.  Since aantron/dream is already included in this overlay, I've found it useful to include this as well.

I thought that was the commit sha of the commit that introduced 1.1.0.  What am I missing/what should I change it to?

I thought that was the commit sha of the commit that introduced 1.1.0. What am I missing/what should I change it to?

rev = “1.1.0”

rev = “1.1.0”

Thanks for pointing that out, completely slipped my mind as a possibility.

Thank you


Suppress requests to the Github API

https://docs.github.com/en/rest/overview/resources-in-the-rest-api#requests-from-github-actions

When using GITHUB_TOKEN, the rate limit is 1,000 requests per hour per repository.

https://github.com/x-motemen/git-pr-release/blob/785e486041821862775064359101ffb95848f302/lib/git/pr/release/cli.rb#L112

fixed


fix: Component id separator

Fixes #1918
This PR changes the usage of ___ component id separator to a much more unique one: (VISUALIZE.ADMIN_COMPONENT_ID_SEPARATOR), as it turns out some cube iris can end with _, breaking the separation and joining logic.
I didn't introduce a new chart config migration to replace the old separator with new one, as the application was not deployed to PROD since component ids were introduced. It means that some TEST and INT charts that were created recently might get broken (but as these are testing environments, it should be fine).
How to test

Go to this link and start a new visualization.
✅ See that it's possible to make a chart.
Switch to an area chart.
Add color segmentation.
Make it interactive.
Remove one or more filters from the multi-filter panel.
✅ See that the filters were also removed from the chart legend.

@bprusinowski tested on the PR and it looks good to me. Thanks for the quick fix

@noahonyejese looked like some Sentry error. I re-deployed and seems to work fine 👍


setup walkcamera tool

Adding setup walk camera tool to default navigation tools

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


Added Almost Binary

Added the 2 future libraries too, as I'll implement them within the next few weeks.

@wsdt Thanks for the addition.
Everything is OK with the formatting :+1:
A few possible improvement :smiley:

Try to make the description about what makes your language unique and esoteric, Ideally, people reading this list should get the basic idea on what the language is about. In this case, I would try to emphasize the idea that it is a multiplatform high-level language where you code in binary. I obviously don' t know a lot about your language so I may be wrong about the core idea. Likewise, I wouldn't put a lot of info on libraries or what can be used for unless it is a core idea of the language itself.

Consider using AlmostBinary as the name for the entry of this list (no space). Most programming languages are named in a single word (at least in this list). Of course, you are the creator of this language and if you feel the proper name for it is using 2 words, feel free to keep it that way :)

Check the following descriptions in case that helps:

Monicelli - Programming using Italian-like gibberish.
Unary - Brainfuck-based language where all the commands are 0.

You are right :-)
Thanks, just modified the ReadMe.

@wsdt Cool, I think the new description greatly helps to understand what the language is about. However, I feel the bit: Code in platform-independent binary may be hard to understand for newcomers (you code in binary vs you code using a platform-independent binary?). How about something similar to: Platform-independent coding in binary or Code in binary and run in any platform?

Done 👍


Update for lucid-2.5

Attempting to build with lucid-2.5 instead of lucid-2.0 results in the following failure:
src/TryHaskell.hs:366:61:
   No instance for (Monoid a1) arising from a use of ‘mempty’
   The type variable ‘a1’ is ambiguous
   Note: there are several potential instances:
     instance Monoid a => Monoid (Control.Applicative.Const a b)
       -- Defined in ‘Control.Applicative’
     instance Monoid () -- Defined in ‘Data.Monoid’
     instance (Monoid a, Monoid b) => Monoid (a, b)
       -- Defined in ‘Data.Monoid’
     ...plus 39 others
   In the second argument of ‘p_’, namely ‘mempty’
   In the first argument of ‘td_’, namely
     ‘(p_ [class_ "haskell-icon"] mempty)’
   In a stmt of a 'do' block: td_ (p_ [class_ "haskell-icon"] mempty)

src/TryHaskell.hs:386:44:
   No instance for (Monoid a0) arising from a use of ‘mempty’
   The type variable ‘a0’ is ambiguous
   Note: there are several potential instances:
     instance Monoid a => Monoid (Control.Applicative.Const a b)
       -- Defined in ‘Control.Applicative’
     instance Monoid () -- Defined in ‘Data.Monoid’
     instance (Monoid a, Monoid b) => Monoid (a, b)
       -- Defined in ‘Data.Monoid’
     ...plus 39 others
   In the second argument of ‘span6_’, namely ‘mempty’
   In a stmt of a 'do' block: span6_ [id_ "console"] mempty
   In the first argument of ‘row_’, namely
     ‘(do { span6_ [id_ "console"] mempty;
            span6_ [id_ "guide"] mempty })’

There's a cabal.config for GHC 7.8.3, I haven't tested on other version sets yet.

Okay, I bumped Lucid to 2.6 to fix this.

Fixed in 07fb75d44a08973a0b458588636aff8a61a06f15.