timed start foo does not stop the currently active project

In the example below I:

Start working on foo
After two minutes, start working on bar (I would expect this to implicitly stop working on foo)
After one minute, stop working on bar
Wait one minute then do timed summary

I would expect timed summary to report 2 minutes worked on foo and 1 minute on bar. In fact it reports 1 minute on bar and 4 minutes on foo. The time recorded against foo will just keep increasing now as time goes by, there's no way to stop it (except editing ~/.timed manually).
Looking at ~/.timed, there's no end time for when I was working on foo, and evidently timed doesn't consider the start time for bar on the following line to be an end time for foo. Timed considers that foo is still running. timed stop does not stop foo from running.
You're now in a broken state that the timed cli doesn't seem able to get out of. Only editing ~/.timed manually can fix it.
Is this intentional - are you supposed to always do timed stop before doing timed foo?
I'd call this a bug - doing timed start ... when there's a currently active project should implicitly stop the active project. You wouldn't even have to change what gets written to ~/.timed, just change timed summary so that if a ~/.timed line doesn't have an end time then the start time on the next line is considered as its end time (unless there is no next line, in which case it's the active project).
seanh@mistakenot ~ cat .timed
seanh@mistakenot ~ timed start foo
starting work on foo
  at 17:54 on 17 Mar 2015
seanh@mistakenot ~ timed start bar
starting work on bar
  at 17:56 on 17 Mar 2015
seanh@mistakenot ~ timed stop
worked on bar
  from    17:56 on 17 Mar 2015
  to now, 17:57 on 17 Mar 2015
       => 0h1m elapsed
seanh@mistakenot ~ cat .timed
foo: 17:54 on 17 Mar 2015 -
bar: 17:56 on 17 Mar 2015 - 17:57 on 17 Mar 2015⏎
seanh@mistakenot ~ timed summary
bar   0h1m
foo   0h4m
seanh@mistakenot ~ timed stop
worked on bar
  from    17:56 on 17 Mar 2015
  to now, 17:57 on 17 Mar 2015
       => 0h1m elapsed
seanh@mistakenot ~ cat .timed
foo: 17:54 on 17 Mar 2015 -
bar: 17:56 on 17 Mar 2015 - 17:57 on 17 Mar 2015⏎

Thanks for reporting.  The intended usage is to run timed stop before timed starting a new project.  I think a good solution to this problem would be to allow to specify an argument to timed stop.  As I don't use timed actively anymore, I will not be fixing this, but I would accept a pull request that does.


remove or fix dist/bundle.js.map

There's no need for a broken/empty .js.map within the bundle. Remove or fix this.
Note: Since the map file won't get published to npm because of .npmignore anyway it's not a problem to just remove it. It may though be useful during development for debugging so i would prefer a fix :)

IMO, we do a .gitignore, and let it be on development machine, it gives me nice mapping on my browser when I try to debug :)

@cyberhck It would do that, IF it wasn't broken. :p That's why i asked for fix or remove ^^

Fixed by #29


[build-script] Fix import of dependencies of each product.

While actually wiring this up, I discovered that I imported these products
incorrectly.
Should be NFC.

@swift-ci python lint

@swift-ci smoke test

@swift-ci clean smoke test


Using vue2 slider component, startAnimation: false still animating

Why is this still animating?
Component
<vue-slider :startAnimation="false" v-bind="fatSlider"v-model="model.carbs">`
Slider options
sliderSettings: {           interval: 1,           fixed: true,           processDragable: true,           piecewiseLabel: false,           tooltip: false,           startAnimation: false         }

try start-animation.

still animating

is it when you programmatically set values it still animates, is there a way to disable that on inital model setting?

Thank you, thats just what i needed


Is this going to be added to the official stable repo?

just curious why this hasn't been added to the official repo yet: https://github.com/helm/charts/search?q=kubernetes-external-secrets&unscoped_q=kubernetes-external-secrets
I cant just do helm install --name kubernetes-external-secrets --set env.AWS_REGION='<aws-region>' stable/kubernetes-external-secrets without downloading the chart locally first

Yes, I think @keweilu is working on it!

Awesome thanks! will make it so convenient!

In the meantime, you can use the helm-github plugin as a workaround to install the kubernetes-external-secrets chart directly from the GitHub repository:
helm plugin install --version master https://github.com/sagansystems/helm-github.git
helm github install --repo https://github.com/godaddy/kubernetes-external-secrets.git \
                              --path charts/kubernetes-external-secrets \
                              --name kubernetes-external-secrets \
                              --set env.AWS_REGION='<aws-region>'

Thats neat I didn't know about that plugin. just curious are there requirements to get a helm chart added to the official repo? I have never done that. Do they require you have tests or perform any kind of validation before accepting it?

Hello! Is there any ETA for getting this in the official stable repo ?
The issue that I found so far with the github plugin is that it doesn't support helm upgrade --install

Hi @bilby91 . I'm not sure how long the process normally takes, but the PR has been submitted. Hopefully someone from helm can look at it soon

Thanks for the PR link @jeffpearce !!

An alternative to this is for godaddy to have its own chart repo and add the godaddy chart repo to helm hub (https://github.com/helm/hub)
Its possible to host the charts in a github repository using tools like https://github.com/helm/chart-releaser and https://github.com/helm/chart-testing you can get a pretty good setup as well :)

it's in helm hub now https://hub.helm.sh/charts/external-secrets/kubernetes-external-secrets


Cleanup remaining calls to restart_if_present

Now that restart_if_present is gone, update some existing handlers to
make use of must_exist=false instead.

:+1:


feat(formly-form): allow passing FormArray in form input.

What kind of change does this PR introduce? feature

This change is

Codecov Report

Merging #748 into master will increase coverage by 0.33%.
The diff coverage is 96.29%.

@@            Coverage Diff             @@
##           master     #748      +/-   ##
==========================================
+ Coverage   94.25%   94.59%   +0.33%
==========================================
  Files          17       17
  Lines         783      758      -25
  Branches      229      224       -5
==========================================
- Hits          738      717      -21
+ Misses         45       41       -4

Impacted Files
Coverage Δ

src/core/src/services/formly.config.ts
90.81% <ø> (ø)
:arrow_up:

src/core/src/services/formly.form.builder.ts
95.58% <100%> (+0.04%)
:arrow_up:

src/core/src/services/formly.form.expression.ts
95.29% <100%> (ø)
:arrow_up:

src/core/src/components/formly.form.ts
92.95% <75%> (+2.14%)
:arrow_up:

Continue to review full report at Codecov.

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


formatted code and  made font-weight bolder

in this commit i have formatted code and added made font-weight bolder for INVALID and VALID

It is now looking much better. Thanks @hirentimbadiya


ERR: lws_context_init_server_ssl: SSL_CTX_load_verify_locations unhappy

Problem/Motivation
The following error is displayed in the logs of the add-on:

ERR: lws_context_init_server_ssl: SSL_CTX_load_verify_locations unhappy

The add-on, however, functions perfectly.
Proposed changes
Find out of this error can be fixed or suppressed.
At least add it to the documentation as a known error.

I get the same error in the log.
ERR: lws_context_init_server_ssl: SSL_CTX_load_verify_locations unhappy
And here, the terminal doesn't load. I get an error stating that: "HASS unexpectedly closed the connection", regardless of whether I use the panel or through browser.
My options:
{
  "debug": false,
  "username": "####",
  "password": "####",
  "ssl": true,
  "certfile": "fullchain.pem",
  "keyfile": "privkey.pem",
  "packages": [],
  "init_commands": []
}

@Aephir That has nothing to do with this error message.
You have enabled SSL, ensure you are using HTTPS:// everywhere (including the iframe panel you've added to your configuration of Home Assistant).

@frenck Thanks, but I am using https:// for everything (both when trying https://MYSUPERSECRET.DUCKDNS.org:7681, and url in panel).
Is it that I can't access it via dns? I didn't try just https://192.168.0.XX:7681 when at home.

Notice has been added to the documentation.
Closing this issue.


Update live-events-recording-policies.md

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/7115
Fixed duplicate

@JohanFreelancer9 Copy editing is needed for this PR. Thanks

@cichur PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Emulator Core Version Update Request - Mame2003plus

Would it be reasonable to update the version of the Mame2003plus Retroarch Core to 444bc10 or newer in the next arkOS update? The reason that I ask is the version I currently get using the Core Downloader in Retroarch is 0422413b, which has issues with the games Pole Position and Pole Position II (possibly others, I'm not sure) and simply crashes Retroarch when the core tries to load them. Having tested these games on both a Linux PC and a Windows PC, I can confirm that the issue I mentioned was resolved by the 444bc10 commit of the Mame2003plus core.

Done.  Just do another retroarch core download.  Enjoy.


Build CTS on windows or linux - Python script proposal

Fixes #537

We would like to review this and will post comments shortly.

We would like to stick to cmake-based builds rather than a build script. We recommend moving any necessary logic into the CMake build and documentation to the readme (refer https://github.com/KhronosGroup/OpenCL-CTS/pull/1021)

Any reason we need separate scripts and can not reuse or build on top of https://github.com/KhronosGroup/OpenCL-CTS/blob/master/presubmit.sh ?

Any reason we need separate scripts and can not reuse or build on top of https://github.com/KhronosGroup/OpenCL-CTS/blob/master/presubmit.sh ?

There are already scripts in repo


Add .navbar component

Hello, thank you for this small CSS framework!
I want to know: is there a way to add a simple navbar (like that but more simpler, of course: http://bootsnipp.com/snippets/featured/navbar-search-add-on-bs-3 ) in Siimple CSS?
Or is there a way to have a invisible navbar menu which can collaspe for mobile?
Thank you.

Hello @Kcchouette , thank you for using it!! 😄
Currently I am working on this feature, a set of layout elements like navbars, jumbotrons, etc... But I don't know when it will be finished. If you like, I will notice you when these elements are ready to use.
Thanks again for your interest!

I'm really interested too, I forked the project yet and I'm interested in collaborate with those components
I'm using Siimple massively on my academic/production front-end projects :)
Greetings from Venezuela

@jmjuanes yes, notice me, please. Thanks

Would be very nice to see that feature. Siimple is awesome!

Hello! I just published the new v2.0.0 🎉 🎉
The layout framework (which will include the navbar element) will be published soon on this repository: https://github.com/siimple/siimple-layout , so please if you have any questions or want to help/collaborate, open a new issue on the layout repository.
Thank you! 😄


Export WASM to folder for self-publishing on personal web site

Discussed in https://github.com/marimo-team/marimo/discussions/1078

Originally posted by astrowonk April  5, 2024
marimo.app is very cool and I was excited to see the WASM support. But, WASM apps should be able to just be turned into a folder than can deployed on any web server (nginx, apache, etc.) Shiny Live for example has an easy shinylive export myapp site command that turns the app into a folder I can upload to my nginx server.
Whereas for now, wasm notebooks seem to require using marimo.app. Is there a plan to have an export option someday? Thanks!

Some follow up from that discussion:
We don't support such an export yet. In the meantime, one can iframe marimo.app URLs (https://docs.marimo.io/guides/wasm.html#embedding).
@astrowonk mentioned working with local files as a use case that makes marimo.app links difficult to use. We do have a beta sandbox that lets you attach files to WASM notebooks, via the browser's storage; I'm not actually sure how you'd do it via a static export.
@astrowonk and others who have requested this: are local files the only reason you need exports? We'd appreciate more feedback before going down this route, since it will take us some time.

I would say I have a few reasons for wanting WASM export, though I could be in the minority even if this is important to me,it is no guarantee  it's worth your development time (yet.)

Ownership. I prefer to host my own things when possible. I can deploy a Marimo app myself via uvicorn ,so it'd be nice to own / host the WASM apps too (but not require spinning up a bunch of new infrastructure/ reverse proxy, etc.)
Using Marimo to demo/show off a new module/repo.  I would imagine I'd use Marimo in some ways like I use Streamlit or Dash. I develop some new project/class/code. Then I have a Marimo notebook in the same git repo that uses whatever I wrote to show off what the code can do. The repo/folder may contain other code, data, etc. Exporting all that to a WASM app would be pretty nifty; as I wouldn't have to deploy any new server.
Proprietary code/data In a work context, I could imagine using Marimo to show dashboard-like data for monitoring our systems and/or algorithms. Putting a Marimo notebook + some local data file (parquet, csv, etc.) and then exporting to WASM on an (already existing) access-controlled web server would give some new interactivity without spinning up new uvicorn/reverse proxies/etc.

If this export of the tutorials was reactive - would that solve your use case?
https://dmadisetti.github.io/quarto-marimo/intro.html

If this export of the tutorials was reactive - would that solve your use case?
https://dmadisetti.github.io/quarto-marimo/intro.html

I think … so? I mean the idea is to get WASM-goodness that is on marimo.app but self-hosted on any web server (plus being able to be import code from other python files in the same directory). I'd love to  someday type marimo export notebook.py notebook_folder and copy that the resulting folder to a directory on my web site, and have a live interactive notebook.

Building on this, it would also be great to have a way to generate the wasm permalink programmatically, similarly to the marimo export html, just with the wasm version in the iframe.

You can do this without the marimo CLI. The link is just https://marimo.app/#code/{url_encoded_code}
url_encoded_code is URL escaped code which is just urllib.parse.quote(code, safe="~()*!.'")
but you will hit a 2mb limit that is imposed by your browsers (not the server)

You can do this without the marimo CLI. The link is just https://marimo.app/#code/{url_encoded_code}
url_encoded_code is URL escaped code which is just urllib.parse.quote(code, safe="~()*!.'")
but you will hit a 2mb limit that is imposed by your browsers (not the server)

Yes, that’s useful for some contexts, but I don’t want it to be limited to what can be encoded in a url. And I want to access other data locally (parquet, csv, sqlite, etc.)

+1
Noticing a few limitations in the WebAssembly links:

only includes notebook code: not accompanying files
can't specify package dependencies: it does automatically install some detected packages, but it can't detect all of them

For many usecases though, the 2MB limit will be problematic anyways. So perhaps making it easy to store data in a Gist or other location and including a reference in the link.
Even though it won't always be suitable, it is really awesome to be able to share a link to a reactive Python notebook that doesn't need a running Python kernel, and can be embedded in webpages. This really puts Python up against Observable notebooks for example (in a way that Jupyterlite doesn't due to lacking reactivity - but all of these are nice projects!).

Hey @declann our Community Cloud has implemented a way to include additional files to get past the 2mb URL limit. We do this by storing your files/assets in a S3-compatible bucket, backed by authentication.  We also support reading from a GitHub repo: we can read the notebook and all the files in a configurable directory. You can create an account here for free: https://marimo.io/sign-up.
We'd like to make some of these components open source for others to use to publish on their own site, but that requires additional maintenance and support that we are not ready to handle at the moment. We do allow embedding these community cloud notebooks in an iframe which has worked great for blogs and tutorials.

Thanks Mike!

Will follow along, the would be a great way to improve upon our notebook sharing at work but is required to be self hosted (or via a github page)

This is now possible in marimo 0.10.0, here are the docs. (related PR).


[WalletConnect] respond to eth_sendTransaction

Currently the WalletConnect safe app doesn't send a response to eth_sendTransaction calls. Neither when transaction is approved wallet-side, nor when it is rejected.
This makes it impossible to await a transaction sent with web3.eth.sendTransaction as web3 internally depends on transaction hash returned in response to eth_sendTransaction, or to know if the user rejected the transacation on the wallet side

Question, when is this going to be reflected in the public testnet deployment of the gnosis safe?

Question, when is this going to be reflected in the public testnet deployment of the gnosis safe?

@hellwolf it should be available on dev environment in the next few days, I'm working on a CI setup because we recently migrated to monorepo. For staging/production deployments I'm not sure

@hellwolf it should be available on dev environment in the next few days, I'm working on a CI setup because we recently migrated to monorepo. For staging/production deployments I'm not sure

Thanks @mikheevm ! Sorry for my ignorance, how could we use gnosis safe from dev environment instead?

Thanks @mikheevm ! Sorry for my ignorance, how could we use gnosis safe from dev environment instead?

@hellwolf https://safe-team.dev.gnosisdev.com/app/#

@hellwolf https://safe-team.dev.gnosisdev.com/app/#

Hello @mikheevm still doesn't seem to work properly on our end.
Promise remains unresolved, both in case of approved and rejected tx.
I see these errors on console on https://safe-team.dev.gnosisdev.com/app/#
TX Approved:

TX Rejected:

Hello @mikheevm still doesn't seem to work properly on our end.
Promise remains unresolved, both in case of approved and rejected tx.
I see these errors on console on https://safe-team.dev.gnosisdev.com/app/#
TX Approved:

TX Rejected:

Hey @mikheevm & Gnosis team. The response to 'eth_sendTransaction' seems to return a wrong txHash.
Link used: https://safe-team.dev.gnosisdev.com/app/#
Network: Rinkeby
Example:
Sent tx's receipt:

Etherscan link of it: https://rinkeby.etherscan.io/tx/0x6b5d378d81f6f67294a3222646474ceb3ce29af730cc80ebf17e741117067fe3
Actual tx hash when looking at the logs of the safe's address: https://rinkeby.etherscan.io/tx/0x6a1dfa751630928b3d3dd74efd478387a0a568fc65263eb237f1bd94a49de618

@devzl
Hey, since safe is a multisig wallet, we may not have the "real" transaction hash since the beginning because the transaction could be possibly signed off-chain.
What we return is the transaction hash used internally in safe contracts: https://github.com/gnosis/safe-contracts/blob/760f6347844d848b4a2034c44e02f1e5463c4db1/contracts/GnosisSafe.sol#L403

Thanks for the prompt reply @mikheevm
How would you ideally track a transaction from the moment the internal hash is generate to the moment it's successfully validated on-chain? Is there a (semi-)direct way to map that hash to a on-chain tx or do we need to figure out transactions from subscribing to logs and sent params and events data? (referencing this for the second part of my question: https://docs.walletconnect.org/smart-wallets#transactions)

There's no easy way to do that due to the nature of multi-signature wallets. You can use Safe Transaction Service GET /transactions/{safe_tx_hash}/ endpoint to check the status of the transaction.
Swagger: https://safe-transaction.gnosis.io/
Docs: https://docs.gnosis.io/safe/docs/services_transactions/

Ok will check that out!
Is there an ETA of when this fix will be deployed to production? @mikheevm

@devzl it's already deployed

Oh, I must've been using the old version, there are two wallet connect apps list on the Apps list on your platform:
This deployment doesn't return a txHash: appUrl=https://ipfs.io/ipfs/QmWwSuByB3B3hLU5ita3RQgiSEDYtBr5LjjDCRGb8YqLKF
While this one does: appUrl=https://ipfs.io/ipfs/QmT3VxxfFtfEcvq8AeaoFAyUGxePRa2zisvnxLTrQXU5Uf
@mikheevm

@devzl the old one must be persisted in local storage, try to clear it


webpack v5 compatibility

Library Affected:

workbox-webpack-plugin

Browser & Platform:
None
Issue or Feature Request Description:
webpack 5 exposes webpack-sources via webpack.sources, and it's advised to use it instead of require('webpack-sources'), if you agree to do it, I can submit a pull request.

There's a number of non-trivial changes that are needed for webpack v5 compatibility. (C.f. https://github.com/webpack/webpack/issues/11425)
I'm working on implemented all of those changes in the Workbox v6 branch, hopefully in time for the next Workbox pre-release.

great to hear! we just moved to webpack 5 and had to remove workbox since it was causing the build to error out.

the WP5 branch is looking really good! would love to give it a test drive for you!

This is now live as part of the v6.0.0-alpha.3 release: https://github.com/GoogleChrome/workbox/releases/tag/v6.0.0-alpha.3


dead lock when decode h264 ？

#0  0x00007fffdfc732cb in YamiParser::BitReader::read(unsigned int) (this=this@entry=0x7fffb9c7e7b0, nbits=nbits@entry=1) at bitReader.cpp:99
#1  0x00007fffdfc7d8c0 in YamiParser::NalReader::readUe() (this=this@entry=0x7fffb9c7e7b0) at nalReader.cpp:72
#2  0x00007fffdfc75690 in YamiParser::H264::Parser::parseSps(std::tr1::shared_ptrYamiParser::H264::SPS&, YamiParser::H264::NalUnit const*) (this=this@entry=0x14741d8, sps=..., nalu=nalu@entry=0x7fffb9c7e8d0) at h264Parser.cpp:398
#3  0x00007fffdfc8d2af in YamiMediaCodec::VaapiDecoderH264::decodeSps(YamiParser::H264::NalUnit*) (this=this@entry=0x14740e0, nalu=0x7fffb9c7e8d0) at vaapidecoder_h264.cpp:1203
#4  0x00007fffdfc9411c in YamiMediaCodec::VaapiDecoderH264::decodeNalu(YamiParser::H264::NalUnit*) (this=this@entry=0x14740e0, nalu=0x7fffa4011c90, nalu@entry=0x7fffb9c7e8d0) at vaapidecoder_h264.cpp:1688
#5  0x00007fffdfc941d3 in YamiMediaCodec::VaapiDecoderH264::decode(VideoDecodeBuffer*) (this=0x14740e0, buffer=) at vaapidecoder_h264.cpp:1727

Hi @waiter89 :
could give some steps or stream to reproduce this?

bitReader.cpp:100
tmp <<= toBeReadBits;
when toBeReadBits==0, it will deak lock ?
But we'd better have a stream to verify it.
and We can add them to our daily regression to make sure it never failed again.

bitReader.cpp:72
tmp = m_cache << ((CACHEBYTES << 3) - m_bitsInCache) >> ((CACHEBYTES << 3) - nbits);
deak lock  again

m_bitsInCache==0

Hi @waiter89
thanks for reporting, but if we do not have the bit stream, we can't verify the code even we fix it. Could you share your stream or tel us how to generate it ?
thanks

@wudping :
lizhong is busy with other things. Please help check this. thanks.

ok, Guangxin

Hi @waiter89
This fixed by https://github.com/01org/libyami/pull/514.

Hi @waiter89
this fixed by e3c2744b0b3bb9f15fd7293cf698193a5aa19192.
BR.


Thumbnail not working

Hi,
I am using FeinCMS and the build-in Medialibrary. The Medialibrary admin change list view is showing large images instead of thumbnails.
Something is going wrong in the generate function of the default Thumbnailer:
def generate(self, storage, original, size, miniature):
    try:
        image = Image.open(StringIO(storage.open(original).read()))

        # defining the size
        w, h = int(size['w']), int(size['h'])

        format = image.format # Save format for the save() call later
        image.thumbnail([w, h], Image.ANTIALIAS)
        buf = StringIO()
        if image.mode not in ('RGBA', 'RGB', 'L'):
            image = image.convert('RGBA')
        image.save(buf, format or 'jpeg', quality=80)
        raw_data = buf.getvalue()
        buf.close()

        storage.delete(miniature)
        storage.save(miniature, ContentFile(raw_data))

        return storage.url(miniature)
    except:
        # PIL raises a plethora of Exceptions if reading the image
        # is not possible. Since we cannot be sure what Exception will
        # happen, catch them all so the thumbnailer will never fail.
        return storage.url(original)

How can I figure out where the exception is being raised? And how come this code worked fine last years and now it is showing large images.
I am using FeinCMS 1.8.4 and above and django 1.6.5 and greater...
What alternatives are there to use as thumbnailer?

Basically the same already in in e873c4b.


home-assistant-custom-components.philips_airpurifier_coap: init at 0.28.0, pythonPackages.aioairctrl: init at 0.2.5

Adds the philips-airpurifier-coap Home Assistant custom component.
Manually confirmed that the integration works on my Home Assistant instance.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Thanks for the suggestions, I've applied them. I actually initialized all of this using an older version of nix-init, before the update to these conventions.

Looks like you didn't push any changes yet?

Weird, I did update and force-push. The branch itself showed the changes, but the PR did not. I'll chalk it up to GitHub weirdness.
In any case, I've rebased on master again.

One last nit: The package set is called python3Packages, not pythonPackages (= python2Packages).

One last nit: The package set is called python3Packages, not pythonPackages (= python2Packages).

Reworded :+1:


[C++][Parquet] Add arrow as a thirdparty dependency

Add Arrow as a thirdparty dependency with a fixed source hash (later version, once there is a release)
Reporter: Uwe Korn / @xhochy
Assignee: Uwe Korn](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=uwe) / @xhochy
Note: This issue was originally created as PARQUET-714. Please see the migration documentation for further details.

Uwe Korn / @xhochy:
Will be done as part of PARQUET-712


blingboard webhooks

Why:
Closes [issue link]

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[ ] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

👋 Hi, I’m @BLINGBOARD
👀 I’m interested in IMPROVING MY WORK ...
🌱 I’m currently START MY DIGITAL ART WORK ...
💞 I,m thankful to everyone who supported me to improving my self ...
📫 How to reach me ...
https://opensea.io/BLINGBOARD
<EMAIL_ADDRESS>--->https://www.reddit.com/u/Green-Illustrator-28/?utm_source=share&utm_medium=ios_app&utm


<EMAIL_ADDRESS>hash check failed

Download: Status Legend:
Download: (OK):download completed.
Checking hash of cpu-z_1.87-en.zip ... ERROR Hash check failed!
App:         extras/cpu-z
URL:         http://download.cpuid.com/cpu-z/cpu-z_1.87-en.zip
First bytes: 3C 21 44 4F 43 54 59 50
Expected:    fad8c570c17bad4dd9154105959d6f591a878ee8ebb3dfb44a4d9744449a875f
Actual:      795c6733b912e7c750746433de7b4fa35c82ce821308ca653ba9fbc3a04102f7

now i checked it. my isp's secure internet filter cause the issue.


Incoming call has video?

Hello All!
I'm attempting to develop a strategy to recognize whether an incoming call has video enabled so that I can then manipulate html/css elements appropriately.  My approach thus far has been to examine the incoming session acquired from: ua.on('invite', function(new_session){incoming_call(new_session)} (which as I can tell is an InviteServerContext object).  I'm attempting to access the value at: new_session.mediaHandler.mediaHint.constraints.video.
This fails at constraints, where it is undefined.
Am I on the right track here, and if not can you steer me in the right direction as to how I can check this?

check the sdp received if it has video.
Maybe more reliable ?

Hi.  This has been asked several times on the mailing list.  This thread in particular may be helpful.  gled-rs is on the right track that checking the SDP is the more reliable way to go.
I'm going to close this ticket, as it is not a bug or feature request of the library itself.  Please follow up on the mailing list if you have more questions.
Thank you!

Thank you! Sorry I didn't know where else to post; I was unaware of the mailing list. Cheers!

O. click on support.  Please excuse my ignorance. X P

Completely excused.  It's not the most obvious thing, and you're far from the first to end up here. :-)

Well that makes me feel slightly better. : ) Thx Again!


Example "basic-ssr-file-based": HTTP status code not set to 404 when page not found

Describe the bug
Using this example: https://github.com/TanStack/router/tree/main/examples/react/basic-ssr-file-based
On a dynamic single post route, when NotFound is thrown, the http status code is not set to 404. I've scoured the documentation, it seems there is currently no way to do this.
I think this is necessary for SEO, so that non-existent pages are not indexed by search engines.
Your Example Website or App
https://github.com/TanStack/router/tree/main/examples/react/basic-ssr-file-based
Steps to Reproduce the Bug or Issue
Thrown NotFound on loader, see http status code.
Expected behavior
when visiting a non-existent page, the HTTP status code should be set to 404.
Screenshots or Videos
No response
Platform

OS: Windows
Browser: Chrome
Version: 120.0.6099.225

Additional context
No response

We're working on some new not found APIs, see #1048, that should allow for this


[BUG] Offhand Sunder

Sunder in the program doesn't get bonus from the off-hand weapon or using nodes related to the off-hand (like sword nodes) but inside the game I got those bonus to the damage (using Axe+Rapier) putting points in Brutal Blade as example.

In this case it is actually the in-game tooltip that is incorrect.
Sunder only uses the main hand weapon, so modifiers that only affect the offhand cannot ever affect the skill's DPS. However, the in-game tooltip DPS has never really been accurate when dual wielding due to multiple bugs, mostly relating to the tooltip incorrectly factoring in weapons that the skill does not or cannot use; so it really cannot be trusted.


Change minitest_runner.rb name in CONTRIBUTING.md

Or we could rename the file?

Thanks! I have no strong preference either way so I'm merging this one!


Abstract Sockets Logic

This PR abstracts the sockets logic I created in a previous PR, added cleanup, and removed junk code.
Before considering merging, are there any other commands we should add or support? Having default commands could make it easier for future addons. Right now we only have a command to get the timer state, and a global emit whenever the timer state changes.

@TCWTEAM Looks good. I think the "get state" and global event emitters are a good start. That already opens up the application for 3rd parties quite a bit. If we wanted to extend this further, we could allow clients to change state via a socket message, such as a pause, reset, etc.

@TCWTEAM Thanks for cleaning up the changelog, too.


Possible mis-use of async await

I've been using this library in production for the past few weeks, and has been great, easy to set up and currently handling ~40k tokens in storage.
I have started seeing some issues with our identity provider, not sure if it is related to this library at all but just pulled the code to check it out.
Seems that you are missing a lot of await calls on async redis database transaction methods: e.g.
https://github.com/AliBazzi/IdentityServer4.Contrib.RedisStore/blob/master/IdentityServer4.Contrib.RedisStore/Stores/PersistedGrantStore.cs#L64
Is this on purpose? I suspect it could well cause the transaction to get executed with only some of the actions applied. In the example above StringSetAsync, SetAddAsync etc are all missing await operators.

Hi @dpix
For your concern, it's on purpose, because StackExchange.Redis prepare all the operations before the library calls Redis to execute a bunch of operations as one transaction by calling await transaction.ExecuteAsync(), so no need to await StringSetAsync, SetAddAsync etc, and this is applied according to StackExchange.Redis docs.
Please let me know what other concerns you have, happy to help.

Hi @dpix
I will close this issue, please feel free to open it again to raise more specific concerns if you have any.


Omit patch versions from Cargo.toml

Fixes #28

See https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#caret-requirements for more details.

See https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#caret-requirements for more details.


.ActivePID() doesn't seem to work in some cases

When I use robotgo to find the process and activate it, .ActivePID() doesn't work normally in most cases.

After minimizing the window, .ActivePID() can no longer activate the window
Most .ActivePID() activated windows give the impression that they are not really activated'

Since English is not my native language, I'm afraid I didn't speak the problem clearly. Next, I'll use git or video to show you the process
https://user-images.githubusercontent.com/48341368/109111442-e2814880-7773-11eb-89a1-4691b938696d.mp4

Robotgo version (or commit ref):    v0.92.1
Go version:                                       1.15.7
Gcc version: 8.1.0 (x86_64-posix-seh-rev0, Built by MinGW-W64 project)
Operating system and bit: Windows 10 x64
Resolution: 1920*1080

I have same issue. Can i get some feedback why .ActivePID() dosen't work? And in my system, i can't use other methods like .ActiveName(), .SetActive().
Here is my system.

Robotgo version (or commit ref): v0.93.2
Go version: go1.16.4 windows/amd64
Gcc version: 8.1.0 (x86_64-posix-seh-rev0, Built by MinGW-W64 project)
Operating system and bit: Windows 10 x64
Resolution: 1920*1080

Thnks

同上

+1
I have same problem.


added missing typings to node path module including tests

@reppners thanks mate!

note
http://nodejs.org/api/path.html#path_path_parse_pathstring


Add Cilium configuration documentation

Two commits:

An overview of how to configure the Cilium processes (Helm, ConfigMap, etc).
Intro and examples for CiliumNodeConfig objects.

Fixes https://github.com/cilium/cilium/issues/22576

OK, comments addressed; ready for re-review.

Should we backport this to v1.13?

Should we backport this to v1.13?

Yes, we should. Labeled.


[Enhancement]: Need Total record count for Airtable queries

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Currently Total records is displayed for all other Queries but not visible for Airtable.

Steps To Reproduce

Add Airtable query
Fetch data from table
Observe there is no visible Total record count in query page

Public Sample App
No response
Version
Cloud/Production

Hi @chandannkumar,
Did you check other SaaS integrations? On my end, I have checked Airtable and Google Sheets, and both are not displaying the total record count. I think all SaaS integrations are not showing the total record count.

Hello @ajinkyakulkarni ,I have raised the PR for this issue ,Please have a look at it or assign an reviewer.


Measure latency with RDTSC, warmup

Monotonic clock is very time consuming. Skip first 1000 measurements if there are enough orders.

This one is too messy

@divaykin I would like to keep libtrading portable across platforms. I'm OK with RDTSC support as a configurable option. Perhaps there's an existing library we could use for that?
I don't think we should skip measurements during "warmup" period because outliers do matter. However, I am thinking of switching to HdrHistogram:
https://github.com/HdrHistogram/HdrHistogram_c
so that we're able to show latency percentiles and not just min/avg/max.

@penberg according to latest micro bench 2 x CLOCK_MONOTONIC is only ~50usec which is acceptable overhead for now so lets leave it.
The warmup period is interesting topic - I've an impression that outliers are much more likely to happen during startup - cache is not fully lined up, C state of CPU could be wrong, etc. I'm more interested in cruise mode outliers than something specific to initial phase.
Yes, I'll do histograms later on. To me HdrHistogram is somewhat an overkill.
If we have enough observation we can estimate quantiles with simple one pass:
quantile += eta * (sgn(sample - quantile) + 2.0 * p - 1.0)

We're already connect to a FIX server and sent and received a logon message when we start measuring. So I doubt CPU cache or page faults play a role here. CPU frequency scaling is a problem but you really need to address that in your system when running such benchmarks. Looking at perf profiles, the tests I'm conducting are dominated by Linux kernel TCP/IP stack so that's a likely source for the outliers.
That said, if you really want to skip first N orders, feel free to implement a command line option for that. What I don't want is to unconditionally and silently do that for everyone.


remove pkg_resources from init.py

pkg_resources is not available in python 3.12.
From the documentation:
Use of pkg_resources is deprecated in favor of importlib.resources, importlib.metadata and their backports (importlib_resources, importlib_metadata). Some useful APIs are also provided by packaging (e.g. requirements and version parsing). Users should refrain from new usage of pkg_resources and should work to port to importlib-based solutions.

relevant for release versions, e.g. v.1.1


Add e2e-bootstrap-secret target

Found during https://github.com/openshift/release/pull/9248

/lgtm
/approve


Pulling misc iree_io_* fixes/cleanups from #15983.

This is all of the tweaks made during #15983 as part of adding the new IO stream dialect. That PR will be revived at some point in the future without these changes needing to be bundled in.

(failure is the flaky mi300 machine)


AsyncButton is not public

Thanks for letting me know! Fixed in 1.1.2.
https://github.com/SwiftfulThinking/SwiftfulUI/releases/tag/1.1.2


Fix failure to display circuits when building notebooks in CI

See failures in run https://github.com/qutip/qutip-tutorials/actions/runs/3123223660/jobs/5065718100

This looks hard to debug. The error message looks like something went wrong with ImageMagick. This error didn't show in the PR. Does it mean that something changed during the last month?

Maybe the PR didn't run these tests? Not sure.

It looks like circuit generation is working fine now, so closing this.


Display a splash screen during application initialization

Unpacking & initialization takes a few seconds, it is best to show a splash screen to avoid misunderstanding.

Pyinstaller supports this, it would be nice indeed

This is very unlikely to be done, I tried but the feature of Pyinstaller is too experimental and I didn't manage to get a working executable with Splash configured on neither Linux or Win

But I do consider this an important part in making using Yukon as smooth as possible


feat: ✨ syn-nav-item

Story
As a user of the Synergy Design System, I would like to navigate through my application
Acceptance criteria

[ ] All DoD items successfully completed
[ ] Should work as link or button
[ ] Accordion functionality
[ ] Use Solid as template (https://github.com/solid-design-system/solid/blob/main/packages/components/src/components/navigation-item/navigation-item.ts)

design: https://github.com/synergy-design-system/design/issues/63


Error when load vearch partition server

Dear team
I used vearch v.03 with rocksdb . After index approximate 40 million items . I restart vearch  partition server but It's not load partition data on server and show error :
File gamma.log:
ERROR 2020-06-19 10:30:35,137 field_range_index.cc:600 Cannot find field [Y��]
ERROR 2020-06-19 10:30:35,137 field_range_index.cc:600 Cannot find field [`�]
ERROR 2020-06-19 11:14:58,110 gamma_index_ivfpq.cc:243 add index from docid 26253000 error!
WARNING 2020-06-19 11:14:58,142 realtime_invert_index.cc:79 exceed the max bucket keys [1280000], not extend memory any more! keys_size [218] bucket_no [55] _cur_ptr->_cur_invert_ptr->_cur_bucket_keys[bucket_no] [640000]

@phamthanhtri Please use the latest version v3.1.0, and compile it with rocksdb, we have increase the bucket limit to 12800000 in v3.1.0

Dear team
I used vearch v.03 with rocksdb . After index approximate 40 million items . I restart vearch partition server but It's not load partition data on server and show error :
File gamma.log:
ERROR 2020-06-19 10:30:35,137 field_range_index.cc:600 Cannot find field [Y��]
ERROR 2020-06-19 10:30:35,137 field_range_index.cc:600 Cannot find field [`�]
ERROR 2020-06-19 11:14:58,110 gamma_index_ivfpq.cc:243 add index from docid 26253000 error!
WARNING 2020-06-19 11:14:58,142 realtime_invert_index.cc:79 exceed the max bucket keys [1280000], not extend memory any more! keys_size [218] bucket_no [55] _cur_ptr->_cur_invert_ptr->_cur_bucket_keys[bucket_no] [640000]

Hi @qiutianme
I build  gamma engine version v0.3.1 with  faiss v1.6.1 , gcc-7.3 , rocksdb-6.2 .  These dependencies build successfully  the version v0.3.0 but  with v0.3.1 , it show error :

vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1121:36: error: ‘DirectMap’ in namespace ‘faiss’ does not name a type
static void write_direct_map(const faiss::DirectMap dm, faiss::IOWriter f) {
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1121:54: error: ISO C++ forbids declaration of ‘dm’ with no type [-fpermissive]
static void write_direct_map(const faiss::DirectMap dm, faiss::IOWriter f) {
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc: In function ‘void tig_gamma::write_direct_map(const int, faiss::IOWriter)’:
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1123:17: error: request for member ‘type’ in ‘ dm’, which is of non-class type ‘const int’
(char)dm->type;  // for backwards compatibility with bool
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1125:19: error: request for member ‘array’ in ‘ dm’, which is of non-class type ‘const int’
WRITEVECTOR(dm->array);
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1093:20: note: in definition of macro ‘WRITEVECTOR’
size_t size = (vec).size();
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1125:19: error: request for member ‘array’ in ‘* dm’, which is of non-class type ‘const int’
WRITEVECTOR(dm->array);
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1076:23: note: in definition of macro ‘WRITEANDCHECK’
size_t ret = (f)(ptr, sizeof((ptr)), n);
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1125:3: note: in expansion of macro ‘WRITEVECTOR’
WRITEVECTOR(dm->array);
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1125:19: error: request for member ‘array’ in ‘* dm’, which is of non-class type ‘const int’
WRITEVECTOR(dm->array);
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1076:37: note: in definition of macro ‘WRITEANDCHECK’
size_t ret = (f)(ptr, sizeof((ptr)), n);
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1125:3: note: in expansion of macro ‘WRITEVECTOR’
WRITEVECTOR(dm->array);
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1126:11: error: request for member ‘type’ in ‘* dm’, which is of non-class type ‘const int’
if (dm->type == faiss::DirectMap::Hashtable) {
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1126:26: error: ‘faiss::DirectMap’ has not been declared
if (dm->type == faiss::DirectMap::Hashtable) {
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1129:55: error: request for member ‘hashtable’ in ‘* dm’, which is of non-class type ‘const int’
const std::unordered_map<idx_t, idx_t> &map = dm->hashtable;
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc: In function ‘void tig_gamma::write_ivf_header(const faiss::IndexIVF*, faiss::IOWriter*)’:
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1141:39: error: cannot convert ‘const std::vector’ to ‘const int’ for argument ‘1’ to ‘void tig_gamma::write_direct_map(const int*, faiss::IOWriter*)’
write_direct_map(&ivf->direct_map, f);
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc: At global scope:
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1155:36: error: variable or field ‘read_direct_map’ declared void
static void read_direct_map(faiss::DirectMap *dm, faiss::IOReader *f) {
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1155:29: error: ‘DirectMap’ is not a member of ‘faiss’
static void read_direct_map(faiss::DirectMap *dm, faiss::IOReader *f) {
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1155:47: error: ‘dm’ was not declared in this scope
static void read_direct_map(faiss::DirectMap *dm, faiss::IOReader f) {
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1155:67: error: expected primary-expression before ‘’ token
static void read_direct_map(faiss::DirectMap *dm, faiss::IOReader *f) {
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1155:68: error: ‘f’ was not declared in this scope
static void read_direct_map(faiss::DirectMap dm, faiss::IOReader f) {
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1355:1: error: expected ‘}’ at end of input
}  // namespace tig_gamma
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1136:13: warning: ‘void tig_gamma::write_ivf_header(const faiss::IndexIVF, faiss::IOWriter)’ defined but not used [-Wunused-function]
static void write_ivf_header(const faiss::IndexIVF ivf, faiss::IOWriter f) {
^
vearch/vearch/engine/gamma/index/gamma_index_ivfpq.cc:1144:13: warning: ‘void tig_gamma::read_index_header(faiss::Index, faiss::IOReader)’ defined but not used [-Wunused-function]
static void read_index_header(faiss::Index *idx, faiss::IOReader *f) {
^
make[2]: *** [CMakeFiles/gamma.dir/index/gamma_index_ivfpq.cc.o] Error 1
make[1]: *** [CMakeFiles/gamma.dir/all] Error 2
make: *** [all] Error 2

@qiutianme  I used lastest faiss v1.6.3 , gcc-7.3 , rocksdb-6.2  build vearch v3.1.0 successfully . But  I load indexed data , it show many error
gamma.log

INFO 2020-06-22 10:58:08,949 gamma_api.cc:297 Engine init successed!
ERROR 2020-06-22 10:58:08,956 vector_manager.cc:75 NO support for retrieval type
INFO 2020-06-22 10:58:08,956 profile.cc:170 Add field name [_id], type [4], index [0]
INFO 2020-06-22 10:58:08,956 profile.cc:170 Add field name [uid], type [0], index [1]
INFO 2020-06-22 10:58:08,956 profile.cc:170 Add field name [imageurl], type [4], index [1]
INFO 2020-06-22 10:58:08,956 profile.cc:170 Add field name [boundingbox], type [4], index [1]
INFO 2020-06-22 10:58:09,161 profile.cc:237 Create table 4 success!
ERROR 2020-06-22 10:58:09,161 gamma_engine.cc:642 Cannot create table!
INFO 2020-06-22 10:58:09,167 profile.cc:237 Create table 6 success!
ERROR 2020-06-22 10:58:09,167 gamma_engine.cc:642 Cannot create table!

@phamthanhtri  Maybe v3.1.0's data format is not compatible to v0.3.0. You can try to recreate space and insert document

Hi @qiutianme
I used new version v0.3.1 and reindex with my data . After indexed ~260 million item vectors ( 130 million items vector per partition) . Vearch can't load this spaces . What's wrong with me ?


Static Reflection

It would be super nice to support this as it remains a major pain point of C++ (at least for myself); introspection of types being the main desired feature

the keyword reflect
an expression of the form reflect type-name
Implementation requirements:

Should present a meaningful interface to the user, perhaps through predeclared types
Should work with generic types
Should have all the information to reconstruct the reflected type (though not required for basic functionality)

small example (syntax is made-up and not decided)
enum Foo<T> {
    Var0: T
    Var1: i32
}

function foo() -> String {
	return match reflect Foo {
        Enum(variants: vs) {
            vs.first().name
        }
        _ => verify_not_reached()
    }
}

Instead of introducing a new keyword, why not do what Zig does and treat types like objects of type Type? Type would act like a compile time struct with some fields like name and an enum of enum, class, struct, function, etc. And each of those would have fields that describe them. So, for function the API would look like:
struct Function {
    name: String,
    arguments: Array<Variable>,
    return_type: Type,
    block: Array<Statement> // Maybe.
}

And it would be used like:
function foo() {
    return match foo.type {
        Function(function: f) => f.name + " returns " + f.return_type.name
        // ...
    }
}

And for enum:
enum BetterOptional<T> {
    Some(T)
    None
}

function bar() {
    return match BetterOptional.type {
        Enum(variants: vs) => vs[0].name // If indexing past Array, throw error at compile time.
        // ...
    }
}

Not exactly sure what the expected behavior would be when taking the name of a templated type, so maybe it would only be valid if either the type has its templates filled out or you don't touch T.

I'm fairly against pretending that typenames are values, we don't have type-level computations, and we don't have kinds, so T wouldn't actually be a "type", it'd have to be magically reflected if you name it as an expression
instead of implicitly doing so, we can keep the parser simpler with a keyword and call it good.
note that my suggestion doesn't place compile-time execution restrictions, reflect(T) is a runtime value.

Slowly moving, but I've got a partial implementation working in https://github.com/alimpfard/jakt/tree/sr.
With the CTFE support present, it's relatively simple to do - just need some time to support enough to be useful.


Non-working example in documentation, with recommendations for how to fix

== cat /etc/issue ===============================================
Linux EricDesktop 4.4.0-96-generic #119-Ubuntu SMP Tue Sep 12 14:59:54 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
VERSION="16.04.2 LTS (Xenial Xerus)"
VERSION_ID="16.04"
VERSION_CODENAME=xenial
== are we in docker =============================================
No
== compiler =====================================================
c++ (Ubuntu 5.4.0-6ubuntu1~16.04.4) 5.4.0 20160609
Copyright (C) 2015 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
== uname -a =====================================================
Linux EricDesktop 4.4.0-96-generic #119-Ubuntu SMP Tue Sep 12 14:59:54 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
== check pips ===================================================
numpy (1.13.1)
protobuf (3.4.0)
tensorflow (1.3.0)
tensorflow-tensorboard (0.1.6)
== check for virtualenv =========================================
False
== tensorflow import ============================================
tf.VERSION = 1.3.0
tf.GIT_VERSION = v1.3.0-rc1-2409-g5ee3804
tf.COMPILER_VERSION = v1.3.0-rc1-2409-g5ee3804
Sanity check: array([1], dtype=int32)
== env ==========================================================
LD_LIBRARY_PATH /usr/local/cuda-8.0/lib64
DYLD_LIBRARY_PATH is unset
== nvidia-smi ===================================================
Sun Sep 24 14:08:03 2017
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 375.82                 Driver Version: 375.82                    |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  GeForce GTX 670     Off  | 0000:03:00.0     N/A |                  N/A |
| 33%   50C    P0    N/A /  N/A |    254MiB /  4031MiB |     N/A      Default |
+-------------------------------+----------------------+----------------------+
|   1  GeForce GTX 670     Off  | 0000:04:00.0     N/A |                  N/A |
| 32%   48C    P0    N/A /  N/A |    253MiB /  4036MiB |     N/A      Default |
+-------------------------------+----------------------+----------------------+
+-----------------------------------------------------------------------------+
| Processes:                                                       GPU Memory |
|  GPU       PID  Type  Process name                               Usage      |
|=============================================================================|
|    0                  Not Supported                                         |
|    1                  Not Supported                                         |
+-----------------------------------------------------------------------------+
== cuda libs  ===================================================
/usr/local/cuda-8.0/doc/man/man7/libcudart.so.7
/usr/local/cuda-8.0/doc/man/man7/libcudart.7
/usr/local/cuda-8.0/targets/x86_64-linux/lib/libcudart_static.a
/usr/local/cuda-8.0/targets/x86_64-linux/lib/libcudart.so.8.0.61
Describe the problem
I have been trying to get a custom op working for many hours, and have finally collected enough information to know that it isn't entirely my fault, and I can help you improve the custom op documentation to save others such effort.

Walkthrough to see what the problem is:

Following the instructions on the documentation page (https://www.tensorflow.org/extend/adding_an_op) I copy the zero_out.cc code
Compile with Bazel
Test with python.  Everything works, this example behaves exactly as it should.
But I want a GPU kernel, so now I try the "example" example.
Copy the example.h, example.cc and example.cu.cc files on the documentation page.  Didn't make any changes.
Now I hit some problems.  The documentation isn't very clear on how to compile this more complicated op with bazel.  I managed to figure it out, but I would strongly recommend notifying users about the "gpu_srcs" argument that can be used inside the BUILD file, since this took me quite a while to discover.

Working BUILD file:
load("//tensorflow:tensorflow.bzl", "tf_custom_op_library")
tf_custom_op_library(
name = "example.so",
srcs = ["example.cc", "example.h"],
gpu_srcs = ["example.cu.cc", "example.h"],
)
Working Bazel command to invoke the BUILD file:
bazel build --config opt --config=cuda --cxxopt="-D_GLIBCXX_USE_CXX11_ABI=0" //tensorflow/core/user_ops:example.so

Now it compiles.  But the op does not load properly in python; the module created by tf.load_op_library does not contain the actual op, as you can see by my tests under the source code section.

After a long time, I discover the extremely obvious problem: the example code in the documentation does not have any REGISTER_OP macro call.  This is clearly an omission in the documentation.

I try copying the REGISTER_OP macro call from the zero_out example, and it doesn't really work since the "example" example is more complicated, but python now at least recognizes the op.  I am sure if I were to spend the time to figure out how to correctly call the REGISTER_OP macro in example.cc, the example would work correctly for me.
I could make some other recommendations about changing the "Adding a New Op" documentation page, though I will refrain for now since I am not sure this is the appropriate place to do so.  I would be happy to take a stab at editing the doc page myself, though since I am very new to github I am concerned that I would end up creating more problems than I would fix, so it might be better to have someone with more experience do it.  I would be happy to help though.
One thing I will say though: I would really love it if some TF expert would add a fully fleshed out template op to the user op documentation page with all the bells and whistles, and with clear instructions on exactly how to compile and run it.  It should have both GPU support and a gradient implementation.  And ideally it would be multi-threaded (if the "example" example isn't already - its not obvious to me either way).  The template would be provided with everything you need, and would have a clearly labeled code block where the user can add their own code:  "here is a for loop that iterates over every element in the tensor.  write whatever you want here."  If you can make it really easy for users to add their own, high-quality operators, you might be able to cut down on your workload responding to problems with custom ops and with users requesting new ops.  And I think that a template that we can fill in would be enough to do that for many people.

Source code / logs
=========================================== start test.py:
import tensorflow as tf
zero_out_module = tf.load_op_library('./zero_out.so')
with tf.Session(''):
print "zero_out:"
print(zero_out_module.zero_out([[1, 2], [3, 4]]).eval())
example_module = tf.load_op_library('./example.so')
with tf.Session(''):
print "example:"
try:
print(example_module.example([[1, 2], [3, 4]]).eval())
except AttributeError as err:
print "Error: ", err
print "Analysis of zero_out_module:"
print zero_out_module.dict.keys()
print zero_out_module.OP_LIST
print "Analysis of example_module:"
print example_module.dict.keys()
print example_module.OP_LIST
====================================== end test.py
Output of running test.py
eric@EricDesktop:~/tensorflow/tensorflow/core/user_ops$ python test.py2017-09-24 14:14:08.183509: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:892] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2017-09-24 14:14:08.183794: I tensorflow/core/common_runtime/gpu/gpu_device.cc:965] Found device 0 with properties:
name: GeForce GTX 670 major: 3 minor: 0 memoryClockRate(GHz): 1.0455
pciBusID: 0000:04:00.0
totalMemory: 3.94GiB freeMemory: 3.66GiB
2017-09-24 14:14:08.208238: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:892] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2017-09-24 14:14:08.208514: I tensorflow/core/common_runtime/gpu/gpu_device.cc:965] Found device 1 with properties:
name: GeForce GTX 670 major: 3 minor: 0 memoryClockRate(GHz): 1.0455
pciBusID: 0000:03:00.0
totalMemory: 3.94GiB freeMemory: 3.66GiB
2017-09-24 14:14:08.208964: I tensorflow/core/common_runtime/gpu/gpu_device.cc:980] Device peer to peer matrix
2017-09-24 14:14:08.208988: I tensorflow/core/common_runtime/gpu/gpu_device.cc:986] DMA: 0 1
2017-09-24 14:14:08.208994: I tensorflow/core/common_runtime/gpu/gpu_device.cc:996] 0:   Y Y
2017-09-24 14:14:08.209000: I tensorflow/core/common_runtime/gpu/gpu_device.cc:996] 1:   Y Y
2017-09-24 14:14:08.209012: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1055] Creating TensorFlow device (/device:GPU:0) -> (device: 0, name: GeForce GTX 670, pci bus id: 0000:04:00.0, compute capability: 3.0)
2017-09-24 14:14:08.209020: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1055] Creating TensorFlow device (/device:GPU:1) -> (device: 1, name: GeForce GTX 670, pci bus id: 0000:03:00.0, compute capability: 3.0)
zero_out:
[[1 0]
[0 0]]
2017-09-24 14:14:08.239952: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1055] Creating TensorFlow device (/device:GPU:0) -> (device: 0, name: GeForce GTX 670, pci bus id: 0000:04:00.0, compute capability: 3.0)
2017-09-24 14:14:08.239970: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1055] Creating TensorFlow device (/device:GPU:1) -> (device: 1, name: GeForce GTX 670, pci bus id: 0000:03:00.0, compute capability: 3.0)
example:
Error:  'module' object has no attribute 'example'
Analysis of zero_out_module:
['_op_def_pb2', '_op_def_lib', '_op_def_registry', '_ops', '_collections', '_common_shapes', 'builtins', 'zero_out', 'package', '_op_def_library', 'OP_LIST', 'LIB_HANDLE', 'name', '_InitOpDefLibrary', 'doc']
op {
name: "ZeroOut"
input_arg {
name: "to_zero"
type: DT_INT32
}
output_arg {
name: "zeroed"
type: DT_INT32
}
}
Analysis of example_module:
['_op_def_pb2', '_op_def_lib', '_op_def_registry', '_ops', '_collections', '_common_shapes', 'builtins', 'package', '_op_def_library', 'OP_LIST', 'LIB_HANDLE', 'name', '_InitOpDefLibrary', 'doc']

Thanks for your detailed description and suggestions.

Would you be able to make these documentation changes and send us a pull request?

Ok, I will give it a shot in the next few days.
On Wed, Oct 18, 2017 at 3:35 PM, Deanna Rubin<EMAIL_ADDRESS>wrote:

Would you be able to make these documentation changes and send us a pull
request?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/13281#issuecomment-337747080,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AeY6kvtaBWwNpWkevFQhldwHu_VvAlw6ks5stn0XgaJpZM4PiC0H
.

Ok, I have made the pull request.  I made four changes, the first two of which should be straightforward, and the second two of which you might want to think about before merging
I am new to GitHub, so let me know if I am there is something I can be doing better.

It has been 14 days with no activity and this issue has an assignee.Please update the label and/or status accordingly.

(their PR is here: https://github.com/tensorflow/tensorflow/pull/13985 but I think it's still in flight)

Nagging Assignee: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @MarkDaoust: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @MarkDaoust: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @MarkDaoust: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Reviewer @MarkDaoust: It has been 15 days with no activity and the awaiting review label was assigned. Can you please take a look?

Nagging Reviewer @MarkDaoust: It has been 15 days with no activity and the awaiting review label was assigned. Can you please take a look?

Nagging Reviewer @MarkDaoust: It has been 121 days with no activity and the awaiting review label was assigned. Can you please take a look?

Nagging Reviewer @MarkDaoust: It has been 136 days with no activity and the awaiting review label was assigned. Can you please take a look?

Nagging Reviewer @MarkDaoust: It has been 151 days with no activity and the awaiting review label was assigned. Can you please take a look?

Nagging Reviewer @MarkDaoust: It has been 166 days with no activity and the awaiting review label was assigned. Can you please take a look?

Closing this issue since https://github.com/tensorflow/tensorflow/pull/13985 is closed.


Dataset licenses - PR 2

Sorry for having been so long, I had another occupation. This is a follow-up from the PR #735 that was closed. Here are the licenses :
Hendrycks maths - MIT license (https://arxiv.org/pdf/2103.03874.pdf)
coqa - multiple licenses (https://stanfordnlp.github.io/coqa/ for details)
asdiv - CC BY-NC 4.0 (https://github.com/chaochun/nlu-asdiv-dataset/blob/master/README.md)
drop - CC BY (https://allenai.org/data/drop)
no license found:
logiqa
sat_analogies
mutual
unscramble
Not sure (see the PR #735 for the discussions):
The Pile - MIT license (according to this paper)
headqa - Custom (at least for the Spanish version)
quac - CC BY-SA 4.0 (https://quac.ai/)
hendrycks_ethics - Ambiguous (the authors declared in the paper that it has no license, on Hugging Face it is a MIT license ; but the common sense portion is derived from Reddit so it may incur complications)
I should have significant time to spare in the month. Do you have any GitHub issue to recommend to me? (an issue can be solved without needing more GPUs that what you can have in Google Colab)

Hendrycks maths - MIT license (https://arxiv.org/pdf/2103.03874.pdf)
This seems correct.

coqa - multiple licenses (https://stanfordnlp.github.io/coqa/ for details)
This seems correct

The Pile - MIT license (according to this paper)
The language here isn't ideal, but I intended to mean that any IP we were creating for the composite entity was under MIT. the actual raw text is under the original license.

sat_analogies
Assuming that these are genuine College Board questions (as opposed to ones written in that style by the authors) then these are unlicensed. In particular, the College Board website ToS says:
All text, audio and video recordings, images, video, animations, content, materials, data, and other information included in, or made available to you through College Board Educator Services (collectively, "Our Content") is the property of College Board and/or third parties from whom College Board licensed it (“Licensors”) and is protected by United States and international copyright laws. Our Content may not be distributed, downloaded, uploaded, modified, reused, performed, reproduced, reposted, retransmitted, disseminated, sold, published, broadcast, or circulated, or otherwise used in any way whatsoever without our express written permission. You may not attempt to decompile, reverse engineer, scrape or data-mine College Board Services or Content.

@glerzing This looks like it's targeting the master branch. As we are now preparing big-refactor to become the new master branch, can you check what the licensing situation is with that branch and report back? This documentary info is worth having in the current master branch but I want to make sure we aren't missing out on it on the new one..

In the big-refactor branch, there is no information about licenses.

In big-refactor we put the dataset scripts that were previously hosted in the eval harness repo onto the HF hub, e.g. https://huggingface.co/datasets/EleutherAI/coqa . So we can simply assign the license labels you've determined to the HF Hub metadata.

Thanks @glerzing for this contribution, I put the license data you added in all the mirrored dataset scripts!

I should have significant time to spare in the month. Do you have any GitHub issue to recommend to me? (an issue can be solved without needing more GPUs that what you can have in Google Colab)
Let's chat further over discord in the #lm-thunderdome channel about issues that are available. If you have API keys for any closed-source models, testing our support there for Claude, or helping to add additional support for other locally-run LM APIs would be a huge get. Aside from that, another often-requested feature is the support for e.g. chat model-specific templates to wrap task prompts in.


Remove agent-settings contract

May bring this back, but for now time zone
and user working hour settings have been
moved to the working-hours contract.
Change-type: major
Signed-off-by: Josh Bowling<EMAIL_ADDRESS>

@balena-ci I self certify!


[BUG] Tests test_window_aggs_for_rows_collect_list_no_nulls  are failing on databricks.

Describe the bug
Tests test_window_aggs_for_rows_collect_list_no_nulls  are failing on databricks.
Steps/Code to reproduce bug
Run the tests test_window_aggs_for_rows_collect_list_no_nulls  on databricks.
Expected behavior
tests should pass
The error complains not finding the plan RunningWindowFunctionExec, which seems to be specific for databricks.
Error log snip:
[2021-02-07T03:07:38.289Z]                     raise Py4JJavaError(
[2021-02-07T03:07:38.289Z]                         "An error occurred while calling {0}{1}{2}.\n".
[2021-02-07T03:07:38.289Z] >                       format(target_id, ".", name), value)
[2021-02-07T03:07:38.289Z] E                   py4j.protocol.Py4JJavaError: An error occurred while calling o174965.collectToPython.
[2021-02-07T03:07:38.289Z] E                   : java.lang.IllegalStateException: Found an aggregation function in an unexpected context Some(!NOT_FOUND <RunningWindowFunctionExec> NOT EVALUATED FOR GPU YET
[2021-02-07T03:07:38.289Z] E                     !Expression <Alias> collect_list(c_int#83071, 0, 0) windowspecdefinition(a#83069L, b#83070 ASC NULLS FIRST, c_int#83071 ASC NULLS FIRST, specifiedwindowframe(RowFrame, unboundedpreceding$(), currentrow$())) AS collect_int#83096 NOT EVALUATED FOR GPU YET
[2021-02-07T03:07:38.289Z] E                       !Expression <WindowExpression> collect_list(c_int#83071, 0, 0) windowspecdefinition(a#83069L, b#83070 ASC NULLS FIRST, c_int#83071 ASC NULLS FIRST, specifiedwindowframe(RowFrame, unboundedpreceding$(), currentrow$())) NOT EVALUATED FOR GPU YET
[2021-02-07T03:07:38.289Z] E                         !Expression <AggregateExpression> collect_list(c_int#83071, 0, 0) NOT EVALUATED FOR GPU YET
[2021-02-07T03:07:38.289Z] E                           *Expression <CollectList> collect_list(c_int#83071, 0, 0) will run on GPU
[2021-02-07T03:07:38.289Z] E                         !Expression <WindowSpecDefinition> windowspecdefinition(a#83069L, b#83070 ASC NULLS FIRST, c_int#83071 ASC NULLS FIRST, specifiedwindowframe(RowFrame, unboundedpreceding$(), currentrow$())) NOT EVALUATED FOR GPU YET
[2021-02-07T03:07:38.289Z] E                           !Expression <AttributeReference> a#83069L NOT EVALUATED FOR GPU YET
[2021-02-07T03:07:38.289Z] E                           !Expression <SortOrder> b#83070 ASC NULLS FIRST NOT EVALUATED FOR GPU YET
[2021-02-07T03:07:38.289Z] E                             !Expression <AttributeReference> b#83070 NOT EVALUATED FOR GPU YET
[2021-02-07T03:07:38.289Z] E                           !Expression <SortOrder> c_int#83071 ASC NULLS FIRST NOT EVALUATED FOR GPU YET
[2021-02-07T03:07:38.289Z] E                             !Expression <AttributeReference> c_int#83071 NOT EVALUATED FOR GPU YET

RunningWindowFunctionExec does not exist outside of Databricks.  From the name it appears to be an optimization that lets them do some window operations in one pass instead of multiple passes.  Either way we need to have someone look at the plans being produced and figure out how to translate them properly to the GPU.

Thanks Bobby&Tom.
I have made the PR https://github.com/NVIDIA/spark-rapids/pull/1783 to fix this issue in the short term. Since this test focuses on the functionality of collect_list over window.
Of course, the long term fix should replace this RunningWindowFunctionExec  with a GPU plan properly.

Filed a new issue (https://github.com/NVIDIA/spark-rapids/issues/1791) to track the support for this RunningWindowFunctionExec , will  close this issue with the PR https://github.com/NVIDIA/spark-rapids/pull/1783


watch stream: don't unmarshal responses when streaming log

We were doing a best-effort unmarshalling for both pod log and regular watches, but we really shouldn't try unmarshalling pod logs into watch events. The existing code would unmarshal a pod log that happens to be a JSON blob and contains a field "object".
/hold
We need an e2e test for basic watch functionality in the main repo first.
/cc @Ark-kun @palnabarun

Codecov Report

Merging #185 into master will increase coverage by 0.05%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #185      +/-   ##
==========================================
+ Coverage   92.46%   92.52%   +0.05%
==========================================
  Files          13       13
  Lines        1487     1485       -2
==========================================
- Hits         1375     1374       -1
+ Misses        112      111       -1

Impacted Files
Coverage Δ

watch/watch.py
100% <100%> (+1.17%)
:arrow_up:

Continue to review full report at Codecov.

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

adding a test in main repo: https://github.com/kubernetes-client/python/pull/1087

kubernetes/e2e_test/test_watch.py::TestClient::test_watch_configmaps Running test against : https://<IP_ADDRESS>:8443
PASSED

/hold cancel
This change is ready for review. Please take a look

Thank you for walking me through the changes and the background personally. :)
/lgtm


Android Plugin for Gradle doesn't play nice with the current libGDX source

Issue details
As previously discussed in issue #4230, starting with version 1.3 the Android Plugin for Gradle doesn't like a module and it's parent sharing the same name. You end up with an error message when trying to do a Gradle Sync or to build and run the Android project from within the IDE.
In both Android Studio and IntelliJ IDEA, you get the following error:
Error:(23, 1) A problem occurred evaluating project ':tests:gdx-tests-android'.
> Failed to apply plugin [id 'com.android.application']
   > Your project contains 2 or more modules with the same identification com.badlogicgames.gdx:gdx-box2d
at ":extensions:gdx-box2d" and ":extensions:gdx-box2d:gdx-box2d".
You must use different identification (either name or group) for each modules.

That part is easy enough to fix. All you need to do is to rename the parent "module".  I have already done this part in commit 50179d2a870c1c938d5a0c5014b9c0a09fc3c4d7 on my fork. Up until a few days ago it was enough to get the code to Sync and Run nicely in the IDE. (I even submitted PR #4316 to that effect.)
Unfortunately, with the refactoring of the Gradle build (0ea025084f59d8feb7711f825bf0f11c440830bd) the Gradle Sync and Instant Run within the IDE are no longer working even with this fix.
We're now left with the following error:
Error:(18, 1) A problem occurred evaluating project ':tests:gdx-tests'.
> Could not find method compile() for arguments [project ':gdx'] on object of type org.gradle.api.internal.artifacts.dsl.dependencies.DefaultDependencyHandler.

Interestingly, the command-line Grade build and installDebug do work without a hitch. It would appear that the Android plugin get confused with the dependencies. Manually adding apply plugin: "java" to the gradle.build file of :tests:gdx-tests will allow the build to proceed further but even when adding it to all the related modules, other errors crop up further along so it's not a viable solution and I had to admit defeat due to my limited knowledge of Gradle.

make sure configure on demand is off in your IDE

Thanks, that did the trick. I don't get why IDEA and Android Studio are ignoring the line in gradle.properties though.


How to train a NN to check if a graph is cyclic

This is more of a help question than a bug/suggestion but this might be a good example of an advanced case for https://github.com/liquidcarrot/carrot/issues/28
What would be the best way to build a NN to check if a graph is cyclic (https://en.wikipedia.org/wiki/Cyclic_graph) or not. I've been thinking about how to input a graph of different sizes and shapes and unsure how to best do this.
Here is a simple example of a cyclic graph:

{a -> b, a -> e, b -> c, c->d, d->a}
I'm not sure what is best to input this kind of data.

Probably the best way to approach this problem is by treating it as an
exploration game in which the NN moves through the graph, time and wrong
guesses are penalized, and a correst guess is rewarded.
Otherwise, the main issue is inputting an unbounded large graph. You would
need to use a recurrent model and input the connections one by one, for
example. I think that this is quite a challenge for a nn to learn using
standard training methods.
On Sun, Jul 7, 2019, 11:01 PM Daniel Ryan<EMAIL_ADDRESS>wrote:

This is more of a help question than a bug/suggestion but this might be a
good example of an advanced case for #28
https://github.com/liquidcarrot/carrot/issues/28
What would be the best way to build a NN to check if a graph is cyclic
https://en.wikipedia.org/wiki/Cyclic_graph or not. I've been thinking
about how to input a graph of different sizes and shapes and unsure how to
best do this.
Here is a simple example of a cyclic graph:
[image: Cyclic graph]
https://camo.githubusercontent.com/0a71c6bcf3874d3204d0eb25027311a3169261cf/687474703a2f2f7777772e616e616c79746963746563682e636f6d2f6e6574776f726b732f677261706874312e676966
{a -> b, a -> e, b -> c, c->d, d->a}
I'm not sure what is best to input this kind of data.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/liquidcarrot/carrot/issues/135?email_source=notifications&email_token=ADLB3ZECQYZ45L63PCD5UJDP6KUXBA5CNFSM4H6XNJ2KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G5X2YKA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADLB3ZBRAJFHDO7OPV27KO3P6KUXBANCNFSM4H6XNJ2A
.

SIDE NOTE:  In example above, what would be the purpose of "E"? A storage node? A past state "echo chamber" node?
I think there's an ENORMOUS amount of value in some cyclical node structures. Would be interesting to have a shape tool....IDK - just a random thought - for after unit test suite v0.

treating it as annexploration game in which the NN moves through the graph, time and wrong
guesses are penalized, and a correst guess is rewarded.

Interesting suggestion, but still stuck on what data do you feed the NN. You basically feeding it temporary IDs.

Otherwise, the main issue is inputting an unbounded large graph

I was thinking the example for this could be limited to 10 nodes max to make the solution easier.
What about feeding the NN one hot encoded data of two nodes (that are connected) at a time.
A = 1, B = 2, C = 3, D = 4, E = 5, F = 6, G = 7, H = 8, I = 9, J = 10
So for the above example {a -> b, a -> e, b -> c, c->d, d->a}, you would feed in
[1,0,0,0,0,0,0,0,0,0 (for a),0,1,0,0,0,0,0,0,0 (for b)]
[1,0,0,0,0,0,0,0,0, 0 (for a),0,0,0,0,1,0,0,0,0 (for e)]
[0,1,0,0,0,0,0,0,0, 0 (for b),0,0,1,0,0,0,0,0,0 (for c)]
[0,0,1,0,0,0,0,0,0, 0 (for c),0,0,0,1,0,0,0,0,0 (for d)]
[0,0,0,1,0,0,0,0,0, 0 (for d),1,0,0,0,0,0,0,0,0 (for a)]
Not sure if that will work. But could give it a try.

In example above, what would be the purpose of "E"? A storage node?

This is more of a thought experiment to a more advanced example in predicting DNA relationships. In this case just another node in the graph. A node that will get in the way to finding the solution.

This is more of a thought experiment to a more advanced example in predicting DNA relationships. In this case just another node in the graph. A node that will get in the way to finding the solution.

Got you! I think HyperNEAT might be able to handle some of these idiosyncratic topologies - but will have to test.

So for the above example {a -> b, a -> e, b -> c, c->d, d->a}, you would feed in (one line at a time)
[1,0,0,0,0,0,0,0,0,0 (for a),0,1,0,0,0,0,0,0,0 (for b)]
[1,0,0,0,0,0,0,0,0,0 (for a),0,0,0,0,1,0,0,0,0 (for e)]
[0,1,0,0,0,0,0,0,0,0 (for b),0,0,1,0,0,0,0,0,0 (for c)]
[0,0,1,0,0,0,0,0,0,0 (for c),0,0,0,1,0,0,0,0,0 (for d)]
[0,0,0,1,0,0,0,0,0,0 (for d),1,0,0,0,0,0,0,0,0 (for a)]

This would be an interesting addition to API - I wonder if there's a way of simplifying the interface...I'll give some things a shot.
SIDE NOTE: This look like a typical GPU like solvable problem; in abstract it also leads me to think that a "shortest path" algorithm can go a long way in situations like this...

With a variation of Graph Convolutional Networks, we could come up with a cool model that works for graphs of any size. I'm also interested in whether the simple model proposed there can actually learn to classify noncyclical vs cyclical graphs.

"we could come up with a cool model that works for graphs of any size"
Yeah, could be handy.
"I'm also interested in whether the simple model proposed there can actually learn to classify noncyclical vs cyclical graphs."
I'll build something this weekend if I have time.

I need some training data.

So I nearly have this all built but I'm a little sure how to pass in the training data. My input is an array of arrays which Carrot doesn't support.  Suggestions?

We're planning to start supporting I/O of any format after an upcoming refactor (eta probably like two weeks or less). Until then, probably the best is to just destructure the inner array into the input array. I'm pretty sure that any kind of destructuring that preserves order will do the job (e.g. input = []; input = input.concat(arr1); input = input.concat(arr2); input.push(variable_number);).

The input would be quite large. If the max is 10 nodes, then I can have up to 45 edges. So the input size would be 45 * 2 * 10 = 900. I guess I could give that a try.


Login issues

In the login menu on the desktop version, pressing "enter" on the keyboard after writing the e-mail and password doesn't do anything, so I had to click on "Log in" using the mouse. Same thing when trying to log in via the SciStarter account, except that I got this message: "Can not log in to that account through SciStarter, because it was not created via SciStarter". Also, when logged-in on either desktop or mobile version, opening "https://sciencenearme.org/my/goals" and "https://sciencenearme.org/my/profile" in new tabs redirects to "https://sciencenearme.org/login?next=my-goals" and "https://sciencenearme.org/login?next=my-profile" respectively, thus prompting the login page again. However, that problem doesn't arise if I opened these same pages in the same tab.

@SciStarter is there some reason you assigned this to me?  Reassign it to me if you disagree.

@SciStarter I fixed the enter thing by just wrpping in form element. But the other part about the new tabs and seeing the 'next' is probably yours.

The prompt for logins is a security precaution that we can't afford to skip, even though it's annoying.


[WIP] Upgrade dependencies and migrate to Scala 2.13.2

Hello !
As there were no updates on https://github.com/bot4s/telegram/pull/88 I wanted to start the migration, this is a draft and is open to discussion and comments.
I removed ScalaJS but we should be able to put it back after playing a bit with some dependencies (we need to bump to ScalaJS 1.0 I guess), the current settings is targeting 2.11, 2.12 and 2.13.
I'm having some VERY VERY strange compilation errors with the current state (in 2.13.2)
mill __.compile
[error] telegram/core/test/src/com/bot4s/telegram/api/CommandsSuite.scala:39:17: type mismatch;
[error]  found   : String("/hello")
[error]  required: com.bot4s.telegram.api.declarative.Command => ?
[error]       onCommand("/hello")(handlerHello)
[error]                 ^
[error] telegram/core/test/src/com/bot4s/telegram/api/CommandsSuite.scala:40:17: type mismatch;
[error]  found   : String("/helloWorld")
[error]  required: com.bot4s.telegram.api.declarative.Command => ?
[error]       onCommand("/helloWorld")(handlerHelloWorld)
[error]                 ^
[error] telegram/core/test/src/com/bot4s/telegram/api/CommandsSuite.scala:57:19: type mismatch;
[error]  found   : String("/cmd")
[error]  required: com.bot4s.telegram.api.declarative.Command => ?
[error]     bot.onCommand("/cmd")(handler)
[error]                   ^
[error] telegram/core/test/src/com/bot4s/telegram/api/CommandsSuite.scala:73:19: type mismatch;
[error]  found   : Symbol
[error]  required: com.bot4s.telegram.api.declarative.Command => ?
[error]     bot.onCommand('cmd)(handler)
[error]                   ^

This seems to be a implicit resolution issue, but I was not able to isolate and recreate the bug, and Intellij (with Implicits hints shows the correct values).
What is even stranger is that some tests ARE passing, for instance on line 39 and 40 there is an issue, but line 41 is fine:
      onCommand("/hello")(handlerHello)
      onCommand("/helloWorld")(handlerHelloWorld)
      onCommand("/respect" & RespectRecipient)(handlerRespect)

And
 bot.onCommand("/cmd")(handler)
seems to be an issue when  bot.onCommand("/cmd")(handler) is not...
I really don't understand what is happening right now, could some please try this locally, to check that this is not related to my setup.
Thanks

I'm very interested in having this, thanks for your work.
I'll try to continue from where you left off, but I am by no means an expert.

I couldn't spend more time on this, but this lead me down a rabbit hole of a bug inside the scala-compiler, check https://github.com/scala/bug/issues/12070
I think that one solution (I did not investigate) would be to drop the support of Symbol in this lib (those will be deprecated in Scala 3 anyway), and/or remove the implicit conversion from string => Command

@XDracam I did a quick fix for the issue described above, I didn't have the time to test it yet (I don't have any telegram token easily accessible right now), could you please pull the changes and try it
Thanks

Wow, thanks for the response. Will do.

I had to migrate the example bots and remove some slogging clutter.
Also, I don't think it's a good idea to make the conversions explicit, as that will drastically deteriorate the quality of the API. Instead, I renamed onCommand(Filter[T]) to onCommandWithFilter. That did not break any example bots during compilation.
The random bot works fine now!
I'll continue to upgrade dependencies to newer versions where possible, run tests etc based on your version, and then I'll open a new PR. If that's fine by you.


LDAP authentication reports Invalid credentials

Hi @heiglandreas , thanks for your work!
As far as I know, I configured everything fine, however I cannot login with LDAP users:
in WP_DEBUG_LOG there is no more than:

LDAP authentication failed with exception: bind was not successfull: Invalid credentials

LDAP is accessible, with the same service user/password, etc., I can configure Authorizer, but your plugin would be better for LDAP - if I could get it to work.
Now, I have ARGON2 hashed passwords in OpenLDAP, and ppolicy overlay to hash clearText passwords (as a workaround because Apache Directory Studio doesn't handle ARGON2, so I pass cleartext from there), if that is relevant.

Hey.
Thanks for raising this.
I assume you have activated the debugging output in the configuration.
And I assume that the issue is not with the actual user but with the LDAP-URI. Can you provide me some more configuration  details - sanitized for passwords - here or via email to authldap AT heigl DOT org?

Thanks for the quick reply!
I think your assumptions are correct: I made an intentionally wrong LDAP-URI - wrong user - which resulted in the same error.
LDAP debugging is enabled.
~My LDAP-URI looks like this: ldap://cn=wordpress,ou=services,dc=example,dc=com:%env:LDAP_PASSWORD%@ldap.example.com:389/dc=example,dc=com~
WOW, what a blunder!
I did try a lot of things. **But a single space made it to the beginning of LDAP-URI :laughing: **
I guess you can close this, as it works now!

Great that it works now!
And thanks for mentioning this!
I will add a trim before storing the LDAP-URI then to pervent such havoc in the future!


[Bug] – Provide a detailed description, as well as steps for anyone to reproduce the issue

[Bug] – Can't create a new branch in sync

Insomnia Version: 7.0.3
Operating System: Windows 10

Details

The message does not convey this, but it also has to be more than 3 characters long. I will update the rule to allow 3-character strings, as that was the original intention.


Publish to npm on tag

We should set up a GitHub Action so that the npm package is built and published when tagging a release:
https://github.com/denoland/dnt#github-actions---npm-publish-on-tag

#11 was just merged, adding the GitHub Action.
Keep this issue open until the next release and close after confirming the npm version was successfully published:
https://www.npmjs.com/package/complaindate

Confirmed, the automatic npm package release worked!


buttons hover effect added.

I have added hover effect on the buttons in about us section , and attached the screenshots also.
Resolves issue #55
Before :

After :

Conflicting files
assets/css/style.css

Please see the conflicts highlighted in the image above. Resolve the conflicts in the mentioned files. Ensure you pull all the latest changes before pushing your feature/bug fixes to the main branch. After resolving the conflicts, please open a new PR for review.
Thank you!


Space application supporter can get, list service instances

Space application supporter can get and list service instances, and list shared spaces relationships of service instances.
Unit tests to prove the above.

Closes #2238

[x] I have reviewed the contributing guide

[x] I have viewed, signed, and submitted the Contributor License Agreement

[x] I have made this pull request to the main branch

[x] I have run all the unit tests using bundle exec rake

[ ] I have run CF Acceptance Tests

Hi @will-gant , it looks like we picked up a few merge conflicts when merging in the PR removing residual perm code. Apologies for any inconvenience this may have caused.
If we get around to looking at this PR today we will try to help resolve them, thanks again for taking the time to make contributions to the codebase!

Hey @sweinstein22,
The conflicts are resolved.
Regards,
Aftab

hey @sweinstein22,
Thank you for your feedback. Changes are done.
Regards,
Aftab