Linkage between candidates and committees needs to be developed

There are several channels being pursued, have the SBOE provide it.
We can parse all the disclosure documents and retrieve the SBOEID and candidate name,

Without this linkage, we can't use the data earlier than 2019


Handle revoked access

If user revoke access of our app from Uber website, he/she will no longer be able to use the command.
This is because the user will be in a status of unauthorized, but our app think it's authorized since we have all the tokens.
This will get RestClient::Unauthorized: 401 Unauthorized error

We need to be able to rescue a 401 response from any api call, then? Is there any case where we could get that response and NOT want to reset the user's tokens?

Should/can we add a command for people to revoke access via slack?

@huangchris

I think if the user revoked the access from Uber, then try to do use the command again, it's pretty obvious that they want to use our app again. So just redoing the oauth process should be sufficient. We can simply set the uber tokens field to nil then just treat the user as a new user.

I don't think its necessary, most app don't do that anyway. But we can have it.


.prettierignore location

Hi, i have this problem:
My directory structure is:
root
  .git
  web-app
    package.json
    .prettierignore

In that package.json i have the script "format:fix": "pretty-quick --staged", however it is not taking the .prettierignore file. I have to place the file in the root folder of the repo so that it works.
Is there a way to avoid copying that file to the root of the repo and take the one inside the web-app folder?

Hi, not currently, we'll have to either:

search both the  git dir and the working dir for .prettierignore, or
add an --ignore-path CLI option

@azz  when .prettierignore and --ignore-path can use in project ?

I have an issue I'm unsure is the same: I can't get my .prettierignore.js file to function anywhere. webpack is running in the same directory as the .prettierignore.js file, both which are two directories into my root git directory.

@azz what about adding a --configDir option that would be used for all prettier config files?
I have the same issue here, and the working dir workaround does not work with commits from IDE, so I think the best way would be an option to manually set the path for pretty-quick, or to go up folders from the pretty-quick folder itself, since most likely the node_modules folder will be in the desired config folder.

I have the same issue as osnoser1, for now I will place in root directory, but it would be ideal to be able to set the ignore-path just as prettier allows

How does your editor pickup your config files if pretty-quick cannot find them? Are you using WebStorm?

@azz Yes, I'm using WebStorm and it picks up the files correctly.
Just in case, the folder structure looks like this, nothing too weird I think:
/ (git root)
  frontend/ (frontend root, prettier config files are there)
  backend/
  other/

Just checked the code, we only need --ignore-path as a CLI option as the config is loaded per-file:
https://github.com/azz/pretty-quick/blob/de790a757ecbc08b460ac3f4b20e780f63986bea/src/formatFiles.js#L8
So pretty-quick --staged --ignore-path=frontend/.prettierignore.
An issue with this is WebStorm might not know how to find your .prettierignore file.

@IronSean I don't think a file named .prettierignore.js is valid. It has to be .prettierignore. (Although it would be nice if a JS module worked...would be a good way to configure multiple projects with the same ignore settings, since you can import other modules in JS files.)


VMWare Resource Bridge Update Clarification

This needs clarification. what does this mean in regard to updating the resource bridge?
"Find and delete the old Arc resource bridge template from your vCenter"
Are you talking about the finding and deleting VM itself? The script doesn't create a Template to my knowledge, but a VM. Are we supposed to just delete the VM and run the script again to upgrade? Also how do we know if there is an upgrade? I can't seem to find a version history.

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

ID: 4a79cab1-7d42-b752-77ee-68d68ea5d2a8
Version Independent ID: 62162227-2f5f-8b90-72d1-c3ace8ccd498
Content: Perform ongoing administration for Arc-enabled VMware vSphere - Azure Arc
Content Source: articles/azure-arc/vmware-vsphere/day2-operations-resource-bridge.md
Service: azure-arc
GitHub Login: @JnHs
Microsoft Alias: jenhayes

@KingDrewcifer
Thanks for your feedback! We will investigate and update as appropriate.

@JnHs, could you please review this issue and share your thoughts. Thank you

Ok, this was all me. My bridge was so out of date that these instructions didn't make sense. I just removed the bridge resources from Azure, removed the outdated bridge from VMWare and redeployed. The new deployment does create a VM Template. . Sorry for the confusion. I would close this but the link to version history might still be a good add.

@KingDrewcifer Thanks for the follow-up. Glad everything is working now. I'll close this issue as you suggest and will follow up with our team to see about adding a link/mentioning versioning issues. Thanks!
#please-close


Buffered List Chunk Reader

Once list chunking is implemented on the server, clients will need to be updated to deal the the fact that lists will be delivered across separate AttributeDataIBs and potentially, across multiple messages.
To reduce the burden of doing this on clients, this commit creates an adapter class (BufferedReadCallback) that sits in between the application and the ReadClient and buffers up each list chunk before reconstituting them into a contiguous list and dispatching that up to the application.
This buffering is done natively in TLV. This avoids the need to understand the specifics of any data model types that the TLV will be decoded into.
We cannot do this buffering at the next level up (i.e at the cluster object level) since those objects are merely 'view' objects and not proper containers. Consequently, there is no way to 'patch' the state in those objects incrementally. By doing it at the TLV level, we can preserve the existing cluster object APIs, which is a significant win.
This PR does not actually fix the ReadClient to correctly handle AttributeDataIBs that contain list operations other than replace (i.e append). That should happen in a separate PR.
Tests: A TestBufferedReadCallback test that exercises all possible permutations of list and non list data to ensure the state machine is validated.

The Darwin test failure reproduces for me locally, and is due to a bug in this PR.
We are getting a crash with the following stack:
hread 2 Crashed:: Dispatch queue: com.zigbee.chip.framework.controller.workqueue
0   com.chip.CHIP                 	0x000000011519382c chip::app::ReadClient::IsSubscriptionType() const + 12 (ReadClient.h:198)
1   com.chip.CHIP                 	0x00000001150935d0 IMReadReportAttributesResponseCallback(chip::app::ReadClient const*, chip::app::ConcreteAttributePath const*, chip::TLV::TLVReader*, chip::Protocols::InteractionModel::Status) + 176 (im-client-callbacks.cpp:379)
2   com.chip.CHIP                 	0x0000000115192a2d chip::Controller::DeviceControllerInteractionModelDelegate::OnAttributeData(chip::app::ReadClient const*, chip::app::ConcreteDataAttributePath const&, chip::TLV::TLVReader*, chip::app::StatusIB const&) + 141 (DeviceControllerInteractionModelDelegate.h:90)
3   com.chip.CHIP                 	0x000000011512e36f chip::app::BufferedReadCallback::DispatchBufferedData(chip::app::ReadClient const*, chip::app::ConcreteAttributePath const&, chip::app::StatusIB const&, bool) + 607 (BufferedReadCallback.cpp:218)
4   com.chip.CHIP                 	0x000000011512e06e chip::app::BufferedReadCallback::OnReportEnd(chip::app::ReadClient const*) + 110 (BufferedReadCallback.cpp:39)
5   com.chip.CHIP                 	0x0000000115161beb chip::app::ReadClient::ProcessReportData(chip::System::PacketBufferHandle&&) + 3115 (ReadClient.cpp:402)
6   com.chip.CHIP                 	0x0000000115160af1 chip::app::ReadClient::OnMessageReceived(chip::Messaging::ExchangeContext*, chip::PayloadHeader const&, chip::System::PacketBufferHandle&&) + 353 (ReadClient.cpp:245)

and indeed BufferedReadCallback::OnReportEnd does this:
    CHIP_ERROR err = DispatchBufferedData(nullptr, mBufferedPath, StatusIB(), true);

which is wrong: it should pass apReadClient for that first arg.


Dead link in documentation

Getting Started links to paper sizes which does not exist.

Fixed by #703


updates don't be able

Microsoft PowerToys version
0.53.0
Running as admin

[X] Yes

Area(s) with issue?
Installer
Steps to reproduce
I cannot update

✔ Expected Behavior
No response
❌ Actual Behavior
I cannot update the app
Other Software
No response

Are you still experiencing this issue? /needinfo


Could use better senders/implementors pattern matching ...

As I point out here, I tend to  use the selector pattern (both senders and implementors) to find likely methods where I actually read the method or look at the class to know if it is the one(s) I am looking for.
It would be much more efficient (for the user) to get a list of all of the methods that match the pattern than a list of symbols that match the pattern. Offhand I don't remember the patterns that I have recently used that give back large numbers of false positives and since I've stopped using the pattern match feature, I'm not likely to generate more ... but just trust me ... it is a thing.
Getting a list back that is just a list of symbols makes it very tedious to use this tool when there are a large number of false positive symbol matches ... I have started using browse methods containing, instead of the pattern matches, because I at least get back a list of methods that I can sort through, rather than having to wade through all of the false positive results provided by matching symbols ...
I've put this in December 2019, becaus I don't want it ignored ...

Should have a fix for this in 3.1.0. Should improve the autocomplete seletions as well.

I think this is working as designed... it's a great change if so as that was very annoying in GBS.


Documentation to setup discord bot on discord developer portal

Edit README to include this documentation.

Hey @milan090 I could take on this task :)

@MaximeKoitsalu sure, go ahead.

@milan090 Added a new readme file with instructions as per your request. Hope that's what you asked for 😊

@MaximeKoitsalu Great! It's almost done. I've requested few necessary changes. Do address them soon and i'll merge the request


Adding a new owner for my package doesn't send email to the new owner

I added 'haacked' as a new owner to one of my packages but he doesn't receive an email notification.

Emails are not sent after registration of new account either.

Fixed, the DB did have use SMTP as true.


Rename package to "fprodents"?

The hyphen / underscore in the package name is really a headache. I think that migrating everything to fprodents (similarly to scikit-learn -> sklearn, scikit-image -> skimage, etc.) will really make everything easier at almost no cost. Nobody prevents us to submit two packages to pypi (fmriprep-rodents and fprodents) or only use fmriprep-rodents for Pypi.
WDYT?

Seems reasonable to me.
"fMRIPrep rodents" isn't very catchy, anyway.
I wouldn't object to continuing with fmriprep-rodents for Pypi alone, I don't really see a need to muddle it with two packages.

Seems reasonable to me.
"fMRIPrep rodents" isn't very catchy, anyway.
I wouldn't object to continuing with fmriprep-rodents for Pypi alone, I don't really see a need to muddle it with two packages.

cc/ @mgxd @effigies any opinions or detail we might just missed out?

cc/ @mgxd @effigies any opinions or detail we might just missed out?

No opinions, really. "fp" doesn't really say "fmriprep" to me, but I don't think it's very important.

No opinions, really. "fp" doesn't really say "fmriprep" to me, but I don't think it's very important.

same, no strong opinion. throwing another name out there: rmriprep 🤷

same, no strong opinion. throwing another name out there: rmriprep 🤷

I thought of FmriPrep -> "fp" by analogy to SciKit -> "sk".
@mgxd I can see an issue with dMRIPrep/rodents there. (Also, r-fMRI is many times understood as resting-state fMRI).

I thought of FmriPrep -> "fp" by analogy to SciKit -> "sk".
@mgxd I can see an issue with dMRIPrep/rodents there. (Also, r-fMRI is many times understood as resting-state fMRI).


Feature request: dump data to text file

Hi,
It would be great if you could dump the workout / pedometer / sleep data to a text file on the device. Doing so would allow you to store your workout data long term without app lock in.

Good feature! I will see if I can fit this in next updates. However I dont think I will write to file but probably will try to write to google drive. I will test out which method is easier. thanks

added integration to google fit


⚠ Adam Brand Music has degraded performance

In 89a1701, Adam Brand Music (https://adambrandmusic.com) experienced degraded performance:

HTTP code: 200
Response time: 5497 ms

Resolved: Adam Brand Music performance has improved in 5171814 after 7 minutes.


Clarify autocompletion support

The README is fairly confusing on this issue:

It's TODO, but it's marked done. 😕 🙃😅 Which is it?

It's done. Thanks for pointing out problem in Readme :)


Add Arm builds

Checklist

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

@JohanMabille @SylvainCorlay can you review this PR for me please?

@JohanMabille @SylvainCorlay ping

@mcbarton I merged your PR after a couple of minor changes.
PS: I noticed that you opened the PR from the main branch. You may want to reset this branch to the new upstream/main.


How to build?

How is this supposed to build? There are no instructions anywhere.

Hello!
Run make && make corebuf then sudo make install
If you want, copy the ".zrc" config file to your home directory for a pre-configured config file with aliases, functions and the like.
Note that the next update will change how file descriptors work so after updating make sure to change the .zrc file too. Syntax will change a lot as the project is still in its infancy, so  every time you get a new clone of the repo make sure to update the config file just to be safe.

Can you please help me some more?
$/home/temp/zrc-0.8b/zrc-0.8b> make
mkdir -p bin
g++ src/main.cpp -o bin/zrc
src/main.cpp:26:22: fatal error: filesystem: No such file or directory
 #include <filesystem>
              ^
compilation terminated.
Makefile:4: recipe for target 'all' failed
make: *** [all] Error 1

Hmm. What C++ version do you have?

Try it now. I modified the Makefile and now it should compile on older gcc versions.

Nope. Exact same output as before.
I have gcc 4.9.2 and 6.3.0.

4.9.2 is quite old. I have tested on all my machines and it compiled. You might want to upgrade your toolchain to a newer version (or I could replace all code that uses std::filesystem with a more classic C-style approach using dirent.h/libgen.h, which is not only more portable across *NIXes but also very easy to implement).

4.9.2 is quite old. I have tested on all my machines and it compiled. You might want to upgrade your toolchain to a newer version (or, if you want, I could just replace all code that uses std::filesystem with a more classic C-style approach using dirent.h/libgen.h, which is not only more portable across *NIXes but also very easy to implement).

Yes, I use an old distro. I have to. It's complicated. I would be using your binary release otherwise.
If you can replace something with something else that makes the whole thing more portable, well then why not?

"also very easy to implement"

Sounds like a done deal.

Alright! I've removed all traces of std::filesystem. Try to see if it works now

It's getting better.
$ make
mkdir -p bin
g++ src/main.cpp -o bin/zrc
src/main.cpp:118:45: error: ‘string_view’ is not a member of ‘std’
   static bool               die            (std::string_view          );
                                             ^~~
src/main.cpp:119:71: error: ‘is_number’ declared as an ‘inline’ variable
   static inline bool        is_number      (std::string_view          );
                                                                       ^
src/main.cpp:119:45: error: ‘string_view’ is not a member of ‘std’
   static inline bool        is_number      (std::string_view          );
                                             ^~~
src/main.cpp:120:71: error: ‘WordList glob’ redeclared as different kind of symbol
   WordList                  glob           (std::string_view          );
                                                                       ^
In file included from src/main.cpp:12:0:
/usr/include/glob.h:146:12: note: previous declaration ‘int glob(const char*, int, int (*)(const char*, int), glob_t*)’
 extern int glob (const char *__restrict __pattern, int __flags,
            ^~~~
src/main.cpp:120:45: error: ‘string_view’ is not a member of ‘std’
   WordList                  glob           (std::string_view          );
                                             ^~~
src/main.cpp:151:45: error: ‘string_view’ is not a member of ‘std’
   static std::string        get_var        (std::string_view          );
                                             ^~~
src/main.cpp:158:71: error: ‘is_expr’ declared as an ‘inline’ variable
   extern inline bool        is_expr        (std::string_view          );
                                                                       ^
src/main.cpp:158:71: warning: ‘is_expr’ initialized and declared ‘extern’
src/main.cpp:158:45: error: ‘string_view’ is not a member of ‘std’
   extern inline bool        is_expr        (std::string_view          );
                                             ^~~
src/main.cpp:178:50: error: ‘fd_parse’ declared as an ‘inline’ variable
   inline bool               fd_parse       (std::string_view,size_t   );
                                                  ^~~~~~~~~~~
src/main.cpp:178:45: error: ‘string_view’ is not a member of ‘std’
   inline bool               fd_parse       (std::string_view,size_t   );
                                             ^~~
src/main.cpp:178:71: error: expected primary-expression before ‘)’ token
   inline bool               fd_parse       (std::string_view,size_t   );
                                                                       ^
src/main.cpp:178:71: error: expression list treated as compound expression in initializer [-fpermissive]
In file included from src/main.cpp:187:0:
src/lex.hpp:54:17: error: ‘std::string_view’ has not been declared
  add_token(std::string_view tok)
                 ^~~~~~~~~~~
src/lex.hpp: In member function ‘void WordList::add_token(int)’:
src/lex.hpp:56:26: error: request for member ‘begin’ in ‘tok’, which is of non-class type ‘int’
   std::string tmp = {tok.begin(), tok.end()};
                          ^~~~~
src/lex.hpp:56:39: error: request for member ‘end’ in ‘tok’, which is of non-class type ‘int’
   std::string tmp = {tok.begin(), tok.end()};
                                       ^~~
src/lex.hpp:56:44: error: could not convert ‘{<expression error>, <expression error>}’ from ‘<brace-enclosed initializer list>’ to ‘std::__cxx11::string {aka std::__cxx11::basic_string<char>}’
   std::string tmp = {tok.begin(), tok.end()};
                                            ^
src/lex.hpp: In function ‘WordList tokenize(std::__cxx11::string, std::istream&)’:
src/lex.hpp:22:41: error: ‘die’ cannot be used as a function
             or die("Unmatched bracket\n");        \
                                         ^
src/lex.hpp:94:5: note: in expansion of macro ‘CHK_LINE’
     CHK_LINE;
     ^~~~~~~~
src/lex.hpp:22:41: error: ‘die’ cannot be used as a function
             or die("Unmatched bracket\n");        \
                                         ^
src/lex.hpp:5:13: note: in expansion of macro ‘CHK_LINE’
             CHK_LINE;                             \
             ^~~~~~~~
src/lex.hpp:117:4: note: in expansion of macro ‘STR_KLUDGE’
    STR_KLUDGE(q);
    ^~~~~~~~~~
src/lex.hpp:131:4: warning: attributes at the beginning of statement are ignored [-Wattributes]
    [[fallthrough]];
    ^
src/lex.hpp:146:20: error: invalid conversion from ‘const char*’ to ‘int’ [-fpermissive]
    wl.add_token(";");
                    ^
src/lex.hpp:54:2: note:   initializing argument 1 of ‘void WordList::add_token(int)’
  add_token(std::string_view tok)
  ^~~~~~~~~
In file included from src/main.cpp:188:0:
src/variable.hpp: In function ‘std::__cxx11::string S(const T&)’:
src/variable.hpp:15:5: error: expected ‘(’ before ‘constexpr’
  if constexpr (std::is_integral<T>::value)
     ^~~~~~~~~
src/variable.hpp:17:2: error: ‘else’ without a previous ‘if’
  else
  ^~~~
In file included from src/main.cpp:189:0:
src/expr.cpp: At global scope:
src/expr.cpp:18:14: error: ‘is_expr’ declared as an ‘inline’ variable
 is_expr(std::string_view str)
              ^~~~~~~~~~~
src/expr.cpp:18:14: warning: ‘is_expr’ initialized and declared ‘extern’
src/expr.cpp:18:14: error: redefinition of ‘bool is_expr’
src/main.cpp:158:29: note: ‘bool is_expr’ previously defined here
   extern inline bool        is_expr        (std::string_view          );
                             ^~~~~~~
In file included from src/main.cpp:189:0:
src/expr.cpp:18:9: error: ‘string_view’ is not a member of ‘std’
 is_expr(std::string_view str)
         ^~~
src/expr.cpp: In function ‘std::__cxx11::string expr(std::__cxx11::string, ExprType)’:
src/expr.cpp:178:16: error: ‘is_expr’ cannot be used as a function
  if (!is_expr(e))
                ^
In file included from src/main.cpp:190:0:
src/dispatch.hpp: In function ‘std::__cxx11::string zrc_builtin_exit(int, char**)’:
src/dispatch.hpp:110:25: error: ‘is_number’ cannot be used as a function
  exit((is_number(ret_val) && !ret_val.empty())\
                         ^
src/dispatch.hpp:116:19: note: in expansion of macro ‘EXIT_SESSION’
 Command(exit)   { EXIT_SESSION; }
                   ^~~~~~~~~~~~
src/dispatch.hpp: In function ‘std::__cxx11::string zrc_builtin_die(int, char**)’:
src/dispatch.hpp:122:30: error: ‘die’ cannot be used as a function
 Command(die)    { die(argv[1]); return "1"; }
                              ^
src/dispatch.hpp: In function ‘std::__cxx11::string zrc_builtin_fork(int, char**)’:
src/dispatch.hpp:110:25: error: ‘is_number’ cannot be used as a function
  exit((is_number(ret_val) && !ret_val.empty())\
                         ^
src/dispatch.hpp:451:3: note: in expansion of macro ‘EXIT_SESSION’
   EXIT_SESSION;
   ^~~~~~~~~~~~
src/dispatch.hpp: In function ‘std::__cxx11::string zrc_builtin_chr(int, char**)’:
src/dispatch.hpp:596:19: error: ‘is_number’ cannot be used as a function
  if (is_number(ret))
                   ^
In file included from src/main.cpp:192:0:
src/subst.hpp: In function ‘std::__cxx11::string get_var(std::__cxx11::string)’:
src/subst.hpp:30:24: error: ‘std::__cxx11::string get_var(std::__cxx11::string)’ redeclared as different kind of symbol
 get_var(std::string str)
                        ^
src/main.cpp:151:29: note: previous declaration ‘std::__cxx11::string get_var’
   static std::string        get_var        (std::string_view          );
                             ^~~~~~~
In file included from src/main.cpp:192:0:
src/subst.hpp: In function ‘void str_subst(std::__cxx11::string&)’:
src/subst.hpp:147:24: error: no match for call to ‘(std::__cxx11::string {aka std::__cxx11::basic_string<char>}) (std::__cxx11::string&)’
     res += get_var(tmp1);
                        ^
In file included from src/main.cpp:193:0:
src/exec.hpp: In function ‘void exec(int, char**)’:
src/exec.hpp:104:16: error: ‘die’ cannot be used as a function
      die("fork");
                ^
src/exec.hpp:128:15: error: ‘die’ cannot be used as a function
     die("fork");
               ^
src/exec.hpp: In function ‘bool str_subst_expect1(std::__cxx11::string&)’:
src/exec.hpp:223:19: error: cannot convert ‘std::__cxx11::string {aka std::__cxx11::basic_string<char>}’ to ‘const char*’ for argument ‘1’ to ‘int glob(const char*, int, int (*)(const char*, int), glob_t*)’
   gbzwl = glob(str);
                   ^
In file included from src/main.cpp:194:0:
src/zlineedit.hpp: At global scope:
src/zlineedit.hpp:67:13: error: variable or field ‘W’ declared void
  sub W(std::string_view sv)
             ^~~~~~~~~~~
src/zlineedit.hpp:67:8: error: ‘string_view’ is not a member of ‘std’
  sub W(std::string_view sv)
        ^~~
src/main.cpp:465:1: error: expected ‘}’ at end of input
 }
 ^
In file included from src/main.cpp:188:0:
src/variable.hpp: In instantiation of ‘std::__cxx11::string S(const T&) [with T = long unsigned int; std::__cxx11::string = std::__cxx11::basic_string<char>]’:
src/variable.hpp:34:22:   required from ‘std::__cxx11::string Array::get(const T&) [with T = long unsigned int; std::__cxx11::string = std::__cxx11::basic_string<char>]’
src/dispatch.hpp:804:33:   required from here
src/variable.hpp:18:10: error: no matching function for call to ‘std::__cxx11::basic_string<char>::basic_string(const long unsigned int&)’
   return static_cast<std::string>(t);
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /usr/include/c++/6/string:52:0,
                 from /usr/include/c++/6/bits/locale_classes.h:40,
                 from /usr/include/c++/6/bits/ios_base.h:41,
                 from /usr/include/c++/6/ios:42,
                 from /usr/include/c++/6/istream:38,
                 from /usr/include/c++/6/fstream:38,
                 from src/main.cpp:27:
/usr/include/c++/6/bits/basic_string.h:549:9: note: candidate: template<class _InputIterator, class> std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::basic_string(_InputIterator, _InputIterator, const _Alloc&)
         basic_string(_InputIterator __beg, _InputIterator __end,
         ^~~~~~~~~~~~
/usr/include/c++/6/bits/basic_string.h:549:9: note:   template argument deduction/substitution failed:
In file included from src/main.cpp:188:0:
src/variable.hpp:18:10: note:   candidate expects 3 arguments, 1 provided
   return static_cast<std::string>(t);
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /usr/include/c++/6/string:52:0,
                 from /usr/include/c++/6/bits/locale_classes.h:40,
                 from /usr/include/c++/6/bits/ios_base.h:41,
                 from /usr/include/c++/6/ios:42,
                 from /usr/include/c++/6/istream:38,
                 from /usr/include/c++/6/fstream:38,
                 from src/main.cpp:27:
/usr/include/c++/6/bits/basic_string.h:511:7: note: candidate: std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::basic_string(std::__cxx11::basic_string<_CharT, _Traits, _Alloc>&&, const _Alloc&) [with _CharT = char; _Traits = std::char_traits<char>; _Alloc = std::allocator<char>]
       basic_string(basic_string&& __str, const _Alloc& __a)
       ^~~~~~~~~~~~
/usr/include/c++/6/bits/basic_string.h:511:7: note:   candidate expects 2 arguments, 1 provided
/usr/include/c++/6/bits/basic_string.h:507:7: note: candidate: std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::basic_string(const std::__cxx11::basic_string<_CharT, _Traits, _Alloc>&, const _Alloc&) [with _CharT = char; _Traits = std::char_traits<char>; _Alloc = std::allocator<char>]
       basic_string(const basic_string& __str, const _Alloc& __a)
       ^~~~~~~~~~~~
/usr/include/c++/6/bits/basic_string.h:507:7: note:   candidate expects 2 arguments, 1 provided
/usr/include/c++/6/bits/basic_string.h:503:7: note: candidate: std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::basic_string(std::initializer_list<_Tp>, const _Alloc&) [with _CharT = char; _Traits = std::char_traits<char>; _Alloc = std::allocator<char>]
       basic_string(initializer_list<_CharT> __l, const _Alloc& __a = _Alloc())
       ^~~~~~~~~~~~
/usr/include/c++/6/bits/basic_string.h:503:7: note:   no known conversion for argument 1 from ‘const long unsigned int’ to ‘std::initializer_list<char>’
/usr/include/c++/6/bits/basic_string.h:476:7: note: candidate: std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::basic_string(std::__cxx11::basic_string<_CharT, _Traits, _Alloc>&&) [with _CharT = char; _Traits = std::char_traits<char>; _Alloc = std::allocator<char>]
       basic_string(basic_string&& __str) noexcept
       ^~~~~~~~~~~~
/usr/include/c++/6/bits/basic_string.h:476:7: note:   no known conversion for argument 1 from ‘const long unsigned int’ to ‘std::__cxx11::basic_string<char>&&’
/usr/include/c++/6/bits/basic_string.h:464:7: note: candidate: std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::basic_string(std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::size_type, _CharT, const _Alloc&) [with _CharT = char; _Traits = std::char_traits<char>; _Alloc = std::allocator<char>; std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::size_type = long unsigned int]
       basic_string(size_type __n, _CharT __c, const _Alloc& __a = _Alloc())
       ^~~~~~~~~~~~
/usr/include/c++/6/bits/basic_string.h:464:7: note:   candidate expects 3 arguments, 1 provided
/usr/include/c++/6/bits/basic_string.h:454:7: note: candidate: std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::basic_string(const _CharT*, const _Alloc&) [with _CharT = char; _Traits = std::char_traits<char>; _Alloc = std::allocator<char>] <near match>
       basic_string(const _CharT* __s, const _Alloc& __a = _Alloc())
       ^~~~~~~~~~~~
/usr/include/c++/6/bits/basic_string.h:454:7: note:   conversion of argument 1 would be ill-formed:
In file included from src/main.cpp:188:0:
src/variable.hpp:18:10: error: invalid conversion from ‘long unsigned int’ to ‘const char*’ [-fpermissive]
   return static_cast<std::string>(t);
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /usr/include/c++/6/string:52:0,
                 from /usr/include/c++/6/bits/locale_classes.h:40,
                 from /usr/include/c++/6/bits/ios_base.h:41,
                 from /usr/include/c++/6/ios:42,
                 from /usr/include/c++/6/istream:38,
                 from /usr/include/c++/6/fstream:38,
                 from src/main.cpp:27:
/usr/include/c++/6/bits/basic_string.h:444:7: note: candidate: std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::basic_string(const _CharT*, std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::size_type, const _Alloc&) [with _CharT = char; _Traits = std::char_traits<char>; _Alloc = std::allocator<char>; std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::size_type = long unsigned int]
       basic_string(const _CharT* __s, size_type __n,
       ^~~~~~~~~~~~
/usr/include/c++/6/bits/basic_string.h:444:7: note:   candidate expects 3 arguments, 1 provided
/usr/include/c++/6/bits/basic_string.h:426:7: note: candidate: std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::basic_string(const std::__cxx11::basic_string<_CharT, _Traits, _Alloc>&, std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::size_type, std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::size_type, const _Alloc&) [with _CharT = char; _Traits = std::char_traits<char>; _Alloc = std::allocator<char>; std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::size_type = long unsigned int]
       basic_string(const basic_string& __str, size_type __pos,
       ^~~~~~~~~~~~
/usr/include/c++/6/bits/basic_string.h:426:7: note:   candidate expects 4 arguments, 1 provided
/usr/include/c++/6/bits/basic_string.h:410:7: note: candidate: std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::basic_string(const std::__cxx11::basic_string<_CharT, _Traits, _Alloc>&, std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::size_type, std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::size_type) [with _CharT = char; _Traits = std::char_traits<char>; _Alloc = std::allocator<char>; std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::size_type = long unsigned int]
       basic_string(const basic_string& __str, size_type __pos,
       ^~~~~~~~~~~~
/usr/include/c++/6/bits/basic_string.h:410:7: note:   candidate expects 3 arguments, 1 provided
/usr/include/c++/6/bits/basic_string.h:397:7: note: candidate: std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::basic_string(const std::__cxx11::basic_string<_CharT, _Traits, _Alloc>&) [with _CharT = char; _Traits = std::char_traits<char>; _Alloc = std::allocator<char>]
       basic_string(const basic_string& __str)
       ^~~~~~~~~~~~
/usr/include/c++/6/bits/basic_string.h:397:7: note:   no known conversion for argument 1 from ‘const long unsigned int’ to ‘const std::__cxx11::basic_string<char>&’
/usr/include/c++/6/bits/basic_string.h:389:7: note: candidate: std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::basic_string(const _Alloc&) [with _CharT = char; _Traits = std::char_traits<char>; _Alloc = std::allocator<char>]
       basic_string(const _Alloc& __a) _GLIBCXX_NOEXCEPT
       ^~~~~~~~~~~~
/usr/include/c++/6/bits/basic_string.h:389:7: note:   no known conversion for argument 1 from ‘const long unsigned int’ to ‘const std::allocator<char>&’
/usr/include/c++/6/bits/basic_string.h:380:7: note: candidate: std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::basic_string() [with _CharT = char; _Traits = std::char_traits<char>; _Alloc = std::allocator<char>]
       basic_string()
       ^~~~~~~~~~~~
/usr/include/c++/6/bits/basic_string.h:380:7: note:   candidate expects 0 arguments, 1 provided
src/main.cpp:122:29: warning: ‘std::__cxx11::string eval_stream(std::istream&)’ used but never defined
   static std::string        eval_stream    (std::istream&             );
                             ^~~~~~~~~~~
Makefile:4: recipe for target 'all' failed
make: *** [all] Error 1

Yeah, I still use some modern C++17 features like std::string_view instead of std::string const&. By the way, have you tried compiling with -std=c++17? This could also fix the issue you had with std::filesystem, so the latest commit could just be undone.

I made a commit. You can now build with C++14. Try it now..

Sadly, now it fails faster.
$ make
mkdir -p bin
g++ src/main.cpp -o bin/zrc
In file included from src/main.cpp:195:0:
src/lex.hpp: In function ‘WordList tokenize(std::__cxx11::string, std::istream&)’:
src/lex.hpp:131:4: warning: attributes at the beginning of statement are ignored [-Wattributes]
    [[fallthrough]];
    ^
src/main.cpp: In function ‘WordList glob(std::string_view)’:
src/main.cpp:240:47: warning: attributes at the beginning of statement are ignored [-Wattributes]
  if (!glob(s.data(), GLOB_TILDE, NULL, &gvl)) [[unlikely]]
                                               ^
src/main.cpp:243:19: error: expected identifier before ‘[’ token
  if (!wl.size()) [[likely]]
                   ^
src/main.cpp: In lambda function:
src/main.cpp:244:3: error: expected ‘{’ before ‘wl’
   wl.add_token(s);
   ^~
src/main.cpp: In function ‘WordList glob(std::string_view)’:
src/main.cpp:244:3: error: expected ‘;’ before ‘wl’
src/main.cpp: In function ‘std::__cxx11::string eval_stream(std::istream&)’:
src/main.cpp:309:23: error: expected identifier before ‘[’ token
      if (*it == "^") [[unlikely]] *it = "2>"; if (*it == "^^") *it = "2>>";
                       ^
src/main.cpp: In lambda function:
src/main.cpp:309:35: error: expected ‘{’ before ‘*’ token
      if (*it == "^") [[unlikely]] *it = "2>"; if (*it == "^^") *it = "2>>";
                                   ^
src/main.cpp: In function ‘std::__cxx11::string eval_stream(std::istream&)’:
src/main.cpp:309:35: error: no match for ‘operator*’ (operand types are ‘eval_stream(std::istream&)::<lambda()>’ and ‘__gnu_cxx::__normal_iterator<std::__cxx11::basic_string<char>*, std::vector<std::__cxx11::basic_string<char> > >’)
      if (*it == "^") [[unlikely]] *it = "2>"; if (*it == "^^") *it = "2>>";

src/main.cpp:310:23: error: expected identifier before ‘[’ token
      if (*it == ">") [[unlikely]] *it = "1>"; if (*it == ">>") *it = "1>>";
                       ^
src/main.cpp: In lambda function:
src/main.cpp:310:35: error: expected ‘{’ before ‘*’ token
      if (*it == ">") [[unlikely]] *it = "1>"; if (*it == ">>") *it = "1>>";
                                   ^
src/main.cpp: In function ‘std::__cxx11::string eval_stream(std::istream&)’:
src/main.cpp:310:35: error: no match for ‘operator*’ (operand types are ‘eval_stream(std::istream&)::<lambda()>’ and ‘__gnu_cxx::__normal_iterator<std::__cxx11::basic_string<char>*, std::vector<std::__cxx11::basic_string<char> > >’)
      if (*it == ">") [[unlikely]] *it = "1>"; if (*it == ">>") *it = "1>>";

Makefile:4: recipe for target 'all' failed
make: *** [all] Error 1

Also, I don't understand your previous suggestion:
$ make -std=c++17
make: invalid option -- '='
make: invalid option -- 'c'
make: invalid option -- '+'
make: invalid option -- '+'
make: invalid option -- '1'
make: invalid option -- '7'

$ make -std='c++17'
make: invalid option -- '='
make: invalid option -- 'c'
make: invalid option -- '+'
make: invalid option -- '+'
make: invalid option -- '1'
make: invalid option -- '7'

I know nothing about C so building from source never works for me. :-(

Also, I don't understand your previous suggestion:
$ make -std=c++17
make: invalid option -- '='
make: invalid option -- 'c'
make: invalid option -- '+'
make: invalid option -- '+'
make: invalid option -- '1'
make: invalid option -- '7'

$ make -std='c++17'
make: invalid option -- '='
make: invalid option -- 'c'
make: invalid option -- '+'
make: invalid option -- '+'
make: invalid option -- '1'
make: invalid option -- '7'

I know nothing about C so building from source never works for me. :-(

You just modify the Makefile, and where you see a g++ command you just copy paste "-std=c++17" after it :P It's that easy.
I think that could have fixed everything with minimal effort, but I think that, for the time being, rewriting for compatibility is still better than forcing users to use C++17.

Sadly, now it fails faster.

But it's almost done. I just need to remove attributes. C++ has these things called attributes (the snippets marked with [[]]). They're annotations that make the code clearer to understand (but also "hints" to the compiler you can use to optimize certain cases). For example, you can do:
if (expression) [[likely]] {
    statement();
} else if (another_expression) [[unlikely]] {
    statement2();
    statement3();
    // ...
}

And the compiler will optimize for the second branch.
It just so happens that some of these attributes don't really seem to work on your configuration, and the compiler freaks out because these attributes were added in a newer C++ version. I'll make a commit that just removes them because they're a cheap way of optimizing anyway...

Yes! It works now. Thank you!
I will see if I can learn it now. I just went about using it as if it were Tkcon and of course it didn't work like I expected. I will read and test some more. Thank you again.

No problem. If you need any help/notice any weird behavior just open an issue.
Happy Tcl-ing! (or rather Zrc-ing, I guess, because this isn't really Tcl).


Auto regeneration doesn't work using Boot2Docker

Hi,
Here is the context:

I'm using a custom build of the Jekyll Docker image that just adds the Asciidoctor gem as you suggested, based on the latest tag of https://registry.hub.docker.com/u/jekyll/jekyll/
I'm on Windows 8.1 and Boot2Docker v1.6.0
My Jekyll site lives in C:\Users\<my-user>\my-jekyll-site. Note that Boot2Docker automatically mounts the C:\Users\ directory to /c/Users/ in the VM

So to run my site, I just go to /c/Users/<my-user>/my-jekyll-site (on my Boot2Docker VM) and run:
docker run --rm -v $(pwd):/srv/jekyll -p 4000:4000 dandelion/jekyll

Problem: as soon as any file is modified in C:\Users\<my-user>\my-jekyll-site, Jekyll doesn't seem to detect any change and so doesn't trigger a site regeneration, despite the log message:
root@a415d7f224d3:/srv/jekyll$ jekyll s
Configuration file: /srv/jekyll/_config.yml
            Source: /srv/jekyll
       Destination: /srv/jekyll/_site
      Generating...
                    done.
 Auto-regeneration: enabled for '/srv/jekyll'
Configuration file: /srv/jekyll/_config.yml
   Server address: http://<IP_ADDRESS>:4000/
  Server running... press ctrl-c to stop.

Any idea?
Thanks!

There is no reason to have a custom fork just for a gem, you can add it to the Gemfile and it will install it for you, see the README.md for details on that.  If we support ASCIIDoc by default (@parkr would have to confirm that because I don't ever use it) then I'd also be happy to include it in the default lineup since we want to target large audiences with this image and a parser is no big deal to add.
As far as the regeneration problem, I don't actually know if this is a problem on our end or a problem on their end, since I've never played with boot2docker I don't actually know if it even supports inotify on Linux or if we have to provide inotify ourselves (since this is typically provided by the Kernel.)  With that said, this will have to wait for a while until I can get on my Windows computer and test it out to see if it's a our problem or a their problem, but I can quickly check right now if it's an our problem by trying it on my LInux system.

Thanks for the tip @envygeeks, indeed it works like a charm using the Gemfile.
However, until the regeneration problem is fixed, I'll keep using my custom image, just because I don't want to recreate tons of containers everytime I need to generate the site. Currently I start a container with bash and hit CTRL+C / jekyll s to manually regenerate the site.
Regarding the regeneration problem, let me know if I can be of any help.
Thanks!

@tduchateau I'm curious does it work when you do --force-polling?

Nop, but with --force_polling it does! :-)
Is there any drawback using polling? Performance maybe?
One important note: jekyll --help doesn't mention this option. It would definitely help.
Huge THANK @envygeeks! :heart:

It's not listed because there are drawbacks but those only apply to extremely large sites, for the most part it won't affect you, it never has me when I needed to use it, not until I hit thousand page scale.

Not sure if it's relevant, but while working with Vagrant in the past on Windows I noticed some hiccups while mounting shared file systems. Specifically when working with dependency trees with lots of files and file watchers. My solution to that prob was to mount only raw source files to Windows so they could be modified with the Editor of choice and let the deps and watcher dump files into a non-shared space.

I'm pretty sure networked filesystems don't support inotify, I'd have to go back and check the API's but last I remember samba and the likes of what virtualbox would use (probably samba's libs) won't support any of the inotify api's which means there is nothing we can do.  I could try and detect the file system and force polling for you but that's not ideal at all.
If one of you wants to boot the docker image in the setup on Windows and give me some system information that it gives you I would know even further what the extents of what I can do is.  Basically I need to know how the VM mounts the system and then how Docker sees that system.

Actually a quick google search points to this being the case actually:

https://github.com/boot2docker/boot2docker/issues/688
http://stackoverflow.com/q/26956623/376678

There is nothing we can do so I added a note about --force-polling to the README.md

Just a note @envygeeks, if I'm not mistaken, there's a typo in the README: --force-polling should be --force_polling

I believe there's a PR out for that
On Fri, May 22, 2015 at 3:52 PM Thibault Duchateau<EMAIL_ADDRESS>wrote:

Just a note @envygeeks https://github.com/envygeeks, if I'm not
mistaken, there's a typo in the README: --force-polling should be
--force_polling
—
Reply to this email directly or view it on GitHub
https://github.com/jekyll/docker-jekyll/issues/14#issuecomment-104764724
.

Indeed, don't pay attention to my comment :-)


StopIteration always at epoch9 step2

Hello, I run into the StopIteration problem when using the fit_generator and do not know how to solve it. Here are the code related.
def generate_train_batch(self, seq_len, batch_size, normalise):
    '''Yield a generator of training data from filename on given list of cols split for train/test'''
    i = 0
    while i < (self.len_train - seq_len):  # 足够生成一个batch时
        x_batch = []
        y_batch = []
        for b in range(batch_size):
            if i >= (self.len_train - seq_len):
                # stop-condition for a smaller final batch if data doesn't divide evenly
                yield np.array(x_batch), np.array(y_batch)
                i = 0
            x, y = self._next_window(i, seq_len, normalise)
            x_batch.append(x)
            y_batch.append(y)
            i += 1
        yield np.array(x_batch), np.array(y_batch)

def train_generator(self, data_gen, epochs, batch_size, steps_per_epoch, save_dir,
                    data_val_gen=None, validation_steps=10):
    timer = Timer()
    timer.start()
    print('[Model] Training Started')
    print('[Model] %s epochs, %s batch size, %s batches per epoch' % (epochs, batch_size, steps_per_epoch))
    save_fname = os.path.join(save_dir, '%s-e%s.h5' % (dt.datetime.now().strftime('%d%m%Y-%H%M%S'), str(epochs)))
    callbacks = [
        ModelCheckpoint(filepath=save_fname, monitor='loss', save_best_only=True)
    ]

    if data_val_gen is None:
        validation_steps = None
    self.model.fit_generator(
        data_gen,
        steps_per_epoch=steps_per_epoch,
        epochs=epochs,
        callbacks=callbacks,
        workers=1,
        validation_data=data_val_gen,
        validation_steps=validation_steps
    )

    print('[Model] Training Completed. Model saved as %s' % save_fname)
    timer.stop()

model.train_generator(
    data_gen=data.generate_train_batch(
        seq_len=101,
        batch_size=64,
        normalise=True
    ),
    epochs=32,
    batch_size=64,
    steps_per_epoch=steps_per_epoch,
    save_dir="saved_models",
    data_val_gen=data.generate_validation_batch(
        seq_len=101,
        batch_size=64,
        normalise=True
    )
)

The training process always stops at epoch 9 and step 2:
Epoch 8/32
2378/2378 [==============================] - 1337s 562ms/step - loss: 0.3470 - acc: 0.8795 - top_k_categorical_accuracy: 0.9975 - val_loss: 0.7842 - val_acc: 0.7937 - val_top_k_categorical_accuracy: 0.9703
Epoch 9/32
2/2378 [..............................] - ETA: 22:07 - loss: 0.2629 - acc: 0.9297 - top_k_categorical_accuracy: 1.0000Traceback (most recent call last):
File "run.py", line 133, in
main()
File "run.py", line 103, in main
normalise=True
File "/home/yangguang/dev/IDPS/keras_model/model_no_configs/model_functional/model_RNN.py", line 73, in train_generator
validation_steps=validation_steps
File "/home/yangguang/machineLearning/venv4ML/lib/python3.6/site-packages/keras/legacy/interfaces.py", line 91, in wrapper
return func(*args, **kwargs)
File "/home/yangguang/machineLearning/venv4ML/lib/python3.6/site-packages/keras/engine/training.py", line 1436, in fit_generator
initial_epoch=initial_epoch)
File "/home/yangguang/machineLearning/venv4ML/lib/python3.6/site-packages/keras/engine/training_generator.py", line 179, in fit_generator
generator_output = next(output_generator)
StopIteration
I have tried to add " While True " in the generate_train_batch, and the training process will not raise StopIteration but will stuck at the same time(training will not continue at epoch 9 step 2).
I have update Keras to the latest Master version.
Could you please help me with this? Thanks a lot.

Add
while True
before your loop, otherwise the generator stops yielding at the second epoch


sntp_client for cc3220sf_launchxl

This PR adds support for cc3220sf_launchxl to the sntp_client sample.
Some modifications are made to the example to make IPv6 and NET_CONFIG_SETTINGS optional, since these are not currently working in the cc3220sf wifi driver implementation.

Codecov Report

Merging #14548 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #14548      +/-   ##
==========================================
- Coverage   52.02%   51.99%   -0.04%
==========================================
  Files         309      309
  Lines       45574    45574
  Branches    10555    10555
==========================================
- Hits        23711    23695      -16
- Misses      17057    17069      +12
- Partials     4806     4810       +4

Impacted Files
Coverage Δ

subsys/testsuite/ztest/include/ztest_assert.h
38.88% <0%> (-38.89%)
:arrow_down:

subsys/testsuite/ztest/src/ztest.c
65.28% <0%> (-7.44%)
:arrow_down:

Continue to review full report at Codecov.

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

NET_CONFIG_SETTINGS optional, since these are not currently working in the cc3220sf wifi driver implementation.

NET_CONFIG_SETTINGS isn't supported on cc3220? But that's one of the most basic networking sub-subsystems in general, and for samples in particular. Almost every sample uses it, do you intend to patch them all? Why the choice is on making such patches instead of implementing NET_CONFIG_SETTINGS support?
The specific commit message:

In this commit, a config.h header file is added to support the case
when the NET_CONFIG_SETTINGS is not used

should mention that this is being done to allow cc3220 support. Actually, I'm a bit proponent of detailed, well-separated changes, but here I'd say it might be better to squash the two commits together.

but here I'd say it might be better to squash the two commits together.

Please do not squash the patches, there is now two different changes which is ok in this case.

NET_CONFIG_SETTINGS isn't supported on cc3220? But that's one of the most basic networking sub-subsystems in general, and for samples in particular. Almost every sample uses it, do you intend to patch them all? Why the choice is on making such patches instead of implementing NET_CONFIG_SETTINGS support?

@pfalcon Implementing NET_CONFIG_SETTINGS fully is non-trivial, as far as I understand. As @GAnthony mentioned in #11889, NET_CONFIG_SETTINGS forces hardcoded values for IP address, which is not desired for an offload driver which already handles DHCP, and brings in other unnecessary functions. It is a current limitation in the SimpleLink driver where we do not support static IP addresses that are set via NET_CONFIG_SETTINGS. So you are right, I'd have to patch the samples that are supported on cc3220xl_launchxl. Fortunately some of them were already following this model where NET_CONFIG_SETTINGS may or may not be defined (e.g. mqtt_publisher), so I am simply following suit.

Updated commit message per comment from @pfalcon, and fixed a build issue in config.h.


undefined local variable or method `cama_root_url' with rails 5.0.

Hi @owen2345 ,
I am getting below error after upgrading to 5.0. this error I am seeing if my existing site has custom root route defined, this was working fine with 4.2.7 version. I never had this issue with any other rails version.
undefined local variable or method `cama_root_url' for #<CamaleonCms::FrontendController:0x007fd7e6306f90> Did you mean? cama_robots_url cama_post_url cama_rss_url cama_admin_forgot_url cama_error_url cama_category_en_url cama_search_url cama_logout_user cama_post_type_7_url cama_post_tag_en_url cama_post_type_2_url cama_robots_path cama_captcha_url cama_sitemap_url cama_ajax_url

  def ecommerce_front_before_load
    @ecommerce_post_type = current_site.post_types.where(slug: 'commerce').first.decorate
    @ecommerce_bredcrumb = [].push(["Home", cama_root_url])
  end

  def ecommerce_admin_before_load

/Users/mazhar/.rvm/gems/ruby-2.3.0@rails5.0.0/bundler/gems/camaleon-ecommerce-a8543f617be2/app/helpers/plugins/ecommerce/ecommerce_helper.rb:16:in `ecommerce_front_before_load'

I have below root route defined in my config/routes.rb file
 root  'high_voltage/pages#show', id: 'home'

Without above line camaleon_cms root route becomes the home page, but I wanted to have static home page provided by the high_voltage gem.
Can you please let me know how I can resolve it.

@mazharoddin,
I fixed the problem, please test using the git version before to publish a new gem version.
Regards,

Hi @owen2345,
Yeah site root issues is got resolved now, thanks. But seeing this error after pulling latest code.
pp/apps/themes/efashion/views/layouts/_header.html.erb where line #32 raised:

undefined local variable or method `current_user' for #<#<Class:0x007f9c7be0c8c8>:0x007f9c7be584f8>
Did you mean?  current_site

	                    <% if cama_current_user.present? %>
	                        <% cart = current_site.carts.set_user(current_user).first_or_create(name: "Cart by #{current_user.id}").decorate %>
						<a href="<%= plugins_ecommerce_checkout_cart_path %>">

							<%= cart.the_price  %>

app/apps/themes/efashion/views/layouts/_header.html.erb:32:in `_app_apps_themes_efashion_views_layouts__header_html_erb__1733713776537051105_70155033032040'
app/apps/themes/efashion/views/layouts/index.html.erb:18:in `_app_apps_themes_efashion_views_layouts_index_html_erb__393460728024828843_70155090220620'

Fixed the problem!

Thanks @owen2345, its working now.


Margin field for each item under the positions

@amit0rana thanks for this excellent tool. would it be possible to enhance betterKite with a Margin column for each item in the positions section? Combined Margin at the header & footer (when positions are selected) also would be very helpful.
Thanks for reading the request!!

This is an excellent feature, even I want it but there is no easy way to calculate how much margin a particular position utilizes.

I have enhanced your tool to fetch the margin.

I might need your support to add the margin value as a new column.

Nice.
Do you recalculate new margin keeping 'margin benefit' in mind? For eg if you select PE margin would be 1.7, only CE margin would be 1.4 but when you select both PE and CE margin would be 1.9only.
Given that most of the times we are interested combined margin, it would make sense to show it at the bottom where you are showing now. Individual margin is not very helpful, moreover adding a column is lot more coding :)

Yes, I do(rather the API I am using) re-calculate margin keeping margin benefit. Examples below:
Only PE

Once CE

PE & CE combined

Agree with you on the additional effort required to add a margin column. I will see if I can add that later when I have more time.

code changes are published part of PR#4


Sources: Operator metering login screen --> reporting-operator

Describe the bug
Wherever we talk about the 'Operator Metering' or 'metering-operator' service
account in this step of adding OCP sources, it should be the reporting-operator service account.
We also need to update the command in this screen:
oc serviceaccounts get-token reporting-operator > ocp_usage_token
Screenshot

For more info
See the Bluejeans recordings at the end of the Cost onboarding Google doc.
Check out the video from Fri 6/14/19 at around 56:00.
@adberglund, can you please verify that I have these details/names (ie. reporting-operator) right?

@boaz0 @dayleparker That is correct. Previously we were using the metering-operator service account but we moved to use the reporting-operator service account.

Cool, thanks a lot for approving this @adberglund

merging in 1..2..3..:bomb:


Possibility to avoid recompiling unchanged files

My projects includes lots of js bundles (around 10) and each time I change something in a bundle when I deploy to production it django-pipeline compiles all the bundles (each taking around 7 seconds.) I was wondering if it's possible to avoid this and only compile outdated bundles. I'm using a custom compiler and I'm checking outdated:
        if not force and not outdated:
            # File doesn't need to be recompiled
            return

But seems like in production environment django-pipeline forces the update. Am I missing something?

How did you solve this issue?
I'm struggling with the fact that the build (compilation of SCSS to CSS is performed each time). I tried setting PIPELINE_ENABLED to True and settings.Debug to False just to see if that would work on a production environment, but I get a 404 for each of css and js files for foundation framework even though it works well when in development.

@h4k1m0u it still compiles all files each time I deploy to production, didn't solve.

I guess I would have to put the compiled CSS link directly inside my html when I deploy it (although I thought that's not really the pipeline way).


[Chore] Add Github action to publish package

We need to add Github's actions to publish this repo as an npm package. We will need to set up an npm secret and create workflow YAML.
question

[ ] which npm account are we going to use? Could we create an organization account? 🤔

I think we should create an account for the organization.


Question about deubiquitinase activity

ubp8  is a deubiquitinase

There is a mapping to cysteine-type endopeptidase.
Should cysteine-type deubiquitinase be a cysteine-type endopeptidase? Otherwise the annotation transfer would seem to be incorrect?

Who would know this?

Actually deubiquitinase cleaves isopeptide bonds, while an endopeptidase cleaves peptide bonds. So, cysteine-type deubiquitinase  is not a type of cysteine-type endopeptidase.


Make rustc dependency optional

This is base on #96

My preference is to have new versions out-of-the-box behave the same as previous versions if possible, so if rustc is the default now, I think it should be the default in the new version.

Thanks for picking this up by the way, I thought the option was a good idea but it was never moved forward.

Thanks!


Update template.yaml

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Mistake?


Update README.md

Voilà Luc, j'ai fait quelques corrections :)

Codecov Report

Merging #28 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #28   +/-   ##
=======================================
  Coverage   93.93%   93.93%
=======================================
  Files          14       14
  Lines        2605     2605
=======================================
  Hits         2447     2447
  Misses        158      158

Continue to review full report at Codecov.

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


[Question] Parallel routes auto redirect

Hi, I'm quite new to flutter world and I've been trying something to implement automatic route redirect based on current authentication state. I haven't found material about that and most of them is to have manual implementation that in a authenticated route keep listening if was unauthenticated, and then navigate back.
Then I got more or less the result that I was looking applying something that I used in RN and Native projects, but I wonder if there is any suggested way to do so.
                          -> HomePage -> SettingsPage
App -> AuthenticationPage
                          -> LoginPage -> RegisterPage

When user login on LoginPage, it would navigate automatically to HomePage
When user logout on SettingsPage, it would navigate automatically to LoginPage
class AuthenticationPage extends StatefulWidget {
  const AuthenticationPage({Key key}) : super(key: key);

  @override
  _AuthenticationPageState createState() => _AuthenticationPageState();
}

class _AuthenticationPageState extends ModularState<AuthenticationPage, AuthenticationController> {
  @override
  void initState() {
    super.initState();
    // addPostFrameCallback is documented to be triggered only once, then listener is discarded
    WidgetsBinding.instance.addPostFrameCallback((_) => autorun((_) => _onRegisterNavigation()));
  }

  void _onRegisterNavigation() {
    final AuthenticationState authenticationState = controller.authenticationState;
    if (authenticationState is AuthenticatedAuthenticationState) {
      Get.offNamedUntil("/home", ModalRoute.withName(Modular.initialRoute)).then((_) => _onHandleRouteReturn());
    } else if (authenticationState is UnauthenticatedAuthenticationState) {
      Get.offNamedUntil("/login", ModalRoute.withName(Modular.initialRoute)).then((_) => _onHandleRouteReturn());
    } else {
      throw UnimplementedError("Not supported $authenticationState");
    }
  }

  void _onHandleRouteReturn() {
    if (!Get.key.currentState.canPop()) {
      SystemNavigator.pop();
    }
  }

  @override
  Widget build(BuildContext context) {
    return Container(
      width: 0,
      height: 0,
      color: Theme.of(context).backgroundColor,
    );
  }
}

Its implementation is totally correct! Are you experiencing any problems with this or just want to confirm that you are doing everything right? Everything seems to be ok for me, and even following good practices.
Personally use SchedulerBinding.instance instead of WidgetsBinding.instance, because it allows you to listen to the end of the build method outside a Stateful and create a "SplashScreen" screen (similar to your authentication screen) to be more user friendly and display a loading, and put all authentication logic in my class's controller.
I also use modular, and my approach is no more correct than yours, maybe it's a different, but equivalent standard, and your logic is correct.
Even though you were a beginner, you managed to do everything correctly. I just wouldn't use SystemNavigator.pop(); because I’ve already had apps rejected on the AppleStore for "unexpected app exit". Test this first on iOS, and see that there is backGesture, and if it makes the app exit, if so, change your code to:
if (! Get.key.currentState.canPop () && GetPlatform.isAndroid) {
      SystemNavigator.pop();
 }
In android this behavior is expected, but in iOS it goes against human guidelines, and you can face problems.
If your application is only android, there is no need to follow this tip.
If your application has a web/PWA version, don't use Flutter's Platform.isAndroid, it imports Dart: io which is incompatible with the WEB (I had to create GetPlatform precisely to give full web support, and I left this Get API exposed for those who need to use it).

Thanks for your feedback, I'll take a look in this SchedulerBinding documentatio -  I was trying to find something like that, but WidgetBinding was looking too much as ViewLayer stuff to put in the Controller/ViewModel
And yes, i just wanted to confirm if this approach is correct using your library.
I have to add this SystemNavigator.pop() because after user press back from homescreen it's redirect to this empty screen , that shouldn't be visible, and considering that /home /login are the first item of the navigation stack, I just wanted to close the app when reach this point.
Tried to handle with RouteObserver instead of listening a Future, but for some reason it wasn't been called when I get back to / path, was trying to read the source code, but haven't found the reason yet.


[JANSA] Updated Translation Files from 2020-07-11

Follow-up to #1654

@mzazrivec Please review.
@simaishi Please merge.


⚠ Bofrost.at has degraded performance

In ab83170, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1561 ms

Resolved: Bofrost.at performance has improved in 43bab79.


Progress Indicator in .xhtml / .txt file

Type of installation
Portable (.zip)
Steps to reproduce
1.Open any .txt / .xhtml file.
2. Shows page indicator instead of progress % when a single .xhtml / .txt file is opened.
✔ Expected Behavior
To show progress %.
❌ Actual Behavior
Shows page indicator.

It's still experimental. But I think that might not be possible. It will still show progress in zen-mode though.
It is not the only issue, and most likely other might not be fixable either, because of how it works,

As I said before in #202, its not really ideal to make a new type of reader, txt and xhtml files are opened in epub-reader but as a manga-item instead, because its structure is same as manga and not a epub, like having a main folder as "manga" and files as "chapter" and many other things.

In my opinion, it would be better to make the .xhtml files (As these are the chapter files). And do not bother about the stylesheet links, images in individual .xhtml files.

Im not linking anything, it's just single file.
The problem here is that, the progress/page-number is not part of reader, I use some global state to decide whether it will show progress or page-number, and that is just the data itself, which currently is either a Manga-Item or EPUB-Item.
Side-list and settings were easy to add as they are inside readers.

Wouldn't it be better to create a new reader.tsx file only for the individual .xhtml files.

It's not a bad solution, until you think about how many issues there are in each reader itself, then imagine if there is some book related problem, i will need to fix it for all the files, and on top of that, almost 90% of it will be same. And if you look at each reader.tsx file, they are very big. It will be hard to manage all that.
I'm already finding it hard to manage Yomikiru related things now. On my personal todo list there are over 10 bugs and around a few more things which work but not like it should, for example I recently added ability to click alphabet to focus element on a select menu, there are lot of such issues left.


Revert asyncificaiton changes

The asyncification changes caused some non-deterministic test failures. I believe that some of these are due to race conditions that are the result of the partial transition to actors.
Instead of merging the asyncification piece by piece, I will collect the changes asyncification changes in a branch and then qualify that branch througougly (running CI multiple times) before merging it into main.
This resets the state of main back to 73bbba385a5d7b45d9f7659a45c5b22a9bb29da8, which was the last commit before all the asyncification changes. Since then, only asyncification PRs have been made to main.

@swift-ci Please test


Fallback for Network Images

When using a image widget with network resource, it will be desirable to be able to specify a fallback widget so that the user is shown a placeholder as things load.
Or just being able to know whether the Network image has been completed loading or not will be useful.
@Hixie

+1. I'm also looking for an onLoad callback for bundled asset images.

Is there a known workaround for this in the meantime? Or are we stuck?

I don't know of any workaround yet.

Actually, a potential workaround for my scenario may be to use a fixed timeout (~500ms).

I think this goes into "make early adopters happy", as it would definitely make one of our current early adopters happy :)

+1 I would like to see this working

Pink currently uses has a workaround by putting the image in stack and when the image fails to load it shows the fallback. :/

@FaisalAbid might be taking a look at this.

Yep, I need it for a project so might as well implement it myself

Cool! You'll be directly helping pretty awesome stuff here.

@FaisalAbid let me know if you still want this or if I should find an owner on my team. :)

Hey! I got it working for the most part internally, will send PR soon after
some cleanup
On Tue, Mar 28, 2017, 6:33 PM Eric Seidel<EMAIL_ADDRESS>wrote:

@FaisalAbid https://github.com/FaisalAbid let me know if you still want
this or if I should find an owner on my team. :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/6229#issuecomment-289925255,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPfBFDEJtNS_U4YRuaAO2LoNUZvSU5-ks5rqYrKgaJpZM4KPd-g
.

--
Sent from my phone

Hey @eseidel - Network image loader
https://github.com/FaisalAbid/pluto
Sorry for the delay, had to extract it from our codebase

@dvdwasibi does @FaisalAbid's package solve your original desire?  Unclear to me if we should keep this open to pull something like pluto into the main Flutter framework, or if having a package is sufficient? :)

I am happy to take feature requests as well!
On Thu, May 11, 2017, 3:05 PM Eric Seidel<EMAIL_ADDRESS>wrote:

@dvdwasibi https://github.com/dvdwasibi does @FaisalAbid
https://github.com/faisalabid's package solve your original desire?
Unclear to me if we should keep this open to pull something like pluto into
the main Flutter framework, or if having a package is sufficient? :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/6229#issuecomment-300887062,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPfBG5ocAm_AjC2mnGwQ-AwD3dmBypSks5r41v_gaJpZM4KPd-g
.

--
Sent from my phone

@FaisalAbid great stuff. didn't notice it up on the Flutter packages page. again... 👍

We have FadeInImage now which supports a fallback image (not widget). Closing this bug for now. If you do need a fallback widget (e.g. a progress indicator), please file a bug describing the use case. Thanks!


Update nvm

installed here
Current version: 0.31.1. I want to clear messages like this:
gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR

This message comes from node-gyp. However, an update to nvm is still needed.

we should make LTS the default version if not specified, rather than latest:
https://github.com/creationix/nvm#long-term-support


Straight almost installs evil correctly

What's wrong
Something I have not yet been able to identify is going wrong when straight installs evil. It is causing some errors in evil output. The same is not the case when evil is installed with package.el.
So I ripped apart my config tracking down this problem only to, quite surprisingly, find that it comes down to how straight is installing evil.
I am still going to guess that its something trivial I am overlooking but I have now gotten it to the point I can reproduce the bug with emacs -Q.
I include two very related scripts that are intended to be run with emacs -Q. evil-debug.el (which uses package.el to install evil) and evil-debug-straight.el (which uses straight.
The correct output for either script should be:
const something = {
       }

        const else = {
        }

        const c = 'bar'
        const a = 'hello'
        const b = 'foo'

Steps to reproduce:

start emacs -Q
open either files
M-x eval-buffer
Press enter to run the evil-ex command that is prepopulated in the minibuffer.
Look at the new opened buffer "foo" and compare to correct output listed above.

Only the script that installs evil using package.el seems to get the correct result.
Version information

Emacs version: 28.2
Operating system: Arch Linux

Scripts
evil-debug.el
;; Set up package.el to work with MELPA
(require 'package)
(add-to-list 'package-archives
             '("melpa" . "https://melpa.org/packages/"))
(package-initialize)
(package-refresh-contents)

;; Download Evil
(unless (package-installed-p 'evil)
  (package-install 'evil))

;; Enable Evil
(require 'evil)
(evil-mode 1)

(setq evil-ex-search-vim-style-regexp t)

(switch-to-buffer "foo")
(insert "const something = {
        a:  'x',
        b:  'x',
        c:  'x',
        d:  'x',
        }

        const else = {
        a: 'x',
        b: 'x',
        c: 'x',
        d: 'x',
        }

        const c = 'bar'
        const a = 'hello'
        const b = 'foo'")

(evil-ex "g/\\{/+1,/\\}/-1d")

evil-debug-straight.el
;; Set up package.el to work with MELPA
(defvar bootstrap-version)
(let ((bootstrap-file
       (expand-file-name "straight/repos/straight.el/bootstrap.el" user-emacs-directory))
      (bootstrap-version 6))
  (unless (file-exists-p bootstrap-file)
    (with-current-buffer
        (url-retrieve-synchronously
         "https://raw.githubusercontent.com/radian-software/straight.el/develop/install.el"
         'silent 'inhibit-cookies)
      (goto-char (point-max))
      (eval-print-last-sexp)))
  (load bootstrap-file nil 'nomessage))

(straight-use-package 'evil)
(require 'evil)
(evil-mode 1)

(setq evil-ex-search-vim-style-regexp t)

(switch-to-buffer "foo")
(insert "const something = {
        a:  'x',
        b:  'x',
        c:  'x',
        d:  'x',
        }

        const else = {
        a: 'x',
        b: 'x',
        c: 'x',
        d: 'x',
        }

        const c = 'bar'
        const a = 'hello'
        const b = 'foo'")

(evil-ex "g/\\{/+1,/\\}/-1d")

A diff of the two directories produces the following on my machine:
n@nbook:evil $ diff ./ /tmp/evil-package.el/elpa/evil-20221122.1450/
diff ./evil-autoloads.el /tmp/evil-package.el/elpa/evil-20221122.1450/evil-autoloads.el
7a8,10
> (add-to-list 'load-path (directory-file-name
>                          (or (file-name-directory #$) (car load-path))))
>
diff ./evil.info /tmp/evil-package.el/elpa/evil-20221122.1450/evil.info
1c1
< This is evil.info, produced by makeinfo version 7.0 from evil.texi.
---
> This is evil.info, produced by makeinfo version 6.7 from evil.texi.
diff ./evil-pkg.el /tmp/evil-package.el/elpa/evil-20221122.1450/evil-pkg.el
1,4c1
< (define-package
<   "evil"
<   "1.15.0"
<   "Extensible Vi layer for Emacs."
---
> (define-package "evil" "20221122.1450" "Extensible Vi layer for Emacs."
7c4,12
<     (cl-lib "0.5")))
---
>     (cl-lib "0.5"))
>   :commit "2e8576188b1d0768fbf92c6bea2fb3fbed9f019f" :maintainer
>   '("Tom Dalziel"<EMAIL_ADDRESS>>   :keywords
>   '("emulation" "vim")
>   :url "https://github.com/emacs-evil/evil")
> ;; Local Variables:
> ;; no-byte-compile: t
> ;; End:
Only in ./: evil-pkg.elc
Only in ./: evil.texi

Running your tests on Emacs 29 I get the same output for both.

package.el:

        a:  'x',
        b:  'x',
        c:  'x',
        d:  'x',
        }

        a: 'x',
        b: 'x',
        c: 'x',
        d: 'x',
        }

        const c = 'bar'
        const a = 'hello'
        const b = 'foo'

straight.el:

        a:  'x',
        b:  'x',
        c:  'x',
        d:  'x',
        }

        a: 'x',
        b: 'x',
        c: 'x',
        d: 'x',
        }

        const c = 'bar'
        const a = 'hello'
        const b = 'foo'

The docstring of evil-ex-search-vim-style-regexp suggests your repro cases may be incomplete:
 Note that this only affects the search command if
‘evil-search-module’ is set to ’evil-search. The isearch module
always uses plain Emacs regular expressions.

However, changing the default value of that variable did not change the output for me.

hmm interesting. Yes you are correct, that I needed to have added:
(evil-select-search-module 'evil-search-module 'evil-search) althought interstingly that didn't change the results in either case.
For me I get the correct output from evil when installed with use-package.el....
I get exactly the output you show if I use straight. But I get the correct output if I use package.el

I just saw you are using emacs 29. Could be thats the reason for the difference?

Possibly. You may be able to find the difference by instrumenting the evil-ex call with edebug and stepping through the execution of both.

Looks like this is being addressed upstream.
I'll close for now. Let me know if we need to re-open.


Update disabled scrollbars to get a focus when focusableScrollbar is true

Checklist

[x] I have read and understand the contribution guide
[x] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
By enabling focus on disabled items, a focus should be able to move from a scrollbar button to the opposite button by page up/down key if focusableScrollbar is true in a scroller or a list.
Resolution
Update a key handling logic to move a focus.
Links
PLAT-78718

LGTM


Window painting over border when clicked on

When I hover over a window the border appears as usual. However, when I click on the window or interact with it somehow, it seems that the window repaints over the border and looks smaller.
This only started happening after I upgraded from bf3a656 to new commit 3e99cee.

Add yabai -m config window_border_topmost 1 to get back the old behaviour.

Does yabai -m config active_window_border_topmost on require SIP to be off?

This setting no longer exists. It was a part of the border implementtatio back in version 2.x.x.
In the latest version (3.x.x), window borders require sip to be disabled.

I ran into a bug that was at least superficially similar, after updating to Big Sur, but seem to have fixed it.
In my case, I think I could only see the borders at the four corners of each window. I'm also not sure what was causing the broken border state, but it seemed to persist until restarting yabai, then inevitably come back after a little while.
I think there's some chance uncommenting the two lines at the top of .yabairc for Big Sur (to load the scripting addition), plus an additional yabai restart, is what fixed it.

In case it's helpful to anyone, I'm on Big Sur and also had the issue of borders being behind windows once you click on them. The "top two lines" @braham-snyder is talking about are these:
sudo yabai --load-sa
yabai -m signal --add event=dock_did_restart action="sudo yabai --load-sa"

Adding these to my yabairc fixed the issue for me!


strings.stripTags() uses simple regex and is therefore unsafe

There are a lot of references why the current regex is insufficient to strip tags from a string:

http://stackoverflow.com/questions/5788527/is-strip-tags-vulnerable-to-scripting-attacks/5793453#5793453
https://www.npmjs.com/package/striptags
http://www.hinnerk-altenburg.de/weblog/strip-all-html-tags-with-perl-regular-expression-regexp-like-php-strip_tags-does/
https://github.com/OWASP/java-html-sanitizer/

What to do about stripTags? I only see a chance to enhance it or to drop it and separate packages can do the job. IMHO the stripTags() approach is broken by design, since an application should normally escape the user input and not call strip tags, since a user might want to display  for some reason.
As an alternative I suggest to port striptags to ringo/utils/strings. It's MIT-based license is compatible with Ringo's Apache License.

Btw. the current unit test is ridiculous:
exports.testStripTags = function () {
    assert.strictEqual('content', strings.stripTags('<tag>content</tag>'));
};

:+1: for dropping stripTags.

I think this can be done together with dropping unwrap() from #311, or?


[JENKINS-26107] StepContext.hasBody

API needed for JENKINS-26107.
@reviewbybees esp. @kohsuke

:bee:

:bee:

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.


fix example for GraphQLInterfaceType

Fixes the GraphQLInterfaceType example in README.md

Thanks a lot for fixing this!


Inline init method.

Failed on assignment to final field.

@geirst Probably a bug introduced in our last rebase.

👍


TRANSCEIVER-6: Telemetry: 400ZR Optics Q-value streaming

README completed at #2374
Ready for Automation


Update index.ts (Change DRAG_AND_DROP_KEY value)

DRAG_AND_DROP_KEY 'dragged tag' throws an error (Element not found) in IE and Edge, changing the value to 'Text' fixes the problem.

Codecov Report

Merging #423 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #423   +/-   ##
=======================================
  Coverage   60.76%   60.76%
=======================================
  Files          13       13
  Lines       36554    36554
  Branches      143      143
=======================================
  Hits        22213    22213
  Misses      14265    14265
  Partials       76       76

Impacted Files
Coverage Δ

modules/core/constants/index.ts
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Hi @DkArteid,
thanks for the PR! Can you update the package.json's version? should be 1.3.2 :)

Hey @Gbuomprisco,
package.json's version updated to 1.3.2

@DkArteid merged :)


Revert "Dev/rebase migration"

Let's not commit unfinished feature to main.

This issue was migrated from an old repository: https://github.com/CiscoDevNet/catalystwan/pull/513


run imageoptim on banner image

Ran imageoptim against the banner image to reduse its size (lossless compression)

Coverage remained the same at 84.282% when pulling edbffd08c147604a0ea4e9af32acf00ee9389202 on feature/optimise-image-banner into 598323571a66a7de01130587fabc332d3e350490 on develop.


Package Manager UI: Update button update logic needs to properly compare versions

Right now it's not checking that the version available is greater than the installed version, it's just checking that it's different (so we'll enable update if it's a downgrade).

Version strings sometimes have weird -, which appear to be treated as if they were . for purpose of comparison.


Test Task - Testing topic related functions in Service/Controller - Create Topic/Delete Topic

Objective: Test the functionality related to topic management in the service.
Tasks:

Create Topic: Ensure that new topics can be successfully created.
Delete Topic: Verify that topics can be deleted properly.

Requirements:
[x] Validate the creation of a topic with valid data.
[x] Check for errors when creating a topic with invalid data.
[x] Confirm that deleting a topic removes it from the database.
[x] Ensure appropriate error handling for delete operations on non-existent topics.

commit 08f404cefe715782b8b7e4630d2b110eefb69d3e


Build-ScopeTableForDeploymentRootScope.ps1 tries to enumerate subscriptions in different tenant via Lighthouse

Describe the bug
Newer version (10x) seems to be trying to get access to management groups for subscriptions which are not in current tenant (via Lighthouse) and fails as a result. Example is below where my account is having access to some resources in external tenant via Lighthouse
To Reproduce
Onboard external resource via Lighthouse into tenant where EPAC shall be working on
Expected behavior
This behavior shall not exist at all since policies are being applied only to current tenant
Screenshots

EPAC Version
10.0.2

@artisticcheese - Can you provide the failure output from the pipeline or debugger?
Are you running EPAC against the lighthouse (managed) tenant or the "parent" tenant?

Here is exact error. This is running under service principal (5b3a2e7a-XXXX-4cec-957a-43e2826d5de0) in Tenant 1 which is through Lighthouse have access to subscription (43909631-XXXX-4b09-9fdb-706fc002ae5f) in Tenant2.
Loaded Module 'Az.Accounts'
Loaded Module 'Az.MSGraph'
Get-AzManagementGroup: /home/vsts/work/1/s/Scripts/Helpers/Build-ScopeTableForDeploymentRootScope.ps1:106
Line |
 106 |  … mentGroup = Get-AzManagementGroup -GroupName $deploymentRootScopeMana …
     |                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | The client '5b3a2e7a-XXXX-4cec-957a-43e2826d5de0' with object id
     | '5b3a2e7a-XXXX-4cec-957a-43e2826d5de0' does not have authorization to
     | perform action 'Microsoft.Management/register/action' over scope
     | '/subscriptions/43909631-XXXX-4b09-9fdb-706fc002ae5f' or the scope is
     | invalid. If access was recently granted, please refresh your credentials.

Add this subscription to exclusion does not help either
 {
         "pacSelector": "prod-environment",
         "cloud": "AzureCloud",
         "tenantId": "d7585f83-XXXX-492f-9b2f-34717df7700c",
         "keepDfcSecurityAssignments": true,
         "deploymentRootScope": "/providers/Microsoft.Management/managementGroups/DevOps-mg", // replaces rootScope
         "globalNotScopes": [
            "/providers/Microsoft.Management/managementGroups/security-mg",
            "/subscriptions/43909631-XXXX-4b09-9fdb-706fc002ae5f"
         ],
         "desiredState": {
            "keepDfcSecurityAssignments": true,
            "strategy": "ownedOnly",
            "excludedScopes": [
               "/providers/Microsoft.Management/managementGroups/security-mg"
            ],
            "excludedPolicyDefinitions": [
            ],
            "excludedPolicySetDefinitions": [
            ],
            "excludedPolicyAssignments": [
            ]
         },
         "managedIdentityLocation": "eastus2"
      }
   ]

This seems to be caused what is default context is chosen for Connect-AzAccount cmdlet. If default subscription is not specified and subscription in Lighthouse tenant is chosen then  Get-AzManagementGroup would fail. You can see reproduction scenario below. Default subscription in this case after Login-AzAccount is executed and set to subscription in Lighthouse tenant and first call to Get-AzManagementGroup cmdlet fails, if I switch AzContext to subscription in home tenant then issue is not present.
PS C:\repo\azure-policies-infra> Get-AzSubscription

Name                              Id                                   TenantId                             State
----                              --                                   --------                             -----
InfoSec Subscription - Production 43909631-XXXX-4b09-9fdb-706fc002ae5f d7585f83-XXXX-492f-9b2f-34717df7700c Enabled
Dev controlplane 1                45b6efbb-XXXX-4a2b-915c-fb954716b0d2 d7585f83-XXXX-492f-9b2f-34717df7700c Enabled
Control Plane 1                   0973ccdf-XXXX-4438-8509-67c1f84ae5a5 d7585f83-XXXX-492f-9b2f-34717df7700c Enabled
Prod 1                            9893b814-XXXX-4d2e-a2aa-cc8d383953ec d7585f83-XXXX-492f-9b2f-34717df7700c Enabled
Sandbox 1                         820b3d00-XXXX-4adf-b34b-a3fe8ca88aca d7585f83-XXXX-492f-9b2f-34717df7700c Enabled

PS C:\repo\azure-policies-infra> Get-AzManagementGroup -GroupName "DevOps-mg" -Expand -Recurse
Get-AzManagementGroup: The client '5b3a2e7a-XXXX-4cec-957a-43e2826d5de0' with object id '5b3a2e7a-XXXX-4cec-957a-43e2826d5de0' does not have authorization to perform action 'Microsoft.Management/register/action' over scope '/subscriptions/43909631-XXXX-4b09-9fdb-706fc002ae5f' or the scope is invalid. If access was recently granted, please refresh your credentials.
PS C:\repo\azure-policies-infra> Get-AzContext

Name                                     Account                                             SubscriptionName                                   Environment                                        TenantId
----                                     -------                                             ----------------                                   -----------                                        --------
InfoSec Subscription - Production (4390… 4ec14295-XXXX-4a8c-95d4-3abc6cf33776                InfoSec Subscription - Production                  AzureCloud                                         d7585f83-XXXX-492f-9b2f-34717df7700c

PS C:\repo\azure-policies-infra> Set-AzContext -Subscription "Dev controlplane 1"

Name                                     Account                                             SubscriptionName                                   Environment                                        TenantId
----                                     -------                                             ----------------                                   -----------                                        --------
Dev controlplane 1 (45b6efbb-2c18-4a2b-… 4ec14295-XXXX-4a8c-95d4-3abc6cf33776                Dev controlplane 1                                 AzureCloud                                         d7585f83-XXXX-492f-9b2f-34717df7700c

PS C:\repo\azure-policies-infra> Get-AzManagementGroup -GroupName "DevOps-mg" -Expand -Recurse

Id                : /providers/Microsoft.Management/managementGroups/DevOps-mg
Type              : Microsoft.Management/managementGroups
Name              : DevOps-mg
TenantId          : d7585f83-XXXX-492f-9b2f-34717df7700c
DisplayName       : DevOps Management Group
UpdatedTime       : 7/28/2022 3:30:32 PM
UpdatedBy         : bd5cfa79-0cca-4045-9265-564bbaee64f4
ParentId          : /providers/Microsoft.Management/managementGroups/d7585f83-XXXX-492f-9b2f-34717df7700c
ParentName        : d7585f83-XXXX-492f-9b2f-34717df7700c
ParentDisplayName : Tenant Root Group
Children          : Live Management Group

Parent of bottom 3

OK, I think the fix you put in just happens to work, but in many situations may not be the case.  I think if you were to directly add a new subscription under the "Security Management Group" MG in the not-lighthouse tenant, and you were to set the context to that subscription, you would get the same error.  I don't think (Still trying to see if I can find a way to verify this) it has to do with the fact that the subscription set in your context has a different "HomeTenantId", I think it has to do with the fact that it is under the "Security Management Group" MG and you have less access (e.g. don't have 'Microsoft.Management/register/action') at that scope, which going off the image appears to be the case; Resource Access vs. Account Admin

Can you send the json for the resource access role?

I'm confused, you want me to send JSON of built-in Reader role?

No, the resource access role

I do not have access to that subscription at all, only to some resources in that subscription

Can you...

Go to the Security Management Group
Access Control
Roles tab
Search for "resource access " role
View details
json tab

Can you email me at<EMAIL_ADDRESS>to set up some time for a call?

@artisticcheese - Can you try testing this proposed fix in your environment?
fix546.zip

Yes, this works

Fixed in v10.1.0


uninitialized constant Aws::Ec2::Fog

Currently released version (0.1.9) has the following bug (which might have been solved already by https://github.com/SearchSpring/aws_security/pull/36 - not sure):
* aws_security_group[<REMOVED>] action create_and_attach
           * chef_gem[fog-aws] action install (up to date)

           ================================================================================
           Error executing action `create_and_attach` on resource 'aws_security_group[<REMOVED>]'
           ================================================================================

           NameError
           ---------
           uninitialized constant Aws::Ec2::Fog

           Cookbook Trace:
           ---------------
           /tmp/kitchen/cache/cookbooks/aws_security/libraries/ec2.rb:50:in `create_aws_interface'
           /tmp/kitchen/cache/cookbooks/aws_security/libraries/ec2.rb:23:in `ec2'
           /tmp/kitchen/cache/cookbooks/aws_security/providers/group.rb:79:in `security_group'
           /tmp/kitchen/cache/cookbooks/aws_security/providers/group.rb:75:in `load_current_resource'
           /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/monkeypatches/chef/runner.rb:78:in `run_action'
           /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/monkeypatches/chef/runner.rb:106:in `block (2 levels) in converge'
           /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/monkeypatches/chef/runner.rb:106:in `each'
           /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/monkeypatches/chef/runner.rb:106:in `block in converge'
           /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/monkeypatches/chef/runner.rb:105:in `converge'

Maybe re-publish to supermarket.chef.io?

This was fixed in #36
A new release needs to get pushed up to the supermarket


invalid box files

In saved box files the space char is missing; all box files containing a space are saved as
space left space bottom space right  space stop
instead of
space space left space bottom space right  space stop

space is delimiter.
IMO it is noses to ask tesseract train contour from space...

Well, I noticed just because  text2image creates training boxfiles with boxes for spaces and sometimes they need to be corrected... Yep, it's dumb, but useful for automatic training


Informix table information: References

When I clicked on Tab References dbeaver did some work and said:

Please post error stacktrace (you can get it in Error Log view or in debug log - https://github.com/serge-rider/dbeaver/wiki/Log-files)

eclipse.buildId=unknown
java.version=1.8.0_111
java.vendor=Oracle Corporation
BootLoader constants: OS=win32, ARCH=x86_64, WS=win32, NL=ru_RU
Command-line arguments:  -os win32 -ws win32 -arch x86_64
org.jkiss.dbeaver.model
Error
Tue Aug 08 12:08:38 MSK 2017
java.sql.SQLException
at com.informix.util.IfxErrMsg.buildException(IfxErrMsg.java:474)
at com.informix.util.IfxErrMsg.getSQLException(IfxErrMsg.java:443)
at com.informix.util.IfxErrMsg.getSQLException(IfxErrMsg.java:394)
at com.informix.jdbc.IfxSqli.receiveError(IfxSqli.java:3477)
at com.informix.jdbc.IfxSqli.dispatchMsg(IfxSqli.java:2379)
at com.informix.jdbc.IfxSqli.receiveMessage(IfxSqli.java:2304)
at com.informix.jdbc.IfxSqli.executeFetch(IfxSqli.java:2038)
at com.informix.jdbc.IfxSqli.getaRow(IfxSqli.java:4082)
at com.informix.jdbc.IfxResultSet.next(IfxResultSet.java:494)
at com.informix.jdbc.IfxDatabaseMetaData.getColumns(IfxDatabaseMetaData.java:3533)
at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCDatabaseMetaDataImpl.getColumns(JDBCDatabaseMetaDataImpl.java:1155)
at org.jkiss.dbeaver.ext.generic.model.TableCache.prepareChildrenStatement(TableCache.java:129)
at org.jkiss.dbeaver.ext.generic.model.TableCache.prepareChildrenStatement(TableCache.java:1)
at org.jkiss.dbeaver.model.impl.jdbc.cache.JDBCStructCache.loadChildren(JDBCStructCache.java:96)
at org.jkiss.dbeaver.model.impl.jdbc.cache.JDBCStructCache.getChild(JDBCStructCache.java:230)
at org.jkiss.dbeaver.ext.generic.model.GenericTable.getAttribute(GenericTable.java:156)
at org.jkiss.dbeaver.ext.generic.model.GenericTable.loadReferences(GenericTable.java:371)
at org.jkiss.dbeaver.ext.generic.model.GenericTable.getReferences(GenericTable.java:185)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at org.jkiss.dbeaver.model.navigator.DBNDatabaseNode.extractPropertyValue(DBNDatabaseNode.java:793)
at org.jkiss.dbeaver.model.navigator.DBNDatabaseNode.loadTreeItems(DBNDatabaseNode.java:479)
at org.jkiss.dbeaver.model.navigator.DBNDatabaseNode.loadChildren(DBNDatabaseNode.java:403)
at org.jkiss.dbeaver.model.navigator.DBNDatabaseNode.getChildren(DBNDatabaseNode.java:190)
at org.jkiss.dbeaver.model.navigator.DBNDatabaseNode.getChildren(DBNDatabaseNode.java:1)
at org.jkiss.dbeaver.ui.navigator.NavigatorUtils.getNodeChildrenFiltered(NavigatorUtils.java:563)
at org.jkiss.dbeaver.ui.controls.itemlist.ItemListControl$ItemLoadService.evaluate(ItemListControl.java:229)
at org.jkiss.dbeaver.ui.controls.itemlist.ItemListControl$ItemLoadService.evaluate(ItemListControl.java:1)
at org.jkiss.dbeaver.ui.LoadingJob.run(LoadingJob.java:86)
at org.jkiss.dbeaver.ui.LoadingJob.run(LoadingJob.java:71)
at org.jkiss.dbeaver.model.runtime.AbstractJob.run(AbstractJob.java:95)
at org.eclipse.core.internal.jobs.Worker.run(Worker.java:55)

It happens sometimes.

This seems to be driver-related problem (https://stackoverflow.com/questions/42509044/schemaspy-error-message-while-generating-the-report)
Could you try to use different Informix driver version?

Yes, it works in version 3.70.JC8DE and not works in 4.10.JC8DE and 4.10.JC9DE

With Informix driver 4.10.JC12 works fine for me (dbeaver 6.0)

Informix driver 4.10.JC12 not works for me (dbeaver up to 6.01) ( for tables with tens references to its )

Informix driver 4.10.JC12 not works for me (dbeaver up to 6.01) ( for tables with tens references to its )

Weird, I have here a table with 120 references and many other tables with +50 references where all them show the references without problem.
Did you have IBM support subscription?
My suggestion is open a CASE there since really looks like a driver defect.
Unfortunately (or not) I'm not able to reproduce here, otherwise, I opened a CASE for that....
I'm using Informix Server v12.10.FC12
If you have other versions of the Informix server for test to validate if the problem is really at the JDBC driver or the version of the Informix server you are using.
Regards
Cesar

I'm using Informix Server 12.10.FC7

The problem here is with some versions of the Informix JDBC driver.  Quick fix is to upgrade to the latest 4.50.1 driver.  It's out on maven.
<!-- https://mvnrepository.com/artifact/com.ibm.informix/jdbc --> <dependency>     <groupId>com.ibm.informix</groupId>     <artifactId>jdbc</artifactId>     <version>4.50.1</version> </dependency>
In detail.  The 4.10.X drivers introduced a statement cleaner thread that runs every 15 seconds and looks for resources that need to be closed as they have no more references and could be garbage collected. The problem, then compounds as a couple JDBC metdata calls incorrectly failed to close resources causing this cleaner thread to close resources in the middle of the work sometimes.  Hence some people see it often, some never. You can work around it with 4.10.JC12 where we added a connection flag to disable the cleaner thread (IFMXCONNECTION_CLEANER_THREADS=0).  Or the best option is to upgrade to the newest 4.50.1 driver which has this issue properly fixed.

Many thanks to Brian.
Newest driver 4.50.1 cleaned resources precisely ( workaround is OK too )
I'll close issue.