Option to keep submit errors on change

See #1708
The change to the if (valid) check in handleSubmit.js is still a bit of an hack. I think the cleanest solution would be to call validate there, similar to asyncValidate.

Current coverage is 100% (diff: 100%)

Merging #1735 into master will not change coverage

@@           master   #1735   diff @@
=====================================
  Files          52      52
  Lines        1115    1118     +3
  Methods         0       0
  Messages        0       0
  Branches        0       0
=====================================
+ Hits         1115    1118     +3
  Misses          0       0
  Partials        0       0

Powered by Codecov. Last update 9186bd3...0d3520d

It's API surface area creep, but it seems reasonable and the PR is excellent.

The persistentSubmitErrors feature is something that I am really keen on. I was wondering when this might be published to npm?

Published in v6.0.3.


Bag Generic Typing

I'd like to make dask.bag.Bag a generic type, so that type checkers and function signatures could include information about what's in a dask bag. For example
def add_lengths(bag: dask.bag.Bag[list[dict[str, Any]]]) -> dask.bag.Bag[list[dict[str, Any]]]:
    return bag.map(add_length)

def add_length(item: dict[str, Any]) -> dict[str, Any]:
    item["length"] = len(item["document"])

I would be happy to write the PR for this, but I wanted to make sure that y'all would be interested in that, and to see if there are any foreseeable pitfalls here.

Hi @jordan-jack-schneider 👋
I don't mind.  I imagine other folks would enjoy it as well.  If you wanted to be careful though you'd wait until Monday.  Some of the folks who probably care about this may be off enjoying their weekend already.

The factory functions for creating Bag objects will hit the problem of https://github.com/python/mypy/issues/6073 due to functions not being subscriptable, and require somewhat nasty workarounds as done in trio: https://github.com/python-trio/trio/blob/master/trio/_channel.py#L90 plus https://github.com/python-trio/trio/blob/master/trio/_util.py#L274

I started looking at this, and some basic cases and methods can definitely be typed, but I'm quickly hitting problems.
But several methods aren't possible to fully type without higher kinded types, especially for cases when Bag is subclassed. e.g. starmap and map_partitions should return Self[T].
And then there are lots of functions that are ~impossible to fully correctly type: map has complicated logic for determining the return type, pluck and unzip are pretty impossible (although you can sort out a couple cases with overloads), for mean it might be possible to catch type errors if the object in the bag can't be summed with float, and probably lots more.

The factory functions for creating Bag objects will hit the problem of python/mypy#6073 due to functions not being subscriptable, and require somewhat nasty workarounds as done in trio: https://github.com/python-trio/trio/blob/master/trio/_channel.py#L90 plus https://github.com/python-trio/trio/blob/master/trio/_util.py#L274

I think this probably isn't too much of a problem though


Failures on long runs.

This is now very bad issue. Sorry folks. After switching from BUFG mode BUFR/BUFIO mode we did see it working well, but just in case we let the loop test to run. Well first time it did run about a week until it failed. We are now running it all the time, every day we check if it has failed. And we are seeing failures almost every day. So there is a likelihood that the OpenHBMC fails within 24 hours of continuous testing.
This is bad, it actually means that OpenHBMC can not be used in real products. As once a day failure can not be tolerated. If it works, it should work and not fail every other day.
We are sure that our target hardware is near ideal for HyperRAM testing - all hyperbus signals are LESS than 4 mm long! This is amazing layout the HyperRAM sits below FPGA and the wires are really all in the range of 2..4mm! It cant be better than this. So it is for sure not signal integrity issue.
Argh! I recall that we have reports from another HyperRAM IP vendor that some HyperRAM devices itself have failures, like real memory content losses. This would mean that the error is really in the HyperRAM device itself. But how to verify it? Right now we just run the memory tests in forever loop, the error is only telling us the memory width at the failing test, we don't see what data was read or written. And its really not helpful for debugging.
As of now we do not also know if the problem is still related to the Xilinx FIFO and be essentially the same failure as with BUFG versions just happening not fast.
We would be really happy to assist in the debugging of this problem, let us know if we can try out something to rule out some possible causes. I myself have little ideas what we could try.

change serdes clock from 300 to 301 MHz?
change IO slew rate to fast?

One interesting option to test the IP would be using CRUVI loopback adapter, but for this testing we would need a HyperRAM emulation model, I am guessing the model offered by Cypress would not work well in FPGA :(
Anyway we are happy to assist with this issue. We really would like to see that HyperRAM would work, and well more than 24 hours!
UPDATE: there is a 3 year old forum entry about errors that happen every 10..20 hours, with different hardware and different IP Core.
https://forum.trenz-electronic.de/index.php/topic,1320.0.html
So there are chances that there is something bad with the HyperRAM chip itself. ?
UPDATE2: different HyperRAM chip, different hardware and different IP Core, and also data corruption:
https://community.infineon.com/t5/Hyper-RAM/HyperRAM-Memory-Corruption/td-p/281115
It would be really nice to see WHAT type of errors happen here at our testing...

Hello!

This is bad, it actually means that OpenHBMC can not be used in real products. As once a day failure can not be tolerated. If it works, it should work and not fail every other day.

Absolutely agree and I want to make every effort to get OpenHBMC as stable, as possible.

We are sure that our target hardware is near ideal for HyperRAM testing - all hyperbus signals are LESS than 4 mm long! This is amazing layout the HyperRAM sits below FPGA and the wires are really all in the range of 2..4mm! It cant be better than this. So it is for sure not signal integrity issue.

OpenHBMC's DRU (data recovery unit) module can handle quite large uncertainty (up to 1/6 of hyperbus clock period) between data bits (among each other) and RWDS. It was proved in a DRU testbench. Also PCB delay is about 165ps/inch, so 2..4mm makes extremely small arrival time difference.

As of now we do not also know if the problem is still related to the Xilinx FIFO and be essentially the same failure as with BUFG versions just happening not fast.

Why do you think there is something wrong with Xilinx FIFO? I don't think so. There is probably some issue in my design.

Anyway we are happy to assist with this issue. We really would like to see that HyperRAM would work, and well more than 24 hours!

Me too! Let's try to fix it together.

So there are chances that there is something bad with the HyperRAM chip itself. ?

No, I can't and don't want to believe that.

It would be really nice to see WHAT type of errors happen here at our testing...

Right! Let me modify memory test software to print additional information about detected error, i.e. address, expected data and actual data. This is quite easy to do. I will commit sources today.

I myself have little ideas what we could try.

change serdes clock from 300 to 301 MHz?
change IO slew rate to fast?

I strongly DO NOT recommend to change ration of serdes and hyperbus clock. DRU was designed to work with 1:3 clock ratio, other values are not supported by design.

Yes, you are free to change slew rate to meet signal integrity of your PCB design. Also FPGAs and RAM's IO strength can be modified.

Started long run memtest

Spartan-7 + IS66WVH8M8ALL-166B1LI
HyperBUS/SERDES clocks: 104/312 MHz
BUFG mode

I am pretty sure this is BUFG error, change to BUFR! We can not run log time testing with bufg at all, as it fails FAST!

we can not rule out that the HyperRAM chip itself is bad, we have a report from industrial customer who says that at OLDER versions of HyperRAM die (a and b?) have problem with data corruption! This is confirmed, those guys really had an issue with commercial IP core and custom hardware.

UPDATE: I double checked the HyperRAM on our board, it is die REV D, the latest and recommended one. So we can assume we have no problems with the HyperRAM device itself?


Responsive HTML5 Output SCORM File with Quiz

Thank you so much for your response.
I have downloaded the latest project and check SCORM file which having quiz is not opening on SCORM player. Quiz slide showing blank. I am attaching the SCORM file having quiz and publish the output in HTML5 format for responsive project, but the file is not loading on player, But I am viewing the file from physical location ,quiz is showing.
Please help me out from this issue.

Fluid_Box_Demo_Responsive.zip
QuizProjectV1.zip
I have tried to upload non responsive project also, in which output format as SWF flash file format. It is working on SCORM player. But Adobe flash is going out from coming December. So we can use only HTML5 output format. Please resolve this issue.
Thank you so much for your valuable support.

I tried running both of the courses you attached.
Both of them are returning a fatal javascript error in a javascript file
(CPXHRLoader.js) that is part of Captivate.
(You can see the error console if you turn on your browser's debugging
tools).
If you google search on the error you get a lot of responses. The most
basic solution that I saw was "Upgrade to the latest version of Captivate".
In any case, it is a known error in Captivate, and so it doesn't seem to be
anything I can fix on the LMS side.
The error is:

Cannot read property 'charAt' of undefined
VM829:1264 Uncaught TypeError: Cannot read property 'charAt' of undefined
at cp.QuizState.readNumber (eval at e (CPXHRLoader.js:formatted:163),
:1264:423)
at cp.QuizState.readString (eval at e (CPXHRLoader.js:formatted:163),
:1266:34)
at cp.PlaybackController.restoreObjectsFromState (eval at e
(CPXHRLoader.js:formatted:163), :1345:20)
at Function.cp.resumeValuesFromDataChunk (eval at e
(CPXHRLoader.js:formatted:163), :1259:460)
at cp.PlaybackController.RestoreQuizState (eval at e
(CPXHRLoader.js:formatted:163), :1386:35)
at Function.cp.NewQuizLibraryInit (eval at e
(CPXHRLoader.js:formatted:163), :982:3)
at Function.cp.CPPreInit (eval at e (CPXHRLoader.js:formatted:163),
:990:246)
at Function.cp.ContinueCPInit (eval at e
(CPXHRLoader.js:formatted:163), :1935:33)
at LoadContent (eval at e (CPXHRLoader.js:formatted:163),
:1258:439)
at InitializeExecuted (eval at e (CPXHRLoader.js:formatted:163),
:1879:36)

I searched on "Cannot read property 'charAT' of undefined CPXHRLoader.js"
On Sat, Aug 8, 2020 at 2:27 PM Kopal-LMS<EMAIL_ADDRESS>wrote:

Thank you so much for your response.
I have downloaded the latest project and check SCORM file which having
quiz is not opening on SCORM player. Quiz slide showing blank. I am
attaching the SCORM file having quiz and publish the output in HTML5 format
for responsive project, but the file is not loading on player, But I am
viewing the file from physical location ,quiz is showing.
Please help me out from this issue.
[image: image]
https://user-images.githubusercontent.com/66865599/89717281-7b1bcb00-d9d2-11ea-86aa-b789820656fb.png
Fluid_Box_Demo_Responsive.zip
https://github.com/dhodges47/SCORM-LearningManagementSystem/files/5046032/Fluid_Box_Demo_Responsive.zip
QuizProjectV1.zip
https://github.com/dhodges47/SCORM-LearningManagementSystem/files/5046034/QuizProjectV1.zip
I have tried to upload non responsive project also, in which output format
as SWF flash file format. It is working on SCORM player. But Adobe flash is
going out from coming December. So we can use only HTML5 output format.
Please resolve this issue.
Thank you so much for your valuable support.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dhodges47/SCORM-LearningManagementSystem/issues/16,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB5OFVMAISSSS5GXL6MJCLR7WKIZANCNFSM4PYY3CEQ
.

Thank you so much for response on my query.
Yes I am getting the same error and report to Captivate team also  and I have received the below response from Captivate team.
Captivate team suggests, there is a problem in SCORM player.

I have also checked both project working on SCORM cloud.
Please help me out from this issue.

I'm out of ideas at this point.
I can't find anything wrong with the LMS.
If I think of something I'll let you know.
On Tue, Aug 11, 2020 at 3:11 PM Kopal-LMS<EMAIL_ADDRESS>wrote:

Thank you so much for response on my query.
Yes I am getting the same error and report to Captivate team also and I
have received the below response from Captivate team.
Captivate team suggests, there is a problem in SCORM player.
[image: image]
https://user-images.githubusercontent.com/66865599/89938061-96d1dc00-dc33-11ea-92fb-309d6605aa41.png
I have also checked both project working on SCORM cloud.
Please help me out from this issue.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dhodges47/SCORM-LearningManagementSystem/issues/16#issuecomment-672205350,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB5OFVUX7HPJ7YCVLZI6ZTSAGJWHANCNFSM4PYY3CEQ
.

Hi Dhodges,
Hope you are doing good.
Did you find any solution on my above query. Because I am totally stuck on responsive SCORM file which having quiz.
Because Captivate team not giving any solution on this because the above SCORM output file working on SCORM Cloud.
Kindly help me out from this LMS issue.
Thanks you so much for your valuable support.
Looking forward to your reply.
Regards,
Kopal Rajpoot

I replied to you a few days ago.
I have not found any solution. I can't see any errors coming from the LMS.
The only error I see is coming from Articulate.
I'm sorry but I don't have any ideas on what could be the problem.
On Thu, Aug 20, 2020 at 2:46 AM Kopal-LMS<EMAIL_ADDRESS>wrote:

Hi Dhodges,
Hope you are doing good.
Did you find any solution on my above query. Because I am totally stuck on
responsive SCORM file which having quiz.
Because Captivate team not giving any solution on this because the above
SCORM output file working on SCORM Cloud.
Kindly help me out from this LMS issue.
Thanks you so much for your valuable support.
Looking forward to your reply.
Regards,
Kopal Rajpoot
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dhodges47/SCORM-LearningManagementSystem/issues/16#issuecomment-677343211,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB5OFSUVIDZ6FBA22JGYXLSBTBFXANCNFSM4PYY3CEQ
.

Hi Dhodges,
Hope you are doing good. I have explored about the responsive project using Captivate, it is working on SCORM cloud. I have debug the source code and getting attached error while using it. Please find the error log with the same.
default-20200818-000.log
default-20200820-000.log
default-20200821-000.log
Attaching the responsive SCORM also.
Fluid_Box_Demo_Responsive_test_with_quiz.zip
Quiz Project Ambika.zip
Kindly upload in your LMS and debug, you will get the same issue.
Looking forward to your reply.
Regards,
Kopal Rajpoot

From your logs I see a lot of database problems. There may be an issue with your database set up. One problem I can fix on my side is from a residual field called "msrepl_tran_version" that is not used in this iteration. I have attached a script to allow nulls in that field, which will resolve some of the errors you are getting.
SCORMFix.sql.zip

I think I fixed the problem with the Articulate Storyline quizes. Please update your solution with the latest commit, which is labelled "Bug fix for Articulate Storyline quizes". The fix is in the file "SCORMRuntimeHelper.cs". After you rebuild your solution, try the quiz again.

Hi Dhodges,
Thank you so much for the great solution. Its really work. Now quiz is working.
One more query I have, I am not getting any interaction data in below highlighted tables.

Could you please suggests.
Thanks & Regards,
Kopal Rajpoot

Please obtain the latest commits - "Fix to cmi_objectives" and "Fix bug in cmi_interactions".  Then try again.

Hi, Really appreciated your reply, its solve many problems. Thank you so much.
Now I am getting another issue and points analyzed on latest updated source code-
1.“[cmi_interactions]” table   data in Column “n_id”  is  not  saving properly . I think  n_id column refers the Question Name ,   but it always updated the question text only for first row  when user  select correct response .
While  student _response  save properly  in different rows (create new row corresponding to each  user response).

Data is not  saving in following table
•	SELECT * FROM [OpenSourceSCORMLMS].[dbo].[cmi_comment_from_learner]
•	SELECT * FROM [dbo].[cmi_student_preferences]
•	SELECT * FROM [dbo].[cmi_comment_from_lms]--blank
•	SELECT * FROM cmi_comment_from_learner--blank

Thank you so much for your support.
Regards,
Kopal

Please get the latest commit. It should solve the problem with cmi_interactions.
For cmi_comment_from_learner, cmi_comment_from_lms, cmi_comment_from_lms: Do you have a scorm module that sets these values? I don't have any way to test these, since none of the modules I have are calling the api for these values.

Hi Dhodges,
Thank you so much for a valuable support. You have solved many issues where I was totally stucked.
Really appreciated for resolving the query. As per my last comment, I  dont have such SCORM file which sets the values in these tables cmi_comment_from_learner, cmi_comment_from_lms, cmi_comment_from_lm
Regards,
Kopal

Thanks Kopal. Because of your persistence I was able to identify and fix several bugs that had not been reported before, so I appreciate your help.
If I get a chance, I'll see if I can set up a test for the comment tables.
I am going to close this issue. If you come across anything else, please open a new issue.
Thanks again,
David


proxy env var now called PROXY_PRESENT

Summary
rename env var PROXY as PROXY_PRESENT
Description for the changelog
@taoyiyang1 reported that we have problems with certain proxy configurations. An environment variable PROXY is used to correct this, and this variable is now named PROXY_PRESENT to fit better with the other env var naming. Also documents the default value
Other info
Hope this is OK with you @taoyiyang1 ?

Thanks @lreading , I had pushed the updated test file to the wrong branch, all working now


load a hex file with a nice name, get Untitled

Create a blocks program and change its name to "fun with with buttons" in the UI.
click save button
reset
load the file "microbit-fun-with-buttons.hex"
Notice that the name is "Untitled".

Specify browser/OS.

Save doesn't save the project before downloading.
Fixed in ec25698c6425274f1ec5cae83e0198b8a3af05f9


Output get confused when flow server is starting

Hey,
Thanks for making this! I noticed that if I was getting the flow starting messages it seems to confuse neomake. I updated the script to be
          \ 'args': ['-c', g:flow_path.' --json 2>/dev/null| flow-vim-quickfix'],

which seems to fix it.

Ah, yeah, I was actually used to this behavior from the original vim-flow project... will add this later to the README after I tried this ... thanks :-)

Tried it and it looks better now! Updated the README ... thanks again :+1:


segmentation fault when making reference index

Hi Ivan,
Whenever I run graphmap (v.022), I get a segmentation fault. My computer is running Ubuntu 15.10 with 30GB of RAM and the parsimonious memory mode.
[Index 14:21:45] Running in parsimonious mode. Only one index will be used.
[Index 14:21:45] Generating index.
Segmentation fault (core dumped)
Do you know what might be causing this error?
Thanks!
Jacob

Hi Jacob,
Could you please share the exact command line arguments and the reference
itself (best: a Url) with us?
Andreas
On 16 Jan 2016 06:38, "Jacob Pfeil"<EMAIL_ADDRESS>wrote:

Hi Ivan,
Whenever I run graphmap (v.022), I get a segmentation fault. My computer
is running Ubuntu 15.10 with 30GB of RAM and the parsimonious memory mode.
[Index 14:21:45] Running in parsimonious mode. Only one index will be used.
[Index 14:21:45] Generating index.
Segmentation fault (core dumped)
Do you know what might be causing this error?
Thanks!
Jacob
—
Reply to this email directly or view it on GitHub
https://github.com/isovic/graphmap/issues/17.

Hello Andreas,
Here is the command:
./bin/Linux-x64/graphmap -I -P -r ../data/ref/hg38.fa
And here is the link to the reference:
ftp://hgdownload.cse.ucsc.edu/goldenPath/hg38/bigZips/hg38.fa.gz
Thank you!
Jacob

On Jan 15 2016, at 5:24 pm, Andreas Wilm<EMAIL_ADDRESS>wrote:
Hi Jacob,

Could you please share the exact command line arguments and the reference
itself (best: a Url) with us?
Andreas
On 16 Jan 2016 06:38, "Jacob Pfeil"<EMAIL_ADDRESS>wrote:
> Hi Ivan,
>
> Whenever I run graphmap (v.022), I get a segmentation fault. My computer
> is running Ubuntu 15.10 with 30GB of RAM and the parsimonious memory
mode.
>
> [Index 14:21:45] Running in parsimonious mode. Only one index will be
used.
> [Index 14:21:45] Generating index.
> Segmentation fault (core dumped)
>
> Do you know what might be causing this error?
>
> Thanks!
>
> Jacob
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/isovic/graphmap/issues/17>.
>

—
Reply to this email directly or [view it on GitHub](https://github.com/isovic/
graphmap/issues/17#issuecomment-172142849).![](https://github.com/notification
s/beacon/AL4vVGHvdAlnadv7xPH3V8WzyCgDy7Pyks5paZNKgaJpZM4HGGDz.gif)

Hi Jacob!
How much memory do you have installed on the machine?
The index will require about 20x the genome size.
Thanks,
Ivan.

Hi Ivan,
I was not aware that the index needs 20x the genome in memory. Is this also
true for the parsimonious mode? I currently have 30GB of memory and a 3GB
reference, so that is the problem.
Thanks,
Jacob

On Jan 16 2016, at 10:21 am, Ivan Sovic<EMAIL_ADDRESS>wrote:

Hi Jacob!

How much memory do you have installed on the machine?
The index will require about 20x the genome size.

Thanks,
Ivan.

—
Reply to this email directly or [view it on GitHub](https://github.com/isovic/
graphmap/issues/17#issuecomment-172242067).![](https://github.com/notification
s/beacon/AL4vVHUu0PLHA1OzBhfNt75Da2UUcNsjks5paoGegaJpZM4HGGDz.gif)

Unfortunately yes - think of it like this: 64-bit integers are used for coding the positions on the reference which includes both fwd and rev sequences (so 2*8 bytes) + the original fwd sequence + the rev sequence = 18x original sequence length. On top of that goes the perfect hash table which consumes far less though on large references, because of small seed sizes.
So, in total it's about 18x per index (20x was an approximation). In parsimonious mode only one index is used, whereas in the default mode two are used at the same time, making memory consumption even larger (~36x the genome size).
We are currently developing other indexing methods which should reduce this number significantly, but it will take some time before it gets released.
Ivan


serve_forever hang on while close_all

serve_forever hang on while close_all

on linux hangs on,
on windows works well

You should provde some mor einfo. How do you stop the server? Via a signal? I suggest you put some print statements in close_all method in order to figure out where it hangs.

I use threading ftpserver class to initialize.
And I start the server in a thread, and then stop via close_all() in another thread.
The close_all can be executed fully.
But the code below server_forever never be executed.
This issue is under Linux, but windows won't hang on.
I guess the epoll has some issue?

Code version is newest, Dec 1

Please paste the code you're using.
Il 11/dic/2015 00:28, "PonyPC"<EMAIL_ADDRESS>ha scritto:

Code version is newest, Dec 1
—
Reply to this email directly or view it on GitHub
https://github.com/giampaolo/pyftpdlib/issues/367#issuecomment-163785407
.

class FtpServer(threading.Thread,metaclass=Singleton):
authorizer = None
server = None
handler = None
def init(self):
self.closed = False
self.running = True
self.authorizer = DummyHashAuthorizer()
dtp_handler = ThrottledDTPHandler
dtp_handler.timeout = 60
dtp_handler.read_limit = 0
dtp_handler.write_limit = 100 * 1024
dtp_handler.auto_sized_buffers = False
self.handler = MyFTPHandler
self.handler.timeout = 60
self.handler.use_gmt_times = False
self.handler.use_sendfile = True
self.handler.authorizer = self.authorizer
self.handler.dtp_handler = dtp_handler
self.handler.abstracted_fs = MyFilesystem
self.handler.passive_ports = range(30000, 31000)
threading.Thread.init(self)
self.setDaemon(True)
def run(self,*args,**kwargs):
while not self.closed:
try:
if self.running:
self.server = ThreadedFTPServer(('<IP_ADDRESS>', 21), self.handler)
self.server.max_cons = 100
self.server.max_cons_per_ip = 3
self.server.serve_forever()
print('stopped')####NEVER EXECUTE UNDER LINUX####
time.sleep(0.5)
except Exception as error:
logger.error(traceback.format_exc())
def stop(self):
self.running = False
self.server.close_all()
def restart(self):
self.running = True
def close(self):
self.closed = True
self.stop()
on main thread:
ftp = FtpServer()
ftp.stop()

I try a temporary solution for this issue:
in ioloop.py class Epoll()
delete this line self._poller.close() in close() procedure.

I came upon the same problem and discovered an error in the docstring of serve_forever():
def serve_forever(self, timeout=None, blocking=True, handle_exit=True):
        """Start serving.

         - (float) timeout: the timeout passed to the underlying IO
           loop expressed in seconds (default 1.0).

1.0 being the default is not quite true because it's set to "None" right here and never changed later.
So when I call serve_forever() without a timeout, calling close_all() wait's indefinitely. When I specify a timeout serve_forever(timeout=10) calling close_all() returns like expected and the server is killed after some seconds.

@muffl0n but how do you deal with forever running except manual shutdown?

@muffl0n Oh.... I catch you. I'll try your solution later....Thanks

@muffl0n after several hours test, it is seem that adding timeout parameter is working very well.
thanks.

I'm always happy to help! :)
I'm still curious what is right: the code or the documentation. @gimpaolo, maybe you could help us here?

Sorry for joining late guys.
So I understand the problem is that you want to shutdown the server via code, you do so by calling close_all but close_all hangs unless you specify a timeout != None (note: docstring stating the default is 1 is clearly wrong).
Also I understand you use a threaded server, and I suppose this does not happen if you use the plan "async" server.
Is this correct?

I don't know whether it does happen by using plain async server. Instead, I use a threaded server.
I mean the serve_forever hangs unless specify a timeout, but close_all can be executed fully.
Firstly, I thought the timeout is the server running time. Obviously I'm wrong. The timeout is killing server timeout.

The timeout parameter is passed directly to the underlying IO loop (which uses select() / epoll() / whatever syscall) and it causes it to wait / hang until something "happens" to the connected file descriptors (read or write events).
As such, I suppose, close_all() closes all fds, but the underlying select / epoll syscall will keep hanging.
Perhaps it makes sense to have timeout default to 1 sec or something for the ThreadedFTPServer class.
Question: are you on Windows?

Centos.
I also use same method to stop and restart http server in python. It is seemed that the http library works well and can stop immediately, although I don't know the reason. ☺ be surely, set timeout to a specific value is useful.

I'm using Gentoo.
+1 for setting the default value to 1 or some other value other than None. Having no timeout at all feels pretty weird.

Done.

Awesome! Thank you! :)

good job

;)

a week later, the problem comes:
After issuing close_all, the threaded ftp server print
[W 2016-10-23 04:00:05] thread <Thread(('xxx', 62906), started daemon<PHONE_NUMBER>16000)> didn't terminate; ignoring it
And hang on again.

[W 2016-10-23 04:00:05] thread ≤Thread(('xxx', 62906), started daemon<PHONE_NUMBER>16000)> didn't terminate; ignoring it

So that means one of the threads kept hanging (e.g. got stuck in a time.sleep() call or something) and as such also the main thread couldn't exit.
I would argue that in this case pyftpdlib is doing the right thing.
As for what concerns your code, you can use os._exit(0) after close_all.
 Not very nice but when dealing with threads that's often used a common solution.

Emm... I don't want to kill the process cause there are other threads to server continuously.
Http server library never hang on after close all at my environment. Is the thread model difference between http and pyftpdlib cause the problem? Or the timeout of sub thread is invalid?

You should identify what thread is hanging and where exactly.
I suppose you're using ThreadedFTPServer instead of async FTPServer because you have blocking operations. Is that blocking operation an HTTP request you do by using httplib? In that case what I would recommend is set a timeout for httplib (say 5 secs), then ThreadedFTPServer.join_timeout = 8 or something.
That way the http request will eventually timeout and raise an exception in the running threaded, but it will "unblock" the thread so that it can be terminated.

After few days finding, I guess there is some issue in ioloop.
I use ThreadedFTPServer for serve FTP. I do nothing blocking in original ftp functions.
httplib is used in my project but it is isolated with ftplib. httplib never hang on its sub thread.
I compared 2500 log files in 8 servers in one year, and found a regular condition.
Thread may not be terminated after user operating this command:
Login -> CWD -> ... -> CWD -> not disconnect after timeout
Login -> CWD -> STOR -> not disconnect after timeout
Login -> CWD -> DELE -> not disconnect after timeout
It is not easy to recurrent this situation. maybe it relies on unstable internet connection.
Then if you run close_all(), you will get an error 'thread didn't terminate; ignoring it'.

I believe this is being cause by issue #48 within the threading module.


Add upgrade support for Multicluster Engine, for development (workaround)

Description of the change:

Include a script that has interactive and automatic run modes
Include a --watch to see the change occur
Update the readme. State this is for development!

Signed-off-by: Joshua Packer<EMAIL_ADDRESS>Motivation for the change:
I didn't want to start fresh every time I make a commit.

/cc @JakobGray
I suspect there is a field on the mce resource I can watch to make sure it gets full configured with the new images?

Watch the mce operator
Watch the mce resource to see that it completes??

This can be part two.

/lgtm

@gurnben @Kyl-Bempah @tpouyer looking for an approve


Is it possible to use mhsendmail to send to a gmail server?

Hello,
This is an excellent tool, thank you. I understand that it was meant for testing but is it possible to use this so that my php webform sends mails directly to my gmail account via mhsendmail?
I saw the setting directly below in the project page and was wondering if it can accept the settings that gmail would need (shown bottom) and if so, can you give any pointers on the syntax?
./mhsendmail --smtp-addr="localhost:1026"<EMAIL_ADDRESS>...
gmail:
tls on
tls_certcheck off
auth on
host smtp.gmail.com
port 587
user<EMAIL_ADDRESS>from<EMAIL_ADDRESS>password yourgmailPassw0rd
Thanks

Sorry, I actually read the documentation and the above is not longer and issue... :-)
Thanks for creating this...


LESS formatting - variable interpolation

Description
LESS files are not aligned properly if variable interpolation is used.
Input
.class-@{name} {
    display: block;
}

Expected Output
The beautified code should look the same as the input.
Actual Output
.class-@ {
    name
}

{
    display: block;
}

Environment
Windows 10
Related
Previously reported in: https://github.com/brackets-beautify/brackets-beautify/issues/222

This does not repro in 1.6.4. @retan can you check this again?

@retan - I cannot repro this anymore.  Please comment and I will reopen if needed.

Sorry for a late reply.
Now it's seams to work as expected (I'm using it through Adobe Brackets extension, so I'm not sure about js-beautify version).

No problem, thanks!


New Rule: Stripe invoice abuse

/update-test-rules

do we care to look for the existence of a phone number or is that in all of these? if the latter, are there 2 numbers or just the 1?

Originally I had the phone number check in there, but OCR was not seeing it at all, and it's not in strings either :/


Update README.md

update readme add how to install , i add plugin to Alcatraz

Thanks! Looks good.
I've added myself as a watcher to alcatraz/alcatraz-packages#520 and can merge this in once it's actually in Alcatraz so the README makes sense.

@dbgrandi  alcatraz accept the pull request please accept

Awesome! Thanks @dimohamdy


BUG: Fix Period and PeriodIndex support of combined offsets aliases

[x] closes #13730
[x] tests added / passed
[x] passes git diff upstream/master | flake8 --diff
[x] whatsnew entry

Essentially, makes sure any freq string passed to a Period or PeriodIndex goes through _Period._maybe_convert_freq(), which calls to_offset(), which is where the logic for combining aliases is.
All the examples in #13730 result in the correct output, and all existing tests pass. I have not written any new ones yet — I first wanted to get the opinion of the maintainers.
This PR builds on #13868 (without it, some existing tests fail).

this seems to be on top of #13868 was that intentional? (usually we try to do independent PR's). if they are not its ok, but then should just put in one / indicate at the top that is the case.
@sinhrks

can you run an asv on this, make sure looks good for perf.

also would need to add tests from the issue. I dont' think we are actually testing this (and that's why it doesn't fail).

I ran the asv benchmarks. Here are the results:
    before     after       ratio
  [59f25575] [01c55da7]
+    3.73ms    16.80ms      4.51  reindex.frame_drop_dup_na_inplace.time_frame_drop_dup_na_inplace
+  898.72ms      3.12s      3.47  plotting.plot_andrews_curves.time_plot_andrews_curves
-   45.25ms    13.87ms      0.31  period.create_period_index_from_date_range.time_period_index

I have no experience whatsoever with these, so I don't know how to interpret the results above, but I find it perplexing that the two tests that run slower have absolutely nothing to do with time series...

Thanks for the test. The slowness looks to be caused by regex search. Maybe we should use dict lookup like get_offset does (related to #4205, merging to_offset and get_offset to make fast/simple impl).

https://github.com/pydata/pandas/blob/master/pandas/tseries/frequencies.py#L508

@sinhrks I'm confused. The asv test that has to do with periods is faster with this PR...

Ah sorry, I looked 1st 2 (of course unrelated to the period). Pls ignore.

Current coverage is 85.22% (diff: 100%)

Merging #13874 into master will decrease coverage by 0.01%

@@             master     #13874   diff @@
==========================================
  Files           140        140
  Lines         50451      50405    -46
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits          43003      42958    -45
+ Misses         7448       7447     -1
  Partials          0          0

Powered by Codecov. Last update b2a1445...01c55da

pls rebase. Need tests to close #13730 ; and add to whatsnew.

Rebased. I have also added tests for Period, Period.asfreq, PeriodIndex, PeriodIndex.asfreq, PeriodIndex.to_datetime and period_range with combined aliases.
There is one standing issue: the function get_standard_freq() does not accepted combined aliases as of now. Apart from tests, it is only called from three places in the whole codebase:

Lines period.py#L609 and period.pyx#L742 both read
freqstr = frequencies.get_standard_freq(other)

They can — and should — be changed to
freqstr = other.rule_code

period.py#L488-L489
freq = Period._maybe_convert_freq(freq)
freq = frequencies.get_standard_freq(freq)

can — and should — be changed to
freq = Period._maybe_convert_freq(freq).freqstr

I have tried these changes in my local copy of the repo and all tests pass.
In view of this, I propose deprecating get_standard_freq() (as is already done with get_offset_name()) — or removing it completely if possible.

I implemented the deprecation proposal in the last commit.

Is "combined offset aliases" okay?

@agraboso Thx, it sounds ok.

@jreback How does it look?

@jreback Done.

you are going to have a flake8 minor issue (long lines). otherwise lgtm.
can you run an asv vs master (just -b period) is fine, and show the results.

you are going to have a flake8 minor issue (long lines). otherwise lgtm.

flake8 does not complain. Which are the long lines you refer to?

can you run an asv vs master (just -b period) is fine, and show the results.

    before     after       ratio
  [768bf495] [81262a57]
-   54.20ms    14.96ms      0.28  period.create_period_index_from_date_range.time_period_index

are u just showing the single change in asv?
there are more period benchmarks

Here is the whole output:
· Creating environments
· Discovering benchmarks
·· Uninstalling from conda-py2.7-Cython-matplotlib-numexpr-numpy-openpyxl-pytables-scipy-sqlalchemy-xlrd-xlsxwriter-xlwt.
·· Installing into conda-py2.7-Cython-matplotlib-numexpr-numpy-openpyxl-pytables-scipy-sqlalchemy-xlrd-xlsxwriter-xlwt..
· Running 24 total benchmarks (2 commits * 1 environments * 12 benchmarks)
[  0.00%] · For pandas commit hash 81262a57:
[  0.00%] ·· Building for conda-py2.7-Cython-matplotlib-numexpr-numpy-openpyxl-pytables-scipy-sqlalchemy-xlrd-xlsxwriter-xlwt....................................
[  0.00%] ·· Benchmarking conda-py2.7-Cython-matplotlib-numexpr-numpy-openpyxl-pytables-scipy-sqlalchemy-xlrd-xlsxwriter-xlwt
[  4.17%] ··· Running gil.nogil_datetime_fields.time_datetime_to_period                                                                                18.29ms
[  8.33%] ··· Running gil.nogil_datetime_fields.time_period_to_datetime                                                                                55.16ms
[ 12.50%] ··· Running period.create_period_index_from_date_range.time_period_index                                                                                14.96ms
[ 16.67%] ··· Running period.period_algorithm.time_period_index_drop_duplicates                                                                                30.93μs
[ 20.83%] ··· Running period.period_algorithm.time_period_index_value_counts                                                                                1.12ms
[ 25.00%] ··· Running period.period_algorithm.time_period_series_drop_duplicates                                                                                16.60ms
[ 29.17%] ··· Running period.period_algorithm.time_period_series_value_counts                                                                                6.94ms
[ 33.33%] ··· Running period.period_setitem.time_period_setitem                                                                                failed
[ 37.50%] ··· Running plotting.plot_timeseries_period.time_plot_timeseries_period                                                                                308.25ms
[ 41.67%] ··· Running timeseries.timeseries_iter_periodindex.time_timeseries_iter_periodindex                                                                                6.57s
[ 45.83%] ··· Running timeseries.timeseries_iter_periodindex_preexit.time_timeseries_iter_periodindex_preexit                                                              55.71ms
[ 50.00%] ··· Running timeseries.timeseries_period_downsample_mean.time_timeseries_period_downsample_mean                                                                  26.72ms
[ 50.00%] · For pandas commit hash 768bf495:
[ 50.00%] ·· Building for conda-py2.7-Cython-matplotlib-numexpr-numpy-openpyxl-pytables-scipy-sqlalchemy-xlrd-xlsxwriter-xlwt....
[ 50.00%] ·· Benchmarking conda-py2.7-Cython-matplotlib-numexpr-numpy-openpyxl-pytables-scipy-sqlalchemy-xlrd-xlsxwriter-xlwt
[ 54.17%] ··· Running gil.nogil_datetime_fields.time_datetime_to_period                                                                                19.20ms
[ 58.33%] ··· Running gil.nogil_datetime_fields.time_period_to_datetime                                                                                51.12ms
[ 62.50%] ··· Running period.create_period_index_from_date_range.time_period_index                                                                                54.20ms
[ 66.67%] ··· Running period.period_algorithm.time_period_index_drop_duplicates                                                                                33.60μs
[ 70.83%] ··· Running period.period_algorithm.time_period_index_value_counts                                                                                1.07ms
[ 75.00%] ··· Running period.period_algorithm.time_period_series_drop_duplicates                                                                                16.89ms
[ 79.17%] ··· Running period.period_algorithm.time_period_series_value_counts                                                                                6.42ms
[ 83.33%] ··· Running period.period_setitem.time_period_setitem                                                                                failed
[ 87.50%] ··· Running plotting.plot_timeseries_period.time_plot_timeseries_period                                                                                294.17ms
[ 91.67%] ··· Running timeseries.timeseries_iter_periodindex.time_timeseries_iter_periodindex                                                                                4.01s
[ 95.83%] ··· Running timeseries.timeseries_iter_periodindex_preexit.time_timeseries_iter_periodindex_preexit                                                              40.75ms
[100.00%] ··· Running timeseries.timeseries_period_downsample_mean.time_timeseries_period_downsample_mean                                                                  24.24ms
    before     after       ratio
  [768bf495] [81262a57]
-   54.20ms    14.96ms      0.28  period.create_period_index_from_date_range.time_period_index

SOME BENCHMARKS HAVE CHANGED SIGNIFICANTLY.

By the way, I just realized that period.period_setitem.time_period_setitem fails even on master.

@jreback

thanks!


[IOTDB-2424] Fix windows cpp compile doc and hide CLI history file

Coverage increased (+0.02%) to 67.811% when pulling 2f910b99daf418dbe1966f498e9a0bfadb772271 on jLine_v2 into 2659d1c1cae29247460ccb28e631d7ec12959cc9 on master.


installfish-app-upload 1.0.0

New Pull Request Checklist
Please mark the points which you did / accept.

[x] I will not move an already shared step version's tag to another commit
[x] I read the Step Development Guideline
[x] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[x] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[x] I read and accept the Abandoned Step policy

Hi @andrewmarmion
Thank you for your PR! Unfortunately I will have to refuse it, because InstallFish is in end of life stage. Please rewrite the titles and descriptions to mirror the rebranding to YourAppBox.
Also, you do not have to escape the underscore in the description strings when referring to the environment variables, as in $BITRISE\_APK\_PATH.
You will have to re-share and open a new PR for the fix.


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter renacidc for common errors and appropiate suggestions to keep this adapter up to date.
This check is based the current head revisions (master / main  branch) of the adapter repository
Please see the result of the check below.
ioBroker.renacidc
   -
ERRORS:

[ ] :heavy_exclamation_mark: [E254] Version "0.1.3" listed at common.news at io-package.json does not exist at NPM. Please remove from news section.

WARNINGS:

[ ] :eyes: [W252] Version of package.json (0.1.3) doesn't match latest version on NPM (0.1.2)

:thumbsup: No suggestions found
Please review issues reported and consider fixing them as soon as appropiate.
Errors reported by repository checker should be fixed as soon as possible. Some of them require a new release to be considered as fixed. Please note that errors reported by checker might be considered as blocking point for future updates at stable repository.
Warnings reported by repository checker should be reviewed. While some warnings can be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
You may start a new check or force the creation of a new issue at any time by adding the following comment to this issue:
@iobroker-bot recheck
or
@iobroker-bot recreate
Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
@mcm1957 for evidence
Last update at Fri, 08 Nov 2024 10:44:46 GMT based on commit a08e0b1dc3331720fc3edb97db5a7589a66d54dc
ioBroker.repochecker 3.2.2

[W252] Version of  (0.1.3) doesn't match latest version (0.1.2) : Created 0.1.4, Please check


Update ElasticSearch Templates to reflect deprecation of "string"

As of ElasticSearch 5.0, elastic has expressed the intention to deprecate the string type in favor of two new data types text and keyword:  https://www.elastic.co/blog/strings-are-dead-long-live-strings
This PR reflects that change by replacing not_analyzed strings as keyword type, and analyzed strings with the text type.  It also simplifies the templates a little bit by disabling _all at the parent type level since all of the fields had the include_in_all parameter set to false.

Codecov Report

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

@@             Coverage Diff              @@
##             master     #366      +/-   ##
============================================
- Coverage     46.83%   46.81%   -0.02%
+ Complexity     2641     2640       -1
============================================
  Files           630      630
  Lines         16816    16816
  Branches       1239     1239
============================================
- Hits           7875     7873       -2
- Misses         8481     8482       +1
- Partials        460      461       +1

Impacted Files
Coverage Δ
Complexity Δ

...m/spotify/heroic/aggregation/simple/MinBucket.java
42.85% <0%> (-9.53%)
4% <0%> (-1%)

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 151c471...15c632a. Read the comment docs.

@jcabmora This looks great!
I am a little bit concerned this will prevent folks from configuring pre keyword and text clusters.
Could we either detect which schema a cluster supports and adapt accordingly, or make it a non-default configuration option for now until we officially bump which ES versions we are compatible with?
Thanks!

@udoprog The problem that I see is that I think the current code base won't work with an ES 2.X cluster. If that is true, then I believe there is no point to try to maintain backwards compatibility for ES templates.
I am not particularly knowledgeable in ElasticSearch, so I could be wrong.  However, I did run a test using an ES 2.2.1 ES cluster and a heroic jar built with the current HEAD, and when I ran the configure HeroicShell task there were multiple exceptions reported and the templates were not created.  Here's an extract of the logs:
21:27:11.654 [elasticsearch[_client_][transport_client_boss][T#1]] WARN  org.elasticsearch.transport.netty4.Netty4Transport - exception caught on transport layer [[id: 0x2fade185, L:/<IP_ADDRESS>:44622 - R:elasticsearchclient/<IP_ADDRESS>:9300]], closing connection
java.io.EOFException: tried to read: 36 bytes but only 3 remaining

The ES docs are vague about this (https://www.elastic.co/guide/en/elasticsearch/client/java-api/5.3/client.html), but it seems that the client must match the version of the server.

@jcabmora good point! we should update the documentation that we no longer support ES 2. Thanks!

Is this PR going to be merged? We want to update our internal version, and it will be nice to have this so our code is easier to rebase.

@jcabmora sorry for the delay - if you rebase against master I'll merge this right away.
I'll look over #371 next. It will also need a rebase once this is merged. Thanks!

@dmichel1 Thanks a lot ! I have rebased my PR branch so it is at the same HEAD as master. Thanks!!


fixed typo in usage.md

Missing double quote around script tags.

Nice catch @CommandLineDesign!  Thanks.


Is there a way to change sample rate for PWM code? Also code restart?

Hello @jmseitz ,
I'm using the LidarLite v3HP with an Arduino Uno. The device is wired using the PWM wiring diagram in the manual. I have installed the PWM 'get distance' code on the Arduino, and it is working great. However, I only need a 10Hz sample rate, not 400Hz for my application.
Is there a way to reduce the sample rate in the code without re-wiring the system?
I also noticed the system crashing when gently moved or bumped. My application requires the system be mounted to moving parts, is there a way to restart the Lidar periodically in the code?
Thank you!

If you are using the PWM functionality, I think you can most easily reduce the rep rate by adding a software timer to the sample code.  That would reduce the rep rate without requiring any changes to the wiring.
It's unclear to me why bumping the system would cause a crash.  Are you sure your connections are not intermittent in a way that would interrupt them when the system is bumped?  There is no soft reset available on the LIDAR-Lite v3HP, but you could cycle the power enable pin if you need to externally apply a signal to reset the device.


"Advanced Node and Express - Set up a Template Engine" passes as correct despite wrong solution

Describe your problem and how to reproduce it:
When setting up Pug for the first time, even though I didn't choose to render the index.pug page, but sent it (using app.sendFile), the FCC checker passed my solution as correct.
Code that wrongly passes tests:
app.route('/')
  .get((req, res) => {
    res.sendFile(process.cwd() + '/views/pug/index.pug');
  });

Actual solution:
app.route('/')
  .get((req, res) => {
    res.render(process.cwd() + '/views/pug/index.pug');
  });

This is especially confusing for a beginner as the boilerplate comes with res.sendFile as a default, simply changing the path results in the behavior shown in the screenshots below.
Add a Link to the page with the problem:
https://www.freecodecamp.org/learn/information-security-and-quality-assurance/advanced-node-and-express/set-up-a-template-engine
Tell us about your browser and operating system:

Browser Name: Firefox
Browser Version: 74
Operating System: Linux Mint 19

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

I've submitted PR's for {freeCodeCamp/boilerplate-advancednode/pull/9} & {freeCodeCamp/freeCodeCamp/pull/38572} as part of this fix.

@mehanix, thank you for reporting this. We should try to test for cases where campers don't update the res.sendFile(...) line to res.render(...) as mentioned in the instructions.
Also thank you @jeremyfiel for submitting PRs for this challenge. I'll leave some comments over in each of them.

fixed in the latest merge.  Ok to close.

There is still an issue of the test not passing with correct solutions:
app.route('/').get((req, res) => {
    res.render(process.cwd() + '/views/pug/index');
  });

OR
app.route('/').get((req, res) => {
    res.render('./pug/index');
  });

There is still an issue of the test not passing with correct solutions:
app.route('/').get((req, res) => {
    res.render(process.cwd() + '/views/pug/index');
  });

OR
app.route('/').get((req, res) => {
    res.render('./pug/index');
  });

hi @Sky020  we've submitted another pr to relax the regex a bit more. it's pending review.
I believe the single period (relative path) the way you've written it doesn't work in even the glitch app. It's pointing at the wrong path.  should be .. or just /pug


Release Training Operator Image for v1.8.0-rc.1

I updated image tag for v1.8.0-rc.1 release.
/assign @kubeflow/wg-training-leads

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 35.414%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
4379

Relevant Lines:
12365

💛  - Coveralls

/hold cancel


OAuth Authentication failing.

After a successful login, [[YMLoginClient sharedInstance] handleLoginRedirectFromUrl:url sourceApplication:sourceApplication] is being called.
and it tries to get OAuth token (https://www.yammer.com/oauth2/access_token.json) Which is failing right now.
response: Request failed: forbidden (403)

Thanks for the report we'll look into this

I'm not having this problem, so it's probably network specific, can you confirm that you have third party apps enabled on your primary network ? You might need an admin to check this for you.

You are right, I'm having this issue because my app isnt global and im trying to post on some other network.
To make it  "Global" seems like i need to contact "help desk". I've googled for "help desk" but cant find anything. after that I called Microsoft helpline the support officer asked me "what is Yammer?" :-/
I'll appreciate if you could guide me to right direction.

Hey Saqib, shoot an email over to<EMAIL_ADDRESS>with your app ID and we'll help you out here. To get your app ID, head to https://www.yammer.com/client_applications/, click on your app and send us the five digit number at the end of the URL.


poll error on c9800platformAccessory.js:92:108

tested on a 17.7.1 i got the following error.
Error getting WLAN state TypeError: Cannot read properties of undefined (reading 'wlan-status')
    at C9800PlatformAccessory.poll (/homebridge/node_modules/homebridge-c9800/c9800platformAccessory.js:92:108)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

changing the poll URL to append the path for wlan-status fixes it after also changing the powerstate const.

Thanks for the fix for 17.7.1.  It worked fine on my 9800CL running 17.3.4c using both versions, so I'm happy to update with your code!


[SPARK-13444] [MLlib] QuantileDiscretizer chooses bad splits on large DataFrames

What changes were proposed in this pull request?
Fixes a bug in QuantileDiscretizer that results in the wrong number of bins for datasets larger than 10K rows and adds regression test.  This PR corrects an issue with PR #11319.
How was this patch tested?
Manual tests and test-only QuantileDiscretizerSuite

Can one of the admins verify this patch?

@oliverpierson When you create the PR, please choose branch-1.6 instead of master and put `[Branch-1.6] in the PR title. I think you need to close this one and create a new PR.

Thanks @mengxr for helping fix this while I was away. Yes it looks good excepting that this needs to be vs branch-1.6 so we can test it with Jenkins explicitly. Thanks @oliverpierson

No worries.  I'll close and open a new one today.  Would it make things easier if I rebase off of branch-1.6 before creating the new PR?


Checks if the page can be resolved

This allows us to change the scope of the page when a promise is resolved.
// ...
$scope.DoCtrlPagingAct = function (page){
    return $http.get({page:page}).then(function(records){
        $scope.records = records;
    }, function(){
        // something
    });
}

<paging
  ...
  paging-action="DoCtrlPagingAct(page)">
</paging>

Hey Anteriovieira,
Sorry for the delay! and thank you for the PR
I will take some time and review the changes hopefully this or next week
Just out of curiosity - would it be possible to include a plunker of the issue this PR resolves?
I think I understand what you are fixing, I'm just surprised I've not run a similar case!
This would also be helpful in creating unit tests around the fix
Thanks again!

I will keep close to solving tests


Update neogov-tutorial.md

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

We'd like to get this documentation updated, please let me know what else is needed on our end to complete this. Kasey Pearce<EMAIL_ADDRESS>

@jeevansd @v-hagamp

#sign-off

@v-hagamp: I'm sorry - only the author of this article, @jeevansd, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@v-hagamp: I'm sorry - only the author of this article, @jeevansd, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

Thanks @v-hagamp


Layer separation in a few components

Testing first proposal for Flavors. Separation of presentational layer into new files under each flavor.

Closing as outdated.


[staging-next] ocaml fixes due to findlib changes

https://github.com/NixOS/nixpkgs/pull/133008/commits/a3ab43d3a45338f845a0f8a12ed34e84c8b79dae moved a required setup step to a preConfigure hook, which wasn't getting triggered by some packages.
Motivation for this change
Fixes e.g. jackline and virt-top on staging-next https://github.com/NixOS/nixpkgs/pull/135477
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Change looks good, I couldn't find the relevant hydra failures though. Perhaps you tested on local ? (it looks as if there are just a bunch of pending jobs right now)
@symphorien might be interested in this.

Yeah there's a lot still queued up but many failures are already visible: https://hydra.nixos.org/eval/1702118?full=1&compare=1700369#tabs-now-fail
E.g. https://hydra.nixos.org/build/151271341 (jackline), https://hydra.nixos.org/build/150650387 (virt-top)

Result of nixpkgs-review pr 136821 run on x86_64-linux 1

1 package marked as broken and skipped:

ocamlPackages.ocaml_extlib-1-7-7

5 packages failed to build:

google-drive-ocamlfuse
libbap
ocamlPackages.bap
python38Packages.bap
python39Packages.bap

22 packages built:

beluga
haxe
haxePackages.hxcpp
haxePackages.hxcs
haxePackages.hxjava
haxePackages.hxnodejs_4
haxe_4_0
haxe_4_1
jackline
ocamlPackages.caqti-driver-mariadb
ocamlPackages.cudf
ocamlPackages.dose3
ocamlPackages.javalib
ocamlPackages.mariadb
ocamlPackages.mccs
ocamlPackages.ocaml-sat-solvers
ocamlPackages.ocaml_extlib
ocamlPackages.ocaml_oasis
ocamlPackages.pgsolver
ocamlPackages.sawja
ocamlPackages.tcslib
virt-top

Perhaps a more correct solution is to have findlib add createOcamlDestDir to postHooks, instead of preConfigureHooks? Causes many more rebuilds though, so not sure if it's appropriate for a staging-next fix.

Result of nixpkgs-review pr 136821 run on x86_64-linux 1

1 package marked as broken and skipped:

ocamlPackages.ocaml_extlib-1-7-7

56 packages built:

beluga
google-drive-ocamlfuse
haxe
haxePackages.hxcpp
haxePackages.hxcs
haxePackages.hxjava
haxePackages.hxnodejs_4
haxe_4_0
haxe_4_1
jackline
libbap
liquidsoap
ocamlPackages.async_smtp
ocamlPackages.bap
ocamlPackages.camomile
ocamlPackages.caqti-driver-mariadb
ocamlPackages.charInfo_width
ocamlPackages.cryptokit
ocamlPackages.cudf
ocamlPackages.dose3
ocamlPackages.eliom
ocamlPackages.email_message
ocamlPackages.gapi_ocaml
ocamlPackages.gettext-camomile
ocamlPackages.graphql_ppx
ocamlPackages.javalib
ocamlPackages.jwto
ocamlPackages.lambdaTerm
ocamlPackages.mariadb
ocamlPackages.mccs
ocamlPackages.ocaml-sat-solvers
ocamlPackages.ocaml_extlib
ocamlPackages.ocaml_oasis
ocamlPackages.ocsigen-start
ocamlPackages.ocsigen-toolkit
ocamlPackages.ocsigen_server
ocamlPackages.pgsolver
ocamlPackages.reason
ocamlPackages.reason-native.console
ocamlPackages.reason-native.dir
ocamlPackages.reason-native.file-context-printer
ocamlPackages.reason-native.fp
ocamlPackages.reason-native.pastel
ocamlPackages.reason-native.pastel-console
ocamlPackages.reason-native.qcheck-rely
ocamlPackages.reason-native.refmterr
ocamlPackages.reason-native.rely
ocamlPackages.reason-native.rely-junit-reporter
ocamlPackages.sawja
ocamlPackages.tcslib
ocamlPackages.utop
ocamlPackages.zed
opa
python38Packages.bap
python39Packages.bap
virt-top

Thank you for this!


doc: process for adding a maintainer if the candidate does not have a role

There is no clear way for a candidate to assume the role of Maintainer if the candidate is neither a Contributor nor a Collaborator. To avoid disputes and confusion, candidates who are neither Contributors nor Collaborators must be approved by the TSC before they can assume the role of Maintainer.

I added DNM label until the Process WG can discuss.  Since this proposes adding another responsibility to the TSC, we probably need to announce this to the TSC as well.

The way I understand it, the same is stated at the top of this page for candidates that are already collaborators:

Collaborator: A highly involved Contributor in one or more areas.
May become a Maintainer with approval of existing TSC voting members.
https://github.com/zephyrproject-rtos/zephyr/pull/77933/files#diff-c557288edbce823dbb1409ebe4e78596230e4f869a9f61374bbf2bcf87c49bd8L18

Am I missing something? Could this be clarified?

This basically says that the collaborator->maintainer path only requires one or more +1 from TSC members on the PR introducing the change.
The proposition in this PR is about requiring a formal vote from the TSC at large

This basically says that the collaborator->maintainer path only requires one or more +1 from TSC members on the PR introducing the change.

Ah okay. I didn't quite understand this when reading the sentence I quoted.

This was discussed at the TSC and no objections raised.


[PhpAmqpLibPublisher][Fix 2.0] Misbehaving header processing with PhpAmqpLib\AMQPArray

This PR is the continuation of #174.
As suggested by @stof, instead of correcting the publisher, we can fix the provider so that we always have array in the Swarrot message, not sometimes an array and sometimes an AMQPArray.
I removed here the previous fix (and corresponding test) and then I fixed the provider and added a test case.

Thanks! :+1:


Add indentation configuration

Ideally, marksman should follow the indentation config of the editor (or currently open file or .editorconfig), however, when that does not work, users should be able to configure indentation via, e.g., indent property which could be either tab string or a number (how many spaces should be used for indentation).
Now, I have configured indentation to three spaces in Markdown files, however, the generated TOC looks like this: Actually, this was my mistake, as I actually used heading levels 1, 3, 4, and 3 instead of 1, 2, 3, 2).

example of my mistake
<!--toc:start-->
- [Heading 1]()
      - [Heading 2]()  <!-- 6 spaces -->
        - [Heading 3]()  <!-- 8 spaces -->
      - [Heading 2]()  <!-- 6 spaces -->
<!--toc:end-->

I would expect it to be as follows:
<!--toc:start-->
- [Heading 1]()
   - [Heading 2]()  <!-- 3 spaces -->
      - [Heading 3]()  <!-- 6 spaces -->
   - [Heading 2]()  <!-- 3 spaces -->
<!--toc:end-->

marksman always uses two spaces for indentation. The relevant line from Marksman/Toc.fs:
https://github.com/artempyanykh/marksman/blob/a511779d2d2ad8c3fdf060d62d4dfb723ed23e2b/Marksman/Toc.fs#L31

Thanks for raising this @tukusejssirs! Adding support for both would be helpful!

indentation config in .marksman.toml and
.editorconfig


Search only defined barcodes to improve performance

We highly recommend to merge this to massively improve performance and not look for barcodes if u just want to take images or something. In our case we are just searching for qr codes, which is done way faster now.
This is just done in iOS and should also be implemented for Android.
Someone may also want to reflect this in ReadMe.
barCodeTypes={[Camera.constants.BarCodeType.qr]}
Keep up the great work!:)

+1

yes! this looks great and is much needed.
We need to consider how to support "all" barcodes. Perhaps barCodeTypes defaults to all if onBarCodeRead is defined.

@rt2zz we could define a key "all" thats adds available types. but not as default this could lead to the same misuse that we had before. for example when someone is using this only to capture an image the device would use the default and check for all barcodes.

@philipheinser my suggestion is that barCodeTypes defaults to null (or []) unless onBardcodeRead is defined. In other words, no barcode scanning should happen unless it is being listened for.
I am wary of overloading the propType with both strings and arrays for barCodeTypes

It's kinda late in Germany. You are absolutely correct. We read over onBardcodeRead.  We will overwork our pull request asap.

Thanks! no worries about the defaults, I took this into v1 branch and implemented the defaulting behavior. 👍


Quests plugin breaks after player attempts to start a quest

What is your Quests version / build number (do not say "latest")?
Ver 3.9.3 / Minecraft 1.13.2
Describe the problem and how to replicate it. Any console errors to post using Pastebin.com?
After a player attempts to start a quest. The whole plugin breaks
If applicable, can you provide an example quest from quests.yml using Pastebin.com?
The error spamming console after a player attempts a quest: https://pastebin.com/wXNqN68A
The error when I attempt a quests reload:  https://pastebin.com/4p8Fe19t
The quests file: https://pastebin.com/t7F2UU1L

Thanks for the report. Is there more to the error? Looks as though it may have been cut off half-way.

https://pastebin.com/YbAgrmZx I think this is the whole error, then it starts looping.

@SoraShiunin Thanks for that information. It's hard to tell whether this is actually the fault of Quests, or some other plugin that's modified the PlayerMoveEvent. In any case, I've pushed a potential fix which you can try here: https://ci.codemc.io/job/PikaMug/job/Quests/115/
If you still get that error, try running your server with only Quests installed temporarily. If the issue goes away, then add plugins back until you find the conflict.

The fix din't work however I found the issue. It was permissions. I recently changed from PEX to LuckPerms and I just had to give quests.quests and etc permissions to the default group. With Pex it worked without giving the permissions so it never crossed my mind.


Select doesn't update when enter 2nd time in the screen with self-reference

      WorkOrderStatus: {
        title: 'Status da Ordem de Serviço',
        fields: {
          name: Fields.text("Name"),
          previousStatus: Fields.fixedReference('Previous Status', 'Status')
        },
        referenceName: "name"
      }

To reproduce, use the entity above, go to list page, click in "+", create the "Status 1", confirm to save it (you'll be redirected to the list page again), click in the "+", you won't see the "Status 1" into the select.
If you go to another page or refresh the browser, the item will be there.

@anderick pretty interesting case. Thanks for filing it!
Yep there's reference value cache in allcountjs-angular-base. Need to invalidate it on every Create, Update, Delete operation.


IntelliSense confuses this in constructor functions when assigning values (JS)

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.70.1
Commit: 6d9b74a70ca9c7733b29f0456fd8195364076dda
Date: 2022-08-10T06:08:33.642Z
Electron: 18.3.5
Chromium: 100.0.4896.160
Node.js: 16.13.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.18363

Steps to Reproduce:

Have a JS file with a constructor function with some properties and a function.

Read-access a member of this. IntelliSense/code completion recognizes this as (Anonymous function) - correct.

Assign a value to a member of this. IntelliSense/code completion recognizes this as the function you are currently in - false.

Code sample:
let foo = new function ()
{
	this.a = "Hello World";
	this.b = 21;

	this.doSomething = function (val)
	{
		// this.a = val;
		console.log(this.a);
	};
};

Looks related to (or possibly the same issue) #50240

Oh yes, you are right.


use after free bug in src/terminal.c

in line 4587 "p1 = line1;"
in line 4631 "vim_free(line1);"
in line 4635 and following lines, p1 is used.

in line 4587 "p1 = line1;"
in line 4631 "vim_free(line1);"
in line 4635 and following lines, p1 is used.

Thanks for reporting.  Strangely I can't make valgrind report this
use-after-free, even though I can see it when debugging.
--
ARTHUR: I've said I'm sorry about the old woman, but from the behind you
looked ...
DENNIS: What I object to is that you automatically treat me like an inferior...
ARTHUR: Well ... I AM king.
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Publishing to AWS IoT Core update topic not working

Hi everyone,
I've been using the PubSubClient successfully for several projects so far and I thought AWS wouldn't be a big deal either.  I got a TLS connection established with PubSub and thought all was on the up and up, however AWS has topics which you must publish to in order to update the thing shadow that begin with a $ such as $aws/things/thingName/shadow/update. I am having an issue publishing to that topic which causes the client to disconnect.  A call to publish returns true but the MQTT connection drops.  Also appears the publish never makes it to AWS.  Any insight on this?  I am able to publish to AWS using MQTT.fx but the pubsub client disconnects every time I try to publish to the topic.

PubSub client state prior to publish to $aws/things/thingName/shadow/update is 0.  State of PubSub client right after publish is 0 as well.  However a check to PubSub client.connected() returns false and checking the state after that results in a -3.  This happens each time an attempt is made to publish to the topic.

How large is the payload you are trying to publish? If the length of payload and topic is >128 bytes then have you also changed the value of MQTT_MAX_PACKET_SIZE to accommodate that?

Hi @knolleary
First off thank you for this fantastic library.  I've used it a lot and it's awesome.
Yes, I have modified MQTT_MAX_PACKET_SIZE to 512.  The packet I'm sending is no where near that though, it's around 197.
I should also mention this is definitely not a connection problem or a TLS issue with connecting to AWS as I am able to publish to topic  things/thingName/shadow/update. The only time I seem to run into this problem is when a $ precedes the topic.

If there is any information I can provide so you can replicate the issue please let me know.  I am running on an ESP32.

It certainly sounds like AWS is kicking you off when you try publishing to that topic. You say you can publish using another client - is that to exactly the same topic as you are trying here?

Yes, I am able to publish to the same topic using the same credentials from MQTT.fx no problem and the shadow document updates on AWS so the publish is actually going through.  It seems specific to the PubSubClient for one reason or another.
I will try to put together a straight forward simple test that you could quickly implement and see for yourself.  Do you have an ESP32 handy by chance?  Also would it be simplest to publish to a local Mosquitto broker?  I'm going to try writing to the Mosquitto broker today to ensure this isn's something on the AWS side for some strange reason.

Please disregard this for now.  It is an issue with AWS permissions.  Apparently my policy document for the AWS thing does not grant permissions for updating the thing shadow as I thought it should.  I'll update back here once I figure out the problem just in case anyone in the future runs into this issue.  Closing issue for now since it is not an issue with the PubSubClient.  I apologize for the wasted time @knolleary

Hello guys, i'm trying to connect to AWS wich request:
wifiClient.loadCACert(ca);
wifiClient.loadCertificate(client_cert);
wifiClient.loadPrivateKey(client_key);
Via wifi i can do it with no problems. But the ethernet library present on arduino do not have these 3 classes above.
Someone know how to connect to AWS via ethernet shield using ESP32?

If you figure out a way to do that let me know.  I went down that rabbit hole for a couple weeks and never got anywhere.  Currently to my knowledge there is no simple way to do this on an ESP32.  PubSub allows you to pass a WiFiClientSecure client to it which handles all the TLS encryption.  Unfortunately there is no EthernetClientSecure and extrapolating the required resources from WiFiClientSecure is near impossible since it is deeply rooted in the low level wifi interface drivers.


static_select in input not emmiting action on Mobile

Description
Not sure where should I report this as it happens on Android(on desktop works fine) (google play comment is not a good place I guess)
In this message block, first static_select (input type) is not emitting anything but second one works fine (section type). (I receive action_id for second one)
I send them as direct message to user from my app.
What type of issue is this? (place an x in one of the [ ])

[x] bug
[ ] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] example code related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
package version: slack/bolt 3.12.1
node version: 14.15.0
android_app_version: <IP_ADDRESS>-90011886-10539
OS version(s): Android 13, iPhone ( a friend said it's not working there also, don't know the version)
Steps to reproduce:
1.Send as direct message to user
2.Select any option from the list
3.Need to have a server to check emitted events (which is not working)
Expected result:
Receiving action_id when static_select got selected as a message.
Actual result:
Not receiving any action_id if it's not part of an accessory.
Attachments:
[
  {
    "element": {
      "action_id": "selected_edit_action_notworking",
      ....
      "type": "static_select"
    },
    ...
    "type": "input"
  },
  {
    "text": {
      ...
    },
    "accessory": {
      "action_id": "selected_edit_action_working",
      ...,
      "type": "static_select"
    },
    "type": "section"
  }
]

Hi @maour, thanks for writing in!
I know this could be confusing but when you use "input" type blocks in either a message and on Home tab and you would like to receive block_actions requests for their interactions, your input blocks should have "dispatch_action": true property, which is mentioned in the document here.
I hope this was helpful to you.

Hello @seratch,
Thanks for the comment, yes, I forgot that dispatch since it was working fine on desktop! (only had problem on mobile, strange!)


Update to<EMAIL_ADDRESS>

/kind enhancement
Ref https://groups.google.com/g/golang-announce/c/oecdBNLOml8/m/Qpa5hGjrBwAJ
Release note:

The Golang version is updated to `1.17.9`.

/lgtm
/approve


Request for adding "Hot corners facilities".

Description of the new feature / enhancement
When moving the cursor to the screen four corner and hit then we can create custom shot cut so that we can do work faster like assume I set Right-Bottom corner for switch between open application so it will help to switch between open application what we do using Alt+Tab shortcut. Like this now assume i set Right-Top corner for sleep mode so when I hit that corner then my computer goto the sleep mode.
There also may a feature lke set a specific key with corner so we can use the same corner for different work.
Scenario when this would be used?
Will improve multitasking. It helps to use shortcut in faster way.And also when using mouse and doing a speicific work then without touching keyboard we can do a lot of tasks.
Supporting information
This feature is available in Unix (Mac Book) and Ubuntu.

/dup #1305

It is a good idea, just already captured by some other users. Check out this list of common issues. If you have more ideas, then you can see if anyone else has already suggested it and +👍 their issue.
https://github.com/microsoft/PowerToys/wiki/Common-Triage-Issue


Update environment.yml to use vtr-optimized (vtr compiled with PGO)

@mithro Could you help me figure out why Conda fails? make env works on my workstation.

My theory is that the condarc file in the package causes this, because vtr-gui also fails: https://github.com/HackerFoo/symbiflow-arch-defs/tree/test-ci

I think I know what's happening. Some dependencies are coming from conda-forge, so that needs to be included in environment.yml.

The fact that there are some conda-forge dependencies is a hint that there is something wrong with the vtr-optmized package. Looking at the package description in litex-hub-conda, I see that there is a condarc file which should be deleted from there: https://github.com/litex-hub/litex-conda-eda/blob/master/pnr/vtr-optimized/condarc.
The problem is that conda-forge generates this kinds of issues and leads to instability, and we should not use it at all.

Unfortunately, there is no other good way to get python-constraints. I can't use pip: https://github.com/conda/conda-build/issues/548

I have created a package for python-constraint to avoid depending on conda-forge.

I'm not seeing an improvement in overall runtime.  This is likely because most of the time is spent during the lookahead computation.  @HackerFoo have you explored/tested PGO improvements on the lookahead computation stuff?

I have not. It might be worth trying a separate PGO build optimized for lookup computation, but it's probably not worth the ~10% performance gain, and I suspect that lookup is mostly memory bound, so it will probably have less effect.
I think the most effective way to improve lookup time is by balancing the parallel workload better, and shortening the tail.


Update bower to include the non-minified version

See https://github.com/bower/bower.json-spec#main
(I hope I'm doing this pull request stuff right! And thanks so much for Lazysizes, it's brilliant)

Thank you. Normally it would be much simpler for me, if you would target the master branch, because this is where the development happens.


Fix docker inspect container only reports last assigned information

fixes #17352
Signed-off-by: Lei Jitang<EMAIL_ADDRESS>

After this the NetworkSettings of docker inspect will be like this
"NetworkSettings": {
        "Bridge": "",
        "EndpointID": "8b91be443a92a6c87984391a56e33172dfc6f0781961f4c7da46d8743ba3d1ee",
        "SandboxID": "865d8030ad69ba9a08d550861296852ba1ca57f9170eb1b2f1259ab8a1bdccc8",
        "Gateway": "<IP_ADDRESS>",
        "GlobalIPv6Address": "",
        "GlobalIPv6PrefixLen": 0,
        "HairpinMode": false,
        "IPAddress": "<IP_ADDRESS>",
        "IPPrefixLen": 16,
        "IPv6Gateway": "",
        "LinkLocalIPv6Address": "",
        "LinkLocalIPv6PrefixLen": 0,
        "MacAddress": "02:42:ac:11:00:02",
        "Networks": {
            "bridge": {
                "EndpointID": "8b91be443a92a6c87984391a56e33172dfc6f0781961f4c7da46d8743ba3d1ee",
                "Gateway": "<IP_ADDRESS>",
                "IPAddress": "<IP_ADDRESS>",
                "IPPrefixLen": 16,
                "IPv6Gateway": "",
                "GlobalIPv6Address": "",
                "GlobalIPv6PrefixLen": 0,
                "MacAddress": "02:42:ac:11:00:02"
            },
            "test": {
                "EndpointID": "15c5c1a3b21b91822ceeb4ed3a069b0cc55325cc5e33bff76bdc6dfe95adbe2f",
                "Gateway": "<IP_ADDRESS>",
                "IPAddress": "<IP_ADDRESS>",
                "IPPrefixLen": 16,
                "IPv6Gateway": "",
                "GlobalIPv6Address": "",
                "GlobalIPv6PrefixLen": 0,
                "MacAddress": "02:42:ac:14:00:02"
            }
        },
        "Ports": {},
        "SandboxKey": "/var/run/docker/netns/865d8030ad69",
        "SecondaryIPAddresses": null,
        "SecondaryIPv6Addresses": null
    },

ping @mavenugo
this could also have tests :angel:

@runcom  working on the tests :)

This is not how we keep backwards compatibility in other cases.
NetworkSettings need to be extracted as an api type, one with the new fields and another with the old ones under api/types/versions/v1p21. The inspect request needs to be versioned at the api level too.

The changes LGTM
BTW, @calavera This  backward compatibility fix is about the format and semantics of the response. Even if somebody uses the latest version of the api, they will expect the meaning of the fields to be the same as previous releases. This commit ensures that.
Having said that, I don't disagree with providing versioning support at the api level as well.

Closing in favor of #17393
Thank you for your prompt work @coolljt0725 ! Really appreciated!


Can you cut a new RC?

Can you cut a new RC?

do you have the proper ssh key to publish?

QBit version 0.9.2-RC5 is released


Tool fails to open on Mac

Version of Archi, Operating System
Archi 4.3.3 on macOS version 10.14.13
Expected Behaviour
The application should open when double clicking on Archi.app
Actual Behaviour
Get error: “Archi.app” is damaged and can’t be opened. You should move it to the Trash.
Steps to Reproduce the Behaviour

Download from www.archimatetool.com
Unzip to extract the contents
Double click on Archi.app

On the download page we have a note to explain this:
Note – when launching Archi on OS X for the first time you may see a dialog box that states that “Archi can’t be opened because it is from an unidentified developer.” Control-click or right click the icon of the app. Select “Open” from the top of contextual menu that appears, then click “Open” in the dialog box. Or you may see the message “Archi is damaged and can’t be opened. You should move it to the Trash.”, to solve this see this.
https://www.tech-recipes.com/rx/45404/mac-downloaded-app-is-damaged-and-cant-be-opened-error-solved/

Closing as resolved.


feat(config): add field and value matchers to commit parser (#194)

Description
This PR introduces the capability of grouping commits by attributes of the commit, i.e. author.name or author.email.
Motivation and Context
This changeset should cover the scenario described in #194.
The reporter of that issue wanted to create a new group based gathering the commits for a specific author name:
 { author.name = "Renovate Bot", group = "Dependency Updates"},

How Has This Been Tested?
In order to test it works, I've modified the cliff.toml file in this project to include a new group like this:
commit_parsers = [
  { field = "author.name", pattern="dependabot\\[bot\\]", group = "Dependency Updates"},
  ...

which generates the following new group:
### Dependency Updates

- *(deps)* Bump regex from 1.9.6 to 1.10.0 ([#308](https://github.com/orhun/git-cliff/issues/308)) - ([a6a890a](https://github.com/orhun/git-cliff/commit/a6a890ac4f0051518b4b4c0d65a5ce83ae0de1c8))
- *(deps)* Bump toml from 0.8.1 to 0.8.2 ([#306](https://github.com/orhun/git-cliff/issues/306)) - ([8422096](https://github.com/orhun/git-cliff/commit/8422096549320604643cda9baab4190a5de272fb))
- *(deps)* Bump regex from 1.9.5 to 1.9.6 ([#305](https://github.com/orhun/git-cliff/issues/305)) - ([e263163](https://github.com/orhun/git-cliff/commit/e2631639bf0438fb8f67e3b99c27354fd7409f57))

Screenshots / Logs (if applicable)

Types of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (no code change)
[ ] Refactor (refactoring production code)
[ ] Other

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly.
[x] I have formatted the code with rustfmt.
[x] I checked the lints with clippy.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Codecov Report
Attention: 20 lines in your changes are missing coverage. Please review.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files
Coverage Δ

git-cliff-core/src/changelog.rs
55.67% <ø> (ø)

git-cliff-core/src/config.rs
41.67% <ø> (ø)

git-cliff-core/src/commit.rs
83.34% <4.77%> (-12.22%)
:arrow_down:

:loudspeaker: Thoughts on this report? Let us know!.

Also, can you resolve the conflicts with main?

Suggestions applied (btw, I really like the refactor for the big switch case I had) and changes from main pulled :-)


replace cuid with @paralleldrive/cuid2

description

paralleldrive/cuid added a depreciation notice (31/Dec/22)

npm install -g @digitak/esrun
# npm WARN deprecated<EMAIL_ADDRESS>Cuid and other k-sortable and non-cryptographic ids (Ulid, ObjectId, KSUID, all UUIDs) are all insecure. Use @paralleldrive/cuid2 instead.

# added 20 packages in 857ms

npm info @digitak/esrun | grep -o dependencies -A5
# dependencies
# @digitak/grubber: ^3.1.4
# chokidar: ^3.5.1
# cuid: ^2.1.8
# esbuild: ^0.17.4

replacement

paralleldrive/cuid2

There's actually no need for esrun to use cuid over uuid, so I decided to remove the cuid dependency completly and instead use the built-in crypto.getRandomUUID() function instead.
<EMAIL_ADDRESS>is now cuid-free.
Thanks for the report!


Some expansion possibilities

I would like to ask if someone can give me some advice or if the author can make some small edits to the extension code:

random value of presets prompt (embellish, style, negative)
dynamic prompt. some like "{close-up, dramatic lighting, intense gaze into camera|full-body shot, dynamic pose, slightly tilted angle}"

regarding the dynamic prompt, I would like to implement the following logic (maybe this can be solved with some other extensions):
the main query generates an ollama. then, in each iteration, a line from the dynamic query is added to this query.
i.e. if the main prompt from the ollama is

Exquisite, ethereal beauty radiates from her porcelain skin, heart-shaped face, and captivating gaze, as she walks through an enchanted forest, adorned in a flowing, iridescent gown that shimmers like moonlight, her raven locks cascading down her back like a waterfall, surrounded by mythical creatures and lush flora, under a starry sky. Eerie, unsettling, dark, grotesque, suspenseful

and the dynamic prompt says

{close-up, dramatic lighting, intense gaze into camera|full-body shot, dynamic pose, slightly tilted angle}

then I should get several generations with queries like
queue 1:

close-up, dramatic lighting, intense gaze into camera, Exquisite, ethereal beauty radiates from her porcelain skin, heart-shaped face, and captivating gaze, as she walks through an enchanted forest, adorned in a flowing, iridescent gown that shimmers like moonlight, her raven locks cascading down her back like a waterfall, surrounded by mythical creatures and lush flora, under a starry sky. Eerie, unsettling, dark, grotesque, suspenseful

queue 2:

full-body shot, dynamic pose, slightly tilted angle, Exquisite, ethereal beauty radiates from her porcelain skin, heart-shaped face, and captivating gaze, as she walks through an enchanted forest, adorned in a flowing, iridescent gown that shimmers like moonlight, her raven locks cascading down her back like a waterfall, surrounded by mythical creatures and lush flora, under a starry sky. Eerie, unsettling, dark, grotesque, suspenseful

I work into adding this.

I made a chain of different nodes so that i can use ollam's prompt and the variant combinator (https://github.com/adieyal/comfyui-dynamicprompts | https://github.com/florestefano1975/comfyui-prompt-composer). But! I can't figure out how to skip the repeated negation of ollam's prompt until all variants of the combinator are tried

Yes I have added a new way where you get AI negatives in the coming update and then I will add the functionality you asked. so the negatives won't repeat as much

There is a new wildcard node works similar to A1111 syntax you add your wildcard ymal files to the presetfolder and it will pick the category base on it's name
here is the rules
With WilcardMode enable
1-. Enter the text prompt using the categoy name __Category__ to indicate the field that should be replace by values from the wildcards.yaml or .json file
2. Use __Category/*__ to select rendom value from the item list use __Category/**__ to select the random value from within the subdirectories
3. Enter the dynamic_prompt select the variable name for the category then enter the equal simbol= and the values between curly brackets{} values separated by pipes| Color={red|blue|white|black} the script will create a prompt for each values overriding the values contained on the wildcards yaml or json file for the specific categories variables
4. select the maximun variant prompts you need with max_variants counter connect the text_list to a display text node
5. Execute the node

I also added a way to get Prefix and Suffix  on the embelishment and styler son when you create a prompt you can automatically pass it trough the new wildcrad node to change those areas around the prompt


Fixed Issue 19 and added logo

I fixed the two code bugs identified and esolves #19. I also added a logo to the docs. I still want to add some more explanatory docs.

Need to add version change to version file. Also it would be great to get a version method.

Version updated now. To check the version of any package, use the following function below (part of python standard library since python 3.8):
from importlib.metadata import version
version('bridgescaler')
# '0.8.0'

Still getting the quantile normalization error after updating the package. See below:
from bridgescaler.distributed import DQuantileScaler
dqt =  DQuantileScaler(distribution='normal', channels_last=False)

BBq = dqt.fit(np.expand_dims(xtrain[:,:,:],1))
save_scaler(dqt,'Quantile_Transform_IVT_fit.json')

vrvr = load_scaler('Quantile_Transform_IVT_fit.json')
ist = vrvr.transform(np.expand_dims(xval,1))
ivt = vrvr.inverse_transform(ist[:,:,:,:])

plt.pcolor(xval[1,:,:])
plt.colorbar()
plt.show()
plt.pcolor(ist[1,0,:,:])
plt.colorbar()
plt.show()
plt.pcolor(ivt[1,0,:,:])
plt.colorbar()
plt.show()

from importlib.metadata import version
version('bridgescaler')
>>'0.8.0'


How can I add a Checkbox into each of the node.

I would like to add a Checkbox (maybe from material-ui) to each of the node, and change node backgroundColor to refect checkbox status (checked or not.) How to do it?
in general, seems like I cannot put complex DOM in the node, am I correct?
thank you.

@bfang711  there is a simple way to do this, simply pass in a button into the button props (takes in array) and then depending on the state, you can pass in a custom class like this className={isChecked ? nodeChecked : ''} an d

@bfang711 Also another option (with regards to rendering a checkbox) you can actually specify a function in nodeData and specify the component that is rendered. This is not very well tested so use at your own risk.
for ex:
{
title:  ()=> <input/>
}

Another alternative is to use nodeContentRenderer to custom render your nodes.
@lifejuggler is correct on how you can change background colors. however, you will have to have to use one of the above 2 options to render a checkbox
Hope that helps!


truncating DB-Views

I'm trying to set up the fixtureFactories plugin on an existing project with existing database.
When I first start a unittest I get the error "Base table or view not found: 1146 Table 'database.tablename' doesn't exist"
I found the following to be the cause:
The error occurs when the DB procedure "TruncateDirtyTables" is executed.
The truncated table is not a table but a view.
The view has been entered in the "test_suite_light_dirty_tables" table.

Thank you for the hint @Rakasch.
The point is that the package relies on the listTableSql method provided by CakePHP, which in turns does not distinguish between views and tables.
I suggest you open an issue directly in CakePHP. You may link it then to the present issue.
Does this make sense? Let me know if you need further advice.

In the meanwhile, you can proceed by writing your own home-made table sniffer and set it in your Datasource configuration as described here: https://github.com/vierge-noire/cakephp-test-suite-light#truncating-tables
The Table Sniffer simply needs to extend the MysqlTriggerBasedTableSniffer and overwrite the fetchAllTables() method with something like this.
But I maintain that opening an issue on the Cake repository would be better in the mid term.

I checked CakePhp Issues and this issue already existed.
A change was rejected with the reason:
"Getting views is a 'feature' not a 'bug'. Removing reflection for views would break existing applications."
See: https://github.com/cakephp/cakephp/issues/12482
So a change is very unlikely.
Thanks for the detailed help. Much appreciated.


Write storage agent

The storage agent will have two tasks:

Sync relevant changes to the database, and
Restart schaufel processes when circumstances require (i.e. changes to topology etc)

So I am thinking about this.

I think the agent should start Schaufel and check periodically to see if it is running.
If Schaufel loses either connection it should die and stay dead  This is because we cannot guarantee 100% that the Postgres -> kvstore replication is always working.
The correct way to start schaufel should be to start the agent.  If the systems in the map can write, then e start schaufel.  If not, we don't.  Similarly if the system's status is offline, we can start it by setting it to online.
The agent probably needs to be able to rewrite the Schaufel config file.  I am not sure the best way to do this (will have to defer to Felix on that).


String position(linenumber, sourcefile) only gets updated on new trans-units

Hi Martin,
i just found out, that the position of "older" trans-units only gets updated if there is a new trans-unit created by ng-xi18n first.
Maybe it would be possible to update while iterating over the nodes and comparing ids.

Hi,
this should already be implemented in the newest version (0.6.0, 0.6.1).
Tell me if it is not working as expected with this version.


Failing test for SEG-9

Ticket:
https://webonise.atlassian.net/browse/SEG-9

@CarlosMecha Updated the test cases, changes the integration as per that. Also shared the event mapping sheet with you contains all finding and reference to changes.

@CarlosMecha Resolved PR comments

@CarlosMecha Resolved PR comments, let me know if anything missing.

Closing this PR as it's merged to upstream.


rename Homeworkdocumentation to documentation

might make sense since this is a public repository, and it might confuse people

Folder is now renamed