[NETBEANS-461] providing ZIP file with extbrowser DLLs during build

Thanks for looking into this incident as I have temporary workaround because I am a NetBeans user.

@geertjanw thanks for looking into this - I assume the files were build from the sources contained in the native subdirectory of extbrowser? This currently breaks builds, because the Verification task fails:
/home/travis/build/apache/incubator-netbeans/nbbuild/build.xml:1532: Failed VerifyLibsAndLicenses test(s):
Some license files have incorrect headers
extbrowser/external/extbrowser-dlls.zip does not appear to include a version number
extbrowser/external/extbrowser-dlls.zip is not associated with any license file

I suggest to rename the zip to something like extbrowser-dlls-<year><month><day>.zip to satisfy the "requires version" requirement. For the "license file requirement" a file extbrowser-dlls-<year><month><day>-license.txt is required. Assuming the files were build from the CDDL sources, lib.profiler/external/profiler-external-binaries-8.2-license.txt can be used as a template.

I think the question also is whether a ZIP file containing the two DLLs is the right approach. Maybe the two DLLs need to be listed separately in the binaries-list, i.e., they'd then be downloaded as DLLs and automatically put in the right place since they'd be in the 'external' folder. That could be better than downloading a ZIP file which will then somehow need to be unzipped?

To answer my own question, this is how it's done, in the module's build.xml:

Not really sure what license to use, so followed @matthiasblaesing suggestion above.

A question at this point that remains is whether 18.03.15 (i.e., today's date) makes sense as a version number.

Hi, the date should be fine but see what the previous versions
reported.  I think the files should be listed in
the binary-list.  How are other similar dll being handled?
from NB 8.2 reports:
org.netbeans.modules.extbrowser                    1.53.1
Enabled
from NB 8.0.2 reports:
org.netbeans.modules.extbrowser                    1.48.1
Enabled
Maybe this release if not the date, use 1.54.1, the date is easier to
remember and document.
My vote is for the date because I have not background on the original
versioning.
Regards,
Efrem McCrimon (efremmc2)
On Thu, Mar 15, 2018 at 3:54 AM, Geertjan Wielenga<EMAIL_ADDRESS>
wrote:

A question at this point that remains is whether 18.03.15 (i.e., today's
date) makes sense as a version number.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-netbeans/pull/454#issuecomment-373290847,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKLVlPftvqdP8Q4R0fSj1d1o3J3LlAQBks5teh5KgaJpZM4Sq7I6
.

Native libs are handled in this way generally, i.e., they’re made available and registered in a ZIP file, then unzipped and put in the right place during the build.

No objections received, time to merge, any problems please file a new issue.


Updating to fix deprecated Kubernetes APIs

Fixes #3403

I'm also working on a bigger refresh using the latest version which is currently 6.0.1. There were more API changes so that's a bigger change. I'll submit it as a separate merge request in the future. That one I will need to do more testing on.

@nicknezis  is this ready for review now?

@joshfischer1108 Yes I think it's ready for review.
The 6.0.1 version of my code will be separate pull request because there might be bigger discussion about the code changes. There is value in taking this pull request in the interim because anyone wanting to use K8s 1.16 will have issues with the yamls.

I'm closing this pull request to continue the work on #3426 which has more Kubernetes Java client api changes. #3426 uses the 6.0.1 version of the client as opposed to the older 3.0.0 which was used in this pull request.

sounds good. thanks!


Ouldouz branch

The file prefixexp.py includes the class Plus

Closing as obsolete, we do not continue with this direction anymore.


Split extra help item computing with help record rendering

I maintain Click Extra, a drop-in replacement for Click which adds colorization of the help screen.
I am currently relying on finicky regular expressions to identify and match the extra items that are displayed at the end of each option's help. These extra items are rendered in square brackets and looks like:

[default: None]
[default: (unlimited)]
[env var: COLOR_CLI8_TIME; default: no-time]
and a variety of other formats.

The problem is that the generation of these extra items are deeply embedded within Click and are impossible to fetch independently.
That's why I propose a simple refactor of the Option.get_help_record() method, and split it in two:

a new method to compute the values of these extra items
keep the original get_help_record() as-is, but dedicate to help message rendering only

I have a PR that is ready at #2517.

Just wanted to ping @janluke on that issue, as he also maintain a Click-based extension that is also deep into help screen customization and coloring.

I've also heard rumors of upcoming big refactors regarding help screen generation, so I propose this small, un intrusive one as a small step towards the big one.


Add support for private registries

Checklist

[x] tests and code linting passes
[x] a test is included
[x] the commit message follows commit guidelines

Affected core subsystem(s)

install
registry

Description of change
By utilizing registry-auth-token, this change ensures that ied resolves which registry to use for a given scope in the same manner as NPM does, and fetches any associated auth token for the registry.
By using the correct registry URL and sending the bearer token along, this allows ied to install private packages from both the public npm registry and private registries.
We're currently using Sinopia at our company, which unfortunately doesn't support the same /<packageName>/<versionSelector> endpoint, so I had to add some logic to resolve the correct version from the full package details endpoint (/<packageName>). Not super happy with the cleanliness of the "legacy mode" as I called it, so feel free to change it however you feel.
I could possibly add some tests, but it's quite hard to mock because of how the rc module caches its results, along with how it resolves .npmrc files. I had to resort to some ugly hacks in registry-auth-token.
Feedback? This should address parts of #7 and possibly #122.

Updated to reduce dependencies on request from @mgcrea

Couldn't wait for having this support. Do you mean this will make ied capable to install modules with git URL like "git+ssh://xxxxxx" from our company's private git lab?
When will you merge and release the new version of ied? I just tested today and having this error:
/usr/local/lib/node_modules/ied/node_modules/rxjs/Observer.js:5
    error: function (err) { throw err; },
                            ^
AssertionError: error status code https://registry.npmjs.org/protobuf/git+ssh://git@git.xxxx.com:12888/service-common/protobuf.git#2.0.3: undefined
    at checkStatus (/usr/local/lib/node_modules/ied/lib/registry.js:97:20)
    at SafeSubscriber._next (/usr/local/lib/node_modules/ied/lib/registry.js:134:12)

This looks great!
I'm wondering: Should we really use registry-auth-token? It looks like we're only interested in those 3 lines:
  var rc = require('rc')('npm', {registry: 'https://registry.npmjs.org/'})
  var url = rc[scope + ':registry'] || rc.registry
  return url.slice(-1) === '/' ? url : url + '/'

Maybe it would make sense to just inline them...
Concerning the test: A simple unit test would be perfect (maybe stub out fs.readFile - I know it's ugly, but it's better than no tests at all).

Actually, registry-auth-token contains both code to resolve both the registry url and the auth token for it. The token logic is slightly more complicated. If npm decides to change the way their tokens or registry info is saved, we'll have to update ied, instead of just updating the dependency. Your call, but the module is pretty slim as it is.
I'll see about adding some tests.

Well, you did ask for tests ;-)
Unfortunately, I found very little tests for the install command and registry parts, so I had to dig a little deep and try to figure out which way to do it. I settled on mocking the http calls using nock and stubbing the registry auth token parts using proxyquire.
I could have mocked out the fs module as you suggested, but then I'd basically be testing both the registry-auth-token and rc modules, which seems wrong for unit tests.
Anyway, I'm heading of to Italy tomorrow for some vacation time, so if you want me to do any changes you'll have to wait for a while or do them yourself ;-)
Happy summer! :sunny: :sunglasses:

Awesome, thanks! Great stuff. Will add this to the next release. Have fun in Italy!

Back from Italy, and rebased! Anything you want me to change here before this can be merged? The tests are a bit flakey, but pass locally - I suspect it's related to the npm registry giving a whole lot of errors lately.

Hey @rexxars ! Cool cool! Thanks for your PR!
I want to make sure the tests are stable before merging this.
(Was thinking of using replay to mock out the registry later on.)
For now, let's try to get this working. I'm rerunning the tests on CI right now.

If this is what I think it is this would extremely nice to have merged in and available. Need any help with getting this finished/finalized?

RIP ied

Luckily pnpm is actively maintained and usage growth.


PR: Add German translation

It's finally done!
The German translation for

Spyder,
Breakpoints Plugin,
Profiler Plugin,
Pylint Plugin
is complete and added to base.py.

I've found some typos in the English source strings. Search for 'TYPO in source' within my translation comments.

Thanks a lot for this!! Since you did this PR against our master branch, this is going to be part of Spyder 4, to be released early next year.
I hope you're ok with that.

We could use the @rlaverde rebase onto thinguie to make it available on 3.x?

I don't think so. There are different strings on master, and at different locations, so this has to go to master.

Have you seen the typo remarks within my translation comments?

Yep, thanks for noticing them. We'll fix them for Spyder 3.2.

Rebase needed?

@be-we, I solved the merge conflict using Github's web interface, so don't worry about it.
I'll merge your PR after our tests pass. Thanks a lot for this, it's a great contribution!

This is the eight language Spyder is translated to, yay!!

The failure in Circle is unrelated to this, so merging.

I see you like the new translation.


Emit events for each order book update type

The following event types are added:
orderbook.diff.asks. deleted
orderbook.diff.asks. inserted
orderbook.diff.asks. updated
orderbook.diff.bids. deleted
orderbook.diff.bids. inserted
orderbook.diff.bids. updated
These events allow a developer to update their own order book during backtesting, rather than having to store a different snapshot at each timestamp.

Looks good. Thank you very much. LGTM! I'll release this in the next version on Friday with some other minor improvements to npm.

Great!


Feature request: 3d egg

Is your feature request related to a problem? Please describe.
There doesn't seem to be a way currently to create a 3d object with different radii to create shapes like 3d eggs, bell's and/or simple rocket nozzle's.
Describe the solution you'd like
A quick & easy way to create a 3d egg-like object with varying curved angle
Describe alternatives you've considered

Using rotate_extrude on a 2d egg doesn't work as you can't rotate a shape within itself. So you can only create a fancy torus this way
Onion's come close but have a fixed angle for the top cone, while an egg/bell shape has a varying curved angle for the top cone
Teardrop's might maybe sort-of work in some use-cases if they'd support rounding edge.

Example Code
na
Additional context
i'd consider:

a 3d egg (🥚) a shape like the thing that comes out of a 🐔 and you use to make an omelet 🍳
a bell (🔔) as a 3d egg with cut-off bottom and maybe negative bottom chamfer/rounding
a simple rocket nozzle as a 3d egg with cut-off bottom (probably to the maximum of the biggest circle)

I understand that this is probably already possible somehow, but given the available 3d shapes this one seems missing. Or in other words, basically the onion shape shouldn't have to exists and could probably be a variant of the 3d egg?


Incompatible with Turbolinks

How can re-instantiate the off canvas menu with a turbo link load? Basically the body is replaced with ajax content, so the scripts are not called again, but I can fire a function on turbo link load, so I could re-setup the object but I cannot see how.

I figured out a workaround, but feels kind of hacky. Let me know if there is a better way.

@matthewscalf hey! Sorry the slow reply to this. You could just init the offcanvas script in the Turbolinks ready event.

Ah to do that we probably need #66 😞


are the install instructions for linux only?

If so, should mention explicitly.

I don't even know, maybe it works for OS X, too.

Let's say it's untested on OS X then

Got this message when building in macOS. Pretty sure SCIP is installed. Be able to see CSIP in julia deps/usr/... But it doesn't go through when building.
INFO: Building SCIP
INFO: Attempting to Create directory /Users/jac0320/.julia/v0.4/SCIP/deps/downloads
INFO: Directory /Users/jac0320/.julia/v0.4/SCIP/deps/downloads already created
INFO: Downloading file https://github.com/SCIP-Interfaces/CSIP/archive/v0.3.5.zip
INFO: Done downloading file https://github.com/SCIP-Interfaces/CSIP/archive/v0.3.5.zip
INFO: Attempting to Create directory /Users/jac0320/.julia/v0.4/SCIP/deps/src
INFO: Directory /Users/jac0320/.julia/v0.4/SCIP/deps/src already created
INFO: Attempting to Create directory /Users/jac0320/.julia/v0.4/SCIP/deps
INFO: Directory /Users/jac0320/.julia/v0.4/SCIP/deps already created
INFO: Path /Users/jac0320/.julia/v0.4/SCIP/deps/src/CSIP-0.3.5 already created
INFO: Changing Directory to /Users/jac0320/.julia/v0.4/SCIP/deps/src/CSIP-0.3.5
make: Nothing to be done for `all'.
INFO: Changing Directory to /Users/jac0320/.julia/v0.4/SCIP/deps/src/CSIP-0.3.5
INFO: Changing Directory to /Users/jac0320/.julia/v0.4/SCIP/deps/src/CSIP-0.3.5
================================[ ERROR: SCIP ]=================================
LoadError: Provider BinDeps.BuildProcess failed to satisfy dependency libcsip
while loading /Users/jac0320/.julia/v0.4/SCIP/deps/build.jl, in expression starting on line 34
================================================================================
===========================================================[ BUILD ERRORS ]===========================================================
WARNING: SCIP had build errors.

packages with build errors remain installed in /Users/jac0320/.julia/v0.4
build the package(s) and all dependencies with Pkg.build("SCIP")
build a single package by running its deps/build.jl script

======================================================================================================================================

And also this...
julia> BinDeps.debug("SCIP")
INFO: Reading build script...
The package declares 1 dependencies.

Library "libcsip"

Providers:

Simple Build Process

Thanks

I don't know anything about building on OSX, but I'm guessing that the file extension .so could be a problem. Should it be dylib or something?

updated instructions with 9592057.


Debian

Hello,
Here is a PR to provide debian package with the build and update several tools.
Changes :

upgrade to kafka <IP_ADDRESS>
upgrade to sbt 13.8
add debian package
update doc to generate the debian package
rename project to a more conventional name (kafka-offset-monitor)

Tested on Jenkins and ubuntu 14.04
Take it if you want :yum:
Enjoy :smile:

nice, I am not sure about the name change. Is that a requirement for Debian pacakages?
Also, to merge, I will need you to sign the contributor agreement:
https://github.com/quantifind/KafkaOffsetMonitor#contributing
Thanks

@Mortimerp9 it's more conventional for a name :-)

Dammit typesafe repository is down ....

@Mortimerp9 i have just sent my contribution agreement ;-)

:+1:


Convert baseline file to match local filesystem path

Problem
Currently, detect-secrets doesn't support multi OS environments very well.
If a Windows developer generates a baseline file, Linux developers will always be alerted on new secrets and vice versa.
This is because baseline files have OS specific paths
Windows:
"filename": "src\blah\appsettings
Linux:
"filename": "src/blah/appsettings
When performing new_secrets = secrets - args.baseline you will always get new secrets because the filenames dont match.
Solution
The best solution would probably be to generate baseline files/read files in a consistent manner to always use Unix paths with something like os.path.normpath Doing this however will mean people will need to update their baseline files.
The solution implemented in the PR instead will using os.sep to determine if we are on UNIX or Windows, then replace slashes in filenames for the loaded baseline file (without changing the baseline file). The comparison will then work regardless of if you are on Linux comparing against a windows generated baseline file and vice versa.

Hey there, requesting any feedback on this PR
@lorenzodb1 @jpdakran

Hi @eddiez9, thank you for opening this PR.
Looks like tests are failing. I'd appreciate if you could take a look and get them all passing.

I've had to change the unit tests as they make assumptions about UNIX pathing wrt. assertions.
To summarize all the changes:
core/potential_secret.py
core/secrets_collection.py
Changed to convert paths to the local file system path seperator for secrets via scans but also via baseline files.
util/path.py
convert to local os path function
tests/audit/audit_test.py
change assertion to use os.path.join instead of hardcoded /
tests/pre_commit_hook_test.py
tests/core/secrets_collection_test.py
change assertions to use os.path.join instead of hardcoded /
tests/core/baselien_test.py
this failing test checks we can read a baseline file and output it exactly the same.
this fails on windows after the path conversion so i've added a check to replace slashes if run on Windows. Another solution for this could potentially have a seperate secrets.baseline.windows file that has windows paths but this complicates pre-commit exclusions


Merge messages at turn end to reduce channel spam

<johnm> grifferz: I think this has been commented on, but it would be good if it rolled up all those instructions into a single line. Perhaps something like: Time for the next card, message me your answers. New Card Tsar: popey, Black Card: ...

The output right now looks like:

<AgainstHumanity> The winner is grifferz, who now has 2 Awesome Points!
<AgainstHumanity> The new Card Tsar is popey.
<AgainstHumanity> Time for the next Black Card:
<AgainstHumanity> → What gets better with age?
<AgainstHumanity> Now message me your answers please!

I would be happy to see this compressed down to:

<AgainstHumanity> The winner is grifferz, who now has 2 Awesome Points! The new Card Tsar is popey. Next Black Card:
<AgainstHumanity> → What gets better with age?


GitHub's new search bar quick jump

GitHub just rolled-out a new feature for its search bar:

Which leads to interferences with this extension:

Yes :/ we will need to do an update to take that feature into account.

Why is this closed? It is still an issue and is unrelated to stylish themes.

This isn’t closed, still an open issue

Just wanted to note that this is still a problem; it makes this extension basically unusable because GitHub defaults to using the value in its own searchbar instead of the one in this extension. Workaround for now is to just use the mouse to select the option brought up in the search, like an animal (/s)

Thanks for pushing @macintacos , I'll check internally who could have some time to work on the extension.


Tititopemike

Worked on the ajv schema, refactored the create role and also implemented the verify role route.
Also created api using jetbrains spotlight.

Worked on the ajv schema, refactored the create role and also implemented the verify role route.
Also created api using jetbrains spotlight.

resolve the conflicts first

Ressolved conflicts Worked on the ajv schema, refactored the create role and also implemented the verify role route.


First Production Went Well!

Didn't find a discord group or other discussion platform that seemed appropriate, so I'll leave this here...
We did our first live production today using TallyArbiter polling an ATEM Production 4K switcher, 5 cameras along with four Atom Matrix listeners and it went very well. Everything worked just as it should. Running the server code on the same Mac Mini as our Wirecast instance. Thinking I might move the TallyArbiter server to a dedicated Raspberry Pi to avoid any possible problems in the future, but don't have any justification for doing so.

And since this isn't an issue, I'll close it!

That's awesome! Thanks for letting me know!


Add link to send email to submitter

Changes in this PR:
A link is added to the editor UI which opens a new email to the submitter, with things like TDR reference in the subject line precompleted.
This was informed by prototyping from @Terry-Price. We should very strongly consider moving structured, repetitive emails like confirmations to this kind of behaviour as an interim before we start automating them entirely.

Codecov Report

Merging #628 (8153c55) into main (63a9d1a) will increase coverage by 0.19%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #628      +/-   ##
==========================================
+ Coverage   70.14%   70.33%   +0.19%
==========================================
  Files          22       22
  Lines         767      772       +5
  Branches       35       35
==========================================
+ Hits          538      543       +5
  Misses        221      221
  Partials        8        8

Impacted Files
Coverage Δ

judgments/views/edit_judgment.py
58.18% <100.00%> (+1.99%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add Websocket Endpoints

Is there any possibility of adding a websockets controller similar to the Java Spring Boot websocket controller? so that there are endpoints of the application and the websockets work in a way similar to REST, I think this would make it easier to use and would be a great abstraction.

I like the concept of this, but because there is no standard for websocket message content (and this would be required to do event routing), I think this should exist as a separate package built on top of Aqueduct.

Well, the idea of becoming a reality, standardizing the sending of messages via websocket, would be a pattern similar to the STOMP protocol, I think it would be a great abstraction, have some idea of where I should start in order to build this on the Aqueduct? I would like to understand more about how Aqueduct works with websockets and how I could interact with it, thanks for the attention.

https://aqueduct.io/docs/http/websockets/


Siamese network tutorial for Torch

Depends on #483 and #485

Very cool tutorial! I especially like the images at the end. However, I'm running into what looks like a Torch version issue.
When I use Torch from a deb installer, I get this error:
2015-12-28 10:03:52 [FAIL] /usr/local/share/lua/5.1/nn/Narrow.lua:14: bad argument #3 to 'narrow' (out of range at /home/bfomitchev/git/d4/extra/cutorch/lib/THC/THCTensor.c:367)
DIGITS Lua Error
stack traceback:
[C]: in function 'narrow'
/usr/local/share/lua/5.1/nn/Narrow.lua:14: in function 'updateOutput'
/usr/local/share/lua/5.1/nn/Sequential.lua:44: in function 'forward'
/home/lyeager/digits/tools/torch/main.lua:622: in function 'Validation'
/home/lyeager/digits/tools/torch/main.lua:779: in main chunk
[C]: in function 'dofile'
/home/lyeager/digits/tools/torch/wrapper.lua:25: in function </home/lyeager/digits/tools/torch/wrapper.lua:25>
[C]: in function 'xpcall'
/home/lyeager/digits/tools/torch/wrapper.lua:25: in main chunk
[C]: in function 'dofile'
/usr/local/lib/luarocks/rocks/trepl/scm-1/bin/th:145: in main chunk
[C]: at 0x00406670

@borisfom why is your home directory hard-coded into the Torch installation!?
When I use my local torch installation, I get the same error:
2015-12-28 10:06:25 [FAIL] /home/lyeager/torch/install/share/lua/5.1/nn/Narrow.lua:14: bad argument #3 to 'narrow' (out of range at /home/lyeager/torch/extra/cutorch/lib/THC/THCTensor.c:367)
DIGITS Lua Error
stack traceback:
[C]: in function 'narrow'
/home/lyeager/torch/install/share/lua/5.1/nn/Narrow.lua:14: in function 'updateOutput'
/home/lyeager/torch/install/share/lua/5.1/nn/Sequential.lua:44: in function 'forward'
/home/lyeager/digits/tools/torch/main.lua:622: in function 'Validation'
/home/lyeager/digits/tools/torch/main.lua:779: in main chunk
[C]: in function 'dofile'
/home/lyeager/digits/tools/torch/wrapper.lua:25: in function </home/lyeager/digits/tools/torch/wrapper.lua:25>
[C]: in function 'xpcall'
/home/lyeager/digits/tools/torch/wrapper.lua:25: in main chunk
[C]: in function 'dofile'
...ager/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:131: in main chunk
[C]: at 0x00406670

Hi Luke and sorry for the late feedback. Do you have three channels in your input images? You should if you're using create_db.py to create your LMDB database.

I'm not sure what I was doing before. It works for me now.
There's a merge conflict from #510 before we can merge.
By the way, that create_db.py script is SUPER slow for me. It wasn't responding when I gave it -c 100000, so I took it down to -c 100 and it still took 199 seconds to finish. It's all disk IO, the CPU is hardly used at all. Are you seeing the same?

Thanks Luke, I have rebased this PR on the tip of the master branch.
Regarding the DB creation slowliness I realized that the script was flushing data to the filesystem after each sample. Creating a database of 100,000 samples was taking about 15 minutes on my SSD drive but slower filesystems (network?) might have been even slower. On #519 I have implemented batch writing. This speeds DB creation up considerably.


Give an option to override the default number of shards in BQ streaming inserts

As per https://github.com/apache/beam/pull/12485 there's no way to override it, though Java does have one:
https://github.com/apache/beam/blob/master/sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/bigquery/BigQueryOptions.java#L58

Let's add a good way to override this.
Imported from Jira BEAM-10680. Original Jira may contain additional context.
Reported by: pabloem.

.take-issue


Bumping a local crate's version forces a registry sync

Happened to me on a train today. If you increment your local crates version, Cargo will hit the registry. There's no need to do that.

Hm, more precisely. I have sever local crates with interdependencies, so I bumped several of their versions as well as their dependencies on each other. All local deps.

Fixed in  https://github.com/rust-lang/cargo/pull/3144


[FLINK-35633][cli] Verify YAML pipeline definitions early

This closes FLINK-35633.
Currently, transform & route rules will be passed to Flink operators as strings, and any malformed configurations will not be exposed until runtime, which makes testing and debugging harder.
This PR adds early verification logic in Flink CDC CLI, and tries to provide more clear and descriptive error messages for input YAML files.

Not a priority for now.


BUG move test file to correct folder and fix class_exists

To fix a bug where not including phpunit in a live deployment broke the site.

Thank you.


chore(docs): tutorial typo nullable schema defined

I am getting start with tutorial.
and I found something wrong typo.
t.nonNull.field("createDraft", {...})

then, generated SDL as
createDraft(body: String!, title: String!): Post!

and
t.nonNull.list.field("drafts", {...}) => drafts: [Post]!
...
t.list.field("posts", { ...}}  => posts: [Post]

ty!


Update warning message

The warning message for Management API Security disabled tells users the old way to do things.
Instead, we should tell users the new way to do things using PRISMA_CONFIG.
The contributing guidelines say that PRs for server should be made against the alpha branch. This is technically a PR for server, but it's just updating warning text. If you're ok, it'd be nice to merge directly to master. Please let me know and I'll change the base fork to master.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Also related, if you'd be so kind: https://github.com/prismagraphql/prisma-templates/pull/18


Issue with framework i think

So im trying to figure this out I've reinstalled CentOS and installed both php5.5 and tried php7.0 after that both come up with the same issues.
If i try to install with the latest version of Composer i get an error so i go down 1 version no errors but when i go in to solder it's self this is what happens
http://i.imgur.com/TwHT4Y7.png
As you can see debug is on and no errors are being printed. I've also made sure the public and app/storage have the right permissions.
Any clue what im doing wrong? i've had it running before on CentOS 6 but for some reason on CentOS7 it doesn't seem to want to work.

Are you hosting this with https?

No.

can you upload the source of that page as a gist and link it here?
you can view the source of the page by hitting ctrlu

https://gist.github.com/giogimic/774be34fdee868b0a6f6
Sure thing feel free to sign in to the solder to view it yourself aswell i havent changed the default password

tbh, i'm stumped.
the first image (@giogimic's site) shows that all of the .css files and title.png are initially canceled, then successfully loaded later (but not applied?).
the second image (my site) shows how the page normally loads (through cloudflare, at least) (ignore the failed ga.js; adblock).

i have no idea what would cause an issue like that...

To add insult to injury
I don't have those cancelled requests

To add insult to injury
I don't have those cancelled requests

@giogimic Can you hop on the discord server? maybe we can get it fixed through the live/voice chat.

Sure. I'm Chuck on Discord.

you guys ever get this figured out? i'm genuinely curious what the issue was...

Nope still having the issue not resolved yet.

No one's messaged me on discord so please if you find a solution post here.

So @giogimic I can offer you a dirtx solution to this, which is.
Replace the href in this line https://github.com/TechnicPack/TechnicSolder/blob/master/app/views/layouts/master.blade.php#L13
with https://maxcdn.bootstrapcdn.com/bootstrap/3.3.6/css/bootstrap.min.cssin your local installation, that seems to fix the issue of not loading bootstrap properly


the right way to update telescope

I know that is a question more related with git, but I don't see anything about it in the documentation.
What is the correct way to update Telescope to the last tagged version?
In the documentation you are pulling from master... so tagged version is last stable version or master is correct?

I personally created stable branch on my fork and update using git checkout stable && git merge vX.X.X command.

@delgermurun thanks!

There's also http://www.telesc.pe/docs/getting-started/updating/


Google et al without Identity + sample.

Initial work for #12091.
I've created a sample app that demonstrates login and logout using Google + Cookies without Identity. I've also created an almost-empty docs page that just contains links to the relevant source snippets that I intend to discuss within the page.
I'm submitting this early as a draft PR so that I can get feedback on two things:

The location and naming of the docs page.
The sample app. I don't expect to add anything else to this so I'd appreciate eyes on whether it's a good basis for the docs page.

I intend for the docs page to focus on Google specifically but to acknowledge and link to the other providers accordingly.

A couple of passing remarks:

Idk if either alternative was considered ...

Use a modified version of the Additional Claims sample for this. (However, note that a big PR is modifying that one now on https://github.com/aspnet/AspNetCore.Docs/pull/11950.)
No sample ... just call out the changes that are required in one or a few snapshot files. If this is possible, it would be great to not have one more sample to update every release. We can't keep up as it is. :cry:

Nit: We try to make GH happy about ending files on a blank line. See the Privacy.cshtml.cs file ... add a return to the last line. It may have even come from the template that way. I'll check the engineering repo. If they need a PR for it, I'll put that in with them.

Since this subject matter pertains to social login, I think @tdykstra will want it in aspnetcore/security/authentication/social/, and the UID would get the social segment. Because of that, I would recommend that having the word "social" again in the URL isn't required. The last segment can be without-identity (or something else that doesn't include "social"). He'll come on and let u know what he wants to do on these points.

Name the sample app SampleApp. We put them into a samples (plural) folder and then inside of a 2.x folder for this version. That way , we can add a 3.x folder with another SampleApp folder later. The samples link in the intro of the topic will go like this (but hold changes tho until u hear from Tom).

[View or download sample code](https://github.com/aspnet/AspNetCore.Docs/tree/master/aspnetcore/security/authentication/social/without-identity/samples) ([how to download](xref:index#how-to-download-a-sample))

... and the full path to a file would be ...

aspnetcore/security/authentication/social/without-identity/samples/2.x/SampleApp/SomeFile.xxxx

Ellipses in the TOC? I'm surprised. Still tho ... my half-penny's worth ... something more like this would be better ...
- name: Social authentication without Identity
  uid: security/authentication/social/without-identity

For most samples we're drop static assets to slim the repo, but these authn/authz ones really suffer without them. We've been leaving the static assets (CSS, JS) in these authn/authz samps, so you're good there. :+1: However, you can drop the cookie policy bits from the sample. Not needed; we're dropping it from samples now; and IIRC, engineering is dropping it in the future anyway (although, they might keep it for auth templates ... not sure).

Thanks for the detailed feedback @guardrex - this is exactly why I decided to get visibility on this as early as possible. I’ll wait for the others to respond accordingly before I look into making changes, as you’ve suggested. It seems like I might already be looking at a do-over so perhaps I’m being a bit too ambitious with contributing on this one... 😟

I hope they say (and u think it can be done) that we don't need a full sample ... just snapshots. I'm still trying to get back to our 2.0 and 2.1 samples (and I think there are even 1.x samples! Yikes!). 🏃😅 .... this is one case where The more the merrier! definitely does NOT apply. :smile: lol

I sent in a patch PR on the line endings. That will speed us up over here on docs. I always stop and add them as I go with a new sample. https://github.com/aspnet/AspNetCore/pull/10192

For most samples we're drop static assets to slim the repo,

Just the minified and a few others that aren't used in local dev. That's automatic via .gitignore

I think the scenario where things get ugly is working in development. There's no fallback for the non-min versions of static assets.

I think the scenario where things get ugly is working in development. There's no fallback for the non-min versions of static assets.

Just use .gitignore. That's our standard. The debug assets are trivial.

My approach totally removes them (in non-auth sample cases). My samps always look marvelous! 😄 .... hopefully, they run ok, too! lol

Don't do that. Just depend on .gitignore

The debug assets are trivial ... Just depend on .gitignore

They aren't. In dev, static assets won't be present on the dev's machine for a downloaded sample. The sample will look great here (on my machine) but when the dev downloads it from the repo (without non-min static assets) it will look like crap when they run it in development.
If you're saying to force the static assets into the repo with .gitignore, ok, but that defeats the goal of having slim samples.
https://github.com/aspnet/AspNetCore.Docs/issues/9186 is still open ... recommend that we get that figured out if there's still a misunderstanding on my part. I'm just going by what I thought we worked out there ... see DR's remark at https://github.com/aspnet/AspNetCore.Docs/issues/9186#issuecomment-432827970.

@guardrex Are there any samples I can help bring up to 2.2 to try and lighten the load?

Are there any samples I can help bring up to 2.2 to try and lighten the load?

I defer to management on that.

@serpent5 take a look at the issues and let me know what you'd like to do. It doesn't matter if the issue is currently assigned to someone else.

What are the next steps for this?

@serpent5 sorry. I'll start reviewing it today.

That's fine. No need to apologise at all. I'm just checking in here as the comments went a bit off topic and I also wanted to make sure you know I'm still keen to try and get this done.

@serpent5  what are the steps to get this working? This is what I tried:
dotnet new webapp -o WebApp1 -au Individual -uld
Remove <partial name="_LoginPartial" /> from the layout file.
Update code to what you have. I run the code and click on the Privacy link, but a break point on OnGetLogin is never hit.
If we do this, I think we'd need to hook up the UI so you could sign-in/sign-out

You don't want to create the project with -au Individual -uld as that sets up Identity, which I expect is conflicting with the AddAuthentication stuff I've added. OnGetLogin is a replacement for what would normally be the "Account/Login" route in MVC/RP and is triggered within the sample I created by clicking on a login link that I added to the navbar to mimic what it would look like with Identity.
If you run the sample app as is, it should all work. If you click "Privacy", it won't trigger the OnGetLogin page but it will send you straight over to Google. OnGetLogin and OnPostLogoutAsync are handlers that get triggered from the navbar in the sample, which I added in an attempt for consistency with the Identity approach. You can see this in the _LoginPartial I added and modified in the sample.
I'd intended for the docs to be a guide that called out the sections most relevant to the auth stuff, but to just mention that the navbar in the sample app is linked to those page handlers. I'm happy to change the approach to a step-by-step guide that starts with an empty app and talks through the start to finish process. Alternatively, we can ditch the page handlers and just detail adding the auth stuff and applying the [Authorize] attribute. This could be based on the webapp template or could be more basic, starting from the empty template.

I like your approach.  I think it's best to keep it as simple as possible.
Let me add some instructions and we'll see what the security folks say.

Moved to #12444

@serpent5 Using postman, I post to https://localhost:44369/Index to logout but I get back 400 bad request.
When I'm not sign-ed in. I have a break-point (BP) on
public IActionResult OnGetLogin()
{
    return Challenge(new AuthenticationProperties
    {
        RedirectUri = Url.Page("/Index")
    });
}

I redirect too google to approve, but I never hit the above BP.

@Rick-Anderson If you're using Postman, it's likely 400ing due to the antiforgery token being absent. The endpoint would also need the Logout handler to invoke the logout code, i.e. https://localhost:44369/Index?handler=Logout. Otherwise, it's just going to render the view without first logging out.
OnGetLogin was only there for the case where we have a "Sign in" button, in order to issue a challenge explicitly. If we don't have a button/link for signing in, we don't need to include a OnGetLogin handler.
Thanks for your patience here. I had intended to flesh out the docs page after the sample was reviewed, but things have gone in a different direction to what I originally anticipated, so I realise that what I've provided isn't very clear now.

@serpent5  thanks. I'll remove the  OnGetLogin handler.  I think all we need is a mechanism to logout.


UUID scope

Hi @mdsadique-inam I appreciate for your work, but there are few things which will conflict with upcomming updates(soon), I just pushed new update which supports uuid for scopes, you can check


🎨: layout search bar plugins: updated  / 🌎: translation update

Summary
Even though plugins will be deprecated soon, I decided to give the plugin search bar a layout update, making it look more like the rest of the new layout. Also added Show Agents Settings/ Completion to the translation files for better understanding.
Change Type

[x] Update Layout
[x] Translation update

Testing
All passed
Before:

After:

Modifications:
Fixed bugs that were causing the code to malfunction.
Refactored the code to make it cleaner and easier to understand.
Added comments to the code to explain what each part of it does.
Used best programming practices to ensure that the code is robust and reliable.
Refactorings:
Improved the structure of the code to make it more organized and maintainable.
Renamed variables and functions to make them more descriptive and easier to understand.
Extracted common code into reusable functions and components.
Simplified the code by removing unnecessary complexity.
New Implementations:
Added a confirmation dialog to prevent users from accidentally uninstalling plugins.

Thank you for this! I will review and test when I get the chance


build: allow mimalloc to be an optional dependency

This PR allows mimalloc to be an optional dependency. I had trouble building datafusion-python on macOS in GitHub Actions CI without this patch. Happy to expand on that if necessary.

@cpcloud hi thanks for the pr - i guess this needs to be rebased


Bug: NPE when INSERT INTO b SELECT * FROM a

Maybe directly related to #10851

CrateDB version:
4.3.2
Environment description:
CrateDB Cloud - Dev Cluster

3x 0.25 vCPUs
3x 2 GB Memory (0.5 GB Heap)

Problem description:

1 TIMESTAMP column (PK)
~ 190 DOUBLE PRECISION columns
~ 10 TEXT columns

CREATE TABLE open_pv (
    DateTime TIMESTAMPTZ PRIMARY KEY ,
  temperature_outdoor_fantasytown_weather DOUBLE PRECISION,
  ...
  c_phase_voltage_brightcounty_pv DOUBLE PRECISION);

103.2 MB (Size primary shard)
23,422 records

when creating a second table open_pv2 with the schema as open_pv and trying to copy the data with
INSERT INTO open_pv2 SELECT * FROM open_pv
-- INSERT ERROR (0.000 sec)

this leads to the following error
NullPointerException[Cannot invoke "org.elasticsearch.common.io.stream.Writeable$Reader.read(org.elasticsearch.common.io.stream.StreamInput)" because "this.reader" is null]

Limiting the number by using LIMIT 10000; the query succedes
INSERT INTO open_pv2 SELECT * FROM open_pv LIMIT 10000;
-- INSERT OK, 10000 rows affected (37.742 sec)

Steps to reproduce:

Create Table open_pv

CREATE TABLE open_pv (
    DateTime TIMESTAMPTZ PRIMARY KEY ,
  temperature_outdoor_fantasytown_weather DOUBLE PRECISION,
  ...
  c_phase_voltage_brightcounty_pv DOUBLE PRECISION);

Insert ~25.000 rows

Create Table open_pv2

CREATE TABLE open_pv2 (
    DateTime TIMESTAMPTZ PRIMARY KEY ,
  temperature_outdoor_fantasytown_weather DOUBLE PRECISION,
  ...
  c_phase_voltage_brightcounty_pv DOUBLE PRECISION);

Try to copy table with INSERT SELECT FROM

INSERT INTO open_pv2 SELECT * FROM open_pv
-- INSERT ERROR (0.000 sec)

Just tested the same on a bigger cluster -> same result
adding (even very high) LIMITs mitigates the issue ...
i.e.
INSERT INTO open_pv2 SELECT * from open_pv LIMIT 1000000;

works

@proddata Thank you for reporting this. Unfortunately we have problems reproducing this.
Can you reproduce the issue even with the stripped table schema you posted?
Thank you again for reporting.

@proddata We've fixed this by #10882, it will be part of the next 4.3 release.
Thank you again for reporting.


Fix logic of new suppress_inline_failure_output option

suppress_inline_failure_output was actually doing the opposite of what it claimed to do. This commit reverses that.

🤕 Sorry about that. Thanks for seeing it through.


Allow objects to input data into EdfReader

Currently working on AWS with EDF objects. Making a feature request to allow some kind of objects similar to StringIO or urllib3.HTTPResponse for input into the EdfReader class instead of a filename. Presumably, this could work using a .read() call on the object. This would save on some IO so that objects wouldn't have to be directly written to local disk before reading and could instead stay remote based.

This is probably not so easily possible, as the library we wrap (edflib) is written in C and accepts strings that point to files on the file system. Unfortunately we cannot easily pass FILE objects or streams to it, and the last time I tried to implement that I didn't get further. It would probably be possible somehow, but would need some adaptation of the C code to work with streams.
If you feel comfortable enough in C to work on something like this let me know!
Else I have to unfortunately put this on the backend, as I don't have enough time right now do work on this.
See also #102 and #100


PerfItDelegatingHandler calls context.Handler.OnRequestStarting for all installed counters

As far as I know due to the nature of the DelegatingHandler we actually do not know the exact WebApi method being called and as a result all installed/registered counters' handlers receive the OnRequestStarting call but only one receives OnRequestEnding call when async send completes. Is it possible that we find a way to identify which WebApi Controller methods is being processed if not shall we consider implementing interception in filter level (like in PerfItMvc ) but not in DelegatingHandler level.

No activity


lon_range, lat_range are clipped

Beyond a certain range, the  lon_range, lat_range are getting clipped.
In this example, the two different ranges do not give substantially different plotted ranges
import numpy as np
import healsparse
import hpgeom
import matplotlib.pyplot as mplt
import skyproj

def main():
    nside = 256

    area = hpgeom.nside_to_pixel_area(nside=nside)

    radius_degrees = np.sqrt(area) / 2
    print(f'radius {radius_degrees:.3g} degrees')

    ras, decs = 200, 45
    pixelnum = hpgeom.angle_to_pixel(nside, ras, decs)

    ra, dec = hpgeom.pixel_to_angle(nside=nside, pix=pixelnum)

    circ = healsparse.Circle(ra=ra, dec=dec, radius=radius_degrees, value=1)
    smap = circ.get_map(nside_coverage=32, nside_sparse=nside, dtype=np.int16)

    for factor in [2, 20]:

        lat_range = [dec-radius_degrees*factor, dec+radius_degrees*factor]
        lon_range = [ra-radius_degrees*factor, ra+radius_degrees*factor]
        print('lat range:', lat_range)
        print('lon range:', lon_range)

        fig, ax = mplt.subplots(figsize=(8, 8))
        sp = skyproj.McBrydeSkyproj(ax=ax)
        im, lon_raster, lat_raster, values_raster = sp.draw_hspmap(
            smap,
            lon_range=lon_range,
            lat_range=lat_range,
        )
        sp.draw_inset_colorbar()

        fname = f'factor{factor}.png'
        print('writing:', fname)
        fig.savefig(fname)

main()

Okay, this is a combination of a lack of functionality and an unclear docstring.  I have added the functionality on this branch and updated the docstring:
zoom : `bool`, optional
    Optimally zoom in projection to computed map. This will override
    lon_range/lat_range if these are larger than the map.
lon_range : `tuple` [`float`, `float`], optional
    Longitude range to plot [``lon_min``, ``lon_max``]. Map will
    be truncated to this range prior to computing optimal zoom
    (if zoom is ``True``).
lat_range : `tuple` [`float`, `float`], optional
    Latitude range to plot [``lat_min``, ``lat_max``]. Map will
    be truncated to this range prior to computing optimal zoom
    (if zoom is ``True``).

So if you use the latest branch and also add zoom=False to your call to draw_hspmap (overriding the default of zoom=True) then it should behave the way you expect.

Yes, the ranges are now different for the two cases in the posted example.

It still seems to be not exactly doing what's requested.  This might be a good thing in most cases.
Here is an example where I requested a very strict limit in the lon range.  It is too tight, so it would have clipped the map.
I requests: lon range: [200.075845621326,<PHONE_NUMBER>7623498]
But skyproj refuses to do that.
import numpy as np
import healsparse
import hpgeom
import matplotlib.pyplot as mplt
import skyproj

def main():
    nside = 256

    area = hpgeom.nside_to_pixel_area(nside=nside)

    radius_degrees = np.sqrt(area) / 2
    print(f'radius {radius_degrees:.3g} degrees')

    ras, decs = 200, 75
    pixelnum = hpgeom.angle_to_pixel(nside, ras, decs)

    ra, dec = hpgeom.pixel_to_angle(nside=nside, pix=pixelnum)

    circ = healsparse.Circle(ra=ra, dec=dec, radius=radius_degrees, value=1)
    smap = circ.get_map(nside_coverage=32, nside_sparse=nside, dtype=np.int16)

    fig, ax = mplt.subplots(figsize=(8, 8))
    sp = skyproj.McBrydeSkyproj(ax=ax)

    lon_range = [ra-radius_degrees*2, ra+radius_degrees*2]
    lat_range = [dec-radius_degrees*4, dec+radius_degrees*4]

    print('lon range:', lon_range)
    print('lat range:', lat_range)

    im, lon_raster, lat_raster, values_raster = sp.draw_hspmap(
        smap,
        lon_range=lon_range,
        lat_range=lat_range,
        zoom=False,
    )
    sp.draw_inset_colorbar()

    mplt.show()

main()
![skyproj-refuses](https://github.com/user-attachments/assets/b464867c-ade4-41f6-914b-860594191b14)

This is a tricky one, and I don't have an answer.  The problem is that this is a very distorted portion of the projection (as you can see from the diagonal lines).  Notice that the RA range is what you asked for, given the dec range, and it goes from the corner to corner.
The only solution is to either (a) choose the lon_0 for the skyproj to be at the center of your map; or (b) use a gnomonic projection (useful for small regions, but you have to set the extent up front when you create the skyproj).

OK, thanks. I'm fine with this behavior as is.


Toggle the timer display triggers the render twice

We need to do something smarter to avoid this problem.
One idea:

Compute render time for opaque, translucent and volume pass
Update the timer
Render overlays

Another idea:

Use a different renderer for geometry and overlays

https://gitlab.kitware.com/f3d/f3d/-/issues/129
https://gitlab.kitware.com/f3d/f3d/-/merge_requests/127

@Meakk labelling as wontfix

may be fixed by #1742


Bump to Kubernetes v1.8.4

Release note: https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG-1.8.md#changelog-since-v183
Note that CA v1.0.3 is included with an important fix.
In case it was your first time to contribute to this project, contrib/bump-version script is useful for this task!

I hope etcd client 3.2.10(https://github.com/kubernetes/kubernetes/issues/56114) to be included in the next patch version of k8s btw.


Using a record as command options

I have different commands and each of them accepts different options. So I'm using a separate record to represent each command's options, but I couldn't get that to work.
Here's an example, I have the file args.hs:
{-# LANGUAGE DeriveGeneric       #-}
{-# LANGUAGE OverloadedStrings   #-}
{-# LANGUAGE ScopedTypeVariables #-}

import Options.Generic

data FirstCommandOpts = FirstCommandOpts
  { xs :: Maybe Int
  , ys :: Maybe Bool
  } deriving (Generic, Read, Show)

data Cmd
  = FirstCommand FirstCommandOpts
  | SecondCommand
  deriving (Generic, Read, Show)

instance ParseRecord Cmd
instance ParseFields Cmd
instance ParseField  Cmd
instance ParseRecord FirstCommandOpts
instance ParseFields FirstCommandOpts
instance ParseField  FirstCommandOpts

main :: IO ()
main = do
  (cmd :: Cmd) <- getRecord "Example"
  case cmd of
    FirstCommand opts -> putStrLn "First Commmand"
    SecondCommand     -> putStrLn "Second Command"

When I run it, I get the following:
$ ./args.hs secondcommand
Second Command

$ ./args.hs firstcommand
Missing: FIRSTCOMMANDOPTS

Usage: args.hs firstcommand FIRSTCOMMANDOPTS

$ ./args.hs firstcommand --xs 2

Invalid option `--xs'

Usage: args.hs firstcommand FIRSTCOMMANDOPTS

My question is, how to get ./args.hs firstcommand and ./args.hs firstcommand --xs 2 to work, and if there's no way to get this to work, shouldn't I get a compilation error instead of a message about wrong usage?

The reason there is no type error is because there the default implementation of ParseField that uses the Read instance for a type.  I believe your example would work if you did:
./args.hs firstcommand 'FirstCommandOpts { xs = Nothing, ys = Nothing }'

Of course, that's probably not what you want.  In general, optparse-generic only supports options one level deep, so you would need to restructure your original type that you parse into as:
data Cmd
  = FirstCommand { xs :: Maybe Int, ys :: Maybe Bool }
  | SecondCommand
  deriving (Generic, Read, Show)

I need to have FirstCommandOpts as a separate record, so I'll go with optparse-applicative to do that.
Thank you for your answer.

You're welcome! 🙂


Information flow between Master Control and Free Navigation AGV?

I want to know how to communicate master control with free navigation AGV.
(like figure: https://github.com/VDA5050/VDA5050/blob/development/assets/Figure2.png)
Is it feasible to do this?
Step 1. Master control sends  only start and destination coordinates. (Shall I still use "order" topic?)
Step 2. The vehicle sends its path to the master control. (What topic shall I use? Or can I define a new topic like "VehicleRequest")
Step 3. I think master control should tell vehicle if the path is valid, so need another topic like "MasterControlResponse"?
Step 4. Then vehilce executes its path (if accepted by master control) or  replan (if rejected).
thanks

In general, I think free navigation isn't well covered by the standard. So here my personal opinion:

Sending only start and end coordinates (or nodes) may only work well if the vehicle has itself a layout which defines where the AGV/AMR is allowed to navigate. Such a layout isn't covered by the standard today as well.
Sending the path to MC also isn't covered by the standard and maybe produces a lot of network traffic (if the path changes).
3,4) Such a concept isn't intended by the standard and would need a much more complex protocol. The idea of the VDA5050 is more like: MC defines the path.

We at Bosch Rexroth use the node and edges inside an order to define a corridor from the start to the destination node. This corridor defines the space in which the AGV/AMR can freely navigate. Maybe you can also work with that concept?


Impossible to add an Announcements

💡 Summary
It has become impossible to add an announcement.
☹ Actual Behavior
When we use the Add button nothing happens.
📌 Environment
Misskey 12.67.0

After a few tests, this bug only appears under Windows with Firefox or Vivaldi, but no problem with QuteBrowser. The problem does not appear under Linux (Archlinux) with Firefox (with the same extensions as under Windows).
I don't understand but it seems more from me than from a Misskey problem.

After a few tests, this bug only appears under Windows with Firefox or Vivaldi, but no problem with QuteBrowser. The problem does not appear under Linux (Archlinux) with Firefox (with the same extensions as under Windows).
I don't understand but it seems more from me than from a Misskey problem.


New version: JetBrains.Rider.EAP version 222.2270.26

Result: Installation Failed

Manifest
Add/Remove Programs

Name
JetBrains Rider (EAP)
CLion 222.2270.30

Version
222.2270.26
222.2270.30

Publisher
JetBrains s.r.o.
JetBrains s.r.o.

ProductCode
JetBrains Rider 221.5080.72
CLion 222.2270.30

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 1638
Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Allow client to configure the expiry on the OTP

Is your feature request related to a problem? Please describe.
Currently the otp:get returns an OTP from the server. The expiry of the OTP lies with the server. Enhance the code for the client to optionally set the expiry for the opt:get
Describe the solution you'd like
Add an optional parameter to the "otp" verb to set the expiry limit.

The changes in at_commons are in: https://github.com/atsign-foundation/at_libraries/pull/422
The changes in at_server are in: https://github.com/atsign-foundation/at_server/pull/1598

Pending PR review

The changes are merged to trunk and released into Canary 3.0.36b


Format any JSON parsable object

Shouldn't have to be a pino object. If its parsable as JSON, apply formatting, if not, just pipe to console.

done


Investigate and adjust the table of content margin and padding

As a user,
I want the table of content to have consistent margin and padding,
so that it aligns properly with the grid, including the bottom line.
At the bottom of the
Acceptance Criteria:

Investigate the current margin and padding settings for the table of content.
Identify any discrepancies or misalignments with the grid, especially considering the bottom line.
Adjust the margin and padding settings to ensure that the table of content sits correctly on the grid.
Verify that the changes are consistent across different screen sizes and devices.
Ensure that the visual appearance of the table of content is improved and aligns with the overall design guidelines.

Visualisation

Link to the issue on Figma https://www.figma.com/design/Wr9b6pK35euFoA6xq3G9Ld/Grid-review?node-id=0-1&node-type=canvas&m=dev

@Majsol Dina, we need your input on this ticket, what is the best approach to the issue you highlighted?


Types and Functions defined in _test.go files are not visible/exported

Go version 1.4.2 release
OS: Linux amd64
The go install command doesn't pick up structs and functions which are defined in source file with names xxx_test.go
I know go test command uses _test.go files as part of the built in testing framework, but I cannot find where it says go install doesn't pick up definitions from these files. In case I am building a lib, and I write tests in _test.go files (define some functions etc.) and run them from package main, but at present those definitions won't be visible and go install throws an error: undefined: TypeName
Is it possible that go install/build can be trained to throw a more verbose error message: For example: Definition found in xxx_test.go, please change the file name to something other than xxx_test.go

Ok, upon going through the code, I found these two places:
https://github.com/golang/go/blob/master/src/cmd/go/build.go#L949
https://github.com/golang/go/blob/master/src/cmd/go/test.go#L951
_test.go files are not included in coverage. Fine but is it possible that go install/build can be trained to throw a more verbose error message? For example: Definition found in xxx_test.go, please change the file name to something other than xxx_test.go

Apparently this issue has the side effect of resulting in inconsistent test runs whether you run go test against a relative path (like ./...) or an absolute (import-like) path, like github.com/user/pkg.
Namely, if you run go test ./..., and your test files live in package foo_test, and they import foo, but they rely on something exported under *_test.go files (a common example is theexport_test.go, like the one in pkg fmt), go test will import the package from $GOPATH, not from wherever go test ./... is run (which makes sense). However, as @faizoff mentioned, the *_test.go files are not included in those builds (not even in tests builds, when the package being tested is other than the package that is imported, minus the _test suffix).
This usually never happens because people tend to run go test ./... while they're in the source tree. However, it does happen with the new Wercker Ewok, which runs Go tests in a different directory (/pipeline/source). This results in CI tests that use go test ./... failing, while CI tests that use go test github.com/user/pkg/... are passing.
/cc @hatchan

Working as intended. The *_test.go files are compiled into the package only when running go test for that package.


Non-pub members are accessible outside of std.

Non-pub members are accessible outside of std. For example, this is done in the hello world example.
example:
https://github.com/zig-lang/zig/blob/101b7745c4da1fdf0ebe723710fe8b195013fc0e/example/hello_world/hello.zig#L4-L8
https://github.com/zig-lang/zig/blob/101b7745c4da1fdf0ebe723710fe8b195013fc0e/std/os/file.zig#L302-L310
https://github.com/zig-lang/zig/blob/101b7745c4da1fdf0ebe723710fe8b195013fc0e/std/io.zig#L30-L38

Thanks for the report. I believe this is #569


fixup: iter_branch_regions

This fixes:
Traceback (most recent call last):
  File "/Users/esc/git/numba-rvsdg/test_fig_scc.py", line 79, in <module>
    bflow = lflow._restructure_branch()
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/esc/git/numba-rvsdg/numba_rvsdg/core/datastructures/byte_flow.py", line 43, in _restructure_branch
    restructure_branch(region.subregion)
  File "/Users/esc/git/numba-rvsdg/numba_rvsdg/core/transformations.py", line 345, in restructure_branch
    head_region_blocks = find_head_blocks(bbmap, begin)
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/esc/git/numba-rvsdg/numba_rvsdg/core/transformations.py", line 244, in find_head_blocks
    assert len(jt) == 1
AssertionError

From: https://github.com/numba/numba-rvsdg/issues/5#issuecomment-1483845948
The basic jist: the state of main that this applies to mixes data and analysis, such that regions may be considered as blocks and stored inside the BlockMap/SCFG as blocks. During extraction of a region the blocks in the region are added to a set, which is used to initialize a new BlockMap/SCFG structure. Since the blocks are in a set the insertion order into the region structure is random, because Python sets are randomized. This means, that the order in which you end up iterating over them is also random / different from run to run. This means the order in which they arrive in iter_branch_regions is random. This is because the underlying bbmap.graph is a dict and the items are returned in insertion-order --- but --- the items are inserted in random order as they come from a set. However, what iter_branch_regions needs is a breadth first ordering. This patch fixes this, by ordering the items in the dict and returning them in a breadth-first (BFS) manner. Additionally, existing regions must be "concealed", that is to say, they must act as though they would be a single block, such that there are no cycles in the control-flow-graph (CFG) as all loops have been identified and are treated as a single linear block. The iterator in this PR also respects that w.r.t. the way the underlying data is stored.

Rebased after merge of #41

After a long OOB discussion with @kc611 -- I learned that on experimental one of the big issues was related to the refactoring of the dominator functions. Especially during branch restructuring, they need to view the graph in a "concealed" fashion, just like the iter_branch_regions in this PR. But there is more. They also need to check if something is within the graph and thus need a __contains__. Hence a simple iterator will not suffice as we need additional accessor methods. In this PR, I propose an object oriented implementation that uses the Facade pattern. Additionally, I chose to inherit from collections.abc.Mapping which effectively provides various methods, including __contains__ automagically. I thus replaced the call to the iterator in iter_branch_regions with the items method of the new view class. The tests are passing fine.

I'll add a test too.


issue with minimizers

Expected Behavior
When HSWP is used together with uglify-js plugin caches turned on only the first cold build should be long - all the subsequent ones should use both HSWP and uglify-js caches
Actual Behavior
three consequent with no code changes in-between

first run: slow module build + slow minification
second run: fast module build (HSWP kicks in) + slow minification
third and subsequent runs: fast build + fast minification

Additional info

uglify's cache folders before and after second run are identical
with HSWP turned off the timeline is

first run: slow + slow
second and subsequent: slow + fast

things that do not change anything

switching uglifyjs for terser
changing cache directories for hs and/or uglify
turning off parallelisation for uglify

Operating System, Node, and NPM dependency versions
Mac OS 10.13.6
Node 8.9.4

closing in favor of #454


Added libdisni RPM/DEB build environment

Hi Patrick,
These are the necessary changes for integrating with MLNX_OFED: ability to build RPM/DEB files
I already noticed that version 1.1 is hardcoded here, and Vlad will take care of it soon
Please share your comments if you have any
Cheers,
Yuval

Vlad, Yuval,
Great, the ability to build RPM/DEB is generally very helpful, thanks a lot! Should be an easy merge..
-Patrick

I notice that this is partially undoing some of the changes @patrickmacarthur did in his recent pull request:
https://github.com/zrlio/disni/commit/a1948b8bd8c7199e05a53ab2468c20de669ae946.
For instance one argument there was since we don't need a static library for JNI anyway, there is also no need to built a libtool library, etc, which I thought made sense. Now we are again adding +libtoolize --force --copy.
Also we are adding +autoconf, is that need given we do autoreconf?
My knowledge about GNU autotools was just sufficient to get the basic setup ready initially but I'm not an expert here. Can we discuss this quickly.

Chiming in:

autoreconf is a wrapper that automatically calls autoconf, automake, autoheader, libtoolize, etc. in the correct order. @vladsokolovsky I'm curious why you believe that it is not sufficient in this case.
I think libtool is still required to (easily) build a shared library with autotools; my changes just made it so it doesn't build a static library by default anymore. Using autoreconf means that we don't need to care in the autoprepare.sh script.
Looking at the entire PR, I don't think installing the header files is useful as the only thing that uses the library is JNI itself (unless there really is something else (ab)using the JNI library).
The Ubuntu convention for JNI libraries seems to be to call them "lib-java-jni", place the library in /usr/lib/$(DEB_HOST_MULTIARCH)/jni, and not create a -dev package. I presume it's the same or similar in Debian.

Hi Patrick,
I addressed your comments and pushed my changes.
Can you please review/merge?
Thanks,
Vladimir
From: Patrick MacArthur<EMAIL_ADDRESS>Sent: Friday, August 25, 2017 1:27 PM
To: zrlio/disni<EMAIL_ADDRESS>Cc: Vladimir Sokolovsky<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [zrlio/disni] Added libdisni RPM/DEB build environment (#13)
Chiming in:

autoreconf is a wrapper that automatically calls autoconf, automake, autoheader, libtoolize, etc. in the correct order. @vladsokolovskyhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fvladsokolovsky&data=02|01|vlad%40mellanox.com|4f31b716bf4f4a6edb8008d4ebe6dda2|a652971c7d2e4d9ba6a4d149256f461b|0|0|636392824183259368&sdata=8dBnkHc0lfEVvseFwCh39AQNL%2F1FIZQcYUcgpa0J7F8%3D&reserved=0 I'm curious why you believe that it is not sufficient in this case.
I think libtool is still required to (easily) build a shared library with autotools; my changes just made it so it doesn't build a static library by default anymore. Using autoreconf means that we don't need to care in the autoprepare.sh script.
Looking at the entire PR, I don't think installing the header files is useful as the only thing that uses the library is JNI itself (unless there really is something else (ab)using the JNI library).
The Ubuntu convention for JNI libraries seems to be to call them "lib-java-jni", place the library in /usr/lib/$(DEB_HOST_MULTIARCH)/jni, and not create a -dev package. I presume it's the same or similar in Debian.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fzrlio%2Fdisni%2Fpull%2F13%23issuecomment-325001935&data=02|01|vlad%40mellanox.com|4f31b716bf4f4a6edb8008d4ebe6dda2|a652971c7d2e4d9ba6a4d149256f461b|0|0|636392824183259368&sdata=lsb4j4%2Fp0zmgzW81JlEG2V7GmH6uWeAM%2Fe%2BA4yXorQs%3D&reserved=0, or mute the threadhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAbx26etPYvVRuBoe5GquN1OtzLUu51mvks5sbxHtgaJpZM4PCzKY&data=02|01|vlad%40mellanox.com|4f31b716bf4f4a6edb8008d4ebe6dda2|a652971c7d2e4d9ba6a4d149256f461b|0|0|636392824183259368&sdata=vVQiKgjWWEI%2FhBGQdcwM1wPyBr6ww3s24s89b7qNQOw%3D&reserved=0.

Looks good to me

Thanks Vlad, Yuval!


Executing javascript from HTML input file through C# code

Hi All,
I'm very new to PhantomJS.
Our .Net Web API needs to read one input HTML file with some Javascript contents for Charts/SVG rendering on browser which needs to be converted to PDF further with the help of some third party conversion tool using their APIs.
But the conversion tool does not fully support JavaScript/Dynamic Contents rendering directly so We Are Trying to Use PhantomJS to make the output Supportable with that tool.
But facing issues in implementing PhantomJS for the same. Can anyone help in resolving the issue with Sample Code??
Scenario: We have one HTML file which contains some JavaScript code inside it/external source. We need to get the contents of the HTML file in form of string using C# code.
Note :We are open to buy the PhantomJS Licence if needed.
We are trying it in following way(using NReco.PhantomJS):
What I have tried:
`public string Conversion()
{
   string script ="var page = require('webpage').create();" +
    "var url = 'Local_File_Path/fileName.html';" +
    "page.open(url, function(status) {" +
     "if (status !== 'success'){" +
     "console.log('Unable to load the address!');" +
    "phantom.exit();" +
    "}else{"+
    "window.setTimeout(function() {"+
     "page.render(MyJavaScriptFuntion);"+
    "phantom.exit();"+
    "}, 1000);"+
    "console.log(status);" +
     "phantom.exit();"+
     ")};";

    string output = string.Empty;

    var phantomJS = new PhantomJS();
    phantomJS.OutputReceived += (sender, e) =>
      {
          output += e.Data;
      };
    phantomJS.RunScript(script, null);
        return output;

}`
We Also tried : page.Evaluate() method and page.setContent() Methods but it didn't work.
Please Help if Anyone have some idea.
It's very very Urgent.

PhantomJS is abandoned. The maintainer recommends you use Chrome Headless instead.
https://groups.google.com/forum/#!topic/phantomjs/9aI5d-LDuNE


Adding W1D2 notebooks

Will update with final youtube links.
This tutorial doesn't conform fully to style guidelines, because it's pretty different in nature than other tutorials and I wanted to respect some of the CC's choices. E.G. I left section sub-headings as TD (To Do) since this tutorial relies on clear calls to action that aren't exercises.

All videos, except for Video 9 in T2.

Were W0D1 files pushed to this branch accidentally? If so I can close and recreate a new PR once the last youtube video arrives?

Updated with remaining video.


Add Grafana LGTM stack

[x] Loki
[ ] Grafana
[ ] Tempo
[ ] Mimir
[ ] Tanka
[x] Agent
[x] k6
[ ] k6-operator

Migrated to https://github.com/uniget-org/tools/issues/57


Form Dialog

As a user
I want to get a notification when i submit a form
so that i know that my input is saved
##Description

AC

[ ] submitting a form opens a dialog

Tasks

[ ] Create dialog component
[ ] The dialog gets a form specific message
[ ] call open the dialog on the submit button

Difficulty
S
Previous stories
#3 #16

Hi @desplodur ✌
Nice feature, nice User Story. I've just a few minor comments.

markdown bug in your description headline

AC:

what (text) is displayed?
what is displayed when the submit has not worked?


Warframe integration

Is it possible to have Warframe integration into the bot, providing us with pretty embeds when world events happen, things like; Sorties, Server resets, Alerts and the rewards that come with it etc.
It would also be awesome if there was the possibility of having a per-person thing where people can use a command to track certain things of value to them and if said item comes up they will receive a PM.
Thanks, kind regards.

For future reference, suggest new features at __https://feathub.com/Kwoth/NadekoBot__
This isn't an issue so 😄
Sent from my Huawei ALE-L21 using FastHub


acs data needs update

the formulas are updated in #205 but i need to push a new version to the quilt bucket. Before that happens, i need to look further into https://github.com/cenpy-devs/cenpy/issues/95

i think a good path forward here would be to incorporate the data from the census planning database which has nationwide tabulations of tracts and blockgroups for most variables of interest. Our current tooling to process ACS data has trouble with some specific geographies from the census api that cause breakage and this would skirt that issue, but we need to double check for consistency in the formulas (currrently ours are defined by those used in LTDB)

our newer tooling collects geodatabases from the census FTP and processes those, so no need to interface with cenpy directly