dce leases list does not support both -p and -s flags

Version information

dce cli v0.3.0
Describe the bug

Providing both a -p and -s flag to dce-cli leases list does not work
To Reproduce
dce-cli leases list -p<EMAIL_ADDRESS>-s Active

...will return all leases where<EMAIL_ADDRESS>no matter the status.
Expected behavior
Should onlu return leases where<EMAIL_ADDRESS>AND leaseStatus=Active
Additional context
The API supports this query, as GET<EMAIL_ADDRESS>

@eschwartz It looks like the current behavior is to replace old leases with new ones in the leases table. E.g. see the example below where I create a new lease for the admin principal. The list of leases still only contains two entries, and the admin's lease has been replaced with a new one. I verified this was consistent with the leases table.
>./dce-cli leases list
[
	{
		"accountId": "258412077092",
		"budgetAmount": 100,
		"budgetCurrency": "USD",
		"budgetNotificationEmails": [
<EMAIL_ADDRESS>		],
		"createdOn":<PHONE_NUMBER>,
		"expiresOn":<PHONE_NUMBER>,
		"id": "8442ce70-fe4c-43e1-9ee8-f840824ebf3f", <---before
		"lastModifiedOn":<PHONE_NUMBER>,
		"leaseStatus": "Inactive",
		"leaseStatusModifiedOn":<PHONE_NUMBER>,
		"leaseStatusReason": "Destroyed",
		"principalId": "admin"
	},
	{
		"accountId": "258412077092",
		"budgetAmount": 100,
		"budgetCurrency": "USD",
		"budgetNotificationEmails": [
<EMAIL_ADDRESS>		],
		"createdOn":<PHONE_NUMBER>,
		"expiresOn":<PHONE_NUMBER>,
		"id": "c092bb05-deb8-44d3-80bf-8923f7096beb",
		"lastModifiedOn":<PHONE_NUMBER>,
		"leaseStatus": "Inactive",
		"leaseStatusModifiedOn":<PHONE_NUMBER>,
		"leaseStatusReason": "Destroyed",
		"principalId": "user"
	}
]
> ./dce-cli leases create --budget-amount 100 --budget-currency USD --email<EMAIL_ADDRESS>--principal-id admin
{
	"accountId": "258412077092",
	"budgetAmount": 100,
	"budgetCurrency": "USD",
	"budgetNotificationEmails": [
<EMAIL_ADDRESS>	],
	"createdOn":<PHONE_NUMBER>,
	"expiresOn":<PHONE_NUMBER>,
	"id": "90746a3c-2b37-4930-b059-559920a11272",
	"lastModifiedOn":<PHONE_NUMBER>,
	"leaseStatus": "Active",
	"leaseStatusModifiedOn":<PHONE_NUMBER>,
	"leaseStatusReason": "Active",
	"principalId": "admin"
}
> ./dce-cli leases list
[
	{
		"accountId": "258412077092",
		"budgetAmount": 100,
		"budgetCurrency": "USD",
		"budgetNotificationEmails": [
<EMAIL_ADDRESS>		],
		"createdOn":<PHONE_NUMBER>,
		"expiresOn":<PHONE_NUMBER>,
		"id": "90746a3c-2b37-4930-b059-559920a11272", <--- after
		"lastModifiedOn":<PHONE_NUMBER>,
		"leaseStatus": "Active",
		"leaseStatusModifiedOn":<PHONE_NUMBER>,
		"leaseStatusReason": "Active",
		"principalId": "admin"
	},
	{
		"accountId": "258412077092",
		"budgetAmount": 100,
		"budgetCurrency": "USD",
		"budgetNotificationEmails": [
<EMAIL_ADDRESS>		],
		"createdOn":<PHONE_NUMBER>,
		"expiresOn":<PHONE_NUMBER>,
		"id": "c092bb05-deb8-44d3-80bf-8923f7096beb",
		"lastModifiedOn":<PHONE_NUMBER>,
		"leaseStatus": "Inactive",
		"leaseStatusModifiedOn":<PHONE_NUMBER>,
		"leaseStatusReason": "Destroyed",
		"principalId": "user"
	}
]

When I apply the -s and -p flags, they seem to work as expected:
> ./dce-cli leases list -p user -s Inactive
[
	{
		"accountId": "258412077092",
		"budgetAmount": 100,
		"budgetCurrency": "USD",
		"budgetNotificationEmails": [
<EMAIL_ADDRESS>		],
		"createdOn":<PHONE_NUMBER>,
		"expiresOn":<PHONE_NUMBER>,
		"id": "c092bb05-deb8-44d3-80bf-8923f7096beb",
		"lastModifiedOn":<PHONE_NUMBER>,
		"leaseStatus": "Inactive",
		"leaseStatusModifiedOn":<PHONE_NUMBER>,
		"leaseStatusReason": "Destroyed",
		"principalId": "user"
	}
]
> ./dce-cli leases list -p user -s Active
[]

It looks like the current behavior is to replace old leases with new ones in the leases table.

@joshmarsh -- that's correct. But you're testing with a single account, so you're missing some of the behavior.
Consider if I create a lease for account=123, end that lease, then create another on account=456. In that case, I want a way to see only my the active account for my user. I don't think the CLI supports that today.
For context, this is causing some pain our DCE pipeline:
https://github.com/Optum/dce/blob/master/pipelines/pipeline.yml#L172

I just tried dce leases list -p my-user -s Active using DCE CLI v0.29.0, and DCE v0.30.0, and it seems to be applying both query params.
I'm going to call this resolved.


Feature/strip

Description:
This enable -no-strip-debug for platform that we were actually not triggering it for (android, ios and windows) and make them equivalent as what we are doing for linux, darwin and freebsd.
Fixes #126
Checklist:

[ ] Tests included.
[ ] Lint and formatter run with no errors.
[ ] Tests all pass.

Also, it seems to break Android builds. Any idea what happened?

I found one suggestion for a comment. However, I am not quite sure what this is trying to do. Where is ctx.StripDebug set and why is it only checked if it isn't targeting Linux, macOS or FreeBSD?

Other platform do not use fyne to do the build. When the build are directly done by fyne-cross it does honor the StripDebug (which is the result of a flag set on the command line). The problem was that before this, the platform build by fyne would not get stripped.

Ah, I see. That makes sense but I think you might want to add a comment about that in the code. It is a bit confusing.

The change in android image are not propagated to the test and so test will still fail. If you recompile locally the image so you can have bundletool in it, everything will work as one expect.

Oh indeed, I tried to commit @Jacalz change from github and it failed due to some wrong email somewhere and forgot after that. Will adress.

The failed Android builds is concerning me.
It seems that the interpretation of the release has changed and now the build is attempting to package for google play instead of verifying the build process.
Adding the bundletool may have been an attempt to fix this, but is it the right fix? Are we attempting to do a store build in these build checks?

We should be able to move this forward with #140

Merging is blocked since the android tests still fail.
They should pass once #141 is merged and the new docker images are available.

@Jacalz @andydotxyz with the new image with latest fyne and go, we are now all green here!

@Jacalz would you mind checking if this PR now match your requested change earlier?


Adding a new returnAndBulkUpsert API in the doc store.

This will help the clients to build more interesting use cases and also cutdown a roundtrip to the server.
A use case where this has even become essential is more like a change data capture kind of use case: A client wants to upsert a bunch of docs but it needs to know what all changed in those docs compared to the previous version of the docs so that it can optimize some processing. CDC (Change data capture) at the doc store level might be helpful but that's much more heavy weight and doesn't have enough use cases currently to bring that in.
Also, it's a known that the new API is sync API and could take a bit longer than other APIs so it should be used wisely based on the use cases.

I am usually reluctant to such additions as it forces sync processing which
isn't good for high load. Could you please describe some of the uses cases
you have in mind?
On Tue, 15 Dec 2020, 03:21 Buchi Reddy Busi Reddy<EMAIL_ADDRESS>wrote:

This will help the clients to build more interesting use cases and also
cutdown
a roundtrip to the server in the cases where they need to immediately get
back
the upserted documents.
You can view, comment on, or merge this pull request online at:
https://github.com/hypertrace/document-store/pull/18
Commit Summary

Adding a new API to bulkUpsertAndGet the documents in the doc store.

File Changes

M
document-store/src/integrationTest/java/org/hypertrace/core/documentstore/mongo/MongoDocStoreTest.java
https://github.com/hypertrace/document-store/pull/18/files#diff-fab258440918057cd0bc0d16483437cc7dbfb88df8e6256b4efb55a763bb9712
(22)
M
document-store/src/integrationTest/java/org/hypertrace/core/documentstore/postgres/PostgresDocStoreTest.java
https://github.com/hypertrace/document-store/pull/18/files#diff-a7415a191ab5ed0893f1d0116760c767adc1290ec45b6647837d027e0fa4d31b
(44)
M
document-store/src/main/java/org/hypertrace/core/documentstore/Collection.java
https://github.com/hypertrace/document-store/pull/18/files#diff-9ca60955d4574c5272bb8c34261cffdf9508edb8e2e8bb52f209524dac6ea9d6
(6)
M
document-store/src/main/java/org/hypertrace/core/documentstore/mongo/MongoCollection.java
https://github.com/hypertrace/document-store/pull/18/files#diff-a4f7c7eb0b57790b80ac31a65d3504b760caf04c168b42c33da9f9364f2f73e7
(62)
M
document-store/src/main/java/org/hypertrace/core/documentstore/postgres/PostgresCollection.java
https://github.com/hypertrace/document-store/pull/18/files#diff-3df0c046f864e7bc937f5d243417a382655bced2f3c5f70e80ec469012ea61bb
(84)

Patch Links:

https://github.com/hypertrace/document-store/pull/18.patch
https://github.com/hypertrace/document-store/pull/18.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hypertrace/document-store/pull/18, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXOYAUGOAMDC4K6X4U5AJTSU3B23ANCNFSM4U3UFAYQ
.

I am usually reluctant to such additions as it forces sync processing which isn't good for high load. Could you please describe some of the uses cases you have in mind?

@jcchavezs updated the description with more details.

@buchi-busireddy I am sorry I am late to the party but I left you some comments.

Hi @buchi-busireddy I wonder if you had time to put up some of the fixed we talked about in this PR post merge.

@jcchavezs Thanks for reminding and sorry that I missed raising the PR. I do have the changes locally and will raise shortly.

@jcchavezs Thanks for reminding and sorry that I missed raising the PR. I do have the changes locally and will raise shortly.


When passing in false to watch it still watches

https://github.com/denali-js/denali/blob/82401b4517b94426c3d723981eafad6356fd39cf/commands/server.ts#L98

Args are always strings, so we need to do an explicit check here.


example from documentation does not work

I copied and paste this into a Scala worksheet:
val str = """[{"myFieldA":1,"myFieldB":"g"},{"myFieldA":2,"myFieldB":"k"}]"""
val json: ujson.Js = ujson.read(str)
json.arr.remove(1)
json(0)("myFieldA") = 1337
json(0)("myFieldB") = json(0)("myFieldB").str + "lols"
and I get
Error:(18, 69) ')' expected but string literal found.
inst$A$A.get$$instance_0$$json(0)("myFieldA"); println("json(0)("myFieldA"): " + {val $$temp$$ = inst$A$A.json(0)("myFieldA") ; MacroPrinter.printDefInfo($$temp$$).replace("inst$A$A.", "") + " = " + ( print$$$Worksheet$$$Array$$$($$temp$$) ).stripPrefix("A$A40$A$A40$")})
^
Any suggestions?

that is a problem with your worksheet, not uPickle. Feel free to open an issue with who-ever implemented your worksheets

The worksheet is in IntelliJ, the most popular Scala IDE I believe.
You are correct. It works fine in regular projects outside the worksheet.
Thanks!


Mailhog does not retry for mongodb connection and panics

If connection to mongodb is interrupted and session is lost, mailhog does not retry the connection.

I can suggest a fix in PR but is this project maintained? Are PRs reviewed?

@tyndyll Please let me know if your can do a review for this fix.. then I can create a PR.

Please do


Typo in README.md

Hey,
While looking at it, there's a typo in README.md.
Line 153: proccess; it should rather be process.

Thank you, just fixed it :)


SNSTopic Crossplane Composite Wrapper Resource

What: SNSTopic resource is in v1alpha1 in the aws-crossplane provider. Once this resource reaches v1beta1 and in order to allow our multi-tenancy users to deploy and manage this resource type we need to create a composition wrapper around it (resource and rbac) and add it to our DFDS Configuration Package
Why: This will allow cluster users to deploy and manage AWS SNS Topic resources using Crossplane in Hellman with appropriate RBAC permissions

[x] Create XRD
[x] Create Composition
[x] Test in Sandbox
[ ] Create Release

Blocked due to being an Alpha resource @ https://doc.crds.dev/github.com/crossplane/provider-aws

XRD not working in my sandbox but the resource itself works. I'm not able to figure out why. need to be reviewed.


How to swap assets on-chain?

SOL <-> token or vise versa, or token <-> token
How does one swap assets on-chain? Within a smart contract or program. Is there an example?

Please refer to https://github.com/raydium-io/raydium-sdk-V2-demo/blob/master/src/amm/swap.ts


Curious behavior with --no-cache option and ts imports from node_modules

Issue :
Hi,
I've recently tried to update ts-jest in one of my projects from 23.1 to 23.10.
Commit: https://github.com/DorianGrey/vue-ts-playground/commit/362999f603f6b3eae3bbf23cc1ba207927b1837c
(There is a level of indirection with @vue/cli, but that should not hurt.
Everything worked fine at first glance, but then my pre-push hook kicked in and complained that it cannot import a .d.ts file:
 FAIL  tests/unit/views/todoList/todoListEntryDisplay.spec.ts
  ● Test suite failed to run

    TypeError: Unable to require `.d.ts` file.
    This is usually the result of a faulty configuration or import. Make sure there is a `.js`, `.json` or another executable extension available alongside `en.ts`.

      2 | import dateTimeFormat from "./dateTimeFormat";
      3 | import messages from "./messages.json";
    > 4 | import vuetifyMessages from "vuetify/src/locale/en";
        | ^
      5 |
      6 | export default {
      7 |   language: "en",

      at getOutput (node_modules/ts-jest/dist/compiler.js:163:23)
      at Object.compile (node_modules/ts-jest/dist/compiler.js:185:29)
      at TsJestTransformer.process (node_modules/ts-jest/dist/ts-jest-transformer.js:101:41)
      at Object.<anonymous> (src/i18n/lang-packs/en/index.ts:4:1)

It's just that the target file is a .ts file, thus, it should not expect a .d.ts file for it to be present at all. Even worse, the test executed fine when executed by my regular test:unit task.
After a bit of fiddling around, I figured out that the problem is somewhat related to the --no-cache option that is used in the pre-push hook, resp. the test:unit:ci task. This flag is not used in the other tasks (which executes fine as mentioned above). Removing that flag from the latter task also makes it work fine.
That flag is enabled since the test:unit:ci task also collects coverage information, which is known too have some issues when using the flag, esp. when used with vue components (see e.g. https://github.com/vuejs/vue-jest/issues/56).
Expected behavior :
Executing test, i.e. transforming the related files, should work regardless of jest's cache being used or not.
Debug log:
Added as file attachment, since it's too long to post it directly:
ts-jest.log
Minimal repo :
https://github.com/DorianGrey/vue-ts-playground/tree/ts-jest-update-import-failure
(If that's too large, I may try to put things together in a smaller repro... though it won't be that much smaller. The affected code is quite small, actually).
The failing test can be executed via
yarn test:unit --no-cache todoListEntryDisplay

Note that the test will work fine if the --no-cache is NOT provided.
Travis build is here: https://travis-ci.org/DorianGrey/vue-ts-playground/builds/439530439 (test execution is somewhere around L700).

Hi @DorianGrey. The "minimal repo" is just that often people cannot give a link to their repo as it is private. So it's fine with your repo as soon as we can reproduce the issue with a command line that does not run 1000 tests ;-)
Can you do this and give the result please:
yarn jest --clearCache
yarn test:unit todoListEntryDisplay

HI @huafu ,
after manually clearing jest's cache manually, the test fails with the same error that occurs when using the --no-cache flag. However, when trying to execute the test again (without clearing the cache) everything works fine... So this seems to be a cache-related issue.

hmmmm. that's definitely a bug! thanks!

I just ran into this same problem and did some debugging. The reason for this is because Typescript is specifically ignoring node_modules with the getEmitOutput() API that ts-jest is using when type checking is enabled. https://github.com/Microsoft/TypeScript/issues/11946
So if you still want full type checking, the solution is to specifically include the node_modules files in the includes in the tsconfig.

@alfaproject Ran into this error as well. We're importing modules from another project which did not have any .d.ts files associated with them. We solved it by adding this to jest.config.js
globals: {
  'ts-jest': {
    isolatedModules: true
  },
},

I had same problem too even with the imported module with d.ts files. But for some reason it worked after setting isolatedModules to true

I ran into this issue as well.
I'm running stuff in docker so cache was never persisting. Fixed that with a bind-mount and adding a cacheDirectory option to jest.config.js that maps to my local filesystem.
Still, errors out with the following message anytime I run tests:
TypeError: Unable to require `.d.ts` file.
This is usually the result of a faulty configuration or import. Make sure there is a `.js`, `.json` or another executable extension available alongside `index.ts`.

This is apparently bc there's no cache yet and the tests seem to not get far enough along for the necessary cache to ever be created.
If I add the following line to my jest.config.js then, then tests will start running w/out error.
  "globals": { "ts-jest": { "isolatedModules": true }, },

Interestingly, once the tests pass once w this option, it can be removed later and the tests will continue passing as long as I don't delete the cache dir or run things with --no-cache.

i will close this issue. If anyone still encounters this with latest jest and ts-jest, feel free to open a new issue with a minimum repo

@ahnpnl, here is a minimal repository to reproduce the issue: https://github.com/mfellner/ts-jest-issue-805-demo
Do you prefer a new issue or could you re-open this one?

Hi, I would prefer a new issue. Would you please create a new one with your repo and describe the issue ?


class ZabbixAPIExtends(ZabbixAPI): NameError: name 'ZabbixAPI' is not defined

Hi
Thank you for your beautiful zabbix agent ansible deployment code
I have this error when I try to use it
TASK [dj-wasabi.zabbix-agent : Updating host configuration with macros] ********
task path: /etc/ansible/roles/dj-wasabi.zabbix-agent/tasks/main.yml:134
Using module file /usr/lib/python2.7/dist-packages/ansible/modules/extras/monitoring/zabbix_hostmacro.py
<localhost> ESTABLISH LOCAL CONNECTION FOR USER: majidgolshadi
<localhost> EXEC /bin/sh -c '( umask 77 && mkdir -p "` echo ~/.ansible/tmp/ansible-tmp-1485256895.75-21818523532 `" && echo ansible-tmp-1485256895.75-21818523532="` echo ~/.ansible/tmp/ansible-tmp-1485256895.75-21818523532 `" ) && sleep 0'
<localhost> PUT /tmp/tmpTr2jiO TO /home/majidgolshadi/.ansible/tmp/ansible-tmp-1485256895.75-21818523532/zabbix_hostmacro.py
<localhost> EXEC /bin/sh -c 'chmod u+x /home/majidgolshadi/.ansible/tmp/ansible-tmp-1485256895.75-21818523532/ /home/majidgolshadi/.ansible/tmp/ansible-tmp-1485256895.75-21818523532/zabbix_hostmacro.py && sleep 0'
<localhost> EXEC /bin/sh -c '/usr/bin/python /home/majidgolshadi/.ansible/tmp/ansible-tmp-1485256895.75-21818523532/zabbix_hostmacro.py; rm -rf "/home/majidgolshadi/.ansible/tmp/ansible-tmp-1485256895.75-21818523532/" > /dev/null 2>&1 && sleep 0'

An exception occurred during task execution. The full traceback is:
Traceback (most recent call last):
  File "/tmp/ansible_8iV0pp/ansible_module_zabbix_hostmacro.py", line 113, in <module>
    class ZabbixAPIExtends(ZabbixAPI):
NameError: name 'ZabbixAPI' is not defined

failed: [XXX.XXX.XXX.XXX -> localhost] (item={u'macro_key': u'apache_type', u'macro_value': u'reverse_proxy'}) => {
    "failed": true,
    "invocation": {
        "module_name": "zabbix_hostmacro"
    },
    "item": {
        "macro_key": "apache_type",
        "macro_value": "reverse_proxy"
    },
    "module_stderr": "Traceback (most recent call last):\n  File \"/tmp/ansible_8iV0pp/ansible_module_zabbix_hostmacro.py\", line 113, in <module>\n    class ZabbixAPIExtends(ZabbixAPI):\nNameError: name 'ZabbixAPI' is not defined\n",
    "module_stdout": "",
    "msg": "MODULE FAILURE"
}

Would you please help me to find out what I have to do with that?

I'm having this problem as well.

Hi,
Which version of Ansible are you using?

I'm using the latest one - but I found out that the problem was that I had...
    zabbix_host_groups:       - Linux Servers
...with capital S on servers, and when I changed it to lower-case it went smoothly.

I get the same error. What is weird, that my instance is properly registered in Zabbix.

Hi @kamilkrzeminski
I believe there was a bug in <IP_ADDRESS> and is fixed in <IP_ADDRESS>, so if you are not using the latest Ansible, please use it.

@dj-wasabi I'm using Ansible <IP_ADDRESS> and the issue persists.

The best way forward is to create an issue in the ansible repository. This seems to be a bug in the Ansible module and is part of the ansible code, this Zabbix Agent Ansible role only uses this module and therefor I can not help you any further.


Jeff Schaller: Watch piramalaranya.com

Jeff Schaller requests the watch of the watch_keyword piramalaranya\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

piramalaranya\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Eliah Kagan in Charcoal HQ


Feed's topic is not limited to 32 bytes

Was looking into what is the limitation of topics as I was a bit confused by API docs and it turned out that the topic is not limited to any specific size and the API only need it hex encoded. It is then internally hashed with the index inside Bee, so that's why.
In code, we have on several places mentioned that the topic is limited to 32 bytes, maybe with some runtime checks as well.
I would therefore propose to remove those. Maybe drop the makeTopicFromString() helper as well?

Currently, the topic is Bee-js is enforced to 32 bytes and if users need arbitrary topic, then he can use the makeFeedTopic, so I am closing this.


use version 0.6.2

currently the is bug with the release candiate, so its best to use an old version until its a new version is released
also saves time having to not rebuild every single time the latest gets updated!

Can you please update version within build and shell file into this commit?

@dm82m done! sorry to nag, the latest was just bugging me until i realised its the release candidate version thats at fault!
maybe you could do another folder for a dev build which uses the latest image instead? would you like me to fo this?

I personally don’t need it. If you need it you can for sure create a pull request.

@dm82m FINALLY GOT THERE! #27
STUPID CODESPACES KEPT COMMITING STUFF IT SHOULDNT BE

Will check it tomorrow, not at home and only my mobile with me.


Extend React.PureComponent instead of React.Component

I was doing some profiling and I noticed that when Waypoint updates, we
call this._handleScroll synchronously, which eventually calls functions
that force layout. Since this happens during render, the layout is
immediately invalidated and then triggered again once rendering is
completed.
We avoid this problem in componentDidMount by deferring this work
until the next tick.
I think we should consider making two changes to Waypoint to help
address this:

Extend React.PureComponent instead of React.Component. In the
place I saw this problem, we were rendering a waypoint with pure
props, so the update in this case could be completely avoided by
using PureComponent.
<Waypoint
  onEnter={this.onEnterWaypoint}
  scrollableAncestor={this.props.scrollableAncestor}
/>

Defer this._handleScroll in componentDidUpdate using
onNextTick, like we do in componentDidMount. This will still
require layout, but it will happen around the same time that the
browser was going to do layout anyway, avoiding the thrash.

This commit implements the first of these two interventions.
Partly addresses #263

(Emphasized part is highlighted)
Would this be a breaking change, as all children components of a Waypoint must now also be pure?

@jamesplease @trotzig Any more thoughts on this or do you think we are good to merge?


Try seafox parser in Prettier, Babel, etc.

I just discovered https://github.com/KFlash/seafox which claims 2X performance, simplistic ISC license. It would be interesting to see if this could be used in tools such as Prettier, Babel, ESLint, etc.
I wonder how it would do in the TC39 test suite https://github.com/tc39/test262

@brodybits I accidentally found this issue, so I will try to answer you.
Regarding the Test262 suite it pass / fail almost 100%. There are a few regex cases that doesn't fail according the spec because I'm using the native regexp validator in NodeJS and browsers.
Also note that by default the Test262 suite doesn't have AnnexB enabled by default - it's separated tests for it. Acorn and others doesn't have an option to turn off the support for additional ECMAScript features for Web Browsers, but you can with Seafox.
I'm sorry, but I didn't find a good way to add the Test262 suite to the repo - I used a hackish version of Acorn's code to verify. And I didn't have time to write my own.
However. I have now added the Test262-parser-tests suite. Except for the few invalid tests in that repo, Seafox pass & fail as it should.
I hope this answers your questions :)

Thanks @KFlash that sounds really awesome. And I do really like the way you added the Test262 suite without copying the whole thing which  I see all too often. So I guess my next step is to try it in one of these tools, unless someone else wants to take this on. (I am in the middle of an urgent work project right now.)
Moving off-topic, I am also curious what led you to choose the ISC license over something more common like MIT?
I actually really like the ISC most due to is simplicity and closeness to both MIT and BSD. There are a couple of factors that make me a little hesitant on my own projects:

The most well-known MIT & Apache 2.0 permissive licenses seem to be the best understood, even if not necessarily the best written.
The ISC license seems to have had a very minor change. My one issue with that is that if a project says "ISC license", how should we know which version they meant?

@brodybits The ISC liecense is something my students are using in their projects, and I just "copied" their workflow. I have no problem to change the licence to MIT or BSD.
I don't have too much experience with the differences between them, but you can do a PR if you want, so I can change it.
All my work is open sourced and no limitations on it.

Thanks, I would actually not be super-certain myself at this point. To be strict, we should probably get permission from the other contributors or maybe just revert their work before changing it (seems to be very little at this point). (Bootstrap actually went through this to change from Apache 2.0 to MIT license). Maybe I am just making too big a deal, wish I had more time to contribute:)
I am now curious what kind of classes and projects you are teaching, sounds really cool!

I'm actually leaving abroad and helping street kids to get back to a normal life. Kids without families. Homeless. We raise them from nothing to get a rich life with education and a future work. It's part of [ALS (alternative learning system)[https://www.deped.gov.ph/k-to-12/inclusive-education/about-alternative-learning-system/].
Some of this students are very smart and I give them private teachings in math, and both frontend and backend programming in different languages. They love JS :)

Wow that sounds really, really awesome. Do you see this expanding into any other countries?
Are you able to find work for some of the students?

Part of this is already in Malaysia where I also live. I switch between Philippines and Malaysia, but as of now I'm stuck in the Philippines.
Yes. Some of the students are  in work. Mostly calling centers. But 2 of them are working in Burger Kind and one already married :)

Married, but not to me!

LOL.

Cool  job, I  guess the  ISC license is  because npm init defaults to this license.


GraphQL Error Unknown field allStrapiArticle on type RootQueryType

**Node.js versionv10.8.0:
**npm version6.2.0:
**Strapi version3.0.0-alpha.13.0.1:
**Operating systemMacOs 10.14 Beta (18A353d):
Success delete html and css files from previous builds — 0.019 s
success open and validate gatsby-config — 0.006 s
success copy gatsby files — 0.018 s
success onPreBootstrap — 0.094 s
⠁ Starting to fetch data from Strapi (article)
Starting to fetch data from Strapi (user)
Fetch Strapi data: 38.918ms
The new node didn't pass validation
ValidationError: child "id" fails because ["id" must be a string]**

(node:3635) Warning: No such label 'Fetch Strapi data' for console.timeEnd()
warning The gatsby-source-strapi plugin has generated no Gatsby nodes. Do you need it?
success source and transform nodes — 0.071 s
success building schema — 0.106 s
success createLayouts — 0.031 s
success createPages — 0.001 s
success createPagesStatefully — 0.019 s
success onPreExtractQueries — 0.001 s
success update schema — 0.068 s
GraphQL Error Unknown field allStrapiArticle on type RootQueryType
file: /Users/omerkunul/gatsby-strapi-tutorial/blog/src/pages/index.js
1 |
2 |   query IndexQuery {

3 |     allStrapiArticle {
|     ^
4 |       edges {
5 |         node {
6 |           id
7 |           title
8 |           content
9 |         }
10 |       }
11 |     }
12 |   }
13 |

same error
"gatsby": "^2.0.19",
"gatsby-plugin-manifest": "^2.0.5",
"gatsby-plugin-offline": "^2.0.5",
"gatsby-plugin-react-helmet": "^3.0.0",
"gatsby-source-strapi": "^0.0.3",

Authenticate Strapi user
Authenticate Strapi user: 148.712ms
Starting to fetch data from Strapi (articles)
Starting to fetch data from Strapi (users)
Fetch Strapi data: 28.169ms
(node:54012) Warning: No such label 'Fetch Strapi data' for console.timeEnd()
success source and transform nodes — 0.198 s
success building schema — 0.131 s
success createPages — 0.000 s
success createPagesStatefully — 0.024 s
success onPreExtractQueries — 0.000 s
success update schema — 0.078 s
GraphQL Error Unknown field allStrapiArticle on type Query
file: /Users/sunjiachao/zhimiao_work/seag/src/pages/index.js
1 |
2 |     query IndexQuery {

3 |         allStrapiArticle {
|         ^
4 |             edges {
5 |                 node {
6 |                     id
7 |                     title
8 |                     content
9 |                 }
10 |             }
11 |         }
12 |     }
13 |

same problem Unknown fieldallStrapiArticleson typeQuery``

I got same error message too. Anyone knows how to solve this issue?

Can you try allStrapiArticles - maybe pluralize issue.

It's caused by strapi/strapi#1911 (comment)

I followed all post and tried every possible way but there is still the error
  38:5  error  Cannot query field "allStrapiArticle" on type "Query"  graphql/template-strings

I tried

go to folder plugins\graphql.
Then, npm install graphql@git://github.com/graphql/graphql-js.git#npm

and

npm install<EMAIL_ADDRESS>--save

and the error still persist

Okay - I found a work around for this... @clabmb
Rename your gatsby-config.js to gatsby.config.js.  Try to recompile.  You should see a new error that says: error GraphQL Error Unknown field \file` on type `Query`s`
Once this new error appears, close your terminal, rename your gatsby.config.js back to gatsby-config.js.  Restart gatsby develop and it magically works.  Why?  I have no idea.

Actually - just use gatsby develop --no-cache

I have the problem true. Could we change the id of a node to string?

@pilotdeveloper your method does not works
I was trying both rename config file and --no-cache flag.
gatsby develop --no-cache
success open and validate gatsby-configs — 0.006 s
success load plugins — 0.257 s
success onPreInit — 0.382 s
success delete html and css files from previous builds — 0.066 s
success initialize cache — 0.008 s
success copy gatsby files — 0.015 s
success onPreBootstrap — 0.007 s
⠁ Starting to fetch data from Strapi (articles)
Starting to fetch data from Strapi (users)
error Plugin gatsby-source-strapi returned an error

  Error: Request failed with status code 403

  - createError.js:16 createError
    [blog]/[axios]/lib/core/createError.js:16:15

  - settle.js:18 settle
    [blog]/[axios]/lib/core/settle.js:18:12

  - http.js:201 IncomingMessage.handleStreamEnd
    [blog]/[axios]/lib/adapters/http.js:201:11

  - next_tick.js:63 process._tickCallback
    internal/process/next_tick.js:63:19

warning The gatsby-source-strapi plugin has generated no Gatsby nodes. Do you need it?
success source and transform nodes — 0.085 s
⠁ Fetch Strapi data: 76.701ms
success building schema — 0.254 s
success createPages — 0.001 s
success createPagesStatefully — 0.043 s
success onPreExtractQueries — 0.003 s
success update schema — 0.153 s
error GraphQL Error There was an error while compiling your site's GraphQL queries.
  Error: RelayParser: Encountered 1 error(s):
- Unknown field 'allStrapiArticle' on type 'Query'. Source: document `IndexQuery` file: `GraphQL request`

  GraphQL request (3:5)
  2:   query IndexQuery {
  3:     allStrapiArticle {
         ^
  4:       edges {

success extract queries from components — 0.134 s
success run graphql queries — 0.011 s — 5/5 558.79 queries/second
success write out page data — 0.004 s
success write out redirect data — 0.003 s
⡀ onPostBootstrapdone generating icons for manifest
success onPostBootstrap — 0.185 s

info bootstrap finished - 3.357 s

Warning: React version not specified in eslint-plugin-react settings. See https://github.com/yannickcr/eslint-plugin-react#configuration .
 ERROR  Failed to compile with 1 errors                       23:49:39

 error  in ./src/pages/index.js

Module Error (from ./node_modules/eslint-loader/index.js):

/home/daniel/exp/gatsby-strapi-tutorial/blog/src/pages/index.js
  28:5  error  Cannot query field "allStrapiArticle" on type "Query"  graphql/template-strings

✖ 1 problem (1 error, 0 warnings)

 @ ./.cache/sync-requires.js 17:54-128
 @ ./.cache/app.js
 @ multi (webpack)-hot-middleware/client.js?path=/__webpack_hmr&reload=true&overlay=false ./.cache/app

✖ ｢wdm｣:
ERROR in ./src/pages/index.js
Module Error (from ./node_modules/eslint-loader/index.js):

/home/daniel/exp/gatsby-strapi-tutorial/blog/src/pages/index.js
  28:5  error  Cannot query field "allStrapiArticle" on type "Query"  graphql/template-strings

✖ 1 problem (1 error, 0 warnings)

 @ ./.cache/sync-requires.js 17:54-128
 @ ./.cache/app.js
 @ multi (webpack)-hot-middleware/client.js?path=/__webpack_hmr&reload=true&overlay=false ./.cache/app
ℹ ｢wdm｣: Failed to compile.
 WAIT  Compiling...                                           23:49:39

ℹ ｢wdm｣: Compiling...
 ERROR  Failed to compile with 1 errors                       23:49:39

 error  in ./src/pages/index.js

Module Error (from ./node_modules/eslint-loader/index.js):

/home/daniel/exp/gatsby-strapi-tutorial/blog/src/pages/index.js
  28:5  error  Cannot query field "allStrapiArticle" on type "Query"  graphql/template-strings

✖ 1 problem (1 error, 0 warnings)

 @ ./.cache/sync-requires.js 17:54-128
 @ ./.cache/app.js
 @ multi (webpack)-hot-middleware/client.js?path=/__webpack_hmr&reload=true&overlay=false ./.cache/app

✖ ｢wdm｣:
ERROR in ./src/pages/index.js
Module Error (from ./node_modules/eslint-loader/index.js):

/home/daniel/exp/gatsby-strapi-tutorial/blog/src/pages/index.js
  28:5  error  Cannot query field "allStrapiArticle" on type "Query"  graphql/template-strings

✖ 1 problem (1 error, 0 warnings)

 @ ./.cache/sync-requires.js 17:54-128
 @ ./.cache/app.js
 @ multi (webpack)-hot-middleware/client.js?path=/__webpack_hmr&reload=true&overlay=false ./.cache/app
ℹ ｢wdm｣: Failed to compile.

Still have this error, and can't fix it yet. Any recommendations?

@Kostanos  remove user, it work
options: {
apiURL: 'http://localhost:1337',
contentTypes: [ // List of the Content Types you want to be able to request from Gatsby.
'article',
// 'user'
],
queryLimit: 1000,
}

same here

It seems permission error "Error: Request failed with status code 403"
Give Permission to both Article and User


debug_traceTransaction sometimes returns no trace steps when on a forked chain

Expected Behavior
The debug_traceTransaction method should consistently return trace steps.
Current Behavior
Sometimes, when running on a forked chain, the debug_traceTransaction method returns an empty list of trace steps, even though the actual list of trace steps was nonempty.  This seems to happen both for transactions before and after the fork.  For a fixed transaction though, within a single Ganache instance, it seems to either always happen or never happen.
Steps to Reproduce (for bugs)
Unfortunately, I can't consistently reproduce this problem.  Start Ganache with a forked chain and try debugging some transactions from before the fork, or make some transactions after the fork and try debugging those.  Sometimes it'll happen, sometimes it won't.
For what it's worth, though, if you want to try it with a transaction from before the fork, one particular transaction I encountered the error with is transaction 0x6a86b051ad13bcf8a4ebfc33f6ccdacced818bd2de3149e7566e9f55cc0fab64 on the Goerli testnet.  Note that sometimes when I've started Ganache this transaction will return no trace steps, while other times it works fine.  However, as mentioned above, within a single instance of Ganache it seems to either work or fail consistently.
Context
I was trying to debug transactions on a forked chain.  Not sure what more there is to say about that.  I personally was trying to do this as a test of the debugger, but from what I hear this is indeed something actual users do with our debugger.
Your Environment

Version used: Ganache CLI v6.8.2 (ganache-core: 2.9.2)
Operating System and version: Release Linux Mint 19.1 Tessa 64-bit

@haltman-at what would you say your failure rate is?

Hm, good question.  This hasn't happened in a while, to tell the truth.

How were you verifying whether or not you were getting the traces? were you manually sending the RPC requests with curl/something similar or were you using the debugger? If you were using the debugger, can you give me reproduction steps on how you would have seen this?

I was using the debugger.  Initially this caused errors but I later altered the debugger to simply warn you rather than causing errors when this occurs.  The reason I say there were no trace steps was because trace.steps, where we store them, was equal to [] when inspected.  (I didn't actually inspect the raw RPC return, but if the trace result were entirely absent rather than being present but empty, this would cause other debugger errors, so I think we can rule that out even though I didn't directly inspect it.)
For repro steps, you can just use truffle debug <txHash>, assuming you can find a transaction where this occurs.  If the trace steps are empty the debugger will (now) warn you about this on startup.

Perfect, thanks. I'll try to keep reproducing this. Let me know if you run into it again; if you do, please let me know which version of truffle and/or ganache you're using

I'm closing this as I can't reproduce it; feel free to reopen or open a new issue if we start seeing this again

OK, I'll let you know if I see this again!


Elasticsearch SearchGuard explanation and debugging

This issue contains my knowledge of SearchGuard configuration in image https://github.com/openshift/origin-aggregated-logging/tree/master/elasticsearch/sgconfig. Which is then used in https://github.com/openshift/elasticsearch-operator.
SearchGuard is configured in these files:

sg_action_groups.yml - ES privileges https://www.elastic.co/guide/en/shield/2.2/privileges-list.html mapped into SG groups . Note that there are cluster and index privileges. These grous are then used in sg_roles.yml
sg_roles.yml - maps users with action groups - e.g. jaeger can READ
sg_roles_mapping.yml - maps users/roles to authentication types

Curator will be authenticated via certificates:
sg_role_curator:
  users:
    - 'CN=system.logging.curator,OU=OpenShift,O=Logging'

Jaeger will use backednroles:jaeger. OpenShiftTokenAuthentication is used as backendrole.
sg_role_jaeger:
  backendroles:
    - 'jaeger'

sg_config.yml - overall SG configuration. It says what auth types SG should use - e.g. OpenShiftTokenAuthentication for service accounts, or authentication_domain_basic_internal.

In this project we are using two auth types: client certs for curator and OpenShiftTokenAuthentication for Jaeger collector and qeury to talk to ES.
OpenShiftTokenAuthentication
This functionality is provided as ES plugin: https://github.com/fabric8io/openshift-elasticsearch-plugin
eg_config.yml
               jaeger:
                 verb: get
                 resource: jaeger
                 namespace: set-in-run-script
                 resourceAPIGroup: elasticsearch.jaegertracing.io

This config uses bearer token from service account to authorize the request. The service account has to be mapped to Role with the same verb, resource, API group and namespace.
The following command can be used to verify whether Jaeger is autorized. Bsically the same request does openshift-elasticsearch-plugin. simple-prod is name of service account maped to jaeger services. Nothe that namespace is overriden in ES run script. We could remove namespace and create ClusterRole instead of Role but then we could deploy only one ES instance in the cluster.
TOKEN=$(oc serviceaccounts get-token simple-prod)
curl -k -v -XPOST  -H "Content-Type: application/json" -H "Authorization: Bearer $TOKEN" https://<IP_ADDRESS>:8443/apis/authorization.k8s.io/v1/selfsubjectaccessreviews -d '{"kind":"SelfSubjectAccessReview","apiVersion":"authorization.k8s.io/v1","spec":{"resourceAttributes":{"group":"elasticsearch.jaegertracing.io","verb":"get","resource":"jaeger", "namespace":"myproject"}}}'

Role:
oc get roles simple-prod-elasticsearch -o yaml                                                                                 3:58
apiVersion: authorization.openshift.io/v1
kind: Role
metadata:
  annotations:
    openshift.io/reconcile-protect: "false"
  creationTimestamp: 2019-02-15T14:29:31Z
  name: simple-prod-elasticsearch
  namespace: myproject
  ownerReferences:
  - apiVersion: io.jaegertracing/v1alpha1
    controller: true
    kind: Jaeger
    name: simple-prod
    uid: 1b925fb1-312e-11e9-91df-8c16456c84e7
  resourceVersion: "24534"
  selfLink: /apis/authorization.openshift.io/v1/namespaces/myproject/roles/simple-prod-elasticsearch
  uid: 1bb109f3-312e-11e9-91df-8c16456c84e7
rules:
- apiGroups:
  - elasticsearch.jaegertracing.io
  attributeRestrictions: null
  resources:
  - jaeger
  verbs:
  - get

SA and RoleBinding:
oc create serviceaccount simple-prod
oc create clusterrolebinding jaeger --clusterrole=simple-prod-elasticsearch --serviceaccount=myproject:jaeger

For easy debugging SearchGuard configuration can be changed in ES image by editing the files. Then it's necessary to run es_seed_acl command to apply changes.
The following commands help to understand the current configuration
es_acl get --doc=rolesmapping
es_acl get --doc=roles
es_seed_acl

The following command can be used to verify the request against ES
oc port-forward es 9200:9200
TOKEN=$(oc serviceaccounts get-token simple-prod)
# TOKEN=$(cat /var/run/secrets/kubernetes.io/serviceaccount/token)
curl -ivX GET -k https://elasticsearch:9200/jaeger-span-2019-01-29/_search -H "Authorization: Bearer $TOKEN"

In https://github.com/jaegertracing/jaeger-operator/pull/325 we have switched from token auth to client certificates for jaeger components.

I think this can be closed, right?

I would like to keep this open as docs issue

I would like to keep this open as docs issue

Did you mean that you want to document this in the operator documentation?

No, this is more like a developer docs.

Where are you creating this? This has been pending for more than a year already.

This issue contains explanation of SearchGuard in ECL Elasticsearch image. It's basically my notes how things work. I thought it will be useful to document it for myself but also for other people who will have to work on it. There was no formal requirement on docs.
If it is a problem to have it here close the issue and I will migrate it somewhere else under my account.

Issues can still be viewed when they are closed, can't they?


usubcription between-device unsubscribe flow - local update?

In section 5.2.2 of the usubscription spec, the upper alt-part of the diagram ('up-app is the last subscriber'): after the unusbscribe call to the remote device returned, there is no update notification going to the local update channel, to indicate status change from UNSUBSCRIBE_PENDING to  UNSUBSCRIBED.
(compare 1st subscription-success case in section 5.1.2, above)
Is this by design, or an omission?

Yes because the uApp just wants to unsubscribe and doesn't care if the remote unsubscribe is successful or not, only the uSubscription Service needs to ensure that the unsubscribe to the remote was successful or not

@AnotherDaniel are we ok to close this?


Added an OIDC AllowGroups option for authorization.

Added an additional OIDC configuration option for a 'groups' claim. The allowed_groups config option is another OIDC authorization option. I tested this with Keycloak by adding a group membership client scope to the OIDC provider.

This looks reasonable, can you add a CHANGELOG.md entry?

@kradalby I added a CHANGELOG.md entry. Let me know if I need to do anything else!

a group membership client scope
How to add  a group membership client scope in keycloak?


ABN-Team

https://abn-team.top/contact.html

Welcome to contact us! Words from YDJSIR!
ABN-Team is welcoming you!


Added a PassDeserializer which turns a pass.json into a Pass object #19

Simple changes to enable converting a pass.json into a Pass object. More work is needed to support the manifest, and extracting this from a zip file.

:+1:

This looks pretty useful, is it going to be merged at some point?


Add property to determine if device is compatible with Apple Pencil

Added a property to the Device enum to determine is the given device can use the Apple Pencil and which version of the Apple Pencil is supported. Current list of supported devices taken off of the Apple Pencil support page: https://support.apple.com/en-us/HT205236

Hello @SomeRandomiOSDev! Could you please rebase your changes onto master - I have merged some breaking changes yesterday and there seem to be merge conflicts. Then I will review the changes, but so far so good and @Zandor300 already looked at the PR in the previous branch and it looked good to him. I will then merge this PR.

Rebasing isn't working out so well due to conflicts between the master branch and these changes. I'm closing this and will open a new PR with my changes cherry picked after the latest changes on the master branch


TensorRT inference error: 'INVALID_ARGUMENT: getPluginCreator could not find plugin InstanceNormalization_TRT version 1'

❓ How to do something using fastreid
I want to train on cuda:4 using "python3 tools/train_net.py --config-file ./configs/Market1501/bagtricks_R50.yml MODEL.DEVICE "cuda:4"", but an error occured: "RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cuda:4!".  And I'm not sure if somewhere fixed to be cuda:0?

You can set the environment variables, for example
CUDA_VISIBLE_DEVICES=4 python3 ..

You can set the environment variables, for example
CUDA_VISIBLE_DEVICES=4 python3 ..

Thank you for your reply! It works.

And I got some errors during transfering onnx to tensorrt engine. So what are the versions of cuda and TensorRT for you? I didn't find your specifications.

You can set the environment variables, for example
CUDA_VISIBLE_DEVICES=4 python3 ..

And I'm now facing some deployment problems. May I ask for what versions of TensorRT and CUDA you're using?

when i run it on tx2 ，errors happens that "no CUDA-capable device is detected"


New version of guest additions

A new version of VirtualBox guest additions has been released (6.0.4 -> 6.0.6): https://www.virtualbox.org/wiki/Downloads
The repo containing the oracle linux vagrant boxes still has 6.0.4 installed. The vagrant-vbguest plugin can be used to automatically upgrade it, but this upgrade has to be done every time you provision the base box which slows things down a little. For the best user experience, a new base box with upgraded guest additions should be created.
I understand that this repo is not for the maintenance of those boxes, but I couldn't figure out where else to bring this up. Hopefully someone here can at least point me in the right direction.

Updated Vagrant Boxes for both OL6 and OL7 are now available.
The same contain all the updated for the OS as well as updated VirtualBox 6.0.6 guest additions.


(modified)button props added "textSizeIOS" for setting button size in…

… ios (ver2)

(Write your motivation here.)
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Bonus points for screenshots and videos!)

Hey, thanks for sending a pull request!\n\nIt looks like your description is missing some necessary information. Can you please add all the details specified in the template? This is necessary for people to be able to understand and review the change you are proposing.


Magic Lamp and RAILS_ENV=test

I didn’t see this in the docs.
So — everything works perfectly… except! (has to be one of those right?)
if I do RAILS_ENV=test before running teaspoon , or do RAILS_ENV=test rake magic_lamp:lint I get:
Error: ActionView::Template::Error: called redirect_to while creating a fixture, this is probably not what you want to have happen.
Is magic lamp “intended” to run with a test rails_env, or only development?

@rstudner It's not intended for any environment in particular. I've even seen people set up a "javascript_test" environment to prevent development data from interfering with fixture data when saving records to the database for use in fixtures.
That being said, it looks like you must be calling a controller method that calls redirect_to conditionally (based on the environment) in one of your fixtures.

I realized after the fact (talking to the more Rails knowledgable folks).  Our development environment, if you weren't logged int, would just make you "user #1"
Thus, RAILS_ENV=test, all the Pundit policies were saying: "hey, you aren't logged in" and redirecting (magic lamp's attempt to render the page) to the login page haha.
So trying to do the magic_lamp_config type setup to actually create/login a user.

We implemented the authstub stuff, so now:
RAILS_ENV=test rake magic_lamp:lint
That totally passes & works.
But trying to run teaspoon, tons of the tests fail still (hrmph).
Trying now to print out the fixture dom's that are being rendered (or not) since of course with no output/visibility its hard to see what is going on hah.

I've seen people do things like this to get around some can can pain:
fixture do
  def self.can?(*args); true; end
  render :edit
end

Teaspoon might have some issues as a result of how it mounts its engine in your application. I know I've personally had issues with Teaspoon + Devise + Better Errors.

All working 100% on our dev boxes, in either environment :)
But -- of course -- it is failing on our jenkins build machine haha (no sure why yet).  One piece at a time!
Thanks for all your help!


fix service ip and port in consul interpolateService

If you use the flag "-internal" is substituted not a valid port to $SERVICE_CHECK_SCRIPT.
Please view and accept the changes, they solve this problem.

SERVICE_CHECK_SCRIPT=nc $SERVICE_IP $SERVICE_PORT | grep OK

@mattaitchison seems reasonable, can I merge?

Hi,
So why is it still not merged?

Anyone alive?

with -internal options, the $SERVICE_IP $SERVICE_PORT got replaced with nil string:
--env SERVICE_CHECK_SCRIPT="nc -vz $SERVICE_IP $SERVICE_PORT" \

2017/07/12 07:34:42 [DEBUG] agent: Check 'service:44f2dc3869c4:registry_dev.1.crke338u27hsscqhmdm0zmpz7:5000' script 'nc -vz  ' output: BusyBox v1.26.2 (2017-06-11 06:38:32 GMT) multi-call binary.

Usage: nc [OPTIONS] HOST PORT  - connect
nc [OPTIONS] -l -p PORT [HOST] [PORT]  - listen

	-e PROG	Run PROG after connect (must be last)
	-l	Listen mode, for inbound connects
	-lk	With -e, provides persistent server
	-p PORT	Local port
	-s ADDR	Local address
	-w SEC	Timeout for connects and final net reads
	-i SEC	Delay interval for lines sent
	-n	Don't do DNS resolution
	-u	UDP mode
	-v	Verbose
	-o FILE	Hex dump traffic
	-z	Zero-I/O mode (scanning)

Is this already in the master image of docker hub?


Increase timeout for cluster pool to 2 hours

Should be more reliable during rush hours.

@ReToCode Sorry, I had to fix the unit test.


Remove some unneeded ClipRect widgets

Stack clips internally if it has visual overflow and Image never draws outside
its bounds.

@Hixie @HansMuller

This change goes beyond just removing unneeded clip rects. The drawer demo had been set up to resemble the drawers in Google apps like GMail, Drive, Photos, etc. Although the new version is simpler, it doesn't demo how to code the dual-view drawer that appears in our products.

The Stack dartdoc still doesn't mention clipping explicitly.

This change goes beyond just removing unneeded clip rects.

The diff might look confusing, but all I did was remove the clip.

Actually, now that I look at it more, I don't think we can remove that clip.

Sorry about that, I have bad diff reading habits.
LGTM

Ok, now I think this patch is just wrong.  I'm going to abandon it.


Loading weights from checkpoint file fails [python/c api]

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): stock example and costume code
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): Model created with pip package, training performed with C api from compiled libtensorflow
TensorFlow version (use command below): 2.0rc
Python version: 3.6
Bazel version (if compiling from source): 0.26.1
GCC/Compiler version (if compiling from source): 7.4
CUDA/cuDNN version:
GPU model and memory:

You can collect some of this information using our environment capture
script
My environment
tf_env.txt
Describe the current behavior
I am using tensorflow's c API to train models created with tf.keras in python. After training in my C program a checkpoint file is created from which a model's weights can be restored for more training or prediction within my C program.
When trying to restore my model in python, trained weights can not be loaded neither by:
model.load_weights("path_to_checkpoint")

nor with
checkpoint = tf.train.Checkpoint(model=model)
checkpoint.restore("path_to_checkpoint")

The model loads successfully with:
model = keras.experimental.load_from_saved_model("path_to_model_folder")

But this is an untrained model. To be precise, this model has the same weights as when the model was first defined.
Describe the expected behavior
In my C program I can load/save/restore my model with no problem, I am expecting to be able to load trained model and make predictions in python as well from the model trained in my C program.
Code to reproduce the issue
There are several steps that I will describe to accurately reproduce my problem.
**1. Create model with:
keras_model.py
python keras_model.py

This will create a keras model and export it in keras_model, this folder name is hard coded in the C program
2. Get data with:
getData.py
python getData.py

This will download the fashion MNIST dataset and stored the training examples and labels as text files in:
data.txt and labels.txt These names are hardcoded in the C program
3. Compile C program
The only requirement for my C program is the tensorflow library shared object files libtensorflow.so and libtensorflow_framework.so and the c API header file c_api.h. These were compiled with bazel via:
./configure
bazel build -c opt //tensorflow/tools/lib_package:libtensorflow

Extracting the shared object files from libtensorflow.tar.gz
My C program can then be compiled with:
gcc -Wall -I path_to_libtensorflow/include -L path_to_libtensorflow/lib -o BellyTF BellyTF.c -ltensorflow

Assuming data.txt, labels.txt and keras_model are in the same directory as the executing command run:
LD_LIBRARY_PATH=path_to_libtensorflow_lib ./BellyTF

This will load saved model, and training data, print predictions from untrained model, train for a number of epochs, print predictions from trained model and save weights as a checkpoint file in the variables folder inside keras_model
4. Load trained mdel in python from checkpoints file
This is where I encounter problems
import tensorflow as tf
from tensorflow import keras
#Get data
fashion_mnist = keras.datasets.fashion_mnist
(train_images, train_labels), (test_images, test_labels) = fashion_mnist.load_data()
#scale and reshape
train_images = train_images/255
train_images = np.reshape(train_images,(60000,784))

#Load model
model = keras.experimental.load_from_saved_model("keras_model")
#Predictions are wrong
model.predict([[train_images[0]]])

#Try to load weights This produces AssertError (traceback at end of issue)
model.load_weights("keras_model/variables/belly")
AssertionError: Some objects had attributes which were not restored:

#Try load model via tf.train.Checkpoint
checkpoint = tf.train.Checkpoint(model=model)
status =  checkpoint.restore("keras_model/variables/belly")

#Prediction is wrong
model.predict([[train_images[0]]])

Other info / logs
Traceback from
model.load_weights("keras_model/variables/belly")
---------------------------------------------------------------------------
AssertionError                            Traceback (most recent call last)
<ipython-input-39-db6e544ddcae> in <module>
----> 1 model.load_weights("keras_model/variables/belly")

~/Projects/tensorflow/env/lib/python3.6/site-packages/tensorflow/python/keras/engine/training.py in load_weights(self, filepath, by_name)
    160         raise ValueError('Load weights is not yet supported with TPUStrategy '
    161                          'with steps_per_run greater than 1.')
--> 162     return super(Model, self).load_weights(filepath, by_name)
    163
    164   @trackable.no_automatic_dependency_tracking

~/Projects/tensorflow/env/lib/python3.6/site-packages/tensorflow/python/keras/engine/network.py in load_weights(self, filepath, by_name)
   1396         # streaming restore for any variables created in the future.
   1397         trackable_utils.streaming_restore(status=status, session=session)
-> 1398       status.assert_nontrivial_match()
   1399       return status
   1400     if h5py is None:

~/Projects/tensorflow/env/lib/python3.6/site-packages/tensorflow/python/training/tracking/util.py in assert_nontrivial_match(self)
    915     # assert_nontrivial_match and assert_consumed (and both are less
    916     # useful since we don't touch Python objects or Python state).
--> 917     return self.assert_consumed()
    918
    919   def _gather_saveable_objects(self):

~/Projects/tensorflow/env/lib/python3.6/site-packages/tensorflow/python/training/tracking/util.py in assert_consumed(self)
    892       raise AssertionError(
    893           "Some objects had attributes which were not restored: %s" %
--> 894           (unused_attributes,))
    895     for trackable in self._graph_view.list_objects():
    896       # pylint: disable=protected-access

AssertionError: Some objects had attributes which were not restored: {<tf.Variable 'dense_2/kernel:0' shape=(784, 128) dtype=float32, numpy=
array([[ 0.01530321, -0.07596322, -0.01604562, ...,  0.03402978,
         0.07713103, -0.05052958],
       [ 0.00116051,  0.04680964, -0.03951511, ..., -0.03624759,
        -0.04315009,  0.06628483],
       [ 0.0188334 , -0.06348058, -0.04110793, ..., -0.02473556,
        -0.04467853, -0.00360004],
       ...,
       [-0.01828651, -0.07760178, -0.06535166, ..., -0.04309935,
        -0.08069813,  0.00902149],
       [ 0.04599006,  0.02525605,  0.06842359, ..., -0.03162104,
        -0.02693111,  0.06349993],
       [-0.02491769,  0.0611873 , -0.00206578, ..., -0.01271369,
         0.00109962, -0.07020341]], dtype=float32)>: ['dense_2/kernel'], <tf.Variable 'dense_2/bias:0' shape=(128,) dtype=float32, numpy=
array([0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0.,
       0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0.,
       0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0.,
       0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0.,
       0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0.,
       0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0.,
       0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0.,
       0., 0., 0., 0., 0., 0., 0., 0., 0.], dtype=float32)>: ['dense_2/bias'], <tf.Variable 'dense_1_1/kernel:0' shape=(128, 10) dtype=float32, numpy=
array([[ 0.16446821, -0.17563525,  0.18357025, ...,  0.08780123,
         0.07823227,  0.10018779],
       [ 0.0915112 , -0.17932694,  0.12889476, ...,  0.20635314,
        -0.09271443,  0.11488943],
       [-0.08094949, -0.14578271, -0.01433699, ..., -0.11150974,
        -0.19056591, -0.01099543],
       ...,
       [-0.08208692,  0.06081717,  0.0688145 , ..., -0.08729573,
         0.20750098, -0.08333559],
       [-0.15593694,  0.14401685, -0.20091254, ..., -0.15566093,
        -0.06369022, -0.12938659],
       [-0.18293834,  0.08427195, -0.02965383, ...,  0.02141935,
        -0.05810072, -0.04387315]], dtype=float32)>: ['dense_1_1/kernel'], <tf.Variable 'dense_1_1/bias:0' shape=(10,) dtype=float32, numpy=array([0., 0., 0., 0., 0., 0., 0., 0., 0., 0.], dtype=float32)>: ['dense_1_1/bias']}

If the C program needs more commenting please let me know

It is challenging to debug large scripts a minimal reproducible code snippet is appreciated.
This question is better asked on StackOverflow since it is not a bug or feature request. There is also a larger community that reads questions there.

Ive tried stackoverflow without success. The C api does not receives a lot of attention so I'm not sure where to go from here. Documentation is very poor in this sense


Build is always 'up-to-date' (GNU)

Environment
GNU-specific duplicate of #214. Pull requesting incoming.

Visual Studio 2015 Community Update 1
Rust 1.7 GNU
VisualRust 0.1.2

Repro

Create a new project
Run the project. Outcome: Hello World!
Add the following to main: println!("Test");
Optional: save main.rs.
Run the project. Outcome: Hello World! Expected: Hello World!<newline>Test.
Right click on the project and select Rebuild.
Run the project. Outcome: as expected.

Summary
The build is not detecting changes to files.
Remarks
An older version of Visual Rust (and Rust) was installed on the environment, but uninstalled prior to installing these latest versions.

This should no longer appear with #234 ... can you confirm that?

This is working perfectly.


Use py.test for regular tests and for slow tests

[x] squash some commits

Changes Unknown when pulling b3a3a5f17b8ed376b2d624e4465a10ca4d65c4ef on use-py.test into ** on master**.

Changes Unknown when pulling b3a3a5f17b8ed376b2d624e4465a10ca4d65c4ef on use-py.test into ** on master**.

Changes Unknown when pulling b93d6ca1128debfeeaf6eb7d6a121cf0eadd73cd on use-py.test into ** on master**.


package request: hastebin server (haste-server)

Project description
great looking pastebin server, as seen on hastebin.com
Metadata

homepage URL: "https://hastebin.com/about.md"
source URL: "https://github.com/seejohnrun/haste-server"
license: mit
platforms: unix

project seems abandoned


[docs] document the Scope struct

The scope::Scope struct in rayon-core needs documentation. The use of scopes is reasonably well documented on the scope::scope function, so probably the type itself just needs to say something like "Represents a fork-join scope which can be used to spawn any number of tasks. See scope() for more information."

Can this issue be closed?

Ah, yes, thanks!


don't overwrite on default

I just lost un-checked in code due to that :-

Can you please provide a short example?

Program was in cwl-upgrader/main.py, CWL description was in
cwl-upgrader/main.cwl.
I invoked cwl2argparse -d cwl-upgrader cwl-upgrader/main.cwl and lost the
contents of cwl-upgrader/main.py
On Tue, Aug 9, 2016 at 6:50 PM, Anton Khodak<EMAIL_ADDRESS>wrote:

Can you please provide a short example?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/common-workflow-language/cwl2argparse/issues/10#issuecomment-238597410,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABROCFAGJplHuJEfVpR-oehQbY7I_8prks5qeKG6gaJpZM4JgLMx
.

--
Michael R. Crusoe
Community Engineer & Co-founder
Common Workflow Language project
https://impactstory.org/u/0000-0002-2961-9670
<EMAIL_ADDRESS>+40 720 781 765
+1<PHONE_NUMBER>

Ouch! Will keep this case in mind

Thanks!
On Tue, Aug 9, 2016 at 8:46 PM, Anton Khodak<EMAIL_ADDRESS>wrote:

Closed #10
https://github.com/common-workflow-language/cwl2argparse/issues/10 via
a318043
https://github.com/common-workflow-language/cwl2argparse/commit/a318043ff94428104c79ba1347d9c9959ecf2c6b
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/common-workflow-language/cwl2argparse/issues/10#event-750207919,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABROCOawnL0Dyvuz6rOonnqIE772j7cIks5qeL0RgaJpZM4JgLMx
.

--
Michael R. Crusoe
Community Engineer & Co-founder
Common Workflow Language project
https://impactstory.org/u/0000-0002-2961-9670
<EMAIL_ADDRESS>+40 720 781 765
+1<PHONE_NUMBER>


Beginner friendly method: Create react app added

Closes #5781.
Added npx create-react-app for beginners trying to create static content.
Moved a element accordingly inside a element.

Thanks for the PR but as described in the deep dive on this page (that you moved), we don't recommend this for new apps. Our hope is that frameworks are actually more beginner-friendly, not less – and they scale with you as your app grows much better than Create React App and similar SPA-only tools do.

Well, that depends. In our company eg we use .Net Core for the backends. And we do not need SSR, ... and additional stuff, therefore we use JUST React (with typescript) and CSR. Of course, altough these frameworks add a lot of functionality, they also add some stuff to think of and keep in mind.
So, eg beginners will have to know more than just how to do react in order to use the boilerplates.
In the end "create-react-app" is JUST React, not more, but not less. I think there is still place and need for this
But will it, altough documentation goes away, still be available and maintained?

Also
https://create-react-app.dev/
still and only mentions the "old" create-react-app. I just don´t understand, why we should hide the way, we used to do it in the past, just because there are now some additional features there. Even if the new ones are the recommended, there is nothing preventing to document and mention the "plain" version
@harish-sethuraman @sophiebits

@LeopoldLerch If you need something more minimal than a full framework, you might want to give Vite a try, with the react or react-ts template: https://v3.vitejs.dev/guide/#trying-vite-online
The docs on the react.dev site lean towards recommending a framework, but it does briefly mention vite
under this popout section:
https://react.dev/learn/start-a-new-react-project#can-i-use-react-without-a-framework


[BUG]:  Previous link remains in the "Add Link" modal when reopening for new link

Describe the bug
When we try to add a new link by reopening the "Create Link" modal form, the modal still contains the previously entered link text and URL. The modal should reset its form fields (link text and URL) when it is reopened, so that the user can start with a fresh form.
Steps to reproduce
1.Open the editor and click to create a link.
2.Enter a link text and URL in the modal and submit it.
3.Click to add another link and observe that the previous link’s data (text and URL) is still present in the modal fields.
Expected behavior
When reopening the "Create Link" modal, the fields for link text and URL should be empty, allowing the user to input a new link.
What is actually happening?
It is not Clearing the modal form fields when the modal is opened for a new link. Reseting the state  is needed.
Screenshots
No response
Any additional comments?
No response

Hello,
I would like to work on resolving this issue . Could you please assign it to me?
I’ll ensure the modal’s state is reset properly when reopening, and I’ll take care of ensuring the link styling and alignment don’t carry over to subsequent text.
Please let me know if there are any specific guidelines or requirements I should follow for this fix. Thank you!

Hi @geekychandan , you can contribute by  forking the repo and raising pr against the issue will check code and merge , you have to follow certain coding guidelines , and code of conduct please read code_of_conduct.md and contributing.md for more information


Avoid GNUC warnings when defining DECLARE_DEPRECATED [1.1.0]

We need to check that GNUC is defined before trying to use it.
This demands a slightly different way to define DECLARE_DEPRECATED.

This is a backport of #6680 for 1.1.0

This is a backport of #6680 for 1.1.0

Along with red cross from travis.

Thanks for the reminder.  Fixup pushed

Itäs looking good so far...

Right you are, @petrovr.  Fixed

Richard, this pr has now deviated from #6680. Do you intend to "forward port" your additional changes to master?

The "additional change" comes from #6689, which should really have been part of #6680.

What you're missing is that no-deprecated is handled differently between 1.1.0 and 1.1.1.
However, the #else part in this corresponds to the change in #6680, does it not?

What you're missing is that no-deprecated is handled differently between 1.1.0 and 1.1.1.

Ok, I understand now. I had thought about that possibility, but I overlooked 973abf52304b454e8f2220a11aa8b6c6512c44fb. Maybe it would be a good idea to add a short note to the CHANGES file?
msp@msppc:~/src/openssl$ grep -r NO_DEPRECATED
CHANGES:1175:  *) Revert default OPENSSL_NO_DEPRECATED setting.  Instead OpenSSL
CHANGES:1446:  *) config has been changed so that by default OPENSSL_NO_DEPRECATED is used.
CHANGES:5496:     this change is covered by the OPENSSL_NO_DEPRECATED symbol. As always,
CHANGES:5558:     if OPENSSL_NO_DEPRECATED is defined, BN_zero() is a void macro.
CHANGES:5611:     OPENSSL_NO_DEPRECATED is defined. Some "openssl" subcommands and a few of
CHANGES:5807:     the new ones. The OPENSSL_NO_DEPRECATED symbol can be used to hide
NEWS:507:        functionality with the OPENSSL_NO_DEPRECATED macro or the

However, the #else part in this corresponds to the change in #6680, does it not?

Agreed. :-)

Maybe it would be a good idea to add a short note to the CHANGES file?

Maybe you should read the whole entry for that line you found (line 1175) ;-)

Ok, you win. And I'll shut up ;-).

... or if you're satisfied, you could approve ;-)

Merged.
dcb8333087d56eef97c482aabb506b8be8299cde Avoid GNUC warnings when defining DECLARE_DEPRECATED
Thank you!


Replacing CBV URN

Hi,
https://github.com/RalphTro/epcis-event-hash-generator/blob/ff05d4782959b4a3087dbf61bec0f1b951844125/epcis_event_hash_generator/hash_generator.py#L138
https://github.com/RalphTro/epcis-event-hash-generator/blob/ff05d4782959b4a3087dbf61bec0f1b951844125/epcis_event_hash_generator/hash_generator.py#L140
https://github.com/RalphTro/epcis-event-hash-generator/blob/ff05d4782959b4a3087dbf61bec0f1b951844125/epcis_event_hash_generator/hash_generator.py#L142
https://github.com/RalphTro/epcis-event-hash-generator/blob/ff05d4782959b4a3087dbf61bec0f1b951844125/epcis_event_hash_generator/hash_generator.py#L144
Shouldn't it be https://ns.gs1.org/cbv/XXX- like detailled in the readme?
If present, any URN-based standard vocabulary value (starting with ‘urn:epcglobal:cbv’) SHALL be expressed in its
corresponding GS1 Web Vocabulary URI equivalent (starting with ‘https://ns.gs1.org’). Example:
‘urn:epcglobal:cbv:bizstep:receiving’ --> ‘https://ns.gs1.org/cbv/BizStep-receiving’

Dear @clementh59 ,
Another good catch - thanks a lot! When this was implemented, there was no final consensus yet as to the URI structure. We also need to think how the short strings for all CBV standard elements (e.g. shipping instead of https://ns.gs1.org/cbv/BIZSTEP-shipping - which will be allowed in JSON/JSON-LD, but not in XML), will impact the construction of the pre-hash string.
This is another thing which was added after the SW package was published. While it makes other things easier, it complicates the construction of the pre-hash string.
How do you like the following suggestion: let's wait until the draft spec are motioned to Public Community Review (expected by End of September), by which time all this should be pretty stable, and then build in an appropriate update?
@Echsecutor : let's discuss on how this could be tackled some time after your holiday. As I do not want us to update the algorithm everytime some values are added to the CBV, my spontaneous thought on how this could be solved quite nicely would be to strip off 'urn:epcglobal:cbv:bizstep:' everytime a URN is recognised in an XML document and to copy and paste the value 1:1 from a JSON/JSON-LD structure.
Kind regards,
@RalphTro

Dear @RalphTro ,

We also need to think how the short strings for all CBV standard elements (e.g. shipping instead of https://ns.gs1.org/cbv/BIZSTEP-shipping - which will be allowed in JSON/JSON-LD, but not in XML), will impact the construction of the pre-hash string.

Yes, good catch too.

How do you like the following suggestion: let's wait until the draft spec are motioned to Public Community Review (expected by End of September), by which time all this should be pretty stable, and then build in an appropriate update?

Yes, good idea.
Best regards,
Clément

New Examples by Ralph in https://github.com/RalphTro/epcis-event-hash-generator/pull/66 -> Continue work on this issue in that PR!

Update
Dear @clementh59 ,
This issue can be closed from my POV - when executing the code (master branch) for e.g. a JSON/JSON LD file accommodating bare string notation, it neatly expands to its canonical form (e.g.  "incorrect_data"--> "https://ref.gs1.org/cbv/ER-incorrect_data" or "in_progress"--> "https://ref.gs1.org/cbv/Disp-in_progress"
Would you agree with this?
Kind reards,
Ralph

Yes, I do!
Thanks a lot,
Clément


Quantifiers for exact numbers of matches do not work

While converting a project from Flex, we found that quantifiers that are supposed to be matched an exact number of times do not work in some cases. When used in pattern such as this one:
[1-4][A]([0-9]|[A-Z]|[a-z])([0-9]|[A-Z]|[a-z])[0-9]{3}
The quantifier does not seem to work. It captures all numbers at the end of the pattern (not just 3). The pattern matches if we remove the quantifier and just put [0-9] three times at the end.
Is this a bug or are we missing something?

My tests do not reveal any issues. Here is the DFA of [1-4][A]([0-9]|[A-Z]|[a-z])([0-9]|[A-Z]|[a-z])[0-9]{3}:

Perhaps the problem is that another pattern in your scanner matches a longer string? You can find out which pattern matched a string, with reflex option -d to enable debugging.

Thanks for your feedback. The pattern matches, but matches even subsequent numbers.
It is quite a strange bug, because we do have similar projects, that we have converted without issue. I guess we will have to look deeper.

Again, I do not see any issue with this pattern and also not in our tests with various range quantifiers.
First thing to test on your side is to verify that it is indeed this specific pattern that misbehaves, which you can do by running the lexer in debug mode. This is important to verify, since other overlapping longer patterns in your lexer specification may match instead, such as [1-4]A[0-9]*. The lexer always accepts the longest leftmost match. If you find that it is the specific pattern that still misbehaves, then you can narrow the problem down by removing as many of the other patterns until the smallest set of patterns still show this issue. Report this set if patterns to me and I'll take a closer look.

Thanks for your help. We used the debug option and we can confirm that is this pattern that misbehaves.
We like your library and that has been very useful. I understand that can be frustrating (and impossible) to debug just a small example. I started the issue mainly to understand if there was something obvious that we were missing.
We are going to assemble that set of patterns, but it will take some time.
I don't know if it is relevant, but we found that using the matcher boost solves our issues. Do you know what could cause that?

The Boost option uses a different regex library.
I'm very interested to learn what combination of patterns cause this very specific pattern to misbehave in the reflex regex library. The library is tested extensively, but we're always looking for ways to improve it. When you have a set of patterns for us to debug please share these so we can release an update soon.

I think we found the problem. There is a bug when two or more range quantifiers are used "in parallel", for example a{3}|a{4} results in accepting a{3,} (more than 3 a's). Will get that fixed!

Impressive, that was quick. As soon as you update the library I will report if it fixes our issue.

The patch is to replace line lib/pattern.cpp:664 with the comparison if (fp->first.loc() >= b_pos):
        // update followpos by virtually repeating sub-regex m-1 times
        Follow followpos1;
        for (Follow::const_iterator fp = followpos.begin(); fp != followpos.end(); ++fp)
          if (fp->first.loc() >= b_pos) // FIXED
            for (Index i = 1; i < m; ++i)
              for (Positions::const_iterator p = fp->second.begin(); p != fp->second.end(); ++p)
                followpos1[fp->first.iter(iter * i)].insert(p->iter(iter * i));

A version update will be released very soon!

This update did not solve our problem. I am still working on the minimal set of rules that cause this error.


255 forward pandas apply function to modify data in row column and table

Closes #[issue_no].
Summary of Changes
Users are now able to call Column#transform to apply an anonymous function to every data point within the column

🦙 MegaLinter status: ❌ ERROR

Descriptor
Linter
Files
Fixed
Errors
Elapsed time

✅ PYTHON
black
2
0
0
0.64s

❌ PYTHON
mypy
2

1
1.61s

❌ PYTHON
ruff
2
2
9
0.05s

✅ REPOSITORY
git_diff
yes

no
0.03s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff
MegaLinter is graciously provided by

:tada: This PR is included in version 0.12.0 :tada:
The release is available on:

v0.12.0
GitHub release

Your semantic-release bot :package::rocket:


PKG-5524-update-0.9.2

Destination channel: defaults
Links

PKG-5524
Upstream repository

Explanation of changes:

Update version, sha, deps

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_cfvnb5iom8/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:45: missing_description: The recipe is missing a description

===== ERRORS =====

clone/recipe/meta.yaml:38: missing_pip_check: For pypi packages, pip check should be present in the test commands
clone/recipe/meta.yaml:45: missing_dev_url: The recipe is missing a dev_url
clone/recipe/meta.yaml:45: missing_documentation: The recipe is missing a doc_url or doc_source_url
clone/recipe/meta.yaml:45: http_url: http://github.com/ContinuumIO/tranquilizer is not https
===== Final Report: =====
4 Errors and 1 Warning were found


vscode-go: changing testFlag setting requires restart

Go version
go version go1.23-20240317-RC00 cl/616607620 +0a6f05e30f X:fieldtrack,boringcrypto linux/amd64
Output of go env in your module/workspace:
doesn't matter

What did you do?
in settings.json, i changed the value of go.TestFlags
What did you see happen?
clicking on the "run test | debug test" line above the function definition no longer works, requiring gopls to be restarted (Developer: reload window)
What did you expect to see?
At least a warning that the window needed to be reloaded. It would be even better if the window did not have to be reloaded.

This doesn't always happen, and I'm not sure when it does or doesn't.


Request to guide about how to include issue templates on github repository without showing on the repository file list

Hello sir,
I was trying to add issue templates to my github repository without showing the .github/issue_templates on my repository file list, as you have done for your repositories.
I have tried many different things, e.g., by using gitignore file or by .git/info/exclude, but could not do it as you have done it.
Can you please guide me of how can i achieve the same functionality for issue_templates.

Still need help with that, @alexcurrann?

Still need help with that, @alexcurrann?

No, i figured out how to do it, still thank you for reaching back


MacForge crashes on MacOS Big Sur M1

Bug Report
Current Behavior
No plugins installed works
To Reproduce
Steps to reproduce the behavior:

Install MacForge on Big Sur
Open MacForge
See Crash Log in Console

Expected behavior/code
A clear and concise description of what you expected to happen (or code).
Plugins work
Screenshots
If applicable, add screenshots to help explain your problem.
Environment

MacForge version(s): [e.g. v0.10.1] v1.0.0
OS version: [e.g. macOS 10.13.4] 11.2
Plugins: [e.g. cleanHUD] AfloatX

Possible Solution

Additional context
Add any other context about the problem here.
Process:               com.macenhance.MacForge.Injector [6415]
Path:                  /Library/PrivilegedHelperTools/com.macenhance.MacForge.Injector
Identifier:            com.macenhance.MacForge.Injector
Version:               ???
Code Type:             X86-64 (Translated)
Parent Process:        launchd [1]
Responsible:           com.macenhance.MacForge.Injector [6415]
User ID:               0

Date/Time:             2022-02-04 09:15:17.369 -0500
OS Version:            macOS 11.3.1 (20E241)
Report Version:        12
Anonymous UUID:        C5F3193D-A4F0-BE7A-28BD-8EC6BD830E99

Time Awake Since Boot: 9300 seconds

System Integrity Protection: disabled

Crashed Thread:        3  Dispatch queue: com.apple.NSXPCConnection.user.com.macenhance.MacForge.Injector.mach.6409

Exception Type:        EXC_BAD_ACCESS (SIGSEGV)
Exception Codes:       KERN_INVALID_ADDRESS at 0x0000000000ff0000
Exception Note:        EXC_CORPSE_NOTIFY

Termination Signal:    Segmentation fault: 11
Termination Reason:    Namespace SIGNAL, Code 0xb
Terminating Process:   exc handler [6415]

VM Regions Near 0xff0000:
-->
    __TEXT                      1046c3000-1046c7000    [   16K] r-x/r-x SM=COW

Application Specific Information:
objc_msgSend() selector name: code

Thread 0:: Dispatch queue: com.apple.main-thread
0   ???                           	0x00007ffe9454eab8 ???
1   libsystem_kernel.dylib        	0x00007fff202fd32a mach_msg_trap + 10
2   com.apple.CoreFoundation      	0x00007fff2042b9c5 __CFRunLoopServiceMachPort + 316
3   com.apple.CoreFoundation      	0x00007fff2042a07b __CFRunLoopRun + 1332
4   com.apple.CoreFoundation      	0x00007fff20429480 CFRunLoopRunSpecific + 567
5   com.apple.Foundation          	0x00007fff210ddb47 -[NSRunLoop(NSRunLoop) runMode:beforeDate:] + 212
6   com.apple.Foundation          	0x00007fff2116ba11 -[NSRunLoop(NSRunLoop) run] + 76
7                                 	0x00000001046c6299 0x1046c3000 + 12953
8   libdyld.dylib                 	0x00007fff2034df3d start + 1

Thread 1:: com.apple.rosetta.exceptionserver
0   runtime                       	0x00007ffdffe9a878 0x7ffdffe99000 + 6264
1   runtime                       	0x00007ffdffea4d70 0x7ffdffe99000 + 48496
2   runtime                       	0x00007ffdffea5a40 0x7ffdffe99000 + 51776

Thread 2:
0   runtime                       	0x00007ffdffef399c 0x7ffdffe99000 + 371100

Thread 3 Crashed:: Dispatch queue: com.apple.NSXPCConnection.user.com.macenhance.MacForge.Injector.mach.6409
0   libobjc.A.dylib               	0x00007fff201d0797 objc_msgSend + 23
1   com.apple.Foundation          	0x00007fff2113d1c6 __NSXPCCONNECTION_IS_CALLING_OUT_TO_EXPORTED_OBJECT_S1__ + 10
2   com.apple.Foundation          	0x00007fff210e80c8 -[NSXPCConnection _decodeAndInvokeMessageWithEvent:flags:] + 2271
3   com.apple.Foundation          	0x00007fff2109f749 message_handler + 206
4   libxpc.dylib                  	0x00007fff20078c24 _xpc_connection_call_event_handler + 56
5   libxpc.dylib                  	0x00007fff20077a9b _xpc_connection_mach_event + 938
6   libdispatch.dylib             	0x00007fff20188886 _dispatch_client_callout4 + 9
7   libdispatch.dylib             	0x00007fff2019faa0 _dispatch_mach_msg_invoke + 444
8   libdispatch.dylib             	0x00007fff2018e473 _dispatch_lane_serial_drain + 263
9   libdispatch.dylib             	0x00007fff201a05e2 _dispatch_mach_invoke + 484
10  libdispatch.dylib             	0x00007fff2018e473 _dispatch_lane_serial_drain + 263
11  libdispatch.dylib             	0x00007fff2018f0c0 _dispatch_lane_invoke + 417
12  libdispatch.dylib             	0x00007fff20198bed _dispatch_workloop_worker_thread + 811
13  libsystem_pthread.dylib       	0x00007fff2032f4c0 _pthread_wqthread + 314
14  libsystem_pthread.dylib       	0x00007fff2032e493 start_wqthread + 15

Thread 3 crashed with X86 Thread State (64-bit):
  rax: 0x0000000000000001  rbx: 0x000000030d2eb1b0  rcx: 0xbf30a3c18fc7005a  rdx: 0x0000000000000016
  rdi: 0x0000000000ff0000  rsi: 0x00007fff7baed50b  rbp: 0x000000030d2eb220  rsp: 0x000000030d2eb178
   r8: 0x000000000000004d   r9: 0x0000000000000d85  r10: 0x00007ffffffffff8  r11: 0x00007fff203b3234
  r12: 0x000000030d2eb1a8  r13: 0x00007fff8093e0e0  r14: 0x00007fa4dc8040a0  r15: 0x00007fff201d0780
  rip: 0x00007fff201d0797  rfl: 0x0000000000000243

Does someone know how do we install de Beta version which includes M1 support?

Does someone know how do we install de Beta version which includes M1 support?

here: https://github.com/MacEnhance/appcast/raw/master/Beta/MacForge/MacForge.1.2.0B1.zip
But in my mac, it doesn't work with AfloatX.


[WIP] Clean up subject selection

I've removed all of the checks to see if we need to call out to Cellect/CellectEx before enqueueing the jobs. Instead, it'll just do that when it runs. However, I can see a case to be made for keeping it. If you like, I would suggest doing that in an override to perform_async in the worker, rather than littering the rest of the codebase with it.

I'll incorporate your comments and merge. I've got more refactorings coming, but let's get the notifications stuff out the door.


Assembly number of 0.7 release is still 0.6.1

I noticed that in the most recent release for some reason the assembly version is still 0.6.1. Not 0.7.
https://github.com/ericsink/SQLitePCL.raw/blob/master/src/cs/AssemblyInfo.cs

Noticed you bumped the version number. Wanted to ping on this in case you do another release :)

Yeah, thanks for the reminder.  This commit was just a test for somebody.
E
On Monday, January 19, 2015, Dave<EMAIL_ADDRESS>wrote:

Noticed you bumped the version number. Wanted to ping on this in case you
do another release :)
—
Reply to this email directly or view it on GitHub
https://github.com/ericsink/SQLitePCL.raw/issues/31#issuecomment-70535089
.


d1g1f25tn8m2e6.cloudfront.net

Hi there,
d1g1f25tn8m2e6.cloudfront.net is a false-positive. From what I gathered, the domain is used for Uber Eats' promotional imagery throughout their application. This domain blocked prevents the images from loading at all. Not sure if there's an analytics payload on this domain, but I may have missed it.
Thanks!

It is not currently blocked, might have been in an old version of the list


Vickel: Watch itjahangir(?:@outlook.com)?

Vickel requests the watch of the watch_keyword itjahangir(?:@outlook\.com)?. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

itjahangir(?:@outlook\.com)? has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by cocomac in Charcoal HQ


--watch not working with helix-editor

What version of Hugo are you using (hugo version)?

$ hugo version
hugo v0.129.0+extended darwin/amd64 BuildDate=2024-07-17T13:29:16Z VendorInfo=brew

Steps to reproduce:

Run hugo server or hugo --watch
Change files in Helix

Expected result
Hugo reacts to file changes and rebuilds pages
Actual result
Hugo reacts to file changes and builds pages, but not all of them and the output is the same as the one without the changes.
This only happens in Helix editor. In vim everything works as expected.

Could you try to run
hugo server --logLevel info

And note the lines with livereload in them when you change something.
You could also try hugo server --poll 1s. If that works, the Helix must doing some non-standard way of saving files that does not trigger the expected fs events.

This is the output with --logLevel info
Change detected, rebuilding site (#1).
2024-07-25 16:35:34.441 +0200
Template added /_default/baseof.htmlc3gzwd.bck
Template added /_default/baseof.html
INFO  build:  step process substep resolve content adapter change set changes 2 checked 2 matches 0 duration 36.901µs
INFO  build:  step process substep resolve page output change set changes 2 checked 448 matches 0 duration 3.843363ms
INFO  build:  step process substep gc dynacache duration 196.695µs
INFO  build:  step process substep rebuild templates duration 8.210313ms
INFO  build:  step process duration 13.327644ms
INFO  build:  step assemble duration 777ns
INFO  build:  step render substep pages site de outputFormat html duration 20.021809ms
INFO  build:  step render pages 0 content 0 duration 20.075914ms
INFO  build:  step render deferred count 0 duration 917ns
INFO  build:  step postProcess duration 9.17µs
INFO  build:  duration 33.51625ms
Web Server is available at http://localhost:1313/ (bind address <IP_ADDRESS>)
Total in 33 ms
INFO  livereload: build changed 0 files

adding --poll 1s fixes the issue.

This is not a hugo issue but a helix issue.

This is not a hugo issue but a helix issue.

Well, that's debatable.
Looking at these 2 lines:
Template added /_default/baseof.htmlc3gzwd.bck
Template added /_default/baseof.html

I'm guessing that the above is hinting at the problem; I suspect that Helix saves to a temp file and then renames it, nos sure.

Well, that's debatable.

adding --poll 1s fixes the issue.

I just spent the day tearing my hair out trying to solve this problem, and this solved it! Maybe there is some better way to deal with this long term than adding --poll 1s?

This is not a hugo issue but a helix issue.

Well, that's debatable.
Looking at these 2 lines:
Template added /_default/baseof.htmlc3gzwd.bck
Template added /_default/baseof.html

I'm guessing that the above is hinting at the problem; I suspect that Helix saves to a temp file and then renames it, not sure.

I'd be surprised if this was the issue. Vim does the same thing.

@kirawi We have a few exclusions related to vim, but nothing for .bck files.
https://github.com/gohugoio/hugo/blob/e1becf1dfe31bec65eca585ac66a2d10f1651e42/commands/hugobuilder.go#L779-L794


[Bug] /x10 is missing from gadgetbridge string lookup in .boot0

I am not sure it this is a mistake or there is something I am doing wrong, I was playing with your code on my p8, and it seems that this code from here:
https://github.com/espruino/BangleApps/blob/master/apps/boot/boot0.js
is not working.
  Bluetooth.on('line',function(l) {
    if (l.startsWith('GB({') && l.endsWith('})') && global.GB)
      try { global.GB(JSON.parse(l.slice(3,-1))); } catch(e) {}
  });

this line:
if (l.startsWith('GB({')
needs a leading /x10 to work in the p8, just wanted to report it in case I am not mistaken.

Thanks! Yes, you're correct - that needs changing.


@protected is not defined in .tailcallrc.graphql

The .tailcallrc.graphql file does not have the @protected defined. It should be autogenerated like other directives.

/bounty 15$

/attempt #1605


4.0.0 -> 4.0.1

Motivation for this change
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Package name is missing from commit title.

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

1 package built:

warzone2100

Package name is missing from commit title.

Oops :sweat_smile:
Fixed


Revert "Fixed RegEx::search missing return type hint"

Reverts godotengine/godot#9379

Wrong PR...


Expose WireMock server port as system property

While it works generally quite well to use the @WireMockTest annotation and pass the (random) port from the WireMockRuntimeInfo to the class under test, this approach only works when I can pass in directly the port/url into the class.
When using e.g. a SpringBootTest with Dependency Injection it's sometimes not possible. Especially, when the server host is read from an application.yaml
Nevertheless, what would work, is when the WireMockExtension would expose random ports as a well known property. Something like
System.setProperty("wiremock.server.port", wireMockServer.port().toString())

in a beforeAll method would make it possible to reference the port in an application.yaml.

Not sure if I understand your question correctly, but would it help to use Spring Boot's @DynamicPropertySource in this case?
@RegisterExtension
static WireMockExtension wireMockExtension = WireMockExtension.newInstance()
        .options(wireMockConfig().dynamicPort())
        .build();

@DynamicPropertySource
static void dynamicProperties(DynamicPropertyRegistry registry) {
    registry.add("remote.url", wireMockExtension::baseUrl);
}

What about this for Micronaut?

What about this for Micronaut?


refactor: Remove StandardBottomSheet, ExpandingBottomSheet and ModalStandardBottomSheet

BREAKING CHANGE: StandardBottomSheet, ExpandingBottomSheet and ModalStandardBottomSheet  are removed from Uno.Themes
GitHub Issue: #86
PR Type
What kind of change does this PR introduce?

Refactoring

Description
Remove StandardBottomSheet, ExpandingBottomSheet and ModalStandardBottomSheet from Uno.Themes
PR Checklist
Please check if your PR fulfills the following requirements:

Commits must be following the Conventional Commits specification
[X] Tested UWP
[X] Tested iOS
[X] Tested Android
[X] Tested WASM
[ ] Tested MacOS
[ ] Contains No breaking changes

[X] Breaking change commit message added

psst, delete:

psst, delete:

Thanks yes forgot :)


Split CupertinoScaffold into CupertinoTabScaffold and CupertinoPageScaffold

Small diffbase on #12102
#10466

Rename CupertinoPageRoute's page.dart to route.dart to avoid confusion with CupertinoPageScaffold.
Break CupertinoScaffold into a CupertinoPageScaffold for grouping content and nav bar and CupertinoTabScaffold for grouping swappable content and tab bar.
Rename CupertinoTabScaffold.rootTabPageBuilder to just tabBuilder.

Most moves are unmodified from previous code.
Next PR, create a CupertinoTab with a Navigator.
Will make API announcement when merging.

This PR introduces an API breaking change.
CupertinoScaffold is being split into CupertinoTabScaffold and CupertinoPageScaffold to better express the navigation hierarchy of iOS where a tab view is at the root of potentially multiple pages.
Tab scaffolds previously created with new CupertinoScaffold.tabbed() is now CupertinoTabScaffold and no longer hosts the navigation bar. Note that constructor parameter rootTabPageBuilder is also renamed tabBuilder
The navigation bar is now in CupertinoPageScaffold which replaces the previous default new CupertinoScaffold() constructor.


Add new EMCD pool tags

Confirmed with EMCD support and one of their product manager.


Use strlen instead of strcmp

Use strlen instead of strcmp to figure out if string is empty

@jbkyang-nvi had this passed CI?

@jbkyang-nvi  Can we close this PR?

Sure. I can create a new PR when I have time to actually change it


Update Documentation

Hello,
Thanks for all the work on updating this library to v2. I saw that some of the documentation was updated, but it looks like there are still a few broken links / outdated guides.
This link is on the Readme here and doesn't seem to go anywhere: https://docs.dolthub.com/reference/python/
Also the notebook example / guide does not seem to run on Deepnote properly anymore: https://docs.dolthub.com/integrations/notebooks
Thanks!

Seems like there is no update so I'll go ahead and close


Display as Key-Value mode on narrow device

Feature idea.
In order to make the responsiveness of table better, we could add a flag that enables the table to be shown in key-value mode. See examples: https://css-tricks.com/responsive-data-tables/
This can be of course achieved with CSS, but it would be a better idea to provide a solution for it in the first place.

that's a cool idea. BTW, table-responsive also can be an alternative.
https://www.w3schools.com/bootstrap/bootstrap_ref_css_tables.asp

It would be great if displaying as key-value would be optional as all use cases might not require it.

Indeed, I also consider we could invent even new paradigms of display-modes. (not easy though :))

We will consider seriously to implement this feature, thanks :)