Added better representations for types

I did this primarily for my own edification and playing with Apropos - I don't know if it defeats the purpose of what you're trying to demonstrate.

That will have to wait until I have a second 😒

Any plans for this branch @AriFordsham ?

I think @delicious-lemon @Geometer1729 felt the concepts were better illustrated the way it was.

I think @delicious-lemon @Geometer1729 felt the concepts were better illustrated the way it was.

Was unaware - thank you. Shall we close the PR?


Services: first build: abs added

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_33m23rlbw_/clone/recipe/meta.yaml:1: missing_pip_check: For pypi packages, pip check should be present in the test commands
WARNING: /tmp/abs_33m23rlbw_/clone/recipe/meta.yaml:15: avoid_noarch: noarch: python packages should be avoided
ERROR: /tmp/abs_33m23rlbw_/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_33m23rlbw_/clone/recipe/meta.yaml:33: missing_license_url: The recipe is missing a license_url
ERROR: /tmp/abs_33m23rlbw_/clone/recipe/meta.yaml:33: missing_dev_url: The recipe is missing a dev_url
WARNING: /tmp/abs_33m23rlbw_/clone/recipe/meta.yaml:33: missing_doc_source_url: The recipe is missing a doc_source_url
ERROR: /tmp/abs_33m23rlbw_/clone/recipe/meta.yaml:33: missing_doc_url: The recipe is missing a doc_url
WARNING: /tmp/abs_33m23rlbw_/clone/recipe/meta.yaml:33: missing_description: The recipe is missing a description
Errors were found


Support for modern CSS layout

Will Weex support CSS-Grid? The Web can expect this revolutionary layout solution very soon as it is already working behind the flags of all major browsers.
Having a quick look around I saw that Weex makes use of CSS-Layout from Facebook. That is not what I had expected (maybe as a stop gap only) because the Facebook API is only based on a subset of the Flexbox spec.
Essentially, CSS-Grid is suitable for robust X, Y and Z layout, while Flexbox is a more one dimensional option. Thanks for your interest in the future, here are a few links to consider:
http://gridbyexample.com/examples/page-layout
http://igalia.github.io/css-grid-layout/index.html

Yes you are right grid layout is awesome, absolutely.
Support this feature means we need some consideration and preparations more:

Know-how for front-end engineers, not very much developers familiar with grid layout today.
Efficiency, wide and strong enough implementation in mobile browsers http://caniuse.com/css-grid and native view (css-layout)
The simplification of tech design of a framework/solution.

Maybe the research should be continued as anyway anyone like. And the discussion could be made again in a more appropriate time when the stuff above come true.
Thanks.

Thanks, it is good to have this on the radar. CSS-Grid is something to anticipate with certainty and learning it for frontend engineers is very easy, there are already numerous tutorials. In addition frontend engineers will drop what they know as soon as they see how incredibly efficient layout has become with CSS-Grid.
Likely an enhancement to the CSS-Layout library is in order ASAP. Maybe a translation of CSS-Grid semantics to something the CSS-Layout requires is a start.


Unable to view available plugins

Clicking "Manage plugins" throws the error "Unable to fetch plugins: error"

This is no longer the case.


lluminate\Database\Grammar::parameterize(): Argument #1 ($values) must be of type array, string given

Hello,
I encountered an issue while saving a new category in my application. Here's the complete error log:
Illuminate\Database\Grammar::parameterize(): Argument #1 ($values) must be of type array, string given, called in /Users/poseidone/projects/oa-cms-laravel-filament/vendor/laravel/framework/src/Illuminate/Database/Query/Grammars/Grammar.php on line 1180
Context:
I have a custom resource that extends the CategoryResource from Filament.
My custom resource uses a custom model, which extends the original Filament Category model. This customization is necessary to make the database fields translatable.
Problem:
When I attempt to save a new category, the error above is triggered. However, when I debug the code, I see that the value being passed appears to be an array, which should be correct.
P.S.
I'm using

Filament v.3
FBlog
Translatable Inline

Does anyone have insights into what might be causing this issue or how to resolve it?
Thank you for your support!

Hey,
it's hard to help with this information.
Please provide a separate repository that only contains the code that reproduces your problem.
Regards


Medication inventory Status

This pull request introduces the drug status feature, enhancing pharmacist inventory management. With the appropriate permissions enabled via the admin page, pharmacists can now access an inventory status page to review available stock levels for selected drugs. Users without the "check inventory status" permission will not have access to this page, ensuring secure role-based visibility.
On the inventory page, pharmacists can select drugs to view their current quantity and stock status. A threshold system indicates whether a drug is "In Stock," "Low Stock," or "Out of Stock," based on predefined levels. This feature lays the groundwork for future enhancements, including automated notifications to alert pharmacists or pharmacy managers when stock levels are low.

I would recommend also allowing the Pharmacy Manager the permissions that the Pharmacist has. It seems to make sense to me that the Pharmacist Manager should have all permissions available.


Update best-practices.md

Add links to official CLong and CULong documentation.
/cc @carlossanlop @danmoseley @gewarren

@carlossanlop Is there something wrong with my doc as you see it? I found the doc at https://docs.microsoft.com/dotnet/api/system.runtime.interopservices.clong and https://docs.microsoft.com/dotnet/api/system.runtime.interopservices.culong.

Apparently the casing is important here - yay.
Any concern @gewarren?


Remove deprecated bower option: version

version option for bower.json it;s deprecated and it's already ignored by bower, as you can read in bower.json specification.

didn't realize. thanks!


translate reference/react/cache into Chinese

https://github.com/reactjs/zh-hans.react.dev/blob/main/src/content/reference/react/cache.md

Working.


Cow: Watch skyfire.xyz

Cow requests the watch of the watch_keyword skyfire\.xyz. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

skyfire\.xyz has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


STDIN is not closed

Not really a big problem (mostly a curiosity, if you will), but something that I also fixed in hyperfine at some point. Try:
bash /tmp/time.sh read

Thank you for the suggestion! This is a very easy change.

I made the change in https://github.com/tdulcet/Benchmarking-Tool/commit/d6908b6219cc1ec1d3755e693b9034fd24cd9937. The read command now produces an error like with hyperfine.

:+1: Thanks.


impv: Correct datax execute and python execute script name

we use PYTHON_LAUNCHER for python script execute and DATAX_LAUNCHER for datax script name

Codecov Report

Merging #14523 (6200d13) into dev (3e3e859) will decrease coverage by 0.03%.
The diff coverage is 46.66%.

:exclamation: Current head 6200d13 differs from pull request most recent head a96e609. Consider uploading reports for the commit a96e609 to get more accurate results

@@             Coverage Diff              @@
##                dev   #14523      +/-   ##
============================================
- Coverage     38.75%   38.72%   -0.03%
+ Complexity     4579     4573       -6
============================================
  Files          1237     1236       -1
  Lines         43534    43486      -48
  Branches       4822     4791      -31
============================================
- Hits          16871    16842      -29
+ Misses        24791    24776      -15
+ Partials       1872     1868       -4

Impacted Files
Coverage Δ

...heduler/api/service/impl/SchedulerServiceImpl.java
29.64% <0.00%> (-0.49%)
:arrow_down:

...duler/plugin/task/api/AbstractCommandExecutor.java
0.00% <ø> (ø)

...nscheduler/plugin/task/api/utils/ProcessUtils.java
0.00% <0.00%> (ø)

...cheduler/plugin/task/pytorch/PythonEnvManager.java
86.66% <ø> (ø)

.../dolphinscheduler/plugin/task/datax/DataxTask.java
33.75% <100.00%> (-0.56%)
:arrow_down:

...inscheduler/plugin/task/openmldb/OpenmldbTask.java
77.27% <100.00%> (ø)

...olphinscheduler/plugin/task/python/PythonTask.java
14.00% <100.00%> (ø)

...heduler/plugin/task/pytorch/PytorchParameters.java
59.09% <100.00%> (ø)

...phinscheduler/plugin/task/pytorch/PytorchTask.java
50.00% <100.00%> (ø)

... and 37 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Shana/sc13936

Adding jaeger tracing and enabling tracing for shipping service
[ch13936]

will create a new PR without agent jar in repo


Folding functions with multiline returns leaves closing bracket

Environment data

Language Server version: v2022.4.3
OS and version: Windows_NT x64 6.1.7601
Python version (& distribution if applicable, e.g. Anaconda): 3.9.10

Expected behavior
Function where the return statements span multiple lines should fold into a single line.
def test():
    return (
        "asd"
        .upper()
    )
# should fold into
# def test()

Actual behavior
Function where the return statements span multiple lines will leave the closing bracket unfolded when folding.
e.g.
def test():
    return (
        "asd"
        .upper()
    )
# folds into
# def test(
#     )

Code Snippet / Additional information
Note that this way of formatting multiline returns (is that even the correct name for this code?) is the default of black for long chained function calls.
If this function is part of a class, folding the class will also leave this bracket remaining.
Might be related (or even the same problem) as https://github.com/microsoft/pylance-release/issues/2333

This issue was fixed in prerelease version 2022.9.21 and is now also available in release version 2022.9.30.


Python 3 conversion incomplete

I realize I didn't deal with the difference in metaclass syntax. It looks like there's no trivial fix here. Would you be willing to accept a dependency on futurize (this seems to be the successor to six for writing 2&3 compatible code).
Here's the Python 2 and 3 compatible way to do metaclasses: http://python-future.org/compatible_idioms.html#metaclasses

I must admit I've been merging without testing :)
I don't mind dependencies if they're pip-installable. Getting things to work with Python 3 has a much greater benefit than any minor cost from a dependency. Judging from the link, this would make the syntax kind of gross, but that seems fine too as long as it works!
Are you going to take a stab at it?

Yes, I'll get something to you. I'll just use the with_metaclass dance.

Any progress on this? I'm working on doing something similar with @slinderman's pyhawkes, and I think I'm running into some issues based on this dependency since it seems to run into unique errors regarding importing submodules when importing from pybasicbayes when running on python3 but not 2.

Everything works under Python 3 right now. The abstractmethod isn't being enforced though. I'll try to finish this tomorrow.

Excellent — thank you for this!

Btw I pushed an updated version to pypi in the last few days.

I noticed — that's why i was asking whether dependencies elsewhere should
be updated to require the newest version, but you explained that the
abstraction version should be backwards compatible which I trust you about.
On Mon, Aug 17, 2015 at 6:18 PM, Matthew Johnson<EMAIL_ADDRESS>wrote:

Btw I pushed an updated version to pypi in the last few days.
—
Reply to this email directly or view it on GitHub
https://github.com/mattjj/pybasicbayes/issues/21#issuecomment-132017900.

It looks like I added 'future' to the setup.py, so I think everything
should work.
On 08/17, Michael Pacer wrote:

I noticed — that's why i was asking whether dependencies elsewhere should
be updated to require the newest version, but you explained that the
abstraction version should be backwards compatible which I trust you about.
On Mon, Aug 17, 2015 at 6:18 PM, Matthew Johnson<EMAIL_ADDRESS>wrote:

Btw I pushed an updated version to pypi in the last few days.
—
Reply to this email directly or view it on GitHub
https://github.com/mattjj/pybasicbayes/issues/21#issuecomment-132017900.

Reply to this email directly or view it on GitHub:
https://github.com/mattjj/pybasicbayes/issues/21#issuecomment-132085326


New SEM Pages - Content Rendering Issues

Main Content section not rendering properly. Need to add inline form for two column layout. Ensure icons and text lay cleanly with background image below (currently overlapping)

Should look like:

Oh @colinvlasak apologies, i closed this issue according to the original issue list! Thanks for adding some more context!

@MarquiseRosier looks much better. Can we move the background image up a bit to match live?
EDS:

Live:


Composite Tests + Benching

Addresses:

#7 add benching
#10 add badges + update readme
#6 add tests for simple compositing conditions

cc @artemp @mapsam

@millzpaugh @artemp I added https://github.com/mapbox/vtcomposite/pull/11/commits/5f4fc8616090c68520c6a100a465d8d32d388e27 which updates the package-lock.json to include vtinfo/mapnik, which weren't being installed since the package-lock.json was present but not updated. Also moves vtinfo/mapnik to devDependencies so we can reduce downstream users package sizes.

Looks like it wasn't truly the package-lock.json file, but a result of installing modules using npm install --ignore-scripts, which intentionally avoids executing any package.json scripts like preinstall or postinstall, which is what we use to execute node-pre-gyp for our addons. If we don't execute this we don't go and fetch the pre-built binaries (or fallback to build if they don't exist) and therefore we run into errors like:
Error: Cannot find module<EMAIL_ADDRESS>    at Function.Module._resolveFilename (module.js:476:15)
    at Function.Module._load (module.js:424:25)
    at Module.require (module.js:504:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/mapsam/mapbox/vtcomposite/node_modules/@mapbox/vtinfo/lib/vtinfo.js:8:31)
    at Module._compile (module.js:577:32)
    at Object.Module._extensions..js (module.js:586:10)
    at Module.load (module.js:494:32)
    at tryModuleLoad (module.js:453:12)
    at Function.Module._load (module.js:445:3)

Per discussion with @millzpaugh we're going to move the vtinfo dependency over to vector-tile-js to remove any C++ testing dependencies.

Outside of the scope of this PR, but one question burning in my head is:
*Can a node addon effectively depend on a node addon?

thanks for digging into that! and surfacing a meta question about node add-ons...
just subbed out vtinfo to vector-tile-js - no longer running into dep errors, but tests are surfacing what seems to be a memory issue. (happens both locally and on travis.)
*** Error in `node': free(): invalid pointer: 0x00000000027a56f8 ***
Aborted (core dumped)

@millzpaugh Looks like something is trying to write to some memory it doesn't own. Have you tried to run under valgrind or gdb?


feat: allow to set default InputValueSerializer

Extended GraphQLProjection to allow setting custom InputValueSerializer per thread.
This change will allow to use any scalars with DgsClient client and setup them once.
@Test
fun test() {
    InputValueSerializerProvider.serializer = InputValueSerializer(
        mapOf(UUID::class.java to UUIDScalar.INSTANCE.coercing)
    )
    /* line below would fail before */
    val query = DgsClient.buildQuery {
        movies(filter = UUID.randomUUID())
    }
}

closes #455

cc @mbossenbroek

Thanks @dwilkolek for adding this! 👏

No problem, I find this project fascinating. I'll keep an eye for good first issues 😉


Improve some portuguese expressions.

Small changes in some words to improve the main idea clearly.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


Not working anymore - Windows 11 Beta?

Text-Grab stopped working and I guess it has something to do with the latest beta release of Windows.
I have tried several versions and the store version, none of them works.
I'm on Windows 11 Beta version 22621.160.
Any suggestions?

Hello, sorry to hear Text Grab stopped working for you, could you be more specific as to how much of Text Grab stopped working?
Does a window pop up? what mode were you trying to use (Fullscreen grab, Grab Frame, Edit Text Window)? What type of computer do you have?
I did test out Text Grab on the Windows Insider Dev channel and Release Preview with no issue. Looking forward to your response.

Hello
Thanks for a fast reply.
I have tried both Fullscreen grab and Grab frame. Both shows and I can select a rectangle in full screen, and words in grab frame.
There is a bit longer delay than  when it worked but then the full screen overlay disappears and grab frame also disappears but nothing is copied to clipboard.

If you open the "Event Viewer" and under "Windows Logs" on the left hand pane, click "Application" then the crashes should show up. I am not seeing any crash data in the developer dashboard for the Microsoft Store version. It could be possible there is a conflict with the version of .NET 6, but that shouldn't not affect the Store version of Text Grab. Hopefully the crash in the Event Viewer will give more information and explain why it is crashing.

There is no crash events in the application log. I tried several "grabs" in a row and sometimes it works and next time I try the same words it doesn't work. I have used it in several applications to get a clear black on white text but that doesn't help.
It's like 1 read and 5 nothing. It feels like OCR can't read it?

What is your keyboard language set to? What language are you trying to OCR? Can you provide reference images?

My keyboard language is swedish. I have tried both english and swedish. It doesn't have any problems with swedish characters like ÅÄÖ.

This one, from the Event Viewer, worked one time of four tries.

Program
Säkerhet
Installation

Hello
My settings page doesnt look like yours, I'm on Windows 11 Beta. But here it is.
I have tried other OCR apps I have found and they can read the text, but that is done by saving a screenshot and then loading the picture into the app.

Hello again
I ran TextGrab in Visual Studio debug mode and got this error:
System.Runtime.InteropServices.COMException
{"OpenClipboard Failed (Exception from HRESULT: 0x800401D0 (CLIPBRD_E_CANT_OPEN))"}
Line 323 in fullscreengrab.xaml.cs     Clipboard.SetText(grabbedText);
Maybe this can explain why it doesn't work for me.
I changed the line to Clipboard.SetDataObject(grabbedText); as recommended by several sources.
So far it is working 👌

This Stackoverflow answer has a helpful explanation. Not sure why SetText would be crashing, but SetDataObject is working. Anyway I some testing and it is working well, Text Grab will use SetDataObject to add the text to the clipboard.

I glad it's solved, it's a great app.


Crash from skeleton arrow firing through portal?

I wasn't able to reproduce it with just Immersive Portals in singleplayer, but a friend of mine was being chased by a skeleton on his server (dedicated) and he ran away through the portal, and was met with this crash shortly after.
Crash report:
https://pastebin.com/pe5wRyxZ

I added the check. It probably won't crash in the next version


Title: A Friendly Beginner's Guide to TypeScript

Article Checklist
NOTE: (1st time contributors - submit ONLY one article to get familiar with the process, quality, and style-once published, you can then submit any number of articles.)
Preliminary Checks - Formatting and Structure

[ ] Does your article follow any of the suggested structure formats? - see example formats
[ ] Is your article properly formatted in Markdown? - see Github Markdown guide
[ ] Have you used the correct folder and file structure? - see Contribution Guidelines
[ ] Is your article over 750 words? - Tool: Wordcounter.net
[ ] Have you used our preferred heading structure? - H3 (###) and up
[ ] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.
[ ] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.
[ ] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images
[ ] In your PR - add a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.

Grammar Checks

[ ] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid -  Grammarly
[ ] Have you checked your article for readability? - Tool: Hemmingway
[ ] Have you added sources for quotes and images that aren't yours?

Technical Checks

[ ] Are your code snippets properly formatted for syntax highlighting - see Syntax guide

Contribution Guidelines
For first-time contributors and for more details, see Contributing Guidelines
@hectorkambow here is the new clean PR.
fixes #1291

@hectorkambow a peer reviewer. Wishing to review this Typescript article

@hectorkambow a peer reviewer. Wishing to review this Typescript article

@hectorkambow a peer reviewer. Wishing to review this Typescript article

Didn't notice. Sorry 😅

@hectorkambow a peer reviewer. Wishing to review this Typescript article

Didn't notice. Sorry 😅

@zolomohan and @ReactifyStudio - This is a duplicate pr. I was reviewing it.
Refer to #1311

@zolomohan and @ReactifyStudio - This is a duplicate pr. I was reviewing it.
Refer to #1311

😂😅

😂😅

@zolomohan you can continue with this. My PayPal chaos continues. No need to.

@zolomohan you can continue with this. My PayPal chaos continues. No need to.

@Olanetsoft Be sure to mention the old PR in the duplicate PR to avoid confusion in the future.

@Olanetsoft Be sure to mention the old PR in the duplicate PR to avoid confusion in the future.

@zolomohan the old PR has been closed by @kayere

@zolomohan the old PR has been closed by @kayere

@kayere Updated Changes

@hectorkambow - Please give this article a final review.

@Olanetsoft
👍
Great overall tutorial - please go over my final edits and changes to incorporate into future articles 🚀
cc @kayere  please do the same 👍


Standardize OpenVR WMR grip press as DeviceInputType.TriggerPress

Overview
It was TriggerPress for one hand and ButtonPress for the other
#5874 assumes TriggerPress, so I standardized on that.
This PR also adds code to allow profiles to double check their default interaction mappings to update when axes / buttons / etc change.

/azp run

/azp run


TypeError: init() got an unexpected keyword argument 'initial_amount'

I got a bug on cell number [18]. Line code number 14 is red underlined.
here is the code which is red underlined:
e_train_gym = StockTradingEnv(df = train, **env_kwargs)
here is the error warning:
TypeError                                 Traceback (most recent call last)
 in ()
12 }
13
---> 14 e_train_gym = StockTradingEnv(df = train, **env_kwargs)
TypeError: init() got an unexpected keyword argument 'initial_amount'
How to solve this bug? thank's before.

Hi team,
I got the same error and when I commented out the initial_amount line of code I got
TypeError                                 Traceback (most recent call last)
[<ipython-input-52-6f99a2a2ef8c>](https://localhost:8080/#) in <module>()
     12 }
     13
---> 14 e_train_gym = StockTradingEnv(df = train, **env_kwargs)

TypeError: __init__() missing 1 required positional argument: 'initial_list'

And in the source code, I saw that the Initial_amount is commented out, is it as simple as un-comment that line? If so I would be happy to submit a PR
Commented out initial_amount line of code

I see, there is a PR to revert the change that commented initial_amount
PR LINK

Trying to get started and cant run the initial notebook because of  this error.

This was also discussed a while back here: https://github.com/AI4Finance-Foundation/FinRL/issues/495
It was closed, but there is still a problem to be addressed.

Using this change I did manage to run the next few lines but when trying to train_model i get
TypeError: 'float' object is not subscriptable
Seems to be caused by something on
available_amount = self.state[0] / (self.state[index + 1]*(1 + self.buy_cost_pct[index]))
Any ideas?

I have submit a PR which has been merged and fix this bug in tutorials/1-Introduction/FinRL_StockTrading_NeurIPS_2018.ipynb

developpers updated the code today: add the num_stock_shares, which should be initialized as [0] * stock_dimension

"initial_amount" should be set 1000000

That works. Thanks!

@PierrickI3  Could you push a PR for this problem?

Fixed. Thanks.

Not sure how I can do that.
Here is the final version of my Trade Data Split step which works:
buy_cost_list = sell_cost_list = [0.001] * stock_dimension
num_stock_shares = [0] * stock_dimension

env_kwargs = {
    "hmax": 100,
    "initial_amount": 1000000,
    "num_stock_shares": num_stock_shares,
    "buy_cost_pct": buy_cost_list,
    "sell_cost_pct": sell_cost_list,
    "state_space": state_space,
    "stock_dim": stock_dimension,
    "tech_indicator_list": config.INDICATORS,
    "action_space": stock_dimension,
    "reward_scaling": 1e-4
}
e_train_gym = StockTradingEnv(df = train, **env_kwargs)

Where did you put this in the code? I'm having issues where it says:
Traceback (most recent call last):
File "C:\Users\User\PycharmProjects\Trader\MultipleStockTrader.py", line 453, in
buy_cost_list = sell_cost_list = [0.001] * stock_dim
NameError: name 'stock_dim' is not defined
Would really appreciate the help! I'm a beginner :)

@flyingsandwich1  which notebook did you run?  Have you revised anything?


Elasticsearch parent/child relationship

Hi there
Wonder if there is a way to define a parent when syncing from mongo to elastic search. I could not find any informations relating to this topic.  How do I go about syncing a child doc with a parent doc
Elastic mapping
mappings: {     family: { ... },     container: {        _parent: { type: 'family' },       ...     }  }
example data
Family: {  id: 1,  name: 'familyName'  ... }
Container: {  id: 100,  familyId: 1,  ... }
Here is the basic pipeline i'm using
var source = mongodb({
"uri": "${MONGODB_URI}"
})
var sink = elasticsearch({
"uri": "${ELASTICSEARCH_URI}"
})
t.Source("source", source, "/family|container/").Save("sink", sink, "/.*/");`

thanks for the detailed report! I'm thinking adding the rename function will handle this case. Can you try the following:
var source = mongodb({
"uri": "${MONGODB_URI}"
})

var sink = elasticsearch({
"uri": "${ELASTICSEARCH_URI}"
})
t.Source("source", source, "/family|container/")
  .Transform(rename({"field_map": {"familyId":"parent"}}), "/container/")
  .Save("sink", sink, "/.*/");

thanks for the reply, unfortunately that doesn't seem to work.  The only records created are the ones for Family and not containers.  I can see the records being process when I use the pretty({}) transform and the field name "familyId" is being rename to "parent":.
 t.Source("source", source, "/family|container/")
  .Transform(rename({"field_map": {"familyId":"parent"}}), "/container/")
  .Transform(pretty({}))
  .Save("sink", sink, "/.*/");

the data
Family: { id: 1, name: 'familyName' ... }
Container: { id: 100, parent: 1, ... }

sorry, after digging into the docs for parent/child associations in elasticsearch, we'll have to make some changes to transporter to support it. today, it makes a naive assumption about the structure and fields.
at a minimum, we'll need to expose configuration options for what field the id should come from and if a parent association should be used. both of those config options would need to be namespace scoped.
is this something you'd be about to help with? also, what version of elasticsearch are you targeting?

The version of elastic i'm using is 5.4.  I haven't done any go programming so I'm not sure how helpful I will be.  I assume adaptor/elasticsearch/clients/v5/writer.go is the file I need to look at right?  I'll try digging into the code a bit and read up on the go programming language.  Can you point me to which file or files I should be looking into.
adaptor/elasticsearch/clients/v5/writer.go: line 92
elastic.NewBulkIndexRequest().Index(w.index).Type(indexType).Id(id).Doc(msg.Data())

this seems like the code where I would need to add: .Parent() after the Id function but I'm not sure how to go about passing in the additional information needed.

yea, you're definitely on the right track
so, the following bit of code
br = elastic.NewBulkIndexRequest().Index(w.index).Type(indexType).Id(id).Doc(msg.Data())

returns the *BulkIndexRequest type so you could have an if statement to check for the presence of a parent field in msg.Data() (similar to https://github.com/compose/transporter/blob/master/adaptor/elasticsearch/clients/v5/writer.go#L80) and do the following:
br.Parent(msg.Data()["parent"].(string))

addressed via https://github.com/compose/transporter/pull/406


Fix clippy warnings

This will make the already very educational code more readable.
Please tag this issue as 'Hacktoberfest' :smile:
Thanks so much for your "Let's build a browser engine!" series. Really cool.

PR #14 is part of this issue

PR #16 also is part of this issue :+1:

And one more, PR #17

Thank you!

My pleasure. Thanks for merging.


refactor to directly return mayBeNullFieldAccess

the method already calls onOverrideMayBeNullExpr and nullnessFromDataflow so there is no point to break out of the switch to call the former again
see also other usage of mayBeNullFieldAccess a few lines below
also remove comments that just restated the obvious

obsolete after the other PR got merged


Feat/swagger auth

add swagger authorization

add auth button in swagger
add auth url to get token (逾7天更新一次，若無token則新增，有token直接取得- one user vs one token)
update core.models.Token(預設 one user vs multiple token)
update BearerAuthentication function(swagger authorization)
update api import BearerAuthentication

Hello @Dawn0472, thank you for working on this. There are some coding style issues, could you help me fix it, thanks!

Codecov Report
Patch coverage: 65.82% and project coverage change: +0.01 :tada:

Comparison is base (4b520ac) 69.56% compared to head (287fc55) 69.57%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1141      +/-   ##
==========================================
+ Coverage   69.56%   69.57%   +0.01%
==========================================
  Files          82       87       +5
  Lines        3364     3444      +80
==========================================
+ Hits         2340     2396      +56
- Misses       1024     1048      +24

Impacted Files
Coverage Δ

src/core/models.py
95.71% <ø> (-0.49%)
:arrow_down:

src/users/api/views.py
36.66% <36.66%> (ø)

src/pycontw2016/urls.py
84.21% <66.66%> (-1.51%)
:arrow_down:

src/core/authentication.py
81.25% <79.31%> (-18.75%)
:arrow_down:

src/attendee/api/views.py
42.85% <100.00%> (ø)

src/events/api/views.py
52.00% <100.00%> (ø)

src/sponsors/api/views.py
41.17% <100.00%> (ø)

src/users/api/urls.py
100.00% <100.00%> (ø)

... and 6 files with indirect coverage changes

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


Add a way to filter tests

For cli:
--test-name-pattern/--testNamePattern/-t Node, Jest, Vitest
--test-skip-pattern Node
I also suggest adding these options to the Poku runner

This idea sounds familiar to me as a --filter.
Could you show me a practical example?
Depending on how this functionality works, it could even replace --filter.

filter - seems to filter by file name
test-name-pattern - by test name
for example https://poku.io/docs/tutorials/cross-platform#using-poku-api-advanced-concept
if run
npx poku -t bun

should only run tests that contain text bun


Parameters with implicite type and range

TEST:
module bug();
  localparam signed [63:0] n = 32'h8000_0000;
  localparam signed [63:0] n2 = n + 1'b1;
  localparam signed [63:0] n3 = 64'h8FFF_FFFF_FFFF_0000;
  localparam signed [63:0] n4 = n3 + 64'h0000_0000_0000_FFFF;
endmodule

There is no information about range of the parameter inside UHDM:
   |vpiParameter:
   \_parameter: (work@bug.n), line:2:28, endln:2:45, parent:work@bug
     |vpiName:n
<EMAIL_ADDRESS>
     |vpiLocalParam:1
     |HEX:80000000

also math operations (on parameters/localparam with implicite type and range) on more than 32bits results in wrong result in UHDM.
Same issue applies to parameters.

Fixed by #2465


Utilizing async 2.6.4 for node 4 compatibility

Good afternoon Winston developers,
This PR is to restore node 4 compatibility while maintain the CVE fix linked here: https://security.snyk.io/vuln/SNYK-JS-ASYNC-2441827
This is in regards to this issue: https://github.com/winstonjs/winston/issues/2130
I attempted to update the testing, but the 2.X branch does not have the CI/CD workflow. So I tested it manually and it passed.
Please let me know if you have any questions.

will there be a published version of 2.4.7?  for example i only up to 2.4.6 at https://www.npmjs.com/package/winston.

@lexinator done!


Able to exchange two regions selected visually with multiple-cursors

Basically if you press ctrl-d on a word, in selects the other words visually and when press c changing them at the same time, like refactoring or in multiple lines. More data at
https://github.com/terryma/vim-multiple-cursors

So would the desired workflow be something like this?

Use multiple-cursors to select multiple occurrences of the same word
Type X to mark the multiple selections for an exchange
Use multiple cursors to select another set of a different word
Type X to invoke the swap of all occurrences of the first word with all occurrences of the second word

Sounds pretty interesting and useful. I don't want to get your hopes up right now, as there's a lot more I need to know before I can judge whether this is feasible. My gut feeling right now is that it will be too complicated to support, and building a one-off specialized solution is probably a better bet.
I can think of a way to achieve this result with a different workflow using another plugin. Abolish.vim provides a :Subvert command (can be shortened to :S) which can help to do a pairwise swap of words. For example, you could use this:
:%S/{foo,bar}/{bar,foo}/gc

That will replace all foos with bar and all bars with foo (the c flag makes it ask for confirmation at each occurrence). It's admittedly not as easy as the workflow you're proposing, but it could fill the void for now.
Some things I need to consider:

How does multiple-cursors actually execute the command?

Does it work for all Visual mode commands?
Does it apply the command to each execution serially?

To support this, I think exchange.vim would need to have a way to let the user specify whether the operation is to mark an operand for exchanging or perform the actual exchange. Right now, it uses the same operator for both.

Actually yes, but multiple-cursors is working in hybrid-X mode. I stopped to use vim-multiple-cursors and i started to use vim-visual-multi. (mg979/vim-visual-multi). Actually its much more improved, also it can uses default visual mode mappings that means it can run every map. So after pressing , it deactivates mode but cursors is stays selected already, and you can do operations on them. I dont know too much things about it. I'm not sure if it would work.

I'll request mg979 to add support for this plugin because understanding of visual multi plugin should be hard. For him much easier.


Request times out

Is this a bug report?
Yes. Edit: No, see comment.
Steps to reproduce
Start proxy, call URL as seen in link below.
Expected behavior
I expected http://localhost:9004/webservices/gis/aktuell/wfs?service=wfs&version=1.1.0&request=GetFeature&typeName=gk:waterlevels&outputFormat=json&Filter=<Filter><Or><PropertyIsEqualTo><PropertyName>water</PropertyName><Literal>ESTE</Literal></PropertyIsEqualTo></Or></Filter> would return the same result as https://pegelonline.wsv.de/webservices/gis/aktuell/wfs?service=wfs&version=1.1.0&request=GetFeature&typeName=gk:waterlevels&outputFormat=json&Filter=<Filter><Or><PropertyIsEqualTo><PropertyName>water</PropertyName><Literal>ESTE</Literal></PropertyIsEqualTo></Or></Filter> does.
Actual behavior
The request hangs until it times out eventually.
Setup
express: 4.17.1
http-proxy-middleware: 0.19.1
Node: 10.16.0
client info
Windows 10, current Chrome / Firefox
target server info
https://pegelonline.wsv.de/webservices/gis/aktuell/wfs is a WFS service providing feature information about water levels.
Reproducible Demo
const proxy = require('http-proxy-middleware')
const app = require('express')()
const port = 9004

app.use('/webservices', proxy({
  target: 'https://www.pegelonline.wsv.de/',
  changeOrigin: true
}))

app.listen(port, () => console.log(`Listening to port ${port}.`))

https://github.com/isthisstackoverflow/repro-waterlevel
Also
I tried various options from the docs, but most of them seem not to be relevant; trying them anyway did not fix the issue. The shorthand results in the same behaviour, too.

I figure the problem must be me trying to proxy https to http. While I did not find a solution to do it, I also no longer consider this a bug report, but a fault on my side.


"Chioibășești" is listed in the drop-down menu

Description:
"Chioibășești" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Chioibășești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Chioibășești, Brăila (Ciocile)".


Revert merge

https://github.com/microsoft/pxt/commit/13d0cb389c717a92276c14319d04d9662380aa15 Accidentally reverted a bunch of changes on master.
Reverting back to the minor bump before that change, and then re-cherrypicking the three commits that came afterwards:

2bb67b590052ec0b96237d7ba6da9bc0c269435f
80833d6f365bdc6fb8bdd1a77f2c1b2eaec06db8
e82ba342119f9222953983445c5a69bcb3f9cc11

Since this is hard to review, here are the exact commands I ran:
git checkout origin/master -b revert-merge
git checkout 948388b641d889a636fff9cf34c67d343a3c7bd2 -- .
git commit -m "Reverting back to 948388b641d889a636fff9cf34c67d343a3c7bd2"
git cherry-pick 2bb67b590052ec0b96237d7ba6da9bc0c269435f
git cherry-pick 80833d6f365bdc6fb8bdd1a77f2c1b2eaec06db8
git cherry-pick e82ba342119f9222953983445c5a69bcb3f9cc11

will you manually put the minor version afterwards? need to skip 6.11.2

@pelikhan will do

https://github.com/microsoft/pxt/pull/7966


[TECH] Migrate the repository on YARN berry

Information
Migrate all workspaces on yarn berry. See the migration guide here:
https://yarnpkg.com/getting-started/migration
You can take as example the @tsed/logger repository.
Acceptance criteria

[ ] All test pass
[ ] The repository build

I am working on it. Almost complete. There may be documentation to update and other small details.


Shorten the list of items in "Related Submissions" box, to only show 5 by default with a show more when there are more

See https://apply.opentech.fund/apply/submissions/3498/, there's waaaaaay to many showing there.

Current test shows more than 5 previous submissions.

@fourthletter Yes, the list is cut off by pixels and not item count. Most titles are long and takes up more than one line so I set it to 300px.

This issue was visible and functional during testing. Thank you!


Gherkin Rule keyword and Protocol Buffer messages

This PR adds a Rule keyword to the Gherkin grammar. The GherkinDocument AST also has a Rule node. The AST itself, as well as Source and Pickle messages all use Protocol Buffers.
This PR supercedes #378 and #414, and implements #250.
TODO:

[ ] Implement messages library

[x] Java
[x] JavaScript
[ ] .NET
[x] Go
[x] Ruby

[ ] Update ASTBuilder to use messages and rename to GherkinDocumentBuilder

[x] Java
[x] JavaScript
[ ] .NET
[ ] Go
[ ] Ruby

[ ] Remove obsolete event-protocol module

[ ] Move docs over to messages/README.md

From #378 @charlierudolph:

Cool. @aslakhellesoy how can I help with this? I wanted to get back to the protobuf work.

It would be great if you could implement the messages library in Go, and then update ast_builder.go and pickles.go to use it. events.go and ast.go can be removed.
I'll crack on with the JavaScript implementation.

@SabotageAndi and/or @gasparnagy are you up for doing the .NET stuff?
@brasmusson or anyone else in @cucumber/cucumber-ruby - who wants to work on the Ruby code?

@aslakhellesoy I am full with work for the next 2-3 month. After that, perhaps. Hopefully @gasparnagy can work on it earlier.

I can work on Ruby
Any insights @aslakhellesoy?

@aslakhellesoy trying to use docker-run locally wasn't working for me, so I had to install protoc locally for development. Is this working for you?
$ ../../scripts/docker-run ../../Dockerfile.cucumber-build make
go get github.com/stretchr/testify
go get github.com/golang/protobuf/protoc-gen-go
protoc --go_out=. gherkin.proto
bash: /usr/local/bin/protoc: No such file or directory
make: *** [Makefile:20: gherkin_pb.go] Error 127

For updating the go gherkin:

I think I need the subrepo for messages to exist so it can be pulled, otherwise not sure how to bring that in
Wanted to double check if you're good removing: https://github.com/cucumber/cucumber/blob/master/gherkin/go/astjson.go

Thanks a lot @charlierudolph! I have been building on my host machine (MacOS) and you're right we need protoc in the docker image.
Since the image is based on Alpine Linux, the protobuf version you've tried to install won't work. There is a protobuf APK package, but the latest version of it does not support option ruby_package. Alpine 3.8 has a more recent version which should support it, but Alpine 3.8 is not dockerised yet :-(
I see a few options here:

Build generated sources locally and check them in
Build the latest protobuf on Alpine
Switch to a different Linux

I'll experiment a little more with this.


Deprecate?

Now that optional chaining is almost everywhere, I think this should print out a deprecation warning.

I agree with you in spirit. But I think the intention is better served using a lint warning to suggest that people use ?. or ?? instead of idx. It's also important that developers switching between the two recognize the functional differences between them. (In places where the former may return undefined, idx may return null.)

A codemod can work even better.

A codemod is not possible because the behavior of ?. is not identical to idx. Closing for now.


Remove machete-pre-rebase hook from this repo

Ofc the support for the hook itself in the plugin should stay as is.
Plus update the docs accordingly.

why?

Needless complication, in hindsight we don't really need to rewrite the versions back on every commit with git filter-branch... just bumping on the last one is enough (esp. given that all these commits will typically be squashed at the end anyway).
The worst that can happen is a non-monotonic sequence of versions in git history... but still, git describe will display the version properly, based on git tags.

Done on #904


Need to read the stream in only 3 channels i.e. RGB not 4 channels i.e. RGBA. How to do it?

I'm doing object detection using the TensorFlow object detection API. For that, the detection model accepts the input tensor in the shape [<tf.Tensor 'image_tensor:0' shape=(None, None, None, 3) dtype=uint8>]; but after using CudaToNumpy the shape is (None, None, None, 4). I tried to convert 4 channel feed to 3 channel feed but there is a loss of information in that.
Captured by gStreamer, original image with 4 channels:

After conversion to 3 channels:

Please tell me if there is any way to convert the 4 channels to 3 channels without losing the information.
For this conversion, I have used :
img = cv2.cvtColor(img, cv2.COLOR_BGRA2BGR)
and passed the numpy array as img.

Use img = cv2.cvtColor(img, cv2.COLOR_RGBA2RGB)


Difference between Lite and Normal version

Hello,
The README should explain the difference between "kot-stream-theme-lite-vscode" and "kot-stream-theme-vscode", because the word Lite can have 999 meanings. For example, Lite can mean Free as opposed to a paid version; Lite can mean Fast-Loading as opposed to a version that is heavy in resources; etc. Looking at the code, the only difference (for now) is the styling of variables. In the Lite version it's:
"foreground": "#000000",
In the Normal version it's:
"foreground": "#6A027E",
"fontStyle": "bold",

Hello!
It interesting thought., in next updates I'll add more description about this themes.
Thank you!


hp_test

@niuqg Thank you for the pull request! But, what exactly is it for?

I'm going to close this now. Feel free to reopen.


Simplify UI scaler

Renaming config option will reset scale for everyone.

I see, how do we handle that?

we don't care about the user?
we don't care about the code?
:D

I see, how do we handle that?
1. we don't care about the user?
2. we don't care about the code?

Reasonable default
or keep old variable, give new variable default -1 for example, and if it is -1, we migrate value to new on the first load. This migration code can be removed in the next release.

I see, how do we handle that?

we don't care about the user?
we don't care about the code?

Reasonable default for example 100% zoom
or keep old variable, give new variable default -1 for example, and if it is -1, we migrate value to new on the first load. This migration code can be removed in the next release.

To be honest, in that case I'd probably leave the variable name as is since its too much hastle for too little gain. I did not consider that when I proposed to rename the variable. But your solution also sounds fine.


doesn't work with current django 1.6, likely not with 1.5

When running the syncdb command from the install instructions, I get:
...
ImportError: No module named markup
Seems to come from the django.contrib.markup import - it's deprecated as of 1.5:
https://groups.google.com/forum/#!topic/django-users/iwfuDFFAltI
There seems to be an alternative, but that requires adjustments:
http://stackoverflow.com/questions/20055603/how-can-i-satisfy-a-django-contrib-markup-templatetags-markup-import-restructure

Should now work with Django 1.8


Switch to stable Rust and tokio 0.2.4

Added

Add dependency on async-trait.

Changed

Upgrade to stable Rust, async/await, stable futures 0.3.1, and stable tokio 0.2.4.
Upgrade patch versions for dieseland r2d2.

Removed

Remove nightly rust-toolchain file.
Remove rustfmt.toml with nightly-only features

Thanks for the handy crate and async abstraction, @mehcode! I was looking for a crate like this for use in a personal project, and I was disappointed to see this one hadn't been updated in a while. This pull request should get it fully up to speed with upstream, and the simple example seems to work fine without modification.
I would appreciate your review and feedback, and I look forward to a new release on Crates.io sometime soon!
This pull request supersedes #5.

Thanks a million for the update.


Node stuck in NotReady status after temporary interface disconnect

Environmental Info:
K3s Version:
k3s version v1.22.2+k3s1 (10bca343)
go version go1.16.8
Node(s) CPU architecture, OS, and Version:
Linux somehostname1 3.10.0-1160.el7.x86_64 #1 SMP Tue Aug 18 14:50:17 EDT 2020 x86_64 x86_64 x86_64 GNU/Linux
Linux somehostname2 3.10.0-1160.el7.x86_64 #1 SMP Tue Aug 18 14:50:17 EDT 2020 x86_64 x86_64 x86_64 GNU/Linux
Linux somehostname3 3.10.0-1160.el7.x86_64 #1 SMP Tue Aug 18 14:50:17 EDT 2020 x86_64 x86_64 x86_64 GNU/Linux
Cluster Configuration:
3 node HA cluster with embedded etcd.
NAME               STATUS   ROLES                       AGE     VERSION
somehostname1   Ready    control-plane,etcd,master   5m      v1.22.2+k3s1
somehostname2   Ready    control-plane,etcd,master   4m      v1.22.2+k3s1
somehostname3   Ready    control-plane,etcd,master   3m47s   v1.22.2+k3s1

Nothing is deployed on the cluster except coredns and local-path-provisioner.
Describe the bug:
Node is stuck in NotReady status after temporarily disconnecting interface and reconnecting.
Steps To Reproduce:
Set up a three node HA cluster with embedded etcd using k3s version v1.22.2+k3s1.

On one node, bring the interface down: ifdown $interface
Wait till Node goes into NotReady status. Determined by watching kubectl get nodes from one of the remaining 2 nodes.
Bring the interface back up: ifup $interface

The node does not return to Ready, although it appears to be reconnected.
Notes:

I've tested this multiple times - disconnecting both the etcd leader and non etcd leader nodes.
Physically unplugging the ethernet cable, waiting, and then plugging the cable back in has the same effect.
Occasionally it does recover by itself (within a minute). However most of the time it does not.
I've let it sit for over 12 hours without it recovering.
Restarting k3s manually (systemctl restart k3s) does restore the node.
etcdctl endpoint status --cluster indicates that the node is back in the etcd cluster after the interface is brought back up.

kubectl describe node on the "failed" node shows:
Conditions:
  Type             Status    LastHeartbeatTime                 LastTransitionTime                Reason              Message
  ----             ------    -----------------                 ------------------                ------              -------
  MemoryPressure   Unknown   Wed, 06 Oct 2021 14:23:52 -0700   Wed, 06 Oct 2021 14:24:52 -0700   NodeStatusUnknown   Kubelet stopped posting node status.
  DiskPressure     Unknown   Wed, 06 Oct 2021 14:23:52 -0700   Wed, 06 Oct 2021 14:24:52 -0700   NodeStatusUnknown   Kubelet stopped posting node status.
  PIDPressure      Unknown   Wed, 06 Oct 2021 14:23:52 -0700   Wed, 06 Oct 2021 14:24:52 -0700   NodeStatusUnknown   Kubelet stopped posting node status.
  Ready            Unknown   Wed, 06 Oct 2021 14:23:52 -0700   Wed, 06 Oct 2021 14:24:52 -0700   NodeStatusUnknown   Kubelet stopped posting node status.

Expected behavior:
When the interface is reconnected the node should return to Ready status.

I see that you're using the ipsec flannel backend. Do you see the same issue with different a different backend?

Okay, the exit makes sense to me - thanks for explaining.
I ran some more tests and confirmed some things - which you may already know - I'm catching up :)
If I disconnect non-etcd-leader nodes (in which case there is no leader change and no "leaderelection lost"/fatal error on any nodes during failover or recovery) everything recovers fine.
The problem arises only when the fatal error occurs (leaderelection lost) on the node that went down. Furthermore, if I manually restart k3s before the node comes back online it recovers fine when it rejoins.
So all that just confirms what you are saying - fix the panic and restart and it should work.
By the way, thanks for your fast responses on this.

if I disconnect the controller-manager node, I wait a little over 15 seconds for the kube-controller-manager lease to be taken by another node, and then the 30 seconds for my node to go into NotReady status. Is there anyway to avoid the 15 second delay?

I don't believe the controller failover resets the timer, as it's not actually a timer as far as I know. The kubelet updates the status on both its Node object, and the kube-node-lease matching its Node name. That is what the controller looks at to determine whether or not the kubelet is healthy. This is documented at https://kubernetes.io/docs/concepts/architecture/nodes/#heartbeats

Good information thanks.
Found this in my logs when the controller manger picks up on a new node during failover.
node_lifecycle_controller.go:1013] Missing timestamp for Node somehostname1. Assuming now as a timestamp.
node_lifecycle_controller.go:1013] Missing timestamp for Node somehostname2. Assuming now as a timestamp.
node_lifecycle_controller.go:1013] Missing timestamp for Node somehostname3. Assuming now as a timestamp.

and I gave node_lifecycle_controller.go a look
if nodeHealth == nil {
		klog.Warningf("Missing timestamp for Node %s. Assuming now as a timestamp.", node.Name)
		nodeHealth = &nodeHealthData{
			status:                   &node.Status,
			probeTimestamp:           nc.now(),
			readyTransitionTimestamp: nc.now(),
		}

My go is not greatest, but here is what I gathered from the rest of the code. The node controller just stores the probe times in memory (in the node heath objects in a map). On spin up it creates a new map and subsequently sets the last probe times to the current time, which of course has the effect of reseting t0 for the grace period.
I suppose it's cheaper this way than constantly persisting and sharing the last probe time to account for a controller manager shift.
In any case, the failover times should be good enough for now.

Right, node health is nil on startup because the nodeHealthMap hasn't been populated yet. The timestamps will get updated almost immediately as the Node watch sees the initial nodeLister.List results come in via the code path in monitorNodeHealth.  The Kubernetes code has a ton of parallel goroutines and as I said earlier, an eventual consistency model so not everything is in its final state at startup.

Performed the same steps mentioned in https://github.com/k3s-io/k3s/issues/4239#issuecomment-956933405 using master branch commit 702fe24afe3f08f96ebc167313c20c0339a5510f


Return a tailored mutate function with the key already applied

Hi, great library!
It would be a nice convenience if, similar to revalidate, a mutate function were returned from useSWR which already had the key applied, e.g.
-import useSWR, { mutate } from 'swr'
+import useSWR from 'swr'

function Profile () {
- const { data } = useSWR('/api/user', fetcher)
+ const { data, mutate } = useSWR('/api/user', fetcher)

  return (

My name is {data.name}.

      <button onClick={async () => {
        const newName = data.name.toUpperCase()
        // send a request to the API to update the data
        await requestUpdateUsername(newName)
        // update the local data immediately and revalidate (refetch)
-       mutate('/api/user', { ...data, name: newName })
+       mutate({ ...data, name: newName }) // look ma, no key!
      }}>Uppercase my name!</button>

  )
}

I think it will be great to merge revalidate and mutate:
revalidate(newDataOrPromise, shouldRevalidateAfterMutation = true)
const { revalidate } = useSWR('/api/user', fetcher)

// ...
revalidate()
revalidate(newData)
revalidate(newData, false)
revalidate(post)
revalidate(post, false)

Adding support in #245.


Deploy fails on Linux if missing $HOME/.local/share

Problem
With all pre-requisites installed, invoking deploy.py failed with the following error:
INFO:deploy:deploying function app api-service.zip
System.Exception: Cannot find the Local Application Data.
   at Azure.Functions.Cli.Utilities.EnsureCoreToolsLocalData() in D:\a\1\s\src\Azure.Functions.Cli\Common\Utilities.cs:line 180
   at Azure.Functions.Cli.Telemetry.PersistenceChannel.StorageService.VerifyOrSetDefaultStorageDirectoryPath(String desireStorageDirectoryPath) in D:\a\1\s\src\Azure.Functions.Cli\Telemetry\PersistenceChannel\StorageService.cs:line 96
   at Azure.Functions.Cli.Telemetry.PersistenceChannel.StorageService.Init(String storageDirectoryPath) in D:\a\1\s\src\Azure.Functions.Cli\Telemetry\PersistenceChannel\StorageService.cs:line 78
   at Azure.Functions.Cli.Telemetry.PersistenceChannel.PersistenceChannel..ctor(String storageDirectoryPath, Int32 sendersCount) in D:\a\1\s\src\Azure.Functions.Cli\Telemetry\PersistenceChannel\PersistenceChannel.cs:line 42
   at Azure.Functions.Cli.Telemetry.Telemetry.InitializeTelemetry() in D:\a\1\s\src\Azure.Functions.Cli\Telemetry\Telemetry.cs:line 93

This is due to an empty result when resolving the LocalApplicationData special folder here.
On Linux, this .NET enum should resolve to the value of XDG_DATA_HOME, which defaults to $HOME/.local/share. Setting XDG_DATA_HOME to a non-default but existing directory resulted in the same error, but ensuring the existence of the default directory did unblock deployment.
Related (closed) upstream issue: dotnet/runtime#27582.
System
OneFuzz: 2.16.0
OS: Ubuntu 18.04 (WSL1)
Packages:

azure-cli: 2.23.0
azure-functions-core-tools-3: 3.0.3442-1
python: 3.7.5

AB#39994529

This is not a bug in our script, but does impact us, so opening for visibility, mitigation, later follow-up.

Seen again on WSL2, azure-functions-core-tools 3.0.3785.

I am not able to run the deploy command for deploying the instance of onefuzz for the first time. I am getting the following error-
"azure-functions-core-tools" not installed.
When trying to install the azure function core tools, it is throwing a lot of errors saying "missing dependencies"
Is there any better way to install the tools and dependencies together?
I am using the below environment-
Ubuntu WSL 20.04
Azure CLI docker image.

@Ashutosh-18-hub Are you following these instructions to install azure function core tools? https://learn.microsoft.com/en-us/azure/azure-functions/functions-run-local?tabs=v4%2Clinux%2Ccsharp%2Cportal%2Cbash#install-the-azure-functions-core-tools
An alternative is to use the devcontainer we've created for this repository which comes preinstalled with dotnet/python/az cli/azure function core tools. Please let us know if you have trouble deploying from the devcontainer/codespaces for this repository.


Function Renaming via Reference Fails in thumb2 architecture

Version and Platform (required):

Binary Ninja Version: [3.4.4271
OS: Windows
OS Version: 11
CPU Architecture: x64

Bug Description:
When loading a baremetal arm firmware with the thumb2 arch, attempts to rename functions via references (e.g. renaming pointers in the Exception Vector) consistently fail. Also, when clicking through a pointer to a thumb-mode function, the cursor lands within the function instead of on the function declaration (as normally occurs in other architectures).
I suspect this is related to the pointer underlying the symbol being off by one due to thumb.
Steps To Reproduce:
A sample binary has been privately shared with @bpotchik on Slack
Please provide all steps required to reproduce the behavior:

Open thumb-mode binary
Attempt to rename a function via 'Rename' on a pointer to the function (not the declaration)
Observe that the function is not renamed
Observe also that the current name of the symbol wasn't pre-populated in the rename modal

Expected Behavior:
I expect that triggering rename on a pointer to a function will rename the targeted symbol
https://binaryninja.slack.com/archives/C0CV88A83/p1688655856862579

Had already been resolved in dev branch at time of report, so closing issue


Brotlin for compression

As I understand how Appimages work, is that they are compressed archives which have an executable component. Google has created a new compression algorithm called Brotlin,  which is open source and compresses even better than gzip. Using that compression algorithm on their play store saved then around 1.5 Petabytes in bandwith, it might also help compression of appimages. While compressing might take a little bit longer than on gzip, decompressing shouldn't take that long, which shrinks the appimage file size. Could this work?

Yes, this could work, if someone implements Brotli compression for squashfs (or another filesystem image format). Looks like someone already had this idea: https://www.topcoder.com/challenges/30053205/?type=develop

Any updates on this?

It is currently not a priority for us. Is it for you?

idk


Error in making result file

Hi,
I have been trying to use find_enrichment.py to perform an enrichment analysis. I originally tried it with my study but it did not work so I tried it with the data provided on the github page as well as the command line example but seem to receive the same error message. It doesnt produce a results file either. Is there something else which i need to do before?
(enrichment) [mbxrs14@login001(Augusta) enrichment]$ find_enrichment.py small_study.txt small_population.txt small_association.txt --pval=0.05 --method=fdr_bh --pval_field=fdr_bh --outfile=results_id2gos.xlsx
go-basic.obo: fmt(1.2) rel(2023-06-11) 46,420 Terms
**WARNING: GO:0000229 NOT FOUND IN DAG
**WARNING: GO:0004871 NOT FOUND IN DAG
**WARNING: GO:0007568 NOT FOUND IN DAG
**WARNING: GO:0008565 NOT FOUND IN DAG
**WARNING: GO:0051186 NOT FOUND IN DAG
HMS:0:00:00.040034   6,267 annotations READ: small_association.txt
Study: 38 vs. Population 2000
WARNING: only 0.39473684210526316 fraction of genes/proteins in study are found in the population background.
ERROR: only 0.39473684210526316 of genes/proteins in the study are found in the background population. Please check.
Any help would be greatly appreciated thank you!

@Rhia15
Can you check to see if most of the genes in your study (38 genes) is a subset of the population (2000 genes)?
The population can be the entire protein set in the genome of interest.

Hi, thank you for the reply!  the input in the example is the txt files I downloaded from this github page in test/data to check if there was anything wrong with the environment or any dependencies. I think it is a subset of the population. It looks like I get the the same warnings about DAG with any data I try though. Could there be something wrong with the DAG? but I have checked the permissions for go-basic.obo and if the IDs are present and everything seems okay, so I am not sure what to do :/

@Rhia15
I just tested with the test files in this repo, it seems to run fine. Have you checked if you have the latest version?
$ find_enrichment.py data/study.txt data/population.txt data/association.txt --pval=0.05 --method=fdr_bh --pval_field=fdr_bh --outfile=results_id2go.xlsx

**WARNING: GO:0000059 NOT FOUND IN DAG
**WARNING: GO:0000060 NOT FOUND IN DAG
...
HMS:0:00:00.263267 127,967 annotations READ: data/association.txt
Study: 276 vs. Population 33239
Load BP Ontology Enrichment Analysis ...
Propagating term counts up: is_a
 87% 28,948 of 33,239 population items found in association

Load CC Ontology Enrichment Analysis ...


transact/send-tx does not support PulseChain

The limitation seems to be Rainbowkit

Here's a gist with fewer dependencies
It also implements Safe tx for arbitrary chains, which can be a nice addition to swiss knife
https://gist.github.com/aviggiano/ba3a2fe1a5907390824752697fa079c1


Remove version: WinSCP.WinSCP.RC version 5.18.5-

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/winscp/files/WinSCP/5.18.5 RC/WinSCP-5.18.5.RC-Setup.exe/download
Status Code: 200
(Automated message - build 897)

Close with reason: Package still available.;


error: explicit specialization of non-template class 'hash'

I get this error when building the tests on macOS:

In file included from test/optional-lite.t.cpp:7:
In file included from test/optional-lite.t.h:12:
include/nonstd/optional.hpp:1070:7: error: explicit specialization of non-template class 'hash'
class hash< nonstd::optional >
^   ~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.

My Clang version:
$ clang --version
Apple LLVM version 8.1.0 (clang-802.0.42)
Target: x86_64-apple-darwin16.5.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

My compiler flags:
-std=c++11

Note to self: investigate CGAL/Nested specialization syntax?.

Above NTS refers to Why can't I specialize std::tuple_element which states the location of the original declaration of hash is not std and should be removed:
//    template < class T>
//    struct hash;

template< class T >
class hash< nonstd::optional<T> >
{
public:
    std::size_t operator()( nonstd::optional<T> const & v ) const optional_noexcept
    {
        return bool( v ) ? hash<T>()( *v ) : 0;
    }
};

However this situation is not (directly) the case here.

gsl lite's issue 63 provides more information and circumvents the Clang Xcode problem by not using hash<T>()( *v ).

I gather #include <functional> is missing.

@njlr Question is: do we now get gsl lite's issue 63

The tests work now.
01-to_int works if I change line 19 to const:
    const char * text = argc > 1 ? argv[1] : "42";

See: https://github.com/martinmoene/optional-lite/pull/6

Thanks!


Whatsapp tab changes from whatsapp to url + notifications on unread messages not seen

This occurs randomly. Notification on un read messages may not appear or show wrong data. Refresh solves the issue for a while.

Hello,
We have divided error #47 (#33 in the task list) into two tasks:
33.1 Incorrect header text format. (5 hours).
33.2 Incorrect data in the counter (incorrect count of unread messages).
33.2.1 Item cannot be completed before item 28 in the task list, because it involves a high-level api.

The point is that previously we also needed access to the high-level api.
1.1 By high-level api we mean access to the site's memory.
Your extension has written code that we can work with freely. But functions which directly affect WA code are not available to us, because we don't have access to the high-level WA API. For example, task 47.2 (number 33 on the task list) is a task which is directly related to WA code and cannot be solved without it.
We write complex algorithms to make your extension functions work like the original WA.
4.1 At the moment we haven't found a way to access the high-level API.
4.2 You can give us permission to find workarounds for the high-level WA API described in task list #28.
4.3.1 Task #28 on the task list is to find access to the high-level API. Which will affect most of your tasks and their solutions. If, of course, we can do it.
4.3.2
Finding solutions and time
28.1 (10hr)
28.2 (5hr)
28.3 (10 hours)
28.4 (6 hours)
Most of your tasks require access to a high-level WA API. If we have it: the time to solve most tasks will be significantly reduced or become the most optimal!

Hi, what's the ETA on this?

When the code will be ready?
When will it be in production?

It is fixed. I will upload as beta version to chrome first


Files get hardly reconized

Hi i tried using the package but i found somt strange issues, i tried to reproduce it with a simple script:
#he is the simple script i used :
 cat asdef.py
#!/usr/bin/python
# PYTHON_ARGCOMPLETE_OK

import argcomplete, argparse

from random import randint

def completereu(prefix, parsed_args, **kargs):
    choices = ['%s_%d_%s' % ('cp1', randint(10, 100), x) for x in range(10)]
    return (c for c in choices if c.startswith(prefix))

parser = argparse.ArgumentParser()
parser.add_argument("--typ1").completer = completereu

argcomplete.autocomplete(parser)
pargs = parser.parse_args()
print pargs

(bashcompletion) user@host:~/snippets$ deactivate
user@host:~/snippets$ mkdir test_2_cmpl
user@host:~/snippets$ cp bashcompletion/qwertz.py test_2_cmpl/
user@host:~/snippets$ cd test_2_cmpl/
user@host:~/snippets/test_2_cmpl$ ls
qwertz.py
user@host:~/snippets/test_2_cmpl$ less qwertz.py
user@host:~/snippets/test_2_cmpl$ python qwertz.py
Namespace(typ1=None)
user@host:~/snippets/test_2_cmpl$ ls
qwertz.py
user@host:~/snippets/test_2_cmpl$ ls -l
total 4
-rwxr-xr-x 1 user user 447 Dec 13 15:28 qwertz.py
user@host:~/snippets/test_2_cmpl$ ls -la
total 12
drwxr-xr-x 2 user user 4096 Dec 13 15:28 .
drwxr-xr-x 7 user user 4096 Dec 13 15:28 ..
-rwxr-xr-x 1 user user  447 Dec 13 15:28 qwertz.py
user@host:~/snippets/test_2_cmpl$ ./qwertz.py
Namespace(typ1=None)
user@host:~/snippets/test_2_cmpl$ activate-global-python-argcomplete
Installing bash completion script /etc/bash_completion.d/python-argcomplete.sh
usage: activate-global-python-argcomplete [-h] [--dest DEST] [--user]
                                          [--no-defaults]
                                          [--complete-arguments ...]
activate-global-python-argcomplete: error: [Errno 13] Permission denied: '/etc/bash_completion.d/python-argcomplete.sh'
Please try --user to install into a user directory, or --dest to specify the bash completion modules directory
user@host:~/snippets/test_2_cmpl$ activate-global-python-argcomplete --user
Installing bash completion script /home/user/.bash_completion.d/python-argcomplete.sh
user@host:~/snippets/test_2_cmpl$ cp qwertz.py asdef.py
user@host:~/snippets/test_2_cmpl$ ./asdef.py
asdef.py   qwertz.py
user@host:~/snippets/test_2_cmpl$ source /home/user/.bash_completion.d/python-argcomplete.sh
user@host:~/snippets/test_2_cmpl$ ./asdef.py
asdef.py   qwertz.py
user@host:~/snippets/test_2_cmpl$ cp asdef.py lkmj.py
user@host:~/snippets/test_2_cmpl$ ./lkmj.py -
-h      --help  --typ1
user@host:~/snippets/test_2_cmpl$ ./lkmj.py -

So if i copy it aftewards and try to execute the copy it seems to work fine, but does not on the orginal file.

By default, the completer requires you to type - to offer completions on options. You can change this behavior with the always_complete_options=True keyword argument.

Actually, never mind, that doesn't explain the issue that you saw.

@cgte Can you run complete | grep asdef and post the output? I'm expecting to see complete -F _minimal ./asdef.py. This appears to be a result of the fallback behavior of the global completion script when you try to complete something that it doesn't match. You should be able to work around this with complete -r ./asdef.py.

It seems this is expected behavior from bash, and I don't think there's much we can do about it. I've added a section to the docs to explain what to do if this happens.

FYI:
complete | grep asdef
complete -F _minimal ./asdef.py

Thanks for following up. If the advice I added in #198 doesn't help, please let me know.


Grouped method entries.

Coverage increased (+0.27%) when pulling ebd8738c25f60b8bb068cb2358e7621cb30a92fb on mh/multidoc into e5c9e4e250332c768e9607a689efe8632c139e8d on master.

Coverage increased (+0.27%) when pulling ebd8738c25f60b8bb068cb2358e7621cb30a92fb on mh/multidoc into e5c9e4e250332c768e9607a689efe8632c139e8d on master.


Update the OTIOView screen shot in the readme

The OTIOView screenshot referenced by README.md is pretty old and missing a lot of the more recent features.  We should update it to the latest.

Addressed by #504


Rasa Actions Quick Restart Bug

Rasa version:
Rasa Version     : 2.2.2
Rasa SDK version (if used & relevant):
Rasa SDK Version : 2.2.0
Rasa X version (if used & relevant):
Rasa X Version   : None
Python version:
Python Version   : 3.7.3
Python Path      : /home/admin/.venv/bin/python3.7
Operating system (windows, osx, ...):
Operating System : Linux-4.19.0-13-cloud-amd64-x86_64-with-debian-10.7
Issue:
If Rasa actions is restarted rapidly, the old connection remains in a TIME_WAIT mode which causes unpredictable behavior
tcp        0      0 <IP_ADDRESS>:5055            <IP_ADDRESS>:*               LISTEN      5235/python3.7
tcp        0      0 <IP_ADDRESS>:60180         <IP_ADDRESS>:5055          TIME_WAIT   -

This basically just needs an SO_REUSEADDR set on the listening socket
Error (including full traceback):
No error. Just doesnt work!
Command or request that led to error:
rasa actions
Content of configuration file (config.yml) (if relevant):

Content of domain file (domain.yml) (if relevant):

Thanks for raising this issue, @rctatman will get back to you about it soon✨
Please also check out the docs and the forum in case your issue was raised there too 🤗

Thanks for raising this issue, @rctatman will get back to you about it soon✨
Please also check out the docs and the forum in case your issue was raised there too 🤗


import: Make.defs: Fix 'make import' for CONFIG_BUILD_KERNEL=y

Summary

I noticed that 'make import' failed due to recent changes in nuttx
This commit fixes this issue by adding logic for -lm

Impact

CONFIG_BUILD_KERNEL=y only

Testing

Build (mkimport, make import) with sama5d4-ek:knsh

See https://github.com/apache/incubator-nuttx/pull/5371 as well


fix #warn choking on apostrophe

#warn directives shouldn't try to process the tokens. Fixes #1439 and adds test

Might be better to instead have the lexer read the rest of the line and save it as the warning/error token's value. Something like this in the lexer:
case "warning": {
    StringBuilder message = new StringBuilder();

    while (GetCurrent() is not '\0' and not '\n') {
        message.Append(GetCurrent());
        Advance();
    }

    token = CreateToken(TokenType.DM_Preproc_Warning, "#warn", message.ToString());
    break;
}

and use (string)token.Value! in HandleErrorOrWarningDirective()


Fix build step not copying console dependencies and translations

Right now, neither the gettext translations nor the html templates for spice & vnc consoles are properly copied during the gulp build step, so neither actually works in the appliance now.
This fixes gulp gettext-copy by fixing the path selector (one more dir level now), and adds gulp console-copy to copy the console templates and their dependencies to the right place.
Also adjusted the console code to find the template using the full path, including /self_service and updated the devel-mode server to actually cope with such a path too.
Assuming darga/yes since without this, consoles and i18n won't work in SSUI..
Fixes #61
https://bugzilla.redhat.com/show_bug.cgi?id=1347312
https://bugzilla.redhat.com/show_bug.cgi?id=1347319
@AparnaKarve could you review please?

Successfully verified in SSUI - both BZs are resolved with these changes. 👍


[WIP] Allow "Jsonify" the option to pretty print JSON

Fixes #5040: Allow "Jsonify" to pretty print JSON
This PR allows Jsonify the option to return pretty-printed output, where the spacing is specified.

As Go doesn't have optional parameters, I modified Jsonify to be a variadic function, however I'm not sure if this is the most elegant and maintainable way to do it.
Any and all feedback greatly appreciated!

Any and all feedback greatly appreciated!

Yes. String options are not optimal -- if we would get more options in the future ...
As I see it it is either that or a map (which you could then say "asdf" | jsonify (dict "delimiter" "   ")
This is how I did it for resources.ToCSS etc. with many options.

Thanks for the feedback @bep - just wanted to let you know that I'm hoping to get another crack at this in the coming weeks after exams for school are done!

Great! Good luck with your exams!


Manual Import of programming exercises from a zip-file

Is your feature request related to a problem?
Currently programming exercises can be shared only inside Artemis between different courses.
On the long run, programming exercises should also be readable from zip-files in the format as defined with feature #3296.
This would enable the smooth exchange of exercises between various Artemis instances, but is also a first step into a better integration of the sharing plattform developed by the CodeAbility project
Describe the solution you'd like
As a first step we would implement a manual import feature as a zip-file upload, based in the export format:
The import comprises:

the title
the problem statement
the categories
the programming language
the difficulty
the content for the exercise, solution and test repositories

Beyond this data, which is part of the sharing platform metadata standard, also the following attributes are set

max points
bonus points
mode: Individual/Team
static code analysis
allow offline Editor
show test names to students
sequential test runs

Describe alternatives you've considered
There are currently no really good alternatives: Only sharing inside the same artemis instance, or manual copy&paste between artemis instances together with resharing of repositories.
The second option is very time consumuing and error prone.
Additional context
This issue is an intermediate step for bringing up a full blown integration with the CodeAbility sharing plattform.

Already implemented :-)


Remove README.md from bundle

The package is 44.7kb, and the README alone is 11.3kb.
~If you just remove the README from the bundle, you can save ~25% in bundle size, which could save 3TB (279,859,270 * 11.3kb) each week.~
We cannot remove but if we put a placeholder, we can save ~631GB/week (3,7×228240517÷1024÷1024) of bandwidth (13.3kB to 10.4kB).

From what I saw, is impossible to remove Readme, but maybe you can rename/remove before packing or try reduce the size of the readme.

You are trying to fix this in the wrong place. You should open an issue on npm instead. Lots of users use the npm page for docs, so we cannot truncate it in any way.


Feature/dynamic circle color

Hi,
thank you for this one, but it seems that the following part (preferred individually, rather than using the respective data- attribute) doesn't work:
$("[id$='circle']").percircle({
	progressBarColor: "green"
});

Could you please investigate it?

@toubou91 Updated :)

Hey @tranthanhhoa ,
I decided to merge as soon as possible, as I noticed that @ahmadajmi prepares something cool (and as I am not sure about my availability in the upcoming days, it would be better to eliminate the chances for bigger conflicts).
Thus, there are still some minor issues left, for the case you still want to continue contributing.
Thank you!


10r-04 6953  BACKUP

congratulations on your work, could you also kindly post a complete backup of your NAVIGON?
thanks in advance

I recently took a backup of my Garmin / Navigon device with the latest maps for western europe (which is from 2021 I think). Here it is (the link is available for 30 days): https://www.swisstransfer.com/d/bf6e46fb-2da9-4b77-8c31-f33f33f861fd
I also have a backup from before the update, which is basically the version that was delivered with the car when it was new in december 2013.


docs(dropdown): add watch badge

Wait, I added the incorrect badge, I am sleepy.

It needed more work (I updated the commit message too).

LGTM


upstream: Don't call UpstreamRequest::resetStream() more than once

Uses the existing reset_stream_ variable to determine if resetStream() should get run.
Logs an Envoy BUG if resetStream() is called more than once on the same UpstreamRequest.

@alyssawilk WDYT of this PR?  I will write tests for it tomorrow, but saw that we set a reset_stream_=true after resetStream() has been called, but then never use the reset_stream_ anywhere.  I added a check to not execute resetStream() again if it's already been called.

no objections as long as we test it.


Datetime field not working properly in Studio

Expected behavior and actual behavior
When entering a date in the date picker and then entering a time in the time picker, the date field is cleared and vice versa.

Works fine in Chrome.
Steps to reproduce the problem
Add a datetime property
press "new record"
enter date
enter time
Safari version 9.1.1 (11601.6.17)
Important Questions
Runninng Mode

[ ] Embedded, using PLOCAL access mode
[ ] Embedded, using MEMORY access mode
[x] Remote

Misc

[ ] I have a distributed setup with multiple servers. How many?
[ ] I'm using the Enterprise Edition

OrientDB Version

[ ] v2.0.x - Please specify last number:
[ ] v2.1.x - Please specify last number:
[x] v2.2.x - Please specify last number: 3

Operating System

[ ] Linux
[x] MacOSX
[ ] Windows
[ ] Other Unix
[ ] Other, name?

Java Version

[ ] 6
[ ] 7
[x] 8

Hi,
Thanks for creating this issue
I am experiencing problems also in Windows 10, using Chrome Version 55.0 (when creating/editing a record from the Schema panel or adding a datetime property to an edge or vertex from the Graph Editor)


[Proposal] Resolve injections in properties

Hi. I have a proposal. There is an example:
export class Config {
    public url = "https://google.com";
}

export class SubService {
    @inject public configModel: ConfigModel;
}

export class MainService {
    public prop = new SubService();
    public props = [new SubService(), new SubService()];
}

@provider(
    Config,
    MainService,
)
export class Html extends React.Component {
    @inject private service: MainService;
}

Your example already works with inject() function syntax (without decorators):
class Config {
  url = "https://google.com";
}

class SubService {
  configModel = inject(this, Config)
}

class MainService {
  prop = new SubService();
  props = [new SubService(), new SubService()];
}

class Html extends React.Component {
  static contextType = InjectorContext;
  service = inject(this, MainService);

  render() {
    html = this;
    return <div />;
  }
}
let html: Html;

const App = provider(Config, MainService)(Html);

render(<App />, document.createElement("div"));
expect(html.service.prop.configModel.url).toBe("https://google.com");
expect(html.service.props[0].configModel).toBe(html.service.prop.configModel);
expect(html.service.props[1].configModel).toBe(html.service.prop.configModel);

But with decorators it can't work. Because @inject doesn't immediately write dependency to decorated property, but rather it define a lazy property getter that resolves dependency on demand. And it is conceptual limitation of TypeScript decorators.


Implement FutureBuilder Widget in the Widget Catalog

FutureBuilder Widget needs to be added to the Widget Catalog. Make sure to include the description of the widget in the Description tab and add an argument to include link to the official docs of the widget (Refer to any example in homepage.dart for better understanding. Also add an argument for category of the widget and give it a category accordingly. Showcase proper implementation of the widget.
Please attach a screen recording of the same once done in your PR.

Hey!!!... I would like to work on this one. Kindly assign me this task please.

Hey!!!... I would like to work on this one. Kindly assign me this task please.

Cool, I will assign it to you. Good luck!


Quarkus provided dependency

I made a Quarkus core dependency provided, and is working as expected. i.e The dependency won't exist in the classpath unless the client does that.
I think this changeset suffice to support Quarkus native builds!
Would be great to hear what you think about this solution and if there are any concerns.
@csviri @kirek007 @adam-sandor @metacosm
Update The scope provided didn't wok :(, I borrowed the Annotation (with full package name) to this project.
I think this should work, how is your feeling about it :D?

Nice try with moving just the annotation class :)
In my opinion this hack is not needed and it's safer to simply use quarkus core even if it's heavier. Shadowing 3rd party classes is never a good idea :)
What do you think @csviri?

Hehe, I think the best would be if quarkus would separate its API as a dependency :)
Don't think this is a bad solutions, Its quite smelly to have the whole framework on the classpath. Especially when somebody would use the sdk with spring.
Probably both ways would work. This is nicer not having the whole quarkus on classpath, but it's less safe since there is a minimal chance that this interface or it's package will change in quarkus.
This is a rare occasion when voting might be a good ide :D
@psycho-ir do I understand correctly that this is still a temp. solution? So at the end we plan to have annotation processor to generate json files to support quarkus without annotation?

@psycho-ir do I understand correctly that this is still a temp. solution? So at the end we plan to have annotation processor to generate json files to support quarkus without annotation?

Yes absolutely, this would just enable the users to build quarks natives images until we come up with a better solution.

@adam-sandor  what do you think?

I'm working on a quarkus extension as we speak in #238.


At spyder start, Kernel is said "dead and restarting" a lot of times (on a slow pc)

Description of your problem
What steps will reproduce the problem?

start Spyder on a slow pc while compiling things
spyder starts , but generates a lot of Kernel death messages (6)
in the finally stabilized Qtconsole, we see 6 "[1]" appearing

What is the expected output? What do you see instead?
report of Kernel death are widely exagerated (in numbers), just because kernel is probably slow to start. could we have less ?
Please provide any additional information below

Versions and main components

Spyder Version: github 2016-09-16
Python Version: 3.5
Operating system: windows 10

Dependencies
Please go to the menu entry Help > Optional Dependencies (or
Help > Dependencies), press the button Copy to clipboard
and paste the contents below:
jedi >=0.8.1     :  0.9.0 (OK)
matplotlib >=1.0 :  2.0.0b4 (OK)
nbconvert >=4.0  :  4.2.0 (OK)
numpy >=1.7      :  1.11.1 (OK)
pandas >=0.13.1  :  0.19.0rc1 (OK)
pep8 >=0.6       :  1.7.0 (OK)
pyflakes >=0.6.0 :  1.3.0 (OK)
pygments >=2.0   :  2.1.3 (OK)
pylint >=0.25    :  1.6.4 (OK)
qtconsole >=4.2.0:  4.2.1 (OK)
rope >=0.9.4     :  0.9.4-1 (OK)
sphinx >=0.6.6   :  1.4.6 (OK)
sympy >=0.7.3    :  1.0 (OK)

We're doing the exact same thing now that qtconsole does to start its consoles, so I don't know what we could do to improve this.

Closing as a duplicate of #3444


Bad encoding of the variables

Based on http://graphql.org/learn/serving-over-http/#post-request, the variables should be sent over as a JSON object, since it is a JSON document after all.
Currently graphlient encodes the json, and sends it over as a string:
{
  "query":"query IntrospectionQuery {\n  __schema {\n    queryType {\n      name\n    }\n    mutationType {\n      name\n    }\n    subscriptionType {\n      name\n    }\n    types {\n      ...FullType\n    }\n    directives {\n      name\n      description\n      locations\n      args {\n        ...InputValue\n      }\n    }\n  }\n}\n\nfragment FullType on __Type {\n  kind\n  name\n  description\n  fields(includeDeprecated: true) {\n    name\n    description\n    args {\n      ...InputValue\n    }\n    type {\n      ...TypeRef\n    }\n    isDeprecated\n    deprecationReason\n  }\n  inputFields {\n    ...InputValue\n  }\n  interfaces {\n    ...TypeRef\n  }\n  enumValues(includeDeprecated: true) {\n    name\n    description\n    isDeprecated\n    deprecationReason\n  }\n  possibleTypes {\n    ...TypeRef\n  }\n}\n\nfragment InputValue on __InputValue {\n  name\n  description\n  type {\n    ...TypeRef\n  }\n  defaultValue\n}\n\nfragment TypeRef on __Type {\n  kind\n  name\n  ofType {\n    kind\n    name\n    ofType {\n      kind\n      name\n      ofType {\n        kind\n        name\n        ofType {\n          kind\n          name\n          ofType {\n            kind\n            name\n            ofType {\n              kind\n              name\n              ofType {\n                kind\n                name\n              }\n            }\n          }\n        }\n      }\n    }\n  }\n}",
  "operationName":"IntrospectionQuery",
  "variables":"{}"
}

Due to this we had to introduce unnecessary complexity in the server to parse that string into a JSON hash, if it's given as a string, and leave it as is when it's coming from comforming clients.

I think you're right, per (spec)[http://graphql.org/learn/serving-over-http/#post-request] a GET should pass query parameters, but a POST should send application/json. This should be fairly easy to fix, feel free to beat me at a PR :)

I'm leaving for vacation tomorrow, so I guess you will win this time :)

Wait, sorry, looks like we already fixed this.
Look at https://github.com/ashkan18/graphlient/blob/master/lib/graphlient/adapters/http/faraday_adapter.rb#L11?

Note that graphql-client does this here, also wrong?

So query IntrospectionQuery {} is not valid JSON. So I actually think we're all doing the right thing, aren't we?

Released as 0.2.0.


Fix cairocffi build

Motivation for this change
cairocffi doesn't build, preventing qtile from build.
(patch doesn't applying correctly)
Patch was refreshed, as well as more packages required for tests added.
/cc @FRidh
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg python2.pkgs.cairocffi python3.pkgs.cairocffi

On Tue, Mar 05, 2019 at 12:29:50AM -0800, Frederik Rietdijk wrote:

@GrahamcOfBorg python2.pkgs.cairocffi python3.pkgs.cairocffi

FYI cairocffi is 3.5+ library now.
Probably I should add assert as well.

@GrahamcOfBorg build python3.pkgs.cairocffi

On Tue, Mar 05, 2019 at 01:10:25AM -0800, Frederik Rietdijk wrote:
Added disabled = pythonOlder "3.5" stanza

@FRidh @avnik I'd propose to add an cairoffi_1 package as well (similar to cairosvg1) to retain Python 2 support for now. This affects thumbor, graphite-web and cairosvg1 and I think it's too early to drop Python 2 support if some dependants still need it.
Also wondering if the Python update script should contain some functionality to actually test the package before bumping it (and pushing this to master).

@Ma27 cairocffi_py2 be better name, than _1.
Other option -- add some backporing patch to this package, I am ok with both solutions.

cairocffi_py2 be better name, than _1.

Ack. Didn't look at the package's version when writing this :sweat_smile:

Other option -- add some backporing patch to this package, I am ok with both solutions.

Can you please be more precise in what you mean? Not sure if I understand correctly what you meant :)

On Thu, Mar 07, 2019 at 08:37:44PM +0000, Maximilian Bosch wrote:

Other option -- add some backporing patch to this package, I am ok with both solutions.

Can you please be more precise in what you mean? Not sure if I understand correctly what you meant :)

I mean, if you take commit, which removing 2.x support, make reversing
patch, and will support it ;)
If you want overtake cairoffi, you are welcomed -- I care about it,
because it transitive dependency of qtile.

I mean, if you take commit, which removing 2.x support, make reversing patch, and will support it ;)

I doubt that it'll be that easy. Python 2.x and 3.x diverge pretty much.
But I'll have a look at this and see how to proceed here :)

On Sat, Mar 09, 2019 at 06:07:44AM -0800, Maximilian Bosch wrote:

I mean, if you take commit, which removing 2.x support, make reversing patch, and will support it ;)

I doubt that it'll be that easy. Python 2.x and 3.x diverge pretty much.

I know. I am in middle of porting mid size project from 2.x and gevent
to 3.x and aio. Lot of pain and sorrows ;)

Just a short update: I created a cairocffi_0_9 package locally and played around with it. I'll push this tomorrow to this PRs branch as I'm fairly sleepy atm :)

I pushed three commits on top of your branch:

8d23630: qasm2image doesn't support Python older than 3.5. It's more obvious to raise an error since this package doesn't support old Python rather than breaking the evaluation when trying to use the cairocffi derivation.

ad550de: move the entire cairocffi build instructions into a generic function that is invoked for cairocffi 1.0 and 0.9. The older version should be supported for now to keep older software like thumbor working. I'd probably revert that before 19.09 as Python 2.x will be dropped by the end of the  year.
This affects the following packages:

cairosvg1: Built with cairocffi 1.x by default, but when using Python 2.x (which is still supported here), cairocffi can be overridden by cairocffi_0_9. (Also wondering if we should add a cairosvg1_py2 to make this more obvious.
thumbor: Python 3.x support is in progress, but until then we can only build with Python 2 here.
graphite-api with Python 2.x (nix-build -E 'with import ./. {}; pythonPackages.graphite_api.override { cairocffi = pythonPackages.cairocffi_0_9; }')
graphite_web (doesn't work with Python 2.x)
and probably some more...

da86a8e: unbreak thumbor (by using cairocffi_0_9 and adding some minor fixes to the test suite).

@FRidh @avnik it would be great if you could review this as well :)

Cherry picked in ab89b6b45b4e79b3d8b62a8533a6eaf51606f48a..3274b1de0fbf9229efc2af26e07cf7281421dcc1.
Btw, I also pushed f721655455fb1868eb483d76873f56a53aa93a38 (cherry picked in cdc714fe4a0f7f136de03eceffd9bbcad1a8b560).

Is there a way to use it with overridden withXcffib like in https://github.com/NixOS/nixpkgs/blob/6004e5dda3da46716cebcff3898fe9c877929b1f/pkgs/applications/window-managers/qtile/default.nix#L4 ?
I can't make it work after this change.

@kamilchm Sorry for messing that up. An easy way to make it work would be to use three files: default.nix and 0_9.nix, which both call common.nix. I'll make the change once I have time for it.


Repro socket strangeness in dev.

Automated message from Dropbox CLA bot
@showell, it looks like you've already signed the Dropbox CLA. Thanks!

This was repro-ing earlier today.  Something changed, and now stuff is working again.

Here is example output:
calling success 3 Object { msg: "", result: "success", success: "3" }  compose.js:374:9
calling success 6 Object { msg: "", result: "success", success: "6" }  compose.js:374:9
calling success 9 Object { msg: "", result: "success", success: "9" }  compose.js:374:9
calling success 12 Object { msg: "", result: "success", success: "12" }  compose.js:374:9
calling success 15 Object { msg: "", result: "success", success: "15" }  compose.js:374:9
calling success 18 Object { msg: "", result: "success", success: "18" }

More clues, we get acks for skipped stuff, but not responses for the skipped ones:
calling success 16 Object { msg: "", result: "success", success: "16" }  compose.js:374:9
WebSocketTransport a[{"type":"ack","req_id":"1500073191:20:17"}]  sockjs-0.3.4.js:1220:9
1500073191:20:17 ack  socket.js:252:13
WebSocketTransport a[{"type":"ack","req_id":"1500073191:20:18"}]  sockjs-0.3.4.js:1220:9
1500073191:20:18 ack  socket.js:252:13
WebSocketTransport a[{"type":"ack","req_id":"1500073191:20:19"}]  sockjs-0.3.4.js:1220:9
1500073191:20:19 ack  socket.js:252:13
WebSocketTransport a[{"type":"response","response":{"msg":"","result":"success","success":"19"},"req_id":"1500073191:20:19"}]  sockjs-0.3.4.js:1220:9
1500073191:20:19 response  socket.js:252:13
calling success 19 Object { msg: "", result: "success", success: "19" }

To trigger this, force a Django restart by changing Python code.

Problem related to Tornado queues fixed in https://github.com/zulip/zulip/pull/5857.

@timabbott @umairwaheed I am closing this, since we eventually simplified the repro to restarting Django.


Fix bad memory access in Darwin select() fallback

Apple's fd_set stores its bits in an array of 32-bit integers, which
means FD_ISSET() may read out of bounds if we allocate storage at
byte granularity. There's also a chance that the select() call could
corrupt the heap, although I didn't investigate that.
This issue was discovered by LLVM's AddressSanitizer which caught
FD_ISSET() trying to read out of bounds.

/cc @indutny

@saghul LGTM. I wonder if it might be related to that TTY issue? #217

@saghul do you want me to land it?

@oleavr can you squash your commits into one?
@bnoordhuis or @indutny can you PTAL?

@saghul Done.

@bnoordhuis Well spotted; can't believe I managed to fail at basic integer division :weary:

Ready for a (hopefully final) review now. :smile:

Thanks Ole, looking good.  Landed in d557ad7.