Update taskruns.md doc

Changes
While reading the documentation, I found there is a broken link to an example around tekton bundles. This PR fixes it, by updating to the correct path of the tekton-bundles.yaml sample.
/kind documentation
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices
[ ] Release notes block has been filled in or deleted (only if no user facing changes)

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
For pull requests with a release note:
Fix wrong path in the taskruns.md document.

/ok-to-test

/ok-to-test

/approve
/lgtm

/approve
/lgtm

Closed/repopened to force EasyCLA check..

Closed/repopened to force EasyCLA check..


SSR error - Cannot find module './vue/server.mjs'

Reproduction
Ensure you run mix assets.build before and there's a file in priv/vue/server.mjs. Then, set ssr_module: LiveVue.SSR.NodeJS in your config and try visiting a page with server-side rendered component.
Explanation
It's related to the upstream package nodejs not supporting importing ESM modules. I've added support for it, waiting for maintainers to merge and release the new version.
You can track PR here:
https://github.com/revelrylabs/elixir-nodejs/pull/84
Workaround
Add in your mix.exs deps:
{:nodejs, github: "Valian/elixir-nodejs", branch: "master", override: true}

Update - there was some activity in the nodejs repo. My PR was rebased and ready to be merged. Hopefully soon ❤

It's done! Everything should be working smoothly with nodejs ~> 3.1 🎉

Could you please update dependency version? It still points to the ~> 2.0.

@vheathen 0.3.9 has been published with corrected dependency ❤

@Valian thanks a lot!


Override associations

ActiveModel serializers allow to override association. This is does not work with postgres_ext-serializers.
https://github.com/rails-api/active_model_serializers#overriding-association-methods

Could you give an example? The URL above doesn't seem to work and you might be talking about an AMS 0.9 or 0.10 feature, but currently only AMS 0.8 is supported.


fix #10 [Accessibility] Coral.Tree fix WAI-ARIA implementation to improve screen reader support

Description
Per: https://jira.corp.adobe.com/browse/CQ-4272964
Fix the Coral-Spectrum Tree implementation to better conform with WAI-ARIA Authoring Practice design pattern: https://www.w3.org/TR/wai-aria-practices-1.2/#TreeView
Related Issue
#10 and https://jira.corp.adobe.com/browse/CUI-7339
Motivation and Context
Ensure accessibility of Tree with desktop and mobile screen readers/
How Has This Been Tested?

Unit tests have been updated.
Manually tested using the following browser/screen reader combinations:

macOS 10.15.2/VoiceOver/Safari 13.0.4 (156<IP_ADDRESS>.3)
macOS 10.15.2/VoiceOver/Chrome 79.0.3945.88 (Official Build) (64-bit)
iOS 13.2.3/VoiceOver/Safari

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.

This PR changes the keyboard behavior to better conform with with WAI-ARIA Authoring Practice design pattern: https://www.w3.org/TR/wai-aria-practices-1.2/#TreeView, but the keyboard behavior is not currently documented in Coral Spectrum.

@icaraps


Add enchantment effects

To complete the encanting system, enchantment effects should be implemented.
daniel0916 started with it, but he didn't finish it, so maybe his branch could be useful.
https://github.com/daniel0916/MCServer/tree/Enchanting (see commits)

Could someone explain to me want Enchantment Effects are?
They don't appear to exist.
What do you mean?

Probably so that enchantments will actually do something.


[Feature Request] Add a "Contents" Section with Jump Links in Posts

Description
Adding a section near the top of an article with a list of headings in the article. Each one would be a jump link to said section.
This is done on some blogs out there, but you can also see it in sites like wikipedia.
Implementation
If you are willing to include this, I am willing to work on it and raise a Pull Request. I can maintain it in a separate fork, if you prefer not to include it in your repo.
We can utilize the markdown-it-anchor plugin and we can also make this a feature that is optionally enabled by configuration if you prefer.

Hello @nezartarbin Apologies for getting to this late, I've been busy and forgetful too.
I'd appreciate the Pull Request if you still have some free time


Fix incompatibility of the TranslatorPass compiler pass with Symfony 3.3+

Q
A

Branch?
master

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Tests pass?
yes

License
MIT

This PR fix issue #13. A quick recap of the problem: since Symfony 3.3 the TranslatorPass is not compatible anymore due to a new argument added to the constructor of the service class it acts on and an exception is thrown.

Coverage decreased (-0.1%) to 99.87% when pulling f8f79be9ee934c970223afa0e9d01cce37a0c6ec on ste93cry:fix/issue-13 into 960866bdf8a6198e7c553e10d392b289fe3483b5 on sonatra:master.

@ste93cry Thank you !


So much so that Bob Haugen added the conversation as a Github Issue:
https://github.com/valueflows/intent/issues/11#issuecomment-350081246
It is also here: https://groups.google.com/forum/?hl=en-gb#!topic/sensorica/ARX7gXex9x8
I invite any who want to to engage them in breaking the ice. I got an away message when trying to engage  Christian Huemer, so maybe bringing him on-board is a January thing.
-Brent

Moved to other channels.


Memory not showing in notebooks

Using systemdspawner in jupyterhub set limit to 1GB.
Install psutil, and nbresuse 0.3.4
memory limit not showing

Jun 11 08:32:08 jupyter bash: File "/usr/local/lib/python3.6/site-packages/nbresuse/prometheus.py", line 34, in call
Jun 11 08:32:08 jupyter bash: metrics = self.apply_memory_limits(memory_metrics())
Jun 11 08:32:08 jupyter bash: File "/usr/local/lib/python3.6/site-packages/nbresuse/prometheus.py", line 49, in apply_memory_limits
Jun 11 08:32:08 jupyter bash: metrics.max_memory = self.config.mem_limit
Jun 11 08:32:08 jupyter bash: AttributeError: can't set attribute

Please use nbresuse==0.3.3 and give it a try.

Working no problem with version 0.3.3


Bail out! Gsk:ERROR

Describe the bug
The app crashes with this error message
Gsk:ERROR:../gsk/gl/gskglrenderjob.c:3987:gsk_gl_render_job_visit_node_with_offscreen: code should not be reached
Bail out! Gsk:ERROR:../gsk/gl/gskglrenderjob.c:3987:gsk_gl_render_job_visit_node_with_offscreen: code should not be reached

To Reproduce
Steps to reproduce the behavior:

Open the app and authenticate
Click on 'Library',
Click on a library icon, I have two megascan library, "Megascan:Tropical" and "Megascan - Lava Field". it crashes with both. But it does not crash with the library "A Boy and His Kite", for example.

Expected behavior
I should be able to click on a library and download it
Desktop (please complete the following information):

OS: Pop!_OS 22.04 LTS
Browser Firefox 101.0
Epic Asset Manager 3.5.0

Let me know if there is something that I can do to help you debugging this issue, and thanks for developing this asset manager, it is really helpful on linux.

That is fine, as long as I can reproduce it I can try working on it to get rid of that crash. Will keep you posted here.

Just gonna put some notes here for me here. The above commit fixes the Lava Field but the Tropical assets still crashes. Plus I discovered one more that crashes but on something slightly different "Autumn Audio Bundle" will try to figure that one out as well

I am also having the same issue on pop os browsing assets even with the same asset it seems to work sometimes but not others , I can only browse a couple of assets before it crashes
arron@pop-os:~$ flatpak run --env=RUST_LOG=debug io.github.achetagames.epic_asset_manager
<epic_asset_manager> - [DEBUG] - /app/share/epic_asset_manager
<epic_asset_manager> - [DEBUG] -
<epic_asset_manager> - [DEBUG] - 3.8.2
<epic_asset_manager::application> - [INFO] - Epic Asset Manager (io.github.achetagames.epic_asset_manager)
<epic_asset_manager::application> - [INFO] - Version: 3.8.2 ()
<epic_asset_manager::application> - [INFO] - Datadir: /app/share/epic_asset_manager
<epic_asset_manager::application::imp> - [DEBUG] - GtkApplication::startup
<egs_api::api::types::download_manifest> - [ERROR] - No header magic
<egs_api::api::types::download_manifest> - [ERROR] - No header magic
<egs_api::api::types::download_manifest> - [ERROR] - No header magic
<egs_api::api::types::download_manifest> - [ERROR] - No header magic
<egs_api::api::types::download_manifest> - [ERROR] - No header magic
<egs_api::api::types::download_manifest> - [ERROR] - No header magic
**
Gsk:ERROR:../gsk/gl/gskglrenderjob.c:3933:gsk_gl_render_job_visit_node_with_offscreen: code should not be reached
Bail out! Gsk:ERROR:../gsk/gl/gskglrenderjob.c:3933:gsk_gl_render_job_visit_node_with_offscreen: code should not be reached
this is with some of the paragon character assets. I am on the latest pop-os 22.04 patch and firefox 107.0 as my browser, asset manager 3.8.2


[Feature]: Add option to change text color in the Editor

What problem does this feature solve?
This feature solves the problem of limited text customization in the Editor. Without the ability to change text color, users are unable to highlight or differentiate text based on importance, theme, or branding requirements. This limitation can hinder content creation, especially for users who need to emphasize certain parts of their text, match specific color schemes, or create visually appealing documents. By adding a text color option, the editor becomes more versatile and user-friendly, addressing these needs and enhancing the overall content creation experience.

Could I add a text color customization feature to the Editor? This would greatly enhance content creation by allowing users to emphasize important text and align with specific color schemes. My GitHub username is Aber72.

hi Aber72 , you can fork the repo and add the feature and create pr , will check and merge if it works fine , you can read code_of_conduct.md and Contributing.md file for guidelines


How to remove "name" tag or customize it dynamically?

Is an existing page relevant?
No response
What karpenter features are relevant?
labels and tags.
How should the docs be improved?
Rather than spin up ec2 instances with a static EC2 tag name of the provisioner, it would be really useful if the EC2 tag name matched the node's name, which is currently the private IP for us in EKS.  Is this possible?  It may require karpenter to update the name after the node is provisioned.
If it is not, is it possible to just not tag the node's name at all?  We currently use datadog that automatically imports EC2 tags into metric tags, but aren't getting much use out of name since they all have the same name tag.
Thanks!
Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

If it is not, is it possible to just not tag the node's name at all?

If Karpenter had to tag the instance with something, what would you prefer it to be? We have this proposal that is in the works and one of the thoughts is to tag the instances that would be launched with the machine name, which we could know ahead of time.

what would you prefer it to be

Whatever shows up when I type kubectl get nodes is my preference.  I don't particularly care what the node names are, but do prefer the kubectl and EC2 UI match.

Or resource id would be also great

Resource Based Naming https://github.com/aws/containers-roadmap/issues/1723#issuecomment-1217343633 causes node name and instance id to converge. EC2 name tag is another thing entirely. It's a bit redundant to have a name tag == instance id. The other challenge is that the instance id isn't known until the instance is created, so the tagging would have to happen later, creating reconciliation / atomicity challenges. Possible, just work to deal with.


Implemented am335x GPIO driver code.

Note: This hardware is actually compatible with TI OMAP4, so at some point they should be merged somehow. Something similar to Linux device trees could be a way to do this.

I forgot to mention I filled out the I2C header. The mux driver still needs to still be finished, but right now this works for a number of beaglebone pins without specifically configuring the mux. The GPIO devices also needed to added to the platform specific hardware and header files in the kernel.

Just realized the more up-to-date repository is in util_libs. Moving code to this repo.


can't install a bp from the portal however its installed well using ays commands

Description
I can't install the following blueprint using the install command in the repo

g8client__test:
g8.url: 'gig.demo.greenitglobe.com'
g8.login: 'gig'
g8.password: 'KrOe6gE9K5nCQdmretfXnj'
g8.account: 'gig'
vdc__helloislam4:
g8.client.name: 'test'
g8.location : 'du-conv-1'
maxMemoryCapacity: 2
maxVDiskCapacity: -1
maxCPUCapacity: -1
maxNASCapacity: 20
maxArchiveCapacity: 20
maxNetworkOptTransfer: 5
maxNetworkPeerTransfer: 15
maxNumPublicIP: 1

This error is producing :
by @rkhamis

ERROR: Cannot get ays service 'os!centrilogic', did not find ((type:input.error  cat:1)

However I can install from the docker machine using the ays install commands
Secnario
From the portal:

Add a new bp to the repo (bp to create a cloud space)
Execute this bp
Install this bp
Check the service state on the instance of the repo.

Cockpit portal
https://centrilogic.aydo2.com

This looks like another blueprint has not been properly deployed before in the same repo.
Cause in the blueprint pasted here I don't see any os service. So if AYS complains about the missing  os!centrilogic a bad manipulation has been done before.


Add an extra set of state tracking for Timeout.timeout edge cases

Thoughts? This isn't strictly necessary, as the chance of this happening is very very small. It is still a chance though.
Normally, Timeout.timeout raises an exception and kills the block and everything is handy dandy. In rare situations, it will raise and then kill execution while the rescue block executes (see https://github.com/mperham/connection_pool/pull/67 for more info).
This adds an extra layer of protection, where if we run into that, we reset the connection anyway.

Seems ugly but so does Timeout :stuck_out_tongue_winking_eye: In absence of a fully asynchronous timeout scheme this seems ok to me.

Yea it's not great unfortunately but such is life :(


New options: offsetTop & autohideDelay, and now use CSS Transform when posible

New options

offsetTop
offset top position for notification frame, default to 0 as before
autohideDelay
timeout for auto-hiding, default to 0 as always shown

New feature
Now use CSS Transform to manupilate frame animation when posible, if not, use Top instead
Other tweaks
All elements nodes will not be created nor appended into document until reference function called

No longer relevant. Thanks though!


DAVIS dataset can't download from Dropbox

could you sent me a training set 、val set and test set to my<EMAIL_ADDRESS>

The link failed to open,Can I get DAVIS dataset link? my email is<EMAIL_ADDRESS>

The links to the datasets were just updated in the README and you'll find the datasets there.
Regards


Make XING branch of typheous work with Ruby 2.2

Maybe this implementation is naive, but it builds cleanly, all the gem tests run and I'm able to run the tests of XING internal gems and one project that depend on it.

See #11


RFC: bring back resolve (or something similar)

I personally found the old resolve builtin (still documented, but gone in master) extremely useful to quickly see where a command is coming from as per the command resolution rules.
Before:
[~]─> resolve ls
▶ <closure 0xc420462a20>   # my ls alias

Now:
[~]─> put $builtin:ls
Compilation error: variable $builtin:ls not found
[tty], line 1: put $builtin:ls
[~]─> put $ls~
▶ <closure 0xc420462a20>
[~]─> search-external ls
▶ /Users/taazadi1/.nix-profile/bin/ls

Of course this is a contrived example, and it could easily be automated in a function, but since Elvish has to resolve the command anyway, I would vote for exposing this functionality again.

To be accurate, the new resolve also should resolve special commands like use. For that reason, it cannot write a callable value. It will write a string instead.

Returning a string is obviously closest to what other shells do with commands like type ls.  I can't think of any reason why it should return a callable. However, it might be better to return something more structured than a simple string. For example, from Fish shell:
$ type and                                                                                5.3  s
and is a builtin
$ type ls                                                                                4 ms
ls is a function with definition
# Defined in /usr/local/share/fish/functions/ls.fish @ line 34
function ls --description 'List contents of directory'
    command ls -G $argv
end
$ type od                                                                                36 ms
od is /usr/bin/od

That is okay when a human is interpreting the output but a structured return value would make the command much more useful to other functions. And likely without any loss of usefulness to an interactive user of the command.

@krader1961 That is interesting, but I have opted to not implement those. The reason is different for functions and externals:

Functions already have their own introspection methods (#560, which is already fixed!). There is one limitation (#617) which I wish to fix though.

In Elvish terminology, "resolve" means strictly static resolution. External commands undergo two processes: static resolution in the compilation phase when they merely gets tagged to be external, and dynamic searching in the evaluation phase when their path is found. The resolve commands only emualtes static resolution and does not do the searching: the search-external command is exactly for that.


Fix evm-private dependency using the public one (v1.1.0)

Fixes https://github.com/moonbeam-foundation/moonbeam/issues/2722

I didn't see you just fix it on eb24406e90776e803d5d13a9f7c206c3db39ce41
Thanks!


Power function in Algebra Section

A feature for finding x to the power y in the algebra section

I want to work on this under JWOC

@VATSAL-IIT This is too small calculator. If u want u can combine it with square root and cube root in the form of drop down menu. For drop down u can refer to circular motion calculator. And one more thing if u want to work then the definition for each should not be copied from other website, instead add somesome of ur own words.

I have seen the refernce and would add dropdown for square-root calculation and cube-root as well .


AST: Revert inherited type request in ConformanceLookupTable

One of the request triggers added to ConformanceLookupTable in https://github.com/apple/swift/pull/68384 can cause circular request evaluation. Revert the request trigger since it doesn't appear to be necessary for the test cases introduced in that PR.
Resolves rdar://115314044

Thank you to @hborla for the reduced test case!

@swift-ci please smoke test


Hide private key export when using a Ledger account

Reported by Duff Lite in our Discord server.
Trying to export a private key from a Ledger account shows "00000000..." instead of what would be the private key. Given Ledger is a hardware wallet, it completely isolates private keys by design, so showing the option to export something that can't be exported is technically a bug, and we can simply solve it by just hiding the option when using a Ledger account.

Had to re-position the other icons so there wasn't a gap when the priv key icon is hidden.
@itamarcps since you're the Ledger boy, can you check it's working for me later? Just wanna make sure, testing with flag overrides is not really the same as testing with the real thing :stuck_out_tongue:

Unfortunately it did not work right away, the "visible" part of the privKeyRect was not properly seeing changes on the isLedger bool flag, I have added a signal with commit 4050e6e solving this issue, but thanks for the base rectangle!


Listing the code string should be optional

But that requires re-structuring the dicts.

It is now.


Fix build (graphing)

Addendum to https://github.com/dimagi/commcare-odk/pull/150

@orangejenny thanks!


Disable restart manager in GCE deployment

https://github.com/containerd/containerd/pull/2318 is useful for other use cases, e.g. using containerd to supervise system daemons.
But k8s deployment on GCE doesn't use it. We should disable it to avoid surprise, although unlikely.

Can users reneable it?

Yeah, it is just a config file change. We just disable the plugin by default in cloud-init.


Fix double vkUnmap calls

Appears to be missed in f289815. The memory should be mapped and unmapped by VulkanDeviceMemoryImpl.
Related to #41.

Hi Ben, Can you please rebase to the ToT?


[Issue] - Health Checker

Exchange Health Checker version <IP_ADDRESS>9
Provide the version of the script that has the issue.
Operating System Information
    Version: Microsoft Windows Server 2019 Standard
    System Up Time: 0 day(s) 0 hour(s) 6 minute(s) 47 second(s)
    Time Zone: W. Europe Standard Time
    Dynamic Daylight Time Enabled: True
    .NET Framework: 4.8
    Page File Size: 8192MB
    Power Plan: High performance
    Http Proxy Setting:  --- Warning this can cause client connectivity issues.
    Http Proxy By Pass List:
    Visual C++ 2012: 184610406 Version is current
    Visual C++ 2013: 201367256 Version is current
    Server Pending Reboot: False

I updated 3 Exchange 2019 Servers to CU11 (own test environment from a bare CU10 installation)
Post install i ran on all three servers the Healtch Checker and observed on all 3 a http proxy warning.
All three installations get the Http proxy warning, and bypass list but it looks like it misses any information about what warnings?
Unfortunately I did not run the healthchecker before the update.
Expected behavior
I think the Http Proxy settings warning should contain the information if found. Now it looks like the warning triggered but no setting was found?
If the healtchecker queries the following?
C:\temp>netsh winhttp show proxy
Current WinHTTP proxy settings:
Direct access (no proxy server).
Found in earlier Healtcheck log (v<IP_ADDRESS>8)
Http Proxy Setting:
Best regards,
Arian!

@grimson73 can you run the Health Checker again with -SaveDebugLog and email the debug log and the xml file to<EMAIL_ADDRESS>I think there is an issue with trying to run the cmdlet within a script block that is causing the problem.
When running on my lab computer that doesn't have the proxy set, it works as intended and not showing an empty HTTP Proxy settings.

Ok I tried multiple versions:
HealthChecker-<IP_ADDRESS>9.ps1
Http Proxy Setting:
HealthChecker-<IP_ADDRESS>2.ps1
Http Proxy Setting:
HealthChecker-<IP_ADDRESS>9.ps1
Http Proxy Setting:  --- Warning this can cause client connectivity issues.
Http Proxy By Pass List:

@grimson73 can you run the Health Checker again with -SaveDebugLog and email the debug log and the xml file to<EMAIL_ADDRESS>I think there is an issue with trying to run the cmdlet within a script block that is causing the problem.
When running on my lab computer that doesn't have the proxy set, it works as intended and not showing an empty HTTP Proxy settings.

I will send the team / you the requested debuglogfiles, thank you!

Ok I tried multiple versions:
HealthChecker-<IP_ADDRESS>9.ps1
        Http Proxy Setting: <None>

HealthChecker-<IP_ADDRESS>2.ps1
        Http Proxy Setting: <None>

 HealthChecker-<IP_ADDRESS>9.ps1
        Http Proxy Setting:  --- Warning this can cause client connectivity issues.
        Http Proxy By Pass List:

Okay, so the previous versions with <None> is how it should show up. But i am curious as to why it was working in a pervious version of the script.
Please provide the debug information from <IP_ADDRESS>2 HC version as well.

Ok I tried multiple versions:
HealthChecker-<IP_ADDRESS>9.ps1
        Http Proxy Setting: <None>

HealthChecker-<IP_ADDRESS>2.ps1
        Http Proxy Setting: <None>

 HealthChecker-<IP_ADDRESS>9.ps1
        Http Proxy Setting:  --- Warning this can cause client connectivity issues.
        Http Proxy By Pass List:

Okay, so the previous versions with <None> is how it should show up. But i am curious as to why it was working in a pervious version of the script.
Please provide the debug information from <IP_ADDRESS>2 HC version as well.

I send the logfiles to the mentioned e-mail address. Also hope this e-mail address accepts a zip file (contains TXT files).
Otherwise wil send all TXT / logfiles as seperate attachments.


Ordered lists not rendered correctly

Cannot render an Ordered list with line breaks in between, or code snippets.
This:
1. One

2. Two

3. Three

Outputs this
1.
One

2.
Two

3.
Three

My goal is to have some multi-line code inside each list item.
If I try to use this:
1. One

/```
testing
some
code
/```

2. Two

3. Three

(added / on purpose so the code ticks are displayed)
I get

Which contains wrong numeration.

That looks like Markdown rendering as expected -- try indenting each line of the embedded snippet 4 spaces. Should keep the number consistent.


Auto install babel presets

Hi, and tks for this great tool,
Since the lib is already using babel why not enabling parsing the .babelrc and auto installing the presets and plugins ?
Thanks

I agree, it will be better. Now it is possible to use babel presets in run-js by manually installing them via npm install. Anyway, great tool!


FileHeader "immediate-origin" should start at position 13 to include full 10 digits

I noticed the "immediate origin" field in the file header starts at position 14 instead it should start at position 13 in order to fit complete 10 digits. Currently, immediate-origin gets truncated to first 9 digits.
FileHeader.java
**@ACHField(start = 14, length = 10, name = IMMEDIATE_ORIGIN, inclusion = MANDATORY)**     public String getImmediateOrigin() {         return immediateOrigin;     }

@pc-btegginamath
fixed in <IP_ADDRESS>


Make TimerTrigger example importable in other projects

Description
The spin-timer example shows how to create a simple trigger for spin applications. The example is a binary crate that can be used by spin up to run a locked application. The goal of this PR is to make the TimerTrigger importable in other projects, so that this trigger can be used not only as a standalone binary, but also when embedding spin. So that plugin developers can use this crate as a blueprint.
This PR has two commits:

The first commit moves everything but the main function to a lib.rs
2. The second commit replaces the manual creation of the plugin manifest with pluginify.

I'd love to hear your feedback 😊

cc @karthik2804 and @itowlson
I know @karthik2804 has been working on a dedicated cron trigger — https://github.com/fermyon/spin-trigger-cron, which we want to support.
(this is compared with the timer trigger from the Spin repo which has historically been an example of how to create one and embed Spin).

Awesome, I wasn't aware of the spin-trigger-cron repo! 😅

@radu-matei For sure.  The intent with the timer trigger, as you note, is educational, to provide a model for trigger devs to follow.  What @0xE282B0 does here is absolutely what we want trigger devs to do, and should be included.

I'm a bit concerned about using the pluginify plugin in the context of this example - or specifically about it being the only way to run it. pluginify is kind of magical; and as this is an example, and its job is to build understanding, it would be good to also show the manual way. But that's not a blocker and others may have different points of view!

@itowlson, maybe I was so excited about how easy pluginify is that I forgot the value of learning the manual way. How about a short setup guide in the readme?

@0xE282B0 sounds good!


Clean up HMC diagnostics, update corresponding tests

Summary:
Addresses issue #1790.
Intended Effect:
Rename divergent iterations and add the value of the Hamiltonian to identify poorly chosen kinetic energies.
How to Verify:
Run unit tests.
Side Effects:
Changes sampler parameter output.  This will break the automated tests that will try to push this into CmdStan -- a super dev friendly problem that @syclik will have to fix by hand.
Documentation:
None -- the Stan manual does not discuss specific HMC diagnostics.
Reviewer Suggestions:
Anyone.

For consistency, should we rename n_divergent in the diagnostics given by
RStan (and ShinyStan) when you add this? I like the name change but that
would break any user scripts that access the sampler parameter diagnostics
by name. Or should we wait until making other changes that break things?

Either support both and deprecate the old one with a warning,
or wait to make breaking changes for a major version update.
Semantic versionining:  MAJOR.MINOR.PATCH
backward compatibility breaking:  MAJOR++
new features without backward compatibility breaks:  MINOR++
bug fixes without new features: PATCH++

Bob

On Mar 8, 2016, at 2:27 PM, Jonah Gabry<EMAIL_ADDRESS>wrote:
For consistency, should we rename n_divergent in the diagnostics given by
RStan (and ShinyStan) when you add this? I like the name change but that
would break any user scripts that access the sampler parameter diagnostics
by name. Or should we wait until making other changes that break things?

—
Reply to this email directly or view it on GitHub.

Right, and that's what we would do if we were talking about the name of a function argument or something like that, but for this kind of change I don't think we can allow both simultaneously with one deprecated. Here's a quick example illustrating that:
If a user has an R script containing something like the following
fit <- stan(...)
sampler_pars <- get_sampler_params(fit)
sampler_pars_chain1 <- sampler_pars[[1]]

then sampler_pars_chain1  will be a matrix with column names corresponding to the names of the various sampler parameters ("accept_stat__", "n_divergent__", etc).  So if we change the name from n_divergent__ to divergent__ and a user has a script that uses column names rather than column numbers to access the columns of the matrix then they would have to use divergent__ and not n_divergent__ and there's no way for us to allow both. Once they have the matrix then they need to play by R's rules.
So this seems like something we should wait to change, at least for anything user-facing.

There are a lot of variables that have been mentioned as being good candidates for renaming.
For example, in "Recommendations and style guidelines" in https://github.com/stan-dev/stan/wiki/User-Interface-Guidelines-for-Developers
I mention "consistent variable names (either num_warmup and num_iter or use warmup and iter, not a mix)"
It would be nice to track them some place centrally so we can do the renaming all at once. While RStan/PyStan/CmdStan may differ I don't think they should differ on variable names.

Or you can introduce a new function name that returns
the new thing while still supporting the old function
that returns the old thing.
And while I have your attention about renaming, I'd recommend
removing "get_" prefixes;  "set_" is OK, but "get_" doesn't
really add any info, just four more characters.

Bob

On Mar 8, 2016, at 3:28 PM, Jonah Gabry<EMAIL_ADDRESS>wrote:
Right, and that's what we would do if we were talking about the name of a function argument or something like that, but for this kind of change I don't think we can allow both simultaneously with one deprecated. Here's a quick example illustrating that:
If a user has an R script containing something like the following
fit <- stan(...)
sampler_pars <- get_sampler_params(fit)
sampler_pars_chain1 <- sampler_pars[[1]]
then sampler_pars_chain1 will be a matrix with column names corresponding to the names of the various sampler parameters ("accept_stat__", "n_divergent__", etc). So if we change the name from n_divergent__ to divergent__ and a user has a script that uses column names rather than column numbers to access the columns of the matrix then they would have to use divergent__ and not n_divergent__ and there's no way for us to allow both. Once they have the matrix then they need to play by R's rules.
So this seems like something we should wait to change, at least for anything user-facing.
—
Reply to this email directly or view it on GitHub.

On Tuesday, March 8, 2016, Bob Carpenter<EMAIL_ADDRESS>wrote:

Or you can introduce a new function name that returns
the new thing while still supporting the old function
that returns the old thing.

Ok yeah, I wasn't thinking about adding a new function but that's probably
the best option.

And while I have your attention about renaming, I'd recommend
removing "get_" prefixes; "set_" is OK, but "get_" doesn't
really add any info, just four more characters.

Bob

Fine by me. I'll add an issue at the RStan repo.

Looks like the unit tests are failing with a compilation error -- both unit and performance tests pass locally on my machine.

Jenkins, retest this please.

It looks like some io stream (not related to the tests) died on Jenkins. I
just restarted it.
On Wed, Mar 9, 2016 at 9:47 AM, Michael Betancourt<EMAIL_ADDRESS>
wrote:

Looks like the unit tests are failing with a compilation error -- both
unit and performance tests pass locally on my machine.
—
Reply to this email directly or view it on GitHub
https://github.com/stan-dev/stan/pull/1791#issuecomment-194326876.

Now it's just the performance test which I changed to get running locally on my machine.  Why is it yielding different results on Jenkins?  This test is a mess.

Yes, the test is a mess. Thanks for reminding me. I'll fix it with the services refactor. Having a direct route to call the algorithm will clean up the test.

"ld: not enough disk space for writing" Jenkins out of space?

Jenkins, retest this please.

Ready to merge.

@betanalpha, let me know if we can use energy__ or something a little more descriptive than E__.
@jgabry,

the current version of ShinyStan will not work with this change if it gets into RStan. Is this correct?
is there a fix you can apply to have ShinyStan handle both existing behavior and this change?
is there a way to tell users that upgrade rstan to also upgrade ShinyStan?
what happens if this change hits the current ShinyStan?

@betanalpha: thanks for changing it. I guess you did something I didn't expect, though -- I wasn't suggesting changing the code... just the output name so it's friendlier on the users. I trust you're ok with the addition of the member variable?
It looks like github.com went down when the Windows test kicked off. I'll start the tests again.

Jenkins, retest this please.

This was going to be done in the next PR anyways, I just
took advantage of having to change things.
On Mar 14, 2016, at 3:35 PM, Daniel Lee<EMAIL_ADDRESS>wrote:

@betanalpha: thanks for changing it. I guess you did something I didn't expect, though -- I wasn't suggesting changing the code... just the output name so it's friendlier on the users. I trust you're ok with the addition of the member variable?
It looks like github.com went down when the Windows test kicked off. I'll start the tests again.
—
Reply to this email directly or view it on GitHub.

Sweet.
On Mon, Mar 14, 2016 at 11:48 AM, Michael Betancourt <
<EMAIL_ADDRESS>wrote:

This was going to be done in the next PR anyways, I just
took advantage of having to change things.
On Mar 14, 2016, at 3:35 PM, Daniel Lee<EMAIL_ADDRESS>wrote:

@betanalpha: thanks for changing it. I guess you did something I didn't
expect, though -- I wasn't suggesting changing the code... just the output
name so it's friendlier on the users. I trust you're ok with the addition
of the member variable?
It looks like github.com went down when the Windows test kicked off.
I'll start the tests again.

—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/stan-dev/stan/pull/1791#issuecomment-196377066.

On Sun, Mar 13, 2016 at 7:59 PM, Daniel Lee<EMAIL_ADDRESS>wrote:

@jgabry https://github.com/jgabry,

the current version of ShinyStan will not work with this change if
it gets into RStan. Is this correct?

Correct, but once we get the change into a branch on the RStan repo it
should be easy for me to make the necessary tweaks to ShinyStan. @bgoodri
and/or @maverickg would know best where any changes to RStan need to be
made.

is there a fix you can apply to have ShinyStan handle both existing
behavior and this change?
is there a way to tell users that upgrade rstan to also upgrade
ShinyStan?

These are related and it depends on how RStan implements the change. For
example, Bob suggested that we add a new function to RStan that has the
same behavior as get_sampler_params but uses the new names instead of
changing get_sampler_params (which would break user's scripts). If that's
what RStan ends up doing then ShinyStan could look for the new function and
use that if the user has it, and if not it could use the old function.

As for the names of the sampler_params, I'm fine with whatever Michael
wants.

Thanks! Merge will happen when tests pass!

On Mar 14, 2016, at 1:13 PM, Jonah Gabry<EMAIL_ADDRESS>wrote:
On Sun, Mar 13, 2016 at 7:59 PM, Daniel Lee<EMAIL_ADDRESS>wrote:

@jgabry https://github.com/jgabry,

the current version of ShinyStan will not work with this change if
it gets into RStan. Is this correct?

Correct, but once we get the change into a branch on the RStan repo it
should be easy for me to make the necessary tweaks to ShinyStan. @bgoodri
and/or @maverickg would know best where any changes to RStan need to be
made.

is there a fix you can apply to have ShinyStan handle both existing
behavior and this change?
is there a way to tell users that upgrade rstan to also upgrade
ShinyStan?

These are related and it depends on how RStan implements the change. For
example, Bob suggested that we add a new function to RStan that has the
same behavior as get_sampler_params but uses the new names instead of
changing get_sampler_params (which would break user's scripts). If that's
what RStan ends up doing then ShinyStan could look for the new function and
use that if the user has it, and if not it could use the old function.

As for the names of the sampler_params, I'm fine with whatever Michael
wants.
—
Reply to this email directly or view it on GitHub.


Upgrade Ruby 2.2.3 -> 2.3.1

Fixes: #75
The tests continue to pass with this version of Ruby. I don't think it requires any other changes.

@floehopper thank you for checking that in such detail, much appreciated. I've got a PR incoming for a mongoid upgrade, agreed we should press on with upgrading others.
release_100 contains the ruby upgrade in this PR - release_99 is on production so we can deploy the ruby change on its own tomorrow.


Fix broken DNS CNAME chain for cross-cluster service discovery

Partly addresses the issue in https://github.com/kubernetes/federation/issues/175
CNAME records were missed for the federated cluster without healthy service shards. This pr fixes the missing CNAME records issue.
/cc @kubernetes/sig-multicluster-bugs
/assign @irfanurrehman

/test pull-federation-verify

/test pull-federation-e2e-gce

/lgtm

/test pull-federation-e2e-gce

/test pull-federation-e2e-gce


docker build -t dry-run-test . fails because s3cmd 1.5+ not found

I'm using Ubuntu 15.10 x64 to set up the development environment of docker, following the official guidance. When executing docker build -t dry-run-test ., it reports error. I posted the whole log here:
root@ubuntu:/repos/docker-fork# docker build -t dry-run-test .
Sending build context to Docker daemon 120.8 MB
Step 1 : FROM debian:jessie
 ---> f50f9524513f
Step 2 : RUN echo "deb http://mirrors.aliyun.com/ubuntu/ trusty main multiverse restricted universe" | tee /etc/apt/sources.list
 ---> Running in 2042d7b1b949
deb http://mirrors.aliyun.com/ubuntu/ trusty main multiverse restricted universe
 ---> 614ebe47173e
Removing intermediate container 2042d7b1b949
Step 3 : RUN echo "deb http://mirrors.aliyun.com/ubuntu/ trusty-backports main multiverse restricted universe" | tee -a /etc/apt/sources.list
 ---> Running in c3ba1b9037b5
deb http://mirrors.aliyun.com/ubuntu/ trusty-backports main multiverse restricted universe
 ---> 6709f9dba635
Removing intermediate container c3ba1b9037b5
Step 4 : RUN echo "deb http://mirrors.aliyun.com/ubuntu/ trusty-proposed main multiverse restricted universe" | tee -a /etc/apt/sources.list
 ---> Running in ab4efde455de
deb http://mirrors.aliyun.com/ubuntu/ trusty-proposed main multiverse restricted universe
 ---> 2712bbdc4c7e
Removing intermediate container ab4efde455de
Step 5 : RUN echo "deb http://mirrors.aliyun.com/ubuntu/ trusty-security main multiverse restricted universe" | tee -a /etc/apt/sources.list
 ---> Running in d890ced18014
deb http://mirrors.aliyun.com/ubuntu/ trusty-security main multiverse restricted universe
 ---> 5ec036bfc3cd
Removing intermediate container d890ced18014
Step 6 : RUN echo "deb http://mirrors.aliyun.com/ubuntu/ trusty-updates main multiverse restricted universe" | tee -a /etc/apt/sources.list
 ---> Running in cc1094c1b7bb
deb http://mirrors.aliyun.com/ubuntu/ trusty-updates main multiverse restricted universe
 ---> 8992104f388a
Removing intermediate container cc1094c1b7bb
Step 7 : RUN echo "deb-src http://mirrors.aliyun.com/ubuntu/ trusty main multiverse restricted universe" | tee -a /etc/apt/sources.list
 ---> Running in e694145732a8
deb-src http://mirrors.aliyun.com/ubuntu/ trusty main multiverse restricted universe
 ---> 7f59a76d1e03
Removing intermediate container e694145732a8
Step 8 : RUN echo "deb-src http://mirrors.aliyun.com/ubuntu/ trusty-backports main multiverse restricted universe" | tee -a /etc/apt/sources.list
 ---> Running in 49b75765da73
deb-src http://mirrors.aliyun.com/ubuntu/ trusty-backports main multiverse restricted universe
 ---> 41600528d1f3
Removing intermediate container 49b75765da73
Step 9 : RUN echo "deb-src http://mirrors.aliyun.com/ubuntu/ trusty-proposed main multiverse restricted universe" | tee -a /etc/apt/sources.list
 ---> Running in a4fac7edb44f
deb-src http://mirrors.aliyun.com/ubuntu/ trusty-proposed main multiverse restricted universe
 ---> 474e99bb1134
Removing intermediate container a4fac7edb44f
Step 10 : RUN echo "deb-src http://mirrors.aliyun.com/ubuntu/ trusty-security main multiverse restricted universe" | tee -a /etc/apt/sources.list
 ---> Running in c44a9fabe7ba
deb-src http://mirrors.aliyun.com/ubuntu/ trusty-security main multiverse restricted universe
 ---> 207f1ad3584a
Removing intermediate container c44a9fabe7ba
Step 11 : RUN echo "deb-src http://mirrors.aliyun.com/ubuntu/ trusty-updates main multiverse restricted universe" | tee -a /etc/apt/sources.list
 ---> Running in d1e793c29531
deb-src http://mirrors.aliyun.com/ubuntu/ trusty-updates main multiverse restricted universe
 ---> 56880873c7b2
Removing intermediate container d1e793c29531
Step 12 : RUN apt-key adv --keyserver keyserver.ubuntu.com --recv-keys 40976EAF437D05B5
 ---> Running in cb665759aea5
Executing: gpg --ignore-time-conflict --no-options --no-default-keyring --homedir /tmp/tmp.JUS793R6r0 --no-auto-check-trustdb --trust-model always --primary-keyring /etc/apt/trusted.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-jessie-automatic.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-jessie-security-automatic.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-jessie-stable.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-squeeze-automatic.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-squeeze-stable.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-wheezy-automatic.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-wheezy-stable.gpg --keyserver keyserver.ubuntu.com --recv-keys 40976EAF437D05B5
gpg: requesting key 437D05B5 from hkp server keyserver.ubuntu.com
gpg: key 437D05B5: public key "Ubuntu Archive Automatic Signing Key<EMAIL_ADDRESS>imported
gpg: Total number processed: 1
gpg:               imported: 1
 ---> e833ba43cd7b
Removing intermediate container cb665759aea5
Step 13 : RUN apt-key adv --keyserver keyserver.ubuntu.com --recv-keys 3B4FE6ACC0B21F32
 ---> Running in d085436bcf53
Executing: gpg --ignore-time-conflict --no-options --no-default-keyring --homedir /tmp/tmp.OoVwwqJPav --no-auto-check-trustdb --trust-model always --keyring /etc/apt/trusted.gpg --primary-keyring /etc/apt/trusted.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-jessie-automatic.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-jessie-security-automatic.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-jessie-stable.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-squeeze-automatic.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-squeeze-stable.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-wheezy-automatic.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-wheezy-stable.gpg --keyserver keyserver.ubuntu.com --recv-keys 3B4FE6ACC0B21F32
gpg: requesting key C0B21F32 from hkp server keyserver.ubuntu.com
gpg: key C0B21F32: public key "Ubuntu Archive Automatic Signing Key (2012<EMAIL_ADDRESS>imported
gpg: Total number processed: 1
gpg:               imported: 1  (RSA: 1)
 ---> 3d0fbf2a751c
Removing intermediate container d085436bcf53
Step 14 : RUN apt-key adv --keyserver hkp://p80.pool.sks-keyservers.net:80 --recv-keys E871F18B51E0147C77796AC81196BA81F6B0FC61 	|| apt-key adv --keyserver hkp://pgp.mit.edu:80 --recv-keys E871F18B51E0147C77796AC81196BA81F6B0FC61
 ---> Running in 3f165b692c38
Executing: gpg --ignore-time-conflict --no-options --no-default-keyring --homedir /tmp/tmp.z654HBmqm2 --no-auto-check-trustdb --trust-model always --keyring /etc/apt/trusted.gpg --primary-keyring /etc/apt/trusted.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-jessie-automatic.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-jessie-security-automatic.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-jessie-stable.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-squeeze-automatic.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-squeeze-stable.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-wheezy-automatic.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-wheezy-stable.gpg --keyserver hkp://p80.pool.sks-keyservers.net:80 --recv-keys E871F18B51E0147C77796AC81196BA81F6B0FC61
gpg: requesting key F6B0FC61 from hkp server p80.pool.sks-keyservers.net
gpg: key F6B0FC61: public key "Launchpad PPA for Native ZFS for Linux" imported
gpg: Total number processed: 1
gpg:               imported: 1  (RSA: 1)
 ---> d1de46833088
Removing intermediate container 3f165b692c38
Step 15 : RUN echo deb http://ppa.launchpad.net/zfs-native/stable/ubuntu trusty main > /etc/apt/sources.list.d/zfs.list
 ---> Running in 01459ad931dc
 ---> d6cac058630c
Removing intermediate container 01459ad931dc
Step 16 : RUN apt-key adv --keyserver hkp://p80.pool.sks-keyservers.net:80 --recv-keys 6084F3CF814B57C1CF12EFD515CF4D18AF4F7421 	|| apt-key adv --keyserver hkp://pgp.mit.edu:80 --recv-keys 6084F3CF814B57C1CF12EFD515CF4D18AF4F7421
 ---> Running in f4fe8d461c84
Executing: gpg --ignore-time-conflict --no-options --no-default-keyring --homedir /tmp/tmp.gcHH7pNSyl --no-auto-check-trustdb --trust-model always --keyring /etc/apt/trusted.gpg --primary-keyring /etc/apt/trusted.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-jessie-automatic.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-jessie-security-automatic.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-jessie-stable.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-squeeze-automatic.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-squeeze-stable.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-wheezy-automatic.gpg --keyring /etc/apt/trusted.gpg.d/debian-archive-wheezy-stable.gpg --keyserver hkp://p80.pool.sks-keyservers.net:80 --recv-keys 6084F3CF814B57C1CF12EFD515CF4D18AF4F7421
gpg: requesting key AF4F7421 from hkp server p80.pool.sks-keyservers.net
gpg: key AF4F7421: public key "Sylvestre Ledru - Debian LLVM packages<EMAIL_ADDRESS>imported
gpg: Total number processed: 1
gpg:               imported: 1  (RSA: 1)
 ---> d8b77b9caf31
Removing intermediate container f4fe8d461c84
Step 17 : RUN echo deb http://llvm.org/apt/trusty/ llvm-toolchain-trusty main > /etc/apt/sources.list.d/llvm.list
 ---> Running in 8f9b76f48740
 ---> 7eb1819c8b77
Removing intermediate container 8f9b76f48740
Step 18 : RUN apt-get update && apt-get install -y 	apparmor 	aufs-tools 	automake 	bash-completion 	btrfs-tools 	build-essential 	clang-3.8 	createrepo 	curl 	dpkg-sig 	gcc-mingw-w64 	git 	iptables 	jq 	libapparmor-dev 	libcap-dev 	libltdl-dev 	libsqlite3-dev 	libsystemd-journal-dev 	libtool 	mercurial 	net-tools 	pkg-config 	python-dev 	python-mock 	python-pip 	python-websocket 	s3cmd=1.5.0* 	ubuntu-zfs 	xfsprogs 	libzfs-dev 	tar 	--no-install-recommends 	&& ln -snf /usr/bin/clang-3.8 /usr/local/bin/clang 	&& ln -snf /usr/bin/clang++-3.8 /usr/local/bin/clang++
 ---> Running in ab8090472365
Ign http://mirrors.aliyun.com trusty InRelease
Get:1 http://mirrors.aliyun.com trusty-backports InRelease [65.9 kB]
Get:2 http://mirrors.aliyun.com trusty-proposed InRelease [65.9 kB]
Get:3 http://mirrors.aliyun.com trusty-security InRelease [65.9 kB]
Get:4 http://mirrors.aliyun.com trusty-updates InRelease [65.9 kB]
Get:5 http://llvm.org llvm-toolchain-trusty InRelease [4215 B]
Get:6 http://ppa.launchpad.net trusty InRelease [15.5 kB]
Get:7 http://mirrors.aliyun.com trusty Release.gpg [933 B]
Get:8 http://mirrors.aliyun.com trusty Release [58.5 kB]
Get:9 http://mirrors.aliyun.com trusty-backports/main Sources [9222 B]
Get:10 http://mirrors.aliyun.com trusty-backports/multiverse Sources [1751 B]
Get:11 http://mirrors.aliyun.com trusty-backports/restricted Sources [40 B]
Get:12 http://mirrors.aliyun.com trusty-backports/universe Sources [38.6 kB]
Get:13 http://mirrors.aliyun.com trusty-backports/main amd64 Packages [10.7 kB]
Get:14 http://mirrors.aliyun.com trusty-backports/multiverse amd64 Packages [1396 B]
Get:15 http://mirrors.aliyun.com trusty-backports/restricted amd64 Packages [40 B]
Get:16 http://mirrors.aliyun.com trusty-backports/universe amd64 Packages [48.6 kB]
Get:17 http://mirrors.aliyun.com trusty-proposed/main Sources [154 kB]
Get:18 http://llvm.org llvm-toolchain-trusty/main amd64 Packages [6196 B]
Get:19 http://mirrors.aliyun.com trusty-proposed/multiverse Sources [767 B]
Get:20 http://mirrors.aliyun.com trusty-proposed/restricted Sources [40 B]
Get:21 http://mirrors.aliyun.com trusty-proposed/universe Sources [22.9 kB]
Get:22 http://mirrors.aliyun.com trusty-proposed/main amd64 Packages [156 kB]
Get:23 http://ppa.launchpad.net trusty/main amd64 Packages [5161 B]
Get:24 http://mirrors.aliyun.com trusty-proposed/multiverse amd64 Packages [40 B]
Get:25 http://mirrors.aliyun.com trusty-proposed/restricted amd64 Packages [40 B]
Get:26 http://mirrors.aliyun.com trusty-proposed/universe amd64 Packages [27.6 kB]
Get:27 http://mirrors.aliyun.com trusty-security/main Sources [134 kB]
Get:28 http://mirrors.aliyun.com trusty-security/multiverse Sources [2548 B]
Get:29 http://mirrors.aliyun.com trusty-security/restricted Sources [3920 B]
Get:30 http://mirrors.aliyun.com trusty-security/universe Sources [38.2 kB]
Get:31 http://mirrors.aliyun.com trusty-security/main amd64 Packages [533 kB]
Get:32 http://mirrors.aliyun.com trusty-security/multiverse amd64 Packages [4852 B]
Get:33 http://mirrors.aliyun.com trusty-security/restricted amd64 Packages [20.2 kB]
Get:34 http://mirrors.aliyun.com trusty-security/universe amd64 Packages [161 kB]
Get:35 http://mirrors.aliyun.com trusty-updates/main Sources [327 kB]
Get:36 http://mirrors.aliyun.com trusty-updates/multiverse Sources [5498 B]
Get:37 http://mirrors.aliyun.com trusty-updates/restricted Sources [5217 B]
Get:38 http://mirrors.aliyun.com trusty-updates/universe Sources [189 kB]
Get:39 http://mirrors.aliyun.com trusty-updates/main amd64 Packages [901 kB]
Get:40 http://mirrors.aliyun.com trusty-updates/multiverse amd64 Packages [14.3 kB]
Get:41 http://mirrors.aliyun.com trusty-updates/restricted amd64 Packages [23.5 kB]
Get:42 http://mirrors.aliyun.com trusty-updates/universe amd64 Packages [439 kB]
Get:43 http://mirrors.aliyun.com trusty/restricted Sources [5335 B]
Get:44 http://mirrors.aliyun.com trusty/main Sources [1335 kB]
Get:45 http://mirrors.aliyun.com trusty/multiverse Sources [211 kB]
Get:46 http://mirrors.aliyun.com trusty/universe Sources [7926 kB]
Get:47 http://mirrors.aliyun.com trusty/multiverse amd64 Packages [169 kB]
Get:48 http://mirrors.aliyun.com trusty/main amd64 Packages [1743 kB]
Get:49 http://mirrors.aliyun.com trusty/restricted amd64 Packages [16.0 kB]
Get:50 http://mirrors.aliyun.com trusty/universe amd64 Packages [7589 kB]
Fetched 22.6 MB in 56s (400 kB/s)
Reading package lists...
Reading package lists...
Building dependency tree...
Reading state information...
E: Version '1.5.0*' for 's3cmd' was not found
The command '/bin/sh -c apt-get update && apt-get install -y 	apparmor 	aufs-tools 	automake 	bash-completion 	btrfs-tools 	build-essential 	clang-3.8 	createrepo 	curl 	dpkg-sig 	gcc-mingw-w64 	git iptables 	jq 	libapparmor-dev 	libcap-dev 	libltdl-dev 	libsqlite3-dev 	libsystemd-journal-dev 	libtool 	mercurial 	net-tools 	pkg-config 	python-dev 	python-mock 	python-pip 	python-websocket 	s3cmd=1.5.0* 	ubuntu-zfs 	xfsprogs 	libzfs-dev 	tar 	--no-install-recommends 	&& ln -snf /usr/bin/clang-3.8 /usr/local/bin/clang 	&& ln -snf /usr/bin/clang++-3.8 /usr/local/bin/clang++' returned a non-zero code: 100

In my opinion, the cause or error is quite simple. The docker container is a Ubuntu 14.04 (or at least equivalent, as I see a debian:jessie base image). I tested with my own 14.04 machine. It only installs s3cmd 1.1. See this:
root@ubuntu:/etc/apt# apt-show-versions | grep s3cmd
s3cmd:all/trusty 1.1.0~beta3-2 uptodate

But the dockerfile of the docker build-environment container says:
s3cmd=1.5.0* \

So the Dockerfile asks for a version of s3cmd that the Ubuntu 14.04 base image can't support. I don't know why this? Is this a bug of docker?

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

@hsluoyz Unfortunately, this isn't something we have control over. It seems to be related to the mirrors you're hitting when building from the main Dockerfile.
I'm very sorry, but this is something to be investigated with those who run that particular mirror and with the Debian project.
@jfrazelle Do you happen to have some suggestions?
I'll close this now since it's not a Docker bug and having the issue closed doesn't stop us from continuing the dialogue.

OK. I switched a Debian source and the issue seems to be solved. Many thanks!


Explorer looses selection

disable automatic reveal of files ("explorer.autoReveal": false,)
open src/vs/base/browser/ui/breadcrumbs/collpased-dark.svg
reveal the file manually and notice that it is positioned in the center
focus the editor
reload
the file is selected
reload again
:bug: the file is no more selected

I would expect the tree to preserve selection as part of the view state, forwarding to @joaomoreno

Repro steps in dev:

Select file
Refresh Explorer
Reload window

File is only focused, not selected


Text pasted into rich text programs does not specify a fallback font

Issue Type: Bug
When pasting text from the editor to another program, such as an email, the HTML generated to specifies only the editor font as the font-family of text elements, so if the user has specified a custom editor font that doesn't exist on most systems, recipients of the code will see it rendered in a default, non-monospaced font. In this case, pasting code from VS Code (with font set to Inconsolata) into an outlook email causes the code to be rendered in Times New Roman for recipients that don't have Inconsolata installed.
Current paste result:
...

Proposed paste result:
...

A good solution might be to always specify a fallback font corresponding to the class of the editor font: monospace, sans-serif, or serif.
VS Code version: Code 1.35.0 (553cfb2c2205db5f15f3ee8395bbd5cf066d357d, 2019-06-04T01:17:12.481Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz (8 x 1992)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.85GB (4.43GB free)

Process Argv

Screen Reader
no

VM
0%

Good suggestion, thanks!


Vulkan: Lingering Ambient Occlusion masks appear when enabling Camera3D preview in the editor

Godot version
v4.0.dev.custom_build.0c7cb9878
System information
Manjaro Gnome, AMD R7 200
Issue description
Even using a little "old" commit, I can confirm this happening on newer ones.

As you can see, its perfectly visible on the top (the free cam of the editor) the ambient occlusion (at least I think it is) from the Camera3D's preview from bellow.
Steps to reproduce

Create a new scene
Add world environment (with HDRI sky)
Add Camera3D
turn your layout into two views (side by side or any other)
in one of them use as camera3d's preview

Minimal reproduction project
FPS-Demo.zip

I've been able to reproduce this since the new SSAO was merged in December 2020. It mainly happens while in the editor, and can sometimes be triggered by resizing the viewport or window.

I think I know what is going on. The SSAO attempts to cache uniform sets between frames, but the uniform set is saved globally rather than per viewport. I'll fix this and make a PR soon.


Lifetime idents support

Is it possible to create lifetimes with paste?
For example:
macro_rules! impl_types {
    ( $($ty:ident),* ) => {
        paste::paste! {
            impl<$( '[< $ty >], $ty ),*> … { … }
        }
    }
}

It should work as long as you put the ' inside the [<.
impl<$([<'_ $ty>],)* $($ty,)*> ….

Ah okay, it also seems like I have to do [< '_ $ty >] instead of just [< ' $ty >]

[< ' $ty >] is not lexically valid Rust tokens so rustc wouldn't allow that in macro input.
If you don't want an underscore in the lifetime then you can use [<"'" $ty>].


[Feat]Adding multi language support system✨

Description
I want to add multi language support system to the website which can help us understand in their native language.
@sharmavikas4 can you assign me this issue.
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

You can proceed @T-Rahul-prabhu-38

@T-Rahul-prabhu-38 what is the progress

@sharmavikas4 If the multi-language feature is not yet implemented, I would like to take the opportunity to work on this issue. Please let me know if I can proceed or if there are any specific guidelines I should follow.


⚠ ClamAV has degraded performance

In a6bce93, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 66 ms

Resolved: ClamAV performance has improved in bbd0c89.


[Parse] Fix nested ifConfig compiler checks

The parser is supposed to avoid looking inside unmatched #if compiler (et al) blocks. This usually means that the following code builds fine
#if compiler(>=100)
foo bar
#endif

however, a logical bug meant that if the check was nested inside an already-inactive #if block, it would not adhere to this evaluation-skipping behavior
#if false
#if compiler(>=100)
foo bar // error!
#endif
#endif

This PR fixes this specific case.

I'm unsure if this fix needs to be ported to SwiftSyntax as well — I couldn't find the "skip parsing unmatched compiler checks" logic there. If it exists, could someone please point me to it?

The change looks good to me. Thanks!
@kabiroberai Could you squash the change and add relevant commit message? Or I if you write it here, I can squash and add the comment when merging.

@rintaro feel free to use the issue title: "[Parse] Fix nested ifConfig compiler checks". Thanks!

@swift-ci Please smoke test

OK, but I hope to put more explanation in the comment.
Let's just use the description:
[Parse] Fix nested ifConfig compiler checks

The parser is supposed to avoid looking inside unmatched `#if` compiler (et al) blocks.
This usually means that the following code builds fine

  #if compiler(>=100)
  foo bar
  #endif

however, a logical bug meant that if the check was nested inside an already-inactive
`#if` block, it would not adhere to this evaluation-skipping behavior

  #if false
  #if compiler(>=100)
  foo bar // error!
  #endif
  #endif

This PR fixes this specific case.


ShadingIT fails on macOS

Expected behavior
The following commands should succeed:
$ ./mvnw install -DskipTests
$ cd testsuite-shading
$ ../mvnw clean integration-test failsafe:verify

Actual behavior
The above command fails with the following error:
[ERROR] Tests run: 2, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 1.19 s <<< FAILURE! - in io.netty.testsuite.shading.ShadingIT
[ERROR] testShadingNativeTransport  Time elapsed: 0.374 s  <<< ERROR!
java.lang.UnsatisfiedLinkError: failed to load the required native library
	at io.netty.testsuite.shading.ShadingIT.testShading0(ShadingIT.java:54)
	at io.netty.testsuite.shading.ShadingIT.testShadingNativeTransport(ShadingIT.java:37)
Caused by: java.lang.NoClassDefFoundError: shaded2/io/netty/channel/kqueue/KQueueStaticallyReferencedJniMethods (wrong name: shaded2//io/netty/channel/kqueue/KQueueStaticallyReferencedJniMethods)
	at io.netty.testsuite.shading.ShadingIT.testShading0(ShadingIT.java:52)
	at io.netty.testsuite.shading.ShadingIT.testShadingNativeTransport(ShadingIT.java:37)

What's the current state of testsuite-shading? It is running as a part of the CI process?
Netty version
Branch 4.1 f901ea2e712d9d16e199aaeb7b622e36944eb8dd
JVM version (e.g. java -version)
❯ java -version
openjdk version "11.0.14" 2022-01-18 LTS
OpenJDK Runtime Environment Zulu11.54+23-CA (build 11.0.14+9-LTS)
OpenJDK 64-Bit Server VM Zulu11.54+23-CA (build 11.0.14+9-LTS, mixed mode)

OS version (e.g. uname -a)
❯ uname -a
Darwin ... 21.4.0 Darwin Kernel Version 21.4.0: Mon Feb 21 20:35:58 PST 2022; root:xnu-8020.101.4~2/RELEASE_ARM64_T6000 arm64

The test passes if I comment out ShadingIT.testShadingNativeTransport(), but why? Why does testShadingTcnative() pass? I can see the shaded native libraries are all there for my architecture (osx_aarch_64):
❯ find target/classes-shaded/ | grep META-INF/native/
target/classes-shaded/META-INF/native/libnetty_tcnative_linux_x86_64.so
target/classes-shaded/META-INF/native/netty_tcnative_windows_x86_64.dll
target/classes-shaded/META-INF/native/libshaded_netty_tcnative_osx_aarch_64.jnilib
target/classes-shaded/META-INF/native/libnetty_tcnative_linux_aarch_64.so
target/classes-shaded/META-INF/native/libshaded2_netty_transport_native_kqueue_aarch_64.jnilib
target/classes-shaded/META-INF/native/libshaded2_netty_tcnative_osx_aarch_64.jnilib
target/classes-shaded/META-INF/native/libshaded_netty_transport_native_kqueue_aarch_64.jnilib
target/classes-shaded/META-INF/native/libnetty_tcnative_osx_x86_64.jnilib

One major difference I found is that lib{shaded,shaded2}_netty_transport_native_kqueue_aarch_64.jnilib doesn't have _osx_ before aarch_64. Could this be why?

I ran the above build commands on my Ubuntu ARM64 VM and saw the same failure:
[ERROR] Tests run: 2, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.417 s <<< FAILURE! - in io.netty.testsuite.shading.ShadingIT
[ERROR] testShadingNativeTransport  Time elapsed: 0.171 s  <<< ERROR!
java.lang.UnsatisfiedLinkError: failed to load the required native library
	at io.netty.testsuite.shading.ShadingIT.testShading0(ShadingIT.java:54)
	at io.netty.testsuite.shading.ShadingIT.testShadingNativeTransport(ShadingIT.java:37)
Caused by: java.lang.NoClassDefFoundError: shaded2/io/netty/channel/epoll/NativeStaticallyReferencedJniMethods (wrong name: shaded2//io/netty/channel/epoll/NativeStaticallyReferencedJniMethods)
	at io.netty.testsuite.shading.ShadingIT.testShading0(ShadingIT.java:52)
	at io.netty.testsuite.shading.ShadingIT.testShadingNativeTransport(ShadingIT.java:37)

Let me change the subject of this issue because it seems to affect both macOS and Linux.

Will check

Thanks for looking, @normanmaurer. It looks like there's a bug in parsePackagePrefix() - it appends an extra / at the end of the package prefix. Let me dig a little bit more and get back.

OK. I have a fix. Let me clean it up and send out a PR tomorrow. I'll also implement the rule documented here, so it handles a peculiar package name correctly, such as one with _ or a non-ASCII character.


RFC for web-servers deps management

Proposes how web-servers dependencies should be management.
Check out this RFC for context.
This is the first stage in managing the following web-servers dependencies:

httpd
nginx
node-engine
yarn

I'm wondering if the parts of web-servers that are actually owned by the Node.js buildpack should be outlined here, or in an RFC decided by that maintainership group. What are everyone's thoughts on these types of shared dependencies?

Plus one too @ryanmoran I am not comfortable approving changes for node-engine or yarn as they are not directly under the Web Server teams control. That being said I am sure that the Nodejs maintainers will be happy to see that someone has already put in some leg work.

WG 2022-08-30 Notes: @TisVictress to update this RFC to remove yarn and node-engine. Most likely will be moved to another RFC specific for nginx deps

I've removed yarn and node-engine sections. This should be good to go now.


Parquet column/modular encryption support for Petastorm

Recently, parquet added support for columnar/modular encryption in version parquet-mr 1.12 (IBM, GitHub), meaning that only the footer file and certain columns of the parquet file are encrypted, minimising encrypt/decrypt overhead while keeping sensitive columns safe. Since Delta is built on top of parquet files and this encryption method became available in Spark 2.3.0 with improved support in Spark 3.0.0, I was wondering if Petastorm already supports this due to being built on the parquet format or if there is a way to make it work. For my application that would be the perfect combination considering current and probable future EU regulation around encryption and cloud storage!
df.write.option("parquet.encryption.footer.key", "k1").option("parquet.encryption.column.keys", "k2:DeviceName,image_id").parquet(STORE_PATH
This is how it works with spark dataframes in Databricks (which is what I am using).
An example of reading the data is as follows (source: IBM):
sc.hadoopConfiguration.set("parquet.encryption.key.list" ,
                   "k1:AAECAwQFBgcICQoLDA0ODw== ,  k2:AAECAAECAAECAAECAAECAA==")
sc.hadoopConfiguration.set("parquet.crypto.factory.class" ,
                   "org.apache.parquet.crypto.keytools.PropertiesDrivenCryptoFactory")
// Read encrypted dataframe files
val df2 = spark.read.parquet("/path/to/table.parquet.encrypted")

This works beautifully in spark dataframes, but for my use it would be great to use Petastorm as I am looking to train a Neural Network in Tensorflow on this data. Apache Arrow already has support for this at least in C++, and I've seen some discussion about implementing it in PyArrow though I can't see if it happened already.
Any help with getting column encryption to work with Petastorm would be highly appreciated, but learning that this is no (yet) supported and that there is no workaround would also be helpful since I can then look at different options. Thanks in advance!

Petastorm uses pyarrow to read parquet. Once pyarrow has the support for this feature, we can make sure petastorm allows you to use it.
The link to the pyarrow discussion you have provided is from 1.5 years ago. I wonder if anything happened since then.

It indeed looks like that, that we should pass some parameters to PyArrow which it then passes on to the parquet library. Implementing support for writing parquet files with encryption and still allowing the Petastorm metadata to be computed might be a bit more complex but as long as make_batch_reader() is able to read an encrypted parquet file I would already be happy. I'll keep an eye on the pull request and let you know when it's merged and released, and then I'll also see if I can add something though my knowledge of the inner workings of Petastorm is indeed limited.
   decryption_properties = crypto_factory.file_decryption_properties(
                                                    kms_connection_config)
   parquet_file = pq.ParquetFile(filename,
                                 decryption_properties=decryption_properties)

In order to create the encryption and decryption properties, a
:class:`~pyarrow.parquet.CryptoFactory` should be created and initialized with
KMS Client details, as described below.

    with pq.ParquetWriter(path,
                          table.schema,
                          encryption_properties=file_encryption_properties) \
            as writer:
        writer.write_table(table)

Sounds good.

The pull request has been committed into the master branch of pyarrow but has not been part of an official release yet. I asked the developers if they had a roadmap for this but so far I have not received a reply.
To me it looks like a fairly simple change for petastorm as I think we can get away with just passing the encryption_properties to the parquet writer and already call crypto_factory.file_encryption_properties inside own code or add crypto_factory.file_encryption_properties to the petastorm code. I'll see if I have some time tomorrow to work on a pull request but I'm curious to hear what you think! (Most helpful files to look at imo are test_encryption.py and sample_vault_kms_client.py )

I think this is a good plan to try. I think taking care of make_batch_reader is sufficient. I am not sure how much of our user base are actually the writer part of petastorm.

That makes sense, then that is the focus now and I hope to have enough time to make a PR for implementing this this week.


Duplicated fileds

There are duplicated files when created a model from DB. The last filed is duplicated.

Ok. I will have a deep look about whole migration code generation process. I have not give much focus there for a long time.


Flattening objects in the schema

Required Functionality
Creating a schema where result has many fields A1..AN but has two other fields, say B and C and only one can be present requires us to create a schema with a one_of at the root.
{
   "type":"one_of",
   "variants": [
    {
       "type":"object",
       "A1":  {...},
       ...
       "AN": {...},
        "B": {...}
    },
    {
       "type":"object",
       "A1":  {...},
       ...
       "AN": {...},
       "B": {...}
    },
  ]
}

This means a lot of repetition. All the A1..AN fields have to be spelt out again.
Proposed Solution
If we could add an attribute to an object which puts all its entries into the parent (like serde's struct flattening at https://serde.rs/attr-flatten.html) we would be able to write this as something like
{
   "type":"object",
   "A1":  {...},
   ...
   "AN": {...},
    "A_OR_B": {
        "type":"variant",
        "attributes":["flatten"],
        "variants":[
            { "type":"object", "A":{...} }
            { "type":"object", "A":{...} }
        ]
   }
}

with that "A_OR_B" never appearing in the output.

The example schema you shown should be something like this, right?
{
   "type":"one_of",
   "variants": [
    {
       "type":"object",
       "A1":  {...},
       ...
       "AN": {...},
        "B": {...}
    },
    {
       "type":"object",
       "A1":  {...},
       ...
       "AN": {...},
       "B": {...} // <==== C ?
    },
  ]
}

The example schema you shown should be something like this, right?

Absolutely.

The example schema you shown should be something like this, right?

I've fixed it in the description now.
There were a few issues in the proposed syntax too, which I've just corrected too. Again copy-paste errors due to me submitting this in a rush - sorry.


Outdated .Net Version

Module: 10
Lab: 11
Task: 2.2
Step: 01
Description of issue
The instructions refer to .Net 3. This version has not been supported by Azure for a long time and should be upgraded to .Net 6.
At the same time, Lab providers should no longer use Visual Studio 2019, but Visual Studio 2022, otherwise .Net 6 will not be supported.
The same applies to all other labs that rely on .Net 3 and Visual Studio 2019.

Also need to update from using MDAL to MSAL as MDAL is deprecated
This is the only lab that refers to .NET 3 and all the rest are .NET 6 (LTS), or still .NET Framework for plugins
The main issue is the use of Xrm.Tools.CrmWebAPI that hasn;t been updated in a long time. Need to replace the use of this helper.

Waiting for new VM with Windows 11 before can update .NET and Visual Studio

New VM is on the way, we're working with Skillable on this now.

Any update on the new VM?

Hi Everyone, The VM will be deployed to this course by early Feb. Thank you for your patience.


Can't install the extention (not found)

Why? Any hidden requirements for the subject?

Linux (Archlinux) 5.4.78-1 (x86_64)
VS code 1.51.1-1
electron9 9.3.3-1

Duplicate of #483
https://github.com/microsoft/pylance-release/issues/483#issuecomment-712376755


Fix to support arrays while evaluating DefaultExpression

Right now, intermediate response is expected to be List, Map, String or Object.

Fixed - 20e55bfa1b13d27d285a6533fba764aedef4e28d


kaggle tpu kernel(tf 2.2.0) "Socket closed" when using subclassing api

When doing model.fit, will always got socket lost/closed error using tpu kernel V3-8 if using subclassing api while functional api is ok. code like below
class Model(keras.Model):
    def __init__(self):
        super(Model, self).__init__()

        bert_layer = hub.load(BERT_GCS_PATH_SAVEDMODEL)
        bert_layer = hub.KerasLayer(bert_layer, trainable=True)
        self.bert_layer = bert_layer
        self.dense = keras.layers.Dense(1)

   def call(self, input):
        input_word_ids = input['input_word_ids']
        input_mask = input['input_mask']
        segment_ids = input['all_segment_id']

        x, _ = self.bert_layer([input_word_ids, input_mask, segment_ids])
        x = self.dense(x)
        self.logit = x
        x = tf.math.sigmoid(x)
        return x

@chenghuige
i ran the code shared by you on nightly and do not face any issues, please let me know if i am missing anything here, gist for the same
please share the error faced by you.

@Saduf2019  I tried to use lates tf in kaggle kernel, following https://www.kaggle.com/kivlichangoogle/jigsaw-multilingual-getting-started but with lates tf, tpu setup will fail, I have modified the colab file you provide.

@chenghuige I would recommend you to post this in Kaggle/kaggle-api as this platform is only for tensorflow issues. Thanks!

Closing this issue as its not related to tensorflow. Please post it in the kaggle platform. Thanks!


Redirect on project after issue created

Closes #17011
If user create an issue with button in project view, after issue is created the user is redirect to project view.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@1f45b3a). Click here to learn what that means.
The diff coverage is 16.66%.

@@           Coverage Diff           @@
##             main   #17211   +/-   ##
=======================================
  Coverage        ?   45.30%
=======================================
  Files           ?      779
  Lines           ?    87278
  Branches        ?        0
=======================================
  Hits            ?    39539
  Misses          ?    41325
  Partials        ?     6414

Impacted Files
Coverage Δ

routers/web/repo/issue.go
38.40% <16.66%> (ø)

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 1f45b3a...6849c8c. Read the comment docs.

:rocket:


How to renderPerEdge from Source to Target Positions

If edges have arrows, the path when using renderPerEdge goes from edge.sourceEndpoint() to edge.targetEndpoint(). Is it possible to render the path from edge.source().position() to edge.target().position()? It seems straightforward if the path is "straight" but not sure how to apply it to "bezier" style. Is there a way to achieve this or maybe consider an option for fullPath?

why do you need the path reversed? what style cannot be done if the default path (which is generated by cytoscape) is being used?

Sorry if my OP was not clear. I hope this pen makes it clear. I would like the blue path to go from a to b nodes and not to target arrow. edge.target().position() gives the position of the node where edge.targetEndpoint() gives the position to target arrow. In case of straight line, I can directly use both psotions to draw a straight line. But for bezier curved line, I'm not sure how it can be extended to the target node.

hm. this plugin doesn't create the path but just piggy packs on the path from cytoscape. So, I'm not sure how to help you
this is the section of the code that does the edge rendering
https://github.com/sgratzl/cytoscape.js-layers/blob/9b2e63e6cd67bc39946956650fb52cb930ab5ccb/src/elements/edges.ts#L83-L109

I found a fix by accessing edge._private.rscratch which has the path data. I updated the pen and here is the relevant code for anyone interested. If want to go from the center of the source node to the center of the target node, you can beginPath from edge.source().position() and extend to edge.target().position().
layers.renderPerEdge(layers.append('canvas'), (ctx, edge, path) => {
  ctx.strokeStyle = 'blue';
  ctx.stroke(path);
  const targetPos = edge.targetEndpoint();
  const pathData = edge._private.rscratch;
  ctx.beginPath();
  ctx.moveTo(pathData.endX, pathData.endY);
  ctx.lineTo(targetPos.x, targetPos.y);
  ctx.stroke();
});


WIP: add RHOAI Notebooks Performance tests to the HOME page

Type of change

[ ] Refactor
[x] New feature
[ ] Bug fix
[ ] Optimization
[ ] Documentation Update

Description
This PR adds PSAP RHOAI Notebooks Performance tests to the HOME page of the CPT Dashboard
Note that this PR includes multiple WIP commits. These commits will not be included in the final version of the PR.
Related Tickets & Documents

Related Issue #
Closes #

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please describe the System Under Test.
Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

Bad bot.

@kpouget is this still being worked on?

it's on hold, too many things to do, too few persons!
But the idea is still open, we'll get to it, sooner or later :)

thanks @vishnuchalla for having a loook,
this work is currently on hold because of lack of time, but we'll get back to it :)


Create profile.md

Welcome to my github profile!
i chhanged something

this is my first git hub change


Need to check return value of libspdm_append_managed_buffer ()

Need to check return value of libspdm_append_managed_buffer () if it is RETURN_BUFFER_TOO_SMALL.

This patch only fixed libspdm_append_message_k(), but not libspdm_append_message_f().
We need additional patch for libspdm_append_message_f().


(Feature proposition) Automating the process of creating, deploying and running excision-mail on vultr

If anyone is interested, I can try to implement a fully-featured ansible playbook, that deploys a OpenBSD serer on vultr, and runs site-preinstall, site-install, site-extra.
So the only necessary thing to do, is to set up the vars and run 1 ansible-playbook...

As a side note: Vultr requires special permission to open port 25.

Yes, you need to be a member I think for 30 days. And then you can create a ticket that will open port 25


added option to use em units for min and max font

This allows for using 'ems' for minFont, maxFont, minimum, and maximum, via a useEm boolean.  fontRatio and lineRatio is essentially untouched.
A dirty example of usage would be:
$('#em').flowtype({
	minimum: 31.25,
	maximum: 75,
	minFont: 0.75,
	maxFont: 2.5,
	useEm: true
});

+1


Move query monitoring to overview and remove dedicate monitoring section

Summary of the changes / Why this is an improvement
Move query monitoring to overview and remove dedicate monitoring section.
closes #733

Happy for feedback @amotl @seut @mfussenegger
Checklist

[x] CLA is signed

Seems good to me. Maybe  @belringer also has some inputs?

Actually I am unsure, if the "Query Per Second" and "Query Speed" isn't a bit misleading overall to show to non-superusers (crate), as it only includes the own queries for non-superusers.
technically this could definitely could use some clean up and deduplication (i.e. load / monitoring graphs are implemented separately)

Looks good to me. Since the monitoring isn't too complex and it is placed on the front page, I don't see the need for a separate section. Plus the front page is already showing monitoring related information.

@belringer Do you think it makes sense to make them a bit smaller and maybe display 2 or 3 graphs next to each other.
--
Plan would be to clean it up a bit more on the implementation side :)

@proddata Sure, maybe something like this:

But then I would also suggest a more refined implementation of the chart toggles, as they break a bit strangely (see screenshot above).

@belringer
how about so 😬  ... would be a bit more in line with the Cloud UI

I would say it is ready for review now :)

Thank you. Let's get it merged in order to cut another release?


Incorrect PCF generation

Avro spec specifies the first three steps of the PCF transformation pipeline as :
[PRIMITIVES]
[FULLNAMES]
[STRIP]
This means that the following schema fragment:
{
"type": {
"logicalType": "timestamp-millis",
"type": "long"
},
"name": "dt"
},
Should transform to (leaving whitespace in):
{
"name": "dt",
"type": {
"type": "long"
}
},
Instead, schema::canonical_form() transforms to:
{
"name": "dt",
"type": "long"
},
I believe this is incorrect, because the PRIMITIVES transformation 'should' run before the STRIP transformation.

You might be right or not ... I will have to re-read the related parts of the spec.
It is always good to cite the sentences/paragraphs which you think are important. Otherwise we have to guess and make mistakes ...
AFAIR the produced fingerprints are the same as the ones for the Java and the Python SDKs, so you might be wrong ...


Stop Dancing MenuItems on standard mode

Maybe related: https://github.com/kitasuke/PagingMenuController/issues/291
That happens if total menu items' width is less than device width.
Original:

Fix:

@kitasuke thanks!

@kitasuke .  Please let me know when i add tableview in a view controller & after TblView.reloadData() getting crash


[Bug]: Cannot connect MSSQL Datasource.

Is there an existing issue for this?

[x] I have searched the existing issues

Description
When im adding a MSSQL Datasource, there is no connection. When clicking Test, there is also no output, not even in the logs.
Datasource does not work. Hostname/IP is reachable and nothing blocked.
Steps To Reproduce

Add MSSQL Datasource
Fill in correct Host,Port,User,Pass
Click Test

Public Sample App
No response
Environment
Production
Issue video log
No response
Version
Self Hosted - v1.9.15

@BRM-SMS are you able to write a query on top of the database?

Hi @BRM-SMS is this a named MsSQL instance ? If so, then please checkout this issue for a workaround: https://github.com/appsmithorg/appsmith/issues/15531


test consul graal integration with both serde and Jackson databind

Ahhh I see, it's a TCK extraction with a failing test on master that should be fixed by #523

exactly

I pushed a commit to fix the Sonar Bug, but Github is lagging again


Fix ineffassign error in masterRole assignment

golangci-lint throws error ineffectual assignment to masterRole. Removed unnecessary assignments here. Default for bool is false in Go anyways.

This change is

Coverage remained the same at 76.744% when pulling 575d90eb003a0a76bdc2325b1d3f4735f9c37157 on terrytangyuan:patch-1 into 2fda21915af061f8846c3cc75019980eb5c44cca on kubeflow:master.

Coverage remained the same at 76.744% when pulling 575d90eb003a0a76bdc2325b1d3f4735f9c37157 on terrytangyuan:patch-1 into 2fda21915af061f8846c3cc75019980eb5c44cca on kubeflow:master.

/test kubeflow-tf-operator-presubmit

/retest


Add notebooks that justify the methods

Recreate the key results form the paper using the methods implemented here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

6 of 9 (66.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.3%) to 95.935%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

emin/source.py
6
9
66.67%

Totals

Change from base Build<PHONE_NUMBER>:
-2.3%

Covered Lines:
118

Relevant Lines:
123

💛  - Coveralls


GLSL 1.5.0 is not supported

Hi,
I followed the instructions for Gibson2 installation. When I run the demo code test:
I get this error:
GLSL 1.5.0 is not supported. Supported versions are ....
I did retry the installation with USE_GLAD set to FALSE in CMakeLists, but this resulted in the installation crashing.
Any ideas on the next steps I can take?

Can you provide a bit more information about your platform? (os, nvidia driver version, etc)
Can you also show the output of glxinfo? Thanks!

Judging from a few posts I read online a potential solution would be to add environment variable
MESA_GL_VERSION_OVERRIDE=3.2 or MESA_GL_VERSION_OVERRIDE=3.3, not sure what is the intuition behind it.
source:
https://github.com/OpenHantek/openhantek/pull/319
https://discourse.slicer.org/t/error-glsl-1-50-is-not-supported/10083

Hi.
I  am on Linux, specifically  a Red Hat Enterprise Server.
I am using an NVIDIA GeForce GPU with 440.64  version driver  and cuda version 10.2 (those numbers from nvidia-smi)
When I  do ldcongif -p | grep EGL I can see

When I run glxinfo, it gives an unknown command error.  Would you happen to know where the command is located?

glxinfo is in mesa-utils in ubuntu, https://packages.ubuntu.com/search?keywords=mesa-utils
I am not sure where it is on redhat.
judging from your ldconfig it looks fine because you have libEGL_nvidia installed.

Do you know where I could make that change?

From: Fei Xia<EMAIL_ADDRESS>Sent: Wednesday, February 10, 2021 12:30 PM
To: StanfordVL/iGibson<EMAIL_ADDRESS>Cc: Koppal,Sanjeev Jagannatha<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [StanfordVL/iGibson] GLSL 1.5.0 is not supported (#62)
[External Email]
Judging from a few posts I read online a potential solution would be to add environment variable
MESA_GL_VERSION_OVERRIDE=3.2 or MESA_GL_VERSION_OVERRIDE=3.3, not sure what is the intuition behind it.
source:
OpenHantek/openhantek#319https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_OpenHantek_openhantek_pull_319&d=DwMCaQ&c=sJ6xIWYx-zLMB3EPkvcnVg&r=UTCUHUgNwObcuZsLsLMIiez54sUEmW8TukUlmQVuvEY&m=QVi4ImGVlKUvOUj3VyeaC68gMQvkk_MYXuKVyI-tQAA&s=V-7bop6mSfoXB2j19bKuGskWbBqbLfJlHa5egjSKH04&e=
https://discourse.slicer.org/t/error-glsl-1-50-is-not-supported/10083https://urldefense.proofpoint.com/v2/url?u=https-3A__discourse.slicer.org_t_error-2Dglsl-2D1-2D50-2Dis-2Dnot-2Dsupported_10083&d=DwMCaQ&c=sJ6xIWYx-zLMB3EPkvcnVg&r=UTCUHUgNwObcuZsLsLMIiez54sUEmW8TukUlmQVuvEY&m=QVi4ImGVlKUvOUj3VyeaC68gMQvkk_MYXuKVyI-tQAA&s=kXF3KZ8m9OhtSMRXZUMSGCqY5sdlTEattFkuh6vsXNw&e=
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_StanfordVL_iGibson_issues_62-23issuecomment-2D776881739&d=DwMCaQ&c=sJ6xIWYx-zLMB3EPkvcnVg&r=UTCUHUgNwObcuZsLsLMIiez54sUEmW8TukUlmQVuvEY&m=QVi4ImGVlKUvOUj3VyeaC68gMQvkk_MYXuKVyI-tQAA&s=RZdwUpLApRAXRiPuDMBXS07xJP3Srz6gSIe0F6ozFdY&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AE74C3PDVOACYM2EOG3M3Q3S6K7CJANCNFSM4XNJ3OGA&d=DwMCaQ&c=sJ6xIWYx-zLMB3EPkvcnVg&r=UTCUHUgNwObcuZsLsLMIiez54sUEmW8TukUlmQVuvEY&m=QVi4ImGVlKUvOUj3VyeaC68gMQvkk_MYXuKVyI-tQAA&s=6w6gTGD6mMb3hCupBBKsxdOpJ3BGkVDpa8MCiH-E8Gg&e=.

Judging from a few posts I read online a potential solution would be to add environment variable
MESA_GL_VERSION_OVERRIDE=3.2 or MESA_GL_VERSION_OVERRIDE=3.3, not sure what is the intuition behind it.
source:
OpenHantek/openhantek#319
https://discourse.slicer.org/t/error-glsl-1-50-is-not-supported/10083

Do you know where I would edit this?

I just want to say that I ran the following on the Linux command line:
export MESA_GL_VERSION_OVERRIDE=3.3
Now the error does not show up. For a split second I can see the environment, but then it seg faults.

Which example code are you trying to run?
On Wed, Feb 10, 2021 at 10:50 AM sanjeevkoppal<EMAIL_ADDRESS>wrote:

I just want to say that I ran the following on the Linux command line:
export MESA_GL_VERSION_OVERRIDE=3.3
Now the error does not show up. For a split second I can see the
environment, but then it seg faults.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/StanfordVL/iGibson/issues/62#issuecomment-776931451,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABTLZRGLJEGA54OJYT335HTS6LIQRANCNFSM4XNJ3OGA
.

I guess it should not be trying to use openGL from mesa in the first place. Can you check the output of glxinfo, that would be the most informative. Other debug guides on this issue can be found in http://svl.stanford.edu/igibson/docs/issues.html

I tried both env_example and demo_static. Here is the output from demo_static:

Ok, this is very helpful, it is showing you are using mesa as openGL renderer, but it should be using nvidia openGL, otherwise you are not using the GPU. It would be good to figure out why nvidia opengl is not used.
Are you operating locally or trying to connect over ssh?

I am operating over an interactive desktop run by TurboVNC (setup by the server management). I start this session on a GPU.
I understood what you said about using nvidia openGL. I am not sure about how to set that up, but I will investigate.

Been investigating, but no luck. You can see from my previous posts that both Mesa and Nvidia openGL are installed, and yet I cannot get it to point only to the Nvidia open gl. Any suggestions?

Also, glxinfo is showing no command found.

Folks, believe it or not, it started working. I changed multiple things at once, so I am not sure what it was:

Added an ubuntu linux container module
Increased GPU memory (my bet is on this)
So far the static has worked.
Thanks for your assistance.


Create basic build_and_test.yml

Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to documentation are included in the PR

Can you check test failures?