[dgoss] Use inspect instead of top in health check

Just started using Goss, very nice! I came across #376 trying to publish JUnit results in my CI. This change tests if the container is running without producing any output to be redirected. This change is working in my environment. Additionally tested with your dgoss-examples repo nginx goss.yaml
$ GOSS_OPTS="--format junit" dgoss run nginx:1.11.10 > junit.xml
INFO: Starting docker container
INFO: Container ID: 9ddf7dec
INFO: Sleeping for 0.2
INFO: Container health
INFO: Running Tests
INFO: Deleting container
$ head -3 junit.xml
<?xml version="1.0" encoding="UTF-8"?>
<testsuite name="goss" errors="0" tests="13" failures="0" skipped="0" time="0.010" timestamp="2019-08-08T22:46:38Z">
<testcase name="Process nginx running" time="0.000">

Doesn't look like the Travis test includes dgoss. I should have mentioned too that I tested with the 3.7.0 release of the goss executable, didn't compile it from source.

Anything additional I should here?

I have the same issue as you, but I was able to resolve it by changing the following:
https://github.com/nathysOne/goss/commit/752c612e30fa772814ddbc928f05dafae3986792

thx @nathysOne , ran into the same issue and removing the docker top fixed it.:)

Looks good to me. I'll test/merge this after testing locally.
Travis test seems to be broken. Don't worry about the PR successful passing tests.

For what it's worth, this PR seems to have broken the dgoss health check for me. It always said container ERROR: the container failed to start.
I committed a fix directly to master, you can find it here:  https://github.com/aelsabbahy/goss/commit/a4a501a4a755c882dd5ab920df3b9c6bb27d9779

Hm, it appears to be the escaped quotes on the format string
[19:39:32]> justin@chymera in ~
$ docker inspect -f \"{{.State.Running}}\" 7cf3008d0e6fde1693d09b6a357858900c266f2f41b7aff4d59965e6c5cd3a59
"true"

[19:39:38]> justin@chymera in ~
$ docker inspect -f "{{.State.Running}}" 7cf3008d0e6fde1693d09b6a357858900c266f2f41b7aff4d59965e6c5cd3a59
true

[19:39:44]> justin@chymera in ~
$ if [ "true" != "$(docker inspect -f \"{{.State.Running}}\" 7cf3008d0e6fde1693d09b6a357858900c266f2f41b7aff4d59965e6c5cd3a59)" ]; then echo ended; else echo running; fi
ended

[19:40:46]> justin@chymera in ~
$ if [ "true" != "$(docker inspect -f "{{.State.Running}}" 7cf3008d0e6fde1693d09b6a357858900c266f2f41b7aff4d59965e6c5cd3a59)" ]; then echo ended; else echo running; fi
running

I just have it ingrained to wrap a command output in quotes for equality tests in case the command doesn't produce any output.

I just have it ingrained to wrap a command output in quotes for equality tests in case the command doesn't produce any output.

Good point, fixed: https://github.com/aelsabbahy/goss/commit/9dddd816d63f7e7217a1021f4e1c5ee0a50f0512


514 bandaid fix

#514
I definitely do not think this is the best solution, but I really don't know what's causing this crash.

Hmm. This does stop the crash in Android for me, but I'm also not sure why it's happening. Maybe certain Android devices are having trouble parsing numbers into a translation string immediately?

@albullington not sure exactly the issue, the error message is pretty vague

I'm going to close without merging since we already applied this bandaid fix to the updated MyObservations code in PR #515


Create dataset loader for Thai-Sum

Dataloader name: thai_sum/thai_sum.py
DataCatalogue: http://seacrowd.github.io/seacrowd-catalogue/card.html?thai_sum

Dataset
thai_sum

Description
We present ThaiSum, a large-scale corpus for Thai text summarization obtained from several online news websites namely Thairath, ThaiPBS, Prachathai, and The Standard. This dataset consists of over 350,000 article and summary pairs written by journalists.

Subsets
-

Languages
tha

Tasks
Summarization

License
MIT (mit)

Homepage
https://github.com/nakhunchumpolsathien/ThaiSum

HF URL
https://huggingface.co/datasets/thaisum

Paper URL
https://www.researchgate.net/publication/367166731_Using_Knowledge_Distillation_from_Keyword_Extraction_to_Improve_the_Informativeness_of_Neural_Cross-lingual_Summarization

#self-assign

#self-assign


gsub should accept numbers as replacement values

As you've written this error yourself you are probably aware but just an issue reminder.
string.gsub("./../../editor/zbstudio.sh %PATH%:%LINE%", "%%LINE%%", "1234")

I think your goal is to replace plaintext and gsub does not have a plainsearch mode.
You need to quote each magics.
I got a similar situation and decided to make a function to quote magics.
local function quote_magics(str)
        local first = str:gsub("[%^%$%(%)%%%.%[%]%*%+%-%?]", "%%%0")
        return first
end

and you sample becomes :
string.gsub( quote_magics("./../../editor/zbstudio.sh %PATH%:%LINE%"), quote_magics("%LINE%") )

The problem is that this works in the original string.gsub and not strung.gsub. In my case I'm using strung as a drop in placement with strung.install()

Hello guys, I don't understand the issue (beside the clumsy error message, which should be improved):
> =strung.gsub("./../../editor/zbstudio.sh %PATH%:%LINE%", "%%LINE%%", "1234")
./../../editor/zbstudio.sh %PATH%:1234	1

> =string.gsub("./../../editor/zbstudio.sh %PATH%:%LINE%", "%%LINE%%", "1234")
./../../editor/zbstudio.sh %PATH%:1234	1

I was too fast to blame strung. I had started using strung again as a drop in replacement because it's actually a lot faster in some cases. I then started getting this error and without really checking I thought it was some bad pattern but I was actually calling gsub with the third argument being nil. It must've been something else I did.
But yeah I guess the error message could be improved. Is there a big performance penalty for having a nice error message or any at all?

None, it's an oversight, it should not have been published like that.
https://github.com/pygy/strung.lua/blob/master/strung.lua#L980-L982
I was likely in the zone and didn't want to get out of it to produce a concise, yet informative message (or lift what Lua is doing), and put a placeholder that I later forgot to fix.
...
Looking at the Lua/LJ source, there's a strung bug! string.gsub also accepts numbers as replacement values. That behavior isn't documented in the manual.
> =string.gsub('f','f',1)
1

> =strung.gsub('f','f',1)
./strung.lua:981: Bad replacement type for GSUB TODO IMPROVE MESSAGE.
stack traceback:
	[C]: in function 'error'
	./strung.lua:981: in function 'select_handler'
	./strung.lua:993: in function <./strung.lua:985>
	[C]: at 0x0100002150

Looking further into it that was actually why it was erroring..

Fixed :-) Now I have to do the LuaRocks dance.

Small setback, I forgot my luarocks password...

Maybe @leafo could help you? Have you already contacted the LuaRocks team?

I tweeted to the luarocksorg account...


Delete KubernetesEventSource, replaced by ApiServerSource

Proposed Changes

Remove KubernetesEventSource, replaced by ApiServerSource

Release Note

KubernetesEventSource will no longer ship with eventing-sources. Migrate to ApiServerSource found in knative/eventing.

follow-up to https://github.com/knative/eventing/pull/1175

/hold
until https://github.com/knative/eventing/pull/1175 lands

Istio still having a bad day over here.
/retest

/hold cancel
This is now ready.

Can you also link a corresponding issue / PR for the documentations website?

This one: https://github.com/knative/docs/issues/1301

/lgtm
/approve


Add Publish Grading Feature

(part of frontend issue #2522)
Currently, the grading states design and business logic cause ambiguity with resubmission of assessments being assumed to be graded immediately, and with grades being displayed when they should not.
This feature aims to solve this issue by introducing a new grading state published and a new backend boolean is_grading_published, allowing explicit publishing of student's grades on the frontend and preventing the fetching of grades if they have yet to be published, hence creating a better logic flow.
This pull request contains changes to be made to the backend to enable this new feature.

Hi! Please fix the format and CI and tag me again when the PR is ready. Thank you!

@YaleChen299 Hello! Sorry, I have fixed the formatting. Thank you!

No worries! Can you check the CI again? I will approve the CI run when I see then.

coverage: 93.886% (-1.5%) from 95.358% when pulling e8f0fbd4b5d2ee09a136fcfc7279409baec5a726 on emptygx:add-publish-grading into 5d09b514ea77a1146b5e13812386a0584dbe2cb1 on source-academy:master.

(part of frontend issue #2522)
Currently, the grading states design and business logic cause ambiguity with resubmission of assessments being assumed to be graded immediately, and with grades being displayed when they should not.
This feature aims to solve this issue by introducing a new grading state published and a new backend boolean is_grading_published, allowing explicit publishing of student's grades on the frontend and preventing the fetching of grades if they have yet to be published, hence creating a better logic flow.
This pull request contains changes to be made to the backend to enable this new feature.

Re-thinking about the problem that we are trying to solve, I think we need to auto un-publish the grades when we unsubmit.
The main point of publish grading is to solve resubmitted submissions shown as graded to graders so that grader will not know if the student is graded after resubmission.
This logic is a bit different then whether the grade is publish to the student or not. Please correct me if i am wrong! Thanks!

Re-thinking about the problem that we are trying to solve, I think we need to auto un-publish the grades when we unsubmit. The main point of publish grading is to solve resubmitted submissions shown as graded to graders so that grader will not know if the student is graded after resubmission. This logic is a bit different then whether the grade is publish to the student or not. Please correct me if i am wrong! Thanks!

Yes, I agree that should be part of the feature, which has already been implemented as part of the logic on the front end, which auto-unpublishes grades whenever an assessment is unsubmitted. Sorry for not mentioning that in the PR description, will make sure to add it into future documentation! Should I add it to the PR description as well?

TODO:
Implement test for is_grading_published after this and #1088 are merged


bug: Nightly 1.2.0 JS broke down scrolling

Type: bug
Platform: desktop browser
Platform: desktop, but probably all
http://s.codepen.io/ionic/debug/mqolp as example
Scrolling doesn't work and doesn't produce any errors in console.

I have same issue on Android and iOS

"By default, Ionic 1.2 will use native scrolling on all platforms."
ion-refresher is based on jsScrolling. If you want ion-refresher you should enable jsScrolling. You can do it by adding
$ionicConfigProvider.scrolling.jsScrolling(true);
in your app config. Ref: http://ionicframework.com/docs/api/provider/$ionicConfigProvider/

@kbalaguru Good to know! Thank you. Sadly all examples refers to "nightly" build and should be fixed then.

thank you

@evgd @ronygb you can also do it easily by view (if you need ion-refresher)
<ion-content overflow-scroll="false">
</ion-content>

Thanks, fixed, will be in 1.2.1 when we publish this week


Add Following node in the documentation:

"Only integer or floats can be stored under the process value topic."

Add tip on how to do that in Node-RED:
Number(msg.payload.yourValue)

or for floats that are too long:
Number(msg.payload.yourValue.toFixed(2))


fix: unsafe attribute reference

Fixes an unsafe reference that can throw an error in debugDOM when it receives a reference to a document in a real browser (ie puppeteer/codepen)

What:

Why:

How:

Checklist:

[ ] Documentation
[ ] Tests
[ ] Ready to be merged
[ ] Added myself to contributors table

We'll need a test to cover this case :)

repro: https://codepen.io/anon/pen/bmYmPd?editors=0011
There is a test but JSDOM seems to implement document.prototype differently than Chrome

@kentcdodds added test

:tada: This PR is included in version 3.11.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


--cnn-mem How to do this?

How to allocate more memory to cnn. There is an error that it ran out of memory each time.

When you have a program compiled with cnn, and assuming that you call cnn's Initialize() function using the command line arguments passed in to the program, it's as simple as adding:
$ my_program --cnn-mem 1024 [other options]
to your command line arguments. Note that the cnn arguments must come before any other arguments.
These command line arguments should be documented, so I'll leave this issue open and add this to the documentation when I have a chance.

A small addition to this: we observe that setting --cnn-mem to values above 2010 or so always fail. Probably relates to some limits on contiguous memory allocation.

@yoavg @neubig it seems the current master has an integer overflow problem on the devices.cc when allocating memory. See this bug fixed in v2 (https://github.com/clab/cnn/blob/v2/cnn/devices.cc#L63) but not sure when will it be merged into master.

Thanks! is v2 operational already?

Yes, it's basically operational. I've switched my experiments over to v2. There are still a few things that need to be done, but I think we'll be merging it into master sooner rather than later.

Information about command line options has been added to README.md

It seems to be the case that the order of the cnn args also matters. I'm using v2 and seeing that the --cnn-mem arg is ignored unless preceded by cnn-gpus or cnn-gpu-ids; it'll just use 512. Can see with the example:
./examples/xor --cnn-mem 1024
[cnn] allocating memory: 512MB

./examples/xor --cnn-mem 1024 --cnn-gpu-ids 3
[cnn] allocating memory: 512MB

./examples/xor --cnn-gpu-ids 3 --cnn-mem 1024
[cnn] allocating memory: 1024MB

./examples/xor --cnn-gpus 1 --cnn-mem 1024
[cnn] allocating memory: 1024MB

It seems to be the case that the order of the cnn args also matters. I'm using v2 and seeing that the --cnn-mem arg is ignored unless preceded by cnn-gpus or cnn-gpu-ids (will use 512 default) and those two args will fail if they follow cnn-mem. Can see with the xor example:
./examples/xor --cnn-mem 1024
[cnn] allocating memory: 512MB

./examples/xor --cnn-mem 1024 --cnn-gpu-ids 3
[cnn] Device(s) selected: 2
[cnn] Bad command line argument: --cnn-gpu-ids
Aborted (core dumped)

./examples/xor --cnn-gpu-ids 3 --cnn-mem 1024
[cnn] allocating memory: 1024MB

./examples/xor --cnn-gpus 1 --cnn-mem 1024
[cnn] allocating memory: 1024MB


Fixed error 'SimpleWorkflow() is not a constructor.

The error started to occur recently. Probably the new version of the AWS-SDK removed/renamed SimpleWorkflow to SWF completely.

@MichaelKovalchuk Thanks for confirming this. Had just recently started using aws-sdk, thought it was a local issue and hacked it to work for the time being. Nice to know "it's not just me."


integration test fix

I don't think this endpoint is supported anymore.  We can use cancel_jobset and that seems to work.

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

Coverage data is based on head (1eee5f3) compared to base (1bbbbef).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2155   +/-   ##
=======================================
  Coverage   46.91%   46.91%
=======================================
  Files         222      222
  Lines       32719    32719
=======================================
  Hits        15351    15351
  Misses      16134    16134
  Partials     1234     1234

Flag
Coverage Δ

armada-server
46.91% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


on server launch, second process opened in task manager

when using the launcher to start the server, it starts a second process which isn't terminated when the launcher is closed.  if anything causes the server to crash, the server process doesn't get properly terminated when being launched using the launcher, namely outdate mods that aren't supported will cause it to crash.

Closing this as I couldn't replicate this


Bug: Screenshots only show visible parts within app viewport

Current: If you select a screen in Reflex, press copy to clipboard or save -> the image is only whatever was within the Reflex app viewport. Anything outside of that is cut off.
Expected: Entire Screen in Reflex should be screenshotted, not just the part which is visible in the app.
v0.7.0-beta.22
Fix branch: #217
Example:

NOTE: This is still an issue, and no clear workaround, since the content outside of the window is not rendered at all.


check SPS/PPS of H264/H265 SDPs

This SPS/PPS check killed a webcam stream that had been working. Is there a config option to disable SPS/PPS checking?
Here's the error in the log:
[RTSP source] media 1 is invalid: invalid SPS: not a SPS

@KanyonKris open an issue and provide a server log with logLevel: debug


openjdk test cases is different between Mode150 and Mode650?

In aqa-tests all openjdk tests are running with both mode150 and mode650. That is the number of testcase should be same for the two mode? However it feels like not. Maybe I misunderstand something or investigation should be taken to figure it out?
For example: compiler/rtm/locking/TestRTMAbortRatio.java is running with Mode650 and is not running with Mode150 on AIX.
jvm_compile_1 Test results: passed: 839; failed: 1
20:33:30  Report written to /home/jenkins/workspace/Test_openjdk11_hs_extended.openjdk_ppc64_aix_testList_0/jvmtest/openjdk/report/html/report.html
20:33:30  Results written to /home/jenkins/workspace/Test_openjdk11_hs_extended.openjdk_ppc64_aix_testList_0/aqa-tests/TKG/output_16504090503753/jvm_compiler_1/work
20:33:30  Error: Some tests failed or other problems occurred.
20:33:30
20:33:30  jvm_compiler_1_FAILED
https://ci.adoptopenjdk.net/job/Test_openjdk11_hs_extended.openjdk_ppc64_aix_testList_0/37/consoleFull
jvm_compile_0 Test results: passed: 821
https://ci.adoptopenjdk.net/job/Test_openjdk11_hs_extended.openjdk_ppc64_aix_testList_1/37/consoleFull
Looking at the test compiler/rtm/locking/TestRTMAbortRatio.java itself doesn't look like compressedRef is filtered.

Can see some tests are excluded in upstream problemlist on x64 , not aix [(https://github.com/adoptium/jdk11u/blob/master/test/hotspot/jtreg/ProblemList.txt)


Facing error while deploying python function on Azure

Repro steps:

Install npm, azure core tools, and .NET
Create a Linux based Python function and try to run it
On a successful run, deploy it and you will see the error.

Action: azureFunctions.deploy
Error type: EPERM

Error Message: EPERM: operation not permitted, scandir 'c:\Users\Manish\Application Data'
Version: 1.0.1
OS: win32
OS Release: 10.0.18363
Product: Visual Studio Code
Product Version: 1.51.1
Language: en

@manish-shegokar you may want to try putting your project in a different folder. The "Application Data" folder might have limited permissions

It worked! Thanks a lot :)


Fixed: define lv_img_decoder_user_data_t

// src/lv_draw/lv_img_decoder.h
typedef void * lv_img_decoder_user_data_t;

This is defined in lv_conf.h

@seyyah
See https://github.com/littlevgl/lvgl/blob/dev-6.0/lv_conf_template.h#L149

Ohhh sorry :(

No problem :)


Multiple rows returned with uselist=False for eagerly-loaded attribute 'PokemonMove.machine

Happens in one of the spline-pokedex tests. Haven't investigated what's triggering it.

lib/python2.7/site-packages/sqlalchemy/orm/loading.py:305: SAWarning: Multiple rows returned with uselist=False for eagerly-loaded attribute 'PokemonMove.machine

This seems to be because Waterfall, Surf, and Fly are both HMs and TMs in S/M and US/UM. I'm not sure why we even have HMs for those games -- i guess they're in the data, but unused?

This should be now fixed (80bc35a764295a86dc09d49672265d9f8b7ce3a2)


Added Godot gitignore (https://godotengine.org/)

Reasons for making this change:
Godot Engine is becoming a major game engine in game development industry, and in preparation for the next major release, Godot Engine 3.0, I've prepared this .gitignore in order to make it easy for new projects to start with a template that will ignore automatic generated files, such as .import ones which will be generated as soon as a compatible file, such as a .png, is added to the project.
Also since Godot now supports C# by using Mono, I added the .mono directory.
Links to documentation supporting these rule changes:
You can check how Godot Engine handles the import process in the official documentation.
If this is a new template:
Godot has a github repository with more than 10.000 stars and over 2.500 forks and an official homepage with contents being updated frequently.

You shouldn't ignore *.import, since in this file are all the settings how to import the asset (eg Filter/Mipmap, etc)

You forgot the logs/ folder.

You forgot the logs/ folder. Unless that should be included for some reason?

That was during unstable development, it should not longer be generated in the project folder.

Closing this out in favour of #2613 which has some more detailed rules (and refers to the example from a demo project)


Add Chad-Blackmon to Contributors list

@chadb4184: Do you wanna reopen this pull request so that I can merge this to the project?


Get list node available versions in machine

Hi thank you for amazing tool I'm pretty love this but I need gonvm to get list available node version on machine.
example command
gonvm list

result
node/8.9.0
node/10.16.0
node/12.18.3

Hope you consider for this feature, thank you.

I think only version installed version number should fine, no need for node/.

Add in #4


DOC fix indent and improve wording for multiagent

The code example was not indented correctly.
Wording

Pull Request Test Coverage Report for Build 3283

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 88.84%

Files with Coverage Reduction
New Missed Lines
%

flow/core/kernel/vehicle/traci.py
1
87.58%

Totals

Change from base Build 3225:
-0.01%

Covered Lines:
7953

Relevant Lines:
8952

💛  - Coveralls


Fix ICU support data loading

This fixes an error (https://github.com/godotengine/godot/issues/97475) in loading the ICU support data on platforms that use neither the builtin icu4c nor the static ICU support data.
Bugsquad edit:

Fix #97475

Thanks! And congrats for your first merged Godot contribution :tada:


Error during javadoc generation

When the action ran, I got this error during the generation of the Javadoc. It fails because it doesn't recognize lambda and stream function:
[INFO] --- maven-javadoc-plugin:3.4.1:jar (attach-javadocs) @ zeebe-cherry-framework ---
[INFO] No previous run data found, generating javadoc.
Error:  MavenReportException: Error while generating Javadoc:
Exit code: 1 - /home/runner/work/zeebe-cherry-framework/zeebe-cherry-framework/src/main/java/io/camunda/cherry/admin/RunnerRestController.java:80: error: method references are not supported in -source 7
        .map(RunnerInformation::getRunnerInformation)
                                ^
  (use -source 8 or higher to enable method references)
/home/runner/work/zeebe-cherry-framework/zeebe-cherry-framework/src/main/java/io/camunda/cherry/admin/RunnerRestController.java:81: error: lambda expressions are not supported in -source 7
        .map(w -> this.completeRunnerInformation(w, // this
               ^
  (use -source 8 or higher to enable lambda expressions)

I tried to add
maven-additional-options: -source 17

without success.
What are the correct parameters?
NB: it should be nice to have this parameters by default to publish JDK 1.17 code, at least in the README.

This is usually related to the runtime version of your workflow or the configured compiler source version


Rename command mode to normal mode

http://vimdoc.sourceforge.net/htmldoc/intro.html#vim-modes-intro
Although normal mode is indeed also known as the command mode, AFAIK, the term normal mode is used more often.
Also since we are now planning the command-line mode, renaming it would make them less confusing.
What do you say, I can make a PR for this.

Yeah I agree, normal mode is what I use most often. I'd merge said PR.

@maxbrunsfeld  how does #508 look like?

I would strongly suggest to fix this ASAP. As more code is written, it will take greater effort to correct this. Command mode should refer to the mode in the command bar.

@maxbrunsfeld up

this is in master, can be closed.  :fireworks:

As mentioned by @bronson, this was resolved when https://github.com/atom/vim-mode/pull/508 got merged.


fix typos

Hallo!
Apart from these, I noticed that questions_<country>_<year>.csv doesn't seem to match the files any more. Aren't they just country>_<year>.csv now? PLMK if I should push a commit correcting that here :-)

👍


Add an option to not changed directory to the one the noxfile is in.

How would this feature be useful?
Currently I'm writing a wrapper to Nox that doesn't publically expose the noxfile.py. Nox then changes directory to my package with no clean way to not do this or get back.
Describe the solution you'd like

Add a --cwd argument that allows the user to specify the directory they want to work in.
Add a --no-change-cwd flag that disables changing the CWD.

Describe alternatives you've considered

Locally I've removed tasks.py#55 which has fixed this.*
Given the comment above this line I don't think this is a valid solution.

Pre-invocation of Nox I could add a file to where I know Nox will change directory. This file will contain the directory I actually want to be in. I could then change directory in each session from this file.
This is really a hack, not a solution.

* It should be noted when removing tasks.py#55, each session is in the unwanted directory but not the global scope. This can be solved by adding the CWD to the global scope and changing in each session. I'm unsure what's causing this however.

Hmm. I'm unsure on this one.

Well I don't see
os.chdir(os.path.realpath(os.path.dirname(global_config.noxfile)))

as being strictly necessary to satisfy the stated reason

This will ensure that the Noxfile's path is on sys.path

This could also be accomplished by modifying / verifying sys.path

I like noxfiles having the guarantee that their cwd is the same directory that they're in.

I'm +1 on this, If you wan to change the cwd, you can just use session.cd https://nox.thea.codes/en/latest/config.html#nox.sessions.Session.cd Otherwise, it wouldn't be deterministic and annoaying to change directories instead of doing nox -f foo/bar/noxfile.py.
I don't fully understand the use case, but you could pass the cwd form the wrapper as a posarg maybe?
Also, maybe nox could provide a way to get the original cwd.

This could also be accomplished by modifying / verifying sys.path - dhermes

I may be wrong, but it doesn't seem like changing sys.path is a requirement. Removing tasks.py#55 and using nox -f not_in_path/noxfile.py works just fine.

If you wan to change the cwd, you can just use session.cd - stsewd

That doesn't solve the problem unless tasks.py#55 is removed, as there's no way to get the old CWD.

Otherwise, it wouldn't be deterministic and annoaying to change directories instead of doing nox -f foo/bar/noxfile.py.

I don't see how --cwd wouldn't be deterministic.
It can also just default to the value tasks.py#55 changes to. Which would make it exactly the same as nox -f foo/bar/noxfile.py with the option for me to do nox -f foo/bar/noxfile.py --cwd ..

I don't fully understand the use case, but you could pass the cwd form the wrapper as a posarg maybe? - stsewd

No as I need access to a user defined config file to make the functions wrapped in nox.session. Take the following MVCE:
import json

import nox

def build_task(task):
    def function(session) -> None:
        session.install(*task['deps'])
        session.run(*task['command'])

    function.__name__ = task['name']
    return nox.session(function)

with open('./user_config.json') as config_obj:
    for task in json.load(config_obj):
        build_task(task)

maybe nox could provide a way to get the original cwd.

I'd be happy with that, but it'd have to be available from nox, not session. As shown above.

The noxfile today assumes all paths are relative to the file itself. I don't quite follow what use case this resolves. It seems like maybe you have a repository that isn't using nox but you would like to but not check in the noxfile, so you keep it out of that directory tree? In this case it seems straightforward to cd in the nox session since the path won't be variable.
Regarding cwd being deterministic: The issue is there are multiple ways a noxfile author could implicitly require the working directory to match. While in your case you have prepared for that, this argument could be used by any passerby on any noxfile. So creating a situation where an arg could have unintended side effects to an execution could be problematic.

@crwilcox As I said before I can't cd in the session, because, as the code I provided above shows, it requires to be in the correct directory to create the session objects. And nox is preventing that from happening.

I'm just going to close this, as I don't see this feature request having a future even if I were to submit a PR.


Update get-started-core-ediscovery.md

Added Exo PLan 2 to the list as org with Exo plan 2 can use the standard features

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/8813


OK-2694 Solve the problem of stc's read only wallet that reports an error when upgrading the password

What does this implement/fix? Explain your changes.
修复了ios下存在只读钱包的时候 升级密码提示“raising by require”的问题
Does this close any currently open issues?
If it fixes a bug or resolves a feature request, be sure to link to that issue.
https://onekeyhq.atlassian.net/browse/OK-2694
Pull request type

Put an x in the boxes that apply

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Where has this been tested?
Python
Any other comments?
…

read_only 钱包为什么要更改密码？

read_only 钱包为什么要更改密码？

不是watch only的要改，是改密码的时候需要遍历全部钱包，需要跳过那些watch only的，不然也是到底层实现那里静默跳过，效果一样，不如这里清晰。


[multistage] m0ar tests! (binary, boolean, char, time)

see description. Also:

fixed a bug in DataBlockBuilder where it's possible to get a byte[] instead of ByteArray from transferable block - (the Segment builder uses PintoDataType#convert, which for PinotDataType.BYTES returns a byte[] and not a ByteArray). Not sure if this is a bug in Pinot v1, but I figured it's safer to just fix DataBlockBuilder to account for both cases.
cleaned up some error messages
added necessary comparators for testing

Codecov Report

Merging #9818 (8b2af1a) into master (73e6129) will increase coverage by 38.76%.
The diff coverage is 60.00%.

@@              Coverage Diff              @@
##             master    #9818       +/-   ##
=============================================
+ Coverage     24.55%   63.32%   +38.76%
- Complexity       53     4776     +4723
=============================================
  Files          1952     1952
  Lines        104676   104684        +8
  Branches      15856    15858        +2
=============================================
+ Hits          25700    66286    +40586
+ Misses        76347    33530    -42817
- Partials       2629     4868     +2239

Flag
Coverage Δ

integration1
25.35% <0.00%> (?)

integration2
24.57% <0.00%> (+0.02%)
:arrow_up:

unittests1
67.89% <60.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

.../pinot/core/common/datablock/DataBlockBuilder.java
74.23% <33.33%> (+74.23%)
:arrow_up:

...a/org/apache/pinot/query/catalog/PinotCatalog.java
47.36% <50.00%> (+47.36%)
:arrow_up:

.../java/org/apache/pinot/query/QueryEnvironment.java
84.84% <100.00%> (+84.84%)
:arrow_up:

.../helix/core/minion/MinionInstancesCleanupTask.java
54.54% <0.00%> (-9.10%)
:arrow_down:

...tream/kafka20/server/KafkaDataServerStartable.java
72.91% <0.00%> (-6.25%)
:arrow_down:

...pache/pinot/core/query/utils/idset/EmptyIdSet.java
25.00% <0.00%> (ø)

...ot/common/function/scalar/ComparisonFunctions.java
42.85% <0.00%> (ø)

...anager/realtime/SegmentBuildTimeLeaseExtender.java
63.23% <0.00%> (ø)

...g/apache/pinot/common/datatable/BaseDataTable.java
78.94% <0.00%> (+0.75%)
:arrow_up:

...e/pinot/common/function/TransformFunctionType.java
100.00% <0.00%> (+0.94%)
:arrow_up:

... and 1240 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


data structure varies depending upon number of rows in input

Thanks for putting together this handy package!
I noticed a small inconsistency in terms of how the data are returned when using axis = 1 vs. pandas' apply in cases where the rows are < 200 vs. >= 200. Here's a minimal reproducible example of the issue:
import pandas as pd
import mapply

mapply.init(n_workers = 2)

dat1 = pd.DataFrame(list(range(1, 200)))
dat2 = pd.DataFrame(list(range(1, 201)))

display([len(dat1), len(dat2)])

display([type(dat1.mapply(sum, axis = 1)), type(dat2.mapply(sum, axis = 1))])

display([type(dat1.apply(sum, axis = 1)), type(dat2.apply(sum, axis = 1))])

[199, 200]
1/? [00:00<00:00, 19.46it/s]
2/? [00:00<00:00, 54.05it/s]
[pandas.core.frame.DataFrame, pandas.core.series.Series]
[pandas.core.series.Series, pandas.core.series.Series]

Note that with mapply one result is a DataFrame whereas the other is a Series, but for pandas apply returns a Series in both cases.

Hi @davedgd,
The default chunk_size=100, combined with n_workers=2 and the dat1 frame with length 199, somehow prevents correct dtype casting.
Will try to have a look tonight.
Thanks for your bug report!

Thank you @ddelange!

Released 0.1.6
Thanks again :)


Support for various languages

The next in line is probably Python.

I recently used the Codeforces API to find out what languages are most popular (in terms of number of submissions sent during the contest by participants in the appropriate division). Here are the results:
Div. 1 (contests 1545, 1540, 1528):
C++                95.29%
Python             1.61%
Java               1.44%
Rust               0.52%
C#                 0.49%
Kotlin             0.32%
D                  0.14%
C                  0.10%
Haskell            0.07%

Div. 2 (contests 1546, 1543, 1542):
C++                90.10%
Python             5.25%
Java               3.46%
C                  0.70%
Kotlin             0.12%
C#                 0.10%
Rust               0.09%

Div. 3 (contests 1551, 1547, 1538):
C++                87.20%
Python             6.94%
Java               4.42%
C                  1.08%
Kotlin             0.08%
C#                 0.08%

The code that gave me these results: gist.

Codeforces Toolbox now supports 99.77%, 99.85% and 99.87% from Divisions 1, 2, 3 respectively. I think that's enough to close this issue.
Again, the results from the last three contests from each division were extracted using this gist.


Creating replication group silently fails if non-NEW_AND_OLD_IMAGES stream exists

In DynamoDBReplicationUtilities.createTableIfNotExists, the current stream specification is compared against the desired stream specification, and the table's config is updated if they don't match.
This works fine if streams aren't enabled in the first place. However, if streams are already enabled, but the stream type is something other than NEW_AND_OLD_IMAGES, the UpdateTable call fails with a ResourceInUse exception.
As a side note, this was pretty difficult to debug, because the resulting exception isn't actually printed - you just get the following log line in CloudWatch:
2015-09-06 12:37:41,108 ERROR com.amazonaws.services.dynamodbv2.replication.coordinator.state.DynamoDBReplicationGroupCreationStarted - Unable to create table for replication member with ARN: <arn>

From the [UpdateTableRequest](http://docs.aws.amazon.com/AWSJavaSDK/latest/javadoc/com/amazonaws/services/dynamodbv2/model/UpdateTableRequest.html#setStreamSpecification(com.amazonaws.services.dynamodbv2.model.StreamSpecification) documentation :

You will receive a ResourceInUseException if you attempt to enable a stream on a table that already has a stream, or if you attempt to disable a stream on a table which does not have a stream.

Seems like the stream has to be disabled before it's type can be changed.

Right, although I think the larger issue is that this fact is not surfaced to the user via the replication console, and is barely surfaced at all via those CloudWatch logs.

Hi,
As a part of the improvement process for cross-region replication library, we have refactored the library completely and released an updated version. As a result, the previous version involving multiple components has been deprecated and I suggest you use the new libraries as command-line tools instead. The error referred to here should surface as an error when attempting to execute via command line.
All the code has also been refactored down to the fundamental components only, stripping away other complicated and unnecessary dependencies. This should make the library much easier to understand, and a lot less cumbersome to manage as a client-side solution.
Thank you!


South Africa Address Parsing (unit, house_number, road, house confusion)

Here is the input:
308 SOHO on Strand, 128 Strand Street, Cape Town, W Cape, 8005, South Africa
Current parsing result:
                house_number:308
                road:soho on strand
                house_number:128
                road:strand street
                city:cape town w cape
                postcode:8005
                country:south africa

It should be:
                unit:308
                house: soho on strand
                house_number:128
                road: strand street
                city:cape town
                state: w cape
                postcode:8005
                country:south africa

Won't fix.

this works: SOHO on Strand, 128 Strand Street, Cape Town, Western Cape, 8005, South Africa
so does this: Unit 308 SOHO on Strand, 128 Strand Street, Cape Town, Western Cape, 8005, South Africa

Those forms are more standard and we're not going to generate naked unit numbers like "308" to the training data as they're difficult to disambiguate from house numbers and humans don't commonly write addresses that way.
Will add the abbreviation "W" for "Western" as opposed to just "West" which will allow the parser to get "W Cape".


Creating tests to the "cartolafc.Api.schemes" method

Coverage increased (+2.0%) to 82.392% when pulling 3fd223ad99deff4c015b98c99e3be234e6ae4171 on dev into 9e3e5c900d63b4ae92878caf05b583bd92e21e48 on master.


Create 0000-01-02-Rahupau99.md

Just learning

Why it is blocked?

Why it is blocked?

PR confused?


[fix][#1191] refactor job_logs

Coverage decreased (-0.2%) to 85.236% when pulling b3c54b27c11ecdaa1f6048c15695bb6f676b6d03 on sunny4381:ss-1191 into bde1741d330a563f726f013f0c4168f7489b77f1 on shirasagi:master.


feat: add overridable default input artifacts #2026

This adds support for overridable default input artifacts as described in  #2026. An earlier PR was opened in #7341 but it was not showing the diff accurately due to https://stackoverflow.com/questions/16306012/github-pull-request-showing-commits-that-are-already-in-target-branch hence this was opened.

@alexec apologies for the new PR but I have added the test for you


Sublime Text 3 inline style expantion on tab doesn't work

In ST2 I press Tab key after zero and for example this

 expands into this

But it doesn't work for me in ST3. In both cases document syntax set to HTML. If I try to expand m0 when syntax set to CSS it works. I tried to remove Emmet and PyV8 packages and re-install them. I didn't helped. I use 3047 build of ST3 and the newest version of Emmet.

Hey guys, i have the same problem, well i think its the same. I'm new in sublime text (and more, i'm french) so, they're certains things i don't understand very well. How did you do Nikolay to solve the problem. I have the same problem since i have dowloaded emmet package ... :/


To Reproduce
Here is a basic reproduction:
https://codepen.io/Booligoosh/pen/ExVyVgb
Comment out the line in the JS saying use_native: true and it will work.
Expected behavior
It should proceed without native lazy loading for video elements, since there is no native lazy loading for video elements
LazyLoad version
15.1.1
Desktop (please complete the following information):

OS: MacOS
Browser Chrome
Version 81.0.4044.92

Actually, I just found the solution you posted here. I'm using that for now. However, it might be worth mentioning something about it in the docs, as it probably causes confusion for some people.

Thanks for your feedback.
Yes native lazy loading only works on images and iframes, for now.
Will update the docs.


extension prevents "live chat" options from appearing on some websites

Description

If the extension is enabled, some websites will not display their "live chat" widget or similar pop up window.
Steps to Reproduce

with the extension enabled go to, for instance, https://www.greengeeks.com/ and see if there is a live chat widget pop up option displayed in the bottom right hand corner

disable the extension, refresh the page and the widget will appear in the bottom right hand corner.

Expected behavior:

There is supposed to be a widget pop-up window displayed in the bottom right hand corner with the title 'GreenGeeks LiveChat'
Actual behavior:

There is no widget
Versions

Extension: 2020.7.9
Browser:  Firefox: 78.0.2 (64-bit)
OS:  Linux Mint 20

Additional Information

DDG indicated to me by email that certain tracking software scripts are blocked by default and that unblocking them to enable tools like LiveChat.com or other live chat scripts would defeat the privacy protection purposes of the extension.
In speaking with LiveChat, I determined all I could really do at this point was test for an instance of the backgroun chat object in question and throw up a popu-up letting them know that they need to turn off privacy protection for this site in order to use the live chatt/support feature.
It's a lousy non-solution, but it is where we are. So long as these live chat platforms use tracking libraries that DDG does not like they will be blocked.

DDG indicated to me by email that certain tracking software scripts are blocked by default and that unblocking them to enable tools like LiveChat.com or other live chat scripts would defeat the privacy protection purposes of the extension.
In speaking with LiveChat, I determined all I could really do at this point was test for an instance of the backgroun chat object in question and throw up a popu-up letting them know that they need to turn off privacy protection for this site in order to use the live chatt/support feature.
It's a lousy non-solution, but it is where we are. So long as these live chat platforms use tracking libraries that DDG does not like they will be blocked.


Server logs suggest there is some issue with continuous batching on GD2.
#testing by sending 5 request on Gaudi2 TGI endpoint. Note that the queue time is increasing for subsequent inference requests
Req1: total_time="3.076226394s" validation_time="449.063µs" queue_time="110.028µs" inference_time="3.075667684s" time_per_token="66.86234ms"
Req2: total_time="3.076173218s" validation_time="3.502745ms" queue_time="70.64658ms" inference_time="3.002024052s" time_per_token="65.261392ms"
Req3: total_time="3.132718439s" validation_time=""786.778µs" queue_time="201.632982ms" inference_time="2.930298993s" time_per_token="63.702152ms"
Req4: total_time="3.197355097s" validation_time="1.277488ms" queue_time="331.050014ms" inference_time="2.865027991s" time_per_token="62.283217ms"
Req5: total_time="3.259123777s" validation_time="924.292µs" queue_time="459.104331ms" inference_time="2.799095535s" time_per_token="60.849902ms"

#Same test as above this time sending 5 requests to a single Nvidia T4 card running TGI docker 2.0.4. Note that the queue time is more or less constant after the first request indicating effective continuous batching

Req1: total_time="1.513475533s" validation_time="1.069695ms" queue_time="52.017µs" inference_time="1.512354236s" time_per_token="32.877266ms"
Req2: total_time="1.507096983s" validation_time="799.031µs" queue_time="54.518157ms" inference_time="1.451780025s" time_per_token="31.560435ms"
Req3: total_time="1.502753387s" validation_time="418.679µs" queue_time="50.525381ms" inference_time="1.451809782s" time_per_token="31.561082ms"
Req4: total_time="1.507244713s" validation_time="841.468µs" queue_time="54.479958ms" inference_time="1.451923498s" time_per_token="31.563554ms"
Req5: total_time="1.503086631s" validation_time="828.972µs" queue_time="50.359691ms" inference_time="1.451898309s" time_per_token="31.563006ms"

Expected result:
Gaudi 2 throughput numbers on the TGI endpoint (with continuous batching) should be at par or better than the static benchmark throughput

Not sure why the queue time is increasing, any idea @kdamaszk ?


[HttpServer] Fixes #28 - React to breaking Web schema changes

This brings the client inline with the latest HttpServer schema changes. Note, this change also effects HUD. Fixes #28.

other than case-sensitive header names, this looks good to me.

Ok I've made the last of the changes/updates. Any other feedback?

OK, lgtm.


Does the gain-tracking algorithm work even if exercising at constant speed?

@xioTechnologies
Hi, Thanks for your great work.
I'm going to do a dead reckoning using IMU in a mobile robot.
For this algorithm, I understand that speed and position are obtained through integration only if there is acceleration.
For constant velocity motion, the acceleration value is very small.
Then can I use this algorithm even in the above situation?
If it's not suitable, do you have any algorithms to recommend in my situation?
I'd appreciate it if you could answer me.
Thanks,

You are unlikely to achieve useful measurements of position using an IMU alone.  Consider that even just a 1 degree inclination error in the orientation measurement would result in an acceleration measurement error of 1g * sin(1) = 0.171 m/s/s.  This would result in a position error of 8.5 meters after 10 seconds of dead reckoning.  IMUs are typically only used for positional measurement when combined with other technologies such as GNSS.

@xioTechnologies
Thank you for your quick and kind reply.
I will find a way to apply other sensor fusion methods.
Thanks,


docs[minor],core[minor]: Add support for mustache templates

Todo:

[x] docs

Seems fine overall, but how does this affect automatic typing? We use the type ParamsFromFString in PromptTemplate.fromTemplate args


Drop older unsupported versions of Django

These are making it hard to validate support for Python3.11.
Builds on #20.

The only thought I had on this -- since it's technically breaking, do you want to put out a release which doesn't have this change, then merge this and #17 and then another release?

Thanks @PeterJCLaw! I cut a release just before this one and called it 1.0 as I think it's perfectly usable. v1.1 will include this.


How to get a non inline toml string from a document

Considering the following toml file:
[package]
authors = [ "me", "I", "myself" ]
name = "proj"
version = "0.0.1"

[profile]

After loading and saving it with the following code (tomlet 5.1.3):
TomlDocument document = TomlParser.ParseFile("file.toml");
using StreamWriter streamWriter= new("result.toml");
document.ForceNoInline = true; // With or withou this, the result is the same
streamWriter.Write(document.SerializedValue);

The content of result.toml file is inlined:
package = { authors = [ "me", "I", "myself", ], name = "proj", version = "0.0.1" }
profile = {  }

How can we get a non inlined string for a document that is loaded not inlined?

You need to set the ForceNoInline property on the package table itself.

It could be nice to not inline the table that is loaded not inlined

@SamboyCoding I am also of the opinion that it should be possible to preserve the original formatting or at least instruct the serializer to emit the same format as the input.
Rationale is that sometimes config files need to be both deserializable / serializable by an application and human readable / editable. Dropping original formatting by the parser kills this dual use case.


When passing undefined value to findById function it returns first row

This is very dangerous issue, when passing undefined value to findById(id) function it returns first row from queries table. I'm observing this issue on MySQL database.

Should already be fixed. What version are you using?

I'm on ^0.5.5

Yep, update to 0.6 and this will work.

Thx :)


GUACAMOLE-1813: Add support for history recording storage extension to Docker

This adds support for the guacamole-history-storage-extension to the Docker container. I went ahead and based it against 1.5.3 - if anyone feels it should get pushed out to 1.6.0 that's fine - it's a bit on the fence as to whether it's actually a bugfix or not.

I agree with this being a bugfix and worth basing on 1.5.3.
It may even be worth expanding the concept of what can be included in these patch releases to anything that doesn't require changes to the docs (ie: anything that we can merge and then call it "done").


fixed issue #221 + ip ranges.

Apologies, not sure why I didn't merge this earlier. Thanks for the fixes!

Thank you ;)


Update condition for missing network

Network creation wasn't working for me until I changed this condition to look for the string 'network not found' instead of 'no such network'. I can't tell, but maybe this worked in the past and the message changed. It is currently defined here.
fixes #28

This is 100% incorrect because ansible does have an implicit AND between conditions and not OR. Please read and correct it to do an or between the two strings, so the conditional is not broken.
See https://docs.ansible.com/ansible/latest/user_guide/playbooks_conditionals.html

I am familiar with the implicit AND, but I don't know why my change is incorrect. The logic used to look like this:
"If there is a message AND the message contains "" AND the network information is not in the result..."
Since that message is not output by Podman under any circumstance (that I'm aware of), the network does not get created even when it should be. I didn't change the structure of the logic, I just corrected the text of the message being tested for:
"If there is a message AND the message contains "" AND the network information is not in the result..."
Maybe my understanding of the situation is incorrect, but I don't think changing any of those ANDs to an OR would be correct.
I've tested this locally and it fixes the problem for me. Maybe the solution isn't universal across all versions of Podman? If I added a Molecule test would it be helpful?

That's very helpful! I'll change it to something like this:
      when:
        - podman_network.results[0].msg is defined
        - "'network not found' in podman_network.results[0].msg or 'no such network' in podman_network.results[0].msg"
        - podman_network.results[0].networks is undefined

I don't think it'll be too difficult to add a test for this in the existing molecule scenario. I'll give it a shot this week. I'm not sure if I'll be able to figure out how to matrix in the different versions of podman.

I don't think it'll be too difficult to add a test for this in the existing molecule scenario. I'll give it a shot this week. I'm not sure if I'll be able to figure out how to matrix in the different versions of podman.

imho, it should be a new scenario and not a modification of default one so it's a little bit more work.


LVP doesn't resize with smaller window size

When running LVP on 2017 MacOS Big Sur (13.3 inch, 2560 × 1600), the side panel UI gets slightly cut off. I can't see the "Enable LED" button. See attached

I am also encountering a similar issue on Windows 10 Home (run through BootCamp) with a Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz   2.30 GHz processor; the side panels get cut off and the 96-well plate cells become dilated (this dilation was not present on MacOS):

The cutoff is undesirable. The usage should work for 1080p and larger resolutions. We need to implement minimum window size. The "dilated" well size was part of the design and the little wells you see in Mac is the wrong size. We need to use dp units instead of px units. For the cutoff on the right panel (even at 1080p), the plan is to transition this into tabs on the top instead of the accordion.


Add zsh fish bash completion support

Add zsh fish bash completion support
Place the completion file of the corresponding shell in the completion folder in the specified location

and the specific effect is as follows
bash

zsh

fish

Hi,
What is the point of adding those generated files here ?
I think this should be closed in favor of #861, or something like that.

Oh, thank you for your reminder. I did not notice that there was already a completion operation, but using LSD on my computer did not have this effect, so I wrote these shell completion myself. If there is no problem running lsd completion on someone else, then this pr can indeed be closed

Arch already provide them : https://gitlab.archlinux.org/archlinux/packaging/packages/lsd/-/blob/main/PKGBUILD#L42-L44
Nix too: https://github.com/NixOS/nixpkgs/blob/master/pkgs/tools/misc/lsd/default.nix#L28
You probably just need to set SHELL_COMPLETIONS_DIR

Arch already provide them : https://gitlab.archlinux.org/archlinux/packaging/packages/lsd/-/blob/main/PKGBUILD#L42-L44 Nix too: https://github.com/NixOS/nixpkgs/blob/master/pkgs/tools/misc/lsd/default.nix#L28
You probably just need to set SHELL_COMPLETIONS_DIR

Wow, thanks, I'll try

https://github.com/lsd-rs/lsd/blob/8acaabec723cdd71b498212c8a47ac952d022738/build.rs#L32-L35
we already support this
/close


Usage of common localStorage variable UID

Hello,
The script makes usage of general localStorage variable names like "uid" which can present a problem when using with different other plugins and scripts.
It's always a good practice to set those variable names with specific and almost a unique way to avoid clashing with other scripts, e.g. "profanityFilter_uid"

have you found this to actually conflict?

Of course, otherwise I wouldn't be posting an issue.

can you describe your issue? the var uid is scoped to the localStorageIsEnabled function.
But if you're having an issue, I'll happily take a PR.


Failed to create start menu shortcut for GUI application

When I want to install a GUI application by the scoop (for example typora or everything), scoop able to successfully install the application, but can not create a shortcut to the Start menu. The output of scoop is as follows

(All shortcuts in the "Scoop Apps" folder are created by myself, including this folder)

search everything.lnk in Everything, any result?

search everything.lnk in Everything, any result?

search everything.lnk in Everything, any result?

no，there is no any .lnk file in everything

search everything.lnk in Everything, any result?

no，there is no any .lnk file in everything

I mean use app Everything to search everything.lnk

I mean use app Everything to search everything.lnk

I mean use app Everything to search everything.lnk

Everthing: uninstall and then install ,nothing

Quicklook: new installed，nothing too

I mean use app Everything to search everything.lnk

Everthing: uninstall and then install ,nothing

Quicklook: new installed，nothing too

remove Scoop Apps folder you created,
run windows powershell (C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe)
exec scoop reset *
scoop can also install pwsh core, but I think there are some problems with the current configuration of the manifest.
other ways to install powershell core:
https://docs.microsoft.com/en-us/powershell/scripting/install/installing-powershell-core-on-windows?view=powershell-7.1
I installed pwsh core by dotNet global tool.
If you have joined the windows insider program: https://github.com/lukesampson/scoop/issues/3941

remove Scoop Apps folder you created,
run windows powershell (C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe)
exec scoop reset *
scoop can also install pwsh core, but I think there are some problems with the current configuration of the manifest.
other ways to install powershell core:
https://docs.microsoft.com/en-us/powershell/scripting/install/installing-powershell-core-on-windows?view=powershell-7.1
I installed pwsh core by dotNet global tool.
If you have joined the windows insider program: https://github.com/lukesampson/scoop/issues/3941

After I reinstall the Windows system. First, I install pwsh and Windows Terminal then run this command in it to install scoop.
iwr -useb get.scoop.sh | iex
I found all the shortcut are created in the pwsh folder rather than "%HomePath%\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\Scoop Apps". The folder also ends with "AppData\Roaming\Microsoft\Windows\Start Menu\Programs\Scoop Apps", this led me to come up with the solution.
I guess scoop was limited to create this folder "%HomePath%\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\Scoop Apps".
So I uninstall Windows Terminal and pwsh just run scoop reset * in Powershell. then PowerShell tells me that "scripts cannot run on this computer." Then I run this command Set-ExecutionPolicy RemoteSigned to remove this limit. Then, run "scoop reset *"
again. The problem was solved perfectly.
Shortly speaking. To solve this problem, follow these 3 steps.

Uninstall Windows Terminal and pwsh. ( I'm not sure if it's necessary, you can have a try.)
Run Set-ExecutionPolicy RemoteSigned in Powershell.
Run scoop reset * in Powershell.
Reinstall Windows Terminal and pwsh as you wash.

After I reinstall the Windows system. First, I install pwsh and Windows Terminal then run this command in it to install scoop.
iwr -useb get.scoop.sh | iex
I found all the shortcut are created in the pwsh folder rather than "%HomePath%\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\Scoop Apps". The folder also ends with "AppData\Roaming\Microsoft\Windows\Start Menu\Programs\Scoop Apps", this led me to come up with the solution.
I guess scoop was limited to create this folder "%HomePath%\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\Scoop Apps".
So I uninstall Windows Terminal and pwsh just run scoop reset * in Powershell. then PowerShell tells me that "scripts cannot run on this computer." Then I run this command Set-ExecutionPolicy RemoteSigned to remove this limit. Then, run "scoop reset *"
again. The problem was solved perfectly.
Shortly speaking. To solve this problem, follow these 3 steps.

Uninstall Windows Terminal and pwsh. ( I'm not sure if it's necessary, you can have a try.)
Run Set-ExecutionPolicy RemoteSigned in Powershell.
Run scoop reset * in Powershell.
Reinstall Windows Terminal and pwsh as you wash.

Thanks, I have solved this problem by using the .msi file to reinstall powershell (previously my powershell core was installed through the Microsoft Store). I think the cause of the problem may be some kind of sandbox mechanism in the Microsoft Store. When the powershell installed through the Microsoft Store executes the script, ~ is not pointing to the current user's folder, but to a temporary user folder inside it


FatalErrorException in StatusCheckProvider.php line 6

Hi,
I am facing this issue whenever trying to access the StatusCheckProvider for and existing order.
FatalErrorException in StatusCheckProvider.php line 6:
Access level to Anand\LaravelPaytmWallet\Providers\StatusCheckProvider::$response must be protected (as in class Anand\LaravelPaytmWallet\Providers\PaytmWalletProvider) or weaker

hi @itsavinash
Thanx for reporting! Just release a patch, update the package.
Peace

Hi,
I have updated composer update with
"anandsiddharth/laravel-paytm-wallet": "1.0.3",
But it's not working. I am getting same error. Is there anything need to changed?


Add support for port-groups

Add support for port-groups and virtual-port-groups.
Expose option nic-type to customers to allow them to choose
the network type they want to configure.

Yes, it does fix the network-name issue when encrypting on the ESX host.


[No QA] fix focus state is undefined

Details

focus state was undefined as we forget to return value from it.
Credits: @kidroca for mentioning it https://github.com/Expensify/Expensify.cash/pull/3551#pullrequestreview-687812815.

@tgolen Please take a look at it, it's a minor change that was left mistakenly.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging in version: 1.0.73-4🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production in version: 1.0.74-0🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Changed behavior for note_01

Description
In between 2022-02 -24 and 2022-04-07 schematron-validation ddb_validierung_mets-mods-ap-digitalisierte-medien.sch changed the way to validate mods:note.
Formerly, a subtree of
          <mods:location>
            <mods:holdingSimple>
              <mods:copyInformation>
                <mods:shelfLocator>Pon Yb 291, 4° (2. Ex.)</mods:shelfLocator>
                <mods:note>digitized copy</mods:note>
              </mods:copyInformation>
            </mods:holdingSimple>
          </mods:location>

resulted into note_01errors, because of the note missing attribute type, which is not the case with the recent version.
The change both affects note_01 and note_02, but I don't have a test-case for note_02 at hand.
Is this really desired behavior?

Regarding the change log entry from 2022-04-01, in my interpretation, the aim was to precise the xPath expression, not to change behavior.

Yes, the DDB only supports mods:note  as top level element or as child of mods:physicalDescription (we don't process other mods:note).  So limiting note_01 and note_02 to [ancestor::mods:mods] was unprecise.
mods:copyInformation is hardly used in the records we get so we just recently noticed this problem.


Please install clang or check configuration clang.executable

I got this error message after I updated the latest vscode.
Then I can see it all the time even I close it for a thousand times.
I think I have installed all the plugin I need.
How do I fix it or disable it?
I can't disable it by commenting that part of codes.
It is really annoying.

I have reported to vscode-cpptools and they tell me to come here.
Thanks!

The clang-format extension does not automatically install the formatter, it assumes that you have it installed already.
As sean-mcmanus said in https://github.com/Microsoft/vscode-cpptools/issues/647, the best way to achieve this, is to download the LLVM compiler at http://releases.llvm.org/download.html and to put the install location to your environment variable "Path". If you prefer to not have LLVM in your Path variable, you could instead point "clang-format.executable" in your settings to the exact location of the executable (clang-format.exe on Windows).
Or if you have https://marketplace.visualstudio.com/items?itemName=ms-vscode.cpptools installed anyway, you could also switch to using their bundled version of clang-format.
I hope this helps, feel free to reopen the issue, if there are still any questions.

I think my "clang-format.executable" points to the correct thing. And I have alreadey installed https://marketplace.visualstudio.com/items?itemName=ms-vscode.cpptools , but that issue still exists. Actually that problem doesn't occur every time I use it. I just open a cpp file but that problem disappeared temporarily. But when it arises, it becomes really annoying if I want to switch tags.

Does formatting work on your C++ files?
What happens if you type clang-format in your terminal? Does it find an executable?
Do you use https://marketplace.visualstudio.com/items?itemName=ms-vscode.cpptools for formatting? (i.e. what's your settings for "C_Cpp.formatting"?)
What am I supposed to see on that second screenshot besides an overview of the default settings?

Format works for me. And I have enabled https://marketplace.visualstudio.com/items?itemName=ms-vscode.cpptools .
You said I could point "clang-format.executable" in settings to the exact location of the executable (clang-format.exe on Windows). And the line of the screenshot shows it is points to the clang format.
And I found the clang-format executable.

hi , download  llvm from http://releases.llvm.org/download.html install and check that make path for all users then ran vscode as administrator it work for me now

@SeakyLuo This extension and https://marketplace.visualstudio.com/items?itemName=ms-vscode.cpptools are not requirements for each other. If anything they are somewhat conflicting. That is, you cannot run both formatters at the same time. Basically if formatting works for you and you have not set the path for clang-format for this extension, you are already using https://marketplace.visualstudio.com/items?itemName=ms-vscode.cpptools and you do not need this extension at all. You could try to disable https://marketplace.visualstudio.com/items?itemName=xaver.clang-format and see if formatting still works for you.
@liliasouma running the editor as administrator should not be necessary and should be avoided. Does your user account have access rights for the directory you installed clang-format into? And have you tried to manually set "clang-format.executable": "D:\\Program Files\\LLVM\\bin\\clang-format.exe"?


mv rewrite failed for join with limit

Steps to reproduce the behavior (Required)

CREATE TABLE
CREATE TABLE aggregate_table_with_null (
k1 date NULL COMMENT "",
k2 datetime NULL COMMENT "",
k3 char(20) NULL COMMENT "",
k4 varchar(20) NULL COMMENT "",
k5 boolean NULL COMMENT "",
v1 bigint(20) SUM NULL COMMENT "",
v2 bigint(20) SUM NULL COMMENT "",
v3 bigint(20) SUM NULL COMMENT "",
v4 bigint(20) MAX NULL COMMENT "",
v5 largeint(40) MAX NULL COMMENT "",
v6 float MIN NULL COMMENT "",
v7 double MIN NULL COMMENT "",
v8 decimal128(38, 9) SUM NULL COMMENT ""
) ENGINE=OLAP
AGGREGATE KEY(k1, k2, k3, k4, k5)
COMMENT "OLAP"
DISTRIBUTED BY HASH(k1, k2, k3, k4, k5) BUCKETS 3
PROPERTIES (
"replication_num" = "3",
"in_memory" = "false",
"storage_format" = "V2",
"enable_persistent_index" = "false",
"replicated_storage" = "true",
"compression" = "LZ4"
);

CREATE TABLE duplicate_table_with_null_partition (
k1 date NULL COMMENT "",
k2 datetime NULL COMMENT "",
k3 char(20) NULL COMMENT "",
k4 varchar(20) NULL COMMENT "",
k5 boolean NULL COMMENT "",
k6 tinyint(4) NULL COMMENT "",
k7 smallint(6) NULL COMMENT "",
k8 int(11) NULL COMMENT "",
k9 bigint(20) NULL COMMENT "",
k10 largeint(40) NULL COMMENT "",
k11 float NULL COMMENT "",
k12 double NULL COMMENT "",
k13 decimal128(27, 9) NULL COMMENT ""
) ENGINE=OLAP
DUPLICATE KEY(k1, k2, k3, k4, k5)
COMMENT "OLAP"
PARTITION BY RANGE(k1)
(PARTITION p202006 VALUES [("0000-01-01"), ("2020-07-01")),
PARTITION p202007 VALUES [("2020-07-01"), ("2020-08-01")),
PARTITION p202008 VALUES [("2020-08-01"), ("2020-09-01")))
DISTRIBUTED BY HASH(k1, k2, k3, k4, k5) BUCKETS 3
PROPERTIES (
"replication_num" = "3",
"in_memory" = "false",
"storage_format" = "V2",
"enable_persistent_index" = "false",
"replicated_storage" = "true",
"compression" = "LZ4"
);
CREATE MATERIALIZED VIEW mv19
COMMENT "MATERIALIZED_VIEW"
DISTRIBUTED BY HASH(k1) BUCKETS 10
REFRESH ASYNC
PROPERTIES (
"replication_num" = "3",
"replicated_storage" = "true",
"storage_medium" = "HDD"
)
AS SELECT t1.k1, t2.k2
FROM duplicate_table_with_null_partition AS t1 INNER JOIN aggregate_table_with_null AS t2 ON t1.k1 = t2.k1;

INSERT INTO data
query
explain SELECT t1.k1 k1, t2.k2 k2 from duplicate_table_with_null_partition t1 join aggregate_table_with_null t2 on t1.k1 = t2.k1 limit 10

Expected behavior (Required)
the query is rewritten by mv19
Real behavior (Required)
the query is not rewritten by mv10
StarRocks version (Required)

You can get the StarRocks version by executing SQL select current_version()

should analyze base table before query to collect statistics of base tables.


Add a track name parameter

Since a track name was fixed to "production", a parameter was added to allow the track name to be changed.

Looks good to me. Have you tried it in a real project?

Thank you for your review!

Have you tried it in a real project?

not yet.
I can't try it right now, but will try it soon.


Fix build failure due to two PRs racing

SwiftSyntax failed to build because https://github.com/apple/swift-syntax/pull/539 and https://github.com/apple/swift-syntax/pull/578 conflicted

@swift-ci Please test


Add gate queuing and commuting for faster simulation

This change updates the sparse state simulation to incorporate queuing of H, Rx, and Ry gates along with some logic for commuting operations through these gates. This allows these three operations, which can change the size of the state vector and reduce sparsity, to be delayed and sometimes skipped. While the logic for queuing introduces some overhead to each gate operation, the gain in speed and memory footprint for certain common gate sequences involving H, Rx, and Ry can be very significant. For example, the individual gate benchmarks for other gates showed about an 8% regression (from 3.4 to 3.6 microseconds) while the three queued gates showed a nearly 70% improvement in performance (from 17 to 5 microseconds), and even the small Grover's Search benchmark showed more than 20% improvement (from 17 to 13 microseconds). Improvements are more pronounced for larger algorithms that primarily operate on a subset of the qubits.
This change includes an update to the testing logic to interleave queued operations between the operations under test in order to verify equivalence of a queued and commuted operation with its non-queued, non-commuted adjoint.
This change also adds top level documentation for the qir_backend crate that points to the paper published on sparse state simulation, https://arxiv.org/abs/2105.01533.

Of note about the updated tests, they already proved useful by identifying a bug in the mcphase operation used to back a controlled Pauli-I rotation where the I forgot to flush the queue before performing the operation.


View service/task logs

Just discovered this tool and it looks awesome!
I'd like to view the logs of a service/task, just like in k9s 😄
Maybe some like Cloudwatch Logs Live Tail can be used for this?
AWS CLI ref: https://awscli.amazonaws.com/v2/documentation/api/latest/reference/logs/tail.html

Was looking into this a bit more, but this would really only be possible if the LogDriver is awslogs. So the logic would need to:

check if the LogDriver is set as awslogs, otherwise we cannot provide logs.
get the awslogs-group and awslogs-stream-prefix from the options object
Open a livetail stream with these options

Asked GPT for the logic in Bash, for reference: https://chat.openai.com/share/28d42258-2285-4462-9ee3-1aadb05f6d91

@tmeijn
Hi, thank you for your investigation. I added in v1.0.11.
Could you check does it fits your needs? 😃

Hey @keidarcy, thanks so much for getting this in, this looks already really awesome!
I think I did discover a bug: as you can see in the screenshot below, the logs repeat themself one time. Not a big one, but maybe there is some double fetching going on? 🤔

A suggestion: since this tool looks a bit like k9s, maybe an idea to keep parity between them in regards to the shortcuts used? Now the logs function is accessed with o, while in k9s it is l. In e1s this shortcut is undocumented, but does seem to have the same function as <enter>.

A possible improvement: would it be easy to make the new reload functionality also work in the log view?
Again, thanks so much for putting this in ❤ .

Hi, @tmeijn . Thank you for using e1s.

double health check logs

You can see there are different IPs. They are from different containers. I guess your task definition has multiple containers healthcheck setup.

keybind

I want to keep vim keybind to move back and forth. If I hear more people think the same, I may change.

reload in log view

I'm considering to add it.

@keidarcy

You can see there are different IPs. They are from different containers. I guess your task definition has multiple containers healthcheck setup.

Sorry, should've been more clear, I mean the whole log repeats: if you look at the screenshot again you can see two times no-op change. If you look at the timestamps you can see they are exactly the same and if you both following them down you see that the log lines are repeating themselves with the same timestamp.

Hey @keidarcy

thinking instead of fetching by API just show aws logs tail output

Yeah that was my original proposal to use the aws logs tail equivalent in the SDK. If this is workable it would the same experience as k9s where the logs also stream in live.

But it doesn't support multiple log groups.

e1s provides logging and the task level and individual container level right? So the only time when some multiple log groups would be needed is if someone decides to put multiple container definitions into a task AND decides to assign these containers different log groups. This does not seem logical to me and would IMHO justify to just not support this.

Hi @tmeijn I added r in log view for realtime log in v1.0.14 😄

Haha, you just decided to support both, really awesome! 💪🏾
There really isn't a place to say this so I'm doing it here: I'm planning a migration off Kubernetes to ECS and this tool being available as an "equivalent" (not yet, but pretty close already!) to k9s was one of the deciding factors as we use k9s a lot. I also made sure you got a shout out at Cloud Posse Office Hours. Really appreciate the work you've done here.
Alright, that's it, if I have any other issues I'll open a new one 😄 .

Hey, @tmeijn! Thanks a lot. I guess that's why it's getting some attention 😂
I'm glad that this project can help you a bit.


Reproduction Issue

Hi, thanks for releasing the code, it's well-organized and easy to run!
However, when I tried to reproduce the result reported in paper for Huffpost and FewRel datasets by running bin/our.sh, the result seems to differ, and cannot be simply explained by the randomness. I ran each experiments for 5 random seeds, for the test acc I got:
Huffpost: 1-shot 41.9(0.72), 5-shot 62.8(0.27).
FewRel: 1-shot 64.5(0.39), 5-shot 82.5(0.50).
All were lower than the reported ones from 1~3%.
Thanks for any help in advance.

Thank you for your interest in our work!
I modified the code to fit the latest PyTorch and Python version this morning and got similar results as yours when running it on our local machine. Moreover, the results for the baselines (attached below) are also lower than the number reported in Table 1. The main conclusion of our paper still holds.
The experiments in Table 1 were conducted 3 years ago on IBM cloud servers. I suspect the performance variation is due to the infrastructure / library difference. Please let me know if there is anything I can help.

Rep.
Alg.
HuffPost (1 shot)
FewRel (1 shot)

Avg
Proto
32.95
40.9

Idf
Proto
32.79
43.2

CNN
Proto
30.63
45.84

Avg
RR
33.33
52.09

Idf
RR
34.15
55.44

CNN
RR
35.07
55.84

Thanks for the quick response! I actually ran the code with the same library version you provided in README.md though, so it probably wasn't the issue.
If it's possible, would you provide any result on your side with the same library version you provided? Thanks.


Nullability bug in _network_image.io.dart

Steps to Reproduce

The following code fails before throwing exception!
if (response.statusCode != HttpStatus.ok) {
// The network may be only temporarily unavailable, or the file will be
// added on the server later. Avoid having future calls to resolve
// fail to check the network again.
await response.drain<List<int>>();
throw image_provider.NetworkImageLoadException(statusCode: response.statusCode, uri: resolved);
}

Expected results:
This should be invoked:
throw image_provider.NetworkImageLoadException(statusCode: response.statusCode, uri: resolved);
Actual results:
I/flutter (18983): The following _CastError was thrown resolving an image codec:
I/flutter (18983): type 'Null' is not a subtype of type 'List<int>' in type cast
I/flutter (18983):
I/flutter (18983): When the exception was thrown, this was the stack:
I/flutter (18983): #1      NetworkImage._loadAsync (package:flutter/src/painting/_network_image_io.dart:99:24)
I/flutter (18983): <asynchronous suspension>

Logs

[✓] Flutter (Channel stable, 2.2.3, on macOS 11.5.2 20G95 darwin-x64, locale en-FI)
    • Flutter version 2.2.3 at /Users/egaga/dev/flutter
    • Framework revision f4abaa0735 (8 weeks ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

Hi @egaga

I/flutter (18983): The following _CastError was thrown resolving an image codec:
I/flutter (18983): type 'Null' is not a subtype of type 'List' in type cast

Can you please provide a reproducible sample for this?

Not yet, not sure if have time to minimize, but I can give you a little bit context:
Image.network has https url to Amazon S3 bucket.
Then the image is removed from the Amazon and the Flutter app screen gets rebuild/re-rendered this happens.
I know it would be important to provide an example, and I am sorry that I haven't.

Here is a working example, requires the template produced by flutter create bug.
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Image Network error bug',
      home: _ImageNetworkBuggy(),
    );
  }
}

class _ImageNetworkBuggy extends StatelessWidget {

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: Image.network("https://upload.wikimedia.org/wikipedia/commons/c/c4/DNA_double_helix_horizontal2.png")
      ),
    );
  }
}

@egaga
Both latest stable and master don't throw mull safety error using your code sample
[        ] For a more detailed help message, press "h". To quit, press "q".
[   +1 ms] An Observatory debugger and profiler on Chrome is available at: http://<IP_ADDRESS>:49342/uKos-S7cet8=
[ +463 ms] DevTools activation throttled until 2021-09-18 00:20:49.454218.
[ +583 ms] The Flutter DevTools debugger and profiler on Chrome is available at:
                    http://<IP_ADDRESS>:9101?uri=http://<IP_ADDRESS>:49342/uKos-S7cet8=
[  +38 ms] ══╡ EXCEPTION CAUGHT BY IMAGE RESOURCE SERVICE ╞════════════════════════════════════════════════════
[   +2 ms] The following ImageCodecException was thrown resolving an image codec:
[   +1 ms] Failed to load network image.
[   +2 ms] Image URL: https://upload.wikimedia.org/wikipedia/commons/c/c4/DNA_double_helix_horizontal2.png
[   +1 ms] Server response code: 404
[   +2 ms] When the exception was thrown, this was the stack
[   +1 ms] Image provider:
[   +1 ms]   NetworkImage("https://upload.wikimedia.org/wikipedia/commons/c/c4/DNA_double_helix_horizontal2.png",
[        ]   scale: 1)
[        ] Image key:
[        ]   NetworkImage("https://upload.wikimedia.org/wikipedia/commons/c/c4/DNA_double_helix_horizontal2.png",
[        ]   scale: 1)
[        ] ════════════════════════════════════════════════════════════════════════════════════════════════════

@TahaTesser That's good. I have upgraded Flutter a week ago to 2.5.<something>.
I will try next week again to ensure I had correct version of stable and if I get still that error, I'll send you current flutter doctor output.

I upgraded to flutter 2.5.1 and still get the same error on iOS:
======== Exception caught by image resource service ================================================
The following _CastError was thrown resolving an image codec:
type 'Null' is not a subtype of type 'List<int>' in type cast

When the exception was thrown, this was the stack:
#1      NetworkImage._loadAsync (package:flutter/src/painting/_network_image_io.dart:99:24)
<asynchronous suspension>
(elided one frame from dart:async)
Image provider: NetworkImage("https://upload.wikimedia.org/wikipedia/commons/c/c4/DNA_double_helix_horizontal2.png", scale: 1.0)
Image key: NetworkImage("https://upload.wikimedia.org/wikipedia/commons/c/c4/DNA_double_helix_horizontal2.png", scal

flutter doctor -v (with IDE and Android specific stuff omitted):
[✓] Flutter (Channel stable, 2.5.1, on macOS 11.5.2 20G95 darwin-x64, locale en-FI)
    • Flutter version 2.5.1 at /Users/egaga/dev/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ffb2ecea52 (3 days ago), 2021-09-17 15:26:33 -0400
    • Engine revision b3af521a05
    • Dart version 2.14.2

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5.1, Build version 12E507
    • CocoaPods version 1.10.1

[✓] Connected device (3 available)
    • iPhone (mobile) • 00008030-000E38992E9A402E • ios            • iOS 14.7.1 18G82

If I edit the flutter code by commenting out like this
//await response.drain<List<int>>();

Then I get the correct exception.
======== Exception caught by image resource service ================================================
The following NetworkImageLoadException was thrown resolving an image codec:
HTTP request failed, statusCode: 404, https://upload.wikimedia.org/wikipedia/commons/c/c4/DNA_double_helix_horizontal2.png

When the exception was thrown, this was the stack:
#0      NetworkImage._loadAsync (package:flutter/src/painting/_network_image_io.dart:100:9)
<asynchronous suspension>
Image provider: NetworkImage("https://upload.wikimedia.org/wikipedia/commons/c/c4/DNA_double_helix_horizontal2.png", scale: 1.0)
Image key: NetworkImage("https://upload.wikimedia.org/wikipedia/commons/c/c4/DNA_double_helix_horizontal2.png", scale: 1.0)
====================================================================================================

Or if I fix the code as the error indicates, like this:
await response.drain<List<int>?>();

It works, too.

Just see the code in master:
https://github.com/flutter/flutter/blob/83dfb2237a22ad92c38e345b3aa85aeecae732aa/packages/flutter/lib/src/painting/_network_image_io.dart#L99
https://github.com/dart-lang/sdk/blob/d45095c4d717fecf5624c07cc0d90656e7e97115/sdk/lib/async/stream.dart#L1091
The documentation for latter file (API) even says:

The [futureValue] must not be omitted if null is not assignable to [E].

And your call-site code has E = List<int> and called with null so you are not conforming to the API requirements AFAIK. Sorry if I'm mistaken.

@egaga You are not alone with this issue. In our Crashlytics we have:

This issue has 955 non-fatal events affecting 218 users

Non-fatal Exception: io.flutter.plugins.firebase.crashlytics.FlutterError: type 'Null' is not a subtype of type 'List' in type cast. Error thrown Instance of 'ErrorDescription'.
at Stream.drain(Stream.java)
at NetworkImage._loadAsync(NetworkImage.java:99)

I haven't had the time to debug it like you, so props to you 👍

I have same bug

Hi @egaga, I was going through the list of issues randomly and found this, upon looking, it looks likes this issue has been fixed by https://github.com/flutter/flutter/pull/89021.
cc/ @TahaTesser

Hey @amanv8060 , it seems the code in master is fixed in different manner:
https://github.com/flutter/flutter/blob/8017883a5dfd13d709c4bd04d617f54a6cb2271f/packages/flutter/lib/src/painting/_network_image_io.dart#L97


Error: exited with null when starting shared process on latest release version

code-server version: 1.408-vsc1.32.0
OS Version: Ubuntu 18.04 x64 , DigitalOcean

Description

I cannot run the command code-server myApp. It throws this error

I followed two tutorials

https://github.com/codercom/code-server/blob/master/doc/admin/install/digitalocean.md
https://www.youtube.com/watch?v=9Emn2YQNDl0

Steps to Reproduce

wget https://github.com/codercom/code-server/releases/download/1.408-vsc1.32.0/code-server1.408-vsc1.32.0-linux-x64.tar.gz
tar -xvzf code-server1.408-vsc1.32.0-linux-x64.tar.gz
cd code-server1.408-vsc1.32.0-linux-x64/
sudo chmod +x code-server
sudo mv code-server /bin/
cd ~/
code-server myApp
Error Appears

UPDATE: attempted same steps but with an older version and got the same error

This error is a issue with the amount of RAM available on your system. Older versions will be worse, so I really only recommend the latest. Do you know how much RAM the droplet has?

1GB RAM

I think we recommend atleast 2GB.

cool it worked


VS code features were not working after a compiler error

Description:
I am working on testing bindgen tool. For that I am trying to generate the bal code for this java code.
I have run these bindgen command to get all dependencies.
bal bindgen java.io.BufferedInputStream java.io.FileInputStream java.io.InputStream java.util.Map java.util.zip.ZipEntry java.util.zip.ZipInputStream
Then I write below bal code, and every thing was working fine.
public function main(string... args) {
    string fileName = args[0];
    FileInputStream|FileNotFoundException fileInputStream = newFileInputStream3(fileName);
    if fileInputStream is FileNotFoundException {
        io:print(fileInputStream.message());
    } else {
        BufferedInputStream bufInputStream =  newBufferedInputStream1(fileInputStream);
        ZipInputStream zipInputStream = newZipInputStream1(bufInputStream); // casting error
    }
}

At this line, ZipInputStream zipInputStream = newZipInputStream1(bufInputStream);, there is a casting issue and I experience following issues.

It suggests adding type cast, but it adds the cast to the wrong place.
After some time, all the features which were working previously, are stuck and got the below error log VS code output

java.util.NoSuchElementException: No value present
	at java.base/java.util.Optional.orElseThrow(Optional.java:382)
	at org.ballerinalang.langserver.util.definition.DefinitionUtil.getLocation(DefinitionUtil.java:82)
	at org.ballerinalang.langserver.util.definition.DefinitionUtil.getDefinition(DefinitionUtil.java:61)
	at org.ballerinalang.langserver.BallerinaTextDocumentService.lambda$definition$3(BallerinaTextDocumentService.java:257)
	at java.base/java.util.concurrent.CompletableFuture$AsyncSupply.run(CompletableFuture.java:1700)
	at java.base/java.util.concurrent.CompletableFuture$AsyncSupply.exec(CompletableFuture.java:1692)
	at java.base/java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:290)
	at java.base/java.util.concurrent.ForkJoinPool$WorkQueue.topLevelExec(ForkJoinPool.java:1020)
	at java.base/java.util.concurrent.ForkJoinPool.scan(ForkJoinPool.java:1656)
	at java.base/java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1594)
	at java.base/java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:183)

Affected Versions:
slpalpha3

import java.io.BufferedInputStream;
import java.io.FileInputStream;
import java.io.IOException;
import java.util.zip.ZipEntry;
import java.util.zip.ZipInputStream;

public class JavaReadZip {

    public static void main(String[] args) throws IOException {

        String fileName = "src/resources/myfile.zip";

        try (FileInputStream fis = new FileInputStream(fileName);
                BufferedInputStream bis = new BufferedInputStream(fis);
                ZipInputStream zis = new ZipInputStream(bis)) {

            ZipEntry ze;

            while ((ze = zis.getNextEntry()) != null) {

            }
        }
    }
}

Do you get Stack Overflow error or OOM error as well?

The above issues are no longer reproducible on master. Closing.


fix(docs): add editURL.enable = true

The MR documents the requirement to enable editURL setting.
A  prior commit the behavior changed from a default of "disable = false" to a default of "enabled = false".

@ytjohn thank you for catching this!


bug: Incorrect IR emitted in generated MokaIR.

Correct me if I'm completely wrong, but looking at the MokaIR example here, around PC #002C:
%40 = int(0)
nop
nop
nop
if %arg1 >= Phi(%40, %64) goto #0046
...

Which is the condition for the for loop at L18 in the TestAnalysis.java source here:
for (int i = 0; i < y; i++) {
  ...
}

Will always evaluate to true when %arg1/y is passed a value over 0, jumping to #0046 (which breaks out of the for loop).
I am not sure if this is a misunderstanding on my part with the use of Phi; however, looking at this from my perspective, it still looks like the arguments for the conditional have been reversed.

Thanks for your interest in this project.
Phi(%40, %64) is the $\phi$ function in the Static single assignment form. In this example, it is not simply %40 or %64, but depending on the actual execution. At the first execution of #002C, it goes through the path #0028 -> #0029 -> .. -> #002C then the value of Phi(%40, %64) will be %40. At the second execution of #002C, it goes through the path #0040 -> #0043 -> #002A -> #002B -> #002C, and the value of Phi(%40, %64) will be %64.
#0028: iconst_0         => %40 = int(0)
#0029: istore_3         => nop
#002A: iload_3          => nop
#002B: iload_2          => nop
#002C: if_icmpge        => if %arg1 >= Phi(%40, %64) goto #0046
...
#0040: iinc             => %64 = Phi(%40, %64) + 1
#0043: goto             => goto #002A

This project is great by the way! I'm using it in two of my own, this is just the first time attempting to use MokaIR. :smile:
So how is the line #002C evaluated at all? Taking the first first execution where Phi(%40, %64) is %40, collapsing the line to:
if %arg1 >= %40 goto #0046

And since %40 is set to int(0), surely the expression is true (assuming %arg1/y is above 0)?
if <some value above 0> >= 0 goto #0046

And then, as you say during the second execution, the value of Phi(%40, %64) is %64, collapsing the line to:
if %arg1 >= %64 goto #0046

Which is declared at #0040 (which I assume to be 1, since first execution will use %40 + 1 - therefore 0 + 1), but then will still break out of the block if %arg1/y is above 1.
So in my head, passing say 5 to %arg1/y, will break out of the loop on first evaluation of #002C, since Phi would collapse to %40, but say even if the block is executed, and then %64 is decalred, the next evaluation of #002C as you say will compare 5 against 1, which will break out again?
Forgive me if I'm wrong again haha, I just can't figure out in this case how that line ever enters the loop block. :confused:

You are correct. I found that it is a bug in my implementation when converting the stack based JVM byte code to register based IR. The bug will make all the binary operations have their operands swapped.
I will fix it soon. Thanks for bringing this into my attention.


Suggestion: Allow site list to be passed in via stdin

Thanks for making this plugin - it looks neat! As a user attached to multiple organizations and access to hundreds of sites looping through all sites I have access to takes a large amount of time.
It would be great to be able to send the results of terminus site:list or terminus org:site:list to cantilever via stdin. The Terminus mass update plugin does this and it is really useful.
For example, I can list sites from a specific org and filter by tag with terminus org:site:list --tag. That takes much less time and then cantilever could just loop through the sites I provide.

@ataylorme that's what this does.
You don't need a way to send the results, this is an extension of the list plugin and does that for you based on the tags and framework you provide.
Then it will print a list of each of the site names, unless you specify a command, in which case it will run the command on each site.
The next version will have tags from the orgs available.
Watch the video linked in the readme or give it a try to see what I mean.

@ataylorme This has moved over to https://github.com/terminus-plugin-project/terminus-cantilever-plugin
I did want to note, that you can currently filter by org with the --org=orgname variable too.
The next version is going to add an extension to org:site:list that will also use the org info to find sites that are tagged on the organization side. So when --tag=sometag is used, it will use any org data from your user account to match the sites, as well as the data for tags from the site side that it currently looks through.


RuntimeError: await wasn't used with future when using pysnmp.hlapi.asyncio.cmdgen

The pysnmp.hlapi.asyncio.cmdgen functions appear to be broken, as this example demonstrates:
import sys
import asyncio
from pysnmp.hlapi.asyncio import *

ipaddr, community = sys.argv[1:]

snmpEngine = SnmpEngine()

async def doit(varBinds):
    (errorIndication,
     errorStatus,
     errorIndex,
     varBindTable) = await bulkCmd(
        snmpEngine,
        CommunityData(community, mpModel=1),
        UdpTransportTarget((ipaddr, 161)),
        ContextData(),
        0, 10,
        *varBinds)

    if errorIndication:
        print(errorIndication)
    elif errorStatus:
        print('%s at %s' % (
            errorStatus.prettyPrint(),
            errorIndex and varBinds[int(errorIndex) - 1][0] or '?'
        )
              )
    else:
        for varBindRow in varBindTable:
            for varBind in varBindRow:
                print(' = '.join([x.prettyPrint() for x in varBind]))

asyncio.run(
    doit([
        ObjectType(ObjectIdentity('SNMPv2-MIB', 'system')),
    ])
)

$ python snmp_async_bulk.py <IP_ADDRESS> public
Traceback (most recent call last):
  File "snmp_async_bulk.py", line 37, in <module>
    loop.run_until_complete(
  File "/usr/lib/python3.11/asyncio/base_events.py", line 653, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "snmp_async_bulk.py", line 10, in run
    (errorIndication,
RuntimeError: await wasn't used with future

I believe the issue arises because the cmdgen functions return futures and consequently the use of the async keyword is inappropriate, as this turns them into coroutines. Alternatively, the functions could be changed to return await future, but this seems an unnecessary elaboration.

You might switch to the new pysnmp.hlapi.asyncio.slim wrapper,
examples/hlapi/asyncio/manager/cmdgen/v2c-bulk.py

Hello,
Actually there is something wrong with @asyncio.coroutine patches to async def, that lead to an API change.
Here is what I have to do now to get it work on both version:
result = await getCmd(
    engine,
    auth_data,
    transport_target,
    context_data,
    *targets,
)
if isawaitable(result):
    error_indication, error_status, error_index, variables = await result
else:
    error_indication, error_status, error_index, variables = result

This is definitely a bug.  All code from this module have the same broken pattern, it uses return future while it should use return await future.
I'll check and provided a PR.

Confirming this fixes my issue. Thanks folks.

@spdkils If you only use v1/v2c, the slim API is more suitable and remains unchanged. For v3 operations, unfortunately the changes in 5.0.29 do break the code.

It's fine, i'll just pin my previous versions to 0.28, and when I update, I'll push everything to 0.29+
Yes I'm on v3.


Add to_string !eval tag

Adds a to_string function for !eval tag.
Goal of this addition is shown by the new test. To achieve this, a support for lists had to be added. This was done by using EvalWithCompoundTypes object instead of the SimpleEval object.
[[1.0, 1.0], [2.0, 2.0]] -> '[[1.0, 1.0], [2.0, 2.0]]'

LGTM! I see that we need to have this feature. It would be good to also add a mention of this feature to the main README and a short explanation why this is needed in addition to just normally casting to string with quotation marks.

Had forgotten to add to_string to README before. Added now :+1:


Throwing std::bad_cast in the guest is not working well

Maybe it's time to find another way to both inform and error out, eg. a custom assertion syscall.

This was solved with https://github.com/libriscv/godot-sandbox/commit/3fef323ba1f8fe4b6d70f24824a59ee0a1198a13


Add umbrella backend

also adds tarball backend

New pull request has been created which covers these changes in addition to recent requests.


TypeError: promise.timeout is not a function

Describe the bug
Using an onWallPost event causes an TypeError: promise.timeout is not a function error, I am running node.js v14.5.0 if thats causing it.
To Reproduce
Steps to reproduce the behavior:

Run this code
`const noblox = require('noblox.js');
const fliters = [{name:"example",regex:/.example./}]
const groupid = "[REDACTED]"

async function run() {
await noblox.setCookie('[REDACTED]');
var onWallPost = noblox.onWallPost(groupid);
onWallPost.on('data', async function(post) {
console.log(post.content)
fliters.forEach(async function(item){
if (item.regex.test(post.content)) {
noblox.deleteWallPost(groupid, post.id)
await noblox.follow(post.author.id)
await noblox.message(post.author.id,Regarding your recent post on the [REDACTED] group wall','Hey there! Sorry to let you know but your recent post on the [REDACTED] Group wall has been automatically deleted due to being catched by one of are filters. If you feel this is in error please let [REDACTED] know with all the info below.\n Post infomation \n Deleted post ID: ${post.id} \n Filter name: ${item.name})
await noblox.unfollow(post.author.id)
}
})
})
onWallPost.on('error', function (err) {
console.error(err.stack);
});
}
run();`
Expected behavior
That it would run as normally

This is because Bluebird isn't imported - I think several of the events have this issue.
I'll open a PR to resolve this & the others later today.