pip install -e . fails with "Module not found: Error: Cannot resolve module 'jupyterlab/lib/editorhandler/plugin' in /Users/grant/Sites/jupyter/jupyterlab/jupyterlab"

It seems like it fails at node scripts/dedupe.js with "Module not found: Error: Cannot resolve module 'jupyterlab/lib/editorhandler/plugin' in /Users/grant/Sites/jupyter/jupyterlab/jupyterlab":
Obtaining file:///Users/grant/Sites/jupyter/jupyterlab
    Complete output from command python setup.py egg_info:
    setup.py entered
    ...

    ><EMAIL_ADDRESS>prepublish /Users/grant/Sites/jupyter/jupyterlab
    > npm run build

    ><EMAIL_ADDRESS>build /Users/grant/Sites/jupyter/jupyterlab
    > npm run build:src

    ><EMAIL_ADDRESS>build:src /Users/grant/Sites/jupyter/jupyterlab
    > tsc --project src && node scripts/copyfiles.js

    npm WARN prefer global<EMAIL_ADDRESS>should be installed with -g

    ><EMAIL_ADDRESS>install /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/fsevents
    > node-pre-gyp install --fallback-to-build

    [fsevents] Success: "/Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/fsevents/lib/binding/Release/node-v48-darwin-x64/fse.node" is installed via remote

    ><EMAIL_ADDRESS>postinstall /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/phosphor-dragdrop
    > npm dedupe

    ><EMAIL_ADDRESS>postinstall /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/phosphor-keymap
    > npm dedupe

    ><EMAIL_ADDRESS>postinstall /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/phosphor-widget
    > npm dedupe

    ><EMAIL_ADDRESS>postinstall /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/phosphor-codemirror
    > npm dedupe

    ><EMAIL_ADDRESS>postinstall /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/phosphor-commandpalette
    > npm dedupe

    ><EMAIL_ADDRESS>postinstall /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/phosphor-menus
    > npm dedupe

    ><EMAIL_ADDRESS>postinstall /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/phosphor-panel
    > npm dedupe

    ><EMAIL_ADDRESS>postinstall /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/phosphor-boxpanel
    > npm dedupe

    ><EMAIL_ADDRESS>postinstall /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/phosphor-splitpanel
    > npm dedupe

    ><EMAIL_ADDRESS>postinstall /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/phosphor-stackedpanel
    > npm dedupe

    ><EMAIL_ADDRESS>postinstall /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/phosphor-tabs
    > npm dedupe

    ><EMAIL_ADDRESS>postinstall /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/phosphor-dockpanel
    > npm dedupe

    ><EMAIL_ADDRESS>postinstall /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/phosphide
    > npm dedupe

    ><EMAIL_ADDRESS>postinstall /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/jupyterlab
    > npm dedupe

    ><EMAIL_ADDRESS>postinstall /Users/grant/Sites/jupyter/jupyterlab/jupyterlab
    > npm dedupe

    ...

    ><EMAIL_ADDRESS>build /Users/grant/Sites/jupyter/jupyterlab/jupyterlab
    > npm update jupyterlab && webpack --config webpack.conf.js

    ><EMAIL_ADDRESS>prepublish /Users/grant/Sites/jupyter/jupyterlab
    > npm run build

    ><EMAIL_ADDRESS>build /Users/grant/Sites/jupyter/jupyterlab
    > npm run build:src

    ><EMAIL_ADDRESS>build:src /Users/grant/Sites/jupyter/jupyterlab
    > tsc --project src && node scripts/copyfiles.js

    ><EMAIL_ADDRESS>postinstall /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/jupyterlab
    > node scripts/dedupe.js

    -<EMAIL_ADDRESS>node_modules/bootstrap
    -<EMAIL_ADDRESS>node_modules/d3-format
    -<EMAIL_ADDRESS>node_modules/jquery-ui
    -<EMAIL_ADDRESS>node_modules/jupyter-js-utils
    -<EMAIL_ADDRESS>node_modules/jupyter-js-widgets/node_modules/backbone
    -<EMAIL_ADDRESS>node_modules/lolex
    -<EMAIL_ADDRESS>node_modules/phosphor-codemirror
    -<EMAIL_ADDRESS>node_modules/phosphor-menus
    -<EMAIL_ADDRESS>node_modules/phosphor-observablelist
    -<EMAIL_ADDRESS>node_modules/sanitizer
    -<EMAIL_ADDRESS>node_modules/scriptjs
    -<EMAIL_ADDRESS>node_modules/jupyter-js-widgets
<EMAIL_ADDRESS>/Users/grant/Sites/jupyter/jupyterlab/jupyterlab
    ├──<EMAIL_ADDRESS>invalid
    └──<EMAIL_ADDRESS>
    ModuleNotFoundError: Module not found: Error: Cannot resolve module 'jupyterlab/lib/editorhandler/plugin' in /Users/grant/Sites/jupyter/jupyterlab/jupyterlab
        at /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/webpack/lib/Compilation.js:229:38
        at onDoneResolving (/Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/webpack/lib/NormalModuleFactory.js:29:20)
        at /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/webpack/lib/NormalModuleFactory.js:85:20
        at /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/async/lib/async.js:726:13
        at /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/async/lib/async.js:52:16
        at done (/Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/async/lib/async.js:241:17)
        at /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/async/lib/async.js:44:16
        at /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/async/lib/async.js:723:17
        at /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/async/lib/async.js:167:37
        at /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/enhanced-resolve/lib/UnsafeCachePlugin.js:24:19
        at onResolved (/Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/enhanced-resolve/lib/Resolver.js:38:18)
        at innerCallback (/Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/enhanced-resolve/lib/Resolver.js:94:11)
        at loggingCallbackWrapper (/Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/enhanced-resolve/lib/createInnerCallback.js:21:19)
        at /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/tapable/lib/Tapable.js:134:6
        at /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/enhanced-resolve/lib/ModulesInDirectoriesPlugin.js:54:23
        at /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/enhanced-resolve/lib/Resolver.js:191:15
        at /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/enhanced-resolve/lib/ModulesInDirectoriesPlugin.js:45:26
        at loggingCallbackWrapper (/Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/enhanced-resolve/lib/createInnerCallback.js:21:19)
        at /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/tapable/lib/Tapable.js:134:6
        at /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/enhanced-resolve/lib/Resolver.js:122:33
        at /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/enhanced-resolve/lib/Resolver.js:191:15
        at applyPluginsParallelBailResult.createInnerCallback.log (/Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/enhanced-resolve/lib/Resolver.js:110:4)
        at loggingCallbackWrapper (/Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/enhanced-resolve/lib/createInnerCallback.js:21:19)
        at /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/tapable/lib/Tapable.js:134:6
        at Tapable.<anonymous> (/Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/enhanced-resolve/lib/DirectoryDescriptionFilePlugin.js:24:12)
        at Storage.finished (/Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/enhanced-resolve/lib/CachedInputFileSystem.js:38:16)
        at ReadFileContext.callback (/Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/graceful-fs/graceful-fs.js:78:16)
        at FSReqWrap.readFileAfterOpen [as oncomplete] (fs.js:365:13)
    resolve module jupyterlab/lib/editorhandler/plugin in /Users/grant/Sites/jupyter/jupyterlab/jupyterlab
      looking for modules in /Users/grant/Sites/jupyter/jupyterlab/node_modules
        /Users/grant/Sites/jupyter/jupyterlab/node_modules/jupyterlab doesn't exist (module as directory)
      looking for modules in /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules
        resolve 'file' or 'directory' lib/editorhandler/plugin in /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/jupyterlab
          resolve file
            /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/jupyterlab/lib/editorhandler/plugin doesn't exist
            /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/jupyterlab/lib/editorhandler/plugin.webpack.js doesn't exist
            /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/jupyterlab/lib/editorhandler/plugin.web.js doesn't exist
            /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/jupyterlab/lib/editorhandler/plugin.js doesn't exist
            /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/jupyterlab/lib/editorhandler/plugin.json doesn't exist
          resolve directory
            /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/jupyterlab/lib/editorhandler/plugin doesn't exist (directory default file)
            /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/node_modules/jupyterlab/lib/editorhandler/plugin/package.json doesn't exist (directory description file)

    npm ERR! Darwin 16.0.0
    npm ERR! argv "/usr/local/bin/iojs" "/usr/local/bin/npm" "run" "build"
    npm ERR! node v6.3.0
    npm ERR! npm  v3.10.3
    npm ERR! code ELIFECYCLE
    npm ERR<EMAIL_ADDRESS>build: `npm update jupyterlab && webpack --config webpack.conf.js`
    npm ERR! Exit status 1
    npm ERR!
    npm ERR! Failed at the<EMAIL_ADDRESS>build script 'npm update jupyterlab && webpack --config webpack.conf.js'.
    npm ERR! Make sure you have the latest version of node.js and npm installed.
    npm ERR! If you do, this is most likely a problem with the jupyterlab-extension package,
    npm ERR! not with npm itself.
    npm ERR! Tell the author that this fails on your system:
    npm ERR!     npm update jupyterlab && webpack --config webpack.conf.js
    npm ERR! You can get information on how to open an issue for this project with:
    npm ERR!     npm bugs jupyterlab-extension
    npm ERR! Or if that isn't available, you can get their info via:
    npm ERR!     npm owner ls jupyterlab-extension
    npm ERR! There is likely additional logging output above.

    npm ERR! Please include the following file with any support request:
    npm ERR!     /Users/grant/Sites/jupyter/jupyterlab/jupyterlab/npm-debug.log
    /Users/grant/anaconda/lib/python3.5/distutils/dist.py:261: UserWarning: Unknown distribution option: 'License'
      warnings.warn(msg)
    rebuilding js and css failed
    missing files: ['/Users/grant/Sites/jupyter/jupyterlab/jupyterlab/build/bundle.js']
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/Users/grant/Sites/jupyter/jupyterlab/setup.py", line 161, in <module>
        setup(**setup_args)
      File "/Users/grant/anaconda/lib/python3.5/distutils/core.py", line 148, in setup
        dist.run_commands()
      File "/Users/grant/anaconda/lib/python3.5/distutils/dist.py", line 955, in run_commands
        self.run_command(cmd)
      File "/Users/grant/anaconda/lib/python3.5/distutils/dist.py", line 974, in run_command
        cmd_obj.run()
      File "/Users/grant/Sites/jupyter/jupyterlab/setup.py", line 51, in run
        raise e
      File "/Users/grant/Sites/jupyter/jupyterlab/setup.py", line 44, in run
        self.distribution.run_command('jsdeps')
      File "/Users/grant/anaconda/lib/python3.5/distutils/dist.py", line 974, in run_command
        cmd_obj.run()
      File "/Users/grant/Sites/jupyter/jupyterlab/setup.py", line 109, in run
        check_call(['npm', 'run', 'build'], cwd=node_root, stdout=sys.stdout, stderr=sys.stderr)
      File "/Users/grant/anaconda/lib/python3.5/subprocess.py", line 581, in check_call
        raise CalledProcessError(retcode, cmd)
    subprocess.CalledProcessError: Command '['npm', 'run', 'build']' returned non-zero exit status 1

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /Users/grant/Sites/jupyter/jupyterlab/

Duplicate of https://github.com/jupyter/jupyterlab/issues/1057


fix: fixes for source tracing [APE-972]

What I did
Fixes several things that came up from people trying to use the source tracing feature....

Add ability to check if compiler supports it (will need this in Coverage feature anyway)
Performance improvement by preventing the calculation of the source traceback more than once in certain situations
Critical issue causing never-ending loop (hanging) when trying to create the source traceback. For this reason, we may want to patch Core ape after this...
Better error when txn hash is not found
Issue where contracts_folder was None when it should not have been (only happened when using normal Python REPLs).
PC values are now present in the source traceback but also requires that ape-vyper PR

How I did it

How to verify it

Checklist

[ ] All changes are completed
[ ] New test cases have been added
[ ] Documentation has been updated

[ ] tests


Make daily blocks run sequentially

This PR divides the list of blocks in 8 lists of 5 blocks each. It creates 8 workers, each of which depends on the previous to begin execution. It will hopefully avoid both the github and the juno's timeouts.
Checklist

[ ] Linked to Github Issue
[ ] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 80.66%. Comparing base (4bd59e9) to head (be5c17c).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #976   +/-   ##
=======================================
  Coverage   80.66%   80.66%
=======================================
  Files         107      107
  Lines       29325    29325
=======================================
  Hits        23656    23656
  Misses       5669     5669

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@FrancoGiachetta
Although I haven't tested it, we may be able to use the max-parallel option, instead of doing it manually:

https://docs.github.com/en/actions/writing-workflows/workflow-syntax-for-github-actions#jobsjob_idstrategymax-parallel

Thanks! Didn't know of that one, I'll try it.

Done


cmdlets are broken because of the path issue

cmdlets are broken because of the path issue
Cannot find path 'C:\graphql\getCloudConfigurationRule.graphql' because it does not exist.

Issue is fixed with the version 0.0.8 (Removed all old versions)

closing the issue! Version 0.0.8 is stable!


Support 'rubric' metadata to show info on a quiz's front page

Pull Request Check List

[ ] Unit Tests & Regression Tests Added (Optional)
[x] Removed Unnecessary Logs/System.Outs/Comments/TODOs
[ ] Peer-Reviewed

Do we need an associated front end change (at least regenerating/ adding ruberic to IsaacApiTypes) so it recognises the changes to IsaacQuizDto?

Did I forget to make the change? Just adding the field should do, anyway.

You made the changes, I didn't see the other PRs, my bad! All good!


NodeJS and v6.11.2/Hello

Hello,
My name is Seth. I could not get started because I have NodeJS v6.11.2 instead of the prerequisite. Because of this version prerequisite not met, I cannot get nodejs-dev.
I am stuck at the beginning. Do you have any recommendations?
Seth

Just apt install *


Blobs are deleted after compaction even if still valid

Here's a code I'm using to create blobs,. The blob is added to the document and the file is created in the attachment folder.
However when I do a compaction (using CBLDatabase_Compact)  the file is deleted from the attachment folder.
The document still has the blob dictionary.
I'm working on a Dart binding, but I have this test C code that has the same problem.
CBLBlob* CBLBlob_CreateWithData_c(
    CBLDatabase *db _cbl_nonnull,
    const char *contentType,
    unsigned char *contents,
    uint64_t length,
    CBLError* outError
    ) CBLAPI
{
    CBLBlob* blob = CBLBlob_CreateWithData(contentType, alloc_slice(contents, length));

    CBLDocument* doc = CBLDatabase_GetMutableDocument(db, "testdoc3");

    MutableDict props = doc->mutableProperties().mutableCopy();

    FLSlot sl = FLMutableDict_Set(props, slice("logo"));
    FLSlot_SetBlob(sl, blob);

    doc->setProperties(props);
    doc->save(db, {kCBLConcurrencyControlLastWriteWins}, internal(outError));

    return blob;
}

Another issue I'm having is that alling "FLSlot_SetBlob(sl, blob)" if the document already has the key gives me this error:
FAILED PRECONDITION: index<_items.size() not true when calling class fleece::impl::ValueSlot &__cdecl fleece::impl::internal::HeapArray::setting(unsigned int) (at HeapArray.cc line 129)
I tried all kinds of ways to get the mutable properties, create a copy, remove the previous key, it always gives me this error. The only workaroudn I could find is delete the whole document and recreate it.

I tried compacting a database created by the Android SDK, and blobs are not deleted so clearly compacting is not the issue. Some required metadata is probably not saved.
This is the output of the database I'm creating blobs using the C library. I see that the revs command does not show that the document has attachments.
(cblite) cat testdoc1
{
  "_id": "testdoc1",
  "foo": "barzz",
  "logo": {
    "@type": "blob",
    "content_type": "image/png",
    "digest": "sha1-tW93BqpY0aIUwPnR6k0cW3DYZU8=",
    "length": 1079275
  }
}
(cblite) revs testdoc1
Document "testdoc1"
 * 30-38da8e673cd2e75ee9c32b45fdd9c02f8afe1d28    ------ #143
   * 31-d38436b08e4b2eb579b40cb2433f6ee4929ca57f  ------ #145
     * 32-dbf935122703d13b6042663647b23b01898a60fc------ #147
.....

And this from the database created on Android
(cblite) cat blobtest
{
  "_id": "blobtest",
  "logo": {
    "@type": "blob",
    "content_type": "image/jpg",
    "digest": "sha1-OllVnWkt36lpI8/aoQjOlGMGtdc=",
    "length": 15499
  },
  "logo1": {
    "@type": "blob",
    "content_type": "image/jpg",
    "digest": "sha1-8gi9kfKfXrSMwCWBwLEKsKMDY0w=",
    "length": 3944
  }
}
(cblite) revs blobtest
Document "blobtest", Has Attachments
 * 1-581ad726ee407c8376fc94aad966051d013893c4     ------ #1
   * 2-e859e667208d6f59d4719a7d780dfd40c45706bf   ------ #2
....
```

Interesting ... the 'has attachments' flag should get set automatically in the core code (LiteCore). I'm not sure why it isn't in this case; I'll have to take a look.

I added a unit test for this situation (compact db after adding a blob), so I'm pretty sure you should be OK now. Thanks for finding this.


Typescript chill-out, Qualification display fix, contribution always on

When showing the output of a qualification, we wrap it in a  so that you can see newlines be rendered.
We want to always show the contribute button, because otherwise we can never see it.
Typescript is now configured to be much more relaxed, and to have an 'any' type for _ while we figure out how to get the types for lodash loaded up.

bors merge


investigate patterns to avoid v8 deopt by borrowing from backbone :D

instead of function a, function a1, function a2 ... maybe something like this:
https://github.com/jashkenas/backbone/blob/678aeee94d7802967b52ed8a9177620659b825ee/backbone.js#L331-L338

@webpack-bot move to webpack/tapable


Fix Typos in Sending RFQs Page

What type of PR is this? (check all applicable)

[ ] ♻ Refactor
[ ] ✨ New Feature
[ ] 🐛 Bug Fix
[x] 📝 Documentation Update
[ ] 👷 Example Application
[ ] 🧑💻 Code Snippet
[ ] 🎨 Design
[ ] 📖 Content
[ ] 🧪 Tests
[ ] 🔖 Release
[ ] 🚩 Other

Description
Fix Typos in Sending RFQs Page
This PR fixes this #1592
Related Tickets & Documents
Resolves #1592
Added code snippets?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed

Added tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 readme
[ ] 📜 contributing.md
[ ] 📓 general documentation
[x] 🙅 no documentation needed

@EbonyLouis Can you please add hacktoberfest-accepted label?


Add UseNocanon to mod_lbmethod_cluster, fix testsuite to use the right module

close #297
close #298
This resolves two issues, but given that they are connected it seemed reasonable to do it at once. Test that ought to be run with balancer were unfortunately run with cluster. The testsuite needed a few changes and new conf files for the second module.

Thanks for the review


TypeScript Functional Components

Hi! It would be great if the types allowed React.FunctionComponent to be the argument of functions like withAuth and others.

#12

All of the concepts of this library have been moved to the official Auth0 SDK for React (https://github.com/auth0/auth0-react), so I'll be archiving this repository shortly.


[FEATURE] getting this error in NAM Failed checks!  In no coder is this something i need to change somewhere? thanks

Thanks for taking the time to write a feature request! Use the following prompts to help you describe what you're looking for.  The more info you provide, the easier it'll be for me to understand what I can do to make it happen for you without having to come back and ask you questions.
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

So, what you should do first is check out what's going wrong--you may have bad data, and, as they say, "Garbage in, garbage out"! 😉
But, if you think that the checks don't know what they're talking about and you want to try anyways,
Then take this...

and make it like this:

Good luck!


Wrapper code for app.ts and server.ts under typescript style classes

The PR is created in response to the issue - https://github.com/Microsoft/TypeScript-Node-Starter/issues/134
This starter project is lacking typescript OOP design. Code should at-least be wrapped under classes. The idea is to make this starter project a true typescript express project. I have started by modifying app.ts and server.ts along with the test cases. All test cases are getting passed.
Next steps will be to wrap controllers under class and refactor the application more in an OOP way.

 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.:x: satya-jugran sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks @welkie. You are right. I just did some initial changes. Its still lacking better design. I try to cover all those along with your considerations.

@welkie I did the necessary changes from my end. Please review and let me know if this PR is good to go for merge.
Changes :

Every code is wrapped under classes.
"@app" decorator configures the required express configurations for App Class.
App class is made singleton. Only one instance should be there to start for server.
Routes have been moved to their respective controller methods. A controller method can be decorated with @route decorator, where http method type, URI and other middlewares can be added.
Database initialization refactored to separate file.
Custom middleware will not pollute App. All custom middlewares will come under /middlewares folder.
The actual "express" instance needs to be exposed via property because of the limitation of SuperTest request method which requires an actual express app instance as a parameter. The test should not take any express instance, rather it should just make an http request to already up and running instance. If we find some alternative, we can hide the actual express under App class only.

All tests are passed and I have tested the app manually and everything looks good.

I think it's great that you've seen something that can be improved and worked to make it happen. Good on you. 👍

Totally agree, good initiative! 💯

@bowdenk7

hello @satya-jugran. I used your code to my project. But, my "this" variable alway return undefined in controller. How do i fixed? Thanks

Hi folks, I'm going through all the open PRs - I don't think this is what we should probably be merged.  Which is kinda how #134 ended up.
I mentioned this in https://github.com/microsoft/TypeScript-Node-Starter/pull/21#issuecomment-503760181 I think class based infra can make sense for an Express app, but I'm not sure these use cases for a simple starter pack where functions are easier and closer to how the official express docs show how to build an app.
So, thanks for the effort - and anyone interested in this pattern, you can totally use this PR as the foundations for your apps but for now I'm going to close.


transform pipelines accept items castable as xs:string as value for relative URI

Refactor a new xsl:template that transforms a sequence of items into a sequence of maps with stylesheet locations.
It has to deal with situations where the context item is absent, so resolving a relative URI against base-uri(/) wont work.

revoked because of bad credentials error


Invoking Multiple services error

Bug or feature request?
Bug
Replicated here: https://codesandbox.io/embed/heuristic-mountain-1kh0n
Please note I made the login a random boolean, so the error path is triggered when it randoms false
Description:
I am invoking 2 services in a state. The first service is a promise and the second one is a machine. I need when the onError action hits from the promise service to send an action to the child service.
(Bug) Expected result:
I would expect that the child machine action would be called
(Bug) Actual result:
index.js:1375 Missing onError handler and/or unhandled exception/promise rejection for invocation 'login'. Original error: '[object Object]'.  Current error is 'Error: Unable to send event to child 'notification' from service 'authentication'.'. Stacktrace was 'Error: Unable to send event to child 'notification' from service 'authentication'.
(Bug) Potential fix:
Required! If you found a bug and can consistently reproduce it, please take a minute to think about what the potential fix might be.
In the attached code sandbox I commented out line 97. To fix the issue you must remove the "reportError" action from line 92 and add back line 97. The reason this is not the ideal solution is that 97 will always be triggered and I will have to add logic to tell whether it is an error or not while if it was triggered on line 92, it will only be triggered on an error.
Link to reproduction or proof-of-concept:
Replicated here: https://codesandbox.io/embed/heuristic-mountain-1kh0n

There is basically a "race condition" here. With the given machine config, you are doing two things at the same time:

Stopping the notification service
Sending an event to the notification service

This should be a ~code smell~ red flag. Why does the notification service need to be stopped?
Personally, I'd refactor this so that the notification service is invoked throughout the lifetime of the machine:
const authMachine = Machine({
  // ...
  invoke: {
    src: notificationMachine,
    id: 'notification'
  },
  states: {
    // ...
  }
  // ...
});

See here for a working example: https://codesandbox.io/s/vigilant-torvalds-u4pvm


[FLINK-21938][docs] Add how to unit test python udfs

What is the purpose of the change
Let users know how to unit test their Python scalar UDF.
Brief change log
Document how to unit test Python scalar UDF.
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

No.
Documentation

Does this pull request introduce a new feature? (yes / no) no
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@rmetzger Hi Robert, is there anything I need to do here?

No, we have to wait for a reviewer to show up. @HuangXingBo could you take a look?

@dianfu Hi Dian, can you please take a look? Thanks

Sure. Very sorry for late response.

@dianfu Feel free to let me know if you have any comments

@YikSanChan I'm considering adding another page to hold this information instead of putting it inside ScalarFunction as it applies also to other kinds of user-defined functions. Besides the unit test, the page could also contain the other informations about user-defined function, e.g. logging, how to load a large data set, etc.

@dianfu I agree with the proposal, though I don't have much experience on other udfs as well as logging/loading large dataset etc, therefore no sure what to add, correctly. Given this, do you like me to add something, or maybe we can move forward with this one (since the little-doc is better than no-doc)?

@YikSanChan What about rebase this PR after that page is added. I can add that page. Will let you know when it is ready. Thank you a lot~

@dianfu Sure, that sounds good! Let me know when you add the page. 👍


1.8 client doesnt connect to proxy

I got an 1.17 and an 1.8 server both with the same default config both say they are connected but the 1.8 server just doent connect i just dont understand why the 1.8 server is java 17 and sk 2.6.1

Using Skungee 2.0.0 doesn't disallow client connects. That'll be your proxy not Skungee if using 2.0.0+


[μsvg] convert SVG 2.0 to micro SVG 1.1

My library/framework/language/software doesn't have a good SVG library, and only supports the current SVG 1.1-Standard.
I would like to convert SVG 2.0 to micro SVG 1.1 (as much as possible) to be compatible with my library/framework/language/software.
I would like to have a option to output a micro SVG 1.1 (I don't mind if it is buggy/slow, but the most common features should be implemented).

I don't mind if it is buggy/slow

I do.
I will try to flatten alpha. Will see if it would work.

I don't mind if it is buggy/slow

I do.

Sorry for writing unclear: I meant the default output (SVG 2.0) should be fast and almost bug-free (especially for resvg), but I prefer buggy&slow SVG 1.1-convertion than having nothing (So Conversion to SVG 1.1 could be a post-process.)

confirm fix, using cargo install usvg resvg --force --git https://github.com/RazrFalcon/resvg/


When will "4.0.0-RC3" be released to the maven central repository?

When will "4.0.0-RC3" be released to the maven central repository?

Please subscribe dev mail group to get more information


Adding unit test to verify grammar against known-good test data

This is just a copy-paste of our internal logic.
Antlr insists that the grammar name be the same as the grammar file minus the .g4 extension, so I've renamed the file to remove the .txt extension. The negative side effect is that when you try to view the grammar on our website you'll get a download instead of just viewing it inline since it's no longer considered text. A follow-up change to augment the mime type database used by Github Pages will fix the issue.

Equivalent functionality ended up being merged in #61. Closing.


[ENG-7069][eas-cli] Use pagianted select prompt for eas build:run command

Checklist

[ ] I've added an entry to CHANGELOG.md if necessary. You can comment this pull request with /changelog-entry [breaking-change|new-feature|bug-fix|chore] [message] and CHANGELOG.md will be updated automatically.

Why
https://linear.app/expo/issue/ENG-7069/add-pagination-to-eas-buildrun-command
How
Use the paginated list and select function created by @wkozyra95 for the build:resign command https://github.com/expo/eas-cli/pull/1575
Test Plan
Test build:run locally

/changelog-entry chore Use paginated select prompt for the eas build:run command


Index is not reading correctly the security levels.

The index.php file use a Sesion_Start() Function to check if there is an open session, but it is not assigning correctly the destination path for each profile.
Must use the same code used in the login page:
if($_SESSION["logged_in_user"]["App_Users_SecurityLevel"] >= 9)     {       $page = "sup_dashboard.php";     }     else if($_SESSION["logged_in_user"]["App_Users_SecurityLevel"] >= 5)     {       $page = "sup_dashboard.php";     }     else if($_SESSION["logged_in_user"]["App_Users_SecurityLevel"] >= 1)     {       $page = "oper_dashboard.php";     }
This is to assign destination pages in the following way:

=9 sup_dashboard.php

=5 sup_dashboard.php

=1 oper_dashboard.php

I need it this way becasue in the near future we will need to include an administrator dashboard and other funtions that require diferenciation.

Dilip, originally, the index.php file had only 2 alternatives. Send you back to the login page or send you to the supervisor dashboard.
As we already include user level verification this causes a problem in some cases for the regular operator. When they are logged in and open a new window, the index.php try to send them to the supervisor dashboard and then sent back to the login page when this is not necessary.
We need to modify the index.php to read correctly the user level of the PHP session and then send the user to the proper dashboard. The example is already on this queue so you can get it easily spot.

This task done

This is ready.


When onPageload, confirm before execute is selected as NO, inappropriate error is thrown in the alert .Instead of Showing failed to execute actions during page load , can we show Query1 was cancelledThis is happening in case of Query module instances only and not JS module instances

https://www.notion.so/When-onPageload-confirm-before-execute-is-selected-as-NO-inappropriate-error-is-thrown-in-the-aler-7b4bfe8f3d59441298ac0f8cd805fc22

this has been fixed!


ISPN-15042 Do not collect clustered stats in some conditions

https://issues.redhat.com/browse/ISPN-15042

@ryanemerson I've added you as a reviewer because you worked on the cluster metrics collection

I've added you as a reviewer because you worked on the cluster metrics collection

Now that seems like a long time ago!
Changes LGTM.


Datpicker button positioned wrong in firefox with outline appearance

Bug, feature request, or proposal:
Bug. All mat-icon-buttons used a prefix/suffix in a mat-form-field with the outline appearance are positioned incorrectly in firefox.
What is the expected behavior?
The button should be positioned inline with the input, as is seen on other browsers
What is the current behavior?
The button is positioned too high
What are the steps to reproduce?
https://stackblitz.com/edit/angular-firefox-datepicker-outline-display-bug?file=app/datepicker-overview-example.css
Is there anything else we should know?
The stackblitz contains a line of css that fixes the issue in my limited testing.

Duplicate pf #12803


feat: support for cython

Image doesn't really fit with other icon, Could you read this please?
It's fine if you can't do it, you can add the language to missing icon list.


tools/ci: Bump Bazel version

Thanks, but we no longer use these scripts for CI. They'll be removed soon.


Dev forecast derivation agent

This PR converts the ForecastingAgent into a derivation agent using the Derived Information Framework's DerivationWithTimeSeries concept.

I addressed most of your comments and further elaborated the documentation to ensure that overwriting existing forecasts is the default behaviour and mostly in line with DIF.
Regarding the support for om:Quantity vs owl:Thing, this is my point of view:

The current implementation supporting owl:Thing is in line with the current time series ontology. As pointed out by Seb, not everybody uses OM; hence, this gives us more flexibility
Using owl:Thing does not cause issues when retrieving derivation inputs. As the agent handles only one instance per rdf type, it is possible to unambiguously derive the target IRI to forecast: even if any other derivation input also has an owl:Thing relationship attached, this particular IRI will also be retrieved for the more specific input type (e.g. ForecastingModel). So, the only IRI which only appears in the input list for owl:Thing is the IRI to forecast. Please see the validate_input_values method for details.

Hence, I believe we should keep the current implementation with om:Quantity as default, but also more general support for owl:Thing.
Lastly, @sm453 have you been able to spin up the agent as part of the stack-manager by now? I just tried again on my end and it works without issues.

Hence, I believe we should keep the current implementation with om:Quantity as default, but also more general support for owl:Thing.

I suppose this is fine for the moment, could you please also add the discussion to the documentation of the codes?


Patches cannot be decoded using utf-8

I'm not able to read patches for Jsoup 52, Compress 7, and Lang 25 using utf-8. I get the following error:
  File "/home/tschweiz/.pyenv/versions/3.8.15/lib/python3.8/codecs.py", line 322, in decode
    (result, consumed) = self._buffer_decode(data, self.errors, final)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xe4 in position 13981: invalid continuation byte

This only happens on these patches. No other bug patches have this issue. Is it the intended behavior because these patches have special characters or is the encoding incorrect?
Replication

Install Python package (https://pypi.org/project/unidiff/)[unidiff]
Read patch with unidiff: PatchSet.from_filename(path/to/patch)

Hi @Thomsch,
The bug-mining procedure collects the diffs as they were in the history of the repository (git, svn).  Whether unidiff is or is not able to parse those out-of-the-box is out of Defects4J's scope, in my opinion.
--
Best,
Jose

Hi @jose, thank you for the reply. I think that is a fair assessment 👍.
We solved this issue by reading the diffs using the latin-1 encoding if someone ever has this issue in the future.


Provide P2 update site for automatic update

Currently the update mechanism of DBeaver is cumbersome. Eclipse E4 provides a standard way to let the RCP app update itself via P2 update site
see https://wiki.eclipse.org/Equinox/p2/Adding_Self-Update_to_an_RCP_Application or http://www.vogella.com/tutorials/EclipseP2Update/article.html

Generally we have P2 update site for Eclipse plugin version (https://marketplace.eclipse.org/content/dbeaver). However it doesn't work for standalone application right now.
A few version ago (3.x) we have tried to use the same approach in standalone version but there were a few issues. Because of massive plugins versions update on each release and some dependencies mess P2 update didn't work as expected.
We are going to try again later in 4.x.

Our standard update site https://dbeaver.io/update/latest/ must work in newer DBeaver versions

Our standard update site https://dbeaver.io/update/latest/ must work in newer DBeaver versions


Added Jenkins name and description

Can one of the admins verify this patch?

[test]

The tests will fail because your PR to the jenkinsfile-library repository was not merged yet. I suggest you to change the build to read from your forked repository temporarily until the PR is merged

You can find how to squash your commits into a single one in step 10 here: https://github.com/forge/core/blob/master/CONTRIBUTING.asciidoc

Also last but not least try to redact the commit message in this format: http://karma-runner.github.io/2.0/dev/git-commit-msg.html

Fixes #173 #177

[test]

[test]

@animuk the changes introduced by your PR in the Jenkinsfile library may have broken the legacy code: https://github.com/fabric8-launcher/launcher-backend/blob/master/addons/legacy-osio-addon/src/main/java/io/fabric8/forge/generator/pipeline/JenkinsPipelineLibrary.java
Can you make sure that points that to a fixed tag?

[test]


Bug in console on upgrading to  1.7.1-1

vue-autosuggest.esm.js?2831:1 Uncaught TypeError: Cannot read property 'clientWidth' of null
    at VueComponent.clickedOnScrollbar (vue-autosuggest.esm.js?2831:1)
    at VueComponent.onDocumentMouseUp (vue-autosuggest.esm.js?2831:1)
clickedOnScrollbar @ vue-autosuggest.esm.js?2831:1
onDocumentMouseUp @ vue-autosuggest.esm.js?2831:1

Fixed in v1.7.1-2
Thanks @stripathix !


Upgrade Play! version from 2.6.0-M4 to 2.6.0-M5

Changes only in ebean enhancer.
Migration from play-ebean 4.0.0-M2 to 4.0.0-M3 requires Ebean enhancement agent dependency version change from 10.1.2 to 10.1.7.

New 1.0.0-beta8-SNAPSHOT snapshot deployed.

Documentation updated.


cmd/ld: Build (go build) does not produce identical binaries

As the title indicates, running go build on the same machine with no changes does not produce identical binaries on Windows even though it does on Linux/BSD.
To see this in action, run the following one-line command on Windows and Linux/BSD:
$ mkdir $GOPATH/src/unstable; cd $GOPATH/src/unstable; echo "package main; func main() {}" > main.go; go build -o a; go build -o b; openssl sha a b

Output on Windows:

SHA(a)= 5d5a08499e18e2475fb51d72eee2100afb8ec277
SHA(b)= 49a016bf798c8a3c0fa88f06372303dd4b3f224f

Output on Linux:

SHA(a)= 28f40782581aece883589efaa5dc5e269a46b71d
SHA(b)= 28f40782581aece883589efaa5dc5e269a46b71d

Output on OpenBSD:

SHA(a)= 962127cf3e97dd22aa8a879c9be58a9643ede0ae
SHA(a)= 962127cf3e97dd22aa8a879c9be58a9643ede0ae

Further, I've compared the output of strings between the two binaries and there is no difference, but by doing $ cmp --verbose --print-chars a b, it appears to be a time stamp that is the cause.

Issue submission questions:

What version of Go are you using (go version)?
What operating system and processor architecture are you using?

$ go version
go version go1.4.1 windows/amd64

What did you do?

I ran the commands provided above.

What did you expect to see?

The generated binary should be identical and therefore have the same hash on Windows like it does on Linux and BSD.

What did you see instead?

The generated binaries are not identical on Windows although they are on Linux/BSD.

It's this line in cmd/ld/pe.c:
fh.TimeDateStamp = time(0);
I will let the Windows folks decide, but I think we should simply remove that line.  It was a mistake for COFF to put a timestamp into the object file format, and it was a mistake for PE to copy it.   Every time you
put it back, I will come in and take it out again.  I'm sorry.  This
field does not belong here.  We fill it with a 0 so it compares the
same but is not a reasonable time. --<EMAIL_ADDRESS>*/
internal_f.f_timdat = 0;

I've been planing to do get rid of the timestamp for a long time.
https://go-review.googlesource.com/3740


syncer(dm): fix failed row skipped due to incorrect checkpoint flush

What problem does this PR solve?

Issue Number: close #5279
What is changed and how it works?

checkpoint flush should wait all dml before it finished.

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Code changes

Has exported function/method change
Has exported variable/fields change
Has interface methods change
Has persistent data change

Side effects

Possible performance regression
Increased code complexity
Breaking backward compatibility

Related changes

Need to cherry-pick to the release branch
Need to update the documentation
Need to update key monitor metrics in both TiCDC document and official document

Release note
`None`.

/cc @lance6716 @GMHDBJD

/run-dm-integration-test

/run-dm-compatibility-test
/run-integration-test
/run-kafka-integration-test

/merge


Errors guidance

What do you need?
Guidance on errors, for example 404s.
There'd be a cross over with UI-KIT content.
See also: https://github.com/AusDTO/gov-au-content-guide/issues/160

And form/field error messages? 🙂

Now being tracked in Trello


WTFcsv: consider increasing the size limit

@rahulbot my students often work with csv data that is larger than 10MB and I think a ton of journalists will as well. What do you think of the following as an experiment -

we up the limit to 100MB (or no limit which would be my preference for teaching - then the students could work with the big dumps of city data across many years)
see how people use it and whether it is taxing server and/or forcing us to pay a ton of money in hosting

OR - this would be a 2.0 thing if we did it - we could consider a "freemium" model where special people with accounts (i.e. me) who paid some small amount of money could have unlimited file upload. that could support the sustainability of databasic if we charged some small amount of money as well as pay for any extra hosting load that large files would incur

Ok Jay can you up the size limit to 100mb for the moment and then we'll
test on dev
El El Fri, Dec 11, 2015 a las 17:30, rahulbot<EMAIL_ADDRESS>escribió:

I'm ok with this, if the processing time is reasonable. We need to see how
long processing a 100mb file takes. It may be that (processing speed), and
not the size, that should dictate our size limit.
Rahul
————————————
Rahul Bhargava
http://datatherapy.org
http://rahul.connectionlab.org
@rahulbot
On Dec 11, 2015, 4:52 PM -0500<EMAIL_ADDRESS>wrote:

@rahulbot(https://github.com/rahulbot)my students often work with csv
data that is larger than 10MB and I think a ton of journalists will as
well. What do you think of the following as an experiment -
we up the limit to 100MB (or no limit which would be my preference for
teaching - then the students could work with the big dumps of city data
across many years)
see how people use it and whether it is taxing server and/or forcing us
to pay a ton of money in hosting
OR - this would be a 2.0 thing if we did it - we could consider a
"freemium" model where special people with accounts (i.e. me) who paid some
small amount of money could have unlimited file upload. that could support
the sustainability of databasic if we charged some small amount of money as
well as pay for any extra hosting load that large files would incur
—
Reply to this email directly orview it on GitHub(
https://github.com/c4fcm/DataBasic/issues/138).

—
Reply to this email directly or view it on GitHub
https://github.com/c4fcm/DataBasic/issues/138#issuecomment-164066739.

ok give that a shot!


EXAMPLE 3 Issues

Hi,
example 3 should say:
'Page','Codeunit','Table','XMLport','Report','Query' | % { Compile-NAVApplicationObject -DatabaseName MyApp -Filter "Type=$_" -AsJob } | Receive-Job -Wait

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d6fd8a12-b14c-ae47-51df-072f67b4bab1
Version Independent ID: 43268fed-f5b1-a98d-0f65-e9fecd4bb975
Content: Compile-NAVApplicationObject (Microsoft.Dynamics.Nav.Ide)
Content Source: businesscentral-ps/Microsoft.Dynamics.Nav.Ide/Compile-NAVApplicationObject.md

Hi, thanks for your feedback. The -DatabaseName argument has been fixed and the Query suggestion has been included in the example (but note it is not necessary). It will available in the next doc refresh.


[Workers KV] Documentation pretty confusing for me

Which Cloudflare product does this pertain to?
KV
Existing documentation URL(s)

https://developers.cloudflare.com/kv/get-started/

What changes are you suggesting?
First of all, my experience with Cloudflare's documentation is overall a 10 out of 10, but wasn't the case of Workers KV documentation.
As usual, to start learning a new tool, I started with Get started. From my experience with other key-value systems, I expected to get working in less of 10 minutes. So I created a namespace from the Dashboard, then I created a new key-value pair, and then I tried to fetch it from the Worker. At first, I thought I had a problem with my code, then I checked the documentation of the framework I'm using (Hono), and I spend a lot of time trying to figure out why it didn't work. So then, I tried to create the namespace and the pairs with Wrangler. I got the same result. I read the steps 3 and 4 several times. To be honest, I didn't move to the 5 step. Then, I created inside the code of the Worker another key-value pair and I found it worked, so I started to look for other pages in the documentation trying to find a response (without know exactly what to look for), and then I asked in the Discord community for more clarification.
At this point I haven't clear what workflow to follow, to have the same namespace both in cloud and local. This night I will test more and I will try to figure out what's the workflow that is best suited for my needs, but, as I described, the documentation it wasn't help me.
My suggestion is to clarify the workflow of local and production development using KV's before the steps 3 and 4. I think it will help a lot to new users of the platform.
Additional information
No response

Addressed on here: https://github.com/cloudflare/cloudflare-docs/pull/13990


Automate the daily summary of open PRs

goal:
To send a message to Slack with the open PRs, including the following info:

Amount of reviews
Presence of conflicts
Whether it is blocked by another PR or it needs to be merged at the same time as a PR in another repo
Mark it as merged when it's merged

We talked with @mgrunwaldt about this PR and decided to close it by now.


Unravel oil bus does not work for shipping yet

The oil demand for shipping is added as n.loads.loc["EU oil shipping oil]. So far, there is no distinction between European and German shipping demand.
Way to solve this in modify_prenetwork:
in snakefile:
params:
     sector=config_provider("sector"),
input:
        pop_weighted_energy_totals=resources("pop_weighted_energy_totals_s{simpl}_{clusters}.csv"),
        shipping_demand=resources("shipping_demand_s{simpl}_{clusters}.csv"),

in script:
    # global shipping information
    options = snakemake.params.sector
    domestic_navigation = (
    pd.read_csv(snakemake.input.pop_weighted_energy_totals, index_col=0) * nyears
    ).loc[:, "total domestic navigation"].squeeze()
    international_navigation = pd.read_csv(snakemake.input.shipping_demand, index_col=0).squeeze() * nyears
    shipping_demand = domestic_navigation + international_navigation
    DE_shipping = shipping_demand.filter(like="DE").sum() * 1e6 / nhours

    # shipping meoh
    if "EU methanol shipping methanol" in n.loads.index:
        efficiency = (
            options["shipping_oil_efficiency"] / options["shipping_methanol_efficiency"]
        )
        shipping_methanol_share = options["shipping_methanol_share"][int(snakemake.wildcards.planning_horizons)]

        p_set_methanol = shipping_methanol_share * DE_shipping * efficiency

        n.add("Load", "DE shipping methanol", bus="DE methanol", carrier="methanol" , p_set=p_set_methanol)
        n.loads.loc["EU methanol shipping methanol", "p_set"] -= p_set_methanol

        # CO2 intensity methanol based on stoichiometric calculation with 22.7 GJ/t methanol (32 g/mol), CO2 (44 g/mol), 277.7,
        # MWh/TJ = 0.218 t/MWh
        co2 = p_set_methanol * costs.at["methanolisation", "carbondioxide-input"]

        n.add(
            "Load",
            "DE shipping methanol emissions",
            bus="co2 atmosphere",
            carrier="shipping methanol emissions",
            p_set=-co2,
        )
        n.loads.loc["shipping methanol emissions", "p_set"] += co2

    # oil shipping
    if "EU oil shipping oil" in n.loads.index:
        shipping_oil_share = options["shipping_oil_share"][int(snakemake.wildcards.planning_horizons)]

        p_set_oil = shipping_oil_share * DE_shipping

        n.madd("Load", "DE shipping oil", bus="DE oil", carrier="oil", p_set=p_set_oil,)
        n.loads.loc["EU oil shipping oil", "p_set"] -= p_set_oil

        co2 = p_set_oil * costs.at["oil", "CO2 intensity"]

        n.add("Load", "DE shipping oil emissions", bus="co2 atmosphere", carrier="shipping oil emissions", p_set=-co2)
        n.loads.loc["shipping oil emissions", "p_set"] += co2

I was wrong, everything working fine in ariadne with shipping


Add nostr compatibility base

Fixes #1455

This PR can be tested with:
{
    "id": "4376c65d2f232afbe9b882a35baa4f6fe8667c4e684749af565f981833ed6a65",
    "pubkey": "6e468422dfb74a5738702a8823b9b28168abab8655faacb6853cd0ee15deee93",
    "created_at":<PHONE_NUMBER>,
    "kind": 1,
    "tags": [
        ["e", "3da979448d9ba263864c4d6f14984c423a3838364ec255f03c7904b1ae77f206"],
        ["p", "bf2376e17ba4ec269d10fcc996a4746b451152be9031fa48e74553dde5526bce"]
    ],
    "content": "Walled gardens became prisons, and nostr is the first step towards tearing down the prison walls.",
    "sig": "908a15e46fb4d8675bab026fc230a0e3542bfade63da02d542fb78b2a8513fcd0092619a2c8c1221e581946e0191f2af505dfdf8657a414dbca329186f009262"
}

Ok I had to fix one issue, but this looks good and seems safe because it uses the file fallback behavior
I'll merge it!

Ok, merged! Let me know how you'd like to continue on this, or if there are other ideas you have for improvement


Create CSV-W for Analyst Function obStatus Concept Scheme

There are two distinct concept schemes to this ticket for creating the marker columns we want to support in the new form of sdmx:obsStatus concept schemes. We need this to produce the high quality statistical data representation in the Explore Subnational Statistics project.
This ticket should be done in concert with #15.
Create a CSV-W which will allow the use of the Analyst Function list of Marker Characters from their Using symbols and shorthand to create an observation status column.
Follow the example CSV-W json and first line of the csv to complete the concept scheme. Then use the code list generated from the inputs of #15 to provide the additional details expected in a well described concept scheme. Use the new csvcubed code list command launched in version 0.3.0 to generate the CSV-W. This will give you two json files to one CSV.
Once you've got the AF concept scheme completed, go back to the SDMX one, and clean up the code lists by following the "cleaner" manual CSV-W we have created in this one.

Hasn't this been done now?


esp_eth.c ETH_CHECK order problem (IDFGH-2829)

I think that from esp_eth.c:290
   esp_eth_driver_t *eth_driver = (esp_eth_driver_t *)hdl;
    eth_driver->priv = priv;
    ETH_CHECK(eth_driver, "ethernet driver handle can't be null", err, ESP_ERR_INVALID_ARG);

should be
   esp_eth_driver_t *eth_driver = (esp_eth_driver_t *)hdl;
    ETH_CHECK(eth_driver, "ethernet driver handle can't be null", err, ESP_ERR_INVALID_ARG);
    eth_driver->priv = priv;

well spotted! We will fix it soon. Thanks.


Update README.md

typo

Thanks for looking over the README! This is not a typo: I intentionally refer to age in the first paragraph, and then say "rage is a Rust implementation of the age tool" in the second paragraph. I'll have a think about how the first two paragraphs could be rephrased to improve the readability.


[#4308] serialization class for /scimeta/elements

Resolves #4308

Pull Request Checklist:

[x] Positive Test Case Written by Dev

[ ] Automated Testing

[ ] Sufficient User and Developer Documentation

[ ] Passing Jenkins Build

[ ] Peer Code review and approval

Positive Test Case

Create a resource

navigate to http://localhost:8000/hsapi/resource/{res-id}/scimeta/elements/

See that metadata are serialized correctly

--previously returned a 500 TypeError

Sign out from hydroshare

navigate to http://localhost:8000/hsapi/resource/{res-id}/scimeta/elements/

See a 403

(I was not able to reproduce the original 500 error for un-authenticated request to this endpoint on a private resource)

Thanks @pkdash I guess you're right that unauthenticated request returns a 403 as expected (testing against HS v2.6.1).
So the original issue is not reproducible. However, this PR resolves the related case in which the request to this scimeta/elements endpoint is authenticated. I added some screenshots to this PR to make it more clear, and will make a note in the original issue


guide: Comparing Experiments

It may be worthwhile to have a section in the Experiment Management about dvc exp show, dvc exp diff and their options.
Related #2548

[ ] Is https://github.com/iterative/dvc.org/issues/2844 still needed after this?

I think it could be included in https://github.com/iterative/dvc.org/pull/2752 for now, as comparing them is probably a basic need to manage them. May not even need an explicit section but rather implied throughout that guide.

"Managing" feels too broad for me. It can include everything we write in the "Experiment Management" chapter.
I understand what you mean by "managing", after we have some set of experiments by running them we compare them, persist them, push them, discard them, etc.

"Managing" feels too broad

Yes it's pretty broad you're right. But it should be clear what we mean given the context: listed under "management" and at the same level as "running", "sharing", "removing".

compare them, persist them, push them, discard them

If there's a better title than "managing" let's change it, but we agree on the contents of the doc I think? Pushing is part of "sharing" though.

p.s. to clarify, we'll include this in the doc being created in https://github.com/iterative/dvc.org/pull/2752 right? No need to be in the same PR (although it wouldn't hurt).

"Listing and comparing" is rather verbose I think. Users will use these commands to compare the experiments. What other cases do you think of using these commands?
"Comparing experiments" might not be the best title but "Listing and comparing experiments" doesn't also seem that best title too.

I think comparing should be fine (at the end you list them primarily for that reason?) ... can be also Navigating experiments (in this case diff won't fit precisely. Naming is hard :)

Navigating and Comparing?
Let's get the content first and the name should be easier to decide then 🙂


Fix record count in compaction sla event

Fixed record count being 0 for a few topics. The compaction jobs for these topics were mapper only. The code previously only looked at reducer counters.
@zliu41 can you take a look at this?

LGTM


⚠ Bofrost.at has degraded performance

In 4788fed, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2493 ms

Resolved: Bofrost.at performance has improved in 0481421 after 7 minutes.


[quill-cassandra] Simple filter query keeps allocating native memory and crashes the jvm

Version: (e.g. 0.5.1-SNAPSHOT)
Module: (e.g. quill-cassandra)
Expected behavior
Query query[TestEntity].filter(_.id == 1) should run ok
Actual behavior
[info]   - map
13:07:06.175 [pool-1-thread-1-ScalaTest-running-QueryResultTypeCassandraSpec] INFO  i.g.s.c.CassandraSourceSession - SELECT id, s, i, l, o FROM TestEntity WHERE id = 1
OpenJDK 64-Bit Server VM warning: INFO: os::commit_memory(0x000000009c880000, 169345024, 0) failed; error='Cannot allocate memory' (errno=12)
#
# There is insufficient memory for the Java Runtime Environment to continue.
# Native memory allocation (mmap) failed to map 169345024 bytes for committing reserved memory.
# An error report file with more information is saved as:
# /home/jilen/workspace/quill/quill-cassandra/hs_err_pid11639.log
Exception in thread "Thread-1" Exception in thread "Thread-5" java.io.EOFException
	at java.io.ObjectInputStream$BlockDataInputStream.peekByte(ObjectInputStream.java:2626)
	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1321)
	at java.io.ObjectInputStream.readObject(ObjectInputStream.java:373)
	at sbt.React.react(ForkTests.scala:114)
	at sbt.ForkTests$$anonfun$mainTestTask$1$Acceptor$2$.run(ForkTests.scala:74)
	at java.lang.Thread.run(Thread.java:745)
java.io.EOFException
	at java.io.ObjectInputStream$BlockDataInputStream.peekByte(ObjectInputStream.java:2626)
	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1321)
	at java.io.ObjectInputStream.readObject(ObjectInputStream.java:373)
	at org.scalatest.tools.Framework$ScalaTestRunner$Skeleton$1$React.react(Framework.scala:953)
	at org.scalatest.tools.Framework$ScalaTestRunner$Skeleton$1.run(Framework.scala:942)
	at java.lang.Thread.run(Thread.java:745)

Steps to reproduce the behavior
@getquill/maintainers

@jilen this doesn't seem to be related to Quill. SBT is trying to allocate memory to run the test and fails.

@fwbrasil Extractlly


Urgent Attention Required:Images upload

Type: Bug
The images which i am uploading in my folder on my pc(c:drive)
are showing up when i am opening visual studio
the same file even after reloading
VS Code version: Code 1.87.2 (863d2581ecda6849923a2118d93a088b0745d9d6, 2024-03-08T15:20:17.278Z)
OS version: Windows_NT x64 10.0.22631
Modes:

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-11390H @ 3.40GHz (8 x 2918)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
15.74GB (6.16GB free)

Process Argv
--crash-reporter-id b68e82cd-541a-4a66-bd7f-2c9ae9cffd23

Screen Reader
no

VM
0%

Extensions (26)

Extension
Author (truncated)
Version

path-intellisense
chr
2.8.5

codeium
Cod
1.8.18

doxdocgen
csc
1.4.0

composer-php-vscode
DEV
1.45.15192

intelli-php-vscode
DEV
0.12.15062

phptools-vscode
DEV
1.45.15192

profiler-php-vscode
DEV
1.45.15192

vscode-html-css
ecm
2.0.9

code-runner
for
0.12.1

c-cpp-runner
fra
9.4.7

vscode-opencl
gal
0.8.3

vscode-pull-request-github
Git
0.82.1

shortcuts
giz
0.0.1

better-cpp-syntax
jef
1.17.2

cmake-tools
ms-
1.17.17

cpptools
ms-
1.20.0

cpptools-extension-pack
ms-
1.3.0

js-debug-nightly
ms-
2024.3.2217

python-coding-tools
nil
1.0.4

LiveServer
rit
5.7.9

html5-boilerplate
sid
1.1.1

cmake
twx
0.0.17

vscode-lldb
vad
1.10.0

vscode-java-debug
vsc
0.57.0

vscode-java-test
vsc
0.40.1

JavaScriptSnippets
xab
1.8.0

(2 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30256860
vscorecescf:30445987
vscod805cf:30301675
binariesv615:30325510
vsaa593:30376534
py29gd2263:30899288
c4g48928:30535728
azure-dev_surveyone:30548225
vscrp:30673768
962ge761:30959799
pythongtdpath:30769146
welcomedialogc:30910334
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
2e7ec940:31000449
pythontbext0:30879054
accentitlementst:30995554
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfnew:31000557
d34g3935:30971562
fegfb526:30981948
bg6jg535:30979843
ccp2r3:30993541
dsvsc020:30976470
pythonaic:30996667
dsvsc021:30996838
g1icg217:30999571

/needsMoreInfo
What exact you are doing and what is expected result?
Please provide screen recording if possible.

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


C++ documentation does not match function signatures

Documentation of Clipper64 here gives functions with FillRule as last parameter.
bool Execute(ClipType clip_type, Paths64& solution, FillRule fill_rule);
bool Execute(Clip_type clipType, PolyTree64& solution, Paths64& open_solution, FillRule fill_rule);
bool Execute(Clip_type clipType, PolyTree64& solution, Paths64& open_solution, FillRule fill_rule);

But the functions found in clipper.engine.h have FillRule as second parameter.  (and there are 4 instead of 3 variations)
bool Execute(ClipType clip_type, FillRule fill_rule, Paths64& closed_paths)
bool Execute(ClipType clip_type, FillRule fill_rule, Paths64& closed_paths, Paths64& open_paths)
bool Execute(ClipType clip_type, FillRule fill_rule, PolyTree64& polytree)
bool Execute(ClipType clip_type, FillRule fill_rule, PolyTree64& polytree, Paths64& open_paths)

And similarly for ClipperD

Just adding some other cases as I encounter them:
Docs for PolyTreeToPaths64 and PolyTreeToPathsD show them as templates without the type suffix
Actual signatures:
Clipper2/CPP$ grep -ri PolyTreeToPath
Clipper2Lib/include/clipper2/clipper.h:  inline Paths64 PolyTreeToPaths64(const PolyTree64& polytree)
Clipper2Lib/include/clipper2/clipper.h:  inline PathsD PolyTreeToPathsD(const PolyTreeD& polytree)

Thanks 😁.
Should be fixed now.


Remove default configuration from client

Is your feature request related to a problem? Please describe.
This will reduce the memory footprint of the client.
Describe the solution you'd like
I'd like to be able to load a pre-packaged config, but it should be optional. I'd like plugins to be replaced with wrappers where possible.
Additional context
Depends on https://github.com/polywrap/monorepo/issues/1122

Some thoughts on the implementation here:
https://github.com/polywrap/toolchain/pull/1045#issuecomment-1225085814

This has been completed as part of the ClientConfigBuilder work.


invalid version/constraint format: improper constraint

I'm having  a problem with what should be a simple task. I have a Helm chart with no templates and this for requirements.yaml:
dependencies:

name: keycloak
repository: http://storage.googleapis.com/kubernetes-charts-incubator
version: "0.4.0"

Running my script gives this:

+ make preview

helm init --client-only

Creating /home/jenkins/.helm

Creating /home/jenkins/.helm/repository

Creating /home/jenkins/.helm/repository/cache

Creating /home/jenkins/.helm/repository/local

Creating /home/jenkins/.helm/plugins

Creating /home/jenkins/.helm/starters

Creating /home/jenkins/.helm/cache/archive

Creating /home/jenkins/.helm/repository/repositories.yaml

Adding stable repo with URL: https://kubernetes-charts.storage.googleapis.com

Adding local repo with URL: http://<IP_ADDRESS>:8879/charts

$HELM_HOME has been configured at /home/jenkins/.helm.

Not installing Tiller due to 'client-only' flag having been set

Happy Helming!

helm repo add incubator http://storage.googleapis.com/kubernetes-charts-incubator

"incubator" has been added to your repositories

helm repo add chartmuseum http://jenkins-x-chartmuseum:8080

"chartmuseum" has been added to your repositories

helm repo add chartmuseum https://chartmuseum.build.cd.jenkins-x.io

"chartmuseum" has been added to your repositories

sed -i -e "s/version:.*/version: /" Chart.yaml

sed -i -e "s/version:.*/version: /" ../*/Chart.yaml

sed -i -e "s/repository: .*/repository: <IP_ADDRESS>:5000\/ppi-engineering\/keycloak-service/" values.yaml

sed -i -e "s/tag: .*/tag: /" values.yaml

echo "  version: " >> requirements.yaml

helm dependency build

Hang tight while we grab the latest from your chart repositories...

...Unable to get an update from the "local" chart repository (http://<IP_ADDRESS>:8879/charts):

	Get http://<IP_ADDRESS>:8879/charts/index.yaml: dial tcp <IP_ADDRESS>:8879: getsockopt: connection refused

...Successfully got an update from the "incubator" chart repository

...Successfully got an update from the "stable" chart repository

...Successfully got an update from the "chartmuseum" chart repository

Update Complete. ⎈Happy Helming!⎈

Error: dependency "keycloak" has an invalid version/constraint format: improper constraint:

make: *** [preview] Error 1

script returned exit code 2

I've tried this with the name incubator/keycloak as well and with the version quoted, not quoted, and absent.
Output of helm version:
Jason-PPI:dial-service jason$ helm version
Client: &version.Version{SemVer:"v2.9.0", GitCommit:"f6025bb9ee7daf9fee0026541c90a6f557a3e0bc", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.9.0", GitCommit:"f6025bb9ee7daf9fee0026541c90a6f557a3e0bc", GitTreeState:"clean"}
Output of kubectl version:
Jason-PPI:dial-service jason$ kubectl version
Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.1", GitCommit:"d4ab47518836c750f9949b9e0d387f20fb92260b", GitTreeState:"clean", BuildDate:"2018-04-13T22:27:55Z", GoVersion:"go1.9.5", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"9", GitVersion:"v1.9.3", GitCommit:"d2835416544f298c919e2ead3be3d0864b52323b", GitTreeState:"clean", BuildDate:"2018-02-07T11:55:20Z", GoVersion:"go1.9.2", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
Kops on AWS

hey @jasoncarreira - that issue has to do with the version constraint specified in the Chart.yaml file of your chart. It will return that error if the version is not parseable. What does the version in the chart look like?
For more info: https://github.com/Masterminds/semver

It's 0.1.0-SNAPSHOT
I guess -SNAPSHOT is not a valid semver?

It's valid semver, but the hypen notes that it's a pre-release as per section 9 under semver law.
After running that command, would you mind posting the output of requirements.yaml and Chart.yaml? I noticed in the output that there's a echo "  version: " >> requirements.yaml under the make preview target. Just wanna check if something odd is being injected into requirements.yaml or Chart.yaml prior to running helm dep build. Thanks!

I've been playing with it and it's a little different now, but I'm getting the same error. Here's Chart.yaml:
description: A Helm chart for Kubernetes
icon: https://raw.githubusercontent.com/jenkins-x/jenkins-x-platform/master/images/java.png
name: preview
version: develop-24```
and here's requirements.yaml:
```dependencies:
- name: keycloak
  repository: http://storage.googleapis.com/kubernetes-charts-incubator
  version: 0.4.0```

and here's the output

```+ make preview
helm init --client-only
Creating /home/jenkins/.helm
Creating /home/jenkins/.helm/repository
Creating /home/jenkins/.helm/repository/cache
Creating /home/jenkins/.helm/repository/local
Creating /home/jenkins/.helm/plugins
Creating /home/jenkins/.helm/starters
Creating /home/jenkins/.helm/cache/archive
Creating /home/jenkins/.helm/repository/repositories.yaml
Adding stable repo with URL: https://kubernetes-charts.storage.googleapis.com
Adding local repo with URL: http://<IP_ADDRESS>:8879/charts
$HELM_HOME has been configured at /home/jenkins/.helm.
Not installing Tiller due to 'client-only' flag having been set
Happy Helming!
helm repo add incubator http://storage.googleapis.com/kubernetes-charts-incubator
"incubator" has been added to your repositories
helm repo add chartmuseum http://jenkins-x-chartmuseum:8080
"chartmuseum" has been added to your repositories
helm repo add chartmuseum https://chartmuseum.build.cd.jenkins-x.io
"chartmuseum" has been added to your repositories
sed -i -e "s/version:.*/version: develop-24/" Chart.yaml
sed -i -e "s/repository: .*/repository: <IP_ADDRESS>:5000\/ppi-engineering\/keycloak-service/" values.yaml
sed -i -e "s/tag: .*/tag: develop-24/" values.yaml
helm dependency build
Hang tight while we grab the latest from your chart repositories...
...Unable to get an update from the "local" chart repository (http://<IP_ADDRESS>:8879/charts):
	Get http://<IP_ADDRESS>:8879/charts/index.yaml: dial tcp <IP_ADDRESS>:8879: getsockopt: connection refused
...Successfully got an update from the "incubator" chart repository
...Successfully got an update from the "stable" chart repository
...Successfully got an update from the "chartmuseum" chart repository
Update Complete. ⎈Happy Helming!⎈
Saving 1 charts
Downloading keycloak from repo http://storage.googleapis.com/kubernetes-charts-incubator
Deleting outdated charts
helm lint
==> Linting .
[ERROR] Chart.yaml: version 'develop-24' is not a valid SemVer
[WARNING] templates/: directory not found

Error: 1 chart(s) linted, 1 chart(s) failed
make: *** [preview] Error 1```

Looks like it's working for me now. Changed it so the Chart.yaml version is like this: 1.1.0-SNAPSHOT-39

cool yeah develop-24 is definitely not valid semver so that error message seems right to me. Is there a particular action item we can extract from this ticket or does this seem good to close?

Should be good to close. I think the script I was using was messing up copying the version


fix bitbucket logo

The bitbucket logo does not resolve, due to the fact they changed their twitter profile picture.

Oops, this issue should have been for /frontend instead of /code-highlighter.


fund-recipient on token transfer

it is possible to fund destination account in case the destination does not have a token account?

solved

solved

How did you solved it?


Adding livenessProb to the jobmanager to improve reliability

With a livenessProb, Kubernetes can restart the job manager. I find this quite useful when enabling HA mode.
For instance I observed that if the zookeeper quorum is not fully ready (but endpoint is responding), the Job manager may hang forever waiting for leader election. With the livenessProb, Kubernetes will detect the issue (if job manager has no leader, it returns an error) and restart the container

I'd have to think more if there are any caveats to this (for instance if it's possible that the HTTP endpoint is up but the main logic of the jobmanager is failing) but it's better than nothing. Thanks.


Bump react-film

Fixes #1759
Fixes #1707
(Replacement to #1826)

Coverage remained the same at 54.065% when pulling 1c051f4412bcfe936aa1ec5e093a3b2b6bdebe0e on corinagum:1826 into f6f23dfb4ad8abe5499d27953536806192b6cf5d on Microsoft:master.


Move the postbuf into HttpSM

Fix #2692 , #2693
Move the postbuf into HttpSM.

[approve ci clang-format]

[approve ci]

The PostDataBuffers is designed to save the post payload while retry to make a new request.
The HttpTunnel is designed to move the data from producer to one or more consumers. It should not keep any data after tunnel.reset().
Any data that is received from ua_session or server_session, the owner of these data is HttpSM.
According to the above analysis, the PR moves the PostDataBuffers from HttpTunnel to HttpSM.
With the PR,

Incoming data is saved into the PostDataBuffers by HttpTunnel
Tunnel is reset if request is redirected. But the HttpSM still keeps the post payload.
HttpSM make a new request and clone the post payload for new HttpTunnel.

Candidate for 7.1.2 ?

@scw00
We should have an autest for this to make sure POSTs work with redirection.

@bryancall . I'll give another pr on this weekends.

@zwoop! Yes it is candidate for 7.1.2, but please waiting for my autest pr.

Alrighty, let me know what other PRs this depends on when ready.

@scw00 #2800 was merged. Do we want to back-port this?

@scw00 @zwoop Ping

Yes we could port this pr to 7.1.x @zwoop .


Update index.html

Updating the support for Development; open-source community efforts

Added in https://github.com/opentofu/manifesto/pull/1137


Refactor DataConverter API and Internals

Summary
The current DataConverter API is a mess. It needs to be refactored to reduce boilerplace and simplify user implementations.
Analysis

Provide a clean IDataConverterEnvironment interface.
Do not distinguish between results, available data and child conversions.
Do distinguish between asking whether data is available and actually handling / using it.
Move all real-world interaction behind the environment interface.

A converter should not determine Resource paths for new results.
A converter should not save Resources.
Maybe, a converter shouldn't even find matching "existing Resource" candidates.

Do not keep track of converter path length, because granularity is an implementation detail. Instead, let each converter specify how costly the operation it proposes would be.
Unify handling an input, specifying an output, suggesting a name for it (and specifying a cost for this path?) in a single call.

Also figure out a way to get rid of the boolean return value.

Some converters do not know whether they're applicable until their child conversions are performed and the results are available.
Most converters will still just early-out in CanConvert, this would just be for the more ambiguous cases.

A convert operation should only do an early-out with readily available resources, if that early-out would handle all available data.

Keep track of what data is available overall (GetFormats + GetData(format) in ConversionData?)

Moving this to the Future milestone since it doesn't seem to be crucial or urgent.


add better exception handler

#121

Closing as noted here: https://github.com/okcoker/taggle.js/issues/121#issuecomment-571961626
Also if we were to add a check for something like this, I think it should happen in initialization to make sure the options passed are valid.


Fix property revert for inherited child nodes

This PR fixes #28279.
get_instanced_node_original_property walks up the node tree to find the instanced / inherited scene node, and query the property value in that scene for the default value. If the property is not overridden in that scene, it will try to use the class default value. But the default value is currently get from the instanced / inherited scene node's class instead of the parameter node's class.
Tested on the current master (0e3b011dee) and 3.2 branch (6aa54c117).

node is used to walk up the scene tree to locate the instanced node / inherited scene root. So that an editable child / inherited scene node can get the property default value correctly.

node is used to walk up the scene tree to locate the instanced node / inherited scene root. So that an editable child / inherited scene node can get the property default value correctly.

Ah right, was reviewing on phone and didn't see that this passes results by value (it would be clearer if the argument was named r_value as we usually do for arguments that are used as return values).

Thanks!

Cherry-picked for 3.2.4.


Adds a projects section. Updates hero image. Adds icons & links

Adds a project section to the web page which features a slideshow of images. Also updates the hero image to take up the whole page. Lastly, adds GitHub and LinkedIn links in the top right corner.

@Tetheta Starting review


Support the ability to upload files. 支持上传文件的能力

[x] I'm sure this does not appear in the issue list of the repository

What problem does this feature solve?
Currently, only ImagePicker supports the feature of image uploading. It is expected that a new component can be added to support the uploading of regular files.

目前只有 ImagePicker 可以支持图片上传的功能，预期新增一个组件，可以支持普通文件的上传

What are the similar cases of this feature
Similar to this: https://vant-ui.github.io/vant/#/zh-CN/uploader. However, only the file upload feature needs to be referenced, as the image preview functionality is almost identical to our ImagePicker.

类似这个组件 https://vant-ui.github.io/vant/#/zh-CN/uploader. 不过仅仅需要参考这里面的上传文件功能，其他的图片上传和预览等和我们的 ImagePicker 几乎是一致的

We can discuss this together if you are interested in this feature.

感兴趣可以一起讨论

I would like to ask if there is any specific design drawing for the style of the file upload component.
For example, if I upload a text file, what should its preview style be like?


feature: add support for http/2

@sbender9, are you intending a change to the specification?
Would need to consider:

mandatory or optional for the server? Difference if Slave or Master?
discovery - how is this included in the Bonjour and http discovery
https only or http as well?
examples

I'm not sure that we need a a change to the specification.
Clients (including all major browsers) are generally smart enough to figure out if the server supports it.
No change needed to discovery, since this is still just https://
http/2 is generally just https (and I could not get spdy to work with plain http)

The issue here is what the minimum specs are for a server and a consumer. If your app which needs this feature is reliant on http/2 then we need to agree that http/2 is mandatory on all servers. If this is not the case then all consumers must be designed to work when http/2 is not available, and we need to document that http/2 MAY be implemented by servers. If this is not done no server (except node-server) will offer http/2.
You mention 'most browsers'. If we require that all signalk consumers (whether browser based or not) have the ability to 'figure out' support we need to document that.
You have also mentioned spdy here. Are you proposing signalk support for both http/2 and spdy? If so is support mandatory for both?
We also need to document whether http/2 is expected over http AND https or just https (which is the defacto web position, but not the http/2 standard).
Finally, there is an issue with the node-server implementation of https anyway in that if the way it is implemented in node-server (ie. signalk-http points to an https endpoint) is correct, then in effect https support is mandatory for all signalk consumers. I do not think this can be correct or sensible. See #482
I'm not saying that this shouldn't be implemented by the way, just that we need to document how and where, so that a user purchasing/using your (or any) signalk app can be guaranteed that it will work with any signalk server.

I definitely don't think that http/2 support should be mandatory for servers or clients. Maybe in version 3 of the spec ;)
I only mentioned spdy because it is the name of the npm package that implements http/2.
I do think that https support should be mandatory for clients. It is perfectly sensible for a server to only support https.

If your app which needs this feature is reliant on http/2

I do not think this is a real world issue. You would have to go out of your way to make your application work only on http/2. Http/2 servers are compatible with http 1.x. On application level http/2 is still http and everything will work, maybe less efficiently.
Still we should add MAY support http/2 so that it is mentioned in the spec.

If we require that all signalk consumers (whether browser based or not) have the ability to 'figure out' support we need to document that.

This is already part of http/2 spec: a client indicates in the initial 1.x http request that is supports http/2 and the server can then upgrade the connection. If the header is not there to indicate client http/2 support the server will use http 1.x. I don't think we need to include any of this in SK spec.

We also need to document whether http/2 is expected over http AND https or just https (which is the defacto web position, but not the http/2 standard).

No, as http/2 use is transparent and automatic if both client and server use it.

then in effect https support is mandatory for all signalk consumers. I do not think this can be correct or sensible.

I agree with sbender, at this day and age https support should be mandatory.


[WebToolsE2E][Aspire] Running the eshop.web solution throws an exception: Method not found: 'Void Npgsql.TypeMapping.INpgsqlTypeMapper.AddTypeInfoResolver(Npgsql.Internal.IPgTypeInfoResolver)'.

REGRESSION INFO: Worked fine with Aspire 8.0.0-preview.1.23557.2
INSTALL STEPS

Clean machine: Win11 x64 23h2 ENU
Install VS 17.9 P2
Install Aspire Preview 2 latest build 8.0.0-preview.2.23610.1
Apply NuGet Feeds
Install Docker Desktop

REPRO STEPS

Clone the eShop repo
Update NuGet.config to use the dotnet-tools feed
Update Directory.Packages.props to use aspire version 8.0.0-preview.2.23610.1
Update package Npgsql.EntityFrameworkCore.PostgreSQL '8.0.0' and Yarp.ReverseProxy '2.1.0'
Open solution eShop.Web.slnf
Build and F5

ACTUAL
System.MissingMethodException
HResult=0x80131513
Message=Method not found: 'Void Npgsql.TypeMapping.INpgsqlTypeMapper.AddTypeInfoResolver(Npgsql.Internal.IPgTypeInfoResolver)'.
Source=Pgvector
StackTrace:
at Npgsql.VectorExtensions.UseVector(INpgsqlTypeMapper mapper)
at Microsoft.EntityFrameworkCore.VectorDbContextOptionsBuilderExtensions.UseVector(NpgsqlDbContextOptionsBuilder optionsBuilder)
at Extensions.<>c.b__0_1(NpgsqlDbContextOptionsBuilder builder) in C:\Users\xxx\source\repos\eShop\src\Catalog.API\Extensions\Extensions.cs:line 11
at Microsoft.EntityFrameworkCore.NpgsqlDbContextOptionsBuilderExtensions.UseNpgsql(DbContextOptionsBuilder optionsBuilder, Action`1 npgsqlOptionsAction)

EXPECTED
F5 to work successfully

This looks like Pgvector needs to be updated to the Npgsql v8.0 compatible version.
@v-sherryfan - can you also update these two lines?
https://github.com/dotnet/eShop/blob/1417a0c6f78b0f6e9da0352cbfc9dc5a46dda088/Directory.Packages.props#L38-L39
to be
    <PackageVersion Include="Pgvector" Version="0.2.0" />
    <PackageVersion Include="Pgvector.EntityFrameworkCore" Version="0.2.0" />

@eerhardt After updating the above package according to your instructions, the error will disappear.

Is this still relevant?


Add repository link to CTAN

It would be helpful to add the link to this repository on CTAN.

Done in the last submission


Added a feature to skip back to the previous branch point in the curr…

Added a feature that's been requested by myself and some others in my circle. This PR adds a navigation feature to quickly return back to the last branch point by pressing shift+left. This helps when navigating long nested branched variations where it can get a bit tricky to navigate.

Thank you for the pull request! Great feature!
By the way, are you okay with the MIT license for your contributions? Would you like to be listed in the readme as a contributor?

Hi! Thanks for the project, you've kept everything so well organized it's very easy to contribute  :)
I'm happy with MIT license. No need to list me as a contributor. Cheers!


Override live test location default to westus

Temporary location override at the go repo level as part of https://github.com/Azure/azure-sdk-tools/issues/3398 to
address capacity restrictions.

/azp run cpp - attestation

/azp run cpp - keyvault

/azp run cpp - storage


adapters: downgrade initialization logging and add timing info

Downgrade the adapter init logging message from info to debug and
include some timing info to see how long it takes for the adapter to
load and to be initialized.

@mstemm this is a simple one

lgtm


Adds support for Certificate resources

Initial support for Certificate resources. Items to note:
We hardcode ValidationMethod to "DNS" because the "EMAIL" validation method means cert renewal is not automateable. See https://docs.aws.amazon.com/acm/latest/userguide/email-validation.html
We have some custom validation of the number of domain validation options. When requesting a public certificate with DNS validation, you can only submit a max of 5 subdomains/CNAME records for use in DNS validation, and since we hardcode DNS validation method, we need to check for this and put the Certificate into a Terminal state if there are more than 5 CNAME records listed in the DomainValidationOptions field.
Finally, we add a simple sleep of 5 seconds after successful creation since
https://docs.aws.amazon.com/acm/latest/APIReference/API_RequestCertificate.html warns us that DescribeCertificate calls will not succeed for several seconds after a RequestCertificate call has returned the CertificateArn...
Issue aws-controllers-k8s/community#482
Signed-off-by: Jay Pipes<EMAIL_ADDRESS>By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@RedbackThomson k, fixed up the stuff.

Agreed over Slack that the static wait can be dealt with in another PR.
/lgtm


GeoSpatialValues() returns invalid values for latitude

Hi All,
I am trying to use your library in my project, but I get an error when calling GeoSpatialValues()
GeoSpatialValues() returns invalid values for latitude
Only first line of latitude is valid...
For example i have the GRIN for 38N 06E - 56N 39E with only one message - temperature
API returns valid GRID only for 56N latitude, for other rows latitude is invalid
{"Latitude":56.0,"Longitude":35.25,"Value":272.7,"IsMissing":false},{"Latitude":56.0,"Longitude":35.5,"Value":272.83,"IsMissing":false},{"Latitude":56.0,"Longitude":35.75,"Value":273.05,"IsMissing":false},{"Latitude":56.0,"Longitude":36.0,"Value":272.91,"IsMissing":false},{"Latitude":56.0,"Longitude":36.25,"Value":272.57,"IsMissing":false},{"Latitude":56.0,"Longitude":36.5,"Value":272.73,"IsMissing":false},{"Latitude":56.0,"Longitude":36.75,"Value":273.06,"IsMissing":false},{"Latitude":56.0,"Longitude":37.0,"Value":273.06,"IsMissing":false},{"Latitude":56.0,"Longitude":37.25,"Value":273.0,"IsMissing":false},{"Latitude":56.0,"Longitude":37.5,"Value":273.16,"IsMissing":false},{"Latitude":56.0,"Longitude":37.75,"Value":273.25,"IsMissing":false},{"Latitude":56.0,"Longitude":38.0,"Value":273.24,"IsMissing":false},{"Latitude":56.0,"Longitude":38.25,"Value":273.22,"IsMissing":false},{"Latitude":56.0,"Longitude":38.5,"Value":273.2,"IsMissing":false},{"Latitude":56.0,"Longitude":38.75,"Value":273.09000000000003,"IsMissing":false},{"Latitude":56.0,"Longitude":39.0,"Value":272.94,"IsMissing":false},{"Latitude":22.982,"Longitude":6.0,"Value":277.75,"IsMissing":false},{"Latitude":22.982,"Longitude":6.25,"Value":277.59000000000003,"IsMissing":false},{"Latitude":22.982,"Longitude":6.5,"Value":277.41,"IsMissing":false},{"Latitude":22.982,"Longitude":6.75,"Value":277.2,"IsMissing":false},{"Latitude":22.982,"Longitude":7.0,"Value":276.94,"IsMissing":false},{"Latitude":22.982,"Longitude":7.25,"Value":276.57,"IsMissing":false},{"Latitude":22.982,"Longitude":7.5,"Value":276.09000000000003,"IsMissing":false},{"Latitude":22.982,"Longitude":7.75,"Value":275.53000000000003,"IsMissing":false},{"Latitude":22.982,"Longitude":8.0,"Value":275.06,"IsMissing":false},{"Latitude":22.982,"Longitude":8.25,"Value":275.0,"IsMissing":false},{"Latitude":22.982,"Longitude":8.5,"Value":274.8,"IsMissing":false}

Can you share your file? What should the lat be in the other cases?

Lat is big negtive number in other rows
{"Latitude":-2288.2780000000016,"Longitude":29.5,"Value":289.77,"IsMissing":false},{"Latitude":-2288.2780000000016,"Longitude":29.75,"Value":290.62,"IsMissing":false},{"Latitude":-2288.2780000000016,"Longitude":30.0,"Value":288.49,"IsMissing":false},{"Latitude":-2288.2780000000016,"Longitude":30.25,"Value":287.82,"IsMissing":false},
{"Latitude":-2321.2960000000016,"Longitude":13.25,"Value":285.71,"IsMissing":false},{"Latitude":-2321.2960000000016,"Longitude":13.5,"Value":287.2,"IsMissing":false},{"Latitude":-2321.2960000000016,"Longitude":13.75,"Value":288.24,"IsMissing":false},{"Latitude":-2321.2960000000016,"Longitude":14.0,"Value":286.88,"IsMissing":false},{"Latitude":-2321.2960000000016,"Longitude":14.25,"Value":287.1,"IsMissing":false},
Example of file https://drive.google.com/file/d/0B9ZhPptl97CbLXRlVmctVHYxcFk/view

Yes, I see what you're talking about. In the sample file, it looks like the jDirectionIncrement is the culprit. It's 33.018 which accounts for the crazy offsets you're seeing. zyGrib seems to handle it properly: http://i.imgur.com/7Pwb2Uh.png.
I'll continue to look into it and contact the grib_api team, since I don't think this has anything to do with my code.
Key: latitudeOfLastGridPoint Val: 10
Key: latitudeOfLastGridPointInDegrees Val: 10
Key: longitudeOfLastGridPoint Val: 20
Key: longitudeOfLastGridPointInDegrees Val: 20
Key: iDirectionIncrement Val: 0.25
Key: jDirectionIncrement Val: 33.018

grib_dump.txt

What happens if you set jDirectionIncrement to 0.25?
msg["jDirectionIncrement"].AsDouble(0.25);

When I did it for the first message in the sample file, I got values like this:
Lat: 20 Lon: 10 Val: 292.83
Lat: 20 Lon: 10.25 Val: 292.96
Lat: 20 Lon: 10.5 Val: 293.07
Lat: 20 Lon: 10.75 Val: 293.3
Lat: 20 Lon: 11 Val: 293.53
...
Lat: 19.75 Lon: 10 Val: 292.59
Lat: 19.75 Lon: 10.25 Val: 292.98
Lat: 19.75 Lon: 10.5 Val: 293.05
Lat: 19.75 Lon: 10.75 Val: 293.15
Lat: 19.75 Lon: 11 Val: 293.58

Thank you!
You make my day!
But i have another BUG with time of the grib message, all message have the same time as the first message time. In the file i have forecast for future dates (
May be we need to reset another key for right value?

http://imgur.com/qc4XJa4

Do the validityTime and validityDate keys work?

I've got a fix. It'll be in the next release. You can figure out correct time by adding P2 or using validityTime & date.

| Do the validityTime and validityDate keys work?


Migrate remaining Alby Hosted Balance to existing channel

On the liquidity page there is a card "Alby Hosted Balance" if your balance is > 100 sats.
There should be a way to move this remaining balance without opening a new channel, by sending it to one of your existing channels.
If you don't have enough liquidity, the user should be notified of this so they know what to do.

Are these the same two functionalities? how do we determine to migrate to a new channel or not? based on if the amount is > 250,000 sats?

I'd say:
if there is a channel with enough (50% more?) incoming capacity we pay an invoice
if there is no channel and it's above 200k we use the pmlsp
if it's below that we tell people to deposit onchain and open a channel.


Saltern Issues, Requests and Feedback

Description

The bookshelfs are still empty, need to be BookshelfFilled

Once the Singulo runs, all the animals in the zoo die of radiation.

What is this budget kitchen at chapel, does it really need to be there?

I think a Cryopod would be nice, especially once considered that there is no tech storage, and sci is forced to speedrun cloners to not die, which is a not optimal setup. The general lack of medicine lockers in the bay is annoying.

Telecomms is hilariously easy to break in, all it takes is to take out one wall and bam, you are in.

This feels like the normal air setup, but with extra steps.

Security has several issues:

There is not one, but two warden lockers.
There is no detectives locker anymore.
The doctors locker should be replaced with a medicine locker.

Maints in general feels underwelming, any a bit too wide for me, but that is personal preference I guess

agree with the posted

detective doesnt exist so should i just put a forensic scanner in hos room

yea thats fine

also comms is piss easy to break in on any map, see:

barratry, just hack 1 (one) door which is public or break 1 (one) wall
moose, same deal
kettle, hack 2 (two) doors


.fa-facebook .fa-twitter are not working

The icon for the .fa-facebook .fa-twitter are not showing up, where as its working for the others.
Check the cheatsheet here http://fortawesome.github.io/Font-Awesome/cheatsheet/.
I think the wrong version of the font file is being used. Even the fontawesome homepage seems broken http://fortawesome.github.io/Font-Awesome/icons/

@devilankur18 OMFG. I thought I was going nuts. Bloody adblock. Thanks XD


Rename Show/String to Debug/Display

Fix needed because of https://github.com/rust-lang/rust/pull/21457

Done in #85.

Wow. That was fast @BurntSushi! Thanks a lot! :horse_racing:

Thanks goes to @ianbollinger  :-)


Create a new pull request by comparing

Pull Request Test Coverage Report for Build 560565184

2 of 2 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 94.647%

Totals

Change from base Build 557839170:
0.03%

Covered Lines:
1904

Relevant Lines:
1967

💛  - Coveralls


Add donors and partner logos

Add donors logos (USAID, DIFD and NORAD) -> will have to confirm again with Matt if this is the correct list.
Parters are Moabi, WRI, HOT, OSM. who else?

@thomas-maschler if you have any of these logos, can you add to GDrive > assets?
Also, should this be a new page, or included in the About page.  (I suggest the latter, unless there's enough content on partners/donors to justify an entire page)

USAID
https://drive.google.com/open?id=0B58e-2phIRx1NjlSN3dWaGRDQTg&authuser=0
UK
https://drive.google.com/open?id=0B58e-2phIRx1NjlSN3dWaGRDQTg&authuser=0
Norway
https://drive.google.com/open?id=0B58e-2phIRx1QWQzT21XclZsMVE&authuser=0
UMD
https://drive.google.com/open?id=0B-Ojr3OO5BkDSlNQTC05aXUtOHM&authuser=0
WRI
http://brand.wri.org/brand-assets/logos

We could list the partners at the bottom of the front page, next to the twitter feeds.
There is plenty of space.
Have a look at our GFW page (front page, bottom) I think they did a nice job.

agreed.
On Thu, May 14, 2015 at 9:13 AM, Thomas Maschler<EMAIL_ADDRESS>wrote:

We could list the partners at the bottom of the front page, next to the
twitter feeds.
There is plenty of space.
Have a look at our GFW page (front page, bottom) I think they did a nice
job.
—
Reply to this email directly or view it on GitHub
https://github.com/crowdcover/logging-roads/issues/8#issuecomment-102031460
.

@thomas-maschler the link to the UK logo is a duplicate of the USAID logo.  Can you update?

https://drive.google.com/open?id=0B58e-2phIRx1MTB0Mk04enN6QkE&authuser=0

OK, added logos except for UMD b/c I couldn't figure out how to convert its white background to transparent.  @BKessle you have any experience with that?

Will check if we have a Adobe Illustrator license somewhere to edit the original vector file to avoid the ugly white borders around the letters

OK.  Ditto, USAID logo would also look cleaner if we could modify the original svg.
Closing for now.  Reopen if we have access to svgs for UMD/USAID.


Dev 2021.1

Things To Remember:

Be sure to merge into the right branch. The choices are:

master if your PR runs on the current version of Tableau.
dev-2020.1
2020.2

You should update at least one of the following:

The changelog of the project you modified.
The root README.md if the PR is a release of the SDK, Connector Packager, or TDVT.
=====

Hi, @saugatt! Thanks for opening this PR!

In the file connector-packager/connector_packager/xml_parser.py:

Line 135:121: E501 line too long (134 > 120 characters)

In the file connector-packager/connector_packager/xsd_validator.py:

Line 177:149: W291 trailing whitespace
Line 177:121: E501 line too long (148 > 120 characters)
Line 176:144: W291 trailing whitespace
Line 176:121: E501 line too long (143 > 120 characters)
Line 18:121: E501 line too long (142 > 120 characters)

For reference: Error Code Reference, PEP8, Python code style