[Core]: Improvement to CAN message classes

What changed?

Renamed HardwareInterfaceCANFrame to CANMessageFrame for consistensy and editor auto-completions
Removed CANLibManagedMessage in favor of having setters in CANMessage
Removed unused uniqueID in CANMessage and resolved small code smells

Do we want these changes? I feel like the result is pretty clean. Though I mostly used 'replace-all' of vscode for renaming and removing parts, so there might still be some artifacts left somewhere.

Well, I guess the reason the other class exists is because I didn't want any application layer consumer of our messages to be able to modify any values in them, since that message may be processed in other callbacks as well. So one solution to that was having zero setters. We could maybe accomplish that by going around and making sure all references to messages the user might touch in callbacks are const?
Originally I was going to use uniqueID as a way for the network manager to know if something was sent if people didn't use the hardware interface, but never implemented that... and since it's so tightly tied together now I think it makes sense to remove it.
I'm ambivalent about renaming HardwareInterfaceCANFrame I guess. It would be more consistent maybe to just have CANFrame I think though, otherwise we might want to also rename CANMessage to CANPacket (to imply it can be many frames), but then we are changing sooooooo much.... which is not super exciting.

Well, I guess the reason the other class exists is because I didn't want any application layer consumer of our messages to be able to modify any values in them, since that message may be processed in other callbacks as well. So one solution to that was having zero setters. We could maybe accomplish that by going around and making sure all references to messages the user might touch in callbacks are const?

I feel like changing all occurrences to const makes more sense here and simplifies it a bit. With the current approach where we have 'protected' variables, it makes it a bit harder to find where they get changed. As theoretically any derived class could be setting them. I don't know, I'm fine with both approaches, but prefer using const references with setters as looks more maintainable and centralized to me.

I'm ambivalent about renaming HardwareInterfaceCANFrame I guess. It would be more consistent maybe to just have CANFrame I think though, otherwise we might want to also rename CANMessage to CANPacket (to imply it can be many frames), but then we are changing sooooooo much.... which is not super exciting.

Hmm, I see. My initial thought was a CANMessage basically exists of one or more frames, and so a frame is part of a message, hence the name CANMessageFrame. But CANFrame sounds also fine to me.
Regarding renaming CANMessage to CANPacket. In the standard (Part 3) they say:

<IP_ADDRESS> General
Messages greater than 8 byte in length are too large to fit into a single classical CAN data frame.
Therefore, they are broken into several smaller packets, and those packets are transmitted in separate
classical CAN data frames.

<IP_ADDRESS> “Packetization”
A multi-packet message is defined as one whose data does not fit into the data field of a single CAN data
frame (i.e. messages with a data field greater than 8 byte).

I interpret this a 'packet' representing 7 bytes of data when transporting a 'message' larger than 8 bytes. So a 'packet' is basically always a 'frame', but a 'frame' is not necessarily a 'packet.  Hence renaming CANMessage to CANPacket seems not the correct approach. Though I must say that the standard is not really that consistent in their naming.

OK, that all makes sense I think. Still a shame this will be such a disruptive change, but it does make it cleaner and consolidate how messages are used. I suppose it also opens up some ability for us to create another version of send_can_message that allows sending of a CANMessage (though that would need a lot of validation checking before sending).

All occurrences should be changed to use const references. I kept the CANMessageFrame in place, @ad3154 let me know if you want me to change it to CANFrame, now is the time 😄. Otherwise the PR is ready for review


IOS is reloaded when it enters the foreground from the background

Bug description:
I use the ios real machine debugging, when I enter the background and then cut to the foreground, the webview will reload again
<WebView
      key="test"
      ref={pageRef}
      cacheEnabled={true}
      source={{uri: 'http://'}}
  />

Expected behavior:
I don't want to be reloaded, I need to keep it the way it was before
Screenshots/Videos:
https://github.com/react-native-webview/react-native-webview/assets/11611057/9787908d-3609-4b60-97d6-dd6e1632d313
Environment:

OS: IOS
OS version: 16.6
react-native version: 0.72.6
react-native-webview version: 13.6.2

I have found that this is a problem with local debugging and will not be refreshed if you use a network link


0058 - Length of Last Word

Description of the Problem
Given a string s consists of upper/lower-case alphabets and empty space characters ' ', return the length of last word (last word means the last appearing word if we loop from left to right) in the string.
If the last word does not exist, return 0.
Note: A word is defined as a maximal substring consisting of non-space characters only.
Explanation:
Input: "Hello World"
Output: 5

Code
class Solution:
    def lengthOfLastWord(self, s: str) -> int:
        splits = s.split()
        if len(splits):
            return len(splits[-1])
        return 0

Link To The LeetCode Problem
LeetCode

Kindly assign this to me as there have been no issues for this problem.

LeetCode Stats:
Runtime: 20 ms, faster than 98.88% of Python3 online submissions for Length of Last Word.

More optimized solution compared to the previously given solution at
#289

PR does not follow Contribution Guidelines - closed
Please make sure there is not already an Issue Open/Problem Solved (Code in Folder) with the ProblemNr you want to solve!


DataRescue

We've discussed in various places doing a DataRescue event of some kind. The Portland Study Group just did one which seemed successful (https://github.com/daniellecrobinson/Data-Rescue-PDX). Maybe @daniellecrobinson can chime in with more.
I'd like to know how long one of these things takes. The PDX one seems to have been four-ish hours. The upcoming MIT one is an entire Saturday. Can we do a mini-DataRescue on one of our usual hour-and-a-half sessions on Wednesdays or do we really need a long time?

Hi @wkearn - We used the Mozilla Office in downtown PDX and the event ran 5-9. We had people stay the entire time and I am planning to do a longer event on Open Data Day 3/4/17

A lot of the work is detective work, figuring out which datasets need metadata / which already have metadata that is not linked or incorrectly linked / etc.
During our event, it took about an hour for people to get the workflow. It's hard to figure out the level of granularity for some datasets - much of the first hour was people asking us "what should I do here" and the organizers saying "hmmmm, not sure, see if you can find a standard, or make a judgement call". So, I'd say that if you have 1.5 hours, it's definitely enough time to get started but I'd recommend a follow up event to allow people to really get into it.

The next time I do this (at Open Data Day) I'll do a workshop on Friday 3/3/17 to go through the workflow, and then lead a team all day Saturday 3/4/17 (people coming and going as they wish).


cross link lint.deno.land

https://docs.deno.com/runtime/manual/getting_started/configuration_file/#lint should link to https://lint.deno.land/ where people can find rules

This page now links to lint.deno.land and there is a separate issue to bring lint.deno.land into the docs site.


Will the ETH-DAI MARKETS param represent SAI post upgrade or DAI?

If SAI can the image please be updated to use the value SAI?

I had a look at the code and it appears to query ddex to translate the MARKET param.
So I'm guessing post upgrade ETH-SAI can be used for SAI and ETH-DAI for DAI?

Hi! We updated the token symbol from DAI to SAI. And will add Multi Collateral DAI (the new DAI) to market later.
Use SAI if you want to bid auction on old DAI market.


Fixing broken link for Mongo Guidelines, needs to point to master for merge.

@rhomes-wcd Good eye.


All Travis builds are failing

All recent Travis builds have been failing with the following:
Running "intern:functional" (intern) task
Listening on <IP_ADDRESS>:9000
Error: Tunnel failed to start: Exit code: 1
  at handleChildExit  <node_modules/digdug/Tunnel.js:336:16>
  at Socket.<anonymous>  <node_modules/digdug/Tunnel.js:379:6>
  at emitOne  <events.js:82:20>
  at Socket.emit  <events.js:169:7>
  at Pipe._onclose  <net.js:469:12>
Warning: Test failure; check output above for details. Use --force to continue.
Aborted due to warnings.
The command "grunt travis" exited with 3.

The CI is very fiddly, but I don't think it fails every time.

I could be wrong, but all the pull requests have failed so far :(

That's fine @chrissherlock. Our CI/CD works properly only on PR pushed by us. I don't see how this can prevent PRs to be merged.

Thanks, can you perhaps make this a bit more clear somewhere though?
Sent from my iPhone

On 20 Oct 2017, at 1:18 am, Anton Parkhomenko<EMAIL_ADDRESS>wrote:
That's fine @chrissherlock. Our CI/CD works properly only on PR pushed by us. I don't see how this can prevent PRs to be merged.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

That's a great point, adjusting the ticket's name to reflect that.

Duplicate of #606


Deleting record from records list reports success even when there was a failure

Is there an existing issue for this?

[X] I have searched the existing issues

Version of Corteza
No response
Current Behavior
No response
Expected Behavior
Error messages should be shown
Steps To Reproduce

Create a workflow where the trigger is before delete
Make the workflow raise an exception
Go to a record list and try and delete one or more records
The page refreshes and shows "deleted succesfully" when the deletion actually failed

Environment and versions
No response
Anything else?
Import Jobs _ CRM _ Low Code.webm

@skamensky please provide a version since it's crucial for debugging.
Will raise this issue


Add votes pagination and load user vote in first position

Fixes #1162
Changes proposed in this pull request:

Always load the vote by the connected address and show it in first position
Add pagination with query instead of only loading 100 votes

Here you say "continued". But I don't see those changes here. So there won't be a modal?

Ok... But totally makes sense I think. 👍🏻 Then #1162 should probably be removed from description and stay open when this gets merged. Looking at the code now.

Reopened here https://github.com/snapshot-labs/snapshot/issues/1217


run10x Trouble creating loom files : cellsorted file not found

Hi,
I ran this command

velocyto run -b "/path/barcodes.tsv" -m "/path/hg38_rmsk.gtf" "/path/outs/possorted_genome_bam.bam" "/path/refdata-cellranger-GRCh38-3.0.0/genes/genes.gtf"

and received the following errors:

2019-06-21 14:42:09,834 - INFO - Starting the sorting process of /path/outs/possorted_genome_bam.bam the output will be at: /path/outs/cellsorted_possorted_genome_bam.bam
2019-06-21 14:42:09,834 - INFO - Command being run is: samtools sort -l 7 -m 2000M -t CB -O BAM -@ 16 -o /path/outs/cellsorted_possorted_genome_bam.bam /path/outs/possorted_genome_bam.bam
2019-06-21 14:42:09,834 - INFO - While the bam sorting happens do other things...
2019-06-21 14:42:09,834 - INFO - Load the annotation from /path/refdata-cellranger-GRCh38-3.0.0/genes/genes.gtf
sort: invalid option -- 't'
sort: invalid option -- 'O'
open: No such file or directory
[bam_sort_core] fail to open file CB
2019-06-21 15:59:20,518 - DEBUG - End of file. Reset index: start scanning from initial position.
2019-06-21 15:59:20,519 - DEBUG - 3416460 reads were skipped because no apropiate cell or umi barcode was found
2019-06-21 15:59:20,519 - INFO - Now just waiting that the bam sorting process terminates
2019-06-21 15:59:20,519 - INFO - bam file #0 has been sorted
2019-06-21 15:59:20,519 - DEBUG - Start molecule counting!

Failed to open file /path/outs/cellsorted_possorted_genome_bam.bam
Traceback (most recent call last):
File "/mnt/software/unstowable/miniconda3-4.6.14/envs/mamba/bin/velocyto", line 10, in
sys.exit(cli())
File "/mnt/software/unstowable/miniconda3-4.6.14/envs/mamba/lib/python3.7/site-packages/click/core.py", line 764, in call
return self.main(*args, **kwargs)
File "/mnt/software/unstowable/miniconda3-4.6.14/envs/ma/lib/python3.7/site-packages/click/core.py", line 717, in main
rv = self.invoke(ctx)
File "/mnt/software/unstowable/miniconda3-4.6.14/envs/ma/lib/python3.7/site-packages/click/core.py", line 1137, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/mnt/software/unstowable/miniconda3-4.6.14/envs/ma/lib/python3.7/site-packages/click/core.py", line 956, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/mnt/software/unstowable/miniconda3-4.6.14/envs/ma/lib/python3.7/site-packages/click/core.py", line 555, in invoke
return callback(*args, **kwargs)
File "/mnt/software/unstowable/miniconda3-4.6.14/envs/ma/lib/python3.7/site-packages/velocyto/commands/run.py", line 116, in run
samtools_memory=samtools_memory, dump=dump, loom_numeric_dtype=dtype, verbose=verbose, additional_ca=additional_ca)
File "/mnt/software/unstowable/miniconda3-4.6.14/envs/ma/lib/python3.7/site-packages/velocyto/commands/_run.py", line 229, in _run
results = exincounter.count(bamfile_cellsorted, multimap=multimap)  # NOTE: we would avoid some millions of if statements evaluations if we write two function count and count_with output
File "/mnt/software/unstowable/miniconda3-4.6.14/envs/ma/lib/python3.7/site-packages/velocyto/counter.py", line 756, in count
for r in self.iter_alignments(bamfile, unique=not multimap):
File "/mnt/software/unstowable/miniconda3-4.6.14/envs/ma/lib/python3.7/site-packages/velocyto/counter.py", line 251, in iter_alignments
fin = pysam.AlignmentFile(bamfile)  # type: pysam.AlignmentFile
File "pysam/libcalignmentfile.pyx", line 736, in pysam.libcalignmentfile.AlignmentFile.cinit
File "pysam/libcalignmentfile.pyx", line 935, in pysam.libcalignmentfile.AlignmentFile._open
FileNotFoundError: [Errno 2] could not open alignment file /path/outs/cellsorted_possorted_genome_bam.bam: No such file or directory

Does anyone know the solution to this?
I cannot find the file called cellsorted_postsorted_genome_bam.bam in the output folder it listed.
Thanks!

Hi there. I have the same problem:
Obviously the call to the "samtools sort" fail because trying to use options that are not available
see in previous message:
sort: invalid option -- 't'
sort: invalid option -- 'O'

Therefore the sorted bam file is never created.
cellsorted_postsorted_genome_bam.bam

Usage:   samtools sort [options] <in.bam> <out.prefix>
Options: -n        sort by read name
-f        use <out.prefix> as full file name instead of prefix
-o        final output to stdout
-l INT    compression level, from 0 to 9 [-1]
-@ INT    number of sorting and compression threads [1]
-m INT    max memory per thread; suffix K/M/G recognized [768M]

sametools
Version: 0.1.19-96b5f2294a
(installed and updated from apt on Linux mint; #41~16.04.1-Ubuntu)
(1) It would be great to include a check for the presence of the file in the workflow, and stop it in case of error
(2) Any idea on how to solve that? The command

samtools sort -l 7 -m 1604M -t CB -O BAM -@ 16 -o ..../cellsorted_possorted_genome_bam.bam ..../possorted_genome_bam.bam

could be run before the beginning of the velocyto command with compatible arguments, maybe, without the "-t CB -O BAM"

I have a similar problem, though the only error I'm getting is a FileNotFoundError which keeps throwing however I've checked the path several times and that file should exist.
$ velocyto run -m /home/alex/lab/drugres/10x/hg19_rmsk.gtf /home/alex/lab/drugres/10x/D0D3FullSeq/Day0/outs/possorted_genome_bam.bam /data2/CellRanger/REFERENCE/hg19/genes/genes.gtf

2019-10-10 18:47:55,273 - DEBUG - Reading /home/alex/lab/drugres/10x/D0D3FullSeq/Day0/outs/cellsorted_possorted_genome_bam.bam
[E::hts_open_format] Failed to open file /home/alex/lab/drugres/10x/D0D3FullSeq/Day0/outs/cellsorted_possorted_genome_bam.bam
Traceback (most recent call last):
File "/usr/local/bin/velocyto", line 11, in
load_entry_point('velocyto==0.17.17', 'console_scripts', 'velocyto')()
File "/usr/local/lib64/python3.6/site-packages/click/core.py", line 764, in call
return self.main(*args, **kwargs)
File "/usr/local/lib64/python3.6/site-packages/click/core.py", line 717, in main
rv = self.invoke(ctx)
File "/usr/local/lib64/python3.6/site-packages/click/core.py", line 1137, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/usr/local/lib64/python3.6/site-packages/click/core.py", line 956, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/local/lib64/python3.6/site-packages/click/core.py", line 555, in invoke
return callback(*args, **kwargs)
File "/usr/local/lib64/python3.6/site-packages/velocyto/commands/run.py", line 116, in run
samtools_memory=samtools_memory, dump=dump, loom_numeric_dtype=dtype, verbose=verbose, additional_ca=additional_ca)
File "/usr/local/lib64/python3.6/site-packages/velocyto/commands/_run.py", line 229, in _run
results = exincounter.count(bamfile_cellsorted, multimap=multimap)  # NOTE: we would avoid some millions of if statements evaluations if we write two function count and count_with output
File "/usr/local/lib64/python3.6/site-packages/velocyto/counter.py", line 756, in count
for r in self.iter_alignments(bamfile, unique=not multimap):
File "/usr/local/lib64/python3.6/site-packages/velocyto/counter.py", line 251, in iter_alignments
fin = pysam.AlignmentFile(bamfile)  # type: pysam.AlignmentFile
File "pysam/libcalignmentfile.pyx", line 736, in pysam.libcalignmentfile.AlignmentFile.cinit
File "pysam/libcalignmentfile.pyx", line 935, in pysam.libcalignmentfile.AlignmentFile._open
FileNotFoundError: [Errno 2] could not open alignment file /home/alex/lab/drugres/10x/D0D3FullSeq/Day0/outs/cellsorted_possorted_genome_bam.bam: No such file or directory

Did anyone ever get a response or a solution to this? I'm running into the same problem and I think the issue is that velocyto is using the wrong syntax for samtools (rather an older syntax).
So, your options are: (a) use an older version of samtools, (b) sort the cells outside of velocyto and create a dummy file called 'cellsorted_possorted_genome_bam.bam' or, (c) find an updated version of velocyto that uses the correct syntax. Not sure if (c) is a possibility.

Guys, I think I found the problem for me:
I had install a samtools using apt-get
Try this:

uninstall this version that does not have the options (apt remove samtools)
get the samtools from github: git clone https://github.com/samtools/samtools.git
get the htslib which is a requirement I believe: git clone https://github.com/samtools/htslib
follow the instructions concerning the installation of those two packages, meaning use the --with-htslib=DIR when compiling saltools
once samtools is build, either it is installed (already in $PATH) or the program is in a directory, and you can just add this directory to the $PATH (I added the PATH in the ~/.bashrc file)
open a new terminal, and try to run again your velocyto run

@karakach -> try to use velocyto run10x which is a wrapper. you have to give the path of the director that contain the outs/ directory
@ArcusGears This seem to be another problem. I suspect the file is not where it is expected "could not open alignment file : cellsorted_possorted_genome_bam.bam". You are giving the pyth to possorted_genome_bam.bam, and the **cellsorted_**possorted_genome_bam.bam should be created just there too. If it is not there, then the samtools did not create it properly.

@GildasLepennetier Thanks for your suggestion. That doesn't work either, I have had to work around that issue. I think this would be a very good time to get a response/suggestion from the authors.
After I circumvented that issue I ended up with a different one that goes like this for over 6 hours:
2019-10-13 13:54:56,888 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:56,891 - DEBUG - Counting for batch 36357, containing 100 cells and 115 reads
2019-10-13 13:54:56,900 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:56,902 - DEBUG - Counting for batch 36358, containing 100 cells and 124 reads
2019-10-13 13:54:56,909 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:56,911 - DEBUG - Counting for batch 36359, containing 100 cells and 119 reads
2019-10-13 13:54:56,928 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:56,931 - DEBUG - Counting for batch 36360, containing 100 cells and 108 reads
2019-10-13 13:54:56,939 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:56,940 - DEBUG - Counting for batch 36361, containing 100 cells and 106 reads
2019-10-13 13:54:56,947 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:56,949 - DEBUG - Counting for batch 36362, containing 100 cells and 111 reads
2019-10-13 13:54:56,956 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:56,958 - DEBUG - Counting for batch 36363, containing 100 cells and 114 reads
2019-10-13 13:54:56,965 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:56,969 - DEBUG - Counting for batch 36364, containing 100 cells and 105 reads
2019-10-13 13:54:56,992 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:56,994 - DEBUG - Counting for batch 36365, containing 100 cells and 118 reads
2019-10-13 13:54:57,007 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,010 - DEBUG - Counting for batch 36366, containing 100 cells and 118 reads
2019-10-13 13:54:57,019 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,022 - DEBUG - Counting for batch 36367, containing 100 cells and 116 reads
2019-10-13 13:54:57,029 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,031 - DEBUG - Counting for batch 36368, containing 100 cells and 121 reads
2019-10-13 13:54:57,039 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,041 - DEBUG - Counting for batch 36369, containing 100 cells and 118 reads
2019-10-13 13:54:57,054 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,057 - DEBUG - Counting for batch 36370, containing 100 cells and 128 reads
2019-10-13 13:54:57,070 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,072 - DEBUG - Counting for batch 36371, containing 100 cells and 148 reads
2019-10-13 13:54:57,087 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,089 - DEBUG - Counting for batch 36372, containing 100 cells and 196 reads
2019-10-13 13:54:57,104 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,107 - DEBUG - Counting for batch 36373, containing 100 cells and 181 reads
2019-10-13 13:54:57,122 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,124 - DEBUG - Counting for batch 36374, containing 100 cells and 138 reads
2019-10-13 13:54:57,138 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,140 - DEBUG - Counting for batch 36375, containing 100 cells and 138 reads
2019-10-13 13:54:57,148 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,150 - DEBUG - Counting for batch 36376, containing 100 cells and 141 reads
2019-10-13 13:54:57,158 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,161 - DEBUG - Counting for batch 36377, containing 100 cells and 141 reads
2019-10-13 13:54:57,169 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,171 - DEBUG - Counting for batch 36378, containing 100 cells and 128 reads
2019-10-13 13:54:57,179 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,183 - DEBUG - Counting for batch 36379, containing 100 cells and 113 reads
2019-10-13 13:54:57,201 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,204 - DEBUG - Counting for batch 36380, containing 100 cells and 124 reads
2019-10-13 13:54:57,212 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,214 - DEBUG - Counting for batch 36381, containing 100 cells and 131 reads
2019-10-13 13:54:57,228 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,230 - DEBUG - Counting for batch 36382, containing 100 cells and 131 reads
2019-10-13 13:54:57,237 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,239 - DEBUG - Counting for batch 36383, containing 100 cells and 138 reads
2019-10-13 13:54:57,247 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,249 - DEBUG - Counting for batch 36384, containing 100 cells and 137 reads
2019-10-13 13:54:57,257 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,263 - DEBUG - Counting for batch 36385, containing 100 cells and 134 reads
2019-10-13 13:54:57,279 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,281 - DEBUG - Counting for batch 36386, containing 100 cells and 131 reads
2019-10-13 13:54:57,296 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,299 - DEBUG - Counting for batch 36387, containing 100 cells and 146 reads
2019-10-13 13:54:57,308 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,311 - DEBUG - Counting for batch 36388, containing 100 cells and 136 reads
2019-10-13 13:54:57,318 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,320 - DEBUG - Counting for batch 36389, containing 100 cells and 141 reads
2019-10-13 13:54:57,328 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions
2019-10-13 13:54:57,330 - DEBUG - Counting for batch 36390, containing 100 cells and 149 reads
2019-10-13 13:54:57,344 - DEBUG - 0 reads not considered because fully enclosed in repeat masked regions

Hi,
I realized there was an issue with samtools sort. Thus I ran this command separately first:
samtools sort -t CB -O BAM -o ~/outputfolder/cellsorted_possorted_genome_bam.bam ~path/possorted_genome_bam.bam
This requires samtools to be installed separately. Once this cellsorted file was generated, I placed it/outputted it in the expected folder where cellsorted is supposed to be found. Then, I subsequently ran velocyto run where it was able to skip this step as the file already exists.
Hope this helps!

Hi,
I realized there was an issue with samtools sort. Thus I ran this command separately first:
samtools sort -t CB -O BAM -o ~/outputfolder/cellsorted_possorted_genome_bam.bam ~path/possorted_genome_bam.bam
This requires samtools to be installed separately. Once this cellsorted file was generated, I placed it/outputted it in the expected folder where cellsorted is supposed to be found. Then, I subsequently ran velocyto run where it was able to skip this step as the file already exists.
Hope this helps!

Hello,
I did the same. I firstly executed samtools sort command and then tried velocyto run. However, I am getting a big issue.
I am doing it without UMI-barcode file, and also i am using -U tag.
My command line is :
velocyto run --without-umi -o /5TB-storage/gaurav.ahuja/sidrah/Data/b_mouse_linda/trimmed/bams/cellsorted/loom -m /5TB-storage/gaurav.ahuja/sidrah/Data/filesforvelocyto/mm10_rmsk.gtf /5TB-storage/gaurav.ahuja/sidrah/Data/b_mouse_linda/trimmed/bams/cellsorted/$line1.bam /5TB-storage/gaurav.ahuja/sidrah/Data/filesforvelocyto/Mus_musculus.GRCm38.101.gtf
I got this error.
ile "/home/gaurav.ahuja/anaconda2/envs/velocyto/bin/velocyto", line 8, in
sys.exit(cli())
File "/home/gaurav.ahuja/anaconda2/envs/velocyto/lib/python3.6/site-packages/click/core.py", line 764, in call
return self.main(*args, **kwargs)
File "/home/gaurav.ahuja/anaconda2/envs/velocyto/lib/python3.6/site-packages/click/core.py", line 717, in main
rv = self.invoke(ctx)
File "/home/gaurav.ahuja/anaconda2/envs/velocyto/lib/python3.6/site-packages/click/core.py", line 1137, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/home/gaurav.ahuja/anaconda2/envs/velocyto/lib/python3.6/site-packages/click/core.py", line 956, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/home/gaurav.ahuja/anaconda2/envs/velocyto/lib/python3.6/site-packages/click/core.py", line 555, in invoke
return callback(*args, **kwargs)
File "/home/gaurav.ahuja/anaconda2/envs/velocyto/lib/python3.6/site-packages/velocyto/commands/run.py", line 116, in run
samtools_memory=samtools_memory, dump=dump, loom_numeric_dtype=dtype, verbose=verbose, additional_ca=additional_ca)
File "/home/gaurav.ahuja/anaconda2/envs/velocyto/lib/python3.6/site-packages/velocyto/commands/_run.py", line 159, in _run
exincounter.peek(bamfile[0])
File "/home/gaurav.ahuja/anaconda2/envs/velocyto/lib/python3.6/site-packages/velocyto/counter.py", line 158, in peek
raise IOError("The bam file does not contain cell and umi barcodes appropriatelly formatted. If you are runnin UMI-less data you should use the -U flag.")
OSError: The bam file does not contain cell and umi barcodes appropriatelly formatted. If you are running UMI-less data you should use the -U flag.
^CTraceback (most recent call last):
File "/home/gaurav.ahuja/anaconda2/envs/velocyto/lib/python3.6/site-packages/llvmlite/ir/_utils.py", line 48, in str
return self.__cached_str
AttributeError: 'GEPInstr' object has no attribute '_StrCaching__cached_str'
Please help me in this regard. Thank you.


[ionic v4] ionic serve think the plataform is Android

Ionic Info
Run ionic info from a terminal/cmd prompt and paste the output below.
Ionic:
ionic (Ionic CLI) : 4.0.1 (C:\Users\Computer\AppData\Roaming\npm\node_modules\ionic)
Ionic Framework : @ionic/angular 4.0.0-beta.0
@angular-devkit/core : 0.7.0-rc.3
@angular-devkit/schematics : 0.7.0-rc.3
@angular/cli : 6.0.8
@ionic/ng-toolkit : 1.0.0
@ionic/schematics-angular : 1.0.1
Cordova:
cordova (Cordova CLI) : 8.0.0
Cordova Platforms : none
System:
NodeJS : v8.9.4 (C:\Program Files\nodejs\node.exe)
npm : 6.3.0-next.0
OS : Windows 10
Environment:
ANDROID_HOME : not set
insert the output from ionic info here
ionic serve think the plataform is Android when is with browser on windows computer, without any emulator

There aren't many details here, but I am assuming you're expecting windows mode instead of material design on a windows computer? We have removed the windows mode from v4. There are only two modes currently: ios and md (see the preview on the right): https://beta.ionicframework.com/docs/components#action-sheet
Material design (md) is the default for all browsers on desktop. More info on that in the theming docs: https://beta.ionicframework.com/docs/theming/platform-styles#ionic-modes
We've discussed adding in Windows fluent design but no development has started on it.
I'm going to close this as working as designed. Thanks!

@brandyscarney
import { Platform } from '@ionic/angular';
constructor(
public plt: Platform
) {
console.log(this.plt.is('android'));

}
this give me: this.plt.is('android')) --> true
when I use browser should be "core"?


Test major functions with the Big List of Naughty Strings

Detailed description
Add test with strings from the Big List of Naughty Strings, to test robustness of the library.
Context
The Big List of Naughty String is "an evolving list of strings which have a high probability of causing issues when used as user-input data." For example, a string with zero-width space (U+200B).
As a text processing library that has to deal with strings of all sorts, both from user-input and from data archive, it is expected that the library should be robust enough to handle variety of character combinations.
Possible implementation

Run major functions through the string list
First step, aim the lib not to break and return a correct output type in a reasonable time.
Correctness is not the goal for now (as there's no definition of correctness yet).

@bact did you mean that adding strings from the Big List of Naughty Strings as an additional test case with all related functions inside pythainlp/tests/ directory which may cause an issue when encounter with Naughty Strings like tokenize or translate engine ?

@pavaris-pm  Exactly. We can start with tokenize tests.

@pavaris-pm Exactly. We can start with tokenize tests.

Cool! i will try with that first. Do we need to test will all naughty string in that repo? or just some sample of it is ok. Since the repo itself has a blns.txt file that divided it into categories like reverse string , special characters, etc. According to that, we can choose about 2-3 strings in each categories in order to test a library with each categories. What do you think? @bact

blns.txt contains 496 test strings (I use egrep -cv '#|^\s*$' blns.txt). Maybe not too big to test all?
But if it takes too long time in the test (which may affect productivity) we can focus on relevant categories.
I would say these categories are more relevant
Group 1: (non-)whitespaces and control characters - as they occurred a lot and sometimes our regular expressions may not well covered them:
#       Special Characters
# ASCII punctuation.
# Non-whitespace C0 controls:
# Non-whitespace C1 controls:
# Whitespace:
# Unicode additional control characters:
# "Byte order marks",

Group 2: string-length related: some non-careful string manipulation may breaks some of these strings. For this group, I think the expected behavior for the testing is for any f(), len(str) should equal to len(f(str)). But I may wrong.
#       Unicode Symbols
#	Two-Byte Characters
#	Strings which contain two-byte letters:
#	Special Unicode Characters Union
#	Changing length when lowercased
#	Japanese Emoticons
#	Emoji


RCE_H26X_PREPEND_SC flag doesn't add start code prefix to all NAL types

I was testing the flag and I noticed that it only adds the start code prefix to certain HEVC NAL types, but not to all types such as SPS or PPS which are also needed. This was confusing trying to figure out which one it adds to, and wouldn't it be better if it added the start code prefix to all types?

Fixed by 54924c2d5dc60fb65ec0902b7b05b122b171abf2


update readme  file

and please add a label of hacktoberfest-accepted

and please add a label of hacktoberfest-accepted

Sorry, but this doesn't count for the Hacktoberfest event.
You can contribute by doing quality contributions. See https://hacktoberfest.com/participation/


Implement "is somebody home" -detection by pinging IP (= phone present in Wifi)

Not reliable. Pings don't arrive when a mobile device is sleeping


[TRANSLATION] Modules 3 and 4 for the EN to VI

December 2024 Student Submission
Module Completed

[x] Module 1: Instruction Tuning (prePR)
[x] Module 2: Preference Alignment (prePR)
[x] Module 3: Parameter-efficient Fine-tuning
[ ] Module 4: Evaluation (WIP)
[ ] Module 5: Vision-language Models
[ ] Module 6: Synthetic Datasets
[ ] Module 7: Inference
[ ] Module 8: Deployment

hi, i'm back. @duydl do you want to review module 3? I'm working on module 4 maybe I will push it tomorrow.


How to use different animations delay on appear than on going away

This is what I would like to achieve:

When the widget appears I want the animation to be delayed by 500 ms and fade in for 300 ms
When the widget disappears I want the animation to start immediately and fade out for 300 ms

Currently when using AnimatedOpacity I achieve it by simply calling setState after a dedicated Timer(Duration(milliseconds: 500)), but I would love to define this declaratively.
Is there a way to achieve it?
I achieved this effect using AnimatedOpacity as follows:
import 'dart:async';

import 'package:flutter/widgets.dart';

/// Widget that delays opacity change when widget
/// appears (in) or disappears (out)
class DelayedAnimatedOpacity extends StatefulWidget {
  const DelayedAnimatedOpacity({
    super.key,
    Duration? delayIn,
    Duration? delayOut,
    this.duration = const Duration(milliseconds: 500),
    required this.child,
    required this.visible,
  })  : delayIn = delayIn ?? const Duration(milliseconds: 500),
        delayOut = delayOut ?? Duration.zero;

  final Duration delayIn;
  final Duration delayOut;
  final Duration duration;
  final Widget child;
  final bool visible;

  @override
  State<DelayedAnimatedOpacity> createState() => _DelayedAnimatedOpacityState();
}

class _DelayedAnimatedOpacityState extends State<DelayedAnimatedOpacity> {
  bool visible = false;
  Timer? _timer;

  @override
  void initState() {
    super.initState();
    if (widget.visible) {
      _timer = Timer(widget.delayIn, () {
        if (mounted) {
          setState(() {
            visible = true;
          });
        }
      });
    }
  }

  @override
  void didUpdateWidget(covariant DelayedAnimatedOpacity oldWidget) {
    super.didUpdateWidget(oldWidget);

    if (widget.visible != visible) {
      if (widget.visible) {
        _timer?.cancel();
        _timer = Timer(widget.delayIn, () {
          if (mounted) {
            setState(() {
              visible = true;
            });
          }
        });
      } else {
        _timer?.cancel();
        _timer = Timer(widget.delayOut, () {
          if (mounted) {
            setState(() {
              visible = false;
            });
          }
        });
      }
    }
  }

  @override
  void dispose() {
    _timer?.cancel();
    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return AnimatedOpacity(
      duration: widget.duration,
      opacity: visible ? 1 : 0,
      child: widget.child,
    );
  }
}

You can use .animate().fade(delay: 500.ms, duration: 400.ms)
Put this code at the end of the your widget.
you have to put time in delay after initializing a page , how much time you want to take .
After delay time ends it will blink for given duration .
I think i understand your question and able to give answer.


Refactor the data generation

This removes the circular dependency that made the app dependent on datafiles that were dependent on the app. Previously the app had to be built first, resource ids generated, the datafiles updated then the app rebuilt with the data files. This was so that the resource ids for all of the i18n strings could be properly embedded.
However it was super fragile. Add a resource and all the ids could change, causing mislabelled objects or, more often, crashes. Even different versions of the Android SDK seemed to be able to cause this. Ugly, brittle and dangerous to developers it had little to recommend it other than we knew no better at the time.
This is still quite ugly (the creation of a module to be included in the app containing the proto as well as the tools app reaching into that module to borrow a source file is not what I wanted), but should be less brittle. A missing resource will now result in an ugly "MISSING LABEL" label in the UI but should at least not crash.

Fairly major change - will need some careful QA.

Profiling shows no appreciable difference vs the old app. The new proto-lite library shrinks the app a tad.

Oh... oh yes... it's so beautiful! (^✧ ∇ ✧)^

Hopefully it'll make life easier.  I'm seeing some crashes on some devices though due to the new protocol buffer library, so need to look into that.

@jaydeetay I'm able to get built APKs to launch now, so I can now get to testing the AndroidX Preference migration and fixing any crashes that appear before the PR is made. I'm actually making some progress on it now, but it requires kind of a large overhaul of class usages in the project. The settings will definitely look different for sure.

Hi - I just want to caution about doing any major refactoring...I'm not
sure I want that at this stage. Our testing and QA isn't adequate right now
and I don't have the time to fix things if they go wrong.
On Sat, Oct 17, 2020 at 12:54 PM Taco<EMAIL_ADDRESS>wrote:

@jaydeetay https://github.com/jaydeetay I'm able to get built APKs to
launch now, so I can now get to testing the AndroidX Preference migration
and fixing any crashes that appear before the PR is made. I'm actually
making some progress on it now, but it requires kind of a large overhaul of
class usages in the project. The settings will definitely look different
for sure.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sky-map-team/stardroid/pull/363#issuecomment-711045590,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD5LWC5UOEBXZCSQTVE5SNTSLHD4XANCNFSM4SMIVDJA
.

@jaydeetay That's fine, I'm probably not going to make it anyway (it's too rough around the edges right now).


The bot not release pokemon randomly

[20:24:18] Scanning area for objects....
[20:24:18] Walking from (40.4173388, -3.6830914) to [40.419138800000006, -3.6830914] 0.20km
[20:25:06] Something rustles nearby!
[20:25:06] A Wild Rhydon appeared! [CP 1390] [Potential 0.47]
[20:25:06] IV [Stamina/Attack/Defense] = [15/4/2]
[20:25:09] Catch Rate with normal Pokeball is low (12.22%). Throwing Razz Berry... (4 left!)
[20:25:09] Catch Rate with normal Pokeball has increased to 18.34%
[20:25:09] Using Ultraball (chance: 34.43%)... (27 left!)
[20:25:10] Captured Rhydon! [CP 1390] [Potential 0.47] [15/4/2]
[20:25:18] Something rustles nearby!
[20:25:19] A Wild Diglett appeared! [CP 36] [Potential 0.38]
[20:25:19] IV [Stamina/Attack/Defense] = [5/12/0]
[20:25:22] Using Pokeball (chance: 78.21%)... (182 left!)
[20:25:23] Captured Diglett! [CP 36] [Potential 0.38] [5/12/0]
[20:25:32] Something rustles nearby!
[20:25:32] A Wild Scyther appeared! [CP 904] [Potential 0.44]
[20:25:32] IV [Stamina/Attack/Defense] = [7/2/11]
[20:25:34] Catch Rate with normal Pokeball is low (21.79%). Throwing Razz Berry... (3 left!)
[20:25:35] Catch Rate with normal Pokeball has increased to 32.68%
[20:25:35] Using Greatball (chance: 46.24%)... (49 left!)
[20:25:35] Captured Scyther! [CP 904] [Potential 0.44] [7/2/11]
[20:25:44] Release config for Scyther: Conf, CP 1000 or IV 0.7 - Poke, CP 904 IV 0.44
[20:25:44] Exchanging Scyther for candy!
[20:25:44] Something rustles nearby!
[20:25:44] A Wild Scyther appeared! [CP 1154] [Potential 0.29]
[20:25:44] IV [Stamina/Attack/Defense] = [10/3/0]
[20:25:48] Catch Rate with normal Pokeball is low (19.15%). Throwing Razz Berry... (2 left!)
[20:25:48] Catch Rate with normal Pokeball has increased to 28.73%
[20:25:48] Using Greatball (chance: 40.96%)... (48 left!)
[20:25:49] Captured Scyther! [CP 1154] [Potential 0.29] [10/3/0]
[20:25:57] Release config for Scyther: Conf, CP 1000 or IV 0.7 - Poke, CP 1154 IV 0.29
[20:25:57] Exchanging Scyther for candy!
My config:
http://pastebin.com/qkW99A5i
Edit: Lastest commit (Can anyone explainme how i can use git to know the commit?)

seems like same issue with #1463

Duplicate of https://github.com/PokemonGoF/PokemonGo-Bot/issues/1463

/closed


Sim card data such as sim serial number cannot be retrieved for android 13

I am facing the same issue using android 13 ? Any update on this ?

yes exactly i'm facing the same problem right now, have you updated for this?? please.

I am facing the same issue using android 13 ? Any update on this ?

have you found an alternative solution to this problem?? please


[US] Integrate User Management System Into DB

As a platform developer, I want to have a user management system integrated to the database system and PostgREST that I can use to build the related views on top of.

MoSCoW
Risk
Story Points

Must have
High
8pts

Acceptance Criteria

All the components from the User Management & Authentication proposal have been properly integrated into the database system
Lowest access required to sensitive data is enforced
No password is stored as plaintext
Functionalities have been tested and work as intended

Risks

Manipulation of sensitive information
Management of access to sensitive information
Some uncertainty regarding how the final system will work

Pre-requisites

[x] Meet with the stakeholder to clarify the requirements
[ ] Complete the requirements of this US

Tasks

[x] Read over carefully the User Management & Authentication proposal (#89 )
[x] Integrate step by step the different components following the guide provided by PostgREST https://postgrest.org/en/stable/how-tos/sql-user-management.html
[ ] Add any other database components necessary that is specific to our system.

Demo Video
https://github.com/WillTrem/UInnovate/assets/64431699/22f8c1c9-c68d-4e31-91e3-20ed450a20b9

Looks good! Signing off!


S2 | Enhanced signup route

API Request
Story

As a developer
I need an API route to Sign a user up

Acceptance Criteria

Route /signup
Method: POST
Request Object:

{
<EMAIL_ADDRESS>        "password":"1234"
}

Response

{
        "id":"abcd-efgh-ijkl-mnop",
<EMAIL_ADDRESS>}

written in a users-api/routes/signup.js file
Should validate that the mock password is at least 6 chars or more
Should validate that the mock password includes letters and numbers (string.match(/[a-z0-9]+/i))
Should validate that the mock email includes the @ symbol and .com || .edu
Should return a 200 if correct
Should return a 401 with the following object if it fails any validation

{
  "message": "Invalid Credentials"
}

Approach

Since we have no database yet, just send back example data located in this ticket.

@ardagokce @shalayiding

// users-api/routes/signup.js
// Sprint 1: A mock API for signing up (#14)
//require express & create router
const express = require('express');
const router = express.Router();
// a test route
router.get('/testRoute', (req,res) => res.send('Test successful.'));
// check for content in username & password fields of body
// if data is missing, send invalid information error
// if something was entered for username and password,
// confirm the signup and display the information
router.post('/', (req, res) => {
if(req.body.username)
{
if((!req.body.username) || (!req.body.password))
{
res.status(403).send('Invalid information.');
}
else
{
res.send('Account Confirmed. \n' +
'Username: ' + req.body.username + '\n' +
'Password: ' + req.body.password);
}

}
//email
if (req.body.email)
{
var password=req.body.password;
var Email_Check=/^([a-zA-Z0-9_.-])+@(([a-zA-Z0-9-])+.)+([a-zA-Z0-9]{2,4})+$/;
var Email=req.body.email;
if((!password)||password.length < 6||!password.match(/[a-z][0-9]+/i)||!Email_Check.test(Email))
{
  res.status(401).send('"message": "Invalid Credentials"');
}
else{
  res.status(200).send(
'id: ' + "abcd-efgh-ijkl-mnop" + '\n' +
'email: ' + req.body.email);

}

}
else
{
res.status(403).send('Invalid information.');
}
});
//export so other modules can access
module.exports = router;

i have some trouble with checking email address , if i put<EMAIL_ADDRESS>it still think this is valid email address.

@shalayiding this has to be in proper PR format, like last sprint.
for checking emails, it doesn't have to be too strict, so I wouldn't worry about it.

#39

We still need this logic in the current signup route


Update api musicanalysis

All methods in partitura.musicanalysis work well with the new note_arrays, and it is now possible to use the methods for both Part, PartGroup, PerformedPart objects, as well as lists of Part objects.
There are two new methods to get structured note_arrays from Parts, PartGroups or lists of Parts: note_array_from_part_list and note_array_from_part.  These methods are now used in the note_array properties of Part and PartGroup objects to avoid unnecessary code duplication.

Commit  7b13650 adds the example of usage of music analysis methods from issue #13


Release cargo-apk-0.9.7

Fixes https://github.com/rust-windowing/android-ndk-rs/issues/375
Depends on #3
Initially opened as https://github.com/rust-windowing/android-ndk-rs/pull/376

Now that we have merged stuff in this repo I think it will be a bit tricky to rebase and change any of the patches on the top of the filtered repo without confusing github (it will change commit hashes)
That's ok if we agree to address any issues via new patches on top + PRs from now on.
Hopefully all the big stuff is fine.

@rib If that's re. my comments on your initial porting commits, I think pushing those on top (via a PR for review) is fine. I wasn't really sure if you were still planning on force-pushing though I was already told to not use the android-ndk-rs repo for these crates anymore, and really wanted to get this release out :)

Right, esp if spinning a release from here we definitely wont want to force push anything here any more, so we can aim to handle any feedback or issues with the patches I made via PRs.

Can filter-branch also transplant the tags for cargo-apk and ndk-build? Maybe to late now to figure that out in retrospective, maybe for the other repos though?

Can filter-branch also transplant the tags for cargo-apk and ndk-build? Maybe to late now to figure that out in retrospective, maybe for the other repos though?

I'm not sure if that was an option but yeah we'll just have to recreate older release tags manually to bring those across

I just pushed the few that I needed to create the release, having the rest would be nice for back-reference but might be better to leave those to the original repo, it's a lot of work unless you start comparing patchids (for the subdirectory if possible) or commit titles that have ndk-build-*/cargo-apk-* tags on them.

It's probably best to just defer to the tags in this repo for older releases, considering that the standalone repo won't build correctly for all those past commits that correspond to releases (unless you would back port the changes I made on top).

Correct, it'll only generate lots of "failed CI" traffic.


update for 4.18

This PR updates the amd and webpack samples to latest 4.18. It also adds a RequireJS sample.

The Dojo and RequireJS samples have been updated.

Updated the Dojo build file
Update Dojo sample to run unbuilt
Update RequireJS sample to run unbuilt

This got merged a little early, I'll make another PR with fixes


CRLP: GAU Rollups not updated if the last Allocation record for a GAU has been deleted

GAU Rollups execute only in Skew Mode where the main object queried is the Allocation object. As a result, if there are no Allocation records for a GAU that had previously had Allocations records that were rolled up, the GAU will not be updated to change the rollup totals to zero/null after the last Allocation is deleted when the next nightly job is executed or when the [Run Batch] button is used in NPSP Settings.
This can affect other Skew Mode rollups as well, though this is unlikely. For example, it would extremely rare for a single Contact to go from having more than 250 related Soft Credit Opportunities to having zero attached Soft Credit Opportunities at once (in a single day).

**lurch: add

Tracking W-033186

Updated bug: W-8276496


increase lint timeout

Description
increase lint timeout To 5 minutes
Checklist

[x] I have tested this code
[ ] I have added unit test to cover this code
[ ] I have updated the documentation (README.md and /website/docs)
[x] I have followed the contributing guide

Coverage: 93.766%. Remained the same when pulling 6e65fbd09934d9c90c6d11f00e5b81ad2111cb46 on increase-lint-timeout into 0e9a271d369d2257300acf178cab7c89a41aeeb5 on main.


Option validation of analyzer and checker configs

Draft for the time being...

Is #4204 superseding this?


I was told to submit this by the website after it couldn't save!

Error: Service Unavailable
Error: Service Unavailable
    at Request.<anonymous> (https://homebrewery.naturalcrit.com/homebrew/bundle.js:73949:19)
    at Request.Emitter.emit (https://homebrewery.naturalcrit.com/homebrew/bundle.js:1898:20)
    at XMLHttpRequest.xhr.onreadystatechange (https://homebrewery.naturalcrit.com/homebrew/bundle.js:74258:10)
{
  "status": 503,
  "method": "PUT",
  "url": "/api/update/BJmVTAorsNtK"
}

Looks like this was a temporary busy moment on the server. Let us know if it happens again.


Make the minimum submit configurable

This is so that in development can use TellorPlayground to submit
transactions faster which will speed up the debugging during
development. At the moment it doesn't work because of the contract
binding, but at some point will see how we can fix this.
Signed-off-by: Krasi Georgiev<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build 454357849

0 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 35.82%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/ops/miningManager.go
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/ops/miningManager.go
1
0%

Totals

Change from base Build 453021484:
0.0%

Covered Lines:
1455

Relevant Lines:
4062

💛  - Coveralls

so how would this work with the playground?  Could you just set the contract to be a playground address and it would auto submit?  I think the problem is that the playground uses a different function for submitting, so we'd need to write an interface

yes I haven't figured out the details yet but just shooting ideas on how to improve the local development instead of waiting 15min for every submit. Regardless of how we implement the calls to the yellow playground would need this period to be configurable in telliot that is why I thought to just add now.


Support Querying

Just thought I'd let you know, some of the endpoints support querying. This isn't documented anywhere yet due to a lack of time on my part: https://github.com/RocketChat/Rocket.Chat/blob/develop/packages/rocketchat-api/server/v1/helpers/parseJsonQuery.js

See here for some documentation: https://github.com/RocketChat/Rocket.Chat/pull/5411

@graywolf336 👍 We will add query support for *.list interfaces

is it okay if I switch to a .list(settings : { offset, count, query : { sort, fields, query }}, callback) signature then? I'd process it in a central list mapper so that on future breaking changes no client would have to change anything, only us.

Why not switch all of your options to the es6 object spread operator? function channelsList({ offset, count, query: { sort, fields, query }}) that way you can easily add more supported parameters without breaking things.

we are breaking things now, that's why I asked :laughing:
With mapping I meant breaking changes in rocket.chat - let's say offset and count would be moved in a page object in a new version, the mapper could handle that change for clients using that library. but I guess that would mean introducing unnecessary complexity while making things harder to find out/debug for the consumer. stupid idea, dropped it :balloon:

But spread operator is a great idea, thx - will have to test if it works with node5, otherwise maybe we should consider dropping support?

The pagination items won't change until v2 of the rest api, so if you're developing this for v1 then it is safe to assume the names will stay exactly the same. So, it shouldn't have any breaking changes for a while. :)

good to know, thx :)
@qeesung tested it, we will have to drop node5 support for object spread operator. Okay for you? That case I'd remove it from travis and implement a first proposal for channels.list

@MatthiasKainer Cool! It is a good idea to switch to ES6. Should we switch all interfaces to ES6?

Jop, step by step - also we should make use of block level scoping and classes and other stuff that comes with es6. Migrated lib/net.js as an example :)

implemented sort, fields and query as well. @graywolf336 can u check if I caught the description correctly in pr https://github.com/qeesung/rocketchat-node/pull/25 pls? Tests are green

for channels is on master. other *.list interfaces will be added iteratively while we implement them for the v1 api.
thx @graywolf336 for the info :)


BIDS-connectome compliant output layout

The current atlas are stored using templateflow convention.
There's BIDS-atlas: https://peerherholz.github.io/bids_atlas/
Connectome:
https://docs.google.com/document/d/1ugBdUF6dhElXdj3u9vw0iWjE6f_Bibsro3ah7sRV0GA/edit#heading=h.mqkmyp254xh6
See 4.1.3 for example

I am considering adding:

group/ along side the existing sub-[label]
add func/ to each sub/ses
Match the functional derivative BEP for time series metadata.
Match the connectome BEP for connectome meta data.

adding this from #52 and closing it
brain atlas data (BIDS extension 17: https://bids.neuroimaging.io/bep038)
as well the output of brain connectivity analysis (BIDS extension 17: Generic BIDS connectivity data schema, https://bids.neuroimaging.io/bep017)
maybe not for the first release though
also should consider dumping some of those naming and metadata handling into nilearn in the long run

Just to log the conclusion that has been reached on my side:
We will let this bids app to output everything according to BIDS compliance.


Process one message at a time

Hi, i am using this library for past one year and very happy with it. The only thing i am not able to figure out is how does i constrain my subscriber to process one message at a time.
In current code (below) , the subscriber starts processing all message at the same time.
var subscribe = function (queueName) {
var subscribeHeaders = { 'destination': '/queue/' + queueName, 'ack': 'auto'};
var connectionManager = new stompit.ConnectFailover(server);
var channel = new stompit.ChannelFactory(connectionManager);
logger.log('info', 'subscriber listening on queue ' + queueName);
channel.subscribe(subscribeHeaders, function (error, message) {
if (error) {
logger.log('fireError', 'subscribe error ' + error.message);
return;
}
message.readString('utf-8', function (error, body) {
if (error) {
logger.log('fireError', 'read message error ' + error.message);
return;
}
logger.log('info', 'received message: ' + body);
message.ack();
processMessage(JSON.parse(body));
});
});
};

The trick to controlling the message cycle is to delay calling message.readString on the next message until after the current message is processed. The subscribe callback function won't be called again until the latest message is read.
stompit doesn't have a feature to serialize message processing, but you can define your own helper function to get the desired behaviour. Example code of such a function:
function synchronisedHandler(callback) {

  var processing = false;
  var nextMessage = null;

  var next = function() {

    if (nextMessage === null) {
      processing = false;
      return;
    }

    var currentMessage = nextMessage;

    nextMessage = null;

    callback.apply(null, currentMessage);
  };

  return function(error, message) {

    if (processing) {
      nextMessage = [error, message, next];
      return;
    }

    processing = true;

    callback(error, message, next);
  }
}

The synchronisedHandler function wraps an existing subscribe callback function and passes a third argument, the next function, that the subscribe callback function calls once it's ready to read and process the next message.
Here is a modification to your message handler code that works with synchronisedHandler function:
channel.subscribe(subscribeHeaders, synchronisedHandler(function (error, message, next) {

    if (error) {
        logger.log('fireError', 'subscribe error ' + error.message);
        return;
    }

    message.readString('utf-8', function (error, body) {

        if (error) {
            logger.log('fireError', 'read message error ' + error.message);
            return;
        }

        logger.log('info', 'received message: ' + body);

        processMessage(JSON.parse(body), function() {
            message.ack();
            next();
        });
    });
}));

The processMessage function should accept a callback argument that is called after processing has completed.

Hi, I'm trying to  implement this in a connectFailover but I'm still recreiving all messages at  the same time, any advice on this?, thanks in advance.
Btw, we're happy using  this library thanks 👍


How to change font size using v-model after editor loaded

@rahulkumarsingh73690  Can be used props option
<ace-editor
   v-model="importJson"
    lang="json"
    :options="{
      fontSize: '16px'
    }"
    theme="textmate"
    style="height: 400px;"
/>

If you want to have responsive control over font size you can add a value wrapping a computed
More option config


handle forward referenced in prefer-const rule

fixes error related to usage of forward references in prefer-const rule
PR checklist

[x] Addresses an existing issue: https://github.com/palantir/tslint/issues/1898
[x] New feature, bugfix, or enhancement - bugfix

[x] Includes tests

[ ] Documentation update - N/A

What changes did you make?
After this change list of variable declarations in block scope is collected immediately after entering the scope. This prevents cases when variable is syntactically located after its usage

Thanks for your interest in palantir/tslint, @vladima! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

@vladima can you turn on circle ci?

Doesn't seem to catch:
module MyModule {
    let moduleMember = 4;
}

@nchen63 yes, if you can tell me how to do it

go to https://circleci.com/add-projects and add your fork

done

another case that isn't caught - inside a switch statement

that's weird, CircleCI isn't running still. Can you fix the linting errors when you run npm run verify?


Interface with OKCupid broken

It appears changes at OKCupid have created problems with okcupyd interface with OKCupid. Attempts to search generate out-of-range errors. Here is the call tree when trying to run a test implementation of my okc automation script.
Traceback (most recent call last):
  File "okctest.py", line 112, in <module>
    main()
  File "okctest.py", line 109, in main
    process_task(task, profiles)
  File "okctest.py", line 5, in process_task
    for profile in profiles[:200]:
  File "/usr/local/lib/python2.7/dist-packages/okcupyd/util/fetchable.py", line 192, in __getitem__
    return self._handle_slice(item)
  File "/usr/local/lib/python2.7/dist-packages/okcupyd/util/fetchable.py", line 218, in _handle_slice
    value = next(iterator)
  File "/usr/local/lib/python2.7/dist-packages/okcupyd/util/fetchable.py", line 164, in _make_nice_repr_iterator
    for item in original_iterable:
  File "/usr/local/lib/python2.7/dist-packages/okcupyd/util/fetchable.py", line 284, in fetch
    for item in self._processor.process(text_response):
  File "/usr/local/lib/python2.7/dist-packages/okcupyd/util/fetchable.py", line 315, in process
    yield self._object_factory(element)
  File "/usr/local/lib/python2.7/dist-packages/okcupyd/search.py", line 232, in <lambda>
    **MatchCardExtractor(match_card_div).as_dict
  File "/usr/local/lib/python2.7/dist-packages/okcupyd/search.py", line 198, in as_dict
    'id': self.id,
  File "/usr/local/lib/python2.7/dist-packages/okcupyd/search.py", line 151, in id
    return int(self._id_xpb.one_(self._div))
  File "/usr/local/lib/python2.7/dist-packages/okcupyd/xpath.py", line 75, in one_
    return self.apply_(tree)[0]
IndexError: list index out of range

The change at OKCupid must have happened within the last two weeks, as the full automation script and this test worked fine not too long ago.

I was also getting this last week, but just now all the profiles I had tried to access that the script barfed on with this error seem to be working fine, so it seems that okcupid reverted whatever change they had made.

@sagittarian Were you using search? I think that this should only happen when you are searching.
The relevant piece of code is:
_id_xpb = xpb.button.with_classes(
        "match_card_rating",
        "binary_rating_button"
).select_attribute_('data-tuid')

So i guess the spot in the dom to look for is the new binary rating button. Maybe there are some cases where this attribute is on the button and others where it isn't?

Nevermind, the error I was getting was different.  It just involved the same line in xpath.py.
I have been running searches this evening however and haven't had a problem.

yeah, that line in xpath will almost always be the last line in any traceback that results from a change in the structure of the okc DOM, so it really could be just about anything.
Do you have a traceback of the error you were getting? it is probably also an issue worth looking in to.

The relevant part is here:
  File "/home/amesha/src/okcupyd/okcupyd/util/__init__.py", line 49, in __get__
    value = self.func(obj)
  File "/home/amesha/src/okcupyd/okcupyd/profile.py", line 203, in id
    return int(self._id_xpb.one_(self.profile_tree))
  File "/home/amesha/src/okcupyd/okcupyd/xpath.py", line 75, in one_
    return self.apply_(tree)[0]
IndexError: list index out of range

It was caused by an attempt to access profile.id.  As I said everything is working now however with no changes.

huh okay, let me know if you see it again. It could be something that only happens on certain profiles, so the target username would also be helpful

I went back and accessed the same profiles in the same way and didn't get the problem, so it's really gone for now.  If I see it again and I'll you know.

@sagittarian Did you say that search is working for you? I am getting the error that @bitromantic reported on my accounts.

@bitromantic This should now be fixed. Can you confirm?

The problem I was getting (with profile.id) seems to have been related to a network issue on my end, nothing to do with changes at okcupid.  The reason that search was working for me was that I had fixed the code when my searches broke and forgot about it.

The problem I was getting (with profile.id) seems to have been related to a network issue on my end, nothing to do with changes at okcupid. The reason that search was working for me was that I had fixed the code when my searches broke and forgot about it.

Cool. Definitely let me know if you have any other fixes that are lying around.
I'm thinking about trying to set up travis to run against the live site every so often, so I can find out when things break, but for now I only learn about breakages from the relatively small group of okcupyd users, since I havent been using okcupyd very actively as of late (since my ip has been banned and my account has been deleted).


VEDIO WAS NOT RUNNING IN MY HTML

plz tell me what i do for that

Could you please provide your code snippet?

MOBILE SA VIDEO KA URL KASA COPY KARA

Give me code snippet of your video tag and URL of your video I'll fix it out!


Support cwd option

This PR closed #6

example:
# ~/.centoria.toml
[lls]
runas = 'alias'
command = 'ls -al'
cwd = '~/'


Bundle maintained and manage Firebase message

Is the bundle maintained?
And can we send push notification with Firebase with this bundle?
I saw a pull request but not merged yet.

Yes - it is maintained albeit slowly. There is one outstanding change on #141

Thanks a lot.
I saw the Pull request, but why is it not merged?

There is one outstanding change that needs to be made. Once this is done, I'm happy to merge & release

ok, thanks a lot for your reactivity.


Greater usability of a VersionsCollection

Changes summary:

Add VersionsCollection::isEmpty() method
Add VersionsCollection::toArray() method
Add immutable VersionsCollection::sortedAscending() and VersionsCollection::sortedDescending() methods; mark VersionsCollection::sort() as deprecated
Add VersionsCollection::first() method
Add VersionsCollection::last() method
Add VersionsCollection::majorReleases() method
Add VersionsCollection::minorReleases() method
Add VersionsCollection::patchReleases() method

@ntzm You might be interested to review/comment this one as well.

Looks good to me, thanks for sorting! 👍

@ntzm Are there any other particular collection features that you might find useful, i.e. for Roave/BackwardCompatibilityCheck library where you're making some contributions?
Sorry for bothering you with these questions, but the only real feedback I got was from your side.

@Ocramius Can you elaborate this one further: https://github.com/Roave/BackwardCompatibilityCheck/pull/61/files#r186212959? I assume you're suggesting that sort() should be returning a new collection (immutability). I wish I had foreseen this earlier, because such a change at this stage would be a BC break.
Also, if I remember well PHPStan detected some issue with the way author tried to use VersionsCollection. Perhaps I can extend it with some first() and last() helper methods, but I would rather try keeping it minimalistic without exposing too much unnecessary staff.

@nikolaposa an improvement could be to make a new sorted() method, and make that immutable.
Then deprecate the old one and remove it in the next major.
Changing the existing method is too implicit and quite problematic.

@Ocramius I've added immutable VersionsCollection::sorted() method.
I also added first() and last() methods, but I'm in a doubt whether I should use nullable return type as it is now or raise an exception instead: https://github.com/nikolaposa/version/pull/18/files#diff-b701aaf410484f65ecf6e06b7f9a411eR41.

as it is now or raise an exception instead

first and last can correspond if the collection is never empty

Yeah, but the collection can be empty: https://github.com/nikolaposa/version/blob/master/src/VersionsCollection.php#L26. Also, empty collection can be produced after filtering via matching().

@nikolaposa makes sense. A sorted() empty collection is still an empty collection, while first() and last() throwing is indeed OK.

@nikolaposa we should probably talk about this lib over a rakia this week :-)

@Ocramius LOL, agreed. :)

@nikolaposa as discussed in person, it would be good to have a simplified API dedicated to:

people writing version comparison software (this project)
people writing release software (next major, next patch, next minor, or figuring out previous versions to create changelogs)

As an example, the API could be similar to:
        $versions
            ->sortedAscending()
            ->previousPatchFor($version);

@Ocramius I was examining different use cases in BackwardCompatibility library, PickLastMinorVersionFromCollection implementation in particular, which gave me the idea for adding VersionsCollection::filterMinor() and VersionsCollection::filterPatch() methods. Picking last minor version from a collection can now be as simple as:
$lastMinor = $versions
    ->filterMinor()
    ->sortedDescending()
    ->first();

@nikolaposa filterMinor() is not that descriptive: what does it do in your mind? Can you write an explanation for it, that we can then distill to a ubiquitous method name? Same for filterPatch()

@Ocramius Yeah, I was struggling to come up with a descriptive name. So for for instance, filterPatch() filters-out everything except patch versions (x.y.Z > 0), as in: https://github.com/nikolaposa/version/blob/improvement/versions-collection-greater-usability/tests/VersionsCollectionTest.php#L331.
Would something like filterPatchVersions(), findPatchVersions() or pickPatchVersions() describe it better?

Do you think something like filterPatchVersions(), findPatchVersions() or pickPatchVersions() describe it better?

Not really, and also, those don't seem to be really that useful... Check https://github.com/Roave/BackwardCompatibilityCheck/blob/6c797c5118e7b081eff97db01bcc62f35aa9daf8/src/Git/PickLastMinorVersionFromCollection.php for a fixed version of that consumer.

I try to avoid methods returning Version object, because things become complicated when I need to figure out what to do in case of exceptional conditions which is when nothing is found in this particular case. Returning null is always a bad idea in my opinion. Introducing Special Case at this stage doesn't make much sense. Raising an exception feels kinda inappropriate for the semantics of routines that "try to find" something.
That's why I wanted to keep VersionsCollection consistent and broad in sense that it always return new collection for filtering operations, and let the consumer invoke isEmpty(), first(), etc. on it:
findMajorReleases()->sortedDescending()->first().
Or, those type of operations could be delegated to some ReleaseResolver class for better separation of concerns, but I'd rather avoid introducing a new component in this milestone.
I think I'll go with extending VersionsCollection API according to your suggestions, but I'll just need to figure out the most meaningful solution for handling exceptional conditions. Ideas are welcome. @Ocramius

@Ocramius I eventually ended up with:
$latestMajorRelease = $releases
    ->majorReleases()
    ->sortedDescending()
    ->first();

Also added test case that covers it: https://github.com/nikolaposa/version/pull/18/files#diff-f83b2394fa806ea8e550f37c251c7f76R375
majorReleases() still might not be an ideal method name, but perhaps tomorrow I'll have more inspiration for naming. :)

->majorReleases()

So this basically fetches all X.0.0, right? That sounds quite understandable, thanks! :+1:

Yes, exactly. Thanks for the feedback!
I think this is it as for this PR, but I'll wait with merging to see what @asgrim was up to with: https://twitter.com/asgrim/status/1003608706687397888


Fixes DCOS-8909 by deleting the history entry for all job runs, if the job spec is deleted.

See https://mesosphere.atlassian.net/browse/DCOS-8909

Looks good, merged :-)


[FEATURE REQ] Spring Autoconfiguration Library should support Recovery

Is your feature request related to a problem? Please describe.
Spring Libraries need to be able to support redundancy for the services it integrates with, in where the service doesn't support or additional coverage is needed.
Describe the solution you'd like
Currently Autoconfigurations only supports connecting to a single client, as supported by the Azure SDK. I should be able to Autoconfigure multiple clients in order to recover from a failed resource.
Describe alternatives you've considered
If additional auto configurations isn't added it should be made more than clear to customers that for services that don't support full service side redundancy that their setup isn't suggested and isn't considered secure.

is there any other spring lib supporting such feature we can take as an example?


Can't access the RethinkDB context set up by the Hapi plugin.

According to the documentation, The RethinkDB library and the connection are bound to the server context inside handlers on  this.rethinkdb and this.rethinkdbConn.
However, I'm not being able to access those, only using request.server.plugins etc. Is it because the context set inside a plugin with server.bind() is only available inside the plugin itself ?
My code is like this:
index.js
const plugins = [{register: require('hapi-rethinkdb'), options: {url: //url here}]

server.register(plugins, () => {
    server.route(require('./routes'))
    server.start()
})

routes.js:
{
   method: 'GET',
   path: '/list-of-things',
   handler: function(request, reply) {

      // I have to use it like this:
      const r = request.server.plugins['hapi-rethinkdb'].rethinkdb;
      const connection = request.server.plugins['hapi-rethinkdb'].connection;

      // Because this throws undefined:
      const r = this.rethinkdb;
      const connection = this.rethinkdbConn
   }

 }

Thank you.

It's really weird; I was checking this (long time without seeing the code) and on index.js L37-40 is making the bind correctly as defined in the documentation :-/

In other words: I don't know what's wrong. Tomorrow will probably write some tests for this.

I was checking the documentation here and it says that

When setting context inside a plugin, the context is applied only to methods set up by the plugin.

Maybe that's why. I'm very new to Hapi js so I don't know much about how plugins work, but perhaps there's a way to bind the library and connect with a shorter syntax. I mean, it's also fine as it is now but it's a bit verbose to make reference to rethinkdb and connection like I mentioned.

You can always import all this to a single file, and make it so that file exports them. Remember when you make an assignation on javascript is just a reference and it's cached because of node, so:
var myModule = {}
function setup (server) {
  myModule = {
    r: server.plugins['hapi-rethinkdb'].rethinkdb,
    connection: server.plugins['hapi-rethinkdb'].connection
  }
}
module.exports = myModule
module.exports.setup = setup

So, you call setup where you have server available and then just import myModule and you will have there your references to the plugin.


pop-gtk-theme: init at 3.1.2

Motivation for this change
Simply includes a  popular theme developed by System76.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
(On recommendation of @worldofpeace on this discourse thread) :
cc @worldofpeace @jtojnar @romildo @c0bw3b

Added a few (hopefully helpful) suggestions based on the package I've had kicking around in a local overlay though probably best to defer to folks you CC'ed.
Would personally welcome the inclusion of the package though so thank you for submitting the PR.

Here's screenshots of the gtk-widget-factory with the theme on nixos-unstable:

Haven't checked out the gnome-shell theme or sound theme.

Pushed update with latest version


[Bug] Equality test: failures => '0' is not of type integer

Hi,
I have an issue with dbt-core/dbt-utils equality tests.
They have refered me here as they think it must be something with the adapter. When I run the sql within exasol outside of dbt I get a decimal. Can you verify the problem somehow? I am not really able to verify what the code does since I am not that deep inside of debugging the python code behind dbt.
I would really appreciate any help or even just if you can reproduce. I didn't copy all of the issue into this one because I already replicated it from utils to core.
If I can help in any way, please let me know.
Am looking forward to any help.
Greetings Timo Kruth

Okay I found a way to solve my problem.
When I manualy cast the output of the fail_calc function of the test to INT it works as expected.
The difference in the value of failures from exasol is: In the old version it returns a DECIMAL in the new casted version it returns BIGINT.

Hi Timo,
this unfortunately is a tough one. We did manage to reproduce, but could not get enough detailed visibility in order to explain or get closer to a solution.
Honestly it has been a while ago since we looked into it. Currently we are trying to get the support for dbt-exasol to a new level and eventually also have it verified. Therefor I will be working on it more and will definitely also look out for this one.
Just to let you know and greetings,
Torsten

Hi @TimoKruth ,
as mentioned in pyexasol#95 , we think this issue is solved with todays release of dbt-exasol 1.2.1, since we now explicitly cast values from resultsets in our adapter implementation to Decimal in case of BIGINT or DECIMAL values that get encoded as strings in pyexasol - as discussed in above issue.
We will try to add a test for this issue in the next steps as we also need to add some basic test left for full support of dbt 1.2.x - but we also need to add v1.3 support already.
If you find the time to retest in the meantime, please let us know here.
Regards, Torsten

I actually tested this and we are confident that dbt-exasol 1.2.1 actually solves this issue.

Hej, great that it is solved. I really appreciate it. Will test it as soon as I can update dbt to the corresponding version.


Fix Caffe docker URL

The URL for Caffe docker is no longer available

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! :smile:


Features/fields translations

Changes needed to translate fields from Zoho to Ruby and remember the proper translations, as explained in #29
This way, I can have an utm_campaing attribute in my Zoho and Ruby will remember that the original attribute is not called Utm Campaign but utm_campaign.
I have tested it with our application (not in production yet) But I couldn't run any spec because I don't have a usable Zoho API Key (the only one I have is populated with real data and I can't run specs over them). Is there any procedure to run specs without a real API Key?

@brenes Had to manually do the merge. Will test and release in the next couple of days. Thanks!

Thanks to you! And sorry for not being able to test it in months :(

@amalc We had an issue with a field containing a '/'. I solved it in my branch by  cleaning it on the ApiUtils#string_to_symbol and everything worked fine.
I don't know if there's a way to obtain a right ruby variable name from a string with strange characters, so this problem doesn't repeat on the future.

I've got your changes for mapping in master to map between fields. Will release as soon as I test. Thanks for the heads up.

I'll add a reg ex to always generate a legal ruby variable. I take it you're using the mapping fixes you added? If you're processing a field with '/'?


feat(markdown-overrides): Add Codepedia WIP component

Overview

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

🚀 Styleguide deploy preview ready!https://5fe26268a52fb40f3c1656db--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/markdown-overrides@0.4.3-alpha.194508.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/markdown-overrides@0.4.3-alpha.f56f18.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fe27df36bbf6735bffd24bf--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/markdown-overrides@0.4.3-alpha.65322b.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fe35a9854191c8e417c5c67--gamut-preview.netlify.appDeploy Logs


Let beta depend on stimulus/response

Let beta be specified like start_v, indexed by stimulus element - response pairs, and use the appropriate beta-value in the decision function.

Fixed long time ago.


Add an option to add lines per module.

Fixes #1307

cc @ajeffrey
r? @fitzgen

Err, meant to cc @asajeffrey

Thanks! Here it is in action... https://github.com/asajeffrey/mozjs/commit/4516ab9abba2403e40079ccd5c688b0d43e8bb04

@bors-servo r+

:pushpin: Commit 786ed31 has been approved by fitzgen

:hourglass: Testing commit 786ed31d471ba01a30343efdc6b341959801b939 with merge 9b8ddae646f942df3d82e6e68f335c0ce106b0e9...

:sunny: Test successful - status-travis
Approved by: fitzgen
Pushing 9b8ddae646f942df3d82e6e68f335c0ce106b0e9 to master...


[bug] wrap connection listener in a try/catch

Description

Wrapping the removal of a connection listener in a try/catch statement to help reduce the number of errors that get logged when the port has already been disconnected.
Testing

[ ] Includes test.
[ ] Manually tested.

Additional notes

No additional information is given

Were you able to confirm this is working as intended?

I didn't. Is there a way to link this so I can take it for a quick spin?

@mikeyamato yeah this should be linkable just like any other npm module. yarn link in the package file should work.


Agents not working

DDPG, TD3, SAC, A2C, PPO, PPO doesn't support the crypto environment
DQN, DuelingDQN, DoubleDQN, and D3QN are working in the crypto environment but the rest doesn't work with the crypto environment, Is there a way the crypto environment can support DDPG, TD3, SAC, A2C, PPO, PPO?

Resolved


Copy Writing Proposal

Hello, i just checked your lovely project and i discovered you have some sections missing out like FAQ, about us, contact us and privacy policy. If you dont mind i will like to write that for you.
My email<EMAIL_ADDRESS>

I don't think they are needed for a project of this size, but you are welcome to start working on them anytime you want, just contact me on Discord (jakipatryk#1263) or on Steem.chat (jakipatryk).


feat: Ship CRD's manifests also in version v1beta1 for kubernetes 1.15 and older

added manifest generation to Makefile
added CRD's in version v1beta1 for kubernetes 1.15-
bumped codegen
used https://github.com/komuw/ote to mark test dependencies
updated install instruction

Manifest tested on kind with 1.15 cluster. All examples required modification of their own CRD's (to match v1beta1 version supported by 1.15) and passed afterwards.

need to adjust integration tests

need to adjust integration tests

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

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


Add shop=pasta, shop focused on selling (fresh) pasta

OSM Tag(s)
shop=pasta
How would you like this tag to see supported?
As a Preset
Label
Pasta Store
Aliases
Fresh Pasta Shop
Terms
ravioli, spaghetti
Link to OSM Wiki page
https://wiki.openstreetmap.org/wiki/Tag:shop%3Dpasta
Status of the Tag
In Use
Usage of the tag
684
Replaces other Tag?
No response
Regional Tag?
No response
Further Information
https://taginfo.openstreetmap.org/tags/shop=pasta#overview

@Robot8A , there's a draft PR #953 , and I'm wondering if you have any suggestions for a better icon?

I sent a mail to the Italian community to ask for their feedback https://lists.openstreetmap.org/pipermail/talk-it/2023-July/074863.html

Please have a look at this page: https://github.com/ideditor/schema-builder/blob/main/ICONS.md, for additional information about icons.

For me personally I am ok with the icon you chose. Let's see if someone else has an opinion about it.

@Robot8A , please feel free to review #953 . It's no longer in draft mode.

the preset was added in #953, thank you!


Remove step attributes

For each Windshaft PR, we should create a new one in Windshaft-carto and deploy them one by one (first in 1 server, and then to all servers)

ok, 🚀

@dgaubert  I have updated the way to make the acceptance and deploy:
https://github.com/CartoDB/Windshaft/issues/644#issuecomment-416475851


Assign Candidates to Panel

DELIVERABLES
1. List all candidates
For a sift it will be the all the candidates on the REVIEW state
For SELECTION DAY it will be all the candidates in the SHORTLISTED state

=============================
As an admin, at different stages of an exercise, I need to assign candidates to a particular panel so that panellists can be provided information related to the candidates they are assessing.
The number of candidates I need to add can vary across exercises and exercise stages, so there shouldn’t be a minimum or maximum number of candidates that can be assigned to each panel.
Candidates will be allocated at different stages of an exercise, so an assign button in each of the stages on the platform will allow me to click the check box and add the candidates I want to each panel listed within task area.
When assigning candidates to a panel at the eligibility sift or paper sift stages I will need name blind capability. In a name blind scenario, candidates should be identified using the characters that appear after the dash within their unique identifier. Eg, if their unique identifier is JAC00351-fiz0003  we need to use fiz0003.
When assigning candidates to a panel for selection days (shortlisted stage) I will need to use the candidates’ full names.

@lloback for your info, I reached out to our senior appointments team to confirm whether they always run name blind sifts. They confirmed that there will be some circumstances where candidates are identified by name at the sift stage. This means that the panel will need to receive the panel packs with documents that haven't been redacted.
Based on this confirmation, can we please allow an option at the sift stage to transfer candidate applications/documents and create a file on Google Drive, both with name blind capability and with the capability to display the candidates name?

@lloback for your info, I reached out to our senior appointments team to confirm whether they always run name blind sifts. They confirmed that there will be some circumstances where candidates are identified by name at the sift stage. This means that the panel will need to receive the panel packs with documents that haven't been redacted.
Based on this confirmation, can we please allow an option at the sift stage to transfer candidate applications/documents and create a file on Google Drive, both with name blind capability and with the capability to display the candidates name?

@lloback Can you please confirm that we will have the capability to remove a candidate from one panel and allocate them to another without causing any duplication of candidates across numerous panels?
An admin would need an option to select a candidate allocated to one panel and reallocate to another panel thereby removing them from the initial list under the first panel, or an ability to delete them from the originally allocated panel and then to go back into the main candidate list to allocate them to a new panel.
Raising the above off the back of a question I received at today’s digital drop-in.

@lloback Can you please confirm that we will have the capability to remove a candidate from one panel and allocate them to another without causing any duplication of candidates across numerous panels?
An admin would need an option to select a candidate allocated to one panel and reallocate to another panel thereby removing them from the initial list under the first panel, or an ability to delete them from the originally allocated panel and then to go back into the main candidate list to allocate them to a new panel.
Raising the above off the back of a question I received at today’s digital drop-in.

Yes. We willl be able to remove the candidates from ine panel and add to others

Yes. We willl be able to remove the candidates from ine panel and add to others


Migrate JS package to hybrid esm/cjs module

Description
Make the javascript package a hybrid esm/cjs module
Motivation & context
This is the continuation of a work started a few weeks ago to make cucumber-js internals compatible with both CJS and ESM packages.
Type of change

Refactoring/debt (improvement to code design or tooling without changing behaviour)

Note to contributors
I bring also two small addition to the PR:

the use of shx to make life easier for non-POSIX (aka windows) contributors
the addition of dedicated vscode eslint configuration. Per default vscode linter looks for some config in the root folder. As this is actually a monorepo, the dedicated configation tells vscode linter where to look for the javascript configuration

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[x] I have added tests to cover my changes. (test:cjs tests)
[ ] I have added an entry to the "Unreleased" section of the CHANGELOG, linking to this pull request.

I’m surprised there are no failures caused by imports that reference directories or files without an explicit .js extension. I think something is missing…

You’ve forgotten to add "type": "module" in package.json. Adding that will expose a few more things that need to be fixed, like imports.

Indeed, I was surprised too, but I was not able to find the issue myself. Thanks to spot it! Everything should be fixed now :)


DBZ-2267 Optionality ignored while converting default value

https://issues.redhat.com/browse/DBZ-2267

LGTM, thx. Just wondering whether that's worth mentioning in the docs? I don't think we state anything on default values at all at this point? WDYT?


k8s社区贡献指南

贡献快速参考https://github.com/kubernetes/community/tree/master/contributors/guide/contributor-cheatsheet
• 贡献指南: https://github.com/kubernetes/community/blob/master/contributors/guide/contributing.md

文档规范(包含markdown): https://github.com/kubernetes/community/blob/master/contributors/guide/style-guide.md

开发:
○ 开发, 编译,测试: https://github.com/kubernetes/community/blob/master/contributors/devel/development.md
○ 代码规范: https://github.com/kubernetes/community/blob/master/contributors/guide/coding-conventions.md
○ K8s API 规范: https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/api-conventions.md
○ 日志规范: https://github.com/kubernetes/community/blob/master/contributors/devel/sig-instrumentation/logging.md
○ 本地开发环境搭建: https://github.com/kubernetes/community/blob/master/contributors/devel/running-locally.md

Pull requests流程: https://github.com/kubernetes/community/blob/master/contributors/guide/pull-requests.md

Github workerflow流程: https://github.com/kubernetes/community/blob/master/contributors/guide/github-workflow.md

Review 指导原则: https://github.com/kubernetes/community/blob/master/contributors/guide/review-guidelines.md

kubeedge 贡献指南比较简单的https://github.com/kubeedge/community/blob/master/contribute.md#getting-started


Fix incorrect mock object calls

The problem with mocked objects is that any properties will return a "fake" method. So mockeddecode.called_with just gives you a made-up method that does nothing. If you assert mocked_obj.called_with('foo'), then the fake method says "ok", returns some other made-up thing, and test passes.
I don't know what the consequence of these failures are, but I want to identify them and hopefully assign someone to either fix them or do something else.

logging failures
FAILED test_app/tests/authentication/authenticator_plugins/test_radius.py::test_authenticator_plugin
E           AssertionError: expected call not found.
E           Expected: authenticate(username='user-73aa1c', password='radius_password-4aa69d')
E           Actual: authenticate(None, 'user-73aa1c', 'radius_password-4aa69d')

FAILED test_app/tests/authentication/authenticator_plugins/test_oidc.py::test_user_data
E           AssertionError: expected call not found.
E           Expected: decode('token', 'VALUE', 'VALUE', 'VALUE')
E           Actual: decode('token', key='-----BEGIN PUBLIC KEY-----\nVALUE\n-----END PUBLIC KEY-----', algorithms='VALUE', audience='VALUE')


Unable to disable modules

Reproduction steps:

config.json: { "disabled": [ "applications" ] }
Launch walker

I'm using the latest version of walker (b21d226).

fixed in latest.


[pytest] New rule idea- Prefer multiple pytest.param(..., id=...) over pytest.mark.parametrize(..., ids=...)

Hi !
This is a new rule suggestion for the pytest-style plugin.
It checks for pytest.mark.parametrize calls using the ids keyword argument and replace each set of test values with a pytest.params(.., id=...) parameter set, effectively binding the id with the set of arguments.
When using the ids=[…] syntax, each id is bound to a set of parameters by index.
This is quite error prone and makes test hard to modify (adding a test case in the middle means adding an id at the corresponding index in the ids list). Moreover, if one test case fails, it's not directly obvious which set of parameters is associated with the failing id. This issue gets worse the more test case you have in your parametrize call.
Pytest seems to agree this is not a good pattern:

In test_timedistance_v1, we specified ids as a list of strings which were used as the test IDs. These are succinct, but can be a pain to maintain.

An autofix would look like this:
Don't use:
@pytest.mark.parametrize(
    "a, b, expected",
    [
        (datetime(2001, 12, 12), datetime(2001, 12, 11), timedelta(1)),
        ... # 10 other test cases
        (datetime(2001, 12, 11), datetime(2001, 12, 12), timedelta(-1)),
    ],
    ids=["forward", ..., "backward"],
)
def test(a, b, expected):
    pass

Use instead:
@pytest.mark.parametrize(
    "a, b, expected",
    [
        pytest.param(
            datetime(2001, 12, 12), datetime(2001, 12, 11), timedelta(1), id="forward"
        ),
        ... # 10 other test cases
        pytest.param(
            datetime(2001, 12, 11), datetime(2001, 12, 12), timedelta(-1), id="backward"
        ),
    ],
)
def test(a, b, expected):
    pass

I've already implemented such fixer in python (using pyupgrade/django-upgrade building blocs) and would be very interested in porting this to ruff.
If you have in mind another MR I could take inspiration from (especially the auto-fix part) it would be awesome!
Also I'm not sure which code to use, PT028 could overlap with a potential new pytest-style rule

Thanks @UnknownPlatypus for the nice write up.
For now, we only rarely add new rules that don't exist in the upstream linter (in this case, pytest-style). Do you know if your lint has been proposed and discussed for flake8-pytest

@MichaReiser To my knowledge, it has not been discussed in flake8-pytest and I intentionally didn't open an issue there because this is the kind of rule that would be extremely cumbersome to manually fix (hence the reason I wrote an autofixer for it only).
But I can open an issue there if you want to get some more input from the pytest community that might be more present there ?

Thank you @UnknownPlatypus !! This rule sounds amazing