CQL function name fix

Now adjusting function names in CQL in order to allow proper execution by the CQL execution engine in JavaScript.
To test, use:
Test111_v5_2_Artifacts.zip
ED1_v5_2_Artifacts.zip

I like the way that each step is commented.

I figured everyone tested the simple case (which I had as well and it worked fine) so I decided to go fancy. Here is the source CQL and measure as it appears in Bonnie, are we okay with this?

@mayerm94 or @jbradl11 please one of you test this. Adam squashed and submitted new code, the checkboxes still say you indicate approval of this untested code.
Thanks.

Tested locally again. Changes look good


Cells are being overwritten in the screen while scrolling V or H

Alright, to start, would like to say this library is a good job and help me a lot as a solution for presenting spreadsheets. Second, would like to know how to solve this issue:
I've made some customization for populate the content within the spreadsheet. But while scrolling, the cells are being drawn above each other, out of order, producing a great mess in the screen.
It goes like this:

But when I scroll H or V:

Would like to have (please) some clarification about it.
Thanks in advance

Rafael,
I haven’t seen that before. Are you sure you don’t have more than 1 spreadsheet view being added to the popup?
This is just a group of UICollectionViews so I’d check to see if something is strange in the delegate calls.
Jeff
On Mar 4, 2015, at 2:57 PM, Rafael Rodrigues<EMAIL_ADDRESS>wrote:
Alright, to start, would like to say this library is a good job and help me a lot as a solution for presenting spreadsheets. Second, would like to know how to solve this issue:
I've made some customization for populate the content within the spreadsheet. But while scrolling, the cells are being drawn above each other, out of order, producing a great mess in the screen.
It goes like this:
https://cloud.githubusercontent.com/assets/2456117/6493141/7f7a19de-c297-11e4-982d-cccec0037168.png
But when I scroll H or V:
https://cloud.githubusercontent.com/assets/2456117/6493165/a13322b4-c297-11e4-8881-68507a05faa1.png
Would like to have (please) some clarification about it.
Thanks in advance
—
Reply to this email directly or view it on GitHub https://github.com/mutualmobile/MMSpreadsheetView/issues/19.

Well, I'm actually using just one spreadsheet inside this viewcontroller but with a slight custom implementation as attached in the code below.
Here is how I'm loading the data source:

...Here, how I'm initializing the spreadsheet:

In the beginning I was assigning the spreadsheet to the main view and the scroll was not working properly, because the spreadsheetview was being drawn out of bounds of main view. Then, I fix it by assigning the spreadsheetview to the child view as its content. At that point, the contents of cells started to be drawn out of order while scrolling.
Really appreciate any help on this!

Is initSpreadSheetView being called more then once? Try adding NSLog(@"%s", PRETTY_FUNCTION); at the top of the function and bring it up a few times (looks like it's a view inside of a popup). That method should only fire once. If it's getting called more than once, there is the problem.

Yes, it's being called just once from viewDidLoad and I could confirm it by logging "PRETTY_FUNCTION". What else it could be? :)

Did you try open and closing self.viewProdutoEmProdutos several times so see if the initializer is getting called more than once?
From what you've shown, I don't see a problem.
You might try just using a UICollectionView in place of the spreadSheetView to see if you get the same problem.

Hi all,
I ran into the same problem when I was using it. It turns out it is a problem with the uicollection view cells not being properly reused.
What I did was every time I added a cell, I removed all its child views.
That fixed it for me.
Here is the code I used:
-(void)resetCell{
for (UIView *view in [self.contentView subviews]) {
[view removeFromSuperview];
}
}

@ITDebicki Are you saying this is a UICollectionView issue or something in @sidan 's cell implementation? Perhaps needing to check or destroy the cell's subviews in prepareForReuse?

I didn't investigate it fully, but I just used this method in the init method of each cell

@ITDebicki , it seems a nice work around, which I've used so far to reload the view while using this library: https://www.cocoacontrols.com/controls/tstableview that I end up with cuz of project's deadline.
Thanks for supporting..


Something seems wrong with the baseline

When I try to load the PUGizmo baseline I get this error. This is on Pharo8, fresh image as of 2019-11-22.
Metacello new
  repository: 'github://fuhrmanator/PlantUMLPharoGizmo/src';
  baseline: 'PUGizmo';
  load.

Thanks @kasperosterbye for reporting this.
I think the latest baseline is expecting Moose in the Image (that would explain the Famix* problems). I need to fix the baseline to not load the PUGizmo stuff for moose if it's not present.
Another issue is that Pharo 8 renamed ZnUrlTests to ZnUrlTest, and it breaks my extension with tests of ZnUrl#isValid.
Are you at least able to use the code?

The only thing I needed to use was the "load a plantuml image from a plantuml source text". I was able to pick that out of the extensions in String. So yes, I got what I needed - very helpful indeed.
I will need to credit you for that part, but there is no licence in your code, so I guess it is fine to just credit you in some appropriate comment somewhere before I push what I am working on.

Oops. License added! Thanks for letting me know. This was started in a Pharo sprint, and I missed a few things.

Interesting. I think Pharo and Moose use the MIT licence. But you are of cause free to pick the licence you like.

See https://github.com/fuhrmanator/PlantUMLPharoGizmo/issues/17#issuecomment-955024063


About Evaluation

For the evaluation I run the given command in readme with "--test" parameter but it gives "Index out of range" error. What might be the cause?
Testing on epoch 25...
Traceback (most recent call last):2 (0.00+0.24), lr 0.003, l = 2.3097, a = 0.5703, avL = 2.5296, avA = 0.6206, g = -1.0000, emL = 2.4391, emA = 0.6370; gqaExperiment
File "main.py", line 850, in
main()
File "main.py", line 777, in main
evalRes = runEvaluation(sess, model, data["main"], dataOps, epoch, evalTest = False, getPreds = True)
File "main.py", line 258, in runEvaluation
minLoss = prevRes["test"]["minLoss"] if prevRes else float("inf"))
File "main.py", line 573, in runEpoch
imagesBatch = loadImageBatch(data["images"], batch)
File "main.py", line 365, in loadImageBatch
imageBatch[i, 0:numObjects] = toFile(imageId)["features"][imageId["idx"], 0:numObjects]
File "h5py/_objects.pyx", line 54, in h5py._objects.with_phil.wrapper
File "h5py/_objects.pyx", line 55, in h5py._objects.with_phil.wrapper
File "/home/ec2-user/conda/envs/tf_gpu/lib/python3.6/site-packages/h5py/_hl/dataset.py", line 553, in getitem
selection = sel.select(self.shape, args, dsid=self.id)
File "/home/ec2-user/conda/envs/tf_gpu/lib/python3.6/site-packages/h5py/_hl/selections.py", line 94, in select
sel[args]
File "/home/ec2-user/conda/envs/tf_gpu/lib/python3.6/site-packages/h5py/_hl/selections.py", line 261, in getitem
start, count, step, scalar = _handle_simple(self.shape,args)
File "/home/ec2-user/conda/envs/tf_gpu/lib/python3.6/site-packages/h5py/_hl/selections.py", line 457, in _handle_simple
x,y,z = _translate_int(int(arg), length)
File "/home/ec2-user/conda/envs/tf_gpu/lib/python3.6/site-packages/h5py/_hl/selections.py", line 477, in _translate_int
raise ValueError("Index (%s) out of range (0-%s)" % (exp, length-1))
ValueError: Index (150458) out of range (0-148854)

Sorry for that! I'm pretty sure that was a bug in https://github.com/stanfordnlp/mac-network/blob/gqa/merge.py. I resolved it since then but if you ran it before there's some chance it still affects your run. I will be able to look into it more in 2 weeks unfortunately, and in the meantime I would recommend trying rerunning merge.py and run the data with --expName "gqaExperiment2" to make sure it generates clean data with the new code. Also see https://github.com/stanfordnlp/mac-network/issues/20.
I hope to get back to you with more detailed solution soon!

Thank you!


Segmentation fault in arm package

Hi @lswiderski
Raspberry users report a problem, I recreated and on my RPi (Debian 11 & 12)
https://github.com/RobertWojtowicz/miscale2garmin/issues/34
BR,
Robert

@RobertWojtowicz
Hi I did some research and found that, unfortunately due to dotnet limitations It do not support ARMv6 that it's inside RPI 0 . It can works on RPI 2,3,4, 2W
https://github.com/dotnet/runtime/issues/3013
https://www.raspberrypi.com/documentation/computers/processors.html

The solution is to switch to Python 3 API wrapper: https://github.com/cyberjunky/python-garminconnect


Get-DbaDbRestoreHistory

Included BackupStartDate along with StopAt & a combination of the two to provide the LastRestorePoint - being either the stopat point or the start of the last restored backup
Type of Change
Bug fix (non-breaking change, fixes # )
[x ] New feature (non-breaking change, adds functionality, fixes Get-DbaDbRestoreHistory - additional restore point info request #7854 )
Breaking change (effects multiple commands or functionality, fixes # )
Ran manual Pester test and has passed (.\tests\manual.pester.ps1)
Adding code coverage to existing functionality
Pester test is included
If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
Unit test is included
Documentation
Build system
Purpose
Adds additional information to the output of the function relating to the point in time of the restore.
Approach
Additional fields returned in SQL statement
Commands to test
Same as before just provide additional lines of output
Screenshots
Shows the new output and the LastRestorePoint being the start of the differential backup.
image
This shows the output of the SQL queries used in the script. Demonstrating the LastRestorePoint returned as the stop_at point:
image
The following shows the output with the last file being a transaction log created after the stop at point but covering the required time. The LastRestoredPoint is shown as the stop at point not the .trn file datetime (UTC, rather than BST)
image
Finally the log backup not being after the stop at time therefore the stop at time can't be used and the point in time will be that of the backup file.
image

Failed again on primarily the same thing, something .net related which is strange as I've only changed parts of the SQL Script.


Usable with Exponent?

Hey,
So this is amazing!
I'm just coming back to RN after a long time and I find that Exponent is the easiest way to build apps now. I had built my own custom keyboard component back in August of last year but I never figured out how to package it for Exponent, which was very new at the time.
How would we use this in Exponent?
Thanks!!

Hi @idibidiart ! You can install any packages that do NOT interact with native modules. For example, my react-native-haptic package wouldn't work out of the box. However, you can pull Exponent client which is open sourced and then modify it to work with your package.

Brent Vatne, whom I think is associated with Exponent, has pointed out that there is discussion around allowing developers to work across both XCode and Exponent using the "detach" idea, where a projected would be detached so it's runnable in XCode and can incorporate custom native components like yours (and some of the ones I had written but not released yet) and then put them in conditionals so they're ignored when you go back to Exponent. Seems like a good idea. Here's the link, fyi: https://t.co/Oyt9oEi0wd  ... Closing this and thanks for your answer!


Reactive.Events Cause Xamarin.iOS Linker to Skip All Framework

Do you want to request a feature or report a bug?
Bug / Problem
What is the current behavior?
Using any Reactive.Events extension API will cause the Xamarin.iOS Linker to skip all iOS framework. This cause iTune submission failed due to Apple will question on all API used on the app such as Call Kit API even though many of them aren't due to the bug.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem

Start a new Xamarin.iOS project.
Include ReactiveUI and ReactiveUI.Events nuget package.
Create a UIButton and use Event extension API such as NewButton.Events().Clicked.Subscribe(_ => DoSomething());
Compile the project with Linker Behavior set to "Link Framework SDKs Only".
Use otool on the app package created for example "otool -L {your_project_path}/bin/iPhone/Release/{app_name).app/{app_name)
You will see a long list of framework included with the package even the app doesn't call most of them.

What is the expected behavior?
It shouldn't cause the linker to include all framework when only UIKit framework is used.
What is the motivation / use case for changing the behavior?
Prevent dev caught in surprise when submitting their app and try to figure out what happen. I need to create a new project, divide the project part by part to figure out which library caused the problem.
Which versions of ReactiveUI, and which platform / OS are affected by this issue? Did this work in previous versions of ReativeUI? Please also test with the latest stable and development snapshot
reactiveui-events 7.4.0, project targeting iOS (I tested on Xamarin.iOS not Xamarin.Form). Not sure if the previous version have the same problem.
Other information (e.g. stacktraces, related issues, suggestions how to fix)
I first posted the problem on Stackover flow which more information about what framework the Linker skipped. https://stackoverflow.com/questions/47897990/xamarin-ios-submission-rejected-call-kit-features-api/47900470#47900470

This makes sense as all Events are added to a single assembly. This would need to be split up into several different assemblies so the ones not actually being used would be linked out.

Is there any update on this issue? This one has jumped up to bite us as we are just about to ship. We don't use Bluetooth API's but since they are not being linked away, AppStore is rejecting the app until we declare why we need access to Bluetooth.
As an aside, perhaps the docs should warn devs about this before recommending use of the ReactiveUI.Events package?

This is really a critical issue which needs to be addressed as soon as possible, especially given the App Store implications.

Realistically there isn't a timeline to fixing this in the short term. I got a couple crash bugs to fix and then will swing back around to this one. The reality of volunteer run open source software.
Happy to take a PR about the warning about the iOS package though in the mean time.

Thanks @glennawatson. I will look into a PR for the docs next week. In the meantime, for my information, I am wondering how much value we get from the Events package. I am comfortable using Observable.FromEventPattern even though the syntax is not as sugary as this.Events(). Are there other benefits to using ReactiveUI.Events?

The events packages are just auto generated FromEvent wrappers to avoid the reflection cost of FromEventPattern but otherwise just a convenience package.

We produce the .cs files as part of our build server. So you could even grab the latest and just copy paste the ones you need.

https://dev.azure.com/dotnet/ReactiveUI/_build/results?buildId=23081&view=artifacts

Consider using https://github.com/reactivemarbles/observableevents


Normative changes to decorator "extra initializer" evaluation order

Acknowledgement

[X] I acknowledge that issues using this template may be closed without further explanation at the maintainer's discretion.

Comment
Per https://github.com/pzuraq/ecma262/pull/12/files, which achieved consensus in the November 2023 TC39 meeting, the "extra initializers" for fields/auto-accessors (added by context.addInitializer) now need to be run independently for each field, immediately after the field itself is initialized and defined on the class instance.
This results in the need to change our emit for Stage 3 decorators as follows:
Given:
class C {
  @dec a;
  @dec b;
  @dec f() { }
}

We currently emit (some parts elided):
let C = (() => {
  ...
  return class C {
    static {
      ...
      __esDecorate(null, null, _f_decorators, { ... }, null, _instanceExtraInitializers);
      __esDecorate(null, null, _a_decorators, { ... }, _a_initializers, _instanceExtraInitializers);
      __esDecorate(null, null, _b_decorators, { ... }, _b_initializers, _instanceExtraInitializers);
      ...
    }
    a = (__runInitializers(this, _instanceExtraInitializers), __runInitializers(this, _a_initializers, void 0));
    b = __runInitializers(this, _b_initializers, void 0);
    f() { }
  };
})();

Here, we collect all "instance extra initializers" in a single array and evaluate them immediately after the this value is bound in the constructor (i.e. immediately upon entry to the constructor in a base class, or immediately after super() is called in a derived class). When fields are present, we evaluate the extra initializers as part of evaluating the initializer for the first field of the class.
After this change, we need to emit something like this instead:
let C = (() => {
  ...
  let _a_extraInitializers = [];
  let _b_extraInitializers = [];
  return class C {
    static {
      __esDecorate(null, null, _f_decorators, { ... }, null, _instanceExtraInitializers);
      __esDecorate(null, null, _a_decorators, { ... }, _a_initializers, _a_extraInitializers);
      __esDecorate(null, null, _b_decorators, { ... }, _b_initializers, _b_extraInitializers);
    }
    constructor() {
      __runInitializers(this, _b_extraInitializers);
    }
    a = (__runInitializers(this, _instanceExtraInitializers), __runInitializers(this, _a_initializers, void 0));
    b = (__runInitializers(this, _a_extraInitializers), __runInitializers(this, _b_initializers, void 0));
    f() { }
  };
})();

Here, we still run the extra initializers for methods/getters/setters first, but we run the extra initializers for a after it is defined on the instance, but before we initialize b, and run the initializers for b at the top of the constructor, which is evaluated after b has been defined on the instance.

What are "extra initializers"?  In the example code given there is no initialization at all as far as I can tell.

In the example code, a and b are initialized to undefined (i.e., void 0), which you can see in the fragment __runInitializers(this, _a_initializers, void 0). These are "normal" initializers, which decorators can hook:
function dec(target, context) {
  if (context.kind === "field") {
    return function (value) { return value; }; // injects a transformation into "normal" initialization.
  }
}

Extra initializers are added by a decorator via context.addInitializer, and are primarily meant to perform some kind of registration using the instance:
function dec(target, context) {
  context.addInitializer(function() {
    registerThisInstance(this);
  });
}

Here's an example of the difference in timing:
class C {
  @dec a = 1;
}
function dec(target, context) {
  context.addInitializer(function() {
    this.a; // before: undefined
            // after: 2
  });
  return function (value) {
    this.a; // before/after: undefined
    value; // 1
    return value + 1;
  };
}


Somethign wrong with project

Hey,
I was just trying to follow a guide for dynamic routing but I get errors when running: npx create-next-app nextjs-blog --use-npm --example "https://github.com/vercel/next-learn-starter/tree/master/dynamic-routes-starter"
This is my error log:
Creating a new Next.js app in /Users/<User>/Code/Learning/nextjs-blog.

Downloading files from repo https://github.com/vercel/next-learn-starter/tree/master/dynamic-routes-starter. This might take a moment.

Installing packages. This might take a couple of minutes.

npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   peer<EMAIL_ADDRESS>|| ^17" from<EMAIL_ADDRESS>npm ERR!   node_modules/next
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   3 more (react-dom, @next/react-dev-overlay, styled-jsx)
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/next/node_modules/use-subscription
npm ERR<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/next
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.````

These are my installed versions of npm, npx and node:

> npm -v
7.0.8

> npx -v
7.0.8

> node -v
v15.1.0

Also having the same issue. I'm following this tutorial: https://nextjs.org/learn/basics/create-nextjs-app/setup
@vercel

This should be solved by https://github.com/vercel/next-learn-starter/pull/24

I've tried to update my react packages to the latest ver but that didn't solve anything.

Can we get #24 merged?

I'm taking care of the package updates now! Thank you all so much.

I'm taking care of the package updates now! Thank you all so much.

https://github.com/vercel/next-learn-starter/pull/23 was merged, resolving the issues.

https://github.com/vercel/next-learn-starter/pull/23 was merged, resolving the issues.


Replace unmaintained cchardet with charset-normalizer

The cchardet package doesn't properly support python 3.11.

Thanks. I hadn't heard of this package. Good to shed unmaintained stuff.

Think you could spin a new release so that I can update this in courtlistener? I think it's the last remaining blocker for python 3.11 support.

Juriscraper release is done.


add section to docs about the new dictionary feature

I think the phrasing could be better, but hopefully the example makes it clear
fyi @rly


test(yaml): add test for deprecated boolean syntax

Changes
This PR adds a test for deprecated boolean syntax with compatMode: true.

Nice! These PRs that target test coverage are very welcome.


Typo in deprecation

https://github.com/pointfreeco/swift-url-routing/blob/380347731cd4c532696f27613fbdb12661ba8949/Sources/URLRouting/Internal/Deprecations.swift#L11
This line should be:
 @available(*, deprecated, renamed: "decodedResponse(for:as:decoder:)")

@dmzza Good catch! Do you want to contribute a quick PR to fix?

I would, but I'm contractually prohibited.


Passing custom args to intentDialog.matches.

I have a dialog that is triggered from a LUIS model:
IntentDialog.matches('luisIntentID', 'dialogID', customArgs);
I want to pass the customArgs into the start of the 'dialogID' dialog but they're overwritten by the LUIS information (entities, intents, confidence, etc). Is there anyway to persist those custom args?
Thanks in advance.

Are you saying your customArgs aren't in the set? I just looked at the code and they should definitely be there.  They will get merged with the intents, entities, and score from LUIS. are you needing them to not be merged?  If so, there's no way to prevent that but what you could do is add a level a redirection so pass in { custom: {customArgs} } instead.
Steve

Thanks Steve,
Now I look at it again, it is merging but not the way I was expecting. customArgs in my case is a string and it seems to separate each character into it's own property.
{
   "0":"c",
   "1":"u",
   "2":"s",
   "3":"t",
   "4":"o",
   "5":"m",
   "score":0.7555546,
   "intent":"isg.lockedOut",
   "intents":[...],
   "entities":[...]
}

Object work as you'd expect and just merge into the args object but strings are split up into their characters and for arrays, each property is merged into the args, instead of remaining in an array; [string, custom] turns into "0":"string",  "1":"custom".
Thanks,
James

customArgs MUST be an object.


Parallelize scripts/build.js

building packages started taking too long. I think it'll be pretty easy to use worker-farm npm package and parallelize the build of multiple packages in our build script

@aaronabramov working on it :)

It's not much faster, I don't think we should do this.


When I use 'ol-source-wmts' or 'ol-source-xyz', can I use 'tileLoadFunction' to add custom request headers?

Now I need ro add wmts with auths, but I can not find an API to add custom request headers. For example:
<ol-source-wmts
  :url="satelliteUrl"
  :tileLoadFunction="tileUrlFunction"
/>

let satelliteUrl = ref('https://<IP_ADDRESS>/iag-gateway/other/ime-cloud/rest/DOM_2000_2019_20CM/wmts')

function tileUrlFunction(tile, src) {
    let client = new XMLHttpRequest()
    let appid = 'xxx'
    let appsecret = 'xxx.'
    let timestamp: any = new Date().getTime()
    let nonce = Uuid(8)
    let sign = sha256(nonce + timestamp + appid + appsecret + timestamp)

    client.open('GET', src)
    client.responseType = 'blob'
    client.setRequestHeader('x-appid', appid)
    client.setRequestHeader('x-timestamp', timestamp)
    client.setRequestHeader('x-nonce', nonce)
    client.setRequestHeader('x-sign', sign)

    client.onload = function () {
      let blob = new Blob([this.response], { type: 'image/png' })
      let data = window.URL.createObjectURL(blob)
      tile.getImage().src = data
    }
    client.send()
  }

Hey, not sure what's exactly not working. I think it should work as you described. Would you mind to provide an example playground using the Stackblitz starter template?
By using the tileLoadFunction you should be able to implement a custom request setting the headers etc.

I tried to add tileLoadFunction to ol-source-xyz, it still does not work. Maybe it is a bug!  verison 0.5.1

I tried to add tileLoadFunction to ol-source-xyz, it still does not work. Maybe it is a bug!  verison 0.5.1


Annotation doesn't work for a controller with name as a variable

And probably with any other .module api
I use webpack and angular
import angular from 'angular';

const MY_CONTROLLER = 'myController';

angular.module('myApp').controller(MY_CONTROLLER, $scope => {});

export default MY_CONTROLLER;

There's some history here about why this is difficult:
https://github.com/olov/ng-annotate/issues/22
In general, in this use-case (and all use-cases, really), I strongly recommend using the explicit 'ngInject'; prologue directives (and explicitOnly configuration option) to annotate your functions.
Also, in your specific example, I'd advise against using a lambda function for a controller declaration, as you'll have no way of accessing your controller's of this.
I'll leave this open, as it's something that we could hypothetically support some day.  However, I want to lean on the side of caution when making any changes that could introduce false-positives.


Get buy now bid transaction from Postgres DB

When completing a buy bow bid transaction all works fine. Except to get the transaction data from Postgres. I was assuming it would end up in pg_metadata_accept_nft_bids but it´s not.
I made a buy now bid transaction (3JuEU8F7ZRuUWxEJYPUnEuVRceA4mLZKXyTL1435H8ff2Vkqe5GWBm) today for nft 13553654e884d955e9e6462244720e39d97a521a20a49c411974c7ec3b9942e7
select * from pg_metadata_accept_nft_bids where nft_post_hash = decode('13553654e884d955e9e6462244720e39d97a521a20a49c411974c7ec3b9942e7','hex')
Is empty. Ok not so weird
select * from pg_metadata_nft_bids where nft_post_hash = decode('13553654e884d955e9e6462244720e39d97a521a20a49c411974c7ec3b9942e7','hex')
Is not empty, but no easy way to see if accepted or not. Or it it was buy now.
select * from pg_nft_bids where nft_post_hash = decode('13553654e884d955e9e6462244720e39d97a521a20a49c411974c7ec3b9942e7','hex')
Empty again. That is a bit odd I think
In pg_metadata_create_nft table there is no buy now info either. And the NFT get´s reset after sale.
Kinda think this data should be easier available. But not sure if another table is needed or just include in pg_metadata_accept_nft_bids  with a isbuynow boolean?
Let me know your thoughts @diamondhands0 @lazynina  right now NFTz is under reporting NFT sales (:. Can create a separate table if needed. But would like to know if this behavior is correct.

@mvanhalen - there is some code missing to support adding to the accepted_nft_bids table in postgres.go. We need to check if a bid is a bid on a buy now NFT that exceeds the buy now price and if so, adds to metadataAccepteNFTBids. I would welcome a PR that appends to this slice around line 1077 in postgres.go

@mvanhalen - there is some code missing to support adding to the accepted_nft_bids table in postgres.go. We need to check if a bid is a bid on a buy now NFT that exceeds the buy now price and if so, adds to metadataAccepteNFTBids. I would welcome a PR that appends to this slice around line 1077 in postgres.go

Thanks for the quick follow up. Will try to beef up my go skills and make an attempt

Added a PR #226 . It compiles, but not sure if I´m doing it right. If you think it´s Ok I´ll setup a sync to check it.

#226 is exactly what I was suggesting. I'm not positive it's right. I believe the NFT entry should still reflect the buy now price, but I'm not positive. Basically it's a question of ordering the transaction.
It may be faster to just run a regtest node locally (run the n0_test script with a postgres URI) and then generate some buy now transactions locally

Ok will run a test tomorrow and let you know. Will check the NFT entry thing. To avoid an extra postgres call.

I've run a test locally and #226 works as expected - just had to set UnlockableText to an empty byte slice! 226 has been merged with that change.


find a potential typo

core/utils.py
line 446
params[k] = ['offing', 'oning', 'stop', 'hander_stop'].index(v)
‘hander_stop’ should be ‘hinder_stop’
since the motor set the run_state as "hinder_stop" after the calibration, this integration will stop work without this fixation.

Duplicate #139 will fix it

Duplicate #139 will fix it


Text wrapper not properly working with soft hyphens

The linebreak logic doesn't seem to be designed to handle soft hyphens. These are characters that indicate where a word may be split in order to prevent a big gap as the end of a line.
http://www.fileformat.info/info/unicode/char/ad/index.htm
In my language composed words are written as one, so "computer screen" becomes "computerscreen". You understand that words can get potentially very long, like "computerscreensizemeasurementaccuracycategory" or something. I put some soft hyphens in this word so it breaks if it doesn't fit on one line so you probably see a dash now, try resizing the text field.
Typing soft hyphens is annoying because you can't usually see them, so it's advised to use unicode, like in JS you can type "I have a very big computer\u00ADscreen."
so Instead of this:
I have a very big
computerscreen.
you get:
I have a very big computer-
screen.
Long story short: in PDFKit all soft hyphen characters are typed as a dash and not just the one at the end of a line. It probably takes a bit of puzzling to get the sizing algorithms working correctly.
Is this something someone would like to look a little more in to?

This is an issue for me as well. I'd love to see a solution for this in upcoming versions!


data "aws_rds_certificate" for AWS CN

Description
data "aws_rds_certificate" "aurora" {
  latest_valid_till = true
}

Error
data.aws_rds_certificate.aurora: Refreshing state...
module.aurora.data.aws_iam_policy_document.monitoring_rds_assume_role: Refreshing state...

Error: multiple RDS Certificates match the criteria; try changing search query

  on main.tf line 82, in data "aws_rds_certificate" "aurora":
  82: data "aws_rds_certificate" "aurora" {

Versions

Terraform: 0.13.6

Provider(s):

aws = {
  source  = "hashicorp/aws"
  version = ">= 3.22.0"
  region = "cn-northwest-1"
}

Module: v2.29.0

Hi @lemon-li !
I don't think your question is related to a bug in this module so please ask it in a better place (eg, discuss.hashicorp.com).


VM allocation on Primary Storage

ISSUE TYPE

Enhancement Request

COMPONENT NAME

Primary Storage

CLOUDSTACK VERSION

N.R.

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

There should be better management regarding resource allocation on primary storages.
This could be done by allowing the user to be able to choose the disk offering when creating a VM via template via UI (as already happens when creating a VM via API).
Also the parameter "Preferred storage pool" (preferred.storage.pool) (The UUID of preferred storage pool for allocation.) should be configurable in the Cluster or Zone settings so the admin could always choose the preferred PS for every zone. This could be very useful also during storage migrations.
STEPS TO REPRODUCE

EXPECTED RESULTS

ACTUAL RESULTS

@cdfgallo , this sounds like more than one feature/enhancement request. Are you working on these? If not, and to enhance the chance of those getting implemented, can you split them and expand with screen shots/depiction of the desired situation.

@DaanHoogland you are right, this is an enhancement request.
Ok, I will split them.
Ty,
Fabio

Not sure but could Flexible Tags functionality help in achieving a preferred pool allocation, @JoaoJandre?

Hello @cdfgallo @shwstppr,
You could use flexible tags to finely choose which hosts receive which VMs and the same with storages and volumes. You can make rules to specifically allow only certain groups of tags into each host/storage. You can read a little more about tags and flexible tags in the documentation. Let me know if you do not understand the docs, and have further questions about flexible tags; I can try to answer them.
However, if you only want preferred and not exclusive hosts/storages, regular tags would do just fine as well.

@JoaoJandre I don't have much idea about flexible tags. Can you provide an example rule of how someone can use flexible tags in a way similar to preferred.storage.pool?

Hi @cdfgallo @JoaoJandre @DaanHoogland @shwstppr @vishesh92 I was investigating a bit about this issue:

For the first part, users are able to choose the prefered pools by overriding the root disk offering (pre-created by an admin, which can contain storage tags matching the storage pools) during the VM creation. This is already possible, what are your suggestions to extend this functionality?
Currently the preferred.storage.pool setting is granular to accounts, I can draft a PR changing its scope to Zone (or Cluster) and do some testing

@JoaoJandre I don't have much idea about flexible tags. Can you provide an example rule of how someone can use flexible tags in a way similar to preferred.storage.pool but per zone or cluster instead of an account?

@vishesh92, @nvazquez  Sorry for the delayed response, after taking a proper look at the way that the preferred.storage.pool config works, simulating it with tags is somewhat painful. I think that extending the config to zone/cluster scope is an easier approach.

@JoaoJandre @nvazquez @vishesh92 @cdfgallo , as @weizhouapache mentions in @nvazquez ' PR there is a bit of a conflict between account and cluster settings. How can we resolve this?

@JoaoJandre @nvazquez @vishesh92 @cdfgallo , as @weizhouapache mentions in @nvazquez ' PR there is a bit of a conflict between account and cluster settings. How can we resolve this?

This might increase the complexity a bit, the only solution I can think of while maintaining the backward compatibility is to allow adding the same global setting per account & as well as a cluster. And the priority of Account would be higher than that of cluster.


Compass source maps not found

SASS source maps work fine with my own files, but I get a 404 errors in the console for all the Compass imported SASS sheets.
For example:
http://localhost:3000/sass/usr/local/var/rbenv/versions/2.1.2/lib/ruby/gems….1.0/gems/compass-core-1.0.0.rc.1/stylesheets/compass/css3/_transform.scss
Which obviously can't be found. This is the default project, what's going wrong here?

Closing this since I removed the compass dependency. Libsass + autoprefixer is totally worth it!

Agreed, works beautifully now.


feat (frontend CRUD): Add furniture to the correct room

This allows the user to add furniture to the correct room. It only works if they get to the furniture page by selecting a room from the homepage. If you start from the /furniture route, there's no roomSelected property, so no furniture is added.

Actually, maybe I'll hold off on this PR until someone can review my code in the morning. Since we're supposed to be doing that. But it's ready, and you'll have to wait until tomorrow to see it!


Fixed insertOrIgnore typehint

[ ] Added or updated tests
[ ] Documented user facing changes
[ ] Updated CHANGELOG.md

Changes
According to the doc : https://laravel.com/docs/8.x/queries#insert-statements insertOrIgnore has the same signature as insert so this typehint has to be adapted.

Thank you!

Thank you!


Create Kubernetes logs input-plugin

Logagent works fine with its docker-plugin on Kubernetes, when docker is used as container runtime. When containerd is used (e.g. IBM Cloud) the log collection via /var/run/docker.sock does not work.
I suggest creating a Kuberntes specific log collection plugin to avoid any trouble when alternative container runtimes are used in Kubernetes.

Would it be possible to collect a generic container plugin that can detect the runtime and automatically use/do the right thing under the hood without the person having to configure Logagent explicitly for Kubernetes or non-Kubernetes env?

The start script can deal with different environments to configure LA.
We can detect the runtime environment and disable docker-collector when docker socket is not available, or running in Kuberntes.
Well, the k8s user must mount /var/log into LA container to make the k8s plugin work.
Therefore we will need Kubernetes specific installation instructions, and use k8s API instead of docker API.

The following DaemonSet should work on IBM Cloud.

support for containerd log headers
file collection and enrichment with podName, namespace, containerName, containerId

apiVersion: apps/v1
kind: DaemonSet
metadata:
  name: st-logagent
spec:
  selector:
    matchLabels:
      app: st-logagent
  template:
    metadata:
      labels:
        app: st-logagent
    spec:
      #nodeSelector: {}
      hostNetwork: true
      dnsPolicy: "ClusterFirst"
      restartPolicy: "Always"
      containers:
      - name: st-logagent
        image: sematext/logagent:latest
        # image: sematext/logagent:2.0.143
        imagePullPolicy: "Always"
        env:
          - name: LOG_GLOB
            value: "/var/log/containers/*.log;/var/log/*.log"
          - name: LOGS_TOKEN
            value: "YOUR_SEMATEXT_LOGS_TOKEN"
          - name: LOGS_RECEIVER_URL
            value: "https://logsene-receiver.sematext.com"
        volumeMounts:
          - mountPath: /var/log
            name: var-log
          - mountPath: /var/data/kubeletlogs
            name: kubeletlogs
          - mountPath: /etc/localtime
            name: localtime
        securityContext:
          privileged: true
      volumes:
        - name: var-log
          hostPath:
            path: /var/log
        - name: kubeletlogs
          hostPath:
            path: /var/data/kubeletlogs
        - name: localtime
          hostPath:
            path: /etc/localtime

![Screen Shot 2019-04-02 at 00 31 18](https://user-imag
es.githubusercontent.com/7289505/55364555-67f51300-54e1-11e9-9bf5-2e18962c5f95.png)
Note, we are still testing and have seen some crashes, which might be unrelated to the new Kubernetes input plugin.

It turns out that Logagent restarts are caused by tail-forever on Node 10 https://github.com/mingqi/tailF/issues/12 -  sematext/logagent:2.0.146 ignores the uncaughtException until the issue gets fixed by a new release of tail-forever. We will try to create a pull-request or publish a new version of tail-forever node module.
The final daemonset for a more stable deployment requires v2.0.146 or later:
apiVersion: apps/v1
kind: DaemonSet
metadata:
  name: st-logagent
spec:
  selector:
    matchLabels:
      app: st-logagent
  template:
    metadata:
      labels:
        app: st-logagent
    spec:
      #nodeSelector: {}
      hostNetwork: true
      dnsPolicy: "ClusterFirst"
      restartPolicy: "Always"
      containers:
      - name: st-logagent
        image: sematext/logagent:2.0.146
        imagePullPolicy: "Always"
        env:
          - name: DEBUG
            value: "tail-forever"
          - name: LOG_GLOB
            value: "/var/log/containers/*.log;/var/log/*.log"
          - name: LOGS_TOKEN
            value: "YOUR_SEMATEXT_LOGS_TOKEN"
          - name: LOGS_RECEIVER_URL
            value: "https://logsene-receiver.sematext.com"
        volumeMounts:
          - mountPath: /var/log
            name: var-log
          - mountPath: /var/data/kubeletlogs
            name: kubeletlogs
          - mountPath: /etc/localtime
            name: localtime
        securityContext:
          privileged: true
      volumes:
        - name: var-log
          hostPath:
            path: /var/log
        - name: kubeletlogs
          hostPath:
            path: /var/data/kubeletlogs
        - name: localtime
          hostPath:
            path: /etc/localtime```

Version sematext/loagagent:2.0.149 uses Node 8 and runs stable without exceptions.

Problem with Node 10 was fixed  mingqi/tailF#12

Documentation: https://sematext.com/docs/logagent/installation-docker/#kubernetes-with-containerd-and-ibm-cloud


Please remove "ddev-get" label if this is not maintained or tested

Hi @MurzNN - It looks like maybe this add-on isn't maintained and the automated tests are not enabled, etc.
If it's not maintained, could you please remove the "ddev-get" topic from the repo so it's not discoverable?
Thanks,
-Randy

The tests were disabled automatically because the project had no new commits for a long time. Now I updated the GitHub workflow to keep them active, and released a new version with improved tests.


fix in setMeasures, combined with thumbnails without set gallery dime…

When using the gallery with thumbnails, the gallery was sometimes displayed correctly. Meaning the gallery got resized to the available width. Most of the times however, the gallery got resized to the width of a thumbnail.
It turned out, that whether it workes or not depended whether the thumb or main index-0 pic was returned from the server first.
This change does fix this problem.

This change does fix this problem.

I received an report that this issue still persists. Unfortunately, I'm still not able to reproduce the issue somewhat reliably (it occurs only very rarely for me), so I can't check for myself.


JAMES-3904 Support LDAPS

04:00:43,824 [ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.1.2:check (check-style) on project james-server-data-ldap: You have 4 Checkstyle violations. -> [Help 1]


Example curl script to download the important files in this repo

For kickstarting new projects.

Completed via: 2d2478495e6b9ec0740ed982bf88747958a7b204


Allow path "/" to behave as default resource.

Description:
Currently "/*" is considered as default resource path. Ideally it shoud be "/" as its the root of the resource.
@http:configuration {
    basePath:"/abc"
}
service<http> hello {
    @http:resourceConfig {
        path:"/"
    }
    resource sample1 (http:Connection conn, http:InRequest req) {
        //
    }

    @http:resourceConfig {
        path:"/xyz/*"
    }
    resource sample2 (http:Connection conn, http:InRequest req) {
        //
    }
}

curl http://localhost:9090/abc
curl http://localhost:9090/abc/
curl http://localhost:9090/abc/id
curl http://localhost:9090/abc/id/product

Above requests should be dispatched to sample1
curl http://localhost:9090/abc/xyz
curl http://localhost:9090/abc/xyz/biz

Above requests should be dispatched to sample2

It was decided to keep the current implementation as it is during the HTTP API Review [1]. Hence closing the issue.
[1] Updated invitation: [Ballerina] HTTP API review @ Tue Apr 3, 2018 2pm - 4pm (IST)


Test dj110

create builds to start testing out django 1.10

Current coverage is 100% (diff: 100%)

Merging #2 into master will not change coverage

@@             master    #2   diff @@
=====================================
  Files             2     2
  Lines            57    57
  Methods           0     0
  Messages          0     0
  Branches          6     6
=====================================
  Hits             57    57
  Misses            0     0
  Partials          0     0

Powered by Codecov. Last update bce3e68...1ab20f0

Coverage remained the same at 100.0% when pulling 1ab20f03daf42a2bc7370d87ccdf9df27f6f80be on test-dj110 into bce3e686a5fd7fa68af83967fb290bb662f9a86f on master.


Changes on context menu

Fix #227
Update snippets

@andriyko Please provide any feedback you may have. If there is no any comment, please merge this.

The change looks good to me. @aaltat , I would like to merge this PR. Could you please take a look at this as well?

Well, that is a very good question. I don't have the time to maintain this project, unfortunately, but true. SeleniumLibrary eats all my time reserved for doing open source projects. Although I do care about this project, my main focus is in SeleniumLibrary.
Therefore I would be happy to give this project responsibility to someone else. Would anyone of you two willing to take the responsibility? I can, if needed, explain the architecture and deeper design principles to the next maintainer (and shead light to the dark corners, there are few design things which I am not proud of.) Also I am not going anywhere from the Robot Framework ecosystem, I am around to answer questions also after the transfer.

I'm happy to keep this plugin alive though. It's the best plugin comparing to the others out there in vscode or pycharm. However, my python knowledge is quite limited and may not be enough to be the maintainer.

@idxn , thank you for your contribution 👍 .
@aaltat , thank you for the great work done on this project 💯

Well @idxn we could have few sessions about the plugin architecture. But before doing that you could look at Sublime plugin API https://www.sublimetext.com/docs/3/api_reference.html and http://docs.sublimetext.info/en/latest/extensibility/plugins.html After that we could discuss about this plugin itself. Are you in Slack?

@aaltat I have only my work slack tanakiat at central.tech

Robot Framework community has also Slack in use, would it be possible to see also there? If yes, please see more details in contact section in https://github.com/robotframework/robotframework/blob/master/README.rst#support-and-contact for how to join.


Support running on non-English Windows system

Description

rakelkar/gonetsh bases on the console output of the netsh command, this would not respect the Windows system which is using non-English language. I decided to write a new component to support this: https://github.com/thxCode/winnet.
Release Note

Replaced rakelkar/gonetsh with thxcode/winnet to support the non-English language Windows system.

@rakelkar , @benmoss , @rajatchopra , please take a look.

Is there a good way of getting the supporting packages to come from an org repo? Moving between vendor/rkelkar to vendor/benmoss vs vendor/thxcode does not seem like a lasting solution.
@ksubrmnn @madhanrm Opinions please.
/cc @squeed

Closing in favor of #1238 .


New version: Readdle.Spark version 3.16.7 (<IP_ADDRESS>552)

Created by 🥟 Dumplings in workflow run #6370.

Log
Updated: <IP_ADDRESS>393 → <IP_ADDRESS>552
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

/azp run

https://github.com/microsoft/winget-pkgs/issues/142854


setTrimQuotes does not trim quotes

        final AgentConfig config = new AgentConfig();
        final CommandLine commandLine = new CommandLine(config);
        commandLine.setTrimQuotes(true);
        commandLine.setSplitQuotedStrings(true);
        commandLine.setCaseInsensitiveEnumValuesAllowed(true);
        try
        {
            commandLine.parse(args);
        }
        catch (CommandLine.ParameterException e)
        {
            System.err.println(e.getMessage());
            commandLine.usage(System.err);
            return;
        }

[picocli DEBUG] Creating CommandSpec for object of class com.package.AgentConfig with factory picocli.CommandLine$DefaultFactory
[picocli INFO] Picocli version: 4.0.0-beta-1b, JVM: 1.8.0_202 (Oracle Corporation Java HotSpot(TM) 64-Bit Server VM 25.202-b08), OS: Linux 4.15.0-51-generic amd64
[picocli INFO] Parsing 8 command line args ["--env", "ACCEPTANCE_MULTIBOX", "--command", "DEPLOY", "--release-revision", "0.28.0-ci", "--servers", "<IP_ADDRESS>-LARGE,<IP_ADDRESS>-LARGE,<IP_ADDRESS>-LARGE,<IP_ADDRESS>-LARGE,<IP_ADDRESS>-LARGE,<IP_ADDRESS>-LARGE"]
[picocli DEBUG] Parser configuration: posixClusteredShortOptionsAllowed=true, stopAtPositional=false, stopAtUnmatched=false, separator=null, overwrittenOptionsAllowed=false, unmatchedArgumentsAllowed=false, expandAtFiles=true, atFileCommentChar=#, useSimplifiedAtFiles=false, endOfOptionsDelimiter=--, limitSplit=false, aritySatisfiedByAttachedOptionParam=false, toggleBooleanFlags=false, unmatchedOptionsArePositionalParams=false, collectErrors=false,caseInsensitiveEnumValuesAllowed=true, trimQuotes=true, splitQuotedStrings=true
[picocli DEBUG] (ANSI is disabled by default: isatty=false, XTERM=null, OSTYPE=null, isWindows=false, JansiConsoleInstalled=false, ANSICON=null, ConEmuANSI=null, NO_COLOR=null, CLICOLOR=null, CLICOLOR_FORCE=null)
[picocli DEBUG] Set initial value for field com.package.environment.Environment com.package.AgentConfig.environment of type class com.package.environment.Environment to null.
[picocli DEBUG] Set initial value for field com.package.command.Command com.package.AgentConfig.command of type class com.package.command.Command to null.
[picocli DEBUG] Set initial value for field String com.package.AgentConfig.releaseRevision of type class java.lang.String to SNAPSHOT.
[picocli DEBUG] Set initial value for field String com.package.AgentConfig.backupId of type class java.lang.String to null.
[picocli DEBUG] Set initial value for field java.util.List<String> com.package.AgentConfig.serviceNames of type interface java.util.List to [].
[picocli DEBUG] Set initial value for field java.util.List<String> com.package.AgentConfig.hosts of type interface java.util.List to [].
[picocli DEBUG] Set initial value for field boolean com.package.AgentConfig.force of type boolean to false.
[picocli DEBUG] Set initial value for field java.util.List<com.package.environment.HostInfo> com.package.AgentConfig.serversForDynamicTopologies of type interface java.util.List to [].
[picocli DEBUG] Set initial value for field java.util.List<java.util.Map$Entry<String, String>> com.package.AgentConfig.configProperties of type interface java.util.List to [].
[picocli DEBUG] Initializing com.package.AgentConfig: 9 options, 0 positional parameters, 2 required, 0 groups, 0 subcommands.
[picocli DEBUG] Processing argument '"--env"'. Remainder=["ACCEPTANCE_MULTIBOX", "--command", "DEPLOY", "--release-revision", "0.28.0-ci", "--servers", "<IP_ADDRESS>-LARGE,<IP_ADDRESS>-LARGE,<IP_ADDRESS>-LARGE,<IP_ADDRESS>-LARGE,<IP_ADDRESS>-LARGE,<IP_ADDRESS>-LARGE"]
[picocli DEBUG] '"--env"' cannot be separated into <option>=<option-parameter>
[picocli DEBUG] Could not find option '"--env"', deciding whether to treat as unmatched option or positional parameter...
...

```java
 @CommandLine.Option(names = {"--env", "-e"}, description = "Environment to execute on", required = true)
    private Environment environment;

    @CommandLine.Option(names = {"--command", "-c"}, description = "Command to execute", required = true)
    private Command command;
...

That looks like a bug, I will take a look as soon as I can.
Thanks for the detailed report!

The cause is that picocli currently only strips quotes from option parameters and positional parameters, not from option names.
I will fix that so that quotes are stripped from all arguments as they are processed.
Just out of interest, can I ask why the option names are quoted in your use case?

I was migrating from JCommander and noticed the break. There is no reason that I'm aware of to quote the options so I've removed it to solve my problem, however the behaviour was very unexpected.

Agreed that it is inconsistent to not strip quotes from named options, :grin: I will fix this before the 4.0 release. Thanks for raising this! :pray:
I'm actually quite interested in use cases involving quotes. Usually the shell removes quotes before passing parameters to the application, so I was wondering how these quotes came to be there.  :thinking:  Any background is useful to know.

I am also interested in general feedback on the experience migrating from JCommander. (If you have time)

We are executing the process from Runtime.getRuntime().exec() and were quoting all the arguments, there may have been a historical reason for this but it works now in our CI environment.
Generally migrating was simple, it was a drop in replacement, the only issue was the quoting.

Another difference with JCommander is Lists are automatically created from a comma separated list of args where picocli requires splitter ","

I have finally made some progress with this issue. The parser will now be able to correctly handle cases like this:

"-x" "abc"  -> recognises value abc for option -x
"a,b","x,y" -> correctly splits into 2 values (not 4): a,b and x,y
"-x" a,b,"c,d,e",f -> correctly finds 4 values for option -x: first is a, second is b, third is c,d,e and last is f

I also updated the user manual section on how the parser handles quotes.
However,  values with nested quotes like this still give it trouble:

"-x=a,b,\"c,d,e\",f" -> unmatched argument error
"-x" "a,b,\"c,d,e\",f" ->  finds 3 values for option -x: first is a,b,c, second is d, and last is e,f

I need to think further about how to deal with this. Ideas welcome. :-)

@juddgaddie Your example uses setSplitQuotedStrings. That made me realize that the documentation for this backwards-compatibility switch was hugely inadequate. I have updated the user manual and javadocs, but I am considering deprecating this method. Can you take a look at https://github.com/remkop/picocli/issues/760 and comment there?

The cases I would expect the parser to handle for this are:
As above:
"-x" "abc" -> recognises value abc for option -x
"a,b","x,y" -> correctly splits into 2 values (not 4): a,b and x,y
For
"-x" "a,b,c" -> to convert to 3 values for option -x?
For the other cases I would not expect the behaviour to be defined, if it is great..

Thanks for the update. I was able to find a solution that handles nested quotes if they are escaped correctly.  Now writing docs.


bringing direct_propagation() up-to-date

for debugging/comparison with plane-to-plane, from working issue: #106

Issue #106 identified an error in the plane-to-plane propagation. This function implements the direct propagation algorithm given in Andersen and Enmark, which was originally included for exactly this purpose of testing the POPPY implimentation of the plane-to-plane algorithm.
Example code snippet:
import poppy
from poppy import fresnel
import astropy.units as u
z=5000*u.m
wf_direct=fresnel.Wavefront(2*u.m,wavelength=2200e-9,npix=1024,oversample=4)
wf_direct *= poppy.CircularAperture(radius=0.5,oversample=wf_direct.oversample)
wf_direct.propagate_direct(z)

gives a result that agrees well with @mperrin's bug fix and PROPER result:

Thanks. The above code would make a nice unit test, to run both the direct and PTP algorithms and check that the output result is consistent.
In what situations do you think it is preferable to use the direct as opposed to the PTP algorithm? Is there a clear winner for speed or numerical accuracy, or anything like that? I am not that familiar with this direct algorithm.


Create Password screen should conform to iOS 11 standards

Here's the relevant info on the subject:
https://developer.apple.com/documentation/security/password_autofill/#topics
I've just tried to register and couldn't use the built in password generator.

Wow, thanks for the issue and link to the docs! Looks pretty easy to integrate, especially into our existing on boarding flow setup. I'll open a PR 👍

Awesome!💪


Support Carthage

Before this PR get merged, you can integrate with
github "cybertk/FLKAutoLayout" "carthage"

This PR also fixed #26

This is handled in #45 👍


feat: warn when font weights are not available

This pull request warns users when some of the font weights are not available, or throws an error when none of the font weights are available (as the latter situation causes a 400 error with an empty array).

Nice idea! Do you have an example of a setup that would trigger this?

@danielroe Sure. If you add weights: [500, 700] to the config in playground, it would be triggered.

Total config is as belows:
export default defineNuxtConfig({
  modules: ['@nuxt/fonts', '@nuxtjs/tailwindcss', '@unocss/nuxt'],
  unocss: {
    disableNuxtInlineStyle: false,
  },
  fonts: {
    providers: {
      custom: '~/providers/custom'
    },
    families: [
      { name: 'Abel', provider: 'bunny' },
      { name: 'Satoshi', provider: 'fontshare' },
      { name: 'Kode Mono', provider: 'none' },
      { name: 'MyCustom', src: '/custom-font.woff2' },
      { name: 'CustomGlobal', global: true, src: '/font-global.woff2' },
      { name: 'Oswald', fallbacks: ['Times New Roman'] },
    ],
    defaults: {
      weights: [500, 700],
      fallbacks: {
        monospace: ['Tahoma']
      }
    }
  },
})


What's the evaluation config setting for the final evaluation ?

I noticed that several parameters were important to final result, such as extract_final_answer_by_prompting_again. Wondering whether you could post the final config setting for the final evaluation for reproducible result on GSM_8K reported from the original paper ?

Thanks for the question! The settings we used for each externally accessible model (text-bison and GPT models) can be found at around https://github.com/google-deepmind/opro/blob/38af4621f784dc7e60044018c80e233d169467b4/opro/optimization/optimize_instructions.py#L689 for prompt optimization, and https://github.com/google-deepmind/opro/blob/38af4621f784dc7e60044018c80e233d169467b4/opro/evaluation/evaluate_instructions.py#L241 for prompt/instruction evaluation.
The settings we used for internal models are:

Pre-trained PaLM 2-L as scorer:

temperature=0.0, max_decode_steps=256, extract_final_answer_by_prompting_again=True, old_instruction_score_threshold=0.01

Pre-trained PaLM 2-L as optimizer:

temperature=1.5, max_decode_steps=256

PaLM 2-L-IT as optimizer:

temperature=1.0, max_decode_steps=1024
Basically, extract_final_answer_by_prompting_again should be set to True when the scorer model is a pre-trained model. When it's an instruction-tuned model, setting it to either True or False should work as long as the outputs follow similar patterns, like "So the answer is xxx".
Note that the exact numbers may not be reproducible because the PaLM 2-L model is Google-internal, and the externally callable text-bison model has been regularly updated. But you should at least be able to get similar numbers with text-bison scorer if you follow the settings linked above.


Fix styling on new-stuff bar

This was the easiest fix. If you have other ideas for how to display the new stuff are, let me know.
Also, since us developers are going to see this forever and some of us are OCD, I fixed the New New annoyingness :)

Nice.  Better is better.  This is the kind of pointed fix that's right smack dab firmly in the spirit of a cleanup week.  Same thing we got, but less broken. :heart:

:+1:

:+1:


Error and also no data in stats

Hi.
Great bundle first of all. I've installed on laravel Laravel Framework version 5.1.25 (LTS)
I manage view the status but they are empty.
Also in my tracker.php i have
'authentication_ioc_binding' => ['auth'],
because with 'authentication_ioc_binding' => ['auth', 'admin'], a get  Class admin does not exist
But after i use php artisan config:cache i get this error http://pastebin.com/Qetnyn9X
This is my config: http://pastebin.com/75PVgQQv
Please let me know how can i bypass this error and also have data in stats
best regards

It seems that if this is true     'log_users' => false, then i have this error 	Illegal offset type in isset or empty

If is false is working well

@antonioribeiro is there a way i can use log_users true and without errors? Thanks

@antonioribeiro any idee how can i use log_users without errors?
Thanks in advance for your time

Hi there,
We are trying to install tracker in our project but we have the same issue.
We are using Laravel 5.1.29. But our error is located in Container.php line 160:
"ErrorException in Container.php line 160: Illegal offset type in isset or empty".
public function isAlias($name)
{
160:        return isset($this->aliases[$name]);
}

Did you get any solution to this problem?
We are thinking that our problem could be around the vinkla/laravel-hashids package that we install to encrypt the ids for secure the urls. Do you use this package also?
Thanks.

Got this error and this one #218
I will create a merge request soon.


CCS renamed hyperparameter to parameter.

This updates CCS support to reflect the changes from https://github.com/argonne-lcf/CCS/pull/15, and will need to be merged if we adopt it.

This is now required since https://github.com/argonne-lcf/CCS/pull/15 has been merged.


Minimal working example of Lilypond lua filter with markdown

Hi, I'm trying to use the lilypond filter to try to convert lilypond code inside a .md file into embedded images.
I'm tryind to test it with a simple example, but I cannot make it to work. I downloaded lilypond.lua into my working directory and this is my file test.md
# Markdown file

Hi

\version "2.20.0"
{
  c' e' g' e'
}

And I call pandoc using pandoc --lua-filter=lilypond.lua test.md -o test.pdf. The error mesage is
Error producing PDF.
! Undefined control sequence.
l.60 \version

I also tried removing the \version line. Then it compiles and generated a pdf, but without image. My pandoc version is 2.11.4, and mi lilypond version si 2.20.0, running on Ubuntu 20.04.
Sorry if the question is stupid (I'm a newbie on both pandoc and lilypond), but I read lilypond's filter README.md and I cannot figure it out. Could you provide a minimal working example?

The lilypond directory has two sample files used for testing, oboe.md and appoggiaturas.md. The exact test command can be found in Makefile, but I agree that this should be made easier to find.

Thank you! Now that you say it, it was deducible, but probably a line in the readme briefly mentioning that those files are examples that can be compiled with make would be nice for the next newbie coming here. :)

Lets keep this issue open until the documentation is fixed.

I'm not sure what I'm missing. But following the advice from PR #249 I run:
pandoc --to=html --lua-filter=lilypond.lua --output="test.html" appoggiaturas.md

whereas I get the following error message:
Error running filter lilypond.lua:
PandocIOError "pipe" lilypond: createProcess: posix_spawnp: does not exist (No such file or directory)
stack traceback:
        lilypond.lua:49: in upvalue 'generate_image'
        lilypond.lua:83: in upvalue 'process_lilypond'
        lilypond.lua:143: in function <lilypond.lua:138>

Looks like pandoc cannot find the lilypond binary; it has to be installed separately.

The lilypond executable must be installed to a location on your PATH

Oh sorry, completely missed that one :sleepy:

You are right though: the error message is rather cryptic and should probably be improved.


UNMET PEER DEPENDENCY<EMAIL_ADDRESS>

Hi,
I am not able to install the component cause I get this error:
+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>`--<EMAIL_ADDRESS>    `-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>|| ^0.14.0 || ^15.0.0

Any help?
Thanks.

Fixed in master


[CPP_RPC][ANDROID] Fix cpp_rpc build failure

CPP build failure for Android devices with NDK version < 23.
NDK below 23 require additional linker flags.

This PR address the rpc build caused by https://github.com/apache/tvm/pull/13118

I used ENV as in CI dockers this is the way we can supply the NDK version.
For individual compilations  exporting in environment or prefixing NDK version should work.
ANDROID_NDK_MAJOR=21 cmake -DCMAKE_TOOLCHAIN_FILE=Android/Sdk/ndk/22.1.7171670/build/cmake/android.toolchain.cmake -DANDROID_ABI=arm64-v8a -DANDROID_NATIVE_API_LEVEL=android-23 -DCMAKE_FIND_ROOT_PATH_MODE_PACKAGE=ON -DANDROID_STL=c++_static ..

@srkreddy1238 You only need to set the toolchain file. ANDROID_NDK_MAJOR is automatically available in CMakeLists.txt once you specify the toolchain file. I think it is better to set the toolchain file location using ENV, not ANDROID_NDK_MAJOR.

https://github.com/apache/tvm/blob/b807613c794321bf4fa765df026b1c3b454811b3/tests/scripts/task_build_adreno_bins.sh#L42
Tool chain is set.

made ANDROID_NDK_MAJOR to be an optional env variable to keep higher NDK versions work default.


Providing values from ConfigMap and Secret resources

To be able to provide values by other means than in-lining them in the HelmRelease resource. We want to provide users with the option to reference to values in ConfigMap and Secret resources.
Spec changes
diff --git a/docs/spec/v2alpha1/helmreleases.md b/docs/spec/v2alpha1/helmreleases.md
index 5c13e45..086f624 100644
--- a/docs/spec/v2alpha1/helmreleases.md
+++ b/docs/spec/v2alpha1/helmreleases.md
@@ -70,6 +70,10 @@ type HelmReleaseSpec struct {
 	// +optional
 	Uninstall Uninstall `json:"uninstall,omitempty"`

+	// ValuesFrom holds references to values in Secrets and ConfigMaps.
+	// +optional
+	ValuesFrom []ValuesReference `json:"valuesFrom,omitempty"`
+
 	// Values holds the values for this Helm release.
 	// +optional
 	Values apiextensionsv1.JSON `json:"values,omitempty"`
@@ -96,6 +100,30 @@ type HelmChartTemplate struct {
 	Interval *metav1.Duration `json:"interval,omitempty"`
 }

+// ValuesReference contains enough information to let you locate the
+// values reference at namespace level, and where the (sub)set of
+// values should be merged at.
+type ValuesReference struct {
+	// Kind of the values referent.
+	// +kubebuilder:validation:Enum=Secret,ConfigMap
+	// +required
+	Kind string `json:"kind"`
+
+	// Name of the values referent.
+	// +required
+	Name string `json:"name"`
+
+	// ValuesKey is the key in the referent the values can be found at.
+	// Defaults to 'values.yaml'.
+	// +optional
+	ValuesKey string `json:"valuesKey,omitempty"`
+
+	// TargetPath is the YAML dot notation path the values should be merged at.
+	// Defaults to 'None', which results in the values getting merged at the root.
+	// +optional
+    TargetPath string `json:"targetPath,omitempty"`
+}
+
 // Install holds the configuration for Helm install actions.
 type Install struct {
 	// Timeout is the time to wait for any individual Kubernetes operation (like Jobs

Changes vs Helm Operator's valuesFrom

Values from external sources, for example an URL, are no longer supported.
This choice was made because you can not reliably restore the cluster state from Git repositories if the configuration of a service relies on some URL being available. In addition to this, having the optional param is even more problematic, as the cluster state will change if the URL is briefly unavailable during reconciliation.

Chart file references are no longer supported, this will instead be part of the HelmChart API. https://github.com/fluxcd/helm-controller/issues/4

It is possible to define a targetPath where the values should be merged at. This works the same as --set <YAML dot notation path>=<value>. With this, the following Helm action values:
helm install \
  --set-file global.identityTrustAnchorsPEM=ca.crt \
  --set-file identity.issuer.tls.crtPEM=issuer.crt \
  --set-file identity.issuer.tls.keyPEM=issuer.key \
  --set identity.issuer.crtExpiry=$(date -d '+8760 hour' +"%Y-%m-%dT%H:%M:%SZ") \
  linkerd/linkerd2

Can be created as a secret using kubectl:
kubectl create secret generic cert-values \
  --from-file=ca.crt=./certs/ca.crt \
  --from-file=issuer.crt=./certs/issuer.crt \
  --from-file=issuer.key=./certs/issuer.key \
  --from-literal=crtExpiry=$(date -d '+8760 hour' +"%Y-%m-%dT%H:%M:%SZ")

And then be declaratively defined in the HelmRelease as:
spec:
  valuesFrom:
  - kind: Secret
    name: cert-values
    valuesKey: ca.crt
    targetPath: global.identityTrustAnchorsPEM
  - kind: Secret
    name: cert-values
    valuesKey: issuer.crt
    targetPath: identity.issuer.tls.crtPEM
  - kind: Secret
    name: cert-values
    valuesKey: issuer.key
    targetPath: identity.issuer.tls.keyPEM
  - kind: Secret
    name: cert-values
    valuesKey: crtExpiry
    targetPath: identity.issuer.crtExpiry

Behavioral notes

Referenced values should be watched, and changes should trigger a reconciliation of the HelmRelease. This may require some documentation around the fact that this may need some coordination during changes to multiple referenced resources, either by:

A pattern of creating new ConfigMap and Secret resources each time using a hash in the name.
Combining values in the same file that are likely to receive updates at the same time.

Tasks

[x] Support referring to ConfigMap and Secret resources
[x] Support merging at TargetPath level
[ ] Watch referred resources

Discussions / references

https://github.com/fluxcd/toolkit/discussions/100

This choice was made because you can not reliably restore the cluster state from Git repositories if the configuration of a service relies on some URL being available.

We could document a workaround for migration.
A user could create a CronJob in the same namespace that curls a URL into a ConfigMap using a config-editor ServiceAccount.
The HelmRelease may then depend on that.
The CronJob can choose what to do when the URL becomes unavailable which allows the user to decide how the error should be handled separately from whether the valueRef's is optional.

having the optional param ...

Do we already support optional vs. required valueRefs?
I don't see it in the current API.
I was thinking we could basically take the same optional field from Pod's ConfigMapEnvSource
It should be noted that even if we are no longer supporting URL's, ConfigMap/Secrets's with values may often not come from the same gitRepo or even a gitRepo at all.
They may come from another gitRepo, be placed in the cluster manually (example above), or be the result of some other controller (ex: cert-manager).
Should we support explicitly optional valueRefs?

optional valueRefs would allow helmReleases to change config based on the live state of the cluster.
This could be useful for allowing a users to extend behavior at a later date with their own repo without modifying the HelmRelease object.
As long as the user can create/modify the referenced Configs, they can turn on/off features and etc.
This works kind of like a UNIX style /config.d/ dir.
I do a similar thing with volumes:
I like to modify my CoreDNS config so that adding keys to a separate coredns-corefile-extras ConfigMap ends up appending to the config from other repos.
This is done through an optional volumeMount for /etc/corefiles.d/.

@stealthybox Hello! I would like to offer my help if I can on getting the watch working on the respective resources?

Hi guys! Not sure where this Watching feature is but when values in ConfigMaps or Secrets change there goes no release upgrades and I have not found anything better rather than do flux delete hr <name> to apply the changes which isn't the desired way of course.

If you're using valuesFrom with a secret ref or configmap the flux reconcile hr command should pick up the changes, or automatically be picked up on the next reconciling event. @XtremeAI is that not the case?
If so you should provide specific reproduction instructions as you may have found a new bug, that is not the subject of this issue report. The watching functionality is only meant to accelerate this behavior, it should be happening on its own if you are patient or if you manually trigger the HelmRelease reconciling.

Awesome
I think the recreate-pods is only supported through rollback spec in Helm Controller. That root cause link you mentioned @XtremeAI is perfect, you can enforce the pods recreation through use of an annotation that changes based on the shasum of the configmap. Then you also get the benefit of only recreating pods when the configmap has actually changed under it!

Something got me really confused with the optional flag. I though that optional was saying that if the values was not found, either the Secret/ConfigMap object OR the valuesKey in the Secret/ConfigMap, it didn't matter.
After some testing, I understand that optional is really only saying that if the Secret or ConfigMap does not exist we don't care but if it does exist, you better have that valuesKey defined in those Secret/ConfigMap otherwise, reconciliation will fail.
That feels counterintuitive. If the valuesFrom is really optionnal, reconcilliation should be successful even if the valuesKey does not exist.
Any though ?

Before I create a new feature request. helm-operator allowed you to specify the source namespace for configmaps and secrets that were used in the valuesFrom.  This was very useful, and I'm wondering if there is a specific reason that functionality was removed?

See https://fluxcd.io/flux/migration/helm-operator-migration/#values-from-configmap-and-secret-resources-in-other-namespaces-are-no-longer-supported

I noticed that when the file I used to create a config can not contain any commas.
kubectl create configmap grafana-config --from-file=grafana.ini

# HelmRelease

spec:
  valuesFrom:
  - kind: ConfigMap
    name: grafana-config
    valuesKey: grafana.ini
    targetPath: grafana.grafana\.ini

# grafana.ini

# hi there here is an okay comment
val1: 1
val2: 2
# bad comment with, a comma. or something []
val3: 3

Otherwise, I'd get

unable to merge value from key 'grafana.ini' in ConfigMap 'monitoring/grafana-config' into target path grafana.grafana.ini':  key map " a comma" has no value


Update and sync copy edits of learn.microsoft.com in azure.yaml.json

https://github.com/Azure/azure-dev/blob/main/schemas/v1.0/azure.yaml.json#L67
                        "description": "Optional. If not specified, the resource name will be constructed from current environment name concatenated with service name (<environment-name><resource-name>, for example 'prodapi')."
From @wbreza, "is not true.  It will default to the behavior of the bicep implementation, but the user can override the name that should be used or azd will infer it from tag name azd-service-name matching the name/key of the service in the azure.yaml."

@puicchan Do you know if this is still needed?

An old issue. No longer relevant. Closing.


Confusing DeliveryMethod comments in NetConstants.cs

I'm confused with the comments on DeliveryMethod in NetConstants.cs:

Unreliable = Packets can be dropped, duplicated, arrive without order
ReliableUnordered = Packets won't be dropped, won't be duplicated, arrive without order
Sequenced = Packets can be dropped, won't be duplicated, arrive in order
ReliableOrdered = Packets won't be dropped, won't be duplicated, arrive in order
ReliableSequenced = Packets can be dropped (except the last one), won't be duplicated, arrive in order

Can someone confirm if my understanding is correct? Can I update the comments for future developers? (I noticed @Korseus also got confused)

A couple of corrections, If I may:

Unreliable = Packets can be dropped, can be duplicated, can arrive without order.
ReliableUnordered = Packets won't be dropped, won't be duplicated, can arrive without order.
Sequenced = Packets can be dropped (except the last one), won't be duplicated, will arrive in order.
ReliableOrdered = Packets won't be dropped, won't be duplicated, will arrive in order.
ReliableSequenced = Packets can be dropped (except the last one), won't be duplicated, will arrive in order

Sequenced = Packets can be dropped (except the last one), won't be duplicated, will arrive in order.
Last one can be dropped too (because UDP) this is main differenece to ReliableSequenced (that reliable)


can you clarify variable names

is global_phi is the canonical tsdf volume?
what is phi_n in the paper literature?
what's phi_n_psi?
what's phi_global_psi_inv?

hi! same notation as in the paper--phi_global is the canonical tsdf, phi_n is the live projective sdf, phi_n_psi is the live projective sdf after applying the deformation field, and phi_global_psi_inv is the canonical tsdf after applying the inverse deformation field


AOT/ngc: config._d.getTime is not a function -> can't import Modules from 'ng2-bootstrap/ng2-bootstrap'

when I try import { CarouselModule } from 'ng2-bootstrap/ng2-bootstrap' it builds correctly but I get the following runtime error:

The same happens for the TooltipModule (haven't tested any others).
If i use import { CarouselModule } from 'ng2-bootstrap/component/carousel' however it works fine.
I see in the documentation  that one can use either method?  why does one of them fail?
NOTE: I am using ngc for AOT compilation & Rollup for tree shaking prior to seeing the error, haven't tested without.

u should post your package.json。
we need know version of package

Sorry for the delay in response, I am using the latest version on npm: 1.1.16
here are my dependencies:
  "dependencies": {
    "@angular/common": "2.0.2",
    "@angular/compiler": "2.0.2",
    "@angular/compiler-cli": "^0.6.2",
    "@angular/core": "2.0.2",
    "@angular/forms": "2.0.2",
    "@angular/http": "2.0.2",
    "@angular/platform-browser": "2.0.2",
    "@angular/platform-browser-dynamic": "2.0.2",
    "@angular/upgrade": "2.0.2",
    "angular2-infinite-scroll": "^0.2.1",
    "bootbox": "^4.4.0",
    "bootstrap": "^3.3.7",
    "core-js": "^2.4.1",
    "es6-shim": "^0.35.1",
    "file-saver": "^1.3.3",
    "hopscotch": "^0.2.5",
    "jquery": "^2.2.0",
    "knockout": "^3.4.0",
    "knockout-mapping": "^2.6.0",
    "knockout-secure-binding": "^0.5.5",
    "lodash": "^4.12.0",
    "ng2-bootstrap": "^1.1.16",
    "optimal-select": "^3.3.0",
    "reflect-metadata": "^0.1.8",
    "rxjs": "5.0.0-beta.12",
    "rxjs-es": "5.0.0-beta.12",
    "systemjs": "0.19.39",
    "zone.js": "^0.6.25"
  },

let me know if I can provide any more info

@goleary can you drop rollup config? and search issues about rollup? usually commonjs plugin is missing

I searched around and tried the solution here: https://github.com/valor-software/ng2-bootstrap/issues/1188 but it didn't fix the problem.
I wouldn't say this is high priority as I already used the other import statement to get it working, I'm just curious why it isn't working as advertised in the docs
rollup config:
import typescript from 'rollup-plugin-typescript';
import nodeResolve from 'rollup-plugin-node-resolve';
import commonjs from 'rollup-plugin-commonjs';

class RollupNG2 {
  constructor(options){
    this.options = options;
  }
  resolveId(id, from){
    if (id.indexOf('rxjs/') >= 0){
      console.log(id, from);
    }
    if (id.startsWith('rxjs/')){
      var result = `${__dirname}/node_modules/rxjs-es/${id.replace('rxjs/', '')}.js`;
      if (!__dirname.startsWith("/")) {
        return result.replace(/\//g, "\\");
      }
      return result;
    }
  }
}

const rollupNG2 = (config) => new RollupNG2(config);

export default {
  sourceMap: false, //to figure out how to use sourceMap later
  plugins: [
    typescript(),
    rollupNG2(),
    nodeResolve({
      jsnext: true,
      main: true,
      browser: true
    }),
    commonjs()
  ]
}

from v1.1.16-9 and ng v2.4
works fine with AoT
new docs preview http://valorkin.github.io/ng2-bootstrap/#/

Thanks, I'll try the upgraded version later today!

just curious, I've never seen major.minor.patch-# versions on npm before.  I had to remove and reinstall the package just to get the new code (instead of updating as I am using 6 operator on version in package.json).  Is this to be expected or is there some problem?

This versioning is achieved via npm version pre-release, actually it is just a semver
You can expect some breaking changes in how you import modules. And removed hack for modals. All else is mostly the same + popover, a lot of fixes, tests and docs


fixed run.sh and run.cmd to accept variable-length CLI parameters

Fixes #196
The Solution
The solution is plain: run.cmd and run.sh have to consider variable-length CLI parameters, not just the first of them.
*nix
Use "$@" instead of $1.
Windows
Use %* instead of $1
Tests
Program that Consumes Variable-Length CLI Parameters
The first test is the program described in #196
*nix
Before
./run.sh 1 2 3 4 5
Exception in thread "main" java.lang.IllegalArgumentException: Can't get() the 1th element of the array, there are just 1 of them
  at org.eolang.EOarray$EOget.lambda$new$0(EOarray$EOget.java:53)

After
./run.sh 1 2 3 4 5
1
2
3
4
5

Windows
Before
> .\run.cmd 1 2 3 4 5
>>> java -cp "target/classes;target/eo-runtime.jar" org.eolang.phi.Main sandbox.app $1
Exception in thread "main" java.lang.NumberFormatException: For input string: "$1"
at java.lang.NumberFormatException.forInputString(Unknown Source)
at java.lang.Long.parseLong(Unknown Source)
at java.lang.Long.parseLong(Unknown Source)

After
> .\run.cmd 1 2 3 4 5                                                                                >>>java -cp "target/classes;target/eo-runtime.jar" org.eolang.phi.Main sandbox.app 1 2 3 4 5
1
2
3
4
5                                                                                >>>pause
Press any key to continue . . .

Program that Consumes One CLI Parameter
This is a regression test. Here, we test that the sandbox/eo/app.eo still works (this is the Fibonacci example).
*nix
Before
$ ./run.sh 10
10th Fibonacci number is 55
$

After
$ ./run.sh 10
10th Fibonacci number is 55
$

Windows
Before
> .\run.cmd 10
>>> java -cp "target/classes;target/eo-runtime.jar" org.eolang.phi.Main sandbox.app $1
Exception in thread "main" java.lang.NumberFormatException: For input string: "$1"
at java.lang.NumberFormatException.forInputString(Unknown Source)
at java.lang.Long.parseLong(Unknown Source)
at java.lang.Long.parseLong(Unknown Source)

After
> .\run.cmd 10                                                                                >>>java -cp "target/classes;target/eo-runtime.jar" org.eolang.phi.Main sandbox.app 10
10th Fibonacci number is 55
>>>pause
Press any key to continue . . .

@nlchar thanks!


Parameter must be an array or an object that implements Countable - PHP 7.2

I'm getting the following error when using a single-product.twig.
Warning: count(): Parameter must be an array or an object that implements Countable in /app/public/wp-includes/post-template.php on line 285

The $pages array is null at the point where the_content() is called in  wp-content/plugins/woocommerce/templates/single-product/tabs/description.php
It appears this is a common error when the_content() is called outside of the loop. but as far as I can see it should be in it. At least it is when using a non-timber theme such as TwentySixteen.
I'm using the timber start theme.

Sorry for the late answer!
You are right, this happens when the_content() or get_the_content() is called outside of the loop. This is probably an issue with WordPress itself, see https://core.trac.wordpress.org/ticket/42814.
I’d advise you to use a custom Twig template for the description tab. Add a views/woocommerce/single-product/tabs/description.twig template to your theme with the following contents:
{% set heading = __('Description', 'woocommerce')
    |apply_filters('woocommerce_product_description_heading')
    |e('esc_html')
%}

{% if heading %}
    <h2>{{ heading }}</h2>
{% endif %}

{{ post.content }}

When we use $post->content(), there shouldn’t be a problem.


Manage deps (vendor directory)

Quoting Introducing client-go version 6:

dep does not currently respect Godeps/Godeps.json in a consistent way, especially not on updates. It is crucial to specify client-go dependencies manually as constraints or overrides, also for non k8s.io/* dependencies. Without those, dep simply chooses the dependency master branches, which can cause problems as they are updated frequently.

Maybe wait for this to be solved ?

See https://blog.golang.org/versioning-proposal


unbundle index.tar.gz

index.tar.gz is passed through most of the pipeline when all most steps need is the .xg and/or .vg (and not the giant kmers file etc).
need to stop lumping these together and keep track of files individually as needed.
(related to #48 and #29)

Duplicate of #55 (blame github crashing everytime i submitted this)


跨域怎么处理？

请问跨域了怎么处理？
Access to XMLHttpRequest at 'http://pipeline.qiniu.com/v2/repos/client/data' from origin '***' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.

请问跨域了怎么处理？
Access to XMLHttpRequest at 'http://pipeline.qiniu.com/v2/repos/client/data' from origin '***' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.

不好意思。。是我们token的问题。


Upload executables

Pull Request Test Coverage Report for Build 718798425

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 84.184%

Totals

Change from base Build 718367476:
0.0%

Covered Lines:
2145

Relevant Lines:
2548

💛  - Coveralls


How to slide a row to show actions just like IOS

Hi, in a table view, I need to slide a row from right to left and show actions in the right end, just like the UITableView RowAction in IOS.  Is there a demo or existing component?
A example like below:

We did not find an existing Widget and finally we develop a new one.

I find an existing issue the same as mine, thanks.  https://github.com/flutter/flutter/issues/14388

@korbin-w I also struggling with this problem.. Can you please post your idea if you have anything..

@prabhuthiran My colleague will share the idea this later.  It's our pleasure.

@prabhuthiran
The code has not yet been collated into a common interface. So first send the demo to you for reference.
lib.zip

Thank you so much @korbin-w , @chaokongzwp  you guys saved my life.. I tried to run the code it is perfectly working.. i have another question is it possible to have  swipe on the both sides of the card..?

Is it possible to have a icon/image instead of background color and text?

@prabhuthiran  it is just demo，
It's relatively simple, follow-up I will provide a way to set widget as a background. In addition, right-scratch features can be done, but I have no time recently and you may need to wait a few days

@chaokongzwp Thank you for your widget, It helps me a lot.


Write short Foundations roadmap for internal site

Objective: Foundations roadmap that we can use to communicate to the stakeholders.
Content:

Business Opportunity
What's changing?
Current Status
Key Activities

Galao goal: Draft is complete

EOS 14 Oct Update: Draft written
Next Steps: Review required from @pseys

Latte Goal: review required and published

Draft can be viewed here (please make edits in additional colour or by copying original frame.)

Had the Bitbucket call with @mark-tate yesterday... @amritadesmet will sync with dev on Monday to get published by EOD on internal site

Update: Created a PR in Bitbucket, needs review

Lungo Goal: page to be published by EOS

31 Oct Update: PR approved in Bitbucket, will be released soon

Has been published


jsx-uses-vars rule doesn't work with ES6 class

With<EMAIL_ADDRESS>&<EMAIL_ADDRESS>$ cat hello.jsx
class HelloMessage extends React.Component {
    static get propTypes() {
        return {
            name: React.PropTypes.oneOfType([
                React.PropTypes.string,
                React.PropTypes.number
            ])
        };
    }
    render() {
        return
Hello {this.props.name}
;
    }
}

React.render(<HelloMessage name='world' />, document.querySelector('#content'));

$ cat .eslintrc
parser:
    babel-eslint

env:
    browser: true
    es6: true

plugins:
    - react

globals:
    React: true

rules:
    strict: 0
    quotes: [1, single, avoid-escape]
    indent: [1, 4]
    quote-props: [1, as-needed]
    no-unused-expressions: 1
    react/jsx-no-undef: 1
    react/jsx-quotes: [1, single, avoid-escape]
    react/jsx-uses-react: 1
    react/jsx-uses-vars: 1
    react/prop-types: 1
    react/react-in-jsx-scope: 1

$ eslint hello.jsx
hello.jsx
  1:6  error  HelloMessage is defined but never used  no-unused-vars

✖ 1 problem (1 error, 0 warnings)

Is that a bug of this plugin?

I have the same error. I used the following versions of the libraries<EMAIL_ADDRESS><EMAIL_ADDRESS>and<EMAIL_ADDRESS>Code:
'use strict';

import React from 'react';

class HelloMessage extends React.Component {
  render() {
    return
HelloWorld!
;
  }
}

React.render(<HelloMessage />, document.body);

$cat .eslintrc
{
  "parser": "babel-eslint",
  "plugins": [
    "react"
  ],
  "rules": {
    "strict": 0,
    "global-strict": 0,
    "quotes": [1, "single", "avoid-escape"],
    "react/display-name": 0,
    "react/jsx-boolean-value": 1,
    "react/jsx-no-undef": 1,
    "react/jsx-quotes": 1,
    "react/jsx-sort-prop-types": 1,
    "react/jsx-sort-props": 1,
    "react/jsx-uses-react": 1,
    "react/jsx-uses-vars": 1,
    "react/no-did-mount-set-state": 1,
    "react/no-did-update-set-state": 1,
    "react/no-multi-comp": 1,
    "react/no-unknown-property": 1,
    "react/prop-types": 1,
    "react/react-in-jsx-scope": 1,
    "react/self-closing-comp": 1,
    "react/sort-comp": 1,
    "react/wrap-multilines": 1
  },
  "env": {
    "browser": true,
    "es6": true
  },
  "ecmaFeatures": {
    "jsx": true,
    "modules": true
  }
}

$ eslint app.jsx
app.jsx
  5:6  error  HelloMessage is defined but never used  no-unused-vars

✖ 1 problem (1 error, 0 warnings)

I tried to find the root of the problem and it turned out that for the simplest case:
class A {};
<App />;

there are two variables with name A. One in scope: scope: { type: 'global', ... } and the second in scope scope: { type: 'class', ... }. The function markVariableAsUsed() marks only one of them with eslintUsed = true (the inner one I guess), so eslint reports an error.


Mac Catalyst support

Looks like this hard fork has already made some awesome progress towards addressing long standing issues in the old Mapbox SDK, like offline vector mbtiles, and XCFramework support.
Now that you already support XCFrameworks, supporting Mac Catalyst should be relatively straightforward. Binary components with custom build scripts will need the following additional architectures:

x86_64-apple-ios14.0-macabi
arm64-apple-ios14.0-macabi

You'll need to target macOS 11.0 (for Mac Catalyst / iOS 14.0) or higher for OpenGL ES support.
Cheers!

I see no reason why we would not support this. It's just one more "iOS" platform to build for afaik.
Although developers may be more interested into proper M1 support in the macOS SDK (I still need to figure out how big the difference there is between native iOS and macOS SDKs).

Sounds like a fun project...

On paper it should be trivial once done with someone familiar with Apple requirements. We can already build the Qt part of the codebase for M1 mac natively so it's just the matter of configuring the macOS and iOS builds properly.

This will be trivial once Bazel has support for it: https://github.com/bazelbuild/rules_apple/issues/658


feat(module:*): support for Angular ivy

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #3667
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

Seems date-fns isn't compatible with NGCC, and it's a direct dependency. What a pity.

@hsuanxyz So, I was able to compile date-fns using a custom ngcc.config.js file.
However there is still a problem with deep imports (https://github.com/angular/angular/issues/32097).
The next next.3 version will include a fix and maybe I'll finally be able to compile my project, even without ng-zorro explicitly targeting Ivy.

next.4 introduces a fix to avoid scanning non-Angular dependencies for entry points.
The ngcc.config.js is no more needed.

close via #4453


fix bad dynamic import scanning

We weren't checking that dynamic imports actually contained a string, before accepting them as valid. Now, we check first and ignore if you're dynamically importing anything other than a string literal.
Resolves https://www.pika.dev/npm/snowpack/discuss/125
/cc @nandoflorestan

Thank you!


spin test fails in meson install if PYTHONPATH is already set

Desription
It seems that this line
https://github.com/scientific-python/spin/blob/8ddc0abea6221bcc6c0ade7ad86f9d1ad1d9b4d8/spin/cmds/meson.py#L32-L33
trips up something in the following build chain, that is not expecting PYTHONPATH to contain more than one PATH. It looks like it's happening after / inside meson install step, though if I run that exact command manually it seems to pass fine, so maybe it's not originating from there.
Way to reproduce
On Linux with the current main of spin (https://github.com/scientific-python/spin/commit/8ddc0abea6221bcc6c0ade7ad86f9d1ad1d9b4d8), setup a development environmet for scikit-image and run
export PYTHONPATH=/home
spin test

Also reproduce-able on Windows it seems: https://discuss.scientific-python.org/t/spin-test-fails-on-windows/830
Output
Invoking `build` prior to running tests:
$ meson compile -C build
INFO: autodetecting backend as ninja
INFO: calculating backend command to run: /home/lg/.local/lib/venv/skimagedev/bin/ninja -C /home/lg/Res/scikit-image/build
ninja: Entering directory `/home/lg/Res/scikit-image/build'
ninja: no work to do.
$ meson install --only-changed -C build --destdir ../build-install
$ export PYTHONPATH="/home/lg/Res/scikit-image/build-install/usr/lib/python3.11/site-packages"
$ /home/lg/.local/lib/venv/skimagedev/bin/python -P -c 'import skimage'
$ export PYTHONPATH="/home/lg/Res/scikit-image/build-install/usr/lib/python3.11/site-packages:/home"
$ cd /home/lg/Res/scikit-image/build-install/usr/lib/python3.11/site-packages:/home
[Errno 2] No such file or directory: '/home/lg/Res/scikit-image/build-install/usr/lib/python3.11/site-packages:/home'; aborting.

Thanks for opening this, I run into this one, too in one of the more recent attempts of getting numpy/scipy builds working again locally.