Add pytorch to ML image (or create separate image)

For LEAP and M2LInES 2i2c Hubs, many researchers will want to use pytorch. Unfortunately, this image does not have pytorch.
When I try to add pytorch-gpu to ml-notebook/environment.yml and call
conda-lock lock --mamba -f environment.yml -f ../pangeo-notebook/environment.yml -p linux-64

I get the error
Encountered problems while solving.
Problem: nothing provides __glibc >=2.17,<3.0.a0 needed by pytorch-1.10.0-cuda102py37h689c94d_1

I found a similar problem in https://github.com/NVIDIA/MinkowskiEngine/issues/354. That issue implies we may need to mix conda channels to get this to work. The workaround they recommend is to install openblas from anaconda and then switch to conda-forge for pytorch.
Any thoughts on the best path here?
cc @yuvipanda
Also related to #188

Encountered problems while solving.
Problem: nothing provides __glibc >=2.17,<3.0.a0 needed by pytorch-1.10.0-cuda102py37h689c94d_1

Could you try to force later pytorch and/or higher cudatoolkit? This error is likely related to the underlying image you're using (or the machine). __glibc <2.17 is pretty old :(
The openblas addition (at least for recent pytorch versions from conda-forge) likely makes no difference.

Or are you trying this on a personal machine. I will test it quickly to test why this is happening.

@rabernat the error is coming from this: https://github.com/conda-forge/pytorch-cpu-feedstock/blob/93333de760d702aee276dcafeedc890458a00417/recipe/meta.yaml#L34
Something is forcing __glibc <2.17. I will look more closely at this later. It is probably also holding up other packages from getting upgraded. Quite a few packages moved to __glibc >=2.17 on conda-forge.

Paging @hmaarrfk who may be able to advise on this situation. Odd because the successful image ends up being 1.10.2 and pulling glibc 2.17 if you look closely at the CI.

It seems that you are using ubuntu images with 20.04 though. So I'm really not sure.

Yep, I would guess it is a simpler conflict between packages somehow that may need to be sorted out manually. Thanks a lot @hmaarrfk and sorry for bothering you with this here!

Can you check the output of
conda info

before running any other conda commands?


Have you got quality result on ADE20K?

Seems that loss stay at 0.65. And indoor scene cannot got quality result.
Have you got quality result on ADE20K dataset? Can you upload your pre-trained model on ADE20K.
Thanks a lot for sharing your code.

have not trained it on ADE20K


DevDB Continued Refresh

Relatively straightforward. Waiting @damonmcc's version pinning of devdb tests
Commits

Move majority of version.env vars into recipe, and grab previous version during planning
update readme
use parquet for a single dataset
more error handling in looking up past versions

Nice, though I do wonder if we're conflating template variables and env variables. Wondering if we might either:

just set the recipe's env variables into the environment at the beginning of a build. Might makes sense to have some dcpy functionality to set env vars, for various reasons
or just call these vars instead of env, and don't bother putting them in the os.environ

In any case, I do really like capturing vars in the recipe, esp if we upload the recipe file to edm-publishing, just to make it explicit what went into the build.
Thoughts @damonmcc @sf-dcp ?

Yeah separating env and template vars seems like a good distinction. Especially since env often contains secrets - not that bash env vars are making it into the recipe's env field, but just seems good to keep that distinction explicit and make sure nobody makes some unfortunate assumptions

Nice, though I do wonder if we're conflating template variables and env variables. Wondering if we might either:

just set the recipe's env variables into the environment at the beginning of a build. Might makes sense to have some dcpy functionality to set env vars, for various reasons
or just call these vars instead of env, and don't bother putting them in the os.environ

In any case, I do really like capturing vars in the recipe, esp if we upload the recipe file to edm-publishing, just to make it explicit what went into the build.
Thoughts @damonmcc @sf-dcp ?

Yeah separating env and template vars seems like a good distinction. Especially since env often contains secrets - not that bash env vars are making it into the recipe's env field, but just seems good to keep that distinction explicit and make sure nobody makes some unfortunate assumptions

There are multiple cases where "input" vars still need to be made part of the bash env for the rest of the build, so maybe we should move all of that to template vars, and then export the entire vars field, rather than the targeted way it's being done for version and version_prev for devdb in the github action in this pr (and pluto as well, though currently no changes in this PR)

@fvankrieken I kinda lean towards

so maybe we should move all of that to template vars, and then export the entire vars field
Just for consistency across the entire build process. But I don't have a strong opinion, and I'm happy if you want to get this in and we can let it percolate.

Going to make that env/var tweak, then re-run devdb build before merging

build all on the latest commit of this branch


Revert "chore(deps-dev): bump octokit from 2.0.14 to 3.1.2"

Reverts ircam-ismm/node-web-audio-api#56

Oh damn, rather boring as it is not used in the CI, anyway thanks for the catch!


Detect C++14 as C++

Description
The language-cpp14 package provides a grammar called ‘C++14’. When a C++ header file is open and set to C++14, running the beautifier causes Uncrustify to assume the file is C, rather than C++. Uncrustify then often fails because there are non-C things in the file.
Expected Results
A file set to C++14 results in Uncrustify treating it as C++.
Steps to Reproduce

Install language-cpp14.
Open a C++ header file.
Set the language to C++14 (if not set already).
Run the atom-beautify.

Debug
—
Checklist

[ ] I have tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
[x] I have reloaded (or restarted) Atom to ensure it is not a caching issue
[x] Searched for existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
so I know this is not a duplicate issue
[ ] Generated debugging information and added link for debug.md Gist to this issue

@biffen I opened a bug with the uncrustify project.  I am not very familiar with uncrustify and they are asking for more information.  Would you mind hopping into https://github.com/uncrustify/uncrustify/issues/763#issuecomment-248305327 and responding?

Please add a link to the required debug.md Gist which includes debugging information that answers our most commonly asked questions: https://github.com/Glavin001/atom-beautify/blob/master/ISSUE_TEMPLATE.md#how-to-create-debugmd-gist
Thank you.

Based on your issue description what needs to be done is add the C++14 grammar to the C++ language within Atom-Beautify: https://github.com/Glavin001/atom-beautify/blob/master/src/languages/cpp.coffee#L10
However, the debug.md is required to confirm this.
If this is the issue then please submit a Pull Request changing the grammar list for C++. See https://github.com/Glavin001/atom-beautify/blob/master/docs/add-languages-and-beautifiers.md and other past Pull Requests for more information. Let us know if you have any questions.

@prettydiff I respectfully disagree. This is simply what @Glavin001 mentions; adding the C++14 grammar to atom-beautify.
@Glavin001 How do I create a debug.md (the template doesn't say)? And is it really necessary? You seem to have a solution already.

@biffen https://github.com/Glavin001/atom-beautify/blob/master/docs/troubleshooting.md

Thanks @prettydiff . That is the correct link. It appears my GitHub saved reply is outdated.

How do I create a debug.md (the template doesn't say)? And is it really necessary? You seem to have a solution already.

In the future please include a debug.md Gist which will provide answers to common questions we ask. However, this sounds like a simple missing grammar for the C++ language file within Atom-Beautify. You can submit a Pull Request after you confirm the recommended fix works for you. Thanks!


skpm-build ignores relative paths of commands

Version<EMAIL_ADDRESS>
This is a problem only when we have commands where the file name referred to by their script happens to be the same. See this repo.
In the input src/manifest.json, there are two commands with different identifier and different script (but the file names happen to both be my-command.js):
{
  // ...
  "commands": [
    {
      "name": "foo",
      "identifier": "my-plugin.foo",
      "script": "./my-command.js"
    },
    {
      "name": "bar",
      "identifier": "my-plugin.bar",
      "script": "./bar/my-command.js"
    }
  ],
  // ...
}

After running skpm-build, the output plugin bundle only contains one my-command.js. The bar/ relative path appears to have been ignored, and the output file of the second my-plugin.bar command overwrites the output file of the first my-plugin.foo command.
Below is the manifest.json in the output plugin bundle. The bar/ relative path of the second command’s script field is missing.
{
  // ...
  "commands": [
    {
      "name": "foo",
      "identifier": "my-plugin.foo",
      "script": "my-command.js"
    },
    {
      "name": "bar",
      "identifier": "my-plugin.bar",
      "script": "my-command.js"
    }
  ],
  // ...
}

Possible solutions
Unsure if this is some configuration that I’m missing or getting wrong. If not, some possible solutions would be:

Make skpm-build always preserve relative paths in the built output, or
Make it explicit in the docs that relative paths are omitted and that file names used in script must be unique, and perhaps do validation and throw an error if we detect otherwise

That’s definitely a bug, thanks for the nice bug report!


Update to latest CoffeeNet parent version 0.34.0

New CoffeeNet parent version 0.34.0 was released! Please check https://github.com/coffeenet/coffeenet-starter/blob/master/CHANGELOG.md for more information

Pull Request Test Coverage Report for Build 49

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

Totals

Change from base Build 48:
0.0%

Covered Lines:
17

Relevant Lines:
23

💛  - Coveralls


Question: set message while using rayon

It is possible to set the message while I did use rayon?
I would like to create a progressbar simlar to cargo run, where you can see, with files are build behind the progress bar.
Building [===>                      ] 30/195: untrusted, termcolor, ansi_term, typenum(build.rs), regex-syntax, lazy_static, spin, shlex

The problem is that that .progress_with(pb) take the owner ship of pb so I can not call pb.set_message(msg) anymore.
	let pb = ProgressBar::new(stickerpack.stickers.len() as u64);
	pb.set_style(
		ProgressStyle::default_bar()
			.template("[{wide_bar:.cyan/blue}] {pos:>3}/{len} {msg}")
			.progress_chars("#> "),
	);
	stickerpack
		.stickers
		.par_iter()
		.progress_with(pb)
		.enumerate()
		.map(|(i, sticker)| {
			pb.set_message(format!("download sticker {}", sticker.file_id))
			pb.set_message(format!("convert sticker {}", sticker.file_id))

Full code: https://github.com/Lukas1818/mstickereditor/blob/9c083d6fdcde87af10c09483f982bc9f04b70929/src/main.rs#L230
rror[E0382]: borrow of moved value: `pb`
   --> src/main.rs:240:8
    |
229 |     let pb = ProgressBar::new(stickerpack.stickers.len() as u64);
    |         -- move occurs because `pb` has type `ProgressBar`, which does not implement the `Copy` trait
...
238 |         .progress_with(pb)
    |                        -- value moved here
239 |         .enumerate()
240 |         .map(|(i, sticker)| {
    |              ^^^^^^^^^^^^^^ value borrowed here after move
241 |             pb.set_message(format!("dowload sticker {}", sticker.file_id));
    |             -- borrow occurs due to use in closure

A ProgressBar contains an Arc internally, so you're expected to just clone it for this sort of thing.

Thanks I did did not know I can clone it. I thought this would create a new independence bar.
Sadly it seams that the bar is not updated even after I call pb.tick() only if I does use pb.enable_steady_tick(200); or one thread is finished the the bar is updated.

What version are you using? That sounds like a separate issue which might have been addressed on the main branch, maybe see if that works for you?

I did use 0.16.2.

Okay, can you file a new issue about this?

Okay, can you file a new issue about this?

done: https://github.com/mitsuhiko/indicatif/issues/321


[📼] ExoPlayer 2.7.0 -> 2.9.1

what ⁉
Subtitles are blocked for now so I:

Bumped ExoPlayer version to 2.9.1
Removed unsightly transparent status bar on 21 and above.

before & after 👀

devices tested
I made sure vides still play on:

Pixel 2, API 28
Galaxy Nexus, API 17

 Updating ExoPlayer library to 2.9.1


feat(anta.tests): Nested test results

Description
Add a new option to allow to expand "atomic results" for a test in report:
Implements CLI, Option in reporter and result_manager
Need more thorough testing
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have run pre-commit for code linting and typing (pre-commit run)
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes (tox -e testenv)

Closing this for now as it complexifies how to write test and stacking tests should be investigated instead to keep things simple + caching may simplify this


Fixed result for asset property queries

Only one result was while multiple assets where queried.

the handleAssetMeasurementQuery function now has unused parameter seriesLimit. If it is no longer needed, remove it from the function signature, otherwise reimplement it

the handleAssetMeasurementQuery function now has unused parameter seriesLimit. If it is no longer needed, remove it from the function signature, otherwise reimplement it

resovled


Extend Data Migration to inlcude Multiple Resources and Rules

Describe the feature request.
Background:
We have encountered resources that are correctly formed but have references pointing to non-existent resources, this requires filtering the target resources together with related resources to help in correcting wrong references created.
Implementation:

Refactor Data Migration implementation for loading target resources to include forward/ rev include of resources
Provide a way to run multiple rules for executing value

Additional context
This will further the work implemented by this PR
Acceptance criteria
Using migration feature to query for target resources and filter for related resources, use values from related resources to update target resources.
Implementation plan (For Engineers)
@ellykits update some plan/more contxt here

@ageryck did you do the test to check whether the CarePlan for the bug on eCHIS has the tasks tied to it?
Please share it if possible

@ageryck did you do the test to check whether the CarePlan for the bug on eCHIS has the tasks tied to it?
Please share it if possible

@dubdabasoduba This is not included as the code adding outcome reference is not the SM


Broken Link to Finance Trial?

[Enter feedback here]
Could you please check that the link for a Finance trial is correct?  If I follow https://aka.ms/trypoce, it appears to be broken.
Document details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7a8e0a73-6f79-8869-a41d-d17c360faf57
Version Independent ID: b09df2b0-0065-d3c8-74d6-e3c314bca1e2
Content: Sign up for Project Operations preview subscriptions for resource/ non-stocked scenarios
Content Source: articles/environment/resource-sign-up-preview-subscription.md
Service: dynamics-365-project-operations
GitHub Login: @sigitac
Microsoft Alias: sigitac

@standswell the link has been corrected and should be aka.ms/try-po


Can't install with engine-strict

Runnning npm install @storybook/react --engine-strict (or with engine-strict in .npmrc) with npm 5 fails with
npm ERR! code ENOTSUP
npm ERR! notsup Unsupported engine for<EMAIL_ADDRESS>wanted: {"npm":"^3.0.0"} (current: {"node":"8.9.1","npm":"5.6.0"})
npm ERR! notsup Not compatible with your version of node/npm<EMAIL_ADDRESS>npm ERR! notsup Not compatible with your version of node/npm<EMAIL_ADDRESS>npm ERR! notsup Required: {"npm":"^3.0.0"}
npm ERR! notsup Actual:   {"npm":"5.6.0","node":"8.9.1"}

npm ERR!      /Users/felix/.npm/_logs/2018-02-19T22_21_14_142Z-debug.log

Afaik yarn has this on by default too.
Dependency tree:
<EMAIL_ADDRESS>/Users/felix/go/src/sourcegraph.com/sourcegraph/sourcegraph/web
└─┬<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    └──<EMAIL_ADDRESS>
Tracked in https://github.com/arunoda/podda/issues/6, but obviously severely impacting storybook.
PR has been open since August: https://github.com/arunoda/podda/pull/5

Related: #2313

I'd base on ideas from https://github.com/storybooks/storybook/pull/1334

Released 3.3.14


WordPress API sorting broken

Using the WordPress API with the "orderby" parameter should sort.  Customizations to the sorting algorithm appear to have broken it.  This makes publishing impossible until the sorting algorithm is restored because there is no way to get the most recent modified date.
These results should be sorted by modified desc...
https://live-drought-ca-gov.pantheonsite.io/wp-json/wp/v2/posts?per_page=10&orderby=modified&order=desc&_fields=slug,modified

Woot!


Smoketest: app.reload did timeout

https://dev.azure.com/monacotools/Monaco/_build/results?buildId=136306&view=logs&j=672276a2-8d3a-5fab-615d-090c51352f92&t=6e7e74bd-a473-53b1-acea-ad5f8c1f1ec9

1) VSCode Smoke Tests (Electron)
       Dataloss
         verifies that 'hot exit' works for dirty files:
     Error: Timeout: get window ids after 20 seconds.

That sounds as if the application itself could not start.

Not sure what happened. Logs don't say much unfortunately. Relying on Playwright to iron these out: https://github.com/microsoft/vscode/issues/128479


Update black.py (only for test)

No changes needing a change description found.

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs


DataManagementApi: implement TransferProcessApiController

TransferProcessApiController currently contains stub implementations. The controller should be implemented by calling the TransferProcessServiceImpl. Tests should be added.
Examples:

ContractNegotiationApiController
ContractAgreementApiController (in #1048)

Implemented in #1104


[Access Request] protected var - report 38 "Trial Balance by Period"

Can you change the variables ColumnValuesAsText and MaxCount to protected var in report 38 "Trial Balance by Period"?

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Fix some problems for some of the requests

Tried to create and delete stacks in a project.
While the create request worked the delete request resulted in an exception because of wrong types.

test this please


⚠ SceneTime has degraded performance

In a766f12, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 10995 ms

Resolved: SceneTime performance has improved in e103eb9.


South Park, Season 9, Episode 4 "Best Friends Forever" will not download from Comedy Central

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.09.01 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
United States/Los Angeles, CA
Provide a description that is worded well enough to be understood
https://southpark.cc.com/episodes/5gkuf7/south-park-best-friends-forever-season-9-ep-4
Similar issue as #2621, but all of the other videos can be downloaded (In chapters, of course) by me without problems.
I did not have the same issue as #2621. Those videos downloaded fine for me, so I'm sorry that I opened a new issue.
And yes, I am using the latest patched yt-dlp of 2<IP_ADDRESS>044 and not 2022-09-01. I thought it would help.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', '--all-subs', '--convert-subs', 'srt', '-f', 'best[width=1280]/best[height=720]/best', '--verbose', '-a', 'Links.txt', '-o', 'S%(season_number)02dE%(episode_number)02d-%(title)s.%(ext)s']
[debug] Batch file urls: ['https://southpark.cc.com/episodes/5gkuf7/south-park-best-friends-forever-season-9-ep-4']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] ytdl-patched/yt-dlp version 2<IP_ADDRESS>044 [9e28f9a] (win32_dir)
[debug] ** This build is unofficial daily builds, provided for ease of use.
[debug] ** Please do not ask for any support.
[debug] Python 3.8.10 (CPython 64bit) - Windows-10-10.0.19044-SP0
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 2022-09-22-git-af919cf780-full_build-www.gyan.dev (setts), ffprobe 2022-09-22-git-af919cf780-full_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.09.14, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[debug] Loaded 1672 extractors
[debug] Fetching release info: https://api.github.com/repos/ytdl-patched/yt-dlp/releases/latest
Latest version: 2<IP_ADDRESS>044, Current version: 2<IP_ADDRESS>044
yt-dlp is up to date (2<IP_ADDRESS>044)
[debug] [southpark.cc.com] Extracting URL: https://southpark.cc.com/episodes/5gkuf7/south-park-best-friends-forever-season-9-ep-4
[southpark.cc.com] south-park-best-friends-forever-season-9-ep-4: Downloading webpage
[southpark.cc.com] 5ffda56c-ecfd-11e0-aca6-0026b9414f30: Downloading info
[download] Downloading playlist: Best Friends Forever
[southpark.cc.com] Playlist Best Friends Forever: Downloading 0 videos of 0
[download] Finished downloading playlist: Best Friends Forever

Note for maintainers: geo-blocked
Also, created using Lesmiscore's yt-dlp daily builds

I did not have the same issue as #2621. Those videos downloaded fine for me, so I'm sorry that I opened a new issue.

It looks like the exact same issue though


Feat(eos_designs): Support VRF-Lite as a design type and add support for core interfaces

Enhancement summary
Feat(eos_designs): Support VRF-Lite as a design type and add support for core interfaces
There are three sub-issues to be considered for this request:
1.) localised VRF under core_interface
2.) add support for the 'vrf-lite' design
3.) need to establish BGP between borderLeaf and Core and no OSPF using core_interfaces
Which component of AVD is impacted
eos_designs
Use case example
This is to be used L3LS with EVPN deployment with OSPF underlay and BGP as overlay, where the Border-Leaf connecting to CORE for connectivity beyond Data Center, or use CORE for inter-VRFs routing for the devices inside Data Center. In this case, there is only BGP on the links between Border-Leaf and CORE.
Describe the solution you would like
1.) Easy way to put an interface in a VRF using Network Services or core_interface without touching the IP addressing
2.) Support from eos_design to vrf-lite design or at least core_interfaces
Describe alternatives you have considered
1.) custom_structured_configuration_ethernet_interfaces under border leaf group in order to put specific interface under each VRF
2.) Using core_interfaces struct (for P2P addressing) and using a custom jinja2 template to generate that yaml config
3.) Enhancement of the module in a fork of the ansible_avd project to be able to disable OSPF on the sub interface
Additional context
No response
Contributing Guide

[X] I agree to follow this project's Code of Conduct

Hi Vicky, can you add an example of the target configuration you are looking for? I believe we can do everything that is asked here by using the l3_interfaces and bgp_peers under network-services.

The underlay for DC facing interface is OSPF, so need to be turned off for the interface facing the CORE.
Here is the switch config:
!!  We already have a good chunk of the code written to build out the interface as well as define neighbors. One restriction today (from what I can tell, anyway) is that BGP peering requires that BGP is the underlay protocol for this model. It may make sense to allow for BGP peering without restricting this to the include_in_underlay_protocol flag, as well as extend an optional vrf flag to define the VRF this peering belongs in - in @buddy_bunny's case, we would likely need the flexibility to define the vrf on each side of the p2p.  For BGP specifically, we'd need a peer-group flag of some kind to match an already built peer-group (ideally); could get more complex pretty quickly with all the neighbor knobs available, may need a profile here instead.
This would allow us to extend routing to firewalls/load balancers/external fabric devices fairly easily.
https://avd.sh/en/stable/roles/eos_designs/doc/l3-edge.html
For the data model example, I'm thinking about something like:
    p2p_links_ip_pools:
        testpool1: <IP_ADDRESS>/24

    p2p_links:
        - id: 1
          nodes: [ border-leaf, core ]
          interfaces: [ Ethernet1/1.1, Ethernet33/1.1 ] # may need support for subinterfaces here too for this particular case.
          as: [65385 , 65001]
          ip_pool: testpool1
          #include_in_underlay_protocol: true # required today for bgp peering to come up assuming bgp is the underlay.
          routing_protocol: bgp # options: ospf, bgp, isis, none. Default: none # if `include_in_underlay...` is selected, it takes precedence
          vrf: [<vrf-name>, <vrf-name>] # Default: default
          bgp_peerGroup: ['peer-group-name' , 'peer-group-name']

Expectation is that this would output something like:
---border-leaf---
interface Ethernet1/1.1
   description P2P_LINK_TO_core_Ethernet33/1.1
   encapsulation dot1q vlan 1
   ip address <IP_ADDRESS>/31
!
router bgp 65385
   vrf <vrf-name> # assume `VRF1` in this case
      neighbor <IP_ADDRESS> peer group <peer-group-name>
      neighbor <IP_ADDRESS> remote-as 65001
      neighbor <IP_ADDRESS> description core

---core---
interface Ethernet1/1.1
   description P2P_LINK_TO_border-leaf_Ethernet1/1.1
   encapsulation dot1q vlan 1
   ip address <IP_ADDRESS>/31
!
router bgp 65001
#vrf <vrf-name> # assume `default` in this case
   neighbor <IP_ADDRESS> peer group <peer-group-name>
   neighbor <IP_ADDRESS> remote-as 65385
   neighbor <IP_ADDRESS> description border-leaf

Thoughts?

We have most/all of this under the network_services data model. I can understand if this is not the perfect location for it, but we have some data model changes coming in the future, which will make it possible to separate from other network services.
Could I ask you to try to model this using the l3_interfaces and bgp_peers under tenants? We have extended the support for vrf default in that model step by step, but may not be all the way still. So it would be good to know what we need to improve.
Subinterfaces are supported under the network services, and I think we create the parent interface even if not set under another l3_interface.

I would also like this to be available in the fabric topology with a new uplink type under the node type key:
uplink_type: p2p-vrf-lite.
This would be useful for the Campus core to campus aggregation layer and other situations where we are constrained by MTU size and can't run an overlay.

This should be tackled as two independent issues:

[ ] Add support for uplink_type: p2p-vrf-lite
[ ] Add support for vrfs under l3_edge / core_interfaces


announce the new system

Reticketing from @webmaven at https://github.com/gratipay/gratipay.com/issues/3390#issuecomment-101873253.

Includes updating the front page at all?

Includes updating the front page at all?

It has to; Gratipay needs to pull people in, both project leaders and potential funders.

+1 from @kaguillera on #3428:

I suggest apart from answering this comment/thread/issue directly that it should be emailed/posted/blogged/tweeted/etc somewhere so that users like myself know what is going on.

I put up a simple notice in #3452. This ticket is for a proper blog post.

Here's a draft: https://medium.com/@Gratipay/2453d3c53077 (still very much in-process).

Blocking on:

[ ] turning subscriptions on in the UI (#3432)
[ ] self-serve PayPal (#2387)
[ ] require identity for PayPal (easy enough, but depends on #3458)
[ ] split out revenue model question (#3456)

Left some feedback on the draft.

Thanks! :)

#3458 is too much work to get done before tomorrow. :-( I'm striking it from the punchlist.

Ready for final review: https://medium.com/@Gratipay/2453d3c53077.

@whit537 Every team link in that post 404's for me.

@clone1018 Yeah, pending #3464—added to punchlist.

Outta time. :-(
I'll be back later tonight ...

Okay! We have PRs open for #3432 and #3456. Once we land those we can give https://medium.com/@Gratipay/2453d3c53077 a once-over and then publish. Then we can follow up with lots of users who are waiting for this post.

We're live!
https://medium.com/gratipay-blog/gratipay-2-0-2453d3c53077
:dancer:

#3476 and https://twitter.com/Gratipay/status/601565078547083265

I followed up with everyone in our support queue and in my private email.

Was the feedback I gave on the post helpful or just distracting (I can't see what I said anymore, or if you responded)?

@webmaven Helpful! I believe I incorporated all or almost all of your suggestions, though further editing may have obscured that.

Reopening to send an email blast. We did this for "Gratipocalypse", so we should do the same for "Gratipay 2.0" to maintain parallelism.

We did this for "Gratipocalypse", so we should do the same for "Gratipay 2.0" to maintain parallelism.

+1. A friend of mine asked me whether Gratipay is still processing payments, coz he 'didn't receive an email after the Gratipocalypse one'

Done in #3510.


npm run ingest error

Hello, i'm having this problem:
c:\Users\Miguel\Desktop\gpt4-pdf-chatbot-langchain-main\scripts\ingest-data.ts:44
    throw new Error('Failed to ingest your data');
          ^
Error: Failed to ingest your data
    at run (c:\Users\Miguel\Desktop\gpt4-pdf-chatbot-langchain-main\scripts\ingest-data.ts:44:11)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at <anonymous> (c:\Users\Miguel\Desktop\gpt4-pdf-chatbot-langchain-main\scripts\ingest-data.ts:49:3)

Node.js v18.16.0

I think that I have a bad pinecone configuration. Please, help me

I am encountering this same erro. Can someone please provide some context? For reference, temp is set to 0, pdf is not corrupted and/or my api key has access to gpt4.

You need to have an openai paíd account

Same error. My openai account is a paid one. @normaescobar did you find a solution?

@ljs19923 yes I was able to find my issue. Look for a cases I closed a couple of days ago! I wrote the response there!! But in short, all we have to do is change the version of yarn


Updated helm chart with new defaults so the documentation works.

new version from https://github.com/redhat-cop/proactive-node-scaling-operator/pull/7


Use Terratest and Localtest for integration testing

The Terratest golang program does not use any external Terraform files to verify the Terraform integration.  The init, plan and apply Terraform commands are performed against LocalStack, and the Terraform output is verified programmatically.
The lambda is also invoked. Given that the lambda's event is difficult to manufacture and we don't want to actually perform the functions in the lambda, the test creates a bad event and checks the resulting exception to verify that the lambda ran and detected the error.

My thoughts on tftest versus Terratest are in the Trello task.


ssh: Improve error message for KeyboardInteractiveChallenge

For issue: https://github.com/golang/go/issues/41823

I've not contributed to this or other go repos before. Did my best to follow the guidelines for new contributors, but let me know if there's anything that I need to add or change.


Add interactives plot in the plot viewer

I would like to make interactive plots with matplotlib in vscode as I do in Jupyter Notebook using the magic code %matplotlib notebook at the top of the code, in order to use the data zoom cursor and the data tracker as I show in the next image:
https://i.imgur.com/b4mLgex.jpg

@grigosback. Thanks for the suggestion. We'll look into this.

@IanMatthewHuff thanks to you!

@grigosback have you had a chance to try out the plot viewer https://code.visualstudio.com/docs/python/jupyter-support#_plot-viewer? It allows you to zoom, pan, and export matplotlib plots. Would love to get your feedback on whether that's sufficient for your needs.

@ronglums yes, I had. The problem with the current plot viewer is that plots aren't interactive, I can zoom the image but I can't zoom in the data like I do in Jupyter Notebook. I don't know if I'm explaining myself well.

@grigosback I see. that makes sense. We'd have to check if we could do the same in the plot viewer.

@ronglums thank you

Since this is all running is a chrome instance, any chance you have the ability to render js in a new tab? A lot of the really good plotting libraries for python spit out javascript, it would be really nice if it could be directly rendered in a plotting tab. Currently the interactive window happily renders .js plots, having them accessible in a tab removed from code would be really helpful for signal processing work (or data science).
Hope my suggestions clear, thanks for the great work!

Any movement on this? I don't see anyone assigned. It would be great to have %matplotlib notebook work as in a browser. Thanks!

Not that this helps with matplotlib, but I have good luck running vscode notebooks with Bokeh and Plotly interactively.
Alternatively .Net Interactive supports (or will when officially released outside Insiders) XPlot.Plotly interactive plots in output cells driven from F#/C#.
Hoping this will open up as JS charting libraries can be exposed through .Net Interactive middleware!

%matplotlib widget should work as an alternative. (https://github.com/matplotlib/ipympl)
%matplotlib notebook uses jupyter notebook specific UI so doesn't load in our UI.

%matplotlib widget should work as an alternative. (https://github.com/matplotlib/ipympl)
%matplotlib notebook uses jupyter notebook specific UI so doesn't load in our UI.

That worked! Thanks!

@rchiodo that worked! thanks!


多机器分布式部署

请问下，有没有比较详细的帮助手册，关于mxnet在多机器的部署介绍。

https://github.com/dmlc/mxnet/tree/master/example/image-classification

According to the tutorial of official website , can be configured successfully,  tutorial :http://mxnet.io/how_to/multi_devices.html,
Of course，piiswrong mean  was right so. This is very simple，maybe  will be some problems of ssh，If you are not familiar with SSH。


try fixing webpack issues

Overview

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: [ABC-123]
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change
[ ] This PR includes testing instructions tests for the code change
[ ] The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

Don't make me tap the sign.
PR Links and Envs

Repository
PR Link
PR Env

Monolith
Monolith PR
Monolith Env

Portal
Portal Link
Portal Env

Another Repo
Another Link
Another Env

🚀 Styleguide deploy preview ready!https://653686ca1c4bb62752923f9f--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://65a8295a241cb406d446ebc4--gamut-preview.netlify.appDeploy Logs


Nterl0k - T1485 Update windows_high_file_deletion_frequency.yml

Details
Update to inlcude sysmon IED26 as a log source, same fields at EID23.
Minor CIM value alignment.
Checklist

[ ] Validate name matches <platform>_<mitre att&ck technique>_<short description> nomenclature
[ ] CI/CD jobs passed ✔
[ ] Validated SPL logic.
[ ] Validated tags, description, and how to implement.
[ ] Verified references match analytic.

Thank you Steven for these updates! This has been tested and released in
https://github.com/splunk/security_content/releases/tag/v4.30.0


Allow bidirectional communication

Is your feature request related to a problem? Please describe.
In current design, if a subscriber has to return assets back to the publisher, they would have to be onboarded as another publisher. This is complex to manage.
Describe the feature you'd like
Please provide an option to create a return path in mediaexchange for publisher -> subscriber -> publisher

Thank you for your suggestion @provanshu. We've carefully considered your feedback and have decided not to include this feature in a future release.


no examples for the api documentation

the boto3 version of the sdk, is no where near the api as described in the aws documentation. There's no such thing as a sagemaker session in there, and that changes everything and makes things very difficult. I had to guess my way around to actually get things working from java

Hi!
The sagemaker session is in the SageMaker Python SDK, not in boto3. Sorry if this wasn't clear from the example you looked at.


⚠ SolarOscp has degraded performance

In 15087b4, SolarOscp ($SOLAROSCP_URL) experienced degraded performance:

HTTP code: 200
Response time: 177 ms

Resolved: SolarOscp performance has improved in 4e878d7 after 32 minutes.


fix removing elements other than top

Hi @astrofishgames, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

Honestly I don't know how PriorityQueue should behave or how it is working, but if this PR intends to fix a bug, there should be an unit test verifying the fix and also showing the bug in the current version.

@astrofishgames Could you give us some insights of what this PR fixes?

I reproduced the bug and rebased the fix from this PR onto the lastest master, closing this in favor of https://github.com/dotnet/reactive/pull/569/commits.


Why does rye use shims?

This is not a criticism, just a question: why does rye use shims? If rye supported Python version management by reattaching paths instead of using shims, as rtx does, it would eliminate the problems caused by shims (which I hear are common in pyenv and asdf, though I don't know much about them). If it's technically difficult to implement the rtx behavior in rye, it's no wonder that rye uses shims.

I’m not familiar with rtx. What does “ reattaching paths” mean?

Shell integrations are very brittle. That's why I really want to avoid going down that path. It increases startup time and requires maintaining a lot of extra complexity for all variations of shells. It also won't work on windows.


Split pystorm from streamparse.

streamparse includes a Storm multilang implementation in Python in addition to many project workflow features. Splitting the core Storm implementation out would allow other Python projects to reuse this multilang implementation, in addition to any other Storm features that are relevant.

https://pypi.python.org/pypi/pystorm is published, and the master branch of streamparse uses this. Once published to PyPI, this is closed.

This release is planned with #84.

Now that the preview of 3.0 is out, I'll consider this done.


feat(blog): hero

closes #29

Please follow guidelines for creating a issue. you issue was not linked to the PR

fix this runtime error


Diffuse prior on mean function

Is it possible to easily implement a non-zero mean function with an (improper) vague prior on coefficients of explicit basis functions, cf. Section 2.7 in the GPML book. Specifically, predictions to be generated using the formula (2.42) rather than ‘naively’, for the reasons explained there. I’ve dug about quite a bit through the package codebase but I’m a bit lost. Or if not maybe there is some workaround?
Many thanks. (In fact many thanks for the wonderfully useful package in general.)

You can definitely do it, but you'll need to implement the model as described in §2.7 - I don't think GPy has an implementation of this model as-is i.e. I don't see an implementation in https://github.com/SheffieldML/GPy/tree/devel/GPy/models.
But yeah, it shouldn't be too much more than implementing (2.41) for prediction and (2.44) for the likelihood function in a new class VagueLinearPriorGP(GP): (or whatever you choose to name it).
For those reading, the "GPML book" is Rasmussen & Williams (2006) and it's free online at http://www.gaussianprocess.org/gpml/


feat(config): allow percentage value for workers option

Description
I can't use maxWorkers option that jest supported after migrating to vitest.
So I modified it so that the percentage value can be used in maxworkers and minworkers.
https://github.com/jestjs/jest/issues/9012#issue-503013549
Although it is a feature request in jest, I would like to get the same feature in viteest.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. If the feature is substantial or introduces breaking changes without a discussion, PR might be closed.
[x] Ideally, include a test that fails without this PR but passes with it.
[x] Please, don't make changes to pnpm-lock.yaml unless you introduce a new test example.

Tests

[x] Run the tests with pnpm test:ci.

Documentation

[x] If you introduce new functionality, document it. You can run documentation with pnpm run docs command.

Changesets

[x] Changes in changelog are generated from PR name. Please, make sure that it explains your changes in an understandable manner. Please, prefix changeset messages with feat:, fix:, perf:, docs:, or chore:.

I want to add test for worker option but I found two problems.

Can't mock node:os inside bundled vitest/node
if get resolveConfig function from package/vitest/node/config.ts throw type error when running test

I don't think I can test the code properly if I can't mock node:os. Is it okay to test with cpus count without mocking?
Or is it okay to test only exception handling other than calculate logic?

I want to add test for worker option but I found two problems.

Can't mock node:os inside bundled vitest/node
If i import resolveConfig function from package/vitest/node/config.ts to test, throw type error from runtime.

I don't think I can test the code properly if I can't mock node:os. Is it okay to test with cpus count without mocking? Or is it okay to test only exception handling other than calculate logic?

You can create a separate utility that accepts the workers number and the amount of CPU and test that function.

I want to add test for worker option but I found two problems.

Can't mock node:os inside bundled vitest/node
If i import resolveConfig function from package/vitest/node/config.ts to test, throw type error from runtime.

I don't think I can test the code properly if I can't mock node:os. Is it okay to test with cpus count without mocking? Or is it okay to test only exception handling other than calculate logic?

You can create a separate utility that accepts the workers number and the amount of CPU and test that function.

Thanks for nice idea! I separate getWorkers function to utility and node:os mock is works. I added test now.

test failed.. because maybe i changed node:os import method. I fix test in a minutes.

LGTM! Thank you! 🎉


CSS usage doesn't work

The usage via css is currently not working.
The icons only will be displayed if I add the following (from iconfont folder) to my own css:
@font-face {
font-family: "Material Icons";
font-style: normal;
font-weight: 400;
font-display: block;
src: url("~material-icons/iconfont/material-icons.woff2") format("woff2"), url("~material-icons/iconfont/material-icons.woff") format("woff");
}
.material-icons {
font-family: "Material Icons";
font-weight: normal;
font-style: normal;
font-size: 24px;
line-height: 1;
letter-spacing: normal;
text-transform: none;
display: inline-block;
white-space: nowrap;
word-wrap: normal;
direction: ltr;
-webkit-font-smoothing: antialiased;
-moz-osx-font-smoothing: grayscale;
text-rendering: optimizeLegibility;
font-feature-settings: "liga";
}
this I have to do for each style (outlined, etc.)
is this the wanted behaviour? then it should be mentioned in the docs.

The files mentioned in the main readme should also be imported in addition to the files mentioned in the css readme:
import 'material-icons/iconfont/material-icons.css'; // Imports font files for all styles
import 'material-icons/css/material-icons.css'; // Allows usage of CSS classes

I will update the readme to better reflect this.

thx for your reply. after adding all imports like in your post everything works as expected.
from the current readme i couldn't read out that I have to import the material-icons.css from iconfont.
but the update of the readme would surely help there.
thx.


meta-sunxi: Thud updates

With changes on https://github.com/mirzak/meta-mender-community-1/commits/thud-update it builds fine. Thanks.

Does it work as well? :D

Sure :+1: thanks for support

Closing this, replaced by #36

And thank you for your work @nandra


Offline app: Five Second Games can't open

Describe the bug
Offline app: Five Second Games can't open
Steps to reproduce the behavior
1.Download https://arcade.makecode.com/offline-app
2.JavaScript Games----> Five Second Games------>Open Example
Actual behavior
Offline app: Five Second Games can't open

Additional context
1.OS: Windows(rs6)
2.arcade version: 1.2.7
3.Microsoft MakeCode version: 6.2.21

This example includes extensions that aren't bundled with the editor, so it won't work offline. This is expected


Kettle medical rework

About the PR

Large sections of Kettle's medical department have been reworked including:

Ward
OR
Cloning
Cryogenics

Why / Balance

Kettle's medical ward was still one of the few wards that consistently gets deconstructed because it is too clunky to work with. This rework introduces a move of the ward such that it is more accessible to the general public, along with a more open layout.
To encourage use of the cryogenics facilities, an additional cryo pod was added for a total of two, and the entire "subdepartment" has been moved closer to the ward. Cloning now has the entire room it used to share with cryogenics.
I may still change the following:

Cryogenics piping (Not entirely proud of this, tho it does work.)
Paramedic location (They could get a proper room.)

This is mainly because this is some large work and still needs feedback.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Medical ward:

Cryogenics and surgery:

Cloning:

Changelog

Adding because this does seem like a significant enough change, if told to remove I will remove it.
:cl:

tweak: Kettle's medical ward was reworked.

cryo piping seems ok to me

add some tile decals to the main treatment room so areas stand out

add some tile decals to the main treatment room so areas stand out

Working on this now.

Decals added in https://github.com/space-wizards/space-station-14/pull/20435/commits/57e5e8c9c5f608b917dbba777a09aad00dc7e30a .

nice

Issue remaining on this merge: There's not enough medical doctor lockers to cover the map prototype's staff allocation (1-1 senior, 3-5 doctors, 8-8 interns). This will be fixed in a future commit.

Added lockers, but actions failure does seem related. I'm gonna see what's up with that.

YAML validator should now be passing. PR is ready for review, no additional work will be done unless requested.


chore(node-wasm): Switch to bundler target for wasm-pack for manifest v3

I once got this when running extension:
2024-10-01T11:41:20.547000064Z ERROR lumina_node_wasm::commands: invalid response, received: InternalPong

Maybe we could increase interval in NodeClient::new to 200ms just in case?


README.md: fix Atom slack URL and badge

Requirements for Contributing Documentation

Fill out the template below. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
The pull request must only contribute documentation (for example, markdown files or API docs). To contribute other changes, you must use a different template. You can see all templates at https://github.com/atom/.github/tree/master/.github/PULL_REQUEST_TEMPLATE.

Description of the Change

Previous Atom slack channel link and badge were no longer working
Release Notes

Fixes Atom slack channel badge and link.

View rendered README.md

Closing this PR for now. It has now been superseded by https://github.com/atom/atom/pull/22446.

thanks.


Center container vertically.

Need a simple method to center a container vertically.

Pick one fitting your usecase http://howtocenterincss.com/ ;)

Using flexbox, it becomes quite easy:
Check out my example on Codepen

Thanks seshaljain, that solved my issue.


Gradients are [None,...] in text GAN model

I'm trying to build text GAN using TF2. But gradients_of_generator always becoming [None].  Provided code below uses paddings in between generated_prediction and discriminator input. BUT even if I directly pass generator output to discriminator input (of course changing input_length of discriminator), gradients always [None].
How compute gradient in such text GAN correclty?
Please notice, I simplifed code to give more clear understanding of the problem.
code:
`generator = keras.Sequential(
[
layers.Embedding(total_words, 80, input_length=max_sequence_length-1),
LSTM(100, return_sequences=True),
LSTM(50),
tf.keras.layers.Dropout(0.1),
Dense(total_words/20),
Dense(total_words, activation='softmax'),
],
name="generator",
)
discriminator = keras.Sequential(
[
layers.Embedding(total_words, 80, input_length=max_sequence_length-1),
layers.Bidirectional(tf.keras.layers.LSTM(64)),
layers.Dropout(0.2),
layers.Dense(64, activation='relu'),
layers.Dense(1, activation='sigmoid'),
],
name="discriminator",
)
#function: train_step
#text - tokenized text

with tf.GradientTape() as gen_tape:

    predicted_next_word = generator(text, training=True)
    pred_to_word = pad_sequences([[np.argmax(predicted_next_word, axis=-1)]], maxlen=max_sequence_length-1, padding='pre')
    fake_output = discriminator(pred_to_word, training=True)
    generator_loss = binary_cross_entropy(tf.ones(fake_output.shape), fake_output) #gives correct loss results

gradients_of_generator = gen_tape.gradient(generator_loss, generator.trainable_variables)`

Gradient are [None,...]

@e8panse,
I could see that the given code was incomplete. Kindly find the gist of it here and provide the complete code with the
dependencies. Thank you!

`
import time
import random
from tensorflow.keras.preprocessing.text import Tokenizer
from tensorflow.keras.preprocessing.text import one_hot
import tensorflow as tf
import tensorflow.keras as keras
import tensorflow.keras.layers as layers
from tensorflow.keras.models import Sequential
from tensorflow.keras.layers import Dense
from tensorflow.keras.layers import Dropout
from tensorflow.keras.layers import LSTM
from tensorflow.keras.callbacks import ModelCheckpoint
from tensorflow.keras.utils import to_categorical
import numpy as np
from tensorflow.keras.preprocessing.sequence import pad_sequences
from tensorflow.keras.callbacks import ModelCheckpoint
from mgan_1_data_loader import DKDataLoader
batch_size=64
learning_rate=0.01
#use fixed length to simplify test
fixed_sentence_length=30
data_loader = DKDataLoader()
input_sequences = data_loader.tokenization()
fake_sequences = data_loader.generate_fake_text()
xs,ys = data_loader.data_prepare(input_sequences)
total_words = data_loader.total_words #63,303
max_sequence_length = data_loader.max_sequence_length #568
#total samples 100,563
Reserve 5,000 samples for validation.
x_val = xs[-5000:]
y_val = ys[-5000:]
x_train = xs[:-5000]
y_train = ys[:-5000]
Prepare the training dataset.
real_train_dataset = tf.data.Dataset.from_tensor_slices((x_train, y_train))
real_train_dataset = real_train_dataset.shuffle(buffer_size=1024).batch(batch_size)
real_train_dataset_iter = real_train_dataset.iter()
Prepare the validation dataset.
val_dataset = tf.data.Dataset.from_tensor_slices((x_val, y_val))
val_dataset = val_dataset.batch(batch_size)
generator = keras.Sequential(
[
#keras.Input(shape=(max_sequence_length-1,)),
layers.Embedding(total_words, 80, input_length=max_sequence_length-1),
LSTM(100, return_sequences=True),
LSTM(50),
tf.keras.layers.Dropout(0.1),
Dense(total_words/20),
Dense(total_words, activation='softmax'),
],
name="generator",
)
class GeneratorModel(tf.keras.Model):
def __init__(self, total_words, max_sequence_length):
    super().__init__()
    #self.generator = generator
    self.l1 = tf.keras.layers.Embedding(total_words, 80, input_length=max_sequence_length-1)
    self.l11 = LSTM(100, return_sequences=True)
    self.l12 = LSTM(50)
    self.l2 = tf.keras.layers.Dropout(0.1)
    self.l3 = Dense(total_words/20)
    self.l4 = Dense(total_words, activation='softmax')

def call(self, x):
    x = self.l1(x)
    x = self.l11(x)
    x = self.l12(x)
    x = self.l2(x)
    x = self.l3(x)
    x = self.l4(x)
    return x

generator = GeneratorModel(total_words, max_sequence_length)
discriminator = keras.Sequential(
[
#keras.Input(shape=(max_sequence_length-1,)),
layers.Embedding(total_words, 80, input_length=max_sequence_length-1),
layers.Bidirectional(tf.keras.layers.LSTM(64)),
#layers.GlobalAveragePooling1D(),
layers.Dropout(0.2),
layers.Dense(64, activation='relu'),
layers.Dense(1, activation='sigmoid'),
],
name="discriminator",
)
def generate_text_from_word_string(generator_model, seed_text):
for _ in range(fixed_sentence_length):
token_list = data_loader.tokenizer.texts_to_sequences([seed_text])[0]
token_list = pad_sequences([token_list], maxlen=max_sequence_length-1, padding='pre')
pred_1 = generator_model(token_list, training=True)
predicted = np.argmax(pred_1, axis=-1)
output_word = ""
for word, index in data_loader.tokenizer.word_index.items():
if index == predicted:
output_word = word
#next_words = [""]
break
seed_text += " " + output_word
return seed_text

def text_to_sequence(seed_text):
token_list = data_loader.tokenizer.texts_to_sequences([seed_text])[0]
token_list = pad_sequences([token_list], maxlen=max_sequence_length, padding='pre')
return token_list
def generate_and_save_text(epoch, seed_text):
with open('saved_generated_text/saved.txt', 'a') as the_file:
the_file.write(seed_text+' ---epoch:'+str(epoch)+' \n')
optimizer_generator=keras.optimizers.Adam(learning_rate=0.0003)
optimizer_discriminator=keras.optimizers.Adam(learning_rate=0.0003)
loss_fn_generator=keras.losses.CategoricalCrossentropy()
loss_fn_discriminator=keras.losses.BinaryCrossentropy()
binary_cross_entropy = tf.keras.losses.BinaryCrossentropy(from_logits=True)
def discriminator_loss(real_output, fake_output):
real_loss = binary_cross_entropy(tf.ones_like(real_output), real_output)
fake_loss = binary_cross_entropy(tf.zeros_like(fake_output), fake_output)
total_loss = real_loss + fake_loss
return total_loss
def generator_loss(fake_output, fake_ys):
return binary_cross_entropy(fake_ys, fake_output)
@tf.function
def train_step(data_batch):
real_x,real_y = data_batch
word_i = random.randint(0, total_words-1)
tl_res = tf.convert_to_tensor(pad_sequences([[word_i]], maxlen=max_sequence_length-1, padding='pre'))

with tf.GradientTape() as gen_tape:
    #gen_tape.watch(generator.trainable_variables)
    pred_1 = generator(tl_res, training=True)
    pred_to_word = tf.convert_to_tensor(pad_sequences([[25]], maxlen=max_sequence_length-1, padding='pre'))
    #even if I change discriminator input and pass generator output directly to dicsriminator it cannot count gradients
    fake_output = discriminator(pred_to_word, training=True)

    gen_loss = binary_cross_entropy(tf.ones(fake_output.shape), fake_output)

gradients_of_generator = gen_tape.gradient(gen_loss, generator.trainable_variables)
print(gradients_of_generator)

optimizer_generator.apply_gradients(zip(gradients_of_generator, generator.trainable_variables))
#optimizer_discriminator.apply_gradients(zip(gradients_of_discriminator, discriminator.trainable_variables))

def train(epochs):
for epoch in range(epochs):
start = time.time()
for data_batch in real_train_dataset:
    train_step(data_batch)

# Produce texts as you go
seed = random.randint(0, total_words-1)
word = str(data_loader.word_from_index(seed))
g_text = generate_text_from_word_string(generator, word)
generate_and_save_text(epoch + 1, g_text)

# Save the model every 15 epochs
if (epoch + 1) % 15 == 0:
    #checkpoint.save(file_prefix = checkpoint_prefix)
    pass

print ('Time for epoch {} is {} sec'.format(epoch + 1, time.time()-start))

# Generate after the final epoch
seed = random.randint(0, total_words-1)
word = str(data_loader.word_from_index(seed))
g_text = generate_text_from_word_string(generator, word)
generate_and_save_text(epochs, g_text)

train(2)
`

There gists:
https://gist.github.com/e8panse/8d64d6c8a8d4b0390c6f5750529d752c
https://gist.github.com/e8panse/d94e75fa838bda979fadd2491957a6e3

@e8panse,
Though the code was provided, it was not indentation in the way to execute. I tried to make it run with the changes but I was not able to execute. Kindly find the gist and provide the code with indentation which helps us to debug the issue. Thank you!


Claiming: Networking (on Linux)

It'll go over

The ip utility
Interfaces
Routes
Subnets
etc.

#61


Support pd.read_gbq and DataFrame.to_gbq

We should support read_gbq and to_gbq in the DataFrame API when gcp extras are installed.
Imported from Jira BEAM-11587. Original Jira may contain additional context.
Reported by: bhulette.

Unable to assign user @svetakvsundhar. If able, self-assign, otherwise tag @damccorm so that he can assign you. Because of GitHub's spam prevention system, your activity is required to enable assignment in this repo.

Sub-tasks:

#22907
#22908
#22909
#22910
#22911
#22912

Closed the wrong issue


Update depreciation and ITC gross basis based on improved understanding of allowable fees

See conversation on https://github.com/NREL/ssc/pull/1178/. We no longer believe reserves and financing fees are allowable costs in the depreciation and ITC basis, but don't  want to remove them in a patch since we have no way to notify users. Remove these expenses from the basis unless we can find cases where they are allowable.

https://www.seia.org/research-resources/cost-basis-itc-and-1603-applications has a good discussion of developer fees for the partnership flip model (the checkbox should also probably apply to these).

For partnership flip, the following are allowable:

(from https://www.irs.gov/pub/irs-pdf/p551.pdf, pg 3)
This matches the help description of the equity closing cost and potentially the "other financing cost"
The SEIA article above discourages developers having an ownership stake if the developer fee is to be rolled into the basis. That said, it might be less confusing for users if fees in "project term debt" are excluded from the basis and fees in "cost of acquiring financing" are included (and adjusted on the depreciation page as needed).

Community solar upfront fees https://sepapower.org/knowledge/community-solar-cost-allocation-strategies-lessons-from-six-utilities/ suggests these should be amortized (therefore not depreciated):

Now that NREL/ssc#1193 is ready:

Update the send to Excel files on a new branch from develop.
Run test_script_ow.lk for updated SSC results (expecting NPVs to decrease and LCOEs to increase).


Minor performances Improvement:

Inline variable
Remove redundant initializer
Use Math.min
Add final var

Coverage decreased (-0.1%) to 84.291% when pulling dd65d1fbbb85a6680093f074178fe94964df9643 on arturobernalg:feature/minor_performance_improvement into a38c52529beb04bf0815b7d95fb4a393ea110dee on apache:master.

Coverage decreased (-0.1%) to 84.291% when pulling dd65d1fbbb85a6680093f074178fe94964df9643 on arturobernalg:feature/minor_performance_improvement into a38c52529beb04bf0815b7d95fb4a393ea110dee on apache:master.

@arturobernalg
Sorry Arturo, I did not see this PR before I made changes. May you please rebase?

Hi @garydgregory
Don't worry. rebased from master.
TY


[BUG] Cannot run docker-compose images that don't specify a command

Describe the bug
If a docker-compose image doesn't specify a command it won't work with exo run.
To Reproduce
Steps to reproduce the behavior:

Create a dockerfile that specifies a command, e.g.

FROM alpine
CMD ["sleep", "infinity"]

Create a docker-compose that references it and doesn't include a command:

services:
  alpine:
    build: .

Run exo run docker-compose.yml, which will give something like:

❯ dexo run docker-compose.yml
GUI available at: http://localhost:4000/#/workspaces/sfbbetx0x4y6bwf8keb9xv0mem
Error: applying manifest: internal server error
applying manifest: internal server error

In the server logs one sees:
20:00:39 server 2021/08/06 19:00:39 http 20b8fkx46p0y8zpfw8w0480e40: error processing request: adding "alpine": creating container: Error response from daemon: No command specified

Expected behavior
The compose file to run.

We do a call to ImageInspectWithRaw in the ensureImage function. The result of that call has all the image config information needed to make this work. However, a quick glance at the docker-compose internals doesn't make it particularly clear which other bits of image config is used. We should look thoroughly and make sure that we just don't fix CMD here, we fix all the possible other config that may come from the image.

@BenElgar Do you want to take this one on?

Fixed this specific issue here: https://github.com/deref/exo/pull/141.
Will work on looking into what else docker compose does.


Use latest rom, rom-sql and rom-repository releases in new project Gemfile

We don't need to use master there anymore.

@timriley its should be closed by https://github.com/dry-rb/dry-web-roda/pull/16 ?

Yes, this is resolved in #16, thanks for the reminder :)


runtime: removing envoy.reloadable_features.proxy_120_103

Risk Level: low
Testing: n/a
Docs Changes: n/a
Release Notes: yes
https://github.com/envoyproxy/envoy/issues/22291

just flagging https://github.com/envoyproxy/envoy/pull/22311 and associated PRs


<EMAIL_ADDRESS>(close #33)

@ljharb, looks like the package used to validate SPDX IDs here had fallen behind.
The package at the bottom that actually contains the list of IDs is spdx-license-ids.

I have no idea how to version this, so I'm going to do it minor, and just be done with it.

5.1.0


docs: convert from issue template to issue form

Other additions/changes

[x] Changes made in this PR are mentioned below
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] I acknowledge that all my contributions will be made under the project's license.
[x] I've read the contributing guidelines

This PR adds/fixes

Convert from issue template to issue form.

Notes

Also do not forget to update the link in the readme

Also do not forget to update the link in the readme

Also do not forget to update the link in the readme

Which link? Is it the badge link?

Also do not forget to update the link in the readme

Which link? Is it the badge link?

Oh yes sorry for the confusion

Also do not forget to update the link in the readme

Which link? Is it the badge link?

Oh yes sorry for the confusion

It's fine :) I'll be making a good first issue for that.


How to load pre-trained model for fine-tuning?

Hi, thank you for your awesome work!
However, I am facing difficulties to load huggingface checkpoints for further fine-tuning using the slurm scripts. It seems that the huggingface configs cannot align with the slurm model configs. I am trying to fix this by adding "target": "caduceus.configuration_caduceus.CaduceusConfig" to the huggingface configs. Is it okay?
What's more, I still have no idea on how to load the model. Could you please offer me some suggestions? Or could you please release the pre-trained checkpoints produced by slurm scripts? Thank you very much in advance!

The tilt target above should be  "__ target __"

I second releasing the pre-trained checkpoints.  It is not too much work to pre-train, but you do have to have 8 A100 and or 8 A6000 GPUs which is sometimes hard to access.
Also, I felt like I couldn't use my checkpoints on my downstream task until I had reproduced all of your results to verify that I had the correct model. It is a lot of extra work for researchers who just want to make use of this wonderful resource.

However, I am facing difficulties to load huggingface checkpoints for further fine-tuning using the slurm scripts. It seems that the huggingface configs cannot align with the slurm model configs. I am trying to fix this by adding "target": "caduceus.configuration_caduceus.CaduceusConfig" to the huggingface configs. Is it okay?

Yes currently the code is not compatible with HF models, but I created a branch here: https://github.com/kuleshov-group/caduceus/tree/hf_finetune where you can now load HF Caducueus models for downstream tasks, e.g., Genmoics Benchmark and Nucleotide Transformer. See these wrapper scripts for how to use them:

https://github.com/kuleshov-group/caduceus/blob/hf_finetune/slurm_scripts/wrapper_run_nucleotide_transformer_hf.sh
https://github.com/kuleshov-group/caduceus/blob/hf_finetune/slurm_scripts/wrapper_run_genomics_hf.sh

Please let me know if you still have questions

I second releasing the pre-trained checkpoints. It is not too much work to pre-train, but you do have to have 8 A100 and or 8 A6000 GPUs which is sometimes hard to access.
Also, I felt like I couldn't use my checkpoints on my downstream task until I had reproduced all of your results to verify that I had the correct model. It is a lot of extra work for researchers who just want to make use of this wonderful resource.

I pushed the models for the Genomics Benchmark and Nucleotide Transformer to the HF Hub. You can use them as described in the comment above
Genomics Benchmark

https://huggingface.co/kuleshov-group/caduceus-ph_seqlen-1k_d_model-118_n_layer-4_lr-8e-3
https://huggingface.co/kuleshov-group/caduceus-ps_seqlen-1k_d_model-118_n_layer-4_lr-8e-3

Nucleotide Transformer

https://huggingface.co/kuleshov-group/caduceus-ph_seqlen-1k_d_model-256_n_layer-4_lr-8e-3
https://huggingface.co/kuleshov-group/caduceus-ps_seqlen-1k_d_model-256_n_layer-4_lr-8e-3

Thank you so much!!

Thank you Yair!


Updating to include textbook information + adding support for Python 3.

Hi there,
first of all thanks for the great work! I just found one issue when having a look at your changes. The print function does not exist in Python 2.7 and therefore you should use from __future__ import print_function, also see https://stackoverflow.com/questions/32032697/how-to-use-from-future-import-print-function
Best wishes

Thanks a lot!

Hey, thank you guys! I apologize for having not touched this in so long. I'm working on getting some test data and actual tests together.


RC Test: Forces and Motion Basics 2.3.0-phetiorc.1

@samreid @jessegreenberg  so they know this is in the queue.
This RC is from the same shas as https://github.com/phetsims/tasks/issues/868, so it should be done after testing the phet features. Here is the link to index.
Root Directory
Testing Matrix
If any new issues are found, please note them in https://github.com/phetsims/forces-and-motion-basics/issues/227 and reference this issue.
Parent issue for the design changes and implementation: https://github.com/phetsims/forces-and-motion-basics/issues/220
Here is the list of all the changes that have been added

All changes are in the "Net Force" screen
NetForceScreenView includes two new CartStopperNodes placed at a location which will stop movement of the cart.
The cart can move about twice as far along the screen since the stoppers are placed farther than the previous version allowed the cart to move.
The pullers stand up when the game is over
The speedometer was added to the center of the cart, showing speed of the moving cart
A "Speed" checkbox was added to the control panel, which toggles visibility of a speedometer

It looks like this RC is using SHAs that don't include all of the necessary patches for https://github.com/phetsims/scenery-phet/issues/312, specifically it references https://github.com/phetsims/scenery-phet/commit/12053009b222af945f6a2268868ada9ad8fa3ef8 in the dependencies.json, which is missing the fix https://github.com/phetsims/scenery-phet/commit/727d5bd7ac8d93a1c6e8b78b27c9fb6a8a4e783d.
@samreid and @jessegreenberg, can this be cherry-picked to a "forces-and-motion-basics-2.3-phetio" branch in scenery-phet, and then have an RC with that fix included?
@phet-steele can you screen other RCs to make sure that they include that commit?
Also, can someone check to see if the following email was received (I sent it out, but saw no replies):

can this be cherry-picked to a "forces-and-motion-basics-2.3-phetio" branch in scenery-phet, and then have an RC with that fix included?

It could, but cherry picking to a new branch might not be necessary.  Could we just update the scenery-phet SHA in forces-and-motion-basics 2.3-phetio to https://github.com/phetsims/scenery-phet/commit/727d5bd7ac8d93a1c6e8b78b27c9fb6a8a4e783d?
@jonathanolson, Yes, I received the email Fri, Jun 23, 2017.

I am going to give this a shot, but might ask for some help if I run into trouble.  @phet-steele please halt testing for this issue due to the above comments.

Yes, could update the SHA directly to that, as it's the very next commit.

A new RC with the update to scenery-phet has been made in #12, closing this issue.

Actually, @jonathanolson had a request for @phet-steele as part of this issue, leaving open.

@phet-steele can you screen other RCs to make sure that they include that commit?

@phet-steele can you screen other RCs to make sure that they include that commit?

@jonathanolson I checked #11, #12, #13, and #14 and they have the fix.


Feature: add roadmap to README

According to https://github.com/jonashackt/plantuml-markdown, it is possible to add a plantuml diagram to Github readme:

Create file .iuml
Add to github
Use raw link to display -> ! [your-UML-diagram-name] (http://www.plantuml.com/plantuml/proxy?cache=no&src=https://raw.githubusercontent.com/jonashackt/plantuml-markdown/master/example-uml.iuml)

The file .iuml can be added to documentation-resources as roadmap.iuml with the content:
@startuml
start
partition Overview {
    :Articulate aim;
    note right
        Create an android app (.apk) from a simple PyQt5 app
    end note
    :Define requirements;
    :Research existing projects;
    note right
        **Pyqtdeploy** / Pyside / Kivy / BeeWare
    end note
}
partition Implementation {
    :Apk generation;
    note right
        Managed by build-app.py
    end note
    :SQL database;
    note right
        WIP
    end note
    :Graphics;
    :Pkg structure;
}
partition Testing {
    :Logging;
    :Tests;
}
partition Deployment {
    :Documentation;
    :Sharing and Advertising;
}
partition Extension {
    :Windows OS;
    :Linux OS;
    :Mac OS;
}
stop
@enduml

Inside the README:

## Roadmap

_This section introduces the broad roadmap for this project._

![Roadmap Diagram](http://www.plantuml.com/plantuml/proxy?cache=no&src=https://raw.githubusercontent.com/achille-martin/simple-pyqt5-android-app/master/roadmap.iuml)


Running build.sh on fedora 38 system reports an error: There are data after the end of archive

The full error is as follows:
Determining version...
Version: 3875a76
Downloading installer components...
--2023-08-05 16:07:34--  https://www.python.org/ftp/python/3.9.6/python-3.9.6-macos11.pkg
Resolving www.python.org (www.python.org)... <IP_ADDRESS>, 2a04:4e42:1a::223
Connecting to www.python.org (www.python.org)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 416 Range Not Satisfiable
The file is already fully retrieved; nothing to do.

Building m1n1...
make: Entering directory '/a/source/asahi/asahi-installer/m1n1'
make: Leaving directory '/a/source/asahi/asahi-installer/m1n1'
Copying files...
Extracting Python framework...
ERRORS:
There are data after the end of archive
gzip: stdin: unexpected end of file
cpio: premature end of archive

I can't reproduce this, a clean fetch and build works on F38. Maybe delete the contents of the dl/ folder and try again?

Sorry, this is because I only installed p7zip (manually created /usr/bin/7z soft link) and did not install p7zip-plugins.


RDCC-3947: Fixing CVE

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/RDCC-3947
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

As we don't have smoke test to verify whether job is running or not, can you please run the load locally to see if the files are processed successfully?

With the help of @sahitya-desireddy I was able to verify that the files are processed successfully:


ISSUE-1015 Quote window when defining tables to avoid reserved word conflicts

Leaving window unquoted causes table creation to fail when using MariaDB v10.2

+1. @navbruce , thanks for the patch. can you create a github issue and update the PR title to include the ISSUE ?

@navbruce @arunmahadevan
Wouldn't it better to escape all the field name for MySQL as we do for Oracle and PostgreSQL?

@HeartSaVioR Probably yes? I just followed the existing pattern in that MySQL create_tables file.

@HeartSaVioR  yes we could escape all. @navbruce, do you want to take it up as part of this PR ?

I looked at the PostgreSQL and Oracle create table statements and tried to match what they're doing as best I could.
MySQL doesn't support double quotes for field identifiers, only back-tick quoting. Other than that, I quoted all field and index names.

@navbruce , the changes looks good. We require non Hortonworks contributors to sign an iCLA document before accepting contributions. If thats the case can you let me know your email-id where I can send the doc?

@navbruce , waiting for you to return the ICLA to merge this in.

Hi Arun,
I’m really sorry, my boss wasn’t available yesterday and I need his approval to sign the agreement. We should be meeting this morning and I’ll take it up then for sure.
Thanks,

Bruce

From: Arun Mahadevan<EMAIL_ADDRESS>Sent: Tuesday, November 21, 2017 12:55 AM
To: hortonworks/streamline<EMAIL_ADDRESS>Cc: Bruce Fletcher<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [hortonworks/streamline] ISSUE-1015 Quote window when defining tables to avoid reserved word conflicts (#1012)
@navbrucehttps://github.com/navbruce , waiting for you to return the ICLA to merge this in.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/hortonworks/streamline/pull/1012#issuecomment-345958424, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ad38Nyhl7Wg5AFJZel9ZQYD0_sUChsThks5s4o_2gaJpZM4QeMNl.

@navbruce , ping. Can you also rebase with latest master ?

I hope that worked. There were a lot of changes to that file so I attempted to add my changes as conservatively as possible.
Also, waiting to get the go-ahead from the company lawyer on the contributor agreement. Management is very supportive, but none of this is as quick as could be hoped for.

@navbruce Hello. Do you have any update here? Just a friendly reminder.


Add copyright header to all source files

Meta's Open Source Team's automated checks highlighted the missing copyright headers on Headers.cpp.
We had a manual exclusion rule on OITraceCode.cpp, but seeing as how we have core code in this file, I added the header to it and removed the exclusion rule.

Thanks for sorting this! My bad for missing Headers.h.


question: %shortcode has no .Inner, yet a closing tag was provided

This is in my fundamentos.markdown:
{{% moreabout title="Nomenclatura" %}}
Convencionalmente, o nome das constantes é escrito com letras maiúsculas para haver uma melhor distinção no código-fonte. Esta convenção não é obrigatória e não irá causar quaisquer erros durante a execução de um programa. Seguir esta convenção apenas torna mais clara a distinção entre variáveis e constantes dentro do código-fonte facilitando tanto a si, como programador, como a outros programadores que vejam o código do seu programa.
{{% /moreabout %}}

And this is moreabout.html shortcodes:

{{ .Get "title" }}

  {{ .Inner }}

And I'm getting this error:
ERROR: 2016/03/13 14:00:10 shortcode.go:266: Shortcode 'moreabout' in page 'fundamentos.markdown' has no .Inner, yet a closing tag was provided
ERROR: 2016/03/13 14:00:10 shortcode.go:266: Shortcode 'moreabout' in page 'fundamentos.markdown' has no .Inner, yet a closing tag was provided

What's wrong here?
Sorry if I shouldn't be asking this here.

Strange. I just stopped Hugo and restarted it and it works now.

Did you edit the shortcode while hugo server was running? This sounds like hugo wasn't reloading the shortcode after a change.

Yes, that was it.

So, this is a real bug then.

I'm using Windows.

I have had this curious experience lately that I have to start the server 2 times for the reload to just magically start working ... But that doesn't make sense ...

@spf13 I would appreciate it if you could chime and and tell me what your thinking behind this was:
https://github.com/spf13/hugo/blob/master/hugolib/site.go#L478
This seems to think that we only need to render the page content only if the content itself changes.

@bep
The logic around this section as I intended it should be as follows.
There are 3 possible conditions that will trigger a re-rendering of the entire site:

a template change
a data file change
a content file change

Each step has prep work that only applies to that specific change before the rendering can take place on https://github.com/spf13/hugo/blob/master/hugolib/site.go#L571.
The template change will prepare the templates https://github.com/spf13/hugo/blob/master/hugolib/site.go#L464
The data change will reload the data files
https://github.com/spf13/hugo/blob/master/hugolib/site.go#L471
The biggest task is to load/unload/replace the specific content files that changed
This could involve removing a file https://github.com/spf13/hugo/blob/master/hugolib/site.go#L518 / https://github.com/spf13/hugo/blob/master/hugolib/site.go#L529
or rereading one https://github.com/spf13/hugo/blob/master/hugolib/site.go#L534
Then the code is intended to run the conversion on that source file of processing the metadata and populating it in the page structure and converting the markup into html.
Once that has happened it will then render the entire site https://github.com/spf13/hugo/blob/master/hugolib/site.go#L571 with the updated content, metadata, data or templates as prepared above.
I suspect that this error may stem from shortcode processing happening during the markup conversion process, instead of after like all of the other templates. I haven't checked and don't remember clearly, but I bet that's what's happening here.
I did quite a bit of testing with templates and in changing them it did produce the expected results. I don't think I did testing of shortcode modifications though.
Does that make sense? If this is the case, the fix is to check if a changed template is a shortcode and if it is then perform the conversion steps prior to the render steps.

Does that make sense? If this is the case, the fix is to check if a changed template is a shortcode and if it is then perform the conversion steps prior to the render steps.

I have a fix for this (see PR somewhere), but I may have something smarter.

I saw the PR and read through it. I think it's doing work that doesn't need to be done, but I haven't dug into it enough to confirm that suspicion. I'm not clear why we need the extra Raw field yet. I'm curious what alternative you are thinking of.

I'm not clear why we need the extra Raw field yet. I'm curious what alternative you are thinking of.

The contentRaw field gets modified, so the shortcode is "lost" -- so if I just do a plain Convert, I end up with a page with shortcode placeholders.
The alternative to my PR would be to do a "stripped-down convert" that just re-renders the shortcodes (skips the extraction part). That is obviously smarter ...

@spf13 My calendar is a little busy these days. Maybe you would like to take a stab at a fix?


I don't  recommend to use this repository

I give you three reasons why not use this repository:

It is not easy to run example.py if you are new coder. you must change file path in some code file.
dropout_rate is not reasonable. as generally, we look forward to get same feature vector for inference a image twice, but we will get difference feature vector if you use dropout_rate under 1.
According to my practical experience, distance of my two face images is larger than distance of my face image and other person face image. so, the provided model is not practical.

how to change file path ?

I do think so, this model is not sutible for new coder. I didn't know what the meanning of the dropout_rate in the example.py at first.


User input

Take user input for number of images to be included - a check-box for "all" and a text box to enter a number

Take user input for number of images to be included - a check-box for "all" and a text box to enter a number

I gave the user a single textbox and the default for the textbox would be to include all images instead of having to use another check-box to do so. Also, I set the minimum number of images to use as 0, max number of images (arbitrarily) as 9001, and step number as 1.

Hey I found the error in the code.
Are you able to compile the code yourself? In that case I can point you to the error.
Otherwise I can compile it and send you the jar file.
Also, i think it's better to focus on the servlet at the moment. This can be fixed later.

Hey I found the error in the code.
Are you able to compile the code yourself? In that case I can point you to the error.
Otherwise I can compile it and send you the jar file.
Also, i think it's better to focus on the servlet at the moment. This can be fixed later.

Would you be able to tell me the instructions to compile it? I can try to compile the jar.


Create ✍ Authors “2022-04-28-28-36”

Automatically generated by Netlify CMS

👷 Deploy Preview for lighthearted-strudel-a80e1c processing.

Name
Link

🔨 Latest commit
c8062d1e0a8c6187aabdb3fab399f657ff33ec26

🔍 Latest deploy log
https://app.netlify.com/sites/lighthearted-strudel-a80e1c/deploys/626a9688cf4208000aa41fa0


[Move Only] Move wallet related things to src/wallet/

No code changes, just move wallet related files to /src/wallet/.

[ ]   General Comment
 moving is ok but it would make patch applying more different, I would not mind to move them if we want but that will require additional time for updating sources. What do other think?

This pull request is now managed with CodeReviewHub. Refresh.

moving is ok but it would make patch applying more different, I would not mind to move them if we want but that will require additional time for updating sources. What do other think?

This would make the sourcecode more intuitive, but as already said it would require a number of man-hours to do.


fix: allow f-string to trigger TRY003

Hello, and thank you for this package!
I stumble across the issue https://github.com/guilatrova/tryceratops/issues/31 and may have a potential fix.
The problem was false negative for rule TRY003, "Avoid specifying long messages outside the exception class", the root cause being the use of f-string in such messages.
The solution simply consists of checking whether the string is an f-string using ast.JoinedStr.
However, this fix is not perfect, as illustrated by the following example:
    elif a == 5:
        raise CustomException(f"code_{a}")  # This should be acceptable

This code triggers TRY003 with the fix, but it should be acceptable since there are no whitespaces, like in the existing test
    elif a == 3:
        raise CustomException("its_code_not_message")  # This is acceptable

The challenge is that, to my knowledge, there is no way to determine the content of an f-string during parsing without evaluating it first. It is probably beyond the scope of static analysis.
I think it is related to why Google discourages the use of f-string for logging in their styleguide. If f-strings are considered a bad practice for exceptions as well,, it might be worth adding a new violation rule in Tryceratops specifically for them.
Please let me know if you have any feedback or suggestions for improving this fix.

For the record, ruff implementation of TRY003 does produce less false positive than this fix. Their solution is to iterate over the str elements of the f-string and trigger a violation if it contains a whitespace:
a="0"
raise CustomException(f"code_{a}")  # This is accepted by ruff, as it should be
raise CustomException(f"message {a}")  # This is not accepted by ruff, as it should be

For the reason mentioned previously (ruff being a static linter as well), it can produce false negative in the following case:
a = " 0"  # there is a space here
raise CustomException(f"message{a}")  # This is accepted by ruff, but it shouldn't

Tested locally and it's fine.
Thank you :)


Triple Mustache between text nodes is rendered out of order

Description:
When a triple mustache is

after a normal text node
which are both together in a disabled section
which is followed by another text node

...toggling the section to be active causes the triple mustache to be rendered before its preceding text node, which is never correct based on the template source.
Versions affected:
0.9.6, 1.0.0-build-9, 1.0.0-build-85
Platforms affected:
Windows 10: Firefox 57.0b10, Chrome 61.0
Reproduction:
https://codepen.io/rockerest/pen/PJrVbx

Awesome turnaround, Chris. :tada:

There was an issue with rendering a triple after initial render related to finding the correct anchor for insertion. This is now fixed on edge and will go out with 0.9.7. Thanks for the report!

Thank you for the super quick fix. I saw that it was closed and thought I had submitted a duplicate or something. Great responsiveness there. Super impressed :)


Choice Field not polulated with correct values

This URL that is used to get the values is wrong: http://localhost:8080/Plone/@vocabularies/http://localhost:8080/Plone/@sources/type_of_talk?b_start=0
The response obviously is The vocabulary 'http:' does not exist

Closing as a duplicate of https://github.com/plone/volto/issues/2604


picard

Can I run picard as a standalone program as follows?
picard CollectInsertSizeMetrics I=Sons_Aligned_Bam_File.bam O=insertmetrics.out H=insertSizeHistogram.pdf
Do i have to write a yaml file for that run it in the pipeline as just one job? Any examples on this?

Hi Brad,
I tried to run the following
nohup picard CollectGcBiasMetrics R=Garvan_NA12878_HG001_HiSeq_Exome/GRCh37.fasta I=Sons_Aligned_Bam_File.bam O=output.txt CHART=gc_bias_metrics.pdf S=summary_metrics.txt VALIDATION_STRINGENCY=LENIENT TMP_DIR=/md1400/workTmp/ &
The tmp has 90T of disk space and still I get the following error
13:38:41.127 INFO  NativeLibraryLoader - Loading libgkl_compression.so from jar:file:/work/local/sha
re/bcbio/anaconda/share/picard-2.17.2-0/picard.jar!/com/intel/gkl/native/libgkl_compression.so
[Sat Feb 03 13:38:41 EST 2018] CollectGcBiasMetrics CHART_OUTPUT=gc_bias_metrics.pdf SUMMARY_OUTPUT=
summary_metrics.txt INPUT=Sons_Aligned_Bam_File.bam OUTPUT=output.txt TMP_DIR=[/md1400/workTmp] VALI
DATION_STRINGENCY=LENIENT REFERENCE_SEQUENCE=Garvan_NA12878_HG001_HiSeq_Exome/GRCh37.fasta    SCAN_W
INDOW_SIZE=100 MINIMUM_GENOME_FRACTION=1.0E-5 IS_BISULFITE_SEQUENCED=false METRIC_ACCUMULATION_LEVEL
=[ALL_READS] ALSO_IGNORE_DUPLICATES=false ASSUME_SORTED=true STOP_AFTER=0 VERBOSITY=INFO QUIET=false
COMPRESSION_LEVEL=5 MAX_RECORDS_IN_RAM=500000 CREATE_INDEX=false CREATE_MD5_FILE=false GA4GH_CLIENT
_SECRETS=client_secrets.json USE_JDK_DEFLATER=false USE_JDK_INFLATER=false
[Sat Feb 03 13:38:41 EST 2018] Executing as<EMAIL_ADDRESS>on Linux 3.10.0-693.17.1.el7.x86
_64 amd64; OpenJDK 64-Bit Server VM 1.8.0_121-b15; Deflater: Intel; Inflater: Intel; Picard version:
2.17.2-SNAPSHOT
INFO    2018-02-03 13:39:54     SinglePassSamProgram    Processed     1,000,000 records.  Elapsed ti
me: 00:00:09s.  Time for last 1,000,000:    4s.  Last read position: 1:45,797,650
INFO    2018-02-03 13:39:58     SinglePassSamProgram    Processed     2,000,000 records.  Elapsed ti
me: 00:00:13s.  Time for last 1,000,000:    3s.  Last read position: 1:146,456,399
INFO    2018-02-03 13:40:02     SinglePassSamProgram    Processed     3,000,000 records.  Elapsed ti
me: 00:00:17s.  Time for last 1,000,000:    3s.  Last read position: 1:186,076,038
[Sat Feb 03 13:40:06 EST 2018] picard.analysis.CollectGcBiasMetrics done. Elapsed time: 1.42 minutes
.
Runtime.totalMemory()=1060110336
To get help, see http://broadinstitute.github.io/picard/index.html#GettingHelp
Exception in thread "main" java.lang.OutOfMemoryError: Java heap space
at htsjdk.samtools.reference.FastaSequenceFile.readSequence(FastaSequenceFile.java:162)
at htsjdk.samtools.reference.FastaSequenceFile.nextSequence(FastaSequenceFile.java:83)
at htsjdk.samtools.reference.ReferenceSequenceFileWalker.get(ReferenceSequenceFileWalker.jav
a:93)
at picard.analysis.SinglePassSamProgram.makeItSo(SinglePassSamProgram.java:141)
at picard.analysis.SinglePassSamProgram.doWork(SinglePassSamProgram.java:84)
at picard.cmdline.CommandLineProgram.instanceMain(CommandLineProgram.java:269)
at picard.cmdline.PicardCommandLine.instanceMain(PicardCommandLine.java:98)
at picard.cmdline.PicardCommandLine.main(PicardCommandLine.java:108)

Murli;
It looks like you're running out of memory with the BAM file you're passing in. You can specify more memory for picard command line runs with:
picard -Xms1g -Xmx4g

at the start of your command. Hopefully adding that will help resolve the issue. Please let us know if you run into more problems.

Thank for your help. I shall try and let you know.


add double newlines in ADOPTERS.md

Using double newlines in markdown makes it easier to read in markdown readers, e.g github

Thanks @drornir this LGTM.

Codecov Report
Base: 56.56% // Head: 56.56% // No change to project coverage :thumbsup:

Coverage data is based on head (80c4cb7) compared to base (3629105).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3823   +/-   ##
=======================================
  Coverage   56.56%   56.56%
=======================================
  Files         105      105
  Lines       10638    10638
=======================================
  Hits         6017     6017
  Misses       3948     3948
  Partials      673      673

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@drornir could you please fix the DCO?

Yes I'm just a bit busy but I'll get around to it this weekend

ping @drornir

Thanks for the patience!