Page with radiobuttons throws an error when dismissed/backbutton

Short description of the problem:
I have a page that opens another page containing 2 radiobuttons.
When hitting backbutton on that page there is an exception thrown:
EXCEPTION: TypeError: Cannot read property 'remove' of null
What behavior are you expecting?
No errors.
Steps to reproduce:

Create a page that opens another page containing radiobuttons
Visit that page and click back-button

Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
3     898346   error    STACKTRACE:
5     898347   groupEnd
6     898348   error    Uncaught TypeError: Cannot read property 'remove' of null, http://localhost:8100/build/js/angular2-polyfills.js, Line: 324
0     952330   log      Angular 2 is running in the development mode. Call enableProdMode() to enable the production mode.
1     959837   group    EXCEPTION: TypeError: Cannot read property 'remove' of null
2     959838   error    EXCEPTION: TypeError: Cannot read property 'remove' of null
3     959839   error    STACKTRACE:
4     959839   error    TypeError: Cannot read property 'remove' of null
at RadioButton.ngOnDestroy (http://localhost:8100/build/js/app.bundle.js:45957:20)
at AbstractChangeDetector.ChangeDetector_RadioPage_0.dehydrateDirectives (viewFactory_RadioPage:286:20)
at AbstractChangeDetector.dehydrate (http://localhost:8100/build/js/app.bundle.js:12904:14)
at AbstractChangeDetector.destroyRecursive (http://localhost:8100/build/js/app.bundle.js:12917:14)
at AbstractChangeDetector.destroyRecursive (http://localhost:8100/build/js/app.bundle.js:12924:25)
at AppView.destroy (http://localhost:8100/build/js/app.bundle.js:22543:29)
at AppViewManager_.destroyViewInContainer (http://localhost:8100/build/js/app.bundle.js:23008:14)
at ViewContainerRef_.remove (http://localhost:8100/build/js/app.bundle.js:22870:19)
at Array. (http://localhost:8100/build/js/app.bundle.js:44020:35)
at ViewController.destroy (http://localhost:8100/build/js/app.bundle.js:45458:30)
5     959840   groupEnd
6     959842   error    Uncaught TypeError: Cannot read property 'remove' of null, http://localhost:8100/build/js/angular2-polyfills.js, Line: 324
Which Ionic Version? 2.x
Plunker that shows an example of your issue
http://plnkr.co/edit/SUpyXm?p=preview
Click "Show radios" then hit back button. An exception are thrown in the console.
Run ionic info from terminal/cmd prompt: (paste output below)
ionic info
Your system information:
Cordova CLI: 6.1.1
Gulp version:  CLI version 3.9.1
Gulp local:   Local version 3.9.1
Ionic Framework Version: 2.0.0-beta.6
Ionic CLI Version: 2.0.0-beta.25
Ionic App Lib Version: 2.0.0-beta.15
OS:
Node Version: v4.4.3

Hmmm good catch! Have you noticed this with any other component or just the radio buttons component?

I have not seen this with other components i have used.

+1

Hello all! Here is a working plunker. The issue (which i should have spotted earlier) was that the ion-list should have had the radio-group attribute on it. Thanks for using Ionic!

Hello @teleaziz ! Are you interested in doing a pr for this?

@jgw96 , thanks! just did :)

Thanks for the issue and the PR! This will be in the beta.7 release.


add new event for Research Data Roundtable

This is the landing page for our Research Data Roundtable. I'll write an update email on this to the team later.
It is a focus group session where we invite researchers to talk about their research data challenges post Twitter API. We'll also leverage this interaction to validate-test our new capabilities, including those that Mat and Rob are working on.
Event document: https://docs.google.com/document/d/1mFOe93iPioQMi4USM0MtIOEY13dxhHCHuJ54Bn77CUw/edit?usp=sharing

@Quiet27 thank you for the edits and the typo fix! Do you think it's good to go?

@Quiet27 I've made some changes based on your comments. Thanks for the suggestion.


Fix tests not being run

@ironman-machine go

Hello @dora-korpar
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "master", "SCALITY_S3_BRANCH": "FIX/multBackendFuncTest"}
Please follow http://ci.ironmann.io/gh/scality/Integration/13484 for CI status.

@ironman-machine go

Hello @dora-korpar
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "master", "SCALITY_S3_BRANCH": "FIX/multBackendFuncTest"}
Please follow http://ci.ironmann.io/gh/scality/Integration/13503 for CI status.

@ironman-machine try

Hello @dora-korpar
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "master", "SCALITY_S3_BRANCH": "FIX/multBackendFuncTest"}
Please follow http://ci.ironmann.io/gh/scality/Integration/13512 for CI status.

@ironman-machine go

Hello @dora-korpar
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "master", "SCALITY_S3_BRANCH": "FIX/multBackendFuncTest"}
Please follow http://ci.ironmann.io/gh/scality/Integration/13532 for CI status.

@ironman-machine gogo

Hello @dora-korpar
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "master", "SCALITY_S3_BRANCH": "FIX/multBackendFuncTest"}
Please follow http://ci.ironmann.io/gh/scality/Integration/13536 for CI status.


[feature] [question] fast forwarding the snake

Hi! Thanks for the blog post.
I was wandering whether changing in the game.rs
Command::Turn(towards) => {
    if direction != towards && direction.opposite() != towards {
        self.snake.set_direction(towards);
    }
}

to:
Command::Turn(towards) => {
    if direction == towards {
        break;
    } else if direction.opposite() != towards {
        self.snake.set_direction(towards);
    }
}

is the proper way to do add a feature of fast forwarding the snake? Is there any other, maybe more idiomatic way to achieve this that you can think of? ^^

Hey! You are very welcome.
Your suggestion does indeed look like a good way to implement a fast-forward function, by pressing in the direction that the snake is facing.
By breaking from the inner loop we're effectively jumping to the next iteration and breaking from the timer that is used to regulate the time in between each snake move, so it does have the effect of pushing the snake onwards each time you press. One thing I did notice when trying locally is that holding the key down registers a lot of keypresses and results in the snake moving super quickly, so that is probably not intended.
It's been so long since I looked at the code for this that I couldn't suggest a better way off the top of my head. I think implementing a fast-forward feature so that it works a bit more reliably, would probably entail re-working the primary game loop in a way that accounts for repeated keypresses.
In all honesty, I'm not much of a Rust expert and building this game was an exercise in brushing up on the little knowledge I had as well as for a blog post to help others. I'm probably not qualified enough to talk about idiomatic Rust.
Thanks again for your feedback - it is much appreciated :)

Howdy, so I know it's been ages I implemented the FForwarding. I just did it by making two new keys up and down and using those to adjust the snake internal speed for example
KeyCode::Char('d') => Some(Command::Adjust(Speed::Down)),
Command::Adjust(towards) => {
    if Speed::Down == towards && self.speed != 0{
        self.speed -= 1;
    }


Fix carbon line client protocol so it can send correct data

Currently when DESTINATION_PROTOCOL is set to line for carbon-relay.
It sends incorrect data to the destination. The order of metric value
and timestamp are changed from the source, for example, it turns
"test.mytest 1<PHONE_NUMBER>" into "test.mytest<PHONE_NUMBER> 1".
CarbonLineClientProtocol was introduced in a previous pull request #601
to support pluggable client protocols along with a number of other
changes. The incorrect order takes place in its method
_sendDatapointsNow().
This fix will correct the order of segments, so metrics path, metric
value and timestamp will be placed in the right order. An unit test is
also added to ensure carbon line client protocol will send correct data
in the future.

Ouch. @iksaif ?


When --runInBand is set Tests Built from a Higher Order Function that Contain a Promise still run in Parallel.

🐛 Bug Report
Tests built from a higher order function that contain a promise still run in parallel when --runInBand is set.
To Reproduce
Steps to reproduce the behavior:

Create a higher order function that constructs tests and runs a promise passed in as an argument
Make several tests using this function
Tests then execute in parallel

Expected behavior
Tests should run sequentially when --runInBand is set.
Link to repl or repo (highly encouraged)
Repo with working replication can be seen here. Log output from running npm run test should be 1, 2, 3, 4, instead is 1, 2, 4, 3. Something also appears to be messing up test timing as if you run fail.test.js alone the timing for test 4 is 0 ms.
envinfo
System:
OS: Windows 10 10.0.18363
CPU: (12) x64 AMD Ryzen 5 3600 6-Core Processor
Binaries:
Node: 12.16.0 - C:\Program Files\nodejs\node.EXE
Yarn: 1.22.0 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
npm: 6.13.4 - C:\Program Files\nodejs\npm.CMD
npmPackages:
jest: ^26.4.0 => 26.4.0

The --runInBand flag is for running multiple test files serially, without parallelization, that's all it is to it. It doesn't control the execution of code inside the test files.

Is there any way to make the tests inside a file execute serially?
Also I'm incredibly confused as to why the same tests executes in different orders consistently when the only difference is one set of tests was created by a higher order function and the other was not.

In your repro you forgot to return the promise from the test function:
const makeTestFn = (fns) => (cb) => {
-  Promise.all(fns).then(() => {
+  return Promise.all(fns).then(() => {
    cb();
  });
};

Making that change still gives the same result.

And regardless there's some deeper issue here, If I set the timeout for 3 to be 2000 and the timeout for 4 to 1000, the test timing ought to say test 3 took ~2000ms and test 4 took 1000ms. But that isn't what happens, instead you get this.

When I set the timeouts to be 1000, 2000 for 3 and 4 respectively the test timings should say ~1000ms for 3 and ~2000ms for 4, instead you get this.

Running work.test.js works as expected with both tests showing the correct timings.


sync color with gitgutter

From README.md:
Also supports dynamic highlighting of signs. In the image above the marks are colored according to the state of the line as indicated by gitgutter.
NOTE: This feature is disabled by default
So how to enable it?

Sorry I found it in documentation.

Where in the documentation did you find it? I couldn't find it?


changed default value of persist_directory in settings

Description of changes
This pull request addresses an issue with the default value of persist_directory in the Settings class. Previously, the default value was set to ".chroma", which is not a valid path. This PR updates the default value to "./chroma", ensuring the path is valid and the directory is created correctly.
Currently, if you create a client with these settings:
from chromadb.config import Settings
client = chromadb.Client(Settings(
    chroma_db_impl="duckdb+parquet",
))

the program will fail
Disclaimer:
I am new to open source! This is my first PR so if I made any mistakes or skipped any steps of this process please let me know! I'd love to learn and contribute more in the future. Thanks.

#338 @HammadB

Went with a different solution here. Thanks for helping to flag this!


[Task]: POC for SAP connector using SAP Java connector library

Connector Name
other (None Listed Above)
Task Description
Working on POC to build up Ballerina connector for SAP BAPIs invocation.

Closing as completed


Statically  & dynamically error on a method with duplicate argument name

Error
[@ @ /jenkins/.rvm/gems/jruby-<IP_ADDRESS>@looker/gems/sorbet-runtime-0.4.5045/lib/types/private/methods/signature.rb:85]:
NoMethodError: undefined method `+' for nil:NilClass
    @ @ /jenkins/.rvm/gems/jruby-<IP_ADDRESS>@looker/gems/sorbet-runtime-0.4.5045/lib/types/private/methods/signature.rb:85:in `block in initialize'
    org/jruby/RubyArray.java:2813:in `zip'
    /jenkins/.rvm/gems/jruby-<IP_ADDRESS>@looker/gems/sorbet-runtime-0.4.5045/lib/types/private/methods/signature.rb:71:in `initialize'
    /jenkins/.rvm/gems/jruby-<IP_ADDRESS>@looker/gems/sorbet-runtime-0.4.5045/lib/types/private/methods/_methods.rb:295:in `build_sig'
    /jenkins/.rvm/gems/jruby-<IP_ADDRESS>@looker/gems/sorbet-runtime-0.4.5045/lib/types/private/methods/_methods.rb:268:in `run_sig'
    /jenkins/.rvm/gems/jruby-<IP_ADDRESS>@looker/gems/sorbet-runtime-0.4.5045/lib/types/private/methods/_methods.rb:200:in `block in _on_method_added'
    /jenkins/.rvm/gems/jruby-<IP_ADDRESS>@looker/gems/sorbet-runtime-0.4.5045/lib/types/private/methods/_methods.rb:364:in `run_sig_block_for_key'
    /jenkins/.rvm/gems/jruby-<IP_ADDRESS>@looker/gems/sorbet-runtime-0.4.5045/lib/types/private/methods/_methods.rb:348:in `run_sig_block_for_method'
    /jenkins/.rvm/gems/jruby-<IP_ADDRESS>@looker/gems/sorbet-runtime-0.4.5045/lib/types/private/methods/_methods.rb:224:in `block in bar'

Input
class A
  extend T::Sig

  sig {params(_: Integer, _: Integer ).returns(String)}
  def self.bar(_, _)
    ""
  end
end

Observed output
The sorbet.run doesn't fail because this is a sorbet-runtime error.
(It would be great to run code and test sorbet-runtime)
I believe the error to be related to having the same argument name twice.

We're unlikely to support methods with multiple arguments with _ as name.
That said, it would be nice to have a better error message

Makes sense -- our codebase is large so there's the pattern a lot of having more than 1 with underscore for unused methods.
(A form of documentation allowing the reader to know the variables are unused)

Fixed in #3345


What is the purpose of the Prior Network?

CLIP is trained to bring the image and text embeddings of an image-text pair as close to each other as possible.
So the question is, why can't we just skip the prior network, and directly input the clip text embeddings to the decoder network? Prior network tries to predict clip image embeddings from clip text embeddings, but by definition (clip training), these two should be already very close to each other. Am I missing something?

Thank you for the quick answer, that makes so much sense. Do you have a link to this second thing you mentioned. "openai reports trying it and it works"?

Apparently it is directly in their paper. Refer to section 5.1. Thanks for pointing this out @rom1504

^ exactly this

Just want to share our paper that might help answering this question:

CLIP is trained to bring the image and text embeddings of an image-text pair as close to each other as possible. So the question is, why can't we just skip the prior network, and directly input the clip text embeddings to the decoder network? Prior network tries to predict clip image embeddings from clip text embeddings, but by definition (clip training), these two should be already very close to each other. Am I missing something?

arXiv:2203.02053 uploaded on 3 Mar 2022

 Figure 1: The pervasive modality gap in multi-modal contrastive representation learning

In the paper, we present modality gap, an intriguing geometric phenomenon of the representation space of multi-modal models. As shown in Figure 1(b), CLIP’s image embeddings and text embeddings are located in two completely separate regions of the embedding space. We find this phenomenon consistently across various multi-modal models, covering texts,
natural images [Radford et al., 2021], videos [Xu et al., 2021], medical images [Zhang et al., 2020], and amino-acid
sequences [EleutherAI]. Interestingly, this phenomenon still holds even when we embed using multi-modal models with
random weights (Figure 1 (c)).
Our systematic analysis demonstrates that this gap is caused by a combination of model initialization and contrastive learning optimization. In model initialization, we show empirically and theoretically that the representation of a common deep neural network is restricted to a narrow cone. As a consequence, in a multi-modal model with two encoders, the representations of the two modalities are clearly apart when the model is initialized. During optimization, contrastive learning keeps the different modalities separate by a certain distance, which is influenced by the temperature parameter in the loss function. Our experiments further demonstrate that varying the modality gap distance has a significant impact in improving the model’s downstream zero-shot classification performance and fairness.
Hope that it will be helpful for future readers!

Just want to share our paper that might help answering this question:

CLIP is trained to bring the image and text embeddings of an image-text pair as close to each other as possible. So the question is, why can't we just skip the prior network, and directly input the clip text embeddings to the decoder network? Prior network tries to predict clip image embeddings from clip text embeddings, but by definition (clip training), these two should be already very close to each other. Am I missing something?

arXiv:2203.02053 uploaded on 3 Mar 2022

Figure 1: The pervasive modality gap in multi-modal contrastive representation learning
In the paper, we present modality gap, an intriguing geometric phenomenon of the representation space of multi-modal models. As shown in Figure 1(b), CLIP’s image embeddings and text embeddings are located in two completely separate regions of the embedding space. We find this phenomenon consistently across various multi-modal models, covering texts, natural images [Radford et al., 2021], videos [Xu et al., 2021], medical images [Zhang et al., 2020], and amino-acid sequences [EleutherAI]. Interestingly, this phenomenon still holds even when we embed using multi-modal models with random weights (Figure 1 (c)).
Our systematic analysis demonstrates that this gap is caused by a combination of model initialization and contrastive learning optimization. In model initialization, we show empirically and theoretically that the representation of a common deep neural network is restricted to a narrow cone. As a consequence, in a multi-modal model with two encoders, the representations of the two modalities are clearly apart when the model is initialized. During optimization, contrastive learning keeps the different modalities separate by a certain distance, which is influenced by the temperature parameter in the loss function. Our experiments further demonstrate that varying the modality gap distance has a significant impact in improving the model’s downstream zero-shot classification performance and fairness.
Hope that it will be helpful for future readers! arXiv:2203.02053
https://modalitygap.readthedocs.io/

COOL!  Just curious, do you think it's possible to get this modality gap eliminated or almost eliminated? (The blue and red dots group together and interleave)


Error communication with api

I am trying open the react-google-maps in my application but the request for the maps API doesn't work, this is my code:
`var React = require('react');
var GoogleMapLoader = require('react-google-maps/lib/GoogleMapLoader');
var GoogleMap = require('react-google-maps/lib/GoogleMap');
var GoogleMap = React.createClass(
{
/** React mount
 *  Todas as funções chamadas na montagem do componente
 *  getInitialState, getDefaultProps, propTypes, mixins, statics, displayName
 */
getInitialState: function () {
	return {
	}
},

render: function()
{
	return (

		<GoogleMapLoader
          containerElement={ <div style={{height: "100%"}} /> }
          googleMapElement={ <GoogleMap defaultZoom={3} defaultCenter={{lat: -25.363882, lng: 131.044922}}> </GoogleMap> }
        />

	);
}

});
module.exports = GoogleMap;
`
This is what happend in firefox:
http://i.imgur.com/ukBt38H.png

I believe this is an issue of googles' own servers. Close for now.
We're also looking for maintainers. Involve in #266 to help strengthen our community!


Make markdown note generation "idempotent"

Currently, the Bear to MD generator creates markdown notes that have to be tweaked just slightly in order to be published to the site.
We need to be able to "trust" the Bear to MD generator to generate notes in their "final form."

Observed Differences / Problems

[ ] When the note "title" is changed in Bear, it generates a different file name for the note in the Personal Site. Filenames need to be consistent in spite of changing titles.

Solve this via simple UUID-based URLs. We will make URLs meaningful later.


Enable OSX build

@iwonasado Thanks for opening the PR for my branch...but it's not ready. Are you looking for a OSX builds or something?

The OSX build passes but the start up down is very slow.


Bulma Calendar widget shows incorrect value

Using the Bulma Calendar widget and notice that it's showing the incorrect date on a form.
Below screenshot shows the values it should be:

When updating the object the Bulma-Calendar widget shows the following:

It appears to be showing a day earlier than the value of the input:

Did a quick look at upstreams bugs and didn't see any for this bug, but when I get some free time I'll give a more thorough search.

Should have noted this is with v2.4.0, and it looks like this bug was fixed with a more recent version of Bulma-Calendar (tho I haven't had a chance to verify this).

I'll make sure to update Bulma-Calendar in the next release of django-simple-bulma, which will happen soon.


[Enhancement] avoid duplicate deletion of schema

Motivation
Currently when I need to delete a namespace forcedly, the NamespacesBase#internalDeleteNamespaceForcefully will firstly list all the topics under the namespace and then delete them concurrently without distinguishing wheather the topic is partitioned or non-partitioned. This behaivor will cause duplicate deletion of schema when the topic list contains partitioned topics, because all the partitions of the partitioned topic share the same schema.
Modifications
Distinguish partitioned topic to avoid duplicate deletion of the same schema.
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): no
The public API: no
The schema: no
The default values of configurations: no
The wire protocol: no
The rest endpoints: no
The admin cli options: no
Anything that affects deployment: no

Documentation
This PR does not need to update the docs.

I believe that this is a good idea.
Can you please add a test case ?

@hangc0276 thanks for triaging PR and issues. When you label PRs, if you find some PRs do not need to update doc,  could you please help label the PR with no-need-doc in the future? Thanks

@hangc0276 thanks for triaging PR and issues. When you label PRs, if you find some PRs do not need to update doc, could you please help label the PR with no-need-doc in the future? Thanks

@Anonymitaet Ok,  I will add doc label in the following PRs, thank you!

Good work
Can you add a test case?

Sorry for the late reply! I will add a test later.

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

@eolivelli @hangc0276 Sorry for the late. I supplemented a test case. PTAL~


docs(web): Use external book link

Because book_integrated doesn't reflect the pages within the book, and we get confusions like https://github.com/PRQL/prql/issues/286#issuecomment-1426383971

@richb-hanover this mistakenly includes your generous change to the ical. I'll leave it in if that's OK!

@richb-hanover this mistakenly includes your generous change to the ical. I'll leave it in if that's OK!

Please feel free to "make it right". Thanks.


Configurable variable delimiter in TEMPLATE.

I use angularJS inside another framework. Therefore, need to have for example '[[' instead of '{{' as a variable delimiter.

By analyzing the blame information on this pull request, we identified @gschammah, @gonzalog and @asharbitz to be potential reviewers


exception in android 12

For me, when using the method (chain.proceed (request)), there is a problem which is the following error.
My error happens in Android 12
I use retrofit
class CustomInterceptor implements Interceptor {     @NonNull     @Override     public Response intercept(Chain chain) throws IOException {         Request request = chain.request ();         return chain.proceed (request);     } }
Targeting S+ (version 31 and above) requires that one of FLAG_IMMUTABLE or FLAG_MUTABLE be specified when creating a PendingIntent.Strongly consider using FLAG_IMMUTABLE, only use FLAG_MUTABLE if some functionality depends on the PendingIntent being mutable, e.g. if it needs to be used with inline replies or bubbles.

Neither Retrofit or OkHttp are relevant for the error with Intent Mutability. I suggest you search and ask on Stackoverflow.

@arsalanNasiri  Did you managed to fix this issue?

I had some issue on Android 12. Please fix that issue..

I have some issue on Android 12. Please fix that issue..

Sorry for the delay, in case it is useful information I'm using  implementation 'com.google.android.gms:play-services-location:20.0.0' apart from OkHttp is the only dependency I wasn't able to update yet because it slows the whole app when I do it.
The errors in the order I got them:
NotificationsManager
java.lang.IllegalArgumentException: com.anon.anon: Targeting S+ (version 31 and above) requires that one of
Strongly consider using FLAG_IMMUTABLE, only use FLAG_MUTABLE if some functionality depends on the
PendingIntent being mutable, e.g. if it needs to be used with inline replies or bubbles.

EventThread
Task threw exception
java.lang.IllegalStateException: Unable to extract the trust manager on Android10Platform, sslSocketFactory is
class com.google.android.gms.org.conscrypt.OpenSSLSocketFactoryImpl
	at okhttp3.OkHttpClient$Builder.sslSocketFactory(OkHttpClient.kt:751)
	at io.socket.engineio.client.transports.WebSocket.doOpen(WebSocket.java:50)
	at io.socket.engineio.client.Transport$1.run(Transport.java:82)
	at io.socket.thread.EventThread.exec(EventThread.java:55)
	at io.socket.engineio.client.Transport.open(Transport.java:77)
	at io.socket.engineio.client.Socket.probe(Socket.java:472)
	at io.socket.engineio.client.Socket.onOpen(Socket.java:485)
	at io.socket.engineio.client.Socket.onHandshake(Socket.java:526)
	at io.socket.engineio.client.Socket.onPacket(Socket.java:499)
	at io.socket.engineio.client.Socket.access$1000(Socket.java:31)
	at io.socket.engineio.client.Socket$5.call(Socket.java:313)
	at io.socket.emitter.Emitter.emit(Emitter.java:117)
	at io.socket.engineio.client.Transport.onPacket(Transport.java:134)
	at io.socket.engineio.client.transports.Polling.access$700(Polling.java:17)
	at io.socket.engineio.client.transports.Polling$2.call(Polling.java:124)
	at io.socket.engineio.parser.Parser.decodePayload(Parser.java:251)
	at io.socket.engineio.client.transports.Polling._onData(Polling.java:134)
	at io.socket.engineio.client.transports.Polling.onData(Polling.java:106)
	at io.socket.engineio.client.transports.PollingXHR$5$1.run(PollingXHR.java:111)
	at io.socket.thread.EventThread$2.run(EventThread.java:80)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1137)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:637)
	at java.lang.Thread.run(Thread.java:1012)

AndroidRuntime
FATAL EXCEPTION: EventThread
Process: com.anon.anon, PID: 31286
java.lang.IllegalStateException: Unable to extract the trust manager on Android10Platform, sslSocketFactory is
class com.google.android.gms.org.conscrypt.OpenSSLSocketFactoryImpl
	at okhttp3.OkHttpClient$Builder.sslSocketFactory(OkHttpClient.kt:751)
	at io.socket.engineio.client.transports.WebSocket.doOpen(WebSocket.java:50)
	at io.socket.engineio.client.Transport$1.run(Transport.java:82)
	at io.socket.thread.EventThread.exec(EventThread.java:55)
	at io.socket.engineio.client.Transport.open(Transport.java:77)
	at io.socket.engineio.client.Socket.probe(Socket.java:472)
	at io.socket.engineio.client.Socket.onOpen(Socket.java:485)
	at io.socket.engineio.client.Socket.onHandshake(Socket.java:526)
	at io.socket.engineio.client.Socket.onPacket(Socket.java:499)
	at io.socket.engineio.client.Socket.access$1000(Socket.java:31)
	at io.socket.engineio.client.Socket$5.call(Socket.java:313)
	at io.socket.emitter.Emitter.emit(Emitter.java:117)
	at io.socket.engineio.client.Transport.onPacket(Transport.java:134)
	at io.socket.engineio.client.transports.Polling.access$700(Polling.java:17)
	at io.socket.engineio.client.transports.Polling$2.call(Polling.java:124)
	at io.socket.engineio.parser.Parser.decodePayload(Parser.java:251)
	at io.socket.engineio.client.transports.Polling._onData(Polling.java:134)
	at io.socket.engineio.client.transports.Polling.onData(Polling.java:106)
	at io.socket.engineio.client.transports.PollingXHR$5$1.run(PollingXHR.java:111)
	at io.socket.thread.EventThread$2.run(EventThread.java:80)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1137)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:637)
	at java.lang.Thread.run(Thread.java:1012)

This suggests you aren't passing in the trustManager. The sslSocketFactory shouldn't fail like that if you do.
Are you using the latest version of that library
This seems correct
https://github.com/socketio/socket.io-client-java/blob/main/src/site/markdown/usage.md

I was already using the trustManager that was not the issue, updating io.socket:socket.io-client from 0.x to 1.x solved the issue. I didn't update that dependency from the begining because the server is on 1.x and the compatibility chart shows that only works with clients in 0.x, but it seems working now...
Thank you for your time.


In the process of Chisel to FIRRTL, the ArrayBuffer how to generate FIRRTL after adding Command?

For example, object Mux will eventually call the pushCommand function in chiselFrontend/internal/Builder.scala. However, pushCommand calls the addCommand function in UserModule. Scala by adding Command to the variable-length array ArrayBuffer [Command]. How does Array Buffer [Command] generate Mux's FIRRTL?
Detailed pictures are as follows

https://groups.google.com/forum/#!topic/chisel-users/D4dJv17xN6c

Answered in that thread.


Loosen constraint on qt for qt-webengine

@conda-forge/qt-main @conda-forge/qt-webengine what do you think about this? I think with the recent icu migration, we would need to rebuild qt-webengine to be compatible with qt-main 5.15.3 so I loosened the constraint all the way down to 5.15.2.
Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.
Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

+1 from me.


Moved TimeStats / Added fields to different structs / Renaming

Moved TimeStats / Added fields to different structs / Renaming
TimeStats
Issues
The TimeStats functions are related to an issue, that's why we moved and adapted the functions to the issues.go file.
We actually kept timestats.go file for backward compatibility.
Merge-Requests
We created the same functions as the issues's timestats functions for the merge-requests in the merge-requests.go file
Structures
Groups
We added a ParentID field in the CreateGroupOptions struct in the groups.go file.
Issues
We added a Search field in the ListProjectIssuesOptions struct in the issues.go file.
Merge Requests

We added a Search and a Labels field in the ListMergeRequestsOptions struct in the merge-requests.go file.
We added a MilestoneID, a Labels and a TargetBranch field in the UpdateMergeRequestOptions struct in the merge-requests.go file.

Renaming
We changed the ListMergeRequests function name and the ListMergeRequestsOptions struct name to ListProjectMergeRequests and ListProjectMergeRequestsOptions, indeed, it was inconsistent compared to the issues.go file using the same function (equivalent route) with ListProjectIssues function and ListProjectIssuesOptions struct.
We actually kept ListMergeRequests function and ListMergeRequestsOptions struct for backward compatibility.

Closed in favour of PR #198


Table textcontainer card selector titlecontainer

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Calendar View

Idea to improvement.
Add Calendar view to Saltcorn.

Yes, a calendar view plug-in is on the roadmap

work-in-progress: https://github.com/saltcorn/fullcalendar

@glutamate You're Best!

@AleksandrSokolov Thank you :)
first version released as "fullcalendar" in the plugin store


Testmo upload does not include describe group  words from tests

This is a nice reporter; I like how it maps the steps.  One issue currently, is the reporter does not include the describe group words from tests.
products/add-product.spec.ts
test.describe('add product from scratch', () => {

    test('adds single', async ({ page }) => {
          ....
    });
});

playwright-testmo-reporter

junit-reporter

Hello @greggjensencart!
Thank you for creating this issue!
I can see why this feature would be useful.
Would you like to implement this yourself and create a PR? If not, then I'd be happy to implement this.

It would take me a bit to get familiar with the code base.  If you can add it, that would be great.

Sure thing! I'll get started on that!

Woops, closed that too quickly.
I have released a new version, v1.2.0.
For reference, you can find the packaged version here: https://www.npmjs.com/package/playwright-testmo-reporter/v/1.2.0
I have verified it locally as working. Please refer to the README to learn about the new testTitleDepth config attribute.
Let me know if it works! And if it does, you can close this issue! 😄

Closing this issue due to inactivity.


Update gapic-generator hash

:D
I'll wait for the github release v0.16.27 to be tagged before merging this

PTAL

I'm starting to hate all these hashes! Debugging things will be a nightmare :)

@alexander-fenster what would be better? like listing exactly which gapic generator commits are included?

@andreamlin Until we deprecate Artman, I believe we will benefit if we mention gapic-generator changes in the release notes, and update Artman version accordingly (not just patch release, but minor or major as well).
In general, the main problem is that Artman patch version bumps have nothing to do with the functionality being added/removed. For the users gapic-generator is a hidden part of the Artman black box, so whenever any new feature makes its way to gapic-generator, we probably should bump minor (for back-compatible changes) or major (for breaking changes) Artman version, with some description in the release notes, even though the Artman code itself does not change. This will probably break the 'letter' of the semantic versioning, but will surely support its 'spirit'.
Example: adding x-goog-request-params headers, in my opinion, was worth bumping a minor, not just a patch.

i will have to fix the bugs


fix(core): format-number for values when precision > 6

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows Conventional Commits
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Refactoring
[ ] Code style update
[ ] Build or CI related changes
[ ] Documentation content changes

What is the current behavior?
Closes # https://github.com/Tinkoff/taiga-ui/issues/1165
What is the new behavior?
Correction work with numbers when the precision is bigger than 6
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-9.2%) to 72.199%

Totals

Change from base Build<PHONE_NUMBER>:
-9.2%

Covered Lines:
139

Relevant Lines:
181

💛  - Coveralls

Thank you!


Bug: DebugBar after switch light and back to dark

PHP Version
8.2
CodeIgniter4 Version
4.4.2
CodeIgniter4 Installation Method
Composer (using codeigniter4/appstarter)
Which operating systems have you tested for this bug?
Windows
Which server did you use?
apache
Database
x
What happened?
https://github.com/codeigniter4/CodeIgniter4/pull/8063#issuecomment-1771967860
Another bug after switch to light and back again dark will change timeline-parent-open to dark

Steps to Reproduce
Open your debug bar in dark
Expected Output

Anything else?
No response

Reverting #7907 seems to solve the issue.
But I approved the PR, so it was okay at that time...
--- a/admin/css/debug-toolbar/_theme-dark.scss
+++ b/admin/css/debug-toolbar/_theme-dark.scss
@@ -150,10 +150,6 @@
         .timer {
             background-color: $g-orange;
         }
-
-        .timeline-parent-open td {
-            color: $t-dark;
-        }
     }
 }

Reverting #7907 and I use Edge, I was able to reproduce the issue.

But after switching to light mode and dark mode again, the issue was disappeared.
I cannot reproduce it now.

I feel switching the light/dark mode should not affect styles, but rather toggle them, or should it?
Is there some JS running on the mode change which alters the DOM elements ? Otherwise it does not make sense to me that the error only  occurs on switching back and forth.

I sent PR #8125


PR - 2 - Fixed testing issues and updated readmes

Build Pipeline -
https://github.com/OfficeDev/Microsoft-Teams-Samples/actions/runs/8060232197

Build Pipeline -
https://github.com/OfficeDev/Microsoft-Teams-Samples/actions/runs/8136527988


Async/Await

Has there been any discussion around introducing an async/await api around Clock.now? Would be really nice to have syntactically!

Yea that's a good idea. Unsurprisingly our large codebase isn't utilizing this much yet. Happy to review some PRs though


Scoreboard

The scoreboard component should show which players are on each team, how many points each team has, and how many of those points each player contributed.


Remove "api requires packaging" errors

Metadata resolution APIs work for packaged and unpackaged applications.  Some metadata lookup uses the package graph, but "do the right thing" if the process has no package graph.

@jonwis : Thanks for your contribution! The author(s) have been notified to review your proposed change.


No populate on request by default

@bluszcz - I have an issue here to resolve (I send it in my next PR) but I back to this PR I think we should no populate by default. I see some proper reasons not to do it:

If Strapi decided not to populate fields by default it means that's their way, so a user should declare populate for a collection.
Populate might be with different types and if it's not default option it's better to give user decisions about populate type (to discuss)
I work on get_data improvement for some collection and I need to add populate option to the method. It doesn't make sense to do populate request for all collections, so this is next reason not to populate by default

My solution
Instead of
def get_document(did)
  uri_document = URI("#{@site.endpoint}/api/#{endpoint}/#{did}?populate=#{populate}")
end

we can have
def get_document(did)
  uri_document = URI("#{@site.endpoint}/api/#{endpoint}/#{did}?#{populate}")
end

with populate method:
def populate
  "populate=#{@config["populate"]}" if @config["populate"]
end

Please let me know what do you think about this.

Another question is maybe we should in get_data request store this populate.
So we have two types of requests:

collection request (get_data method)
resource request (get_document method)

We don't use populate in collection (but we might), but we can just add populate to parameters and it works. But here we must duplicate this in the config (populate for collection, populate for resource) and it might be confusing.
This is an open question to discuss how we should treat populate parameter.

Related PR: https://github.com/bluszcz/jekyll-strapi-4/pull/27

I think about parametrs for a collection and a resource but this might be overwhelming


ENH: TransferFunction normalize should be optional

Is your feature request related to a problem? Please describe.
Hello,
I'm calculating control models using scipy.signal.TransferFunction. After connecting multiple in-series filters, the coefficients of my resulting TransferFunction numerator have values below 1e-14. Due to the "normalize" function included in the init of the TransferFunction all except one of my coefficients are eliminated, changing the resulting performance of my system.
Describe the solution you'd like.
It would be very interesting that the normalize function was optional or the "tolerance" value that considers a number as 0 was configurable.
Describe alternatives you've considered.
Example code that shows the issue:
from scipy import signal
signal.TransferFunction([2e-16, 1e-16],[1, 1],dt=1/20000)

TransferFunctionDiscrete(
array([1.e-16]),
array([1., 1.]),
dt: 5e-05
)

Additional context (e.g. screenshots, GIFs)
No response

See also gh-7345 and gh-11030; the latter notes that normalize is intended for continuous time transfer functions.
FWIW test scipy.signal.test.test_filter_design::TestSos2Zpk::test_fewer_zeros inadvertently demonstrates the problem with normalize; using the same 24th-order (!!) bandpass filter as there:
import numpy as np
from scipy import signal

sos = signal.butter(12, [5., 30.], 'bandpass',
                    fs=1200., analog=False, output='sos')

print('first numerator',sos[0][:3])
print('poles of first numerator',np.roots(sos[0][:3]))
print('sos2zpk of first section',signal.sos2zpk(sos[:1]))

gives (scipy 1.7.0):
first numerator [3.80521183e-15 7.61042367e-15 3.80521183e-15]
poles of first numerator [-1. -1.]
/home/rory/miniconda3/envs/work/lib/python3.9/site-packages/scipy/signal/filter_design.py:1631: BadCoefficients: Badly conditioned filter coefficients (numerator): the results may be meaningless
  warnings.warn("Badly conditioned filter coefficients (numerator): the "
sos2zpk of first section (array([0.+0.j, 0.+0.j]), array([0.90852419+0.06519289j, 0.90852419-0.06519289j]), 3.805211834355076e-15)


Mobile View for Tare Page

Description
@faithlos Gave use the wonderful design for a more responsive tare page on mobile
Figma

Updated mobile design for the Investment Details subsection

When the load more is pressed

mobile view changes:

[ ]  header not visible
[ ]  top section

[ ]  change back button from text to left arrow
[ ]  reduce side margins so that tags are next to each other more

[ ]  description

[ ]  shorter descriptions
[ ]  ‘read more’ button replaces with down arrow
[ ]  ‘read more’ button on bottom right

[ ]  other subsections

[ ]  put the colored underline back
[ ]  bigger subheaders
[ ]  shorter description
[ ]  change flow to vertical

[ ]  investment details table

[ ]  rotate orientation
[ ]  carousel type movement instead of scroll
[ ]  reduce ‘read more’ side padding


nightly release build is broken

the last nightly run executed on April 2nd. This coincides with the implementation of the Kokoro dedicated pool. Looking into what might have cause that to break the nightly release.

Can we turn on notifications and add a build orb to the readme for nightly?
On Thu, Apr 19, 2018 at 10:45 AM Etan Shaul<EMAIL_ADDRESS>wrote:

the last nightly run executed on April 2nd. This coincides with the
implementation of the Kokoro dedicated pool. Looking into what might have
cause that to break the nightly release.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-intellij/issues/2041#issuecomment-382763830,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHf5HazWZ2uGAioq_bDXllq-0ol0_bhQks5tqKMlgaJpZM4Tb6No
.

a build orb for sure. What notifications do you mean? Email?

Yeah, I haven't noticed any emails from that build failing.
On Thu, Apr 19, 2018 at 10:48 AM Etan Shaul<EMAIL_ADDRESS>wrote:

a build orb for sure. What notifications do you mean? Email?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-intellij/issues/2041#issuecomment-382764551,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHf5HeYFduaTykHymhxJPAc9nb_cxC2-ks5tqKOlgaJpZM4Tb6No
.

oh - it hasn't ever failed. it just stopped running according to the cron I specified (for some unknown reason).
notifications should already be on.

should be fixed now.
Per Kokoro team, there is a bug - b/74211584 - that requires you to manually trigger the cron enabled job after migration to get it working again.
Leaving open until I can confirm that its working again.


support for Plot 0.6

Hello, thank you for supporting Observable Plot! I've published an Observable notebook to demonstrate using olli with Plot.
Note that we've changed the internal representation of channels between Plot 0.5 and Plot 0.6 — the following addition to packages/adapters/src/ObservablePlotAdapter.ts should help to support both versions:
diff --git a/packages/adapters/src/ObservablePlotAdapter.ts b/packages/adapters/src/ObservablePlotAdapter.ts
index 0c003bd..d3450da 100644
--- a/packages/adapters/src/ObservablePlotAdapter.ts
+++ b/packages/adapters/src/ObservablePlotAdapter.ts
@@ -45,7 +45,7 @@ function plotToFacetedChart(plot: any, svg: Element): FacetedChart {
         plot.facet.y ?
             plot.facet.y :
             plot.facet.x :
-        plot.marks.find((mark: any) => mark.ariaLabel === 'line').channels.find((c: any) => c.name === "stroke").value;
+        flatChannels(plot.marks.find((mark: any) => mark.ariaLabel === 'line').channels).find((c: any) => c.name === "stroke").value;
     if (hasFacets(plot)) {
         charts = new Map(Object.values(chartSVG.children)
             .filter((n) => n.getAttribute('aria-label') === 'facet')
@@ -112,7 +112,7 @@ function plotToChart(plot: any, svg: Element): Chart {
 function parseAxis(plot: any, svg: Element): Axis {
     const axisType = svg?.getAttribute('aria-label') === 'y-axis' ? 'y' : 'x'
     const plotMark = plot.marks.filter((mark: any) => mark.ariaLabel !== 'rule')[0]
-    const channel = plotMark.channels.find((c: any) => c.scale === axisType)
+    const channel = flatChannels(plotMark.channels).find((c: any) => c.scale === axisType)
     const field: string = typeof channel.value === 'object' ? channel.value.label : channel.value
     const ticks: string[] = Object.keys(svg.children).map((k: string) => {
         const cObj: Element = svg.children[parseInt(k)]
@@ -152,7 +152,7 @@ function parseAxis(plot: any, svg: Element): Axis {
  */
 function parseLegend(plot: any, svg: Element): Legend { //TODO: Does not support 'ramp' legend types when the legend is rendered as an SVG
     const plotMark = plot.marks.filter((mark: any) => mark.ariaLabel !== 'rule')[0];
-    const channel = plotMark.channels.find((c: any) => c.scale === 'color'); // TODO channel hardcoded to color
+    const channel = flatChannels(plotMark.channels).find((c: any) => c.scale === 'color'); // TODO channel hardcoded to color
     const values: string[] = Object.keys(svg.children).map((k: string) => {
         let c = svg.children[parseInt(k)];
         if (c.nodeName !== 'STYLE') {
@@ -226,3 +226,7 @@ function plotMarkToOlliMark(m: string): OlliMark | undefined {
             return undefined
     }
 }
+
+function flatChannels(channels: any) {
+    return channels.find ? channels : Object.entries(channels).map(([name, chan]) => ({...chan as {}, name}))
+}

Hope this helps. As you can see I still have to find my way in this project — I'll try and contribute in a more efficient manner next time. I'm also happy to answer any question re: Plot, and open to suggestions that could help bridge the gap.

Hi Fil, thanks so much for taking the time to do all of this! I've added a link to your example notebook to the quickstart docs on the website, and made your suggested changes to the adapter. I'll plan to publish the patch within the next day or so, pending a few other fixes I want to make too.
Really appreciate your interest, and I hope we can chat a bit more about the kinds of information Olli needs for accessibility, and potentially making it easier to surface these things in the Plot API. I'm hoping to add more documentation about adapters soon too, which hopefully will help structure that conversation.

fixed in 3d1b4e0f4131da85a4dd3e74d3c7d00b9033d3bb – 7f4d8409dd9dd1026a34ce12cf7bf2ada1903a4a


Feature request: Check for data checksums in old postgres cluster

I had to redo (cleaning up everithing leftover) the upgrade because my old  postgres (version 16) used data checksums
------------------------------------
New database initialisation complete
------------------------------------
---------------------------------------
Running pg_upgrade command, from /var/lib/postgresql/data
---------------------------------------
Performing Consistency Checks
-----------------------------
Checking cluster versions                                     ok

old cluster uses data checksums but the new one does not
Failure, exiting

Starting pgautoupgrade with -e POSTGRES_INITDB_ARGS=-k solved the problem.
It would be great if the script could check if data checksums is enabled on the old cluster and use the correct parameter (-k) in the initdb command.
You can check if data checksum is on using show data_checksums
postgres=# show data_checksums ;
 data_checksums
----------------
 on
(1 row)

Oh, that's a very good idea.  We should definitely do that.
Thanks for the suggestion @zukka77. :smile: Sorry for you having to redo your upgrade. 😦


strict css parser

@maditya, this is the prototype of strict css parser for the html purifier, the idea is to enforce a strict lexical definition on tokens, like string etc.
for example, i redefined the definition of string here.
https://github.com/neraliu/css-js/blob/strict-css-parser/src/l/css.strict.l#L32

CLA is valid!

:+1: looks good


Small package of fixes for migration process

Archive Migration tooling consist of two applications:

berkeley_migration (#12906)
berkeley_account_tables (#14339)

During testing I found a couple of improvements/ small fixes which are necessary for testing and correct run:

extract parameters from berkeley_migration:
a) bucket location
b) number of batches for precomputed block fetch
c) corner case when there are no precomputed blocks to fetch

Closes #14551

ci-build-me!

Did you see my message about writing the migrated transaction hashes?
When migrated, the old transaction hashes will be written as new transaction hashes. In my message, I explained how to avoid that.
You might want to add a check that the migrated transaction hashes are syntactically the same as the old ones.

I agree with paul that you want to add some check that checks the migrated transaction hash is syntactically the same as before.

!ci-build-me

!ci-build-me

!ci-nightly-me

coverage: 41.516% (-20.0%) from 61.515%
when pulling 5552936a2365b75fa8154ea3ef32c9bd085073a7 on dkijania/migration_app_enh
into d3642cb6b5194bd87ebd1dac16360ee414e583e4 on berkeley.

!approved-for-mainnet


When copying the picture, there will be write picture copy is not in

The second picture cannot be copied. Is there any requirement for the picture address?
< div id="copyDiv1" data-clipboard-target="#copyDiv1" @click="clickCopy('#copyDiv1')" ref="copyDiv1">
< img src="https://www.baidu.com/img/bd_logo1.png" />123
< img src="http://img.api.yunzhinang.net/Fkb2Qt0sy3hrzCTJTuKq0VmqNAoS.png" />
< img src="http://www.mikeblog.cn/content/demo/copyimage/images/1.jpg" />

This error should be fixed by providing the appropriate DOM selector. Closing since it didn't provide a JSFiddle.


Register plugins

Hi I'm trying to register a plugin using Yasgui. I have created a new instance of Yasgui and trying to register the gallery plugin or chart plugin but am unable to find the import for it ?
Javascript.
    import Yasgui from "@triply/yasgui";

  useEffect(() => {

    // Yasr.registerPlugin(chart, plugin: Plugin)

    const instance = new Yasgui(ref.current, {
      requestConfig: { endpoint: "https://dbpedia.org/sparql" },
      copyEndpointOnNewTab: false,
      persistencyExpire: 0,
      persistenceId: null,
      yasqe: {
        readOnly: "nocursor",
      },
      yasr: {
      },
    });

    instance.getTab().yasqe.on("change", onChangeEditor);

    return () => {
      if (instance) {
        instance.getTab().yasqe.off("change", onChangeEditor);
        instance.destroy();
        instance = null;
      }
    };
  }, []);

Based on the instance above how would I register a plugin ?
The docs has the below code but it does not state where the Plugin import is coming from ?
Yasr.registerPlugin('gallery', plugin: Plugin ? )
Where would I import the gallery or chart plugin from ? https://triply.cc/docs/yasgui#gallery
As shown here https://yasgui.triply.cc/
I'd also like to disable the download button in the table response is there a way to disable this, apart from hide it using css ?

Hello and sorry for the confusion, we've just updated our documentation about the difference between the base plugins and the additional plugins.
Unfortunately, these plugins on yasgui.triply.cc, or within TriplyDB
As for disabling the download button, the quickest way is indeed to hide it with CSS. The first thing that comes to mind to nicely disable the download button is to extend the original table plugin like this
import Yasgui from "@triply/yasgui";

class MyTable extends Yasr.plugins.table {
  download=undefined
}
Yasgui.Yasr.plugins.table = MyTable

Let me know if the snippet helps! Have a nice day.

Thanks @GerwinBosch
I added the Yasgui.Yasr.plugins.table = MyTable to the top of my useEffect method before the instantiation of new Yasgui()
but it still does not hide the download button. I used css for now to hide it.
As for the plugins got it "they can't be used in a project programmatically."
Thanks for getting back to me.


[WIP] Spike manage super admin privileges v2

Alternative route for "allow GDS to invite another super admin"
Moves the super_admin boolean to organisation. So now to invite a super admin, Steve would just need to invite them to the "GovWifi Super Administrators" organisation.
Another few options: https://github.com/alphagov/govwifi-admin/pull/597

I feel like this a cleaner approach to the task.
Allowing a super admin to be regarded as an organisation could make it easier to work with for future features.
PR #597 (the alternative version) could also mean that the revoke/grant super admin links would be attached to every user in every organisation in the super admin view.
If by accident, a link was pressed unintentionally, it might be difficult to track the user and increase the risk of information exposure.
It could also increase the ambiguity surrounding which organisation a super admin can belong to, although this might be unlikely.


Intellisense Javascript for simple classes

Intellisense for javascript does not detect simple Javascript classes.
Given this (ES6/TypeScript) class:
class Greeter {
    constructor(message) {
        this.greeting = message;
    }
    greet() {
        return "Hello, " + this.greeting;
    }
}
var greeter = new Greeter("world");
greeter. // I get intellisense here

If I write the same class by hand in ES5 (the code generated by TypeScript):
var Greeter = (function () {
    function Greeter(message) {
        this.greeting = message;
    }
    Greeter.prototype.greet = function () {
        return "Hello, " + this.greeting;
    };
    return Greeter;
}());

var greeter = new Greeter("world");
greeter. // no suggestion here

I get no intellisense. I think this pattern should be detected automatically...

This issue was moved to Microsoft/TypeScript#7370

This works for me on master, and I believe there were some fixes recently around IIFE type checking.
@scharf, can you test this by installing typescript@next in a test package and in Code's settings.json temporarily setting "typescript.tsdk": ".../node_modules/typescript/bin", where ... is the path of your test npm package? (You'll have to restart Code or Reload Window to get the changes.)


Suggestions on how to split a line along the international date line

Hi,
I'm looking at the LineString class, and I essentially want to turn any segments that cross the international dateline into 2 separate segments. But I'm at a bit of a loss on where to start.
I'm doing this right now:
var fpLine = new Feature { Geometry = new LineString(points) };

I guess I will have to create two separate Feature, if I loop through the points and see 1 point to another cross the IDL. Is this the "correct" way to go about it? I want to try to do it in a way where I can contribute this code back into the mapsui project. Like a utility or helper that will return a list of LineString which then can be used to add as separate Features/layers.
List<LineString> lineSegments = GetIDLSegments(points);
foreach (var line in lineSegments) {
  var segmentFeature = new Feature { Geometry = line };
  layers.Add(segmentFeature);
}

Or I don't know if there's a way to do this within LineString, but it seems to me that it represents one connected line and should probably stay that way.

Actually I just saw MultiLineString, so I guess I should add the geometries to that.
https://datatracker.ietf.org/doc/html/rfc7946#section-3.1.9

The MultiLineString might be a good workaround.
A more fundamental problem is that Mapsui has no wrap-around. I think if we would have a good implementation for this you would have no problem. You would just have one Geometry and we would draw it at either side of the international dateline.

Yeah. I found this implementation and translated it over:
https://gitlab.com/avandesa/geojson-antimeridian-cut/-/tree/master/
It's working quite well, and I'm translating the tests over now. I'll include the code here for if someone finds it useful; or if you think there's value in a PR, I can do that. The main thing I'm unsure are the coordinate formats

using Mapsui.Geometries;
using Mapsui.Projection;
using System;
using System.Collections.Generic;

public static class MapUtils {

    /// <summary>
    /// Split the lines/points across the international date line
    ///
    /// Roughly follows:
    /// https://gitlab.com/avandesa/geojson-antimeridian-cut/-/blob/master/src/geometry/lines.js#L64
    /// </summary>
    /// <param name="points"></param>
    /// <returns></returns>
    public static IGeometry SplitLineForIdl(List<Point> points)
    {
        var crossings = CrossingPoints(points);
        if (crossings == null || crossings.Count == 0) {
            return LatLonPointsToLineString(points);
        }

        var newLine = new List<List<Point>>();

        // Add the first segment
        var segment = points.GetRange(0, crossings[0] + 1);
        if (segment.Count != 0) {
            newLine.Add(segment);
        }

        for (var i = 1; i < crossings.Count; i++) {
            segment = points.GetRange(crossings[i - 1] + 1, crossings[i] + 1);
            if (segment.Count != 0) {
                newLine.Add(segment);
            }
        }

        // And the last segment
        var lastIndex = crossings[crossings.Count - 1];
        var remainingCount = (points.Count - 1) - lastIndex;
        if (remainingCount > 0) {
            segment = points.GetRange(lastIndex + 1, remainingCount);
            if (segment.Count != 0) {
                newLine.Add(segment);
            }
        }

        // Get the points on the IDL
        for (var i = 1; i < newLine.Count; i++) {
            // Get the left point on the left and right
            var left = newLine[i - 1];
            var lastLeft = left[left.Count - 1];
            var right = newLine[i];
            var firstRight = right[0];

            var intersect = AntiMeridianIntersect(lastLeft.ToDoubleArray(), firstRight.ToDoubleArray());
            left.Add(new Point(180 * Math.Sign(lastLeft.X), intersect));
            right.Insert(0, new Point(180 * Math.Sign(firstRight.X), intersect));
        }

        // Now convert it to the proper mercater projection
        var multiLineString = new MultiLineString();
        foreach (var linePoints in newLine) {
            multiLineString.LineStrings.Add(LatLonPointsToLineString(linePoints));
        }

        return multiLineString;
    }

    /// <summary>
    /// Convert our lat/lon point into a spherical mercator point to display on the map
    /// </summary>
    /// <param name="point"></param>
    /// <returns></returns>
    public static Point Wg84PointToMercator(Point point)
    {
        return SphericalMercator.FromLonLat(point.X, point.Y);
    }

    /// <summary>
    /// Convert a list of lon/lat points into a LineString but converted to the right projection
    /// </summary>
    /// <param name="points"></param>
    /// <returns></returns>
    public static LineString LatLonPointsToLineString(List<Point> points)
    {
        var mercatorPoints = new List<Point>(points.Count);
        foreach (var p in points) {
            mercatorPoints.Add(SphericalMercator.FromLonLat(p.X, p.Y));
        }

        return new LineString(mercatorPoints);
    }

    /// <summary>
    ///
    /// </summary>
    /// <param name="lon"></param>
    /// <returns></returns>
    public static double BringLonWithinBounds(double lon)
    {
        var ret = lon;
        while (ret > 180) {
            ret -= 360;
        }

        while (ret < -180) {
            ret += 360;
        }

        return ret;
    }

    /// <summary>
    /// Checks if the line drawn between two points would cross the antimeridian.
    ///
    /// https://gitlab.com/avandesa/geojson-antimeridian-cut/-/blob/master/src/util.js
    /// </summary>
    /// <param name="lonA"></param>
    /// <param name="lonB"></param>
    /// <returns></returns>
    public static bool StraddlesAntimeridian(double lonARaw, double lonBRaw)
    {
        if (lonARaw - lonBRaw >= 360 || lonARaw - lonBRaw <= -360) {
            return true;
        }

        var lonA = BringLonWithinBounds(lonARaw);
        var lonB = BringLonWithinBounds(lonBRaw);

        if (Math.Sign(lonA) == Math.Sign(lonB)) {
            return false;
        }

        return lonA > 0 ? lonB < -(180 - lonA) : lonA < -(180 - lonB);
    }

    /// <summary>
    /// Finds the points at which a list of coordinates crosses the antimeridian
    ///
    /// https://gitlab.com/avandesa/geojson-antimeridian-cut/-/blob/master/src/util.js#L51
    /// </summary>
    /// <param name="points"></param>
    /// <returns></returns>
    public static List<int> CrossingPoints(IList<Point> points)
    {
        var ret = new List<int>();
        for (int i = 1; i < points.Count; i++) {
            var a = points[i - 1].X;
            var b = points[i].X;

            if (StraddlesAntimeridian(a, b)) {
                ret.Add(i - 1);
            }
        }

        return ret;
    }

    /// <summary>
    /// Make a longitude positive
    /// </summary>
    /// <param name="lon"></param>
    /// <returns></returns>
    public static double MakeLonPositive(double lon)
    {
        return lon < 0 ? MakeLonPositive(lon + 360) : lon;
    }

    /// <summary>
    /// Calculates the latitude at which a line drawn between two coordinates would intersect
    /// the antimeridian.Assumes that:
    ///
    /// The line would cross the antimeridian
    /// -180 < lon{A,B} < 180
    /// -90 < lat{A,B} < 90
    ///
    /// https://gitlab.com/avandesa/geojson-antimeridian-cut/-/blob/master/src/util.js#L87
    /// </summary>
    /// <param name="pointA"></param>
    /// <param name="pointB"></param>
    /// <returns></returns>
    public static double AntiMeridianIntersect(double[] pointA, double[] pointB)
    {
        var lonANorm = MakeLonPositive(pointA[0]);
        var lonBNorm = MakeLonPositive(pointB[0]);

        var slope = (pointB[1] - pointA[1]) / (lonBNorm - lonANorm);

        return slope * (180 - lonANorm) + pointA[1];
    }
}

Thanks for the code, this can already help some users.
If this would be a merge request it should go into master and there we replaced Mapsui.Geometries with NTS, so that would alter this MR.
About the projection. This is indeed not something to do at this level. It is something to configure in the layers. There has been a big change in projection in master. There the ProjectingProvider should be used. See here: https://mapsui.com/documentation/projections.html

Thanks - I was reading through the docs and that looked new to me. I wasn't sure how it was applied to WriteableLayer, which is what I use, but I guess I can look into that on the next release. I will figure out how to adapt this, though.

WriteableLayer indeed complicates this. In that case you need to do a transformation in the other direction when writing. Perhaps the ProjectingLayer should be extended to support that.


Your docker file has small bug

COPY ./sql-client/docker-entrypoint.sh /
you are missing ./ due to this when doing docker-compose up --build sql-client throws error

I am on windows 10 machine
On Mon, Jan 9, 2023 at 7:21 AM Francesco Tisiot @.***>
wrote:

Hi @soumilshah1995 https://github.com/soumilshah1995,
What platform are you in? Seems to work fine in my build
—
Reply to this email directly, view it on GitHub
https://github.com/aiven/sql-cli-for-apache-flink-docker/issues/27#issuecomment-1375547932,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJMF5P3J4GYVCQY2EAAVXE3WRP7GLANCNFSM6AAAAAATRJF4YI
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Thanking You,
Soumil Nitin Shah
B.E in Electronic
MS Electrical Engineering
MS  Computer Engineering
+1-646 204 5957

Should be fixed now!


[feature] "Go To Inheritor" or "Go To Derived Symbols"

Suppose I have a BaseClass. It would be nice to be able to go to any places in the code inherit from BaseClass.
F.e., if I see BaseClass in the code, I could run Go To Inheritors (or something similarly named) and see all sub classes, and navigate to them.
It'd also be sweet to be able to Go To Inheritor Reference Sites or similar, to be able to see all places that use BaseClass in any inherited fashion (so for example instead of just navigating to code like class SubClass extends BaseClass, it would also show locations for code like const s = new SubClass, const a = new SubClass2, etc).
This would great improve code navigation.
For more example of the idea, here's the same question for Visual Studio:

https://stackoverflow.com/questions/282377

Someone pointed to Resharper which adds a feature similar to the Go To Inheritor feature I just described (but not the Go To Inheritor Reference Sites):

https://www.jetbrains.com/help/resharper/Navigation_and_Search__Go_to_Inheritor.html

/duplicate of https://github.com/microsoft/vscode/issues/47126


Fix grammatical issue

Fix grammatic issue

Signed CLA.

Thanks for opening this PR, @appledaily0 but I believe the grammar is correct as is. Closing this issue.


Combine PRs security fix

I'm the maintainer of https://github.com/hrvey/combine-prs-workflow and we just made a new release - https://github.com/hrvey/combine-prs-workflow/releases/tag/1.2.0 - to fix a potential injection attack based on a PR with a malicious branch name. I wanted to let you know since a GitHub search showed me you're using a prior version.

Thank you!

:tada: This issue has been resolved in version 3.4.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Having issues with testing post-exporting

Hello team! I've successfully exported my model with the two files, onnx.json, and onnx. However, when I attempt to write this after exporting:
echo 'This is a test.' |    piper -m /home/suhayb/Desktop/RealTraining/piper/out-train/lightning_logs/version_9/checkpoints/suhayb-real-voice_high.onnx --output_file /home/suhayb/Desktop/test.wav
It gives me this message:
Unknown option -m
Anyone know a quick solution? Thank you in advance!

nevermind, found a solution.
Need to just install piper using -
https://github.com/rhasspy/piper/releases/download/v1.2.0/piper_amd64.tar.gz if linux ofcourse
and just run the command -
echo 'Welcome to the world of speech synthesis!' |
./piper/piper --model model_name.onnx --output_file welcome.wav


Fixed mentorship form's description textarea to have placeholder text.

resolves #101

Looks good to me +1


refactor: cleanup transactions files

non-breaking
@stacks/transactions:

move functions from common to files: signature and address
auto-cleanup imports

Closing in favor of another cleanup


Push new release to PyPi

Can you add push a new release to PyPi which now includes the Python 3 support?

Bump

Somethings weird with the version in PyPi.
unifi===fatal-.Not.a.git.repository.-or.any.parent.up.to.mount.point.-tmp-Stopping.at.filesystem.boundary.-GIT-DISCOVERY-ACROSS-FILESYSTEM.not.set-.

:+1: uploaded 1.2.3


Add LICENSE file

Fixes: #16
Signed-off-by: Valentin Rothberg<EMAIL_ADDRESS>

I guess we could also remove the license mentioning within the README.md since it's now obvious.

I guess we could also remove the license mentioning within the README.md since it's now obvious.

I'll do that in a second PR :^)


nom rewrite and AST node locations

Hi,
I've noticed you're trying a rewrite in nom, I'm wondering if the plan would be to also include position spans for each of the parsed AST nodes (like in the Go implementation).
Perhaps by using something like nom_locate

Hi there!
Yes, I'm currently evaluating the replacement of pest with nom, mainly to see how easy it is to replicate the current parser and if it makes implementing some improvements easier. I'm planning support for emitting the complete AST of an HCL document at one point, mainly to support use cases like https://github.com/martinohmann/hcl-rs/issues/112. This would also include span information.
I'm still undecided if I want this to be part of the general purpose parser that just emits structural elements, expressions and templates (like it does right now). The overhead might harm performance. If this is the case, I might end up with two parsers, one for the common use case, and a specialized one that emits the full AST including comments and whitespace. These two could share a lot of code of course, but the API and data types emitted would differ.
I'm interested to hear what your use case would be for the location spans!

I base my parsing on hcl::structure::body::Body and having location spans would enrich my error messages

Our use case is very much like a linter. We're building a security scanner for HCL files. So:

We deserialize to get the attributes, blocks, as data
We lint the data with a datalog-like language (OPA/rego)
Then when finding a bad value we want to point the user into the line:col coordinate to check for these issues

As a security linter we're focused on the values inside these files, and the composition of it, meaning the syntax might be perfectly fine.
Having location spans can support any linting usecase and not just a security linting example.
Hope that makes sense - LMK if you'd like some more detail of exactly how we do it 👍

Thanks for the feedback, this makes complete sense to me. I'll try to make span information available as part of the nom rewrite. I was already aware of nom_locate and will probably use it to track the spans.

Great!
If I can be of any help -- feel free to pull me in :)

@jondot Hey there, I'm happy to announce that I released the first version of hcl-edit, which also includes span information (accessible via the hcl_edit::repr::Span trait). There are still rough edges and usage examples and a lot of docs are missing, but the parser is functional and even over 100% faster than the original one in hcl-rs.
I'll continue to improve this crate and will also integrate it with hcl-rs, but this will take some more work. Happy to receive some initial feedback if you want to try it out :).
https://docs.rs/hcl-edit


install does not work with latest react-native

I tried installing today using a fresh project:
$ react-native -v
react-native-cli: 2.0.1
react-native: n/a - not inside a React Native project directory
$ react-native init ReduxFormsTest
$ cd ReduxFormsTest
$ react-native run-ios
$  npm i -s react-native-clean-forms
(SIMULATOR ERROR)
If I now embed the sample code and reload the simulator I get this error:
Unable to resolve module react-native-clean-form/redux-formfrom/Users/tomasvykruta/Documents/playground/ReduxFormsTest/forms_test.js`: Module does not exist in the module map or in these directories:
/Users/tomasvykruta/Documents/playground/ReduxFormsTest/node_modules/react-native-clean-form
,   /Users/tomasvykruta/Documents/playground/node_modules/react-native-clean-form
This might be related to https://github.com/facebook/react-native/issues/4968
To resolve try the following:

Clear watchman watches: watchman watch-del-all.
Delete the node_modules folder: rm -rf node_modules && npm install.
Reset packager cache: rm -fr $TMPDIR/react-* or npm start --reset-cache.

RCTFatal
-[RCTBatchedBridge stopLoadingWithError:]
__25-[RCTBatchedBridge start]_block_invoke_2
_dispatch_call_block_and_release
_dispatch_client_callout
_dispatch_main_queue_callback_4CF
CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE
__CFRunLoopRun
CFRunLoopRunSpecific
GSEventRunModal
UIApplicationMain
main
start
0x0
Next I install redux-forms:
$ npm i -s redux-form
``
Now I get this new error:
Unable to resolve module react-native-clean-form/redux-formfrom/Users/tomasvykruta/Documents/playground/ReduxFormsTest/forms_test.js`: Module does not exist in the module map or in these directories:
/Users/tomasvykruta/Documents/playground/ReduxFormsTest/node_modules/react-native-clean-form
,   /Users/tomasvykruta/Documents/playground/node_modules/react-native-clean-form
This might be related to https://github.com/facebook/react-native/issues/4968
To resolve try the following:

Clear watchman watches: watchman watch-del-all.
Delete the node_modules folder: rm -rf node_modules && npm install.
Reset packager cache: rm -fr $TMPDIR/react-* or npm start --reset-cache.

RCTFatal
-[RCTBatchedBridge stopLoadingWithError:]
__25-[RCTBatchedBridge start]_block_invoke_2
_dispatch_call_block_and_release
_dispatch_client_callout
_dispatch_main_queue_callback_4CF
CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE
__CFRunLoopRun
CFRunLoopRunSpecific
GSEventRunModal
UIApplicationMain
main
start
0x0
`
I can wipe that directory:
rm -rf node_modules/react-native-clean-form/redux-form
And I get the same error.

Thank you for the report, I try to test this soon.
I haven`t upgraded my current project to 0.43, but will try this :)

I tried this now. If you use npm install https://github.com/esbenp/react-native-clean-form.git everything installs without your error. The npm package is not up to date. If you look at https://www.npmjs.com/package/react-native-clean-form there is version 0.1.0 instead of 0.3.0

Opened a pull request that is work in progress actually to update the example app to RN 0.43.3. The app worked but looks broken because the react dependency to the beta version is a problem and they worked on the ScrollView that is used by the Form component.
Try to fix that soon. But I am also a bit busy.

I believe this is fixed.
I have:
aj$ react-native -v
react-native-cli: 2.0.1
react-native: 0.44.0

And installed react-native-clean-form today and got:
"react-native-clean-form": "^0.4.1",

And no issues. :)


implement function that wait for a "fully" bootstrapped minion, dinamically

look here:
https://github.com/SUSE/spacewalk-testsuite-base/blob/master/features/centos_salt.feature#L19
the problem is that at moment all minion we bootrap , do after the bootstrap pkg and hw refresh.
We should wait until  the hardware and pkg refresh are done, sucessefully. In this way we are also 100% sure that we boostrap right a minino at moment the gui doesn't provide this

this function for more flexibility, should just wait for a list of  named event, via xmlrpc
 wait for success
 	Hardware List Refresh
	Package List Refresh
        Apply states

A first try : https://github.com/SUSE/spacewalk-testsuite-base/pull/290


Not working in top level domains and sub domains

i am working on a project the code works fine in my own local server i.e http://localhost:5173 but when i build it and deploy to demo.projectname.com the pop shows and closes automatically after selecting an account. I have made sure that the javascript origin and the redirect uri are all correct but still the issue persists.
The error shows
message:
"Popup window closed"
stack:
"Error: Popup window closed\n    at new ap (https://accounts.google.com/gsi/client:224:388)\n    at iq.B (https://accounts.google.com/gsi/client:274:198)\n    at mc (https://accounts.google.com/gsi/client:56:389)\n    at _.l.dispatchEvent (https://accounts.google.com/gsi/client:55:340)\n    at _.Ne.G (https://accounts.google.com/gsi/client:125:179)"
type:
"popup_closed"
can you please look into this?

@gosunoobie sorry I am not able to replicate this issue. Ensure that your popup window isn't being blocked by popup blockers. Also try disabling all browser extensions.

thanks for the reply, it's a very weird issue but i think that its due to the redirect url mismatch just to make sure if i have a javascript origin set to demo.turbo.com then what should the redirect uri should be in the google api console?

@gosunoobie Does this issue still exists? Redirect uri is something you provide which will handle the response from google after a successful login, redirect flow is entirely different where instead of popup you will leave from your application and redirected to a google login page, and this only happens when you set ux_mode to 'redirect' (like this). Please re-open if issue still exists


CODE OF CONDUCT FILE

Hi @djm-1
There is no code of conduct file in this repository
code of conduct is a file that will be required to a open source project for the contributors to follow the decorum
Kindly let me know if you can assign me this issue

Keep going!!!

Keep going!!!

Hi @djm-1
Kindly pls do add DWOC and Level-3 or Level-2 label

Hi @djm-1
Kindly pls do add DWOC and Level-3 or Level-2 label

Yes👍

Yes👍


Explain that Kibana web interface communicates directly with ElasticSearch using AJAX requests

Replaced by https://clusterhq.atlassian.net/browse/FLOC-840
The ELK example should explain that the Kibana application does not require links because its web UI uses AJAX requests to communicate directly with the ElasticSearch server.
Furthermore, we might mention that Flocker's routes allow these AJAX requests (which are sent to the same origin as the Kibana server) to be transparently forwarded to the node hosting ElasticSearch if it is moved.
XXX: Check how Kibana AJAX requests work. They connect to a different port than the web server so why aren't they restricted by same-origin policy? Web Sockets maybe?

http://en.wikipedia.org/wiki/Same-origin_policy

We are moving our development planning to JIRA. This issue is now being tracked at https://clusterhq.atlassian.net/browse/FLOC-840. You are welcome to file additional issues in GitHub if that's easier for you.


Add test to make sure WATCH works

I've set the LIMIT 0, because I was unable to create another connection, and send INSERT INTO on the test suite, but I've tried locally, and it works fine.

Closes https://github.com/long2ice/asynch/issues/74

Please fix ci first, thanks!

Done. 👍


Credentials references

Allow users to add references to Authentication records in their Workflow#credentials property without having to embed the credentials directly in the record.
This uses a JSON Path style so for example: {"username" => "$.service-now.userid", "password" => "$.service-now.password"} would query for an authentication record with the name service-now and then retrieve the userid and password fields.
It is critical that the credential names be unique within the tenant, something which isn't guaranteed by the current Authentication model so I've added the validation to the Workflows::AutomationManager::Authentication subclass.
I'm also attaching the credential records to the seeded Workflows::AutomationManager here so they have something to relate to but that isn't strictly required for this to work.
TODO:

[x] Enforce RBAC when querying the credentials (tenant+user)

Depends:

[x] https://github.com/ManageIQ/manageiq-providers-workflows/pull/12
[ ] Add miq_group_id to workflow_instances

Is there a way for us to create a foreign entity (hopefully one that has rbac already on it) and let the user use that? GenericProvider (new) or GenericObject comes to mind.

I think that greatly complicates the authoring of Workflows, since rather than $.one-unique-field they'll have to deal with multiple intermediate models each with different rules for uniqueness.  Uniquely referencing a VM vs a Host vs an EMS vs a Generic Object are all slightly different.

Yes, this does seem notably more complicated.
ok, so I'm thinking the authtype will dictate how the workflow connects to a host. Some use ssh keys, and others use username/password. Think that would be out of the hands of the workflow author and a connect($authrecord) would be used
We were talking about supporting an existing ems (which has multiple authentication types).
The secondary object (host, vm, generic) seems the only addition of complexity though. But it is taking complex and increasing it.
For the existing solution, the user would see what in the dropdown?
"#{ems.name} - #{object.name || ""} - #{auth_type}" maybe userid
Almost feel like ems_id needs to be in authentications. (yes, many times this will be the resource_id or indirectly associated with the resource_id)

I think I'd rather create a new Credentials model if we aren't able to use Authentications due to Rbac/Tenancy.
So the question is, are we not able to use Authentications due to Rbac/Tenancy?

Now that we're using ConfigurationScriptBase there is an authentications join-table, I figure to get this in without all of the RBAC/OwnershipMixin stuff we could have whoever is managing the Workflow associate credential records with it "manually" for now.  Really just want to unblock this PR somehow.

@Fryguy could you take a look at this one?

For global providers (automate, workflows), they are seeded and linked to the global tenant. we can only use Rbac directly on this object
For tenant based providers (vmware, openshift), do we want to use rbac from the ems?
Or is it possible copy it across from the provider to the auth record? So it is simpler/consistent?
Lets get this nailed down.
This has stalled us (Adam) too much


Document API for getting all update runs under update location

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-js

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

Automation for azure-sdk-for-node
Nothing to generate for azure-sdk-for-node

Automation for azure-sdk-for-java

Can one of the admins verify this patch?

@yudomsft any updates? If you're not ready to edit the PR, I may close it and you can reopen it when you're back to it. Thanks!

Closing per no activity, please add a comment or reopen when you're back to it. Thanks.


[WIP] FlightTaskOrbit: Improvements

My goal is to take the orbit flight task from a draft to a usable state which can be triggered via SDK.
This pr is to fix bugs and improve flight performance.

I'm currently adding the MAVLink interface for Orbit such that it's available with newest QGC daily.
Command: https://github.com/mavlink/mavlink/pull/924
UI: https://github.com/mavlink/qgroundcontrol/pull/6610

In SITL the execution of the MAVLink command now works. Needs still a lot of testing especially also for corner cases like failsafes.


invalid python plugins

It seems the python plugins are invalid on this page:
https://github.com/pinpoint-apm/pinpoint-c-agent/blob/master/DOC/PY/Readme.md

1,my pinpoint version is v2.3.3
2,I use flask,so I follow the link above and download pinpoint-python-plugins
3,It dosen't show call chain map,every service just show itself alone.
4,I download pinpoint-c-agent-0.4.6.tar.gz and decompress it ,replace python plugin with pinpoint-c-agent-0.4.6/plugins/PY/pinpoint(much appreciate to Evelyn)
5,finally,it works ok,the relationship with each service is showing.

Doc Fixed!
https://github.com/pinpoint-apm/pinpoint-c-agent/pull/391
The latest plugin: https://github.com/pinpoint-apm/pinpoint-c-agent/releases/download/V2020.12.17/pinpoint-python-plugins.tar.gz


Privacy & Display screen: Need text instead of Lorem ipsum.

"Some of these settings will have to be set again in case you Recover your app"
@cakesoft-swati

Verified this issue on keeper app v 1.1.12(299)


Add support for response schema override via annotation

@pcl

It looks like a number of the changes here are simply to whitespace and / or code formatting. I'd rather not include those changes, or at the least, include them in a separate dedicated whitespace-converting commit.
Also, you've moved some test source files from the resources/ directory to test/ instead. Why was this necessary? Those test source files were intentionally kept out of the class path, to more closely model a real-world execution context.


Exception: The program BioNetGen was not found in the default search path(s)

I follow the instructions to install bionetgen to the letter, no matter what I do it does not work. I installed it with conda as well, but it doesn't help. I did the installation of the vs code plugin and perl, and .bngl files are able to open, but nope.
And no matter how often I dod 'pip install bionetget', I get get
Conda users can install BioNetGen using the following command:
conda install -c alubbock bionetgen

I also tried the conda thing, of course.

Are you working on Windows, macOS, or Linux? If Windows or Mac, are you working in a container or other virtual environment?

Windows, directly on my machine (no virtual environment)


Weird delay for onSave event

Looking at the output pane, it takes 7-10 seconds after I save the document to trigger the file write. The execution itself is fast, it's the notification that doesn't get sent by the client immediately.
Not sure if that's the intended behavior (debouncing) as I'm not familiar with the vscode extension library. Ideally we could speed this up.

Doesn't happen anymore, maybe it was because it was compiled in debug mode.. Will do more testing

I've also noticed delays in debug mode. Rust is notoriously slow in debug mode, so it wouldn't shock me if that's the whole problem. I'll close for now, but might reopen/open a new issue if the cause is different, or debug builds are too slow to be practical for testing.

No worries, from the logging I think it's actually the VSCode extension debug mode that is slow, because the notification doesn't get sent to the LSP during that time. Once the notification is received, the processing on the rust side is fast.


862 add canary guidance to site

Summary of the changes
Created a new canary section on the components page, which includes guidance and code examples for the canary date-input, as well as an index page which includes a link to the storybook.
Related issue
#862
Checklist

[x] I have manually accessibility tested any changes, if relevant.
[x] I have raised this pull request against the develop branch

Tested with canary-docs import and all is well!

there seem to be quite a few date-pickers in the images, which I wouldn't really expect on the date-input page. in particular the "when not to use" image is confusing as it says to use 3 separate fields, but then shows a date picker

Some of the guidance or component will need to be updated, for example:

There is guidance on potentially changing the placeholder text but this is not possible as it uses 3 'micro' inputs.
A screenshot shows an example where all the placeholders are not visible with a cursor present. When selecting within an input, the other two inputs will still display their placeholder text if not populated
By default, the helper text is 'use format ', but our images show 'Enter in format '
There is no prop to hide the helperText so we may need to update placeholder text 'do not use' images. A workaround is that the user adds an empty string to the helperText prop but it would not remove the element.

Some of the guidance or component will need to be updated:

There is guidance on potentially changing the placeholder text but this is not possible as it uses 3 'micro' inputs.
A screenshot shows an example where all the placeholders are not visible with a cursor present. When selecting within an input, the other two inputs will still display their placeholder text if not populated
By default, the helper text is 'use format ', but our images show 'Enter in format '
There is no prop to hide the helperText so we may need to update placeholder text 'do not use' images. A workaround is that the user adds an empty string to the helperText prop but it would not remove the element.

Have made amendments based on these. Screenshots and guidance has been updated. For the last bullet point, I would argue that it is still valid since the ability to add an empty string means that users could avoid displaying a helper text, and this guidance highlights why that is bad practise.

There seems to be an issue with the padding on .focus-indicator. If you view the component within the Storybook, the padding is correct but not on the guidance site. It looks like there is an issue with the cascading on the focus-indicator class. It wasn't there when I previously looked at the PR so it may be the Gatsby build doing something with the CSS.

There seems to be an issue with the padding on .focus-indicator. If you view the component within the Storybook, the padding is correct but not on the guidance site. It looks like there is an issue with the cascading on the focus-indicator class. It wasn't there when I previously looked at the PR so it may be the Gatsby build doing something with the CSS.

It's going to need more research to find out why Gatsby is importing the CSS in the wrong order (although there are many threads about it online so we're going to make the .focus-indicator rule in ic-date-input.css more specific. Will fix it on fix/ic-date-input-focus-indicator-fix

PR created to address the styling issue: https://github.com/mi6/ic-ui-kit/pull/1787


Move stream handling to an async method, clean up example

This is a pretty large commit, there are several things I was interested in achieving

Moving the handling of the Sink + Stream objects into an async context. It's much less error prone there because we are in the "control flow register" [1]
Exploring what an API which allows connecting (and disconnecting) mutliple streams to the repo dynamically would look like. This would fit much more naturally with the way I would expect people to use a library like this (e.g. accepting TCP connections from individual peers as a stream + sink).
Clean up the examples a bit so it's easier to understand what they are doing.

To that end the largest change is the RepoHandle type and its connect_stream method, which takes a stream and returns a future which must be driven to process messages for the given stream.
In the example code I've split the example up into multiple files, implemented the APIs for the relay server and the peer servers separately, and introduced an abstraction over the stream of messages from the relay peer (it's obviously still very incomplete as this is just an example).
I don't think we should merge this as is, not least because I think main has changed significantly, but hopefully it's useful to look at.
[1] https://without.boats/blog/the-registers-of-rust/

I have removed the collections from the main branch. I propose that I take this branch, rebase, and also adapt it to a "one stream per syncing connection" concept. Let's talk later... @alexjg

Closing since we moved on...


Remove characters from pool option and/or option to generate more than one team.

Hey, thanks for the awesome work - this is looking to be a very practical tool! There is one feature that I would like to see that hasn't been listed in the current issues yet.
I like doing abyss randomizers and I like to reroll new teams for every floor without reusing characters. This can be achieved by either adding the option of having characters be removed from the pool after they have been rolled once or by rolling mutliple teams at once (here it can be an option if characters can appear in multiple teams or not).
Keep up the good work!

Hi @Elive1,
Thanks for the feedback, having a customizable amount of teams to generate sounds like something I could implement. I'll consider it.
Unfortunately I don't think I'm going to have time for this project anytime soon.

Hey, can I work on it?

Feature to generate multiple teams at once released in 1.4.0


Add zstd compression level option

What does this PR do?
Adds a new config option to set the zstd compression level, defaulting to the default level (5). Previously the default level was always used.

Motivation
Using the default level has the potential to use up a lot more memory. Initial experiments show that using a lower level - ie 1, uses a lot less and still produces a significant improvement in compression ratio over zlib.

Additional Notes

Possible Drawbacks / Trade-offs

Given that this is a configurable option, no difference should be noticed in behaviour unless the user chooses to do so.
Describe how to test/QA your changes
Add the following config options to datadog.yml:
serializer_compressor_kind: zstd
serializer_zstd_compressor_level: 1

On a separate note, I think all these go.mod updates comes with the fact that you include DataDog/zstd in the config package, which I think we don't want: some builds don't want zstd around (CGo).
A trade-off would be to declare a const DefaultZstdCompressionLevel = 5 directly in config.go, WDYT?

/merge

/merge


i want the button work by use accepts_nested_attribute_for

the rails_admin.rb
config.model TagRecommend do
visible false
edit do
  field :tag do
    visible false
  end
  field :name
end

list do
  field :tag
  field :name
end

show do
  field :tag
  field :name
end

end
in the Tag model
has_many :tag_recommends
accepts_nested_attributes_for :tag_recommends
i want to show the add form by click the button,just like click button it will show the block to me,double click it will hide.
i try many ways ,it always not work.
i apprciate any help!

@yangman-tutu I don't really get your point. Can you rephrase a bit?

oh , i'm sorry .
in development environmen ,i want to add a object for Model by use rails_admin, when i click the add button, it will appear two forms, i wanna the one form.
how to fix this.


Fix bugs and update copyright signs on ANT manager app.

Fix crash on initializing Wi-fi Direct using Raspberry Pi 3 (#12)
Update libgstreamer_android.so to resolve CameraViewer issue (#12)
Test mode of ANT manager app is added
Large data mode (Wi-fi Direct) unit test is added to test mode
Resolve synchronization issue on Wi-fi direct device controller
Update copyright signs

After applying this patch, I can find that Wi-fi Direct connection and CameraViewer work well with Raspberry Pi 3.


Write client to samen.generated.ts instead of samen.ts

This generates the client to samen.generated.ts, giving the dev the possibility to create a samen.ts where the client is initialised.

Nice!


Announcements, News and Moving Away From Modals

Improved communication tools, less modals, and a bit of better editing support.

Deploy preview for openbuildservice ready!
Built with commit c85ac4cc8cdc5727d50f9e1b4f40340d4b06c3af
https://deploy-preview-256--openbuildservice.netlify.app

this is not about us. what changed or will change regarding annoucements, modals and menus for the people who read this?

I've just push a new commit extending the Announcement's section. There I explain the reasons and the improvements.

@danidoni I would adjust the publishing date in the filename and squash the commits :) besides that no complains :+1:

@danidoni I would adjust the publishing date in the filename and squash the commits :) besides that no complains +1

Date updated


Added 4 Tools

BLOKK wireframing font
Blobmaker vector creator
Death to Stock image service
LoremPixel stock image API

Checklist

[x] I made something good today 💐.
[x] I put links in the correct format: [Tool](link) — description.
[x] My description starts with the lower-case letter and ends with the full stop.
[x] I put the tool in the alphabetical order.

Optional

[x] I added the appropriate labels: free, open-source or mac.
[x] I put the tool only to one category.
[x] I followed Contribution Guidelines.

Wow, @aaroncawte! You added really nice tools, thanks a lot!