ch_sendraw with callback input as blob

It is currently impossible to communicate with an external application with TCP that uses a binary protocol for the output.
I can send a blob using ch_sendraw(), but the callback always gets a string as input parameter.
Since a string in vim replaces the bytes 00 (= ascii NUL) to 10 (= ascii NL), the returned binary message is not readable.
I would like to have a way to use ch_sendraw() but with a binary/blob callback input, so that I can read the output of the external application.
I would see that as an option of the callback when defining the channel or when sending the message:
call ch_sendraw(l:channel, l:message, { 'callback': l:Handler, 'callback_input': 'string (default)|blob' })
or a separate function:
call ch_sendrawblob(l:channel, l:message, { 'callback': l:Handler })

Hmm, what about adding a new channel mode (bin/binary ?) implementing a raw channel but without the string-conversion woes. The input and output would be blobs rather than strings to avoid problems with embedded NULs.

That is even better.

Hmm, what about adding a new channel mode (bin/binary ?)
implementing a raw channel but without the string-conversion woes. The
input and output would be blobs rather than strings to avoid
problems with embedded NULs.

It might be that the channel input uses something else than a blob.
E.g. sending a name or keyword, getting back a blob.  Thus setting the
callback to receive a blob is the most flexible.
--
GUARD #2:  It could be carried by an African swallow!
GUARD #1:  Oh, yeah, an African swallow maybe, but not a European swallow,
that's my point.
GUARD #2:  Oh, yeah, I agree with that...
The Quest for the Holy Grail (Monty Python)
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

It is currently impossible to communicate with an external application
with TCP that uses a binary protocol for the output.
I can send a blob using ch_sendraw(), but the callback always gets a
string as input parameter.
Since a string in vim replaces the bytes 00 (= ascii NUL) to 10 (=
ascii NL), the returned binary message is not readable.
I would like to have a way to use ch_sendraw() but with a binary/blob
callback input, so that I can read the output of the external
application.
I would see that as an option of the callback when defining the
channel or when sending the message:
call ch_sendraw(l:channel, l:message, { 'callback': l:Handler, 'callback_input': 'string (default)|blob' })
or a separate function:
call ch_sendrawblob(l:channel, l:message, { 'callback': l:Handler })

I would think blob is the only alternative that matters here.  The
alternative is to get a string with NUL bytes replaced by NL.  There
isn't really another type we can use.  When using a blob the callback
can then convert it with the existing blob functions.
So, to keep it simple, we could add a flag "callback_blob", which you
can set to true or false.  Would that work?
Implementing this is a bit of work but not very complicated.  It
probably needs to happen in channel_get_all(), changing it to set
argv[1], or an alternate function that returns a blob.
Pehaps you can suggest a way to test this, once it has been implemented.
--
I still remember when I gave up Smoking, Drinking and Sex.  It was the
most horrifying hour of my life!
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

It might be that the channel input uses something else than a blob.

Is that a problem? Strictly speaking strings are blobs without embedded NULs, sending a string or a blob wouldn't invalidate any invariant, the returned value would always be a blob given there may be NULs in the response.
The "raw" channel mode is quite asymmetrical since it allows strings and blobs to be sent (thus allowing NUL bytes) but implicitly converts NULs to NL on the receiving side, that's not so nice IMO.

So, to keep it simple, we could add a flag "callback_blob", which you can set to true or false.  Would that work?

How would that work with ch_evalexpr? The return type should also change from string to blob if the flag is true.

Concerning the testing protocol, I imagine a simple TCP server that takes any request and answers with binary big endian (for readability) (u32)42: we can send whatever blob to the opened channel the received blob should then be (dec)42<=>(hex)0000.002A, and not (hex)AAAA.AA2A ; if (bin)00 is replaced by (bin)10 then (bin)0000 becomes (bin)1010<=>(hex)A.


fix(source:complete): missing lazy-loaded items

The reason for this PR is that there may be some snippets that don't make it into the completion result.
Only some snippet items were shown on the completion menu.

As you can see the lreq is listed in LuaSnipListAvailable command result.

But it's missed from the completion result

And this is the expected result.

I'm not quite sure about this as the point of caching things was to avoid iterating over the files for every request. We can get the same behaviour by just removing caching?
We could possibly just empty the cache when everything is lazy loaded. Which should solve the issue?

Luasnip could emit some event as soon as new snippets are lazy-loaded (maybe pass the filetype(s)), listening on that and refreshing/adding the new snippets should work nicely.

Luasnip could emit some event as soon as new snippets are lazy-loaded (maybe pass the filetype(s)), listening on that and refreshing/adding the new snippets should work nicely.

Yeah, that would be great. If that's possible then we no longer have to iterate over the snippets for every request.

Check out Luasnip/load_event, LuasnipSnippetsAdded is triggered upon lazy-loading new snippets.
The filetype of the snippets that were loaded can be accessed via require("luasnip.session").latest_load_ft.

Check out Luasnip/load_event, LuasnipSnippetsAdded is triggered upon lazy-loading new snippets.
The filetype of the snippets that were loaded can be accessed via require("luasnip.session").latest_load_ft.

This looks great. @captainko do you want to update the PR if not I can make a new one

I will update the PR ASAP.

@L3MON4D3 could you create a hook instead of vim User Event. I could register a callback instead of relying on the require("luasnip.session").latest_load_ft. I have looked into from_vscode and wondering if LuaSnip could emit an event there too.

I'm not sure if the added complexity is worth it, we'd have to register and store callbacks and call all of them vs. just doautocmd and having users access that variable.
And you're right, we should only trigger the event there asload() is asynchronous, so we can't be sure that all snippets are loaded before the first call to completion (could especially happen if there's some conditional loading via packer), I'd welcome a PR to change that :D

imo we should just use the loader event and empty the relevant filetypes. It's a pretty simple change no need to over complicate things

imo we should just use the loader event and empty the relevant filetypes. It's a pretty simple change no need to over complicate things

Yeah, I have overcomplicated it. The main reason for that is because I want to build the completion items as soon as we received the filetype change events instead of wait for cmp to call the complete() method. Now I see there's no need for that :smile:

I'm not quite sure about this as the point of caching things was to avoid iterating over the snippets for every request. We can get the same behaviour by just removing caching?

I have updated the PR. Now we just remove the cache when we receive the LuasnipSnippetsAdded event.

Can you avoid adding the function to global name space and add it to the module instead.
Also please move the autocmd to existing augroup in after/plugin/cmp_luasnip.lua

Will do.

Resolved by author


Set up Tool Bar Panel expand collapse functionality for lab book stuff

the tool bar panel is getting too big, need to add feature to make parts collapsible.
Separate parts: Tool bar, meta data adder? able to minimise both?

Closed in latest PR


there is no Settings > Partner Settings > Programs > Get Started.

it brings me to https://partner.microsoft.com/en-US/ which is more of a marketing page for what partners can do. I'm always so confused with what I'm looking at when inside or linking to the partner network and/or dashboard.

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

ID: e16147fb-8da4-6c41-be0f-b2856043c7c9
Version Independent ID: dab8f3c3-c3ff-883b-406d-405701a53224
Content: Create a Commercial Marketplace account in Partner Center
Content Source: articles/marketplace/partner-center-portal/create-account.md
Service: marketplace
Sub-service: partnercenter-marketplace-publisher
GitHub Login: @qianw211
Microsoft Alias: dsindona

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

adding content owner, @qianw211 to review the doc enhancement request.

@jimbarrett33  Please use the following link when signing in to partner center:
https://partner.microsoft.com/dashboard/account/v3/enrollment/introduction/partnership.
We appreciate your feedback, and we will be updating the documentation shortly.  Thanks!
#please-close


Testcafe integration with Create-react-app and Electron not firing correctly.

What is your Test Scenario?

I want to run a simple e2e test on this test react-electron application
What is the Current behavior?

I can get TestCafe to launch but depending on .testcafe-electron-rc config, it shows a blank page not loading the bundler or shows the application and hangs acting as if it didnt load.
What is the Expected behavior?

TestCafe to complete tests and interact with the application in an electron window.
What is your web application and your TestCafe test code?

https://github.com/calebswank11/testcafe-electron
Steps to Reproduce:

Create React App "app name"
Change start/dev scripts via tutorial
https://medium.freecodecamp.org/building-an-electron-application-with-create-react-app-97945861647c
implement testcafe and "testcafe-browser-provider-electron": "^0.0.8"
write sample test in ./src/e2e/e2etest.test.js (redundant naming I know)
run yarn testcafe which equals "testcafe": "testcafe \"electron:.\" \"./src/e2e/**/*.js\""

.testcafe-electron-rc
for white page that doesnt load bundle.js
{
  "mainWindowUrl": "./build/index.html",
  "appPath": "./"
}

for loading the app but not responding to events and recognizing it.
{
  "mainWindowUrl": "./public/index.html",
  "appPath": "./"
}

Your Environment details:

testcafe version:                   ^1.1.0
node.js version:                    8.11.3 => a littl old, but works with testcafe 1.1.0 on other projects
command-line arguments:  see package.json or above for list
browser name and version: Chrome Version 72.0.3626.119 (Official Build) (64-bit)
platform and version:          macOS Mojave 10.14

Notes:
I believe it to probably be a config issue, but I've tried alot of different things and have searched the vastness of the interwebs for a solution.
reference: https://stackoverflow.com/questions/54988921/test-cafe-electron-create-react-app-not-loading-bundle-js-properly

@AndreyBelym Thank you very much for taking the time to look at this. I knew it had to be something simple like this, just couldn't quite piece it together. Duley noted about the deprecated .test. suffix as well. I have got it working now.

You are always welcome 😄


Add the ability to inject SSH options

Small pull request that allows the user to inject some additional SSH options with TUNNELER_SSH_OPTIONS in their .env file.

Thanks!


TypeChain ver 1.0.0

Goals:

[x] complete ABI parsing to typesafe structures

[x] use a discriminative union for EvmTypes
[x] support overrides

[x] split targets to separate packages
[x] support custom targets
[x] add prebublish hooks for subpackages
[x] better error messages on missing target packages
[ ] readme
[ ] readme for each package
[x] revive truffle example

Related https://github.com/ethereum-ts/TypeChain/issues/150
Closes https://github.com/ethereum-ts/TypeChain/issues/65
Closes https://github.com/ethereum-ts/TypeChain/issues/106

Pull Request Test Coverage Report for Build 262

107 of 121 (88.43%)  changed or added relevant lines in 6 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.7%) to 89.66%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/targets/ethers/generation.ts
15
16
93.75%

lib/targets/legacy/generation.ts
26
27
96.3%

lib/parser/parseEvmType.ts
29
31
93.55%

lib/targets/truffle/generation.ts
8
10
80.0%

lib/parser/abiParser.ts
18
26
69.23%

Files with Coverage Reduction
New Missed Lines
%

lib/targets/truffle/generation.ts
1
82.72%

lib/parser/abiParser.ts
2
81.48%

Totals

Change from base Build 251:
-0.7%

Covered Lines:
429

Relevant Lines:
465

💛  - Coveralls

I'll try to test the new version with our project (on ethers) later today, to see if there are no regressions.

Just tested the newest beta version with ethers target, works fine!
Two issues though:

At first, I didn't install the typechain-target-ethers package, and the error message is rather cryptic. If I didn't read the code before, I would have no clue I have to install it. If target is one of the "official" targets, the error should say to install the package, and for other targets, say something about invalid target package (or a link to custom targets FAQ in the future).

<EMAIL_ADDRESS>npx typechain --target ethers --outDir $TYPECHAIN_OUT_DIR "build/contracts/*.json"
Error occured:  module.default is not a constructor

I've reviewed changes in the generated files. The big difference is that the functions are now also available on contract object's toplevel, which is correct with how ethers generates them. The difference I'm curious about is that methods for contract variable getters -- they now also have the transactionOverrides parameter. It's not necessary there, maybe even invalid -- constant functions are not called as transactions. Example:

solidity:

    bool public isActive = true;
    address public owner;

typechain 0.3.19:

    isActive(): Promise<boolean>;
    owner(): Promise<string>;

typechain 1.0.0-beta.2 + typechain-target-ethers 1.0.0-beta.1:

    isActive(overrides?: TransactionOverrides): Promise<boolean>;
    owner(overrides?: TransactionOverrides): Promise<string>;

ad 2. it can be related to a bug in isConstantFn. Thanks for bringing this up!

I just fixed ad2 👍, I updated todo in the PR description.

ad 1, oh actually the error message was supposed to be much more civilized but you managed to hit another bug 😆 Fixing it now. It will say something like:
$ npx typechain --target x
Error occured:  Couldnt find x. Tried loading: typechain-target-x, /Users/krzkaczor/Workspace/ethereum-ts/TypeChain/x.
Perhaps you forgot to install typechain-target-x?

@krzkaczor - I got the below error message when I haven't installed web3-1.0.0 target.
➜  ETHSurvey git:(develop) ✗ npm run typechain

><EMAIL_ADDRESS>typechain ~/ETHSurvey
> typechain --target=web3-1.0.0 --outDir ./src/types './build/contracts/*.json'

Error occured:  Couldnt find web3-1.0.0. Tried loading: typechain-target-web3-1.0.0, ~/ETHSurvey/web3-1.0.0.
Perhaps you forgot to install typechain-target-web3-1.0.0?

When I tried to do npm install typechain-target-web3-1.0.0, it threw a missing package error. As the package available on npm is under name typechain-target-web3-1. We need to fix the error message being displayed as well as probably change the targets now to web3-v1 and web3-v2 rather than 1.0.0 and 2.0.0. Let me know your thoughts. Thanks!

Hey!
So the name of the target changed from web3-1.0.0 to web3-1. TBH I don't see anything particularly wrong with this naming scheme ie. web3-1 and web3-2 but maybe adding v would make it clearer. @quezak thoughts?

yeah, I think web3-v1 would be clearer. web3-1 souds like it's a web package with version 3.1 :D web3-1.0.0 is also confusing, as it indeed looks like a version specification for web3, and not related to typechain.

@krzkaczor - Let me know if you need any help here in wrapping up this PR. I'll get started with the web3-v2 target once this is merged 🙂

@krzkaczor heads up -- I've renamed the web3-v1 integration test dir too, plus some other occurences you missed.


winpr: Convert time zone generator programs to C# scripts

Converts the C# programs (.cs modules) used to generate the time zone structures in winpr/libwinpr/timezone/timezone.c to C# scripts (.csx files). This change allows the files to be run directly using the C# interpreter (csi.exe) without having to stand up an MSBuild or Visual Studio project.
As a convenience to the developer running the script, the resulting output .txt files are .gitignored.
As an added convenience, the WindowsZones.csx script downloads the CLDR timezone data directly from unicode.org instead of relying on a previously downloaded windowsZones.txt file.

@freerdp-bot test

Looks like the scripts do what they are supposed to do.
@rjcorrig One question though, csi.exe is available with which versions of visual studio?

@akallabeth Should be available since Visual Studio 2015 Update 1

@bmiklautz @hardening Any objections to merging?


Issue converting wdl files with input{} in workflow {}

I have a wdl file from Bammetrics that is unable to convert to cwl and throws an error wdl.parser.SyntaxError: Unexpected symbol (line 26, col 5) when parsing parse__gen10.  Expected rbrace, got <string:26:5 input "aW5wdXQ=">.
This occurs with any WDL file having a format:
workflow xyz  {     input {abc} }
Since $input is not considered a $wf_body_element. I want to make changes to the parser.py but am unable to generate it using Hermes Parser Generator since I don't have the original grammar file to modify and input into the tool. May I get assistance on where to find it?

Hello @Nitromon , it looks like the parser is no longer supported: https://github.com/broadinstitute/pywdl which contains a dead link that I think is supposed to go to https://github.com/openwdl/wdl/tree/master/versions/draft-2/parsers/python (but that is obviously out of date as you have a version 1 and not a WDL draft-2 document)
Perhaps this project could be updated to use https://github.com/chanzuckerberg/miniwdl which is also written in Python
There appears to be a v1.0 Hermes Grammer at https://github.com/openwdl/wdl/blob/master/versions/1.0/parsers/hermes/grammar.hgr

Here is the corresponding auto-generated WDL v1.0 parser for Python: https://github.com/openwdl/wdl/tree/master/versions/1.0/parsers/hermes/python


PyPI malware

https://blog.sonatype.com/python-packages-upload-your-aws-keys-env-vars-secrets-to-web

Resolved with f3e51f0


Attempt to invoke virtual method 'void android.view.ViewGroup.addView(android.view.View)'

Describe the bug
Exception java.lang.NullPointerException: Attempt to invoke virtual method 'void android.view.ViewGroup.addView(android.view.View)' on a null object reference
  at com.getcapacitor.community.admob.banner.BannerExecutor.lambda$createNewAdView$4 (BannerExecutor.java:290)
  at com.getcapacitor.community.admob.banner.BannerExecutor.$r8$lambda$1EMVLVvjFaJLwYRq7cDZGsVkudM
  at com.getcapacitor.community.admob.banner.BannerExecutor$$ExternalSyntheticLambda0.run
  at android.os.Handler.handleCallback (Handler.java:942)
  at android.os.Handler.dispatchMessage (Handler.java:99)
  at android.os.Looper.loopOnce (Looper.java:226)
  at android.os.Looper.loop (Looper.java:313)
  at android.app.ActivityThread.main (ActivityThread.java:8757)
  at java.lang.reflect.Method.invoke
  at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run (RuntimeInit.java:571)
  at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:1067)

To Reproduce
Steps to reproduce the behavior:

Open the App
Crash the App
Close automatically the App

Expected behavior
The App has to open normally and to show ads
The App is crashing on these devices
Device List on Excel-Data Admob_Capacitor_Crash_DeviceList.xlsx
Tools
React 18.2.0, Capacitor 5 & Admob Capacitor 5

any update here?

Expecting binder but got null!
endAllActiveAnimators on 0x7911aed94200 (RippleDrawable) with handle 0x7910bedd2840
To native (Capacitor plugin): callbackId: 129802094, pluginId: AdMob, methodName: addListener
callback: 129802094, pluginId: AdMob, methodName: addListener, methodData: {"eventName":"bannerAdLoaded"}
To native (Capacitor plugin): callbackId: 129802095, pluginId: AdMob, methodName: showBanner
callback: 129802095, pluginId: AdMob, methodName: showBanner, methodData: {"adId":"","isTesting":true,"adSize":"ADAPTIVE_BANNER","position":"BOTTOM_CENT
Ad ID:
Considering local module com.google.android.gms.ads.dynamite:0 and remote module com.google.android.gms.ads.dynamite:240304704
Selected remote version of com.google.android.gms.ads.dynamite, version >= 240304704
Instantiated singleton DynamitePackage.
Instantiating com.google.android.gms.ads.ChimeraAdManagerCreatorImpl
Use RequestConfiguration.Builder().setTestDeviceIds(Arrays.asList("B3EEABB8EE11C2BE770B684D95219ECB")) to get test ads on this device.
Unable to open '/data/user/0/io.ionic.starter/cache/1700159444520.dm': No such file or directory
Waiting for a blocking GC ClassLinker
Failed to measure fs-verity, errno 1: /data/app/~~RjlomzNY6NLNdk-hW9F8uQ==/io.ionic.starter-juGrsCLM1higoiTWvs-GtQ==/base.apk
Invoke Firebase method getInstance error.
The Google Mobile Ads SDK will not integrate with Firebase. Admob/Firebase integration requires the latest Firebase SDK jar, but Firebase SDK is either missing or out of date
Dynamic lookup for intent failed for action: com.google.android.gms.ads.service.CACHE
Shutting down VM
FATAL EXCEPTION: main
Process: io.ionic.starter, PID: 2098
java.lang.NullPointerException: Attempt to invoke virtual method 'void android.view.ViewGroup.addView(android.view.View)' on a null object reference
at com.getcapacitor.community.admob.banner.BannerExecutor.lambda$createNewAdView$4(BannerExecutor.java:290)
at com.getcapacitor.community.admob.banner.BannerExecutor.$r8$lambda$1EMVLVvjFaJLwYRq7cDZGsVkudM(Unknown Source:0)
at com.getcapacitor.community.admob.banner.BannerExecutor$$ExternalSyntheticLambda0.run(Unknown Source:4)
at android.os.Handler.handleCallback(Handler.java:942)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loopOnce(Looper.java:201)
at android.os.Looper.loop(Looper.java:288)
at android.app.ActivityThread.main(ActivityThread.java:7872)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:548)

any update?

@bayonmbayo this problem happens because you are calling show before prepare.
Please check the documentation. There are other several issues with this explanation.

I solved by adding some line in the java "BannerExecutor.java", it is about to handle "NullException" coming from Android-Layout
I upload here the updated file
BannerExecutor.java.txt

The extension of the file finish with ".txt", delete it after uploading
-> It has to end with ".java" like this "BannerExecutor.java"

Then replace the old one with the new file

After updating the file "BannerExecutor.java", ANR and Crash was reduced drastically in the new version of the App 😎
Tools
React 18.2.0, Capacitor 6 & Admob Capacitor 6


Dashboard: Facilities Overcrowding Rates

Business Context:
I want to visualize how overcrowded the different Hawaii prison system facilities are.
In Scope

Create the graph: Create a bar chart with the different facilities occupation rate and capacity.

When hovering over a bar a tooltip will display the full facility name, the occupation percentage and the occupation / capacity numbers.

Plotly link: https://plotly.com/javascript/bar-charts/

Note: Capacity numbers are on the [capacity] tab on this sheet.. Those are static numbers that won't change.

The overcrowding numbers would be coming from the raw population data sheet. The population data is the roster that we update often.

Overcrowding should be calculated excluding the furlough population.

a. Excluding furlough when calculating capacity: We are excluding the sublocations on this list, meaning that we should use the "final capacity" column on this sheet as the reference number for each facility capacity:
Furlough facilities:
HCCC - HALE NANI
MCCC- Dorm 5
OCCC- LAUMAKA
OCCC - MODULE 20
WCCC- MAUNAWILI D
OCCC - XFUR
b. Excluding furlough facilities when calculating current occupation: the facilities / sublocations above should be excluded when calculating current occupation, meaning PiC in those facilities should be excluded from the count of total residents for the respective facility
HCCC - HALE NANI
MCCC- Dorm 5
OCCC- LAUMAKA
OCCC - MODULE 20
WCCC- MAUNAWILI D
OCCC - XFUR
KCCC - XFUR
KCCC - Cabin B
WCCC - XFUR
example of roster (this one is outdated, please use the one in the backend.
Open question
Add a note regarding facilities exclusion on the mockups @Concepcion-c
Acceptance Criteria

Scenario:
Given I go to /public-data using mobile, tablet or desktop
When I select the Facilities Overcrowding Rates graph on the list of graphs on the data dashboard
Then I see the Facilities Overcrowding Rates graph

Scenario:
Given I go to /public-data using mobile, tablet or desktop
When I select the Facilities Overcrowding Rates graph on the list of graphs on the data dashboard
AND I see the furlough facilities numbers excluded from capacity and current occupation numbers,

Mockups
Link to the figma mockups
https://www.figma.com/file/SHMDMgzr7damRoT12ToEz5/Hawaii-CSOC-Advocacy--2?type=design&node-id=428%3A3237&mode=design&t=UZP5VFPI8pVjkP6G-1
You can find Desktop/Tablet/Mobile View in the Dev Handoff Section
Checklist (should be reviewed before grooming as the BA or Pair are writing the card—NOT reviewed for the first time at grooming)

[ ] Ensure to include all necessary mockups to this card.
[ ] Discuss what success/failure user feedback messages would be present, if any occur (front end/back end).
[ ] Discuss Feature Flag: Where could a development flag go to flag off incomplete work? Do we want to keep the flag long-term?
[ ] If this card is part of a sequence of cards, discuss if this card should be blocked until the testing of previous cards in the sequence have been finished.
[ ] Have we added edge cases as ACs that we're considered from a QA perspective
[ ] Should the new functionalities be offered only to users with certain permissions?
[ ] How should this functionality differ between different orgs, and if it does differ, what is the hinge point we should use for that functionality?

Notes:
Media-Query Breakpoints
mobile: 360px max-width
tablet: 834px max-width
desktop: 1920px max-width
-> To be done in the future**
Changes for the breakdown between cells and dorms reflected on the graph

Updated data from Cara:

Facility
Full Capacity
Furlough Capacity(Furlough unit)
Capacity minus Furlough

HCCC (Hawaii)
226
100(Hale Nani)
126

OCCC (Oahu)
954
176(LAUMAKA = 96, Module 20 = 80)
778

MCCC (Maui)
301
32(Dorm 4 = 16, Dorm 5 = 16)
269

KCCC (Kauai)
128
0(Some of the people living in the cabins are in furlough. The Capacity for cabins is 0 because the three cabins do not meet fire repression standards or have a required second exit)
128

WCCC (Women’s)
260
20(Maunawili D)
240

KCF (Kulani)
160
–
–

WCF (Waiawa)
334
–
–

HCF (Halawa)
1124
–
–

Saguaro(private prison in AZ)
Do not have this.  They are under capacity.
–
–

Updated capacity and occupancy data: https://docs.google.com/spreadsheets/d/1mmO2Ex-IfHh4xLG1iDSvzw-f-NhCvPQ6Gt_A8YQ_qsU/edit#gid=0


how to express "Fmy_time < NOW() - INTERVAL 5 SECOND"

hi,
I want to express this sql, but i don't know how to express "Fmy_time < NOW() - INTERVAL 5 SECOND",
It should be noted that 5 here is a runtime variable
UPDATE my_table
SET Fmy_time = NOW()
WHERE Fmy_time < NOW() - INTERVAL 5 SECOND;

I think that sqlpp does not support SQL functions like now(), so you can either use custom queries or perform the date/time computations in your C++ code and pass the computed values to sqlpp.
This is an example of how I do the interval checks in my code
db_model::records records{};
const auto now = ::date::floor<::std::chrono::microseconds>(std::chrono::system_clock::now());
const auto rows = g_db(
	select(records.record_id)
	.from(records)
	.where(records.record_ts < now - std::chrono::seconds{5})
);

considering this answered.


I have a bit of problem here

Can anybody help me with this? I'm pretty new to programming and i would love to get this up and running. I'm trying to make it work on my mac :-)

You need to run npm install inside of the PokemonGo-Bot-Desktop directory

It's not working quite properly right now. Is it the servers?

Yeah, that means the servers are busy. They have been pretty bad lately. It makes testing really difficult

oh okay :-) but thanks man!

I have another issue with another app i'm trying to run. Do you think you can help?

What does this mean?

i have same error

This is another sign that the servers are busy

I have this error  : Python: [2016-07-31 14:57:40] [#]
Python: Traceback (most recent call last):
File "./pokecli.py", line 220, in
Python:     main()
File "./pokecli.py", line 206, in main
bot.start()
File "/Users/fruysshaert/ownCloud/pokemongo/PokemonGo-Bot-Desktop/gofbot/pokemongo_bot/init.py", line 32, in start
Python:     self.stepper = Stepper(self)
File "/Users/fruysshaert/ownCloud/pokemongo/PokemonGo-Bot-Desktop/gofbot/pokemongo_bot/stepper.py", line 31, in init
Python:     self.steplimit2 = self.steplimit**2
TypeError: unsupported operand type(s) for ** or pow(): 'NoneType' and 'int'

@JVenberg Why won't it import the gpxpy file?
Python:     from follow_path import FollowPath
File "/Users/kennethbolo/PokemonGo-Bot-Desktop/gofbot/pokemongo_bot/cell_workers/follow_path.py", line 3, in
import gpxpy
ImportError: No module named gpxpy


[epics-db] support macro without quotes in record field value

lines:
record(longin, "${P=}${FCT=Core}Dimension") {
    field(HOPR, ${INF=65535})
}

tree-sitter message:
$ tree-sitter parse --stat  $file
(source_file [0, 0] - [2, 1]
  (record_instance [0, 0] - [2, 1]
    type: (record_type [0, 7] - [0, 13])
    name: (record_name [0, 15] - [0, 42]
      (macro_expansion [0, 16] - [0, 21])
      (macro_expansion [0, 21] - [0, 32])
      (string_text_fragment [0, 32] - [0, 41]))
    (field [1, 4] - [1, 29]
      name: (field_name [1, 10] - [1, 14])
      (macro_expansion [1, 16] - [1, 28])
      value: (json_value [1, 28] - [1, 28]))))
gscan_core.template	0 ms	(MISSING "null" [1, 28] - [1, 28])
Total parses: 1; successful parses: 0; failed parses: 1; success percentage: 0.00%

fix:
record(longin, "${P=}${FCT=Core}Dimension") {
    field(HOPR, "${INF=65535}")
}

We already talk about this case. I know it is complicated to solve. I just note it here for next case.

Same as #4, #5, and #6


EOmaps v6.0.1

A bugfix release for EOmaps v6.0

❗ Make sure to check the release-notes for EOmaps v6.0

🌳 New

m.add_colorbar(extend=...) now has an explicit kwarg to set how extension-arrows should be treated

🔨 Fixes

fix infinite recursions with compass objects on zoom events
fix updating scalebar background patches when moving the scalebar
fix recursions in m.on_layer_activation() callbacks
fix inherited  colorbar axis positions if extension arrows are present
fix performance issues with the companion-widget for large number of layers
fix sizing of TabWidget
make sure a draw-event is triggered if limits are set on data-plot
make sure colorbars trigger layer-re-draws
fix axis limits when updating layers for existing layer-slider widgets

Codecov Report

Merging #141 (e573d0d) into master (ff27967) will decrease coverage by 0.10%.
The diff coverage is 72.22%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master     #141      +/-   ##
==========================================
- Coverage   77.03%   76.94%   -0.10%
==========================================
  Files          19       19
  Lines        8070     8098      +28
==========================================
+ Hits         6217     6231      +14
- Misses       1853     1867      +14

Impacted Files
Coverage Δ

eomaps/utilities.py
60.47% <22.22%> (-1.57%)
:arrow_down:

eomaps/helpers.py
77.38% <68.00%> (+0.04%)
:arrow_up:

eomaps/colorbar.py
75.45% <73.68%> (-1.47%)
:arrow_down:

eomaps/eomaps.py
74.73% <77.77%> (-0.06%)
:arrow_down:

eomaps/scalebar.py
79.68% <84.21%> (+0.32%)
:arrow_up:

eomaps/_data_manager.py
79.00% <100.00%> (+0.61%)
:arrow_up:

eomaps/_webmap.py
81.88% <100.00%> (ø)

... and 1 more

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


Make file names of documents / images customizable : max_length

Issue Summary
Adding files into a collection, I'm stuck with an exception about the length of file names :
(I'm having a long path from /home to media directory on my dev platform)
SuspiciousFileOperation: Storage can not find an available filename for "<long long path>/<file name>". Please make sure that the corresponding file field allows sufficient "max_length".

You could introduce a parameter in settings to avoid this:
WAGTAIL_FILE_NAME_MAX_LENGTH = 255 in settings for example
Steps to Reproduce

Having a long path to the MEDIA_ROOT directory, or really long file names (for now, max_length is 100 chars)
Try to add files into a collection, through UI or django command

Technical details

Python version: Python 2.7.12+
Django version: Django==1.11.1
Wagtail version: 1.10
Browser version: (not relevant)

I've written a monkey patch:
class OfficeDocument(AbstractDocument):
    ...

# Hook to update the max_length of the
def _update_file_max_length(klass, field_name='file'):
    for item in klass._meta.fields:
        if item.name == field_name:
            item.max_length = 255
            return

_update_file_max_length(OfficeDocument)

...

@frague59 thanks for this detail, it looks like the filename needs to be 100 characters max due to backwards compatibility with Django.
https://code.djangoproject.com/ticket/9893
Also see
https://github.com/wagtail/wagtail/blob/2231906ebe2cda0f0634c565adf4468c3e48c6a2/wagtail/wagtailimages/models.py#L121


Python 3.11/3.12 test suite runs slower than 3.9 on GH Actions

They are running the same tests, so why is it so much slower?
We need to find out which tests are taking so long on 3.11 vs 3.9.

Something upstream with 3.12. We'll revisit.


Add metal3 example

Assumptions:

Two networks, provisioning and public network. Provision network is internal and use for bare metal provisioning. Public network can access all the public endpoints.
Public network is VLAN.

What this PR does / why we need it:

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Checklist:

[ ] squashed commits into logical changes
[ ] includes documentation
[ ] adds unit tests
[ ] adds or updates e2e tests

Thanks @guangyee , this looks quite good!


HalfDay with border style on first and/or last day

Is there a way to break line to 50% if the last or the first day of event is a half day like this :
I add a parameter like
first-day : "morning", "afternoon", "all_day";
last-day : "morning", "afternoon", "all_day";

This option already exists, but only for the "background" style. See the startHalfDay and endHalfDay options of the data source:

I don't think that there is a way to make what you want using CSS. If you find a way to do this, please tell me !


Investigate usage and eliminate MomentJS from codebase

By far the largest asset is our compiled and compressed javascript file all.js
Part of that is jQuery, which is large and difficult to eliminate at this time (because of Yii's ActiveForm as well as Bootstrap's plugins).
MomentJS is also very large, and we only use it in two (I think?) places. We may be able to do the same functionality in a different way.
Eliminating Moment would impact the size of this site in a meaningful way and increase the site's speed for users across the board.

Not possible right now. ChartJS timeseries charts have a dependency on them. Turns out it's easier (and cuts down size even more) to eliminate jQueryUI (see issue #130)


centralize all converters in a single yml file to avoid duplicates

we should have only a single yml file that lists ALL converters and render different tables based on their datatypes, because currently we have duplicates of some converters (BIDSme is another one)
Originally posted by @Remi-Gau in https://github.com/bids-standard/bids-website/issues/441#issuecomment-2189004528

@iamdamion pinging you on this one in case you got time.

status: in progress, no blockers


fix: update goss to be more explicit with platform in docker tests

Does this also happen to unblock updating the base AMI (which would fix #1387)?


(#15 一部) buttonを隠すときにもアニメーションを追加

@hrl7 よろしくお願いします
#15

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


feat: improve runtime bundle size

Optimizes TypeScript files in src/runtime to produce smaller JavaScript files

📦 Published PR as canary version: 0.9.1--canary.25.c02907e.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

:rocket: PR was released in v0.10.0 :rocket:


fix: search BooleanColumn by true or also and add search by boolean column label

This fixed issue #141 and also adds little extra functionality, as such: if you have multiple boolean columns (most typical example is the users table: is_staff, is_active, is_superuser etc.) searching 'true' will not help much since most columns will have it. Instead you would like to be able to just filter "active" users, or "staff" users.

Commit requested to be pulled in this is 6b53bfd


With the RealmManager, We must not close database manually!

In the RealmManager, we have this:
public void closeLocalInstance() {
        checkDefaultConfiguration();
        Realm realm = localRealms.get();
        if(realm == null || realm.isClosed()) {
            throw new IllegalStateException(
                    "Cannot close a Realm that is not open.");
        }
        realm.close();
        // noinspection ConstantConditions
        if(Realm.getLocalInstanceCount(Realm.getDefaultConfiguration()) <= 0) {
            localRealms.set(null);
        }
    }

So, when we use try-with-resource, or try-catch-finally (Or in totally, close the opened realm manually), we get the exception:

Cannot close a Realm that is not open.

Do you think it was better if you removed throw new IllegalStateException, and just ignore it?
With the current approach, it's a limitation on try-with-resource usage.
Or totally, you say it's not necessary to use try-catch-finally/try-with-resource in our code, like:
Realm realm = null;
    try {
        realm = mRealmManager.getLocalInstance();
        realm.executeTransaction(new Realm.Transaction() {
            @Override
            public void execute(Realm realm) {
                DBProduct dbProduct = realm.createObject(DBProduct.class, UUID.randomUUID().toString());
                dbProduct.product_id = Product_id;
                if(dbProduct.totalQuantity.get() == null){
                    dbProduct.totalQuantity.set(0);
                    dbProduct.totalQuantity.increment(countOrder);
                }
                else
                    dbProduct.totalQuantity.increment(countOrder);

                DBBasket dbBasket = realm.createObject(DBBasket.class, UUID.randomUUID().toString());
                dbBasket.dbProducts.add(dbProduct);
                dbBasket.totalProducts = numCart;
            }
        });
    } finally {
        if(realm != null) {
            realm.close(); //<- With the RealmManager, we must not use this anymore?
        }
    }

        realm.close(); //<- With the RealmManager, we must not use this anymore?

Correct.
Unfortunately you can't try-with-resource on the openLocalInstance() call.


Update README.md to show 4.0.0 API

The documentation demonstrates how to use the 2.0.0 API (and states so), but does not demonstrate how the 4.0.0 API works.

Ah, I see that the API is the same. I'll submit a PR soon so someone reading it isn't puzzled as to why the documentation only covers 2.0.

I think you're mistaking this library for react-router. Router is at 2.0. This library (react-router-redux) is at 4.0.

Wow, that's embarrassing :flushed: my mistake! Thanks for pointing it out to me.


Investigate data-oriented design techniques

This investigates data-oriented design to improve iteration performance of the tree. This required modifying the tree trait to allow for the disjointed nature of DoD and I have adjusted the names of methods to match existing std::collections structures. I will be comparing the performance of these approaches later this week.
WIP

Using a linked list might be a bit more intuitive, the left child can be retrieved by pop_front() and right child from pop_back(). This would also allow for non-binary trees

True, although note that in the current setup, we are able to pre-allocate all of the nodes by passing a capacity to the nodes vector. E.g. estimate the length of the vector as length_of_file / 16.
Also, the DfsTree is a non-binary tree. Each node holds the indexes to its optional left and right siblings.
Here is pseudocode for getting the left child:
leftchild(N):
  if nodes[N+1].siblings.left != null:
    return nodes[N+1]
  else:
    return null

And the right child:
rightchild(N):
  C = N+1
  while nodes[C] != null and nodes[C].siblings.right != null:
    nodes[C] = nodes[C].siblings.right
  return nodes[C]

Using DoD in the way I suggested leads to performance problems if we wanted to mutate the tree. I will close this because I can't think of better way to do this.


Train 134

High Level Description
Train 134
Related Issues
#1252
#1264
#1292
#1291
#1288
#1282
#1280
#1276
#1274
#1094
Checklist for all PR's

[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[x] Test Results: [link to CI job test results for component]
[x] Code Coverage (if available): [link to code coverage report]

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and Review Process
What is the review process and when will my changes land?
All PRs require 2 approvals using github's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

continuous-integration/jenkins/pr-head  waits for  https://github.com/dcos/dcos/pull/1291 to be merged. Safe to ignore for now/checked manually.

recreating the train, new pr


chore: remove unused module

Removing top level CDK module
Adjusting README
Adding CLI example

Fixes #20

Codecov Report

Merging #22 (6c2dfa5) into main (e8bc0db) will decrease coverage by 0.27%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main      #22      +/-   ##
============================================
- Coverage     87.88%   87.61%   -0.28%
+ Complexity      335      333       -2
============================================
  Files            54       52       -2
  Lines          1593     1437     -156
  Branches         53       53
============================================
- Hits           1400     1259     -141
+ Misses          169      154      -15
  Partials         24       24

Impacted Files
Coverage Δ

...device/pool/ddb/LockingMechanismDynamoDBModel.java
84.05% <0.00%> (+4.34%)
:arrow_up:

...li/device/pool/s3/ContentTransferAgentS3Model.java
85.45% <0.00%> (+7.27%)
:arrow_up:

...ce/pool/s3/ContentTransferAgentFactoryS3Model.java
91.66% <0.00%> (+16.66%)
:arrow_up:

Continue to review full report at Codecov.

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


Add backup functionality

I needed a way to make (part of) my clipboard permanent, so I added a rudimentary backup function to clipctl and a handler for the CM_BACKUP environmental variable (full path to backup directory) in clipmenud.

Hi! Sorry, but I think I'm unlikely to merge this -- it seems pretty niche.
clipmenu-agnostically, it should be possible to do something like the following:
for file in some_dir/*; do
    xclip -sel clip < "$file"

    # clipboard notifications in X are async, so wait a bit to avoid drops
    sleep 1
done

Thank you for the input!


Database Relation

There is no database Present in the Project Directory. So I cant understand the Relation of tables in according to the Given Queries Please Help Me Out from this

This was made specifically for our organisation, but you can tweak the database and the queries as per your need.
portal.xml.zip
this is the portal database that you can import from phpmyadmin.

I am sorry we cannot share as its private stuff.

I need this to understand my personal Use.. But as You mention its Private So its Ok Thanks for response.. Best of Luck for Project.
Have a Blessed Day


can't pickle Epochs with preload=False

To replicate:
from pathlib import Path
import mne
import pickle

sample_dir = Path(mne.datasets.sample.data_path())
sample_fname = sample_dir / 'MEG' / 'sample' / 'sample_audvis_raw.fif'

raw = mne.io.read_raw_fif(sample_fname)
raw.crop(tmax=60)

events = mne.find_events(raw, stim_channel='STI 014')
event_id = {'auditory/left': 1}
epochs = mne.Epochs(raw, events=events, event_id=event_id,
                    tmin=-0.2, tmax=0.5, baseline=(None, 0),
                    preload=True)

epochs.save('toto-epo.fif', overwrite=True)
epochs = mne.read_epochs('toto-epo.fif', preload=False)

# del epochs._raw[0].fid  # uncomment this line to fix it

fname = "toto.pkl"
with open(fname, 'wb') as fid:
    pickle.dump(epochs, fid)

it's due to the fid in epochs._raw[0].fid
@larsoner thoughts?
reported by @gemeinl in braindecode

I can't 100% remember why we store the fid instead of just opening the file(s) when we need to. One reason is that we support reading file-like objects, at least for FIFF, and probably soon will for other formats (#8976). But we had it that way before we added that support, so there is probably some good way to keep the code that way.
I'm not sure how to handle pickling of file pointers. One workaround would probably be to do read_raw_fif(..., preload='memmap') or so. @gemini would using memmapping like this work for your use case?


Configure the production site to run over SSL

Probably use lets encrypt to achieve this? Or check if we can reuse what ci.ocamllabs.io uses.

Yes, they also use LetsEncrypt! 😄

Implemented by @gs0510 and @art-w


#28 added slf4j wrappers for Takes,Back ,Front ,Take,Pass

Thanks, let me find someone who can review this pull request

@longtimeago this pull request is for you, please review

@dmzaytsev Thanks for the PR.. Basically it's good but it's TOO big. It's really hard to review such complex PRs. I don't think you've spent 30 min on it. Next time, please, implement something simple to start and create todos in the code.
Also, see several remarks in the code above

@longtimeago Thanks for review, you are right I should been split the task.
I asked some questions above

@longtimeago review again please

@dmzaytsev thank you!

@rultor merge please

@longtimeago Thanks for your request. @yegor256 Please confirm this.

@dmzaytsev the first question is why did you decided to implement all of them in one go? You could have done just one and create puzzles for all others. You obviously spent too much time on this ticket and created too much code, which is difficult to review and modify.
Second, I don't like the idea behind LogWrap. In general, using inheritance in order to share some functionality between objects is an anti-pattern. Instead, use encapsulation. Make an instance of LogWrap (I would rename it to Target.Slf4j) and encapsulate it in every class. This will also make testing way easier, since you will be able to mock this Target interface and intercept all logging calls.
Third, try to run pdd -f /dev/null, you will see issues with your PDD puzzles.

@longtimeago @yegor256 Let's try again
I'll create stubs for BkLogged,FtLogged,PsLogged likeTsLogged.

@yegor256 I fixed your comments. Review the changes please

@dmzaytsev see some comments above

@dmzaytsev btw, I think LogWrap should be removed from the branch, right?

I fix all comments besides [this] (https://github.com/yegor256/takes/pull/89#discussion-diff-27631173)
I need your confirmation. If we use org.apache.log4j.Level, you'll have to add the log4j dependency in pom.xml. Is not it?

@yegor256 sure, I alreday  removed LogWrap

@yegor256 take a look the changes please

@dmzaytsev we're almost there, a few more comments above

@yegor256 review again please

@dmzaytsev two minor comments above

@yegor256 take a look

@rultor looks very good now, pls merge

@rultor looks very good now, pls merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor looks very good now, pls merge

@yegor256 Oops, I failed. You can see the full log here (spent 18min)
[INFO] artifact com.sun.tools.jxc.maven2:maven-jaxb-schemagen-plugin: checking for updates from oss.sonatype.org
[INFO] artifact com.sun:tools: checking for updates from oss.sonatype.org
[INFO] artifact com.sun:tools: checking for updates from central
[INFO] artifact org.apache.maven.plugins:maven-antrun-plugin: checking for updates from oss.sonatype.org
[INFO] artifact ant-contrib:ant-contrib: checking for updates from oss.sonatype.org
[INFO] artifact ant-contrib:ant-contrib: checking for updates from central
[INFO] artifact org.apache.maven.plugins:maven-assembly-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.apache.maven.plugins:maven-changes-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.apache.maven.plugins:maven-clean-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.apache.maven.plugins:maven-compiler-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.apache.maven.plugins:maven-dependency-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.apache.maven.plugins:maven-deploy-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.apache.maven.plugins:maven-failsafe-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.apache.maven.plugins:maven-help-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.apache.maven.plugins:maven-install-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.apache.maven.plugins:maven-invoker-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.apache.maven.plugins:maven-jar-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.apache.maven.plugins:maven-javadoc-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.apache.maven.plugins:maven-plugin-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.apache.maven.plugins:maven-release-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.apache.maven.plugins:maven-resources-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.apache.maven.plugins:maven-site-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.apache.maven.doxia:doxia-module-markdown: checking for updates from oss.sonatype.org
[INFO] artifact org.apache.maven.doxia:doxia-module-markdown: checking for updates from central
[INFO] artifact org.apache.maven.plugins:maven-source-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.apache.maven.plugins:maven-surefire-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.apache.maven.plugins:maven-war-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.codehaus.mojo:cobertura-maven-plugin: checking for updates from oss.sonatype.org
[INFO] artifact org.codehaus.mojo:versions-maven-plugin: checking for updates from oss.sonatype.org
[INFO] Generating "JavaDocs" report             --- maven-javadoc-plugin:2.9.1:javadoc
[ERROR] Error fetching link: http://xml.apache.org/commons/components/external/apidocs/package-list. Ignored it.
[ERROR] Error fetching link: http://json-processing-spec.java.net/apidocs/package-list. Ignored it.
[ERROR] Error fetching link: http://www.jcabi.com/jcabi-http/apidocs/package-list. Ignored it.
[ERROR] Error fetching link: http://www.jcabi.com/jcabi-xml/apidocs/package-list. Ignored it.
[ERROR] Error fetching link: https://github.com/hamcrest/JavaHamcrest/hamcrest-core/apidocs/package-list. Ignored it.
[ERROR] Error fetching link: https://github.com/yegor256/xembly/apidocs/package-list. Ignored it.
[ERROR] Error fetching link: http://restfb.com/apidocs/package-list. Ignored it.
[ERROR] Error fetching link: http://projectlombok.org/apidocs/package-list. Ignored it.
[ERROR] Error fetching link: http://jsonp.java.net/apidocs/package-list. Ignored it.
[ERROR] Error fetching link: http://code.google.com/p/guava-libraries/guava/apidocs/package-list. Ignored it.
[ERROR] Error fetching link: http://www.jcabi.com/jcabi-matchers/apidocs/package-list. Ignored it.
[ERROR] Error fetching link: https://jsr311.dev.java.net/apidocs/package-list. Ignored it.
[ERROR] Error fetching link: https://jersey.java.net/jersey-client/apidocs/package-list. Ignored it.
[ERROR] Error fetching link: http://saxon.sourceforge.net/apidocs/package-list. Ignored it.
[ERROR] Error fetching link: http://saxon.sourceforge.net/apidocs/package-list. Ignored it.
[ERROR] Error fetching link: http://uel.java.net/apidocs/package-list. Ignored it.
[ERROR] Error fetching link: https://github.com/hamcrest/JavaHamcrest/hamcrest-library/apidocs/package-list. Ignored it.
[ERROR] Error fetching link: http://www.mockito.org/apidocs/package-list. Ignored it.
[INFO]
4 warnings
[WARNING] Javadoc Warnings
[WARNING] javadoc: warning - Error fetching URL: http://www.h2database.com/javadoc/package-list
[WARNING] javadoc: warning - Error fetching URL: http://jolbox.com/bonecp/downloads/site/apidocs/package-list
[WARNING] /home/r/repo/src/main/java/org/takes/Request.java:50: warning - Tag @link: reference not found: org.takes.rq
[WARNING] /home/r/repo/src/main/java/org/takes/Response.java:55: warning - Tag @link: reference not found: org.takes.rs
[INFO] Generating "Cobertura Test Coverage" report --- cobertura-maven-plugin:2.5.2:cobertura
[INFO] Cobertura <IP_ADDRESS> - GNU GPL License (NO WARRANTY) - See COPYRIGHT file
Cobertura: Loaded information on 240 classes.
Report time: 6650ms

[INFO] Cobertura Report generation was successful.
[INFO] Generating Sitemap.
[INFO] Rendering 1 generated Doxia document: 1 xdoc
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 04:16 min
[INFO] Finished at: 2015-04-02T19:10:44+00:00
[INFO] Final Memory: 44M/434M
[INFO] ------------------------------------------------------------------------
+ mvn clean
[INFO] Scanning for projects...
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] Building takes 1.0-SNAPSHOT
[INFO] ------------------------------------------------------------------------
[INFO]
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ takes ---
[INFO] Deleting /home/r/repo/target
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 2.693 s
[INFO] Finished at: 2015-04-02T19:10:52+00:00
[INFO] Final Memory: 10M/303M
[INFO] ------------------------------------------------------------------------
++ pwd
+ pdd --source=/home/r/repo --verbose --file=/dev/null
INFO: "my version is 0.14"
INFO: "reading /home/r/repo"
INFO: "352 file(s) found"
INFO: "puzzle 101:0/IMP at src/main/java/org/takes/facets/slf4j/TkLogged.java"
INFO: "puzzle 28:30/DEV at src/main/java/org/takes/facets/slf4j/TsLogged.java"
ERROR: "puzzle src/main/java/org/takes/facets/slf4j/TkLogged.java:74-79 has an estimate of 0 minutes, which is lower than 15 minutes"
/var/lib/gems/1.9.1/gems/pdd-0.14/lib/pdd.rb:149:in `rules': 1 errors, see log above (PDD::Error)
	from /var/lib/gems/1.9.1/gems/pdd-0.14/lib/pdd.rb:104:in `xml'
	from /var/lib/gems/1.9.1/gems/pdd-0.14/bin/pdd:90:in `<top (required)>'
	from /usr/local/bin/pdd:23:in `load'
	from /usr/local/bin/pdd:23:in `<main>'

@yegor256 is  estimated time mandatory in puzzle description?

@dmzaytsev yep

@yegor256 merge please

@rultor merge this

@rultor merge this

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge this

@yegor256 Done! FYI, the full log is here (took me 16min)

@elenavolokhova please, let us know what do you think about this ticket, according to our QA rules

@rultor please deploy

@rultor please deploy

@davvd OK, I'll try to deploy now. You can check the progress here

@rultor please deploy

@davvd Oops, I failed. You can see the full log here (spent 5min)
Building native extensions.  This could take a while...

ruby-filemagic-0.6.1 [2014-08-19]:

* Account for options passed to FileMagic.fm (issue #7 reported by Adam
  Wróbel).

Successfully installed nokogiri-1.6.5
Successfully installed ruby-filemagic-0.6.1
Successfully installed rake-10.4.2
Successfully installed pdd-0.14
4 gems installed
Installing ri documentation for nokogiri-1.6.5...
Installing ri documentation for ruby-filemagic-0.6.1...
Installing ri documentation for rake-10.4.2...
Installing ri documentation for pdd-0.14...
Installing RDoc documentation for nokogiri-1.6.5...
Installing RDoc documentation for ruby-filemagic-0.6.1...
Installing RDoc documentation for rake-10.4.2...
Installing RDoc documentation for pdd-0.14...
++ jq -r .version
++ curl -K ../curl-appveyor.cfg --data '{accountName: '\''yegor256'\'', projectSlug: '\''takes'\'', branch: '\''master'\''}' https://ci.appveyor.com/api/builds
+ version=394
+ true
++ curl -K ../curl-appveyor.cfg https://ci.appveyor.com/api/projects/yegor256/takes/build/394
++ jq -r .build.status
+ status=queued
+ '[' queued == success ']'
+ '[' queued == failed ']'
+ echo 'waiting for AppVeyor build 394: queued'
+ sleep 5s
waiting for AppVeyor build 394: queued
+ true
++ curl -K ../curl-appveyor.cfg https://ci.appveyor.com/api/projects/yegor256/takes/build/394
++ jq -r .build.status
waiting for AppVeyor build 394: running
+ status=running
+ '[' running == success ']'
+ '[' running == failed ']'
+ echo 'waiting for AppVeyor build 394: running'
+ sleep 5s
+ true
++ curl -K ../curl-appveyor.cfg https://ci.appveyor.com/api/projects/yegor256/takes/build/394
++ jq -r .build.status
+ status=running
+ '[' running == success ']'
+ '[' running == failed ']'
+ echo 'waiting for AppVeyor build 394: running'
+ sleep 5s
waiting for AppVeyor build 394: running
+ true
++ curl -K ../curl-appveyor.cfg https://ci.appveyor.com/api/projects/yegor256/takes/build/394
++ jq -r .build.status
+ status=running
+ '[' running == success ']'
+ '[' running == failed ']'
+ echo 'waiting for AppVeyor build 394: running'
+ sleep 5s
waiting for AppVeyor build 394: running
+ true
++ curl -K ../curl-appveyor.cfg https://ci.appveyor.com/api/projects/yegor256/takes/build/394
++ jq -r .build.status
+ status=running
+ '[' running == success ']'
+ '[' running == failed ']'
+ echo 'waiting for AppVeyor build 394: running'
+ sleep 5s
waiting for AppVeyor build 394: running
+ true
++ jq -r .build.status
++ curl -K ../curl-appveyor.cfg https://ci.appveyor.com/api/projects/yegor256/takes/build/394
+ status=running
+ '[' running == success ']'
+ '[' running == failed ']'
+ echo 'waiting for AppVeyor build 394: running'
+ sleep 5s
waiting for AppVeyor build 394: running
+ true
++ curl -K ../curl-appveyor.cfg https://ci.appveyor.com/api/projects/yegor256/takes/build/394
++ jq -r .build.status
+ status=running
+ '[' running == success ']'
+ '[' running == failed ']'
+ echo 'waiting for AppVeyor build 394: running'
+ sleep 5s
waiting for AppVeyor build 394: running
+ true
++ curl -K ../curl-appveyor.cfg https://ci.appveyor.com/api/projects/yegor256/takes/build/394
++ jq -r .build.status
+ status=running
+ '[' running == success ']'
+ '[' running == failed ']'
+ echo 'waiting for AppVeyor build 394: running'
waiting for AppVeyor build 394: running
+ sleep 5s
+ true
++ curl -K ../curl-appveyor.cfg https://ci.appveyor.com/api/projects/yegor256/takes/build/394
++ jq -r .build.status
+ status=

@davvd This ticket looks good from QA point of view.

@davvd This ticket looks good from QA point of view.

@elenavolokhova thanks, got it

@longtimeago just added 10 mins to @elenavolokhova (for QA), payment ID is 54710489; Much appreciated! 18 mins was added to your account, payment ID 54710498, time spent is 133 hours and 21 mins; you're getting extra minutes here (c=3); +18 added to your rating, at the moment it is: +3757


[CIT-506] Use nanoid to generate request ids

Proposed changes

Change the type of request ids from usize to string.
Introduce two constructors per request type:

::new() use nanoid to generate the id of the request
::new_with_id() use the given id.

The responses are built after a request re-using the request id

Types of changes
What types of changes does your code introduce to thin-edge.io?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Improvement (general improvements like code refactorings that doesn't explicitly fix a bug or add any new functionality)
[ ] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have read the CONTRIBUTING doc
[ ] I have signed the CLA (in all commits with git commit -s)
[x] I ran cargo fmt as mentioned in CODING_GUIDELINES
[ ] I used cargo clippy as mentioned in CODING_GUIDELINES
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Codecov Report

Merging #368 (f27795c) into main (acc8e36) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #368      +/-   ##
==========================================
+ Coverage   71.15%   71.19%   +0.03%
==========================================
  Files          91       91
  Lines        5314     5335      +21
==========================================
+ Hits         3781     3798      +17
- Misses       1533     1537       +4

Impacted Files
Coverage Δ

sm/json_sm/src/lib.rs
100.00% <100.00%> (ø)

sm/json_sm/src/messages.rs
85.42% <100.00%> (+0.67%)
:arrow_up:

mapper/tedge_mapper/src/collectd_mapper/batcher.rs
80.35% <0.00%> (-1.79%)
:arrow_down:

Continue to review full report at Codecov.

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


Command Enhancements: L-R menus for some embeds

Have some embeds be left-right menus that list items and their previews. Use left-right react buttons to switch between different pages. Will be useful for #1 and also just generally looks really cool.

Example : ~listpins


Need ability to ignore community string with trap_listener.

Regarding v1.0.4 of SNMP I have an issue.
On line 680 of manager.rb for trap_listener we see..
if @community == message.community <-------------------------
trap = message.pdu
if trap.kind_of?(InformRequest)
@transport.send(message.response.encode, source_ip, source_port)
end
trap.source_ip = source_ip
select_handler(trap).call(trap)
end
Which is causing an issue in my environment where we have multiple comm strings used by different parts of my company.
It would be nice to set the community string to nil or some other value in order to accept all SNMP traps regardless of which string they use.
Can we get this incorporated into the next release? Thank you for writing a great GEM as ruby-SNMP has served us quite well.

I have tested the change and it works fine for us. I tested using both nil and an array of known legit comm strings and it all passed. Can we assume 1.0.5 will have this feature? Thank you much for your help.

I just published version 1.1.0. The TrapListener change is included.


model: aya-expanse

8b: https://github.com/janhq/models/actions/runs/11508220100
32b: https://github.com/janhq/models/actions/runs/11508226092

Model worked perfectly with Cortex


docs: fix typos and improve wording for the FAQ page

Fix typos and improve wording for the FAQ page
This PR introduces a grammar check to the FAQ page of the docs website. Changes include adding missing punctuation marks, splitting sentences where the continuation is not related to the rest of the sentence, and clarifying some words and sentences to make the docs more accessible to developers with more limited knowledge of English.
Overview

What is it?

[ ] Feature / enhancement
[ ] Bug
[X] Docs / tests / types / typos

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Use cases and why

One use case

Another use case

Checklist:

[X] My code follows the developer guidelines of this project
[ ] I have performed a self-review of my own code
[X] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

Thanks @brkyurun ! great job!


neovim: pass wrapper args only once

Description of changes
fix for #231172 cc @teto
neovim.tests.run_nvim_with_ftplugin failed, but I have the same behavior without any changes
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

this means additional wrapperArgs wont be used when generating the manifest but that's probably fine.

indeed sry I was looking at the wrong version of the code.


fix: adjust sql to limit by block and reverse

Credits per Dollar=  (1,000,000 / 0.04)

=25,000,000 tokens per dollar
@GentikSolm think this math is right.


Adds an ILogger parameter to the TextFileConfiguration Get methods

This PR:

Adds an ILogger parameter to the TextFileConfiguration Get methods and updates the calling code.

This change was requested on https://github.com/stratisproject/StratisBitcoinFullNode/pull/1478.

@noescape00, this PR adds the change that you requested during the review of PR https://github.com/stratisproject/StratisBitcoinFullNode/pull/1478.

test LoadConfigWithAssumeValidZeroSetsToNull is failing

I added one more final change to respect the LogLevel setting.


Codewars STDOUT

Codewars stdout doesn't print part of a string when between < and >

Which language?
When I was adding Go, I encountered this problem and I had to use a special tag to prevent the output from interpreted as HTML. See this comment and the reply.
I'd suggest opening an issue in codewars-runner-cli. Manually escaping before outputting might work as a workaround for now.

I think #831 is the same issue in C.
This seems to happen when the character following < is [a-zA-Z].

Sounds similar to my #765 too...

There's AT LEAST one Kata that uses this as a feature to display inline images:
https://www.codewars.com/kata/image-processing/c

HTML output is a feature and there are many kata using it. This issue needs more info, but the renderer does need improvements on checking whether the string contains HTML fragment or not.
The example in #831 is probably mistaking .
                            ]*>
<~1I4T&>9Z0Y6[EH-:*h&<,bD_-$<EM;d=<a<CLh<@lZ/,3(HGC~>
                            ^^^
<~1I4T&>9Z0Y6[EH-:*h&<,bD_-$

If you are a kata author running into this issue, you can specifically escape the output by prefixing it with <LOG:ESC:>

It was a mistake of the original site design to allow for HTML by default within the log output. As a way to try to deal with that design flaw in a way that is backwards compatible with older content, I'm adding some auto-detection to try to determine if content contains any HTML and if not, it will render the content as text.
I will be releasing this change today, I already have it up on preview.codewars.com for you guys to test. @g964 @kazk @dinglemouse2250 @dinglemouse2250 @OverZealous @Dan-Nolan I would like to hear your feedback. Here are the basic rules in place.
Basic HTML Detection Regex
/<[a-z]*( [a-z-]*=.*)*( ?\/)?>/ will be used to auto-detect if HTML is present within the content. It does not require matching closing elements to be present but it does require that all elements and attributes by lower cased.
Already Escaped Detection Regex
Some content has already been created which escapes characters on its own, so we want to render them as HTML. Right now I am using /(<|>|&#\d{4,6};)/ to detect escaped <, > and char codes.
Passed/Failed/It/Describe statements:
These statements use HTML detection only since char codes may in fact by part of the description and and its not likely that someone has intentionally escaped < or > (though I could be wrong).
Log Output:
For log output (console.log, puts, etc) I am using both HTML and Escape detection
Have I missed anything? Is this a terrible idea? If you guys have any content that you think might be affected by this would you mind testing on preview?
My next step towards this is think of how best to configure a default MIME type for the kata. Once this is determined and implemented, then we would consider all legacy content to be a special "auto-detect" value, where as all future (or updated legacy) content could be specifically set to HTML.
In the meantime I've also added a HTML log type so now you can use:
<LOG:HTML:>This code will always be HTML
<LOG:ESC:>This code will always be escaped

if you decide to use the  prefixed messages, remember that line breaks need to be replaced with <:LF:>

BTW I've also fixed scripts not running and the issue where sometimes encoding characters  would randomly be re-encoded to something else.

I like it in general. It all makes sense to me.
I might suggest adjusting the already escaped test. I'm not sure I'd bother checking for explicit char codes (&#\d{4,6};), but you might want to add in " and & for your already escaped detection.
The "only lowercase HTML" could possibly cause minor issues, but there aren't too many cases where people still type tags in all <UPPERCASE>, so I think that's a good choice.

Would this affect markdown parsing? Example Challenge: https://preview.codewars.com/kata/simple-fun-number-1-seats-in-theater/train/javascript
Preview:

Production:

That looks like an issue I've seen on Qualified, I'm guessing CW has the updated markdown parser, too, where it won't render headers without a space between the # and the first character.
i.e.: This works:
# Hello

But this doesn't:
#Hello

That is caused by how myjinxin2015 likes to setup his markdown, using - #Header. I've been manually correcting it as I find them but I need to get a hold of him and ask him to use something like # Header instead.

Cool, totally unrelated then. So far I haven't seen it break any content in preview 👍

Actually it looks like my solution is no longer working for Image Processing like @nomennescio mentioned.
STDERR:
[eval]:376
   <canvas id="cvs_${displayID}"
    ^
SyntaxError: Unexpected token <
    at Object.<anonymous> ([eval]-wrapper:6:22)
    at
    at evalScript (node.js:536:25)
    at startup (node.js:80:7)
    at node.js:906:3

For the markdown issue, I upgraded markedjs. That seems to be why - #Header no longer works. I'll look into image processing (I had just fixed the script issue on that, but due to content size restrictions it looks like the last 2 tests don't render properly)

Add &#x[\da-fA-F]+; for already escaped. And do &#\d+; instead. I think the spec allows any number of digits digits.

For my opinion about this, I don't think this is terrible, but really hard to do this well.

Cool, if it helps it looks like it works when you send it to CodeRunner without ciphered test cases, like within the Kata Editor versus the Kata Trainer

Decimal numeric character reference
The ampersand must be followed by a "#" (U+0023) character, followed by one or more ASCII digits, representing a base-ten integer that corresponds to a Unicode code point that is allowed according to the definition below. The digits must then be followed by a ";" (U+003B) character.
Hexadecimal numeric character reference
The ampersand must be followed by a "#" (U+0023) character, which must be followed by either a "x" (U+0078) character or a "X" (U+0058) character, which must then be followed by one or more ASCII hex digits, representing a base-sixteen integer that corresponds to a Unicode code point that is allowed according to the definition below. The digits must then be followed by a ";" (U+003B) character.
https://www.w3.org/TR/html5/syntax.html#character-references

Looks like Image Processing works on Node v6, for some reason training again reverted it to v0.10 without me realizing.

I think it's less work in long term to make HTML opt-in. Introduce <LOG:HTML:> then escape by default and fix the existing kata that intentionally uses HTML.
Majority of the kata doesn't use HTML and I'm sure there will be lots of edge cases doing HTML detection. Trying to fix all that will be painful.

I don't know how kata is stored in the database, but is it possible to do HTML detection on them to see the number of affected katas?

That's a good point, if it's only a few dozen kata, we'd be able to scan & fix it once, rather than every time the result is rendered.
The only uncertain aspect to that is some kata (and Qualified challenges) don't necessarily have the raw HTML visible within the code. For example, it might look like this:
var output = '' }
output += value
...

Or they might only render the solution, which is HTML. So there might not be any HTML at all.
And a third scenario, there might only be HTML rendered if the solution fails (I've written one like that, where failed results are rendered in a pre block, but successes are not rendered at all).

Yeah it will be hard to detect like @OverZealous said. We would have to check for things like .toCharCode in various different languages. If we stored the output of the reference solution this would be a lot easier. Also adding <LOG:HTML:> would have to be manually done per each language, especially since if multi-lines are used we would also need to replace /n with <:LF:>. Like you said though, its probably not more than 50 kata that are affected.
@kazk I agree though that this is going to be a tricky thing to keep up with. My biggest concern is the poor guy who is trying desperately to do something and can't for the life of him/her figure out why the hell sometimes code is escaped and sometimes it isn't.
My hope is that we can at least cover the common cases that affect our existing content as a quick stop-gap, then introduce maybe either a preloaded config (i.e. // @config: mime-type auto-detect) or output token (ie. <CONFIG::MIME>auto-detect), then one by one identify the content that needs this behavior and add those tokens to it, then once those tokens are pretty well added to our content base, switch the default so that auto-detection is no longer the default (text/plain would be).

For Codewars, is it possible to pick one valid and one invalid solution and run to inspect the output? Kind of similar to doing solution invalidation, but not running every solution.

My biggest concern is the poor guy who is trying desperately to do something and can't for the life of him/her figure out why the hell sometimes code is escaped and sometimes it isn't.

I agree, and this is why I'd prefer not to HTML detect. This poor guy might be doing something unexpected but completely valid.
For the amount of work, I still think it's less even with language variations in long term.
But I now see the difficulties detecting :(
If you decide to do HTML detection, I found is-html doing quick npm search. He might also have a package for this problem. 😆

@jhoffner I don't know if I'm understanding your last paragraph. Is this intended to be temporary and are you planning to gradually shift to defaulting to plain text?

Is this intended to be temporary and are you planning to gradually shift to defaulting to plain text?

Yes

is-html

Interesting. The html-tags dependency this package uses would be very helpful. Whitelisting the tags that can be considered html would probably go a long way in preventing false positives. I'll add this in.

Ok, then I don't see a problem in this approach :+1:

Some random thoughts:

HTML detection - wondered why only lower case tags detected

Escapes - I use this one fairly often

It's not just the Kata/tests that have HTML - sometimes I use HTML tags in my solution for output/debugging even when it was not used by the Kata.

I'm confused - is all this effort only fix the problem of things between < and > not rendering properly because they get mis-interpreted as HTML? (like sometimes happens with "view solution" in discourse comments). If so - maybe it is easier if CW just embeds <LOG:ESC:> for those few known cases instead of overhead to attempt HTML detect everywhere else.

Stdout font - A related problem seems a recent change to variable width fonts in the stdout. That ruined a lot of output for tables and pictures drawn with chars for me. I don't want to have to use HTML to enforce the fixed font when it previous did this by default (earlier today I saw it making bad output - now it seems ok again - maybe this is already being modified as I type?)

eof

wondered why only lower case tags detected

Valid HTML is supposed to be lower-case. From what I've seen everyone so far has used lower-case, and not supporting uppercase may help prevent content that looks like HTML but isn't from getting detected.

I use this one fairly often

Good point, I'm just going to use &[a-z]{2,6}; to test for character entities

is all this effort only fix the problem of things between < and > not rendering properly because they get mis-interpreted as HTML

Yeah pretty much. adding LOG:ESCs all over the place is a manual process of adding and I'm trying to get a fix that solves the problem now. We can deprecate auto-detection later once we have cleaned the content up.

Stdout font

I've got this fix queued up for today, should already be on preview.

The new markdown parser seems to be choking some katas which have descriptions with single line code blocks using 3 backticks because it picks up the whole line as if it's a language while it's not, which result in a parse error. e.g:
https://www.codewars.com/kata/a-simple-music-decoder/
If you enter the train screen it's put to a halt while loading, with error message:

(Incidentally, that kata also has improper headers without spaces between # and the actual text. Goes on to show that how easy it is for someone to write non-conformal markdown!)

I found another kata that causes problem to test output:
https://www.codewars.com/kata/calculate-the-function-f-x-for-a-simple-linear-sequence-easy/train/javascript
(Try submitting the code and observe the test output. No errors from console this time but the test comes out completely unformatted.)

Passed/Failed/It/Describe statements:
These statements use HTML detection only since char codes may in fact by part of the description and and its not likely that someone has intentionally escaped < or > (though I could be wrong).

Update, I've had to change this behavior as it was causing issues with certain content. I believe I had come across other content that needed this behavior but has been broken for a while.
It makes a lot more sense to not render describe/it blocks as HTML as most typical use cases would be more likely something along the lines of "should render bold".

@jhoffner I think you should also update this kumite demonstrating the use of AngularJS on test output you wrote a long time ago:
http://www.codewars.com/kumite/56d7779373b8348def00023b?sel=56d7779373b8348def00023b

Another kata that has invalid headers (no whitespaces between # and actual text) in the description:
https://www.codewars.com/kata/deficiently-abundant-perfect-amicable-numbers

What's the latest with this HTML tag auto-detect for STDOUT?
I used to be able to do Java System.out.println("
something") but today it is just printing the
 as text instead of a horizontal line like it used to and like I want it to.


Add PBR support

As described in this issue, I would like PBR support in a wavefront .obj loader, as it is in fact supported by Blender, and used by yours truly.
So... Here it is!
The changes:

Added roughness, metallic, sheen, clearcoat_thickness, clearcoat_roughness, anisotropy, anisotropy_rotation, ambient_map_path, roughness_map_path, metallic_map_path, sheen_map_path, emissive_map_path and normal_map_path
Moved bump_map_path below specular_map_path in the ordering, to better match the ordering of the Wikipedia page
Added .zig-cache to the gitignore

Excellent - thank you very much! :pray:


feat: add support for getRawMany() and row count as it is

Hi,
In this PR, I try to solve two following issues:

https://github.com/ppetzold/nestjs-paginate/issues/265  --> Primary issue that I need to solve.
https://github.com/typeorm/typeorm/issues/5127  , https://github.com/typeorm/typeorm/issues/7454

I totally open for any feedback, and I do really need @ppetzold guidance to improve my changes in this PR.
Thank You

Looks solid at first sight. Could you add new config params description to README?

Hi,
Thank you for your reply,
Ok I will help to update the documentation (README).
And I have some concerns about usage of queryBuilderCloned . I need your opinion, about that. Is it better to use queryBuilderCloned or move lines: https://github.com/ppetzold/nestjs-paginate/blob/master/src/paginate.ts#L222-L230   into below / under line: 389, then we process the counting first and then data rows fetching.

Hi Folks,
Referring to the concept and principle of SQL aggregate functions and GROUP BY . After I tried to understand (rethink)  and reflect more deeply. I think the changes in this PR is not relevant anymore. Putting pagination in aggregation-query is doesn't make sense. We will get invalid results.
I close this PR.

Hi! I'd like to weigh in here on point 3:

3.1: returnRaw, fetchRaw
3.2: preserveRowCount, exactRowCount

Is there any updates here?


Fix abort request

https://stackoverflow.com/a/3648331

@FaridSafi Could you take a look at this PR?


UI Don't pre-select all users for approval

In the approval admin tab, this removes the pre-selection of all users to approve for site or event sign up.
For instance, there were quite a few spam sign ups on ramp.studio lately and the current default can result in accidentally approving those spam accounts, particularly by new admins.
If all users need to be selected, it's still easy enough with 1 click.

LGTM, thanks @rth . Maybe @agramfort can look at it, a 30sec decision. I think it's a good change also for ramp.studio.

LGTM @rth. It will be useful for ramp.studio. Thanks !

Thanks! The CircleCI  issue is unrelated, I'll fix it in a separate PR.


async issue

I'm developing an application which uses youtube-dl for some of it's functionality
however, I get a different result when using a build from dotnet build -c Release  and the corert,
dotnet build -c Release:

corert:

relevant code:
main
async function
How to reproduce

git clone https://github.com/tim241/corert-async-bug
install youtube-dl and add it to PATH
use dotnet publish -c Release -r win-x64 (in the AudioControlServer folder) to build for the corert

I'm not sure why the corert is giving me a different result, maybe it doesn't fill the JsonList?

The problem is race condition in FillJsonList function in your program. The function is async; but nothing is waiting for it to finish - it is called without await. It means that it will run in parallel with the rest of your program.
When your program runs on CoreCLR, it is slowed down by JITing and nothing actually runs in parallel.
When your program runs on CoreRT, it runs much faster and the parallel execution causes unexpected results.
You can fix the problem by changing the FillJsonList function to be a regular (synchronous) function - there is no reason for it to be async given how it is used:
        private void FillJsonList()
        {
            using(Process p = new Process())
            {
                p.StartInfo = new ProcessStartInfo
                {
                    FileName = "youtube-dl",
                    Arguments = $"--skip-download --dump-json {Url}",
                    UseShellExecute = false,
                    RedirectStandardError = true,
                    RedirectStandardOutput = true,
                    RedirectStandardInput = true,
                };

                try
                {
                    p.Start();
                }
                catch(Exception e)
                {
                    throw new Exception("Failed to launch youtube-dl! " + e.Message);
                }

                string line;
                while (!String.IsNullOrEmpty(line = p.StandardOutput.ReadLine()))
                {
                    JsonList.Add(line);
                }

            }
        }

Thanks you for giving CoreRT a try!


8: Robot walks over ball instead of kicking it

Duplicate of #32
this is a problem of the "Roboter am Ball" block which doesn't behave correctly yet


refactor: use attrsOwner directly in is_cfg_enabled

As promised @edwin0cheng here is the PR to resolve the comment here.

LGTM, thanks !
bors r+


Reduce rules line weight

The first thing I noticed about the proof trees is that the line weight of the rules are thicker than the ones in bussproofs. They don't look right to me, because 1pt for the stroke weight is too much. I reduced it to 0.5pt which looks to be the same of bussproof's
Alternatively you should put it as a parameter of prooftree(), but it's not that convenient, because the function doesn't support #show and #set rules, so the tree can't be customized for the entire document

I'm ok with changing the default, but it would also be nice to have a parameter on prooftree. You can "set" it for the current scope if you do something like let prooftree = prooftree.with(line-stroke: 42pt)


Authz for node actions (approve/reject etc).

We should ensure we're happy with the authz for any node actions (such as approve/ reject) and that it is clearly documented how this is achieved.

At present node are approved/rejected based on their name. Further, they are indexed in the node store based on their name. This leads to undefined behavior when two nodes with the same name are in the network.


Octal prefix is incosistent.

Just looking through the docs it seems like 0b is used for binary, 0x is used for hex, then why just 0 for Octal? wouldn't 0o makes more sense?

Well, that's a common scheme that you can find in many languages.

The 0 comes from the old Unix days it seems.
Yes but later versions of Perl, Python, and ES have moved away from 0 only to Haskell like 0O/0o. Ruby also at least as of version 2 supports the 0o notation.

I think I'd like to only have the 0o notation, as @omeid suggests. I dislike the 0 prefix to mean octal, like in C/Ruby, it's not very intuitive. Right now it's like that because we ported it from Ruby, but we can remove it. Thoughts?
/cc @waj @bcardiff

+1 from me on leaving only consistent 0o version
--
Best Regards,
Alexey Fedorov,
Sr Ruby Developer, Clojure Developer, Rust Developer,
+49 15757 486 476
2015-06-06 17:57 GMT+03:00 Ary Borenszweig<EMAIL_ADDRESS>
I think I'd like to only have the 0o notation, as @omeid
https://github.com/omeid suggests. I dislike the 0 prefix to mean
octal, like in C/Ruby, it's not very intuitive. Right now it's like that
because we ported it from Ruby, but we can remove it. Thoughts?
/cc @waj https://github.com/waj @bcardiff https://github.com/bcardiff
—
Reply to this email directly or view it on GitHub
https://github.com/manastech/crystal/issues/740#issuecomment-109593322.

+1 for 0o

Yeay!

Thanks a lot for accepting this improvement suggestion.


Docs: Ansible sphinx theme

This PR updates the Sphinx configuration to use the Ansible theme. Also makes changes to remove warnings from docs build.

@ddonahue007 Can you add an approval for merge or should we wait on more reviewers? Thanks!


Add LLM Self-play paper

This pull request is about adding the recent Arxiv paper "Self-playing Adversarial Language Game Enhances LLM Reasoning". Thanks for your consideration!

Thanks for the submission! Unfortunately, this list is dedicated to robotics + LLMs; while it includes reasoning it's specifically reasoning methods applied to robotics problems. From my read of this paper, I don't see a robotics component. Let me know if I'm mistaken though.


StringRange not taken into account when mapping Brigadier suggestions to cloud

Just add an argument MultiplePlayerSelectorArgument and observe the suggestions don't work past the initial @e suggestion. The cause is that brigadier suggestions, after typing in @e return [. The default suggestion filter removes suggestions that do not start with the current word, in this case @e. If brig returned @e[, then it would work.
I think the best thing, would be to not filter that by default, instead allowing plugins to add such filtering if they wanted. That would mean that within any custom suggestions in custom arguments, you would have to manually match against the current word to return only matching suggestions. Or maybe it could be defined per argument type.
Initial discussion on discord found here.

The issue here isn't with filtering but with how brig suggestions are mapped to cloud suggestions. The default filtering is something we want to revisit for v2 though.


Translations update from Weblate

Translations update from Weblate for OpenRefine/Translations.
Current translation status:

Hi @wetneb , here occurred the error which is falling tests in your pr #3634 . Looks like @weblate doesnt update the prs with the updated master hence the error occurred on this branch yet was fixed on master.

Wow, I really don't get it! How could a weblate PR be the cause of a linting failure in #3634? It did not touch the offending files at all, right?

Yes it didn't touch the offending files, what happened here is when the PR https://github.com/OpenRefine/OpenRefine/pull/3601 was made, we did the change in ci scripts to use yarn lint which was already committed on master, but it was not committed in the fork's branch. So the old code was run into the modified ci script which also didn't let the ci fail as it was mistake over a mistake. I know how confusing this can be but to sum it up, unlisted files were committed in PR https://github.com/OpenRefine/OpenRefine/pull/3601 which were not detected by our ci due to the pull_request_target trigger which takes the ci workflow of the Openrefine/master but use the code of {Fork_User}/branch from where the PR was made.

Let me fix it asap!


xpanes:Warning: 'xpanes' may not work correctly! Please check followings.

I installed tmux and tmux-xpanes from the corresponding GitHub-repositories and I get the following error message everytime I run xpane:
xpanes:Warning: 'xpanes' may not work correctly! Please check followings.
* tmux is installed correctly.
* Supported tmux version is installed.
  Version 1.8 and over is officially supported.

Here are the versions:
$ xpanes -V
xpanes 4.1.0-beta

$ tmux -V
tmux next-2.9

Hi @ddeimeke
Thanks for using tmux-xpanes with latest version (Wow, 4.1.0-beta !).
This error message is displayed if the installed tmux verison is out of support.
But it seems that xpanes recognizes your tmux version incorrectly.
xpanes checks the result of tmux -V to check whether supported version of tmux is used or not.
But the latest tmux (on master branch) returns the irregular format like next-2.9 and it causes the message.
I have no idea how to avoid the message functionally, because I cannot expect how tmux developers name the version number during their development in the future.
However, tmux-xpanes has the internal parameter to suppress the message.
Here is the workaround.
Workaround

Install tmux-xpanes v4.1.0 ( v4.1.0 just released by the way ).
Add this statement to your shell's ~/.*rc file.

export TMUX_XPANES_TMUX_VERSION=2.9

The warning message will be gone.
Because xpanes recognizes tmux version as 2.9.
This variable is not on the README.md because it's originally used for testing purpose.
Thanks!

Hi @greymd
Thanks you for your quick answer.
It works perfectly well.
Thank you!
Cheers
Dirk


Do not allow exceptions in destructors

Exceptions raised in destructors are ignored and a warning message is displayed. Catch exceptions raised and prevent them from propagating.
Raising exceptions in the ObjectBase destructor was causing stack overflow in the unit test tests/python/unittest/test_auto_scheduler_cost_model.py if xgboost was not installed in both windows and linux. This PR eliminates the stack overflow in that case.

It is exactly a bug that troubles me for years! Thanks Bob! CC: @tqchen

I know we should not be raising exceptions from __del__, but we kinda went down this path and now exceptions raised in __del__ can serve as a note to the user that they left objects live before terminating the program. in a server setting, that might be important.
in this case, the underlying problem was that a Node subclass did not call super().init() straightaway, and then raised an exception. This caused Object.__del__ to enter a recursive loop because self.handle slot was not populated. #7481 makes del robust to missing handle. I prefer we merge that rather than suppress all of these error messages, as it's fairly opinionated of our library that users may not care they are leaking memory

There is a better solution. It is still not correct to raise exception from a destructor and we should not rely on bad code for error messages. A proper alternative is to print an error message when we catch an exception. This will give you the exact result you want. raising an exception in a destructor does not raise an exception, it just prints a message that you are trying to raise an exception where you are not supposed to.

@rkimball please rebase against the latest main.

@tqchen I think we are not going to merge this one

Thanks @areusch for the catch. Superseded by  #7481


pw numbers to notable changes to prompt

Hi, Im in the process to compare CLI-prompt travel and these nodes. Im not getting much prompt travel effects here. Can you suggest any pw values?

are you using the value schedule as the input for the variable or are you using a static number in the widget input? I normally plug in a value schedule and use the links I provided at the top of the readme.
https://www.framesync.xyz/ and https://www.chigozie.co.uk/keyframe-string-generator/

static numbers won't change across the batch