[INLONG-4986][Agent] Support MQTT Source

Prepare a Pull Request
(Change the title refer to the following example)

Title Example: [INLONG-XYZ][Component] Title of the pull request

(The following XYZ should be replaced by the actual GitHub Issue number)

About #4986

Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve?
Modifications
Describe the modifications you've done.
Verifying this change
(Please pick either of the following options)

[ ] This change is a trivial rework/code cleanup without any test coverage.

[ ] This change is already covered by existing tests, such as:
(please describe tests)

[ ] This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a follow-up issue for adding the documentation

@haibo-duan here we need to add MQTT Source to the JobProfileDto.class. You can refer to MongoDB in #5674.
maybe other sources you added in agent also add the job definition.

@haibo-duan here we need to add MQTT Source to the JobProfileDto.class. You can refer to MongoDB in #5674. maybe other sources you added in agent also add the job definition.

It has been modified. Please review it for me.


lowrie.py has unicode problems in Python 3

This line (from data_files/lowrie/lowrie_example.dat), does not decode correctly with utf-8, but I am not sure what encoding to use.
318-U1359A-002H-1-W-109              0.00     0.4  2.288E-3    73.5   -46.8     07/04/12;15:10;¡C;0.0000;microT;Saiko;G34; 1

switched to Latin-1, not sure this is a high quality solution, but it is working for now.

sio_magic should have this same problem, probably want to fix it in pmag.magic_read (try using utf-8, if it is broken, use latin-1)

Fixed in these commits:  29d5ce840ae0ee467b55255b5ab08545230f1486, 3a13e96e33819bfb1717fa0f4adc9a8d4edc88e3

New function pmag.open_file deals appropriately with missing file an improper encoding.  In general, this can be used instead of open() in future code.


Target overrides the ALWAYS_EMBED_SWIFT_STANDARD_LIBRARIES

When I run pod update:

[!] The *** [Debug] target overrides the ALWAYS_EMBED_SWIFT_STANDARD_LIBRARIES build setting defined in Pods/Target Support Files/Pods-bootmark/Pods-bootmark.debug.xcconfig'. This can lead to problems with the CocoaPods installation     - Use the$(inherited)` flag, or
- Remove the build settings from the target.
[!] The *** [Release] target overrides the ALWAYS_EMBED_SWIFT_STANDARD_LIBRARIES build setting defined in Pods/Target Support Files/Pods-bootmark/Pods-bootmark.release.xcconfig'. This can lead to problems with the CocoaPods installation     - Use the$(inherited)` flag, or
- Remove the build settings from the target.

I am using cocoapods 1.1.0.rc.2 with a deployment target of 10.0. I am also using alamofire 4.0. Any suggestions?
These people are also experiencing the same issue so far:
@GZaccaroni @jonybur @SMR @realmike33 @msilva
I thought I would make this it's own issue since no one seemed to be getting back to the closed issue (which totally makes sense). SEE ISSUE: #1531

@GZaccaroni @jonybur @SMR @realmike33 @msilva @PlayApple
Alright guys not the cleanest solution to the problem, but here is how I fixed it. Turns out this isn't really an Alamofire issue and shouldn't be on here specifically. Basically I went into the build settings and just cleared all the settings to default. So anything that was bold coloured, I selected and then I pushed delete. This is what worked for me.

Thanks for posting this - It really needs to go in the Cocoapods Readme.

Pleae check issue #5981 and follow the steps posted by johnantoni on 9 Jan 2017. It worked for my project with cocoapod dependencies. Xcode Version 8.1

This is fixed in CocoaPods 1.2.0, so there should be no need for that workaround anymore.

@jshier I'm still seeing this bug.  Im currently on cocoapods 1.2.1, any ideas?


20240124 blekeyboard improvement

Advanced mapping for BLEKeyboard mode.
I can now map, for each button, also special keys (defined in https://github.com/T-vK/ESP32-BLE-Keyboard/blob/master/BleKeyboard.h).
For each button I can also add a  modifiers (shift, ctrl, alt) and repeat  for n times.

I would like to do the same for Looper Mode. Do you think is better work on other branch ?
Or, if you merge this PR, I can create a new PR in the future from the same branch?  Never done before

@itarozzi I also have never done this before. I understood that I need to review and test in my repository and if approved, the change will be merged into my repository. My understanding would be that you can continue to develop on the same branch if you prefer, you can create another pull request later on this one.
I need to check in the next days when I have some time to check and approve. I will keep you updated here.


Relative Links not working

Relative links seems to be not working.
I tried this in a *.md file : [a link to another article](news2.md) a unfortunately the link to news2.md was not working.
That should be fixed. Then Firefox + markdown-viewer becomes the simplest way for setting up a small personal wiki !

I think there is no need is secure parsing as soon as our new HTML is not coming from server.
So just setting innerHTML is enough. Here is my modification (it is chrome/content/markdown-viewer.js). May be it will help someone.
window.addEventListener('load', function load(event) {
    window.removeEventListener('load', load, false);
    MarkdownViewer.init();
}, false);

function BrowserSetForcedCharacterSet(aCharset) {
    var wnd = (gContextMenu ? document.commandDispatcher.focusedWindow : window);
    if ((window == wnd) || (wnd == null)) wnd = window.content;
    const Ci = Components.interfaces;
    var webNav = wnd.QueryInterface(Ci.nsIInterfaceRequestor).getInterface(Ci.nsIWebNavigation);
    var docShell = webNav.QueryInterface(Ci.nsIDocShell);
    docShell.QueryInterface(Ci.nsIDocCharset).charset = aCharset;
    webNav.reload(nsIWebNavigation.LOAD_FLAGS_CHARSET_CHANGE);
}

if (!MarkdownViewer) {

  var MarkdownViewer = {

    init: function() {
        var appcontent = document.getElementById('appcontent');
        if (appcontent)
            appcontent.addEventListener('DOMContentLoaded', this.onPageLoad, true);
    },

    onPageLoad: function(aEvent) {
        var document = aEvent.originalTarget;
        var markdownFileExtension = /\.m(arkdown|kdn?|d(o?wn)?)(#.*)?(.*)$/i;

        if (document.location.protocol !== "view-source:"
            && markdownFileExtension.test(document.location.href)
            && document.contentType !== "text/html") {

            if (document.characterSet !== 'UTF-8') {
                BrowserSetForcedCharacterSet('utf-8');
                return;
            }

            var textContent = document.documentElement.textContent;
            // definitely safe as soon as HTML comes from safe function
            document.body.innerHTML = '
' + marked(textContent) + '
';
            document.title = document.location.href.split("/").pop();

            var link = document.createElement('link');
            link.rel = 'stylesheet';
            link.type = 'text/css';
            link.href = 'resource://mdvskin/markdown-viewer.css';
            document.head.appendChild(link);

            var meta = document.createElement('meta');
            meta.name = 'viewport';
            meta.content = 'width=device-width, initial-scale=1';
            document.head.appendChild(meta);
        }
    }

  };

}

Using innerHTML is a big no-no on http://addons.mozilla.org

Sure it may be. But HTML may also be intentionally malformed and we are ok with it. The main aim of safe parsing is to strip off js code. But in this case it may be unwanted (https://github.com/Thiht/markdown-viewer/issues/26).
As far as I see, this is kind of bug in FF. So FF sanitize does this:

does not allow relative links, so links should be resolved manualy before passing to parseFragment
does not allow links to local files
This seems nice and reasonable. But I do not know why it does not allow local links from if baseURI is local. Anyway, here is what works if FF allows such links:

function parseHTML(html, rootDocument) {

    var Cc = Components.classes;

    var ioService = Cc["@mozilla.org/network/io-service;1"]
         .getService(Components.interfaces.nsIIOService);

    var baseURI = ioService.newURI(rootDocument.location.href, null, null);

    var parser = Cc["@mozilla.org/xmlextras/domparser;1"]
               .createInstance(Ci.nsIDOMParser);

    parser.init(null, baseURI, baseURI);

    var doc = parser.parseFromString(html, "text/html");

    for (var i = 0; i < doc.links.length; i++)
        doc.links[i].setAttribute("href", doc.links[i].href);

    html = Cc["@mozilla.org/xmlextras/xmlserializer;1"]
         .createInstance(Ci.nsIDOMSerializer)
         .serializeToString(doc.documentElement);

    var parser = Cc["@mozilla.org/parserutils;1"]
         .getService(Ci.nsIParserUtils);

   // this guy strips off hrefs and reports security error in browser console
   return parser.parseFragment(html, 0, 0, baseURI, rootDocument.documentElement);

}

Anyway it fixes relative links in non local files.

It works now!


iPad/iPhone crashing on the big page

Version
2.3.2
Reproduction link
https://codepen.io/Bububo/full/qmPPOE
Steps to reproduce
iPhone 6 plus and iPad 3 are crashing on the big pages. iPhone 6s and iPad air 2 work ok.
I see this issue if I use a following javascript and div#app contains about 3000 DOM nodes or more.
new Vue({
	data() {
		return {
			test: 'test',
		}
	},

}).$mount('#app');

What is expected?
The user can see the page.
What is actually happening?
Crash. Message: A problem repeatedly occured on  {url}

iPhone 6s and iPad air 2 work ok.

It worked fine when I tested on the simulator (this kind of errors should also appear on the simulator)... Vue didn't even seem to have an impact on the speed of the page. If you manage to have a js stack trace from safari we may have a look but it looks like something related to your device
For big amount of content, you may want to use something like  https://github.com/Akryum/vue-virtual-scroller But It doesn't look like the error you have

@posva
It is crashing on the simulator also. I've tested on www.browserstack.com (iphone 6s safari)
Screenshot http://prntscr.com/f4dlt7

We are having this issue as well on one site with (at least) iPad model A1416. It's really hard to debug since the dev tools also crash. I can reproduce the problem with the codepen example provided by @bububo using the latest Vue version.
I reckon the problem lies when Vue hooks to the dom. On another website it barely works and Vue takes almost 30 seconds to initialize on the iPad. I'm afraid that with a couple more components introduced to the page it too will crash.
Can you please reopen this issue and investigate? @posva

Hi, we are having the same issue with Vue in the pages where we have a lot of components rendered, such as a list of <product></product>. It works fine on some phones and on some others it throws this error. I can reproduce it via Browserstack anytime. Here is a printscreen

@posva Can you please reopen this issue?
Thanks in advance

I have found the issue in our case, seems to be inline-template. After we moved the template into the .vue file, everything works fine now :)


Blocked a frame with origin ... from accessing a cross-origin frame.

Hello, I think the library is great for managing twitter login, but I have a problem, when I want to use the component to make the login, I get the following error Error: Blocked a frame with origin" http: // localhost: 3000 " from accessing to cross-origin frame. In my specific case I am using my localhost` to be able to use proof of concept, and I enabled the cors, but I can not make it work. Is there any indication on this? or I'm doing something wrong when using the library

Hi,
This should be working. Also in example folder you can find an example with Node.js/Express.js application and React application. Both are hosted on localhost, but on two separate ports. Also i've created tutorial that explains similar setup.
You shouldn't have any issues.

I currently manage a project in which everyone has work with their own server (backend and frontend), the part of the backend worked excellent for me, but when using the react component, I see that a window.open is created, to be able to open the twitter registration window, when entering the polling that window has problems with the cors permissions, so that part is the one that has stopped me, both on the backend server (api), as in the client enable the cors permission as you have it in the tutorial and I could not pass the setInterval cycle, some recommendation ?? , thanks for answering

Sorry for late response, but you should not have any other configuration for cors in frontend. Pay attention that you have correctly set up cors on backend.

Hello, I get the same issue, and my cors in the backend are correctly set... however the problem seems to come from the frontend that can not be able to get data from a popup without the same origine (like twitter.com). Did you found a workaround ?

Hi @Veldars,
I haven't manage to reproduce this issue. Do you use latest version of library? What is the value for credentials prop that you are using?
I just need more information about the issue.

Ok, thanks for your quick answer.
Yes I download the last version of the library.
I tried with every possible value for credentials, but the error is still there.
But for me I don't understant how it could work. I had read the code of your library and after some internet research, The parent window of a popup cannot access to anything from the popup if the origine of the popup is not the same that the parent window.
So if you try to get data like popup.location.hostname the browser will throw an error if the origin is not the same. I haven't found any solution for this kind of problème.
If you need it, here is my call of the TwitterLogin.

  <TwitterLogin loginUrl={`${Conf.BaseURL}auth/twitter`}
              onFailure={this.onFailed} onSuccess={this.onSuccess}
              requestTokenUrl={`${Conf.BaseURL}auth/twitter/reverse`}/>

I have removed the credential parameter because you do not use it in your examples.
Thanks

I modified the component to use the callback offered by tweet, since it was impossible to obtain the token directly to authenticate

Thanks, I did it too, but I'd like to understand how it could work without modification...

I think it's because of some change that twitter made in its way of authenticating

One question how do you end on different domain? Because, popup should go back to application. So I am creating popup from http://localhost:3000 and after authentication in popup will be opened http://localhost:3000/#callback? This is same host. What is your configuration for callback URL on Twitter application?

@josejaguirre, I'm experiencing the same issue, could you please elaborate a little bit on a workaround for it?

@sherlaimov hi! sure.
I used the process where the application redirects to a twitter login and then returns to the application using the callback I hope it serves you.
I was using node for that, i used https://github.com/drudge/passport-twitter-token library.
The steps you took were the following:
1.- Node Backend Use the library and create custom url for get request token from twitter link (https://api.twitter.com/oauth/request_token)
2.- React Component Edit getRequestToken from TwitterLogin and use the custom link and get the oauth_token prop
3.- React Component Redirect https://api.twitter.com/oauth/authenticate?oauth_token=${oauth_token}
4.- Twitter Setup Twitter app callback to return to your application
5.- Node Backend Setup authorization middleware using callback url.

@josejaguirre thank you for you response, I also use NodeJS, in fact I tried following through @ivanvs's tutorial on implementing Twitter Authorization with React, however, I'm using KoaJS and had to use the oAuth module to authorize my 'node-fetch' requests - all seem to be working just fine, except for the frontend/react part.
Can you please explain what kind of customized link should be used in getRequestToken? I'm fetching the following requestTokenUrl link  http://localhost:8080/auth/twitter/reverse in getRequestToken and it does return an oauth_token which is passed to a popup with location of https://api.twitter.com/oauth/authenticate?oauth_token=${data.oauth_token}&force_login=${this.props.forceLogin}.
My react app is running on port 3000, could the port difference be the reason causing the CORS issue?

If you are running application on port 3000, shouldn't your requestTokenUrl be something like this: http://localhost:3000/auth/twitter/reverse? In tutorial my backend was working on port 4000 so I have used this url: http://localhost:4000/api/v1/auth/twitter/reverse.

@ivanvs you are right, it is working now, thank you. It's clearly my gap in knowledge of what CORS really is.

Hey Guys i write a react componet to login to OPENID but have a error :
Failed to load https://sso.csoc.net/openid/authorize?response_type=code&client_id=948084&redirect_uri=https%3A%2F%2Flocalhost%3A4000%2Fopenid%2Fcallback&scope=openid auth_web openid profile email&state=qmICanszt40y7HQphWsmd7zg:
Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'null' is therefore not allowed access. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.
client React : https://localhot:3000
Expres server : https://localhot:4000 connect to server authentication
How i can  fix by pass CORS ?
i try set header :
const customHeader = {};
customHeader['Access-Control-Allow-Origin'] = '';
customHeader['Origin'] = '';
customHeader['Accept'] = 'application/json';
customHeader['Content-Type'] = 'application/json';

Hey Guys i write a react componet to login to OPENID but have a error :
Failed to load https://sso.csoc.net/openid/authorize?response_type=code&client_id=948084&redirect_uri=https%3A%2F%2Flocalhost%3A4000%2Fopenid%2Fcallback&scope=openid auth_web openid profile email&state=qmICanszt40y7HQphWsmd7zg:
Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'null' is therefore not allowed access. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.
client React : https://localhot:3000
Expres server : https://localhot:4000 connect to server authentication
How i can fix by pass CORS ?
i try set header :
const customHeader = {};
customHeader['Access-Control-Allow-Origin'] = ''; customHeader['Origin'] = '';
customHeader['Accept'] = 'application/json';
customHeader['Content-Type'] = 'application/json';

same issue +1


Optimize - 24 Mar 2019

Pull Request Test Coverage Report for Build 139

36 of 36 (100.0%)  changed or added relevant lines in 24 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 90.664%

Totals

Change from base Build 138:
0.1%

Covered Lines:
310

Relevant Lines:
330

💛  - Coveralls


Backspace button is not working on some Android devices (Samsung especially)

On a few android devices (so far all from Samsung) the backspace on the virtual keyboard of the device, doesn't so anything.
It doesnt delete chip nor text.
The same code works perfectly on all iOS devices and other Android devices.
I noticed that on Web, the backspace of a real keyboard, doesn't do anything.
So I tried to do this, in the chips_input code, I added this snippet:
RawKeyboardListener(
    focusNode: _effectiveFocusNode,
    onKey: (event) {
  if (event.isKeyPressed(LogicalKeyboardKey.backspace)) {
           setState(() {

      _value = _value.copyWith(
        text: "",
        selection: TextSelection.collapsed(offset: 0),
        composing: TextRange.empty,
      );
      });
})

so basically I m emptying manually the whole _value, and this works on the Android devices where the backspace is not working.
So this means that on those Android devices, the virtual keyboard is actually considered as a "RawKeyboard" as on those Devices, this listener is firing!
But the same code, on the devices where we have no issue with backspace, this code snippet does not fire!
Any suggestions?
And how can I adjust the code, so when a "backspace" is found on a RawKeyboard, it just deletes the last character or removes the latest chip if no text.

I Have same problem

I've fixed this by wrapping NotificationListener  widget with RawKeyboardListener
RawKeyboardListener(
      focusNode: _effectiveFocusNode, // or FocusNode()
      onKey: (event) {
        final str = currentTextEditingValue.text;
        if (event.runtimeType.toString() == 'RawKeyDownEvent' &&
            event.logicalKey == LogicalKeyboardKey.backspace &&
            str.isNotEmpty) {
          updateEditingValue(
              TextEditingValue(text: str.substring(0, str.length - 1)));
        }
      }, child: Put widget // NotificationListener<SizeChangedLayoutNotification> Here //
      });

Ive fixed this by wrapping the NotificationListener widget with RawKeyboardListener.
override back space click.
return RawKeyboardListener(
      focusNode: _effectiveFocusNode, // or FocusNode()
      onKey: (event) {
        final str = currentTextEditingValue.text;
        if (event.runtimeType.toString() == 'RawKeyDownEvent' &&
            event.logicalKey == LogicalKeyboardKey.backspace &&
            str.isNotEmpty) {
          final sd = str.substring(0, str.length - 1);
          updateEditingValue(TextEditingValue(
              text: sd, selection: TextSelection.collapsed(offset: sd.length)));
        }
      },
      child: NotificationListener<SizeChangedLayoutNotification>(
        onNotification: (SizeChangedLayoutNotification val) {
          WidgetsBinding.instance?.addPostFrameCallback((_) async {
            _suggestionsBoxController.overlayEntry?.markNeedsBuild();
          });
          return true;
        },
        child: SizeChangedLayoutNotifier(
          child: Column(
            children: <Widget>[
              GestureDetector(
                behavior: HitTestBehavior.opaque,
                onTap: () {
                  requestKeyboard();
                },
                child: InputDecorator(
                  decoration: widget.decoration,
                  isFocused: _effectiveFocusNode.hasFocus,
                  isEmpty: _value.text.isEmpty && _chips.isEmpty,
                  child: Wrap(
                    crossAxisAlignment: WrapCrossAlignment.center,
                    spacing: 4.0,
                    runSpacing: 4.0,
                    children: chipsChildren,
                  ),
                ),
              ),
              CompositedTransformTarget(
                link: _layerLink,
                child: Container(),
              ),
            ],
          ),
        ),
      ),
    );

getting same issue backspace bottom not working for samsung and vivo Phones.
please share solution if any exist .
Thank You...!

Ive fixed this by wrapping the NotificationListener widget with RawKeyboardListener. override back space click.
return RawKeyboardListener(
      focusNode: _effectiveFocusNode,
      onKey: (event) {
        final str = currentTextEditingValue.text;
        /// Make sure to filter event since without checking 'RawKeyDownEvent' will trigger this multiple times (2) because of RawKeyUpEvent
        if (event.runtimeType.toString() == 'RawKeyDownEvent' &&
            event.logicalKey == LogicalKeyboardKey.backspace &&
            str.isNotEmpty) {
          final sd = str.substring(0, str.length - 1);
          /// Make sure to also update cursor position using the TextSelection.collapsed.
          updateEditingValue(TextEditingValue(
              text: sd, selection: TextSelection.collapsed(offset: sd.length)));
        }
      },
      child: NotificationListener<SizeChangedLayoutNotification>(
        onNotification: (SizeChangedLayoutNotification val) {
          WidgetsBinding.instance?.addPostFrameCallback((_) async {
            _suggestionsBoxController.overlayEntry?.markNeedsBuild();
          });
          return true;
        },
        child: SizeChangedLayoutNotifier(
          child: Column(
            children: <Widget>[
              GestureDetector(
                behavior: HitTestBehavior.opaque,
                onTap: () {
                  requestKeyboard();
                },
                child: InputDecorator(
                  decoration: widget.decoration,
                  isFocused: _effectiveFocusNode.hasFocus,
                  isEmpty: _value.text.isEmpty && _chips.isEmpty,
                  child: Wrap(
                    crossAxisAlignment: WrapCrossAlignment.center,
                    spacing: 4.0,
                    runSpacing: 4.0,
                    children: chipsChildren,
                  ),
                ),
              ),
              CompositedTransformTarget(
                link: _layerLink,
                child: Container(),
              ),
            ],
          ),
        ),
      ),
    );

Change as following to support 'emoji'.
RawKeyboardListener(
      focusNode: _effectiveFocusNode,
      onKey: (event) {
          ...
          final sd = String.fromCharCodes(str.runes, 0, str.runes.length - 1);
          ...
      }

Are there any outstanding PRs? @lanistor it would be great if you made one with that fix

Forward here: https://github.com/flickerlist/flutter_chips_input/tree/br/1.10.1
Fix: android delete cannot be triggered in some devices
Fix: iOS Simulator Jiantipinyin cannot input

I cannot create a pull request, for version code  management, i created from v1.10.0, which is not existed in danvick/flutter_chips_input, but existed in pub.dev.


Set up OSX CI

Set up OSX CI using Travis
@pruten to provide information on the build process

Fixed in #7


Population of scopes is not consistent

The language specification specifies that top level scopes are populated with entities in the following manner (Sect. Imports):
The {\em public namespace} of library $L$ is the mapping that maps the simple name
of each public top-level member $m$ of $L$ to $m$.
The scope of a library $L$ consists of the names introduced by all top-level declarations
declared in $L$, and the names added by $L$'s imports (\ref{imports}).

This means that the top level scope of $L$ will bind the name of each class declared in $L$ to that class. So we can look up a class as such in the current library. This is confirmed by the following (Sect. Mixin Application):
The effect of a class definition of the form \code{\CLASS{} $C$ = $M$; } or the form
\code{\CLASS{} $C<T_1, \ldots, T_n>$ = $M$; } in library $L$  is to introduce the name
$C$ into the scope of $L$, bound to the class (\ref{classes}) defined by the mixin
application $M$.

I couldn't find a similar statement about regular class declarations, but you could claim that both mixin applications and regular class definitions are covered by the general rule about library scopes above.
The library namespace is used to define how imports work, and this means that we can look up classes as such from libraries that are imported 'immediately' (without a prefix).
However, when we import with a prefix we obtain a getter which returns the Type (Sect. Imports, talking about the members of prefix objects):
For every type $T$ named $id$ in $NS_n$, a corresponding getter named $id$ with return
type \code{Type}, that, when invoked, returns the type object for $T$.

So we sometimes have the getter which is used dynamically for obtaining a Type, and in that situation we don't have a suitable notion of what's in scope for the static analysis (where we'd want to look up the class as such), and we sometimes have the classes as such, and then we don't have a suitable getter which is needed at runtime (foo() => C;).
We can't just add both entries and use the resulting scopes both for static analysis and at runtime, because that would cause a name clash for every type name.
So we probably need to distinguish between static and dynamic scopes.
Members are defined in bits and pieces:
% Sect. Libraries and Scripts
The members of a library $L$ are those top level declarations given within $L$.

% Sect. Classes
The instance members of a class are its instance methods, getters, setters and
instance variables. The static members of a class are its static methods, getters,
setters and static variables. The members of a class are its static and instance members.

So in both cases members are declarations, including variables.
But we also have this:
% Sect. Variables
A library variable introduces a getter into the top level scope of the enclosing library.
A static class variable introduces a static getter into the immediately enclosing class.
An instance variable introduces an instance getter into the immediately enclosing class.
A mutable library variable introduces a setter into the top level scope of the enclosing library.
A mutable static class variable introduces a static setter into the immediately enclosing class.
A mutable instance variable introduces an instance setter into the immediately enclosing class.

So a library scope may contain both a variable x and a getter x induced by the variable, which creates a name clash for every single variable.
For that, I guess we can just insist that variables are not added to any scopes, only the induced setters and getters are. This should work dynamically as well as for the static analysis.
Finally, I can't find any indication that type parameters are added to any scopes. We do get a specification that there is a scope for them (Sect. Classes):
A class has several scopes:
\begin{itemize}
\item A {\em type-parameter scope}, which is empty if the class is not generic
  (\ref{generics}). The enclosing scope of the type-parameter scope of a class is
  the enclosing scope of the class declaration.
\item A {\em static scope}. The enclosing scope of the static scope of a  class is
  the type parameter scope (\ref{generics}) of the class.
\item  An {\em instance scope}. The enclosing scope of a class' instance scope is
  the class' static scope.
\end{itemize}

This means that the usual problem with other scopes is avoided: Since we don't add type parameters to any scopes we don't have a name clash between the type parameter as such and a getter with return type Type.  But if we fill in that missing information we'd need a similar solution.  ;-)

No milestone now: This clarification is independent of Dart 2.


Is the example code outdated?

Example is showing an advantage over standard isolates api during hight pressure of tasks workflow.
If you find example as not really representing, please open pr with enhancements of explanation.


[Bug] Templating library has not been initialized when first requests are being served.

Describe the bug
Templating middleware initializes request context in after_server_start handler however
at this point the server is already serving requests.
Screenshots
Traceback (most recent call last):
  File "handle_request", line 62, in handle_request
    SanicException,
  File "_run_request_middleware", line 28, in _run_request_middleware
    AnyStr,
  File "/app/.venv/lib/python3.10/site-packages/sanic_ext/extensions/templating/extension.py", line 50, in attach_request
    request.app.ctx.__request__.set(request)
AttributeError: 'types.SimpleNamespace' object has no attribute '__request__'

To Reproduce
Start the Sanic server and immediately make a request.
Expected behavior
When the server is ready to handle requests the context variable should already be set.
Environment
Google Cloud Platform / Cloud Run / docker image: python:3.10-slim-bullseye
Dependencies handled by Poetry
Jinja2 = "^3.1.2"
sanic = { version = "^22.3.2", extras = ["ext"] }

Additional context
__request__ is not a good name for this, it implies that it is a magic variable typically belonging to Python itself, PEP8 suggests never inventing new ones.
I am guessing the intent was name mangling, but that only works on class attributes when prefixed with two underscores.
Load the context with before_server_start handler instead?
On closer inspection it does not appear that __request__ is used at all and this whole initialization chain can be stripped out?
Mitigations
# I need templates: Reach into the `app._future_middleware` and fish out the broken function.
>>> app._future_middleware[0].middleware.__module__
'sanic_ext.extensions.templating.extension'

# I don't need templates: Initializing extensions manually without builtins, then picking the modules used.
>>> app.extend(built_in_extensions=False, extensions=[HTTPExtension, OpenAPIExtension])

Templating middleware initializes request context in after_server_start handler however
at this point the server is already serving requests.

That is not true. Requests are not yet being received when after_server_start runs. The server is running and is bound to a socket, but it is not yet receiving anything.

Start the Sanic server and immediately make a request.

Some sample code would be helpful.

I am guessing the intent was name mangling

Nope.

can be stripped out?

Yup. Functionality has been replaced and is now part of Sanic itself as of v22.6: Request.get_current()


Unable to send null values in formdata

I am creating a FormData object like so:
dio.FormData _formData = dio.FormData.fromMap(vals);
Unfortunately .fromMap removes null values, I need to send null values, is there any way supported by Dio to do this? As opposed to a work around, I can work around around it on my own accord.
Thanks

Months later I run into this being a problem again, is there any way to do this?

I have same issue. Why formdata can't have null values?

I also have same issue. I can not send empty string or null value via FormData.


Search does not display results for a long time

On searching a word , it displays '0 search results' for some time before loading the search results.(medium)

tried searching few words/ verse. results were as expected and loading time was fine.
hence moving to done.


Not working

<Popable
content={
<View
style={{
padding: 10,
alignItems: 'center',
justifyContent: 'center',
backgroundColor: 'white'
}}
>
{mtdlegend}

}>
<Text style={{
fontFamily:'MaisonNeue-Bold',
fontSize: 12,
color: mtd_color,
textTransform: 'uppercase'
}}>{mtd}

Hi,
I'll need the full section using Popable to understand where the issue is coming from.
Good day

Hi,
Since I couldn't reproduce the issue you mentioned and haven't heard back from you, I'll close the issue for now.
Feel free to comment again and provide a full example if the issue has persisted.


Update the authservice to the Istio 1.9+

The doc and the setup requires some setup. This issue captures the changes including

ExternalAuthz
UX improvements, etc localhost and port forwarding for quicker "get started".
Doc structure improvement for different deployment model.
Purge the outdated content.

Should I abandon the PR #145 and work with you on this instead?
Although Ext-authz is recommended I was thinking we should still provide EnvoyFilter as a backward compatibility solution when Istio decide to change ext-authz.

As highlighted in the doc: The following information describes an experimental feature, which is intended for evaluation purposes only.

2 things I am working on right now are:

ConfigMap is easier to people to see and understand. But it should be better provided as a Secret instead.
And the mesh.config can be done via Istio-operator other than manually edit it.

Should I create a new PR for these changes?

even though that is an experimental feature, but it's added to beta authorization policy. I would expect the performance stability can be improved as it goes to alpha beta and stable, but the API semantics should still be the same. however, envoy filter on the other hand, Istio would not maintain any promise anyway. I originally thought it's fine to add envoy filter for backward compatbility. but last version without external authz, 1.8 should be out of support already. https://istio.io/latest/docs/releases/supported-releases/. It might also confuses users for which one to choose.

ConfigMap is easier to people to see and understand. But it should be better provided as a Secret instead.

I agree. this avoids the sensitive information putting in the plaintext in the config directly.

And the mesh.config can be done via Istio-operator other than manually edit it.

Good idea. This can also help to streamline the installation onboarding process.

Sure, no problem. I will try to work on it in the weekend. But this is my first time contributing so it may take longer than what you expected.
Anyway, regarding the new docs for istio 1.9+. I tried to implement it with httpbin and got this error in authservice/istio-proxy container. And when trying to access the Ext-authz path I only received 403 and there is no redirect to the OIDC login page. I don't think I change anything major.
2021-08-18T23:53:01.570508Z     warning envoy config    gRPC config for type.googleapis.com/envoy.config.endpoint.v3.ClusterLoadAssignment rejected: malformed IP address: auth-service.default.svc.cluster.local. Consider setting resolver_name or setting cluster type to 'STRICT_DNS' or 'LOGICAL_DNS'
2021-08-18T23:53:01.570514Z     warning envoy config    gRPC config for type.googleapis.com/envoy.config.endpoint.v3.ClusterLoadAssignment rejected: malformed IP address: auth-service.default.svc.cluster.local. Consider setting resolver_name or setting cluster type to 'STRICT_DNS' or 'LOGICAL_DNS'
2021-08-18T23:53:01.570520Z     warning envoy config    gRPC config for type.googleapis.com/envoy.config.endpoint.v3.ClusterLoadAssignment rejected: malformed IP address: auth-service.default.svc.cluster.local. Consider setting resolver_name or setting cluster type to 'STRICT_DNS' or 'LOGICAL_DNS'
2021-08-18T23:53:01.570527Z     warning envoy config    gRPC config for type.googleapis.com/envoy.config.endpoint.v3.ClusterLoadAssignment rejected: malformed IP address: auth-service.default.svc.cluster.local. Consider setting resolver_name or setting cluster type to 'STRICT_DNS' or 'LOGICAL_DNS'

From the log it seems like there's a cluster endpoint with name rather than IP. Not sure how it ends up like this. Have you tried the bookinfo-example/README.md with updated helm packages? This works for me.

@anhdle14 sorry i speak too soon. you are right, there's some issues will give you 403. I already find the problems, will update the doc and config soon.


Lasair WP3.5 brief

See https://lsst-uk.atlassian.net/wiki/spaces/LUSC/pages/3224141825/Brief+for+WP3.5+Lasair+the+UK+s+community+broker

Stephen started this?

19/JUL/23

@tms-epcc started draft based on Phase C proposal with expectation this will be completed AFTER the latest version of the Lasair LTP is ready.

04/OCT/23

@RoyWilliams will update the WP3.5 brief based on the long-term plan and the briefs for the corresponding Phase B WPs

12/OCT/23

Roy and Terry have updated the brief further

15/NOV/23

no concerns raised


Security group rule description not validated

cfn-lint version: cfn-lint 0.29.5
I came across a small problem similar to the issue described in #173. Security group rule descriptions have some constraints on characters/length allowed (docs) and it seems like these are not being checked by cfn-lint.
The following snippet will be accepted by cfn-lint but cause an issue during deployment:
LbSecurityGroup:
    Type: AWS::EC2::SecurityGroup
    Properties:
      GroupDescription: Load Balancer SG
      VpcId: !Ref VPC
      SecurityGroupIngress:
        - Description: -> doesn't work
          CidrIp: <IP_ADDRESS>/0
          FromPort: 80
          ToPort: 80

Which would result in this:
Invalid rule description. Valid descriptions are strings less than 256 characters from the following set: a-zA-Z0-9. _-:/()#,@[]+=&;{}!$*
Unsure if this also applies when using Ingress rule resource AWS::EC2::SecurityGroupIngress as there is no similar constraint in the docs.
Thanks!

As described here, this would need something like:
https://github.com/aws-cloudformation/cfn-python-lint/blob/a46773c752247c51effef415bd3462eaec10ab0b/src/cfnlint/data/ExtendedSpecs/all/03_value_types.json#L33-L36
https://github.com/aws-cloudformation/cfn-python-lint/blob/a46773c752247c51effef415bd3462eaec10ab0b/src/cfnlint/data/ExtendedSpecs/all/04_property_values.json#L330-L336
and the rest of the necessary changes should be auto-generatable by running:
cfn-python-lint $ pip3 install -e . --user
cfn-python-lint $ cfn-lint --update-specs

There should be no functional difference, but src/cfnlint/data/ExtendedSpecs/all/03_value_types and src/cfnlint/data/ExtendedSpecs/all/04_property_values are more organized than src/cfnlint/data/ExtendedSpecs/all/03_value_types.json and src/cfnlint/data/ExtendedSpecs/all/04_property_values.json, so they should be preferred locations for new constraints


Rewrite to use a vendored circleci v1.1 api client

This deletes the go.mod rewrite rule since it looks like those PRs aren't going to ever get merged. Additionally, some cleanup refactoring has gone on at the same time.

I meant to open this to my own fork, sorry.

If there is something you feel like merging upstream please open another PR!

If there is something you feel like merging upstream please open another PR!

I'm happy to upstream stuff (the code is in the open already). At this particular moment I'm trying to move pretty quickly so I forked to control my own releases. I've unleashed some of my own golang opinions on the codebase so the scope of my changes isn't particularly narrow as I wasn't explicitly planning to upstream right away. If you're open to the things I've done I'll definitely open a PR just let me know!

On quick glance the existing changes don't look PR friendly, as you mentioned. Stuff like using Uber's zap is obviously perfectly fine but not a conventional choice for Terraform providers, among other patterns introduced.
That said, strong 👍🏻 for vendoring go-circleci and eliminating github.com/tgermain/go-circleci if you're up for submitting just that. No sense replacing one unmaintained fork with another, particularly with a separate org/user.


fix(graphql-mapper): Catch printSchema errors

Description
The method graphql.printSchema can fail.
This PR adds the error handling for this case.

This is already throwing, so why should we rewrite the error?

Those are the 2 points I thought about:

it allows us to handle the error as we prefer, f.e. deciding if we wanna throw or just log it without throwing, but at least we can decide it (without leaving the choice to the 3rd party library)
we can add additional logs regarding the input passed to the GraphQL function, that's extremely useful to debug


[Feature] Introduce bootcamp workshop check-in

User Story
As a user
I want to want to register for the bootcamp workshop at the service
So that there is a place for me at the event itself, and also so that the host knows that I want to attend his workshop.
Details
API side

[ ] Storage model for workshop and check-ins (workshop: name, alias, capacity, comments, date, timeslots; check-in: user_id, workshop_id, date timestamp)
[ ] API routes to check-in and check-out
[ ] API route to view workshop check-ins
[ ] API route to view user's check-ins
[ ] Optional: add constraints based on time (registration from evening to 12 noon before workshop)

Frontend side

[ ] View workshops separated by days
[ ] Select workshop and view workshop's fields
[ ] Check-in \ Check-out to workshops
[ ] View list of attendees (with sorting by timestamp and with separating line for capacity constraint)
[ ] View list of user's check-ins

Should be done via #46


Error using query_iterative()

I am getting an error while iteratively querying data from the Dimensions API using query_iterative(). How do I fix this (as previous iterations succeed)?
Code:
query = """search publications where year in [2020] and category_for.name in ["4701 Communication and Media Studies", "3605 Screen and Digital Media"] return publications"""
df_result = dsl.query_iterative(query).as_dataframe()

Error:

Query worked for me.
Possibly a temporary server outage?

If it happens again, the error message you got has instructions on how to deal with this e.g. limit and force=True - see also https://digital-science.github.io/dimcli/modules.html#dimcli.core.api.Dsl.query_iterative


Quest List: Quests are not filtered on completion

Screenshots

Environment (please complete the following information):

OS: Windows 10
RuneLite version: 1.5.19-SNAPSHOT

Tried !linkerror and that didn't fix it, issue is still present on latest build.

Also probably related, sometimes when I open the quest list the filtering isn't applied at all.

I will look into this once I get home from work.

Not totally sure on that one yet, I'll post here if I manage to figure out a reliable way to reproduce it.

I've also had the issue where quests are sometimes not filtered even though the filter is selected. Haven't figured out whats causing it yet. I did a few quests yesterday so it might have occurred on completion or something. I'll update if I figure it out.

I've also had the issue where quests are sometimes not filtered even though the filter is selected. Haven't figured out whats causing it yet. I did a few quests yesterday so it might have occurred on completion or something. I'll update if I figure it out.

The filter not being applied is caused by hopping, the widgets all get recreated and at the time the quest list widget is loaded, the header is hidden so the filter doesn't update.

Oh that makes sense, that is what was happening. Thanks

Can confirm the filter does not apply correctly on hopping worlds.

The reason for not filtering when the header is hidden was to avoid an exception in the scrollbar update script. Not familiar at all with how the scripts work, so not sure what's best. Perhaps always filter and check if the scrollbar script should run on tab switch?

To add to this, I noticed the quest filter breaks even on teleport, at least to Lumbridge. I tried it teleporting in place in Lumbridge with it filtered, then clicked on the quest tab to see it became unfiltered again.

Can confirm, filtering breaks on teleport if the quest list is not open. This is the most likely cause of the unexplained issue I was experiencing with it.


safety-worker.js: same source code as ngsw-worker.js

Which @angular/* package(s) are the source of the bug?
service-worker
Is this a regression?
No
Description
We are currently trying to get rid of an old service worker using the safety-worker.js. While writing the corresponding scripts for replacing the ngsw-worker.js with the safety-worker.js I noticed that the contents of the files are the same.
I checked the source of https://github.com/angular/angular/tree/15.1.x/packages/service-worker to see if this change might be intentional and the safety-worker.js mentioned there is a small file containing few lines of code.
Out of curiosity i checked the safety-worker.js of angular io - same as the ngsw-worker.js!
https://angular.io/safety-worker.js
https://angular.io/ngsw-worker.js
This seems like an issue. If this is the expected behaviour, the documentation does not mention it.
Please provide a link to a minimal reproduction of the bug
https://github.com/000panther/service-worker-issue
Please provide the exception or error you saw
No response
Please provide the environment you discovered this bug in (run ng version)
Angular CLI: 15.1.4
Node: 16.15.1
Package Manager: npm 8.11.0
OS: linux x64

Angular: 15.1.3
... animations, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router
... service-worker

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1501.4
@angular-devkit/build-angular   15.1.4
@angular-devkit/core            15.1.4
@angular-devkit/schematics      15.1.4
@angular/cli                    15.1.4
@schematics/angular             15.1.4
rxjs                            7.5.7
typescript                      4.9.5

Anything else?
Can I use the safety-worker.js from the service-worker repository as a workaround for now?

Thanks! I am now serving the file you mentioned as safety-worker.js, and it works perfectly! Would be awesome to get this fixed and have correct funtionality out of the box again.


Fix a typo in 138th line of demo.py

Thank you for your contribution!! Please make sure the citation format is in line with the listed references in this repo.


Refactor of top level tests

Example of running a local cross cloud test:
podman-compose -f docker-compose.yml -f docker-compose.azurite.yml -f docker-compose.minio.yml -f docker-compose.testing.yml up -d

[!note]
A directory called test-bucket must be in the _data directory for the local volume for minio, where this is will differ between systems.

Getting this when running locally
{"time":"2024-09-26T17:32:09.308398739Z","level":"ERROR","source":{"function":"github.com/cdcgov/data-exchange-upload/upload-server/cmd.main","file":"/code/cmd/main.go","line":92},"msg":"error starting app, error initialize dex handler","app_info":{"System":" DEX","Product":"UPLOAD API","App":"UPLOAD SERVER","Env":"DEV"},"error":"failed to connect to edav deliverer target not enough information given to connect to container edav"}


Update npm docs with deprecation warning

https://www.npmjs.com/package/spark still shows the old documentation and doesn't mention that the library is deprecated

Thanks. I've updated npm, and it will show up once their cache clears.


Use "Amazon EKS Amazon ECR private repositories" instead of public.ecr.aws for EKS ADOT Addon

Hello everyone,
When installing the EKS ADOT Addon the images used are hosted on "public.ecr.aws".
Would it be possible to change this to use "Amazon EKS Amazon ECR private repositories"?
See this page: https://docs.aws.amazon.com/eks/latest/userguide/add-ons-images.html
This has a couple of benefits:

Standardises with other EKS Addons
Enables the EKS ADOT Addon to be used in clusters without internet access/restricted environments
Enables the use of the ECR VPC Endpoint

We will also share this request with our AWS Enterprise support team.
If you have any questions I would be glad to discuss!
Thank you in advance!
Joachim

Heya @joachimdegroot, ADOT PM here. Thanks for raising this feature request and this is certainly something worth exploring, not on our immediate roadmap, though.

As a work-around to using a private ECR, we had to
kubectl get deploy opentelemetry-operator -n $NAMESPACE -o yaml | \
sed 's|public.ecr.aws/aws-observability|{{company-private-ecr}}/aws-mirror/adot|g' | \
kubectl apply -f -

It's klunky but works until the feature is supported natively by AWS.

See also https://github.com/aws/containers-roadmap/issues/2139

Any updates on this one. How do we get the addon enabled if you are using a IAC to setup your infrastructure. Can you prioritise this? Most enterprise apps as deployed in private network where there is no direct access to public images. How is open telemetry collector supposed to work in that environment?

Please help to update here is there any latest update in this. I am as well stuck with same situation my cluster is completely private .,Don't have access to Internet. After Add-on Deployment it is failing with public repo access error.


PST-577 Support for use readonly role

https://keboola.atlassian.net/browse/PST-577

Thanks odin, I wasn't sure if typehints are supported here yet.


feat: submit multiline with Opt+Enter on Mac

Is your feature request related to a problem? Please describe.
Multiline questions say (Finish with 'Alt+Enter' or 'Esc then Enter')
Apparently only the latter works on Macs because they do not have the Alt key
Describe the solution you'd like
I would not even change the help text, but simply make Option+Enter work since that key typically functions as Alt

AFAICT this is controlled by the underlying prompt-toolkit library.
Their README says,

To execute the input in multi-line mode, you can either press Alt+Enter, or Esc followed by Enter. (If you want the first to work in the OS X terminal, you have to check the "Use option as meta key" checkbox in your terminal settings. For iTerm2, you have to check "Left option acts as +Esc" in the options.)

That should be a fix, right?


Fix diagnostic bug for structs

Before we were checking all structs, but we should only check the public ones.
A bug came in for the following scenario
namespace DiagnosticTests
{
    public sealed class PrivateSetter
    {
        struct ExampleStruct
        {
            int i;
            public void SetValue(int value) { i = value; }
        }
    }
}

I tested this scenario in WinMDExp and it is valid. So I added a valid case, and changed the implementation to only raise diagnostics for public structs.

Closing, this got fixed in #737


Updated example "Manual generation with the package manager from buil…

Updated example "Manual generation with the package manager from build.zig".

thanks


Extract Text From PDF with Python.

I would like to contribute to this issue as well.
Please assign it to me.

I would like to contribute to this issue..So kindly assign me this issue.

@queenish001 go for it


Advanced task: Readme Assets

Hi i have combined all design configurations of the app included

Figma link
All screens
Readme
Images

great... nicely done

great... nicely done

Thank you ma'am :)


what trigger on_change function in dog mode example?

hello,  dear developers.
I'm learning how to use this tool. at first i thought the function which is subscribe_data_points  was trigger by changing data points.but
I thought i was wrong when i see the log ,could you tell me who trigger the subscribe_data_points functions?

Hello justgod3,
sorry, for late reaction but our "inbox" notification seems not to work correctly.
Not 100% sure, what you exactly asking about.
"Subscribing data points" actually means that an app is doing a subscription for being notified in case the content of at least one of the data points being part of the subscription is changing.
The data point subscription in the dog-mode app is defined by the annotation @subscribe_data_points in front of the on_change function. This is triggered during app start already.
With starting the runtime the component called feedercan is changing contents of those data points in the data broker, which in turn notifies the app; in detail this results in calling the on_change function with the updated values.
Hope that helps and feel welcome to reply if you've further questions.

Closing as question is answered. Please reopen if something needs being clarified.


Serving files without minification ?

Is it possible to Minify::serve files without any minification, for
developpment environnment purposes

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 28 Jun 2012 at 11:59

http://code.google.com/p/minify/wiki/Debugging#Javascript/CSS_Problems

or if using Minify::serve directly, use the 'debug'/'minifiers' options:
http://code.google.com/p/minify/source/browse/min/lib/Minify.php#130

Original comment by<EMAIL_ADDRESS>on 28 Jun 2012 at 2:09

Changed state: Invalid

thanks

Original comment by<EMAIL_ADDRESS>on 28 Jun 2012 at 2:14


Full-Node error

Hi, was wondering if anyone is able to help with this?  Full node runs for a couple of hours and then all services stop, cli shows this error message.  When it's running its fine works as expected I have won a block even.
Starting daemon
spare_harvester: started
spare_farmer: started
spare_full_node: started
spare_wallet: started
Traceback (most recent call last):
File "asyncio\runners.py", line 43, in run
File "asyncio\base_events.py", line 574, in run_until_complete
File "asyncio\base_events.py", line 541, in run_forever
File "asyncio\base_events.py", line 1750, in _run_once
File "selectors.py", line 323, in select
File "selectors.py", line 314, in _select
ValueError: too many file descriptors in select()
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "spare\server\start_full_node.py", line 65, in
File "spare\server\start_full_node.py", line 60, in main
File "spare\server\start_service.py", line 234, in run_service
File "asyncio\runners.py", line 46, in run
File "asyncio\runners.py", line 62, in _cancel_all_tasks
File "asyncio\base_events.py", line 574, in run_until_complete
File "asyncio\base_events.py", line 541, in run_forever
File "asyncio\base_events.py", line 1750, in _run_once
File "selectors.py", line 323, in select
File "selectors.py", line 314, in _select
ValueError: too many file descriptors in select()
[6284] Failed to execute script start_full_node

I was able to fix this by editing the config file where it shows "target_peer_count: 800" I have adjsuted this to 80, this is now the same as chia and other forks and working fine.


perf: performance overhaul

NOTE:
This is a continuation and a rework of the previous PR that had some unexplainable issues so I decided to rewrite it. I have opened this to rework it more carefully and take care of all the edge cases and random errors. (For more info, please refer to #25771)
NOTE 2:
The 70 docs commits are due to me forgetting to flush the original patch-9 branch when I was done with that one. I usually start these PR's remotely, which means I don't get to name them something unique, and I just got onto the github app to try and re-run the failed tests, which still had the old patch-9 commits. That's my bad. I used the origin's files in every conflict so it ended up just not changing anything (as can be seen in the files changed).

🔗 Linked issue

Related: #25771
❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

This is a PR that aims to improve both build time and runtime performance to the max by refactoring everything that comes to mind with performance best practices, some of which include smarter and fewer object/array iterations, reduced indexes, smarter variable usage for constant values, and more.
Some key improvements in this PR:

Refactors of known costly javascript methods:
In some places throughout the app, there are operations that rely on certain javascript methods that are known to be more expensive at certain conditions than manual implementation
(similar to endsWith in  #24746
and startsWith in  #24744).
One key change under this category is the usage of  Array.prototype.includes.
Array.prototype.includes appears to be slower when checking for existence of a certain input between at the very least 2 elements than directly checking each element with strict equality (like when dealing with strings).
After thorough benchmarking, both browser and node, direct comparisons showed marginally better performance, benchmarks attached below in that same order with the following benchmark:
const a = "#text"
console.time("start")
for(let i = 0; i< 1000000; i++) {
  const inc = ['#comment', '#text'].includes(a)
}
console.timeEnd("start")
console.time("stop")
for(let i = 0; i< 1000000; i++) {
  const inc  = a === "#comment" || a === "#text"
}
console.timeEnd("stop")

Browser:

Node:

Reduced and reworked iterations
Originally the main change in this PR, many parts in nuxt involve some sort of array/object manipulations, and in some places they are either repeated, redundant, or simply costly in terms of performance. The main examples of such cases are the small side PR's I have submitted like #24717, #24730,  and numerous others. I couldn't get all of it at the same time so I have decided to group the rest here, and also handle the iteration logic as opposed to mere count.

Reduced indexes:
Some places in nuxt include working with indexes, like the length property of array/strings. While that alone isn't expensive, it still makes that repeated index, which especially with bigger inputs (like in HTML parsing or AST traversal) will gradually cost more and more microseconds, and might eventually become slightly noticeable in page performance.

Early returns
Similar to #25647, there are some places where some iterations are made on potentially empty values, which increase the overhead and therefore decrease performance. This PR adds guards to ensure there won't be iterations over an empty value.

Avoiding amortized operations where possible
Methods like Array.prototype.push have an amortized complexity (O(1) in the case of push), which for the most part would serve most use cases well. But if we want to really fine-tune performance and take care of that amortization, we can remove them in favor of plain complexity if we know the size of the array. In multiple places around the Nuxt source, there are usages of push where the exact array size is known. This PR uses a hybrid approach that refactors said parts with known sizes by using the new Array(size) constructor with direct index modification, which would result in plain O(1) complexity, potentially outperforming the amortized O(1) complexity of push, while keeping push for dynamic arrays, variable sizes, and other parts where the size cannot be accurately obtained without causing increased time complexity (like Object.keys to obtain the size of an object).

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have added tests (if possible).
[ ] I have updated the documentation accordingly.

An important note:
The added commits are due to me forgetting to flush the original patch-9 branch when I was done with that one. That's my bad, however I used the origin's files in every conflict so it ended up just not changing anything (as can be seen in the files changed).

@danielroe Just got back to this, I don't see anything that I changed that can cause the autofix.ci to error stating "appId" is not exported? Which file could I have messed that up in?

Hey @danielroe , any idea why it fails to import /public.svg? I haven't changed anything there..

I can keep finding more areas to improve and more files to change (already set my sights on a couple as of writing this message), but I know that it's impossible to get it all completely because new things are always being added and I find new ways to improve stuff every day, so in stead of keeping this a draft for eternity, I'll likely take care of a last few of the most important and most impactful areas, and it'll be ready for review 😊

Accidentally hit the wrong button, sorry about that!

I believe as an initial refactor round this is good enough. It would definitely serve as good foundation for a more targeted refactor round that will involve structural changes (there were already a few in this one). Please let me know if there's anything you think I should add/change/revert!

Actually, just realized an area that could potentially really benefit from a refactor, looking into it.

Marking as draft to resolve type issues

Marking this as ready for review. The type issues don't appear to be related to a change of mine, seeing as it also impacts a production app I have:

(Cropped image to show that this is, in fact, not a blank project)

I'm curious, why not just make all the changes here and then merge as a whole? It would be less time consuming compared to making multiple pr's, which I'm sure you have way more important things to invest that time on

I find it easier to review in smaller batches, and there's less risk of merge conflicts.
Plus, there are a lot of judgement calls here, where I need to edit the code - it's not a straightforward single feature or change to review.

I see. Well, all I can offer is my help, if you would like me to make any changes so you won't have time to spend time making them I'll happily make them


mxnet.rnn.BucketSentenceIter has bug

suppose we have sentences with length [11, 12, 13, 14, 19], all <= 20,
and we provide two bucket keys [30, 40],
after we call mxnet.rnn.BucketSentenceIter will raise an

IndexError: too many indices for array

this happens with the following program statement,
label[:, :-1] = buck[:, 1:]
because bucket with key 40 is empty
one possible solution is to add the following program statement after line 104
self.data = [i for i in self.data if i]

Yes. Would you like to submit a PR?

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!

I'm trying to follow the code example in https://github.com/apache/incubator-mxnet/blob/master/example/rnn/lstm_bucketing.py and I get this error also.


docs: Update Cilium Sphinx RTD Theme reference

Follow-up to cilium/sphinx_rtd_theme#20: Fix hidden search bar on all media types.
This updates Documentation/requirements.txt to reference a new commit hash on the theme's v1.0 branch. This will trigger an RTD build. The new build should contain changes fixing the hidden search bar.
I read @qmonnet's post about requirements-min/requirements.txt before submitting.
Fixes: #21938

Travis hit known flake #21730

Thanks! This should go live on https://docs.cilium.io/en/latest soon I think. If we're happy with the results then we can add needs-backport/1.10, needs-backport/1.11, needs-backport/1.12 labels to this PR and it'll be backported + eventually show up on all branches.

Thank you very much, @joestringer! Thank you for sharing the Travis flake as well. I'm excited to look into this. 🔍

Seems like the /latest docs are handling the formatting pretty well so I just went ahead and marked this for backport.

We didn't backport the Sphinx/add-ons version updates to 1.10, 1.11 so the new theme is not expected to work there.

As there seems to be an issue with the original fix, please skip any actions on this for now. I'll investigate why this didn't resolve the same way as it did here.
After testing that this change will work on latest, I'll locally test the theme change on Cilium 1.10 and 1.11 with the updated Sphinx/add-ons versions. If all goes well, the Sphinx/add-ons version updates will be added for backport to 1.10 and 1.11. This also includes any changes that might've caused the original PR to not render in prod on latest.


docker: Update entrypoint.sh for multiple networks

Overview
This commit adds a --p2p.network flag to the start command in entrypoint.sh. This resolves https://github.com/celestiaorg/celestia-node/issues/1502 and https://github.com/celestiaorg/celestia-node/issues/1508

Checklist

[ ] New and updated code has appropriate documentation
[ ] New and updated code has new and/or updated testing
[ ] Required CI checks are passing
[x] Visual proof for any user facing features like CLI or documentation updates
[x] Linked issues closed with keywords

thank you @renaynay - I'll be getting to that this week and will try that out too

Codecov Report

Merging #1522 (f0679af) into main (1846a08) will decrease coverage by 0.27%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1522      +/-   ##
==========================================
- Coverage   55.46%   55.18%   -0.28%
==========================================
  Files         204      211       +7
  Lines       12167    12957     +790
==========================================
+ Hits         6748     7150     +402
- Misses       4745     5078     +333
- Partials      674      729      +55

Impacted Files
Coverage Δ

nodebuilder/p2p/genesis.go
0.00% <0.00%> (-33.34%)
:arrow_down:

header/store/init.go
72.72% <0.00%> (-27.28%)
:arrow_down:

nodebuilder/header/constructors.go
54.54% <0.00%> (-20.46%)
:arrow_down:

nodebuilder/share/module.go
58.73% <0.00%> (-13.82%)
:arrow_down:

share/ipld/namespace_hasher.go
90.24% <0.00%> (-9.76%)
:arrow_down:

header/core/exchange.go
31.81% <0.00%> (-9.10%)
:arrow_down:

header/p2p/helpers.go
52.17% <0.00%> (-8.25%)
:arrow_down:

header/p2p/exchange.go
67.29% <0.00%> (-7.40%)
:arrow_down:

nodebuilder/header/config.go
46.55% <0.00%> (-6.90%)
:arrow_down:

header/sync/sync_head.go
53.71% <0.00%> (-4.96%)
:arrow_down:

... and 43 more

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

what's the proper way to test this out? when working off of this PR, I can't seem to get image to build from this file
docker build docker/

[+] Building 0.5s (10/14)
 => [internal] load build definition from Dockerfile                  0.0s
 => => transferring dockerfile: 37B                                   0.0s
 => [internal] load .dockerignore                                     0.0s
 => => transferring context: 2B                                       0.0s
 => [internal] load metadata for docker.io/library/ubuntu:20.04       0.4s
 => [internal] load metadata for docker.io/library/golang:1.19        0.4s
 => CANCELED [builder 1/6] FROM docker.io/library/golang:1.19@sha256  0.0s
 => => resolve docker.io/library/golang:1.19@sha256:660f138b4477001d  0.0s
 => => sha256:660f138b4477001d65324a51fa158c1b868651 2.36kB / 2.36kB  0.0s
 => CACHED [stage-1 1/3] FROM docker.io/library/ubuntu:20.04@sha256:  0.0s
 => [internal] load build context                                     0.0s
 => => transferring context: 65B                                      0.0s
 => ERROR [stage-1 2/3] COPY docker/entrypoint.sh /                   0.0s
 => CACHED [builder 2/6] WORKDIR /src                                 0.0s
 => ERROR [builder 3/6] COPY go.mod go.sum ./                         0.0s
------
 > [stage-1 2/3] COPY docker/entrypoint.sh /:
------
------
 > [builder 3/6] COPY go.mod go.sum ./:
------
failed to compute cache key: "/docker/entrypoint.sh" not found: not found

when running in /docker directory
docker build .

[+] Building 0.2s (10/14)
 => [internal] load build definition from Dockerfile                                        0.0s
 => => transferring dockerfile: 37B                                                         0.0s
 => [internal] load .dockerignore                                                           0.0s
 => => transferring context: 2B                                                             0.0s
 => [internal] load metadata for docker.io/library/ubuntu:20.04                             0.2s
 => [internal] load metadata for docker.io/library/golang:1.19                              0.1s
 => CACHED [stage-1 1/3] FROM docker.io/library/ubuntu:20.04@sha256:0e0402cd13f68137edb026  0.0s
 => CANCELED [builder 1/6] FROM docker.io/library/golang:1.19@sha256:660f138b4477001d65324  0.0s
 => => resolve docker.io/library/golang:1.19@sha256:660f138b4477001d65324a51fa158c1b868651  0.0s
 => => sha256:660f138b4477001d65324a51fa158c1b868651b44e43f0953bf062e9f38b 2.36kB / 2.36kB  0.0s
 => [internal] load build context                                                           0.0s
 => => transferring context: 65B                                                            0.0s
 => ERROR [stage-1 2/3] COPY docker/entrypoint.sh /                                         0.0s
 => CACHED [builder 2/6] WORKDIR /src                                                       0.0s
 => ERROR [builder 3/6] COPY go.mod go.sum ./                                               0.0s
------
 > [stage-1 2/3] COPY docker/entrypoint.sh /:
------
------
 > [builder 3/6] COPY go.mod go.sum ./:
------
failed to compute cache key: "/docker/entrypoint.sh" not found: not found

running from celestia-node directory: docker build . -t celestia-node -f docker/Dockerfile

afaik, the image works correctly, although arabica is down
docker run -e NODE_TYPE=light -e P2P_NETWORK=arabica celestia-node celestia light start --core.ip https://limani.celestia-devops.dev --gateway --gateway.addr <IP_ADDRESS> --gateway.port 26659 --p2p.network arabica
log for build: https://app.warp.dev/block/WjhAiA5z9ONE7vUwnuCW1u
log for run (errors as expected): https://app.warp.dev/block/7tObINfBU6WX2wXRDKYYSa

you're welcome. let me know if there's other testing to do!

updated docs ahead of v0.6.2 release: https://github.com/celestiaorg/docs/pull/381/commits/f080832b2c61f28650680deba1575699b3731136


[WIP]: Portico page polish.

This applies some styling polish to the portico page set.

Automated message from Dropbox CLA bot
@brockwhittaker, it looks like you've already signed the Dropbox CLA. Thanks!

Can you post screenshots showing these?

I merged the password reset one; I'm not sure what the rest do.
The box-shadow thing looks nice, I think!

This is fixed.

@brockwhittaker can you rebase?  I think we've merged a few chunks of this.

This is rebased now.

Merged, after adding portico: to the start of the commit messages to make it clear what part of the product is involved, and removing the hunk in the last commit that modified TERMS_OF_SERVICE.  Thanks @brockwhittaker!


How properly scroll to Android element?

I have screen with elements not visible on screen. I need to scroll my screen for making that elements visible and accessible by Appium. I found tons of examples at internet, but I want to ask about better solution there too.
Currently works for me:
TouchAction action = new TouchAction(driver);         action.longPress(bottomElement).moveTo(topElement).release().perform();
How do you solve scrolling to element?
it's give me enough scrolling, but would be perfect to have native Appium method for scrolling to element:
scrollTo(MobileElement targetElement, ScrollDirection scrollDirectionEnum) ;
and that method will scroll until find element or reach end of available scrolling.

The most reliable way is to use a little bit of UI Automator code.
See the final example here: https://github.com/appium/appium/blob/master/docs/en/writing-running-appium/uiautomator_uiselector.md

Looks like this link does not work anymore. Can you please update the link?

@pawarvaibhav
Yes > link

Thanks @pumano .
Its working brother.

Thanks @pumano , still works till this day!


feat: added vizolv chat

Can u be a bit more verbose on what are u implementating? Can we have some screenshots and more contexts. Closing for now feel free to reopen if u think it's worth it


On the topic of self hosted .mrpack drawbacks

Enhancement Type
Improve an existing feature
Describe the enhancement
From what I've noticed (and it could me being blind), there seems to be no clean way of updating a modpack when providing a url to your self hosted .mrpack. I tried bumping the version but it didn't catch on. So far best solution I've found is to jump into the container and remove the existing .mrpack. Preferably the container would check for an existing .mrpack and compare versions and names, if the name+version don't match, redownload the mods.

For efficiency of avoiding a redownload it only compares the URL previously provided. My assumption was that you would change the filename at the URL to reflect a change in version.
I can have it download the file and check the versionId within the manifest, but that would require a redownload of the whole modpack on every startup. Would that be acceptable?
I could also have it save off the last update time and use https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/If-Modified-Since but not all web servers are compliant with that.

For efficiency of avoiding a redownload it only compares the URL previously provided. My assumption was that you would change the filename at the URL to reflect a change in version.
I can have it download the file and check the versionId within the manifest, but that would require a redownload of the whole modpack on every startup. Would that be acceptable?
I could also have it save off the last update time and use https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/If-Modified-Since but not all web servers are compliant with that.

Perhaps having it redownload the whole thing every time the server is started might even be a preferable option since having to change the url each time on both the server and the container is less than optimal administration wise.
Primarily because the user does not restart the server that often(?) and if they do for memory reasons it could be done in off hours.
I have an another proposal, adding a MODRINTH_FORCE_REDOWNLOAD_MODPACK env var that can be set for the frature to be enabled. Or if it's going to be a cross type feature we could omit the modrinth prefix. Or even shorten it further, it's already a mouthful.

I have had to provide support for many  users with terrible network connectivity, so redownloading is not necessarily a trivial matter. With that said, I like the suggestion to accept an option to request such redownload.
I'll give this some more thought.

That's interesting cause I expected most ppl running it off a VPS like I do but they could also be running a bigger modpack.
Do you have any cleaner idea on how to address this instead of changing the url or dropping into a shell and nuking the pack from within the container?
That being said, thnx for a quick reply and support big man, much appreciated <3

Those are the only two approaches I can think of for now. Again the expectation in current implementation is that URLs would be formatted like https://example.com/yours-1.5.2.mrpack

Does the url itself get checked for a semver tho?

No, it's just checking for any difference.

I went with the URL method and strangely enough I had issues getting it to detect a different URL too. To be 100% clear, I host my .mrpack at for e.g. https://whatever-files.com/modpacks/modpack-1.0.0.mrpack and then switch the version number to smth like https://whatever-files.com/modpacks/modpack-1.1.0.mrpack and restart the server to no effect. It could be a me specific issue since the URL is passed as an env-var from portainer but also might not be.


first push for review

What?
Why?
How?

@joey2022-eng Also, please note that if you do not understand anything, you can always ask. I will be available to reply under this thread.

Hi @joey2022-eng ,
You have done very well coming this far with this project! There are a lot of issues to fix but we will tackle them one at a time.

First of all, please re-organize your files. Take for example this video. If you skip to 2:14:40, you will realize on the left, all the files have been organized in their appropriate folders.

Please tackle this issue first

Please am done with the changes you ask me to do.
The only issue right now is how to commit and push

Hi @joey2022-eng ,
You have done very well coming this far with this project! There are a lot of issues to fix but we will tackle them one at a time.

First of all, please re-organize your files. Take for example this video. If you skip to 2:14:40, you will realize on the left, all the files have been organized in their appropriate folders.

Please tackle this issue first

Please am done with it.

Hi @joey2022-eng ,
You have done very well coming this far with this project! There are a lot of issues to fix but we will tackle them one at a time.

First of all, please re-organize your files. Take for example this video. If you skip to 2:14:40, you will realize on the left, all the files have been organized in their appropriate folders.

Please tackle this issue first

Please am done with the re-organization.

Done re-organizing it


Tighten rules for "goto" with anonymous functions and local functions

Originally reported in #926, I don't believe we currently handle the scoping of labels properly when it comes to anonymous functions and local functions.
I think we're already covered for "into":

A label can be referenced from goto statements (§13.10.4) within the scope of the label.

Note: This means that goto statements can transfer control within blocks and out of blocks, but never into blocks. end note

A local function declaring a label would have to have a block body, so I don't think you'd be able to goto a label within it.
Worth checking whether goto outside the local function (or anonymous function) is actually prohibited in the spec though. (It clearly shouldn't be allowed.)
class Program
{
    static void Main()
    {
        LocalFunction();
    testlabel:

        Action action = () =>
        {
            if (DateTime.UtcNow.Second > 10)
            {
                goto testlabel;
            }
        };

        if (DateTime.UtcNow.Second < 5)
        {
            goto testlabel;
        }

        void LocalFunction()
        {
            if (DateTime.UtcNow.Second > 10)
            {
                goto testlabel;
            }
        }
    }
}

Same error for both: CS0159 No such label 'testlabel' within the scope of the goto statement
(Interestingly enough, VS does autocomplete testlabel, and renaming testlabel renames the targets of the goto...)
I suspect we might be able to fix this in 7.3, which currently states:

Each block or switch_block creates a separate declaration space for labels. Names are introduced into this declaration space through labeled_statements, and the names are referenced through goto_statements. The label declaration space of a block includes any nested blocks. Thus, within a nested block it is not possible to declare a label with the same name as a label in an enclosing block.

Although declaring testlabel again within LocalFunction gives a different error:

CS0158 The label 'testlabel' shadows another label by the same name in a contained scope

Thanks for finding that, @KalleOlaviNiemitalo. I'll create a PR for the equivalent for local functions. Hopefully this is nice and straightforward, in that case. (Feels a bit of a shame for that not to be in the goto statement itself, but never mind.)

Possibly. It wouldn't hurt to add that redundantly to 13.10.4, keeping the restrictions in local/anonymous functions too.


Create front end MVP

Front end MVP should have:

Time series graph (from past year)
Button to quick-add an entry for today

For chart, use chart.js.
For component library, use Bulma or w/e.
Bring both in as script tag in header.
Mobile view only.
Raw HTML, CSS, JS.
AC:

Mobile-oriented, local view of graph & a button
Graph should be a time-series. X should be the twelve months. Y just number, 0-30.

Shouldn't take me more than 2 days, probably 1.

This took me like one hour..

Actually, let's add some more ACs to this. This isn't minimally viable yet.

Header showing % increase / decrease in habit occurrence

Research the algorithm to find this. What am I looking for?

If the percentage above has increased, the font color should be RED. If percentage has gone down, the font color should be GREEN.
Can this look a bit nicer? Put it in a card or something?


Dont update the docs until version is available on npm

Issue
Some of the docs provided on https://material-ui-next.com/ are not as useful until the version documented matches the npm version available

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
Documentation version and npm version should match
Current Behavior
Documentation version and npm version do not match
Context
Trying to use Tabs and the behavior does not match
1.0.0-beta.32
Your Environment

Tech
Version

Material-UI
1.0.0-beta.32

React
16.2

browser
63.0.3239.132

I always forget to publish material-ui-next (the official repository is material-ui). Done.


[CI] Publish KubeRay operator / apiserver images to Quay

Why are these changes needed?
if: contains(fromJson('["refs/heads/master"]'), github.ref)

Because I am not a collaborator of this GitHub repo, I cannot read the secrets of this repo. Hence, we will not log in to either DockerHub or Quay for PR. We will only log in to the image registries after the PR is merged into the master branch.
Related issue number
#1133: We can close the issue if the PR works well after merge.

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] This PR is not tested :(

We can only verify this PR after it is merged.

This PR works well! Check https://quay.io/organization/kuberay for more details!


Problems with long integers

I got a csv with this number for example:
311503010000139674
After reading it, i get this:
3.115030100001396E+17
How can i ignore this feature? I need the exact number.

Is it gone when you open the excelfile and double click on the column sizing to autoresize the column?
else try to format the cells with "0".
http://www.maatwebsite.nl/laravel-excel/docs/reference-guide#formatting

It is a .txt file in .csv format. I get them from a ftp server and i need to read them every 5 minutes.
Just doing this:
$cdr = \Excel::load($file, function($reader) { })->get();

Try typecasting it as an integer or string.
If that doesn't work you'll need to write a custom value binder.

@patrickbrouwers See my other ticket. I cannot make a binder. I get iconv() error.

I've little knowledge about the custom value binders to be honest. Someone wrote a PR for it. He might know more about it. I tagged him in the issue.

On 24/03/2015 19:49, Patrick Brouwers wrote:

Try typecasting it as an integer or string.
If that doesn't work you'll need to write a custom value binder.
—
Reply to this email directly or view it on GitHub
https://github.com/Maatwebsite/Laravel-Excel/issues/380#issuecomment-85666096.

I've already modified the default binder in PHPExcel recently to treat
numbers beginning with zeroes as a string, I can probably tweak it again
to check for integer-type strings that are longer than PHP_MAX_INT
--
Mark Baker

|.  \     -3
|J/ PHP |
|| |  __  |
|| |m|  |m|
I LOVE PHP

Great! But I guess these changes aren't tagged yet?
Op dinsdag 24 maart 2015 heeft Mark Baker<EMAIL_ADDRESS>het
volgende geschreven:

On 24/03/2015 19:49, Patrick Brouwers wrote:

Try typecasting it as an integer or string.
If that doesn't work you'll need to write a custom value binder.
—
Reply to this email directly or view it on GitHub
<
https://github.com/Maatwebsite/Laravel-Excel/issues/380#issuecomment-85666096
.

I've already modified the default binder in PHPExcel recently to treat
numbers beginning with zeroes as a string, I can probably tweak it again
to check for integer-type strings that are longer than PHP_MAX_INT
--
Mark Baker

|. \ -3
|J/ PHP |
|| | __ |
|| |m| |m|
I LOVE PHP
—
Reply to this email directly or view it on GitHub
https://github.com/Maatwebsite/Laravel-Excel/issues/380#issuecomment-85689589
.

--
Patrick Brouwers Film
www.patrickbrouwersfilm.eu http://www.patrickbrouwersfilm.eu
<EMAIL_ADDRESS><EMAIL_ADDRESS>17 - 6245 LJ Eijsden

Solved this problem with PHPExcel_Cell_DataType::TYPE_STRING. I think this problem occur when the export cel formats are not correct. But in my case i not create the file.
The code:
use PHPExcel_Cell;
use PHPExcel_Cell_DataType;
use PHPExcel_Cell_IValueBinder;
use PHPExcel_Cell_DefaultValueBinder;
class StubValueBinder extends PHPExcel_Cell_DefaultValueBinder implements PHPExcel_Cell_IValueBinder
{
public function bindValue(PHPExcel_Cell $cell, $value = null)
{
if (is_numeric($value))
{
        $cell->setValueExplicit($value, PHPExcel_Cell_DataType::TYPE_STRING);

        return true;
    }

    // else return default behavior
    return parent::bindValue($cell, $value);
}

}

Glad to hear you found a way to get it working :)

or just use " ", to make it string. but it'll show number like this: "3112323"

After searching and exploring a code for almost 1 hour, I found an option to change the default value binder.
We solved it by changing this value https://github.com/Maatwebsite/Laravel-Excel/blob/3.1.30/config/excel.php#L209 to PhpOffice\PhpSpreadsheet\Cell\StringValueBinder::class. so we get all the values as strings. And then as we do import, Laravel will cast it automatically.
Great library with nice customization options @patrickbrouwers 👍

The workaround by @mitulgolakiya works for me. However, all the cells are now treated as strings, I don't know if there's a way to only implement this to a column.

Implement your own string value binder which only applies it to 1 column

I found ValueBinder confusing. But I found a way to make things work, though the cells are still treated as numbers; it at least prevents the formatting of the number to exponent form.
namespace App\Exports;

use Maatwebsite\Excel\Concerns\WithColumnFormatting;

class VendorDataSheet implements WithColumnFormatting
{
    public function columnFormats(): array
    {
        return [
            // B is the column
            'B' => '0'
        ];
    }
}


Add unit tests for state/execution.go and app/app.go

still need tests for state/execution.go and app/app.go

@rigelrozanski I'm closing this, since you merged the PR, okay?


add waf custom rules

DESCRIPTION

CHECKLIST

[ ] This pull request was tested on:

[ x] PowerShell 5.1 on Windows
[x ] PowerShell 6.x on Windows
[ x] PowerShell 6.x on Linux

[x ] Resources not created by the scripts in the pull request are explicitly mentioned in a comment at the head of the file, and have user-supplied variables or function arguments which correspond to the required resource identifiers.
[x ] All user-supplied values are set in variables at the head of the file, after any comments.
[ x] All passwords are user-supplied values, or generated by a secure random string generator.
[x ] All Azure identifiers required to be universally unique are guaranteed to be so.
[x ] All scripts only use commands available in the latest release of the Azure PowerShell Az module. (Command reference)

[ ] I have an exception! (optional - state for which service)
[ ] I'm requesting an exception! (optional - you must include your MS alias for further discussion before PR review)

[x ] All scripts contain only ASCII characters (no 'smart quotes' or other wide characters)

@sptramer This is ready to merge.


Automatically sorts by key

The plugin automatically sorts the information by their key vs just showing how it is output.
I would think as a developer this shouldn't be default or a way to disable it.
I spend several hours today wondering why asort() wasn't working for my array and come to find out the plugin was sorting the output vs showing the exact output.

I am having same issue.
ISSUE: The JSON arrays are being "re-sorted" before being displayed.
SOLUTION: The items in an array should display in the order they are provided.

Bump. This also had me baffled for several hours. Please look into this.

Agreed. Luckily I didn't spend too much time bashing my head trying to figure it out. This default behavior is a head scratcher because most developers need arrays in a particular order, so seeing it sorted differently than how I'm programming it to sort is odd.

Yes agreed, although it doesn't look like this project is active, sorting should be off by default. Provide the option to sort if you like, but we need to be confident the data has not been modified in any way by the plugin.
I had been back and forth with developers thinking they were sorting, but as it turns out my results were sorted! lol.

Any updates on this?
Is there any fork that resolved this?


docs: clarify server shutdown

This PR closes #360 and modifies the README > Start server section.
When a server is started in a job, it continues running until the job completes. The shutdown of a server is not linked to the end of testing as such. The current text was misleading in this respect.
An example can be seen by viewing actions/workflows/example-start.yml, for example job<PHONE_NUMBER>

See Start server to review.

I think it's clear enough from the community discussion over there that it's a confusing issue. This PR should not have closed that other issue.

@karlhorky

Issue #360 from @bahmutov in April 2021 originally contained just a question, and that question was actually already answered by the documentation. The documentation was however inconsistent, and this PR corrected the description in README > Start server. So in my view the issue was not incorrectly closed.

You would like the ability to kill a server process and you would like that documented. I don't believe that re-opening #360 would serve that purpose well, however I'm happy to let @nagash77 comment on that and decide. This is a complex topic and to do it justice it should have a clear separate new issue.

@karlhorky if you feel there is functionality missing that you feel would be a beneficial addition to Cypress please open a feature request ticket and our product team will review and give it consideration.

My comment above does not have to do with missing behavior from Cypress.
It has to do with the original reason for #360 existing - that it was unusual behavior that the process was not stopping. This will be a sharp edge that some of your users will run into, with (now that #360 was closed) no open issue mentioning it.
This behavior feels like a bug.
What I was suggesting in my comment above was this:

how to stop a server started by cypress-io/github-action:

using multiple jobs
using a command to allow stopping between steps in a single job

For 1., apparently using multiple jobs should be fairly straightforward according to @MikeMcC399. So it would be a documentation block showing multiple jobs.
For 2., if you check out #360, there are two workarounds for how to run another step after the Cypress GitHub action (and maybe the Cypress team has other recommendations for different commands). This works around the weird behavior of the Cypress GitHub Action, and offers clear recourse to users having this problem.
This should have been a part of #915, but since it wasn't, it should be a part of whatever PR closes #360.

@karlhorky
I'm sorry that you don't like what has been done here. When I considered suggesting closure I also looked at the example in #360 using v2 of the action and the link to the repository https://github.com/bahmutov/cypress-grep-example.
I found the workflow https://github.com/bahmutov/cypress-grep-example/blob/main/.github/workflows/main.yml there and this no longer had a problem because the job had been split up, so the issue description no longer represented a reproducible example using a current version of the action. The most effective open issues are ones which are reproducible on current versions.


Only check highlight style if typeof object

The styleParser must not assign an empty highlight style.
Otherwise the highlight interaction will re-style an mvt layer, even if there is nothing to re-style.

This can be tested by setting a breakpoint in the styleFeature method. This shouldn't break if there is nothing to style.
Screencast from 17-07-24 11:59:30.webm


Lasso Regression - Documentation

I want to work on the documentation of Lasso Regression algorithm. Can you assign it to me?
@ashwani-rathee

Task : Create detailed documentation of the Lasso Regression ML algorithm by updating readme.md
Level-1 : 10 points

Sure @divyanshu887 ,You have been assigned the task to create detailed documentation of Lasso Regression.
Try to submit a PR in 2 days.

#64 is ready


安装好所有模块 ，如下提示

TYLORGAN-MB0:manim omen$ python3 -m manim example_scenes.py WriteStuff -pl
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/Users/omen/ManimInstall/manim/manim.py", line 2, in
import manimlib
File "/Users/omen/ManimInstall/manim/manimlib/init.py", line 4, in
import manimlib.extract_scene
File "/Users/omen/ManimInstall/manim/manimlib/extract_scene.py", line 9, in
from manimlib.scene.scene import Scene
File "/Users/omen/ManimInstall/manim/manimlib/scene/scene.py", line 12, in
from manimlib.camera.camera import Camera
File "/Users/omen/ManimInstall/manim/manimlib/camera/camera.py", line 9, in
import cairo
ModuleNotFoundError: No module named 'cairo'
TYLORGAN-MB0:manim omen$

Your installation of Python doesn't include pycairo which is the module that draws the shapes into images.

3q.and i akready solved this problem.but i have a new question that i can't use Chinese by textmobject
获取 Outlook for iOShttps://aka.ms/o0ukef

发件人: Nathan Graule<EMAIL_ADDRESS>发送时间: Wednesday, August 21, 2019 7:46:15 PM
收件人: 3b1b/manim<EMAIL_ADDRESS>抄送: Campanulata<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>主题: Re: [3b1b/manim] 安装好所有模块 ，如下提示 (#701)
Your installation of Python doesn't include pycairo which is the module that draws the shapes into images.
―
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/3b1b/manim/issues/701?email_source=notifications&email_token=AFU5P4KK347EHK77VORWP43QFUTIPA5CNFSM4IOFNRW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4ZMKMQ#issuecomment-523420978, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFU5P4OYGJEYA3S6S53QMRTQFUTIPANCNFSM4IOFNRWQ.

pip install -U pycario

could help you. macOS is a Unix-like system, you won't get trouble in C compiling, in most cases...


Fix typo in v1alpha2

Fixed a typo that we also seem to have copy pasted in a lot of places.
Fixes #176

/assign @munnerz

/lgtm


[BitwiseCopyable] Fragile modules always infer.

Gating inference on visibility only makes sense in the context of a resilient module.

@swift-ci please test