Add files via upload

This is the game of Random dice rolling where you can carry Sprig as a dice in your pocket and Play Game of  Dares. In this game you have to prepare a bowl of Dares and write numbers on the chits , now you can use you Sprig and roll the dice to give each other the dares and have fun.

This game is about rolling the dice and playing the game of dares with your friends , you basically prepare the bowl of chits with dares written on them, you also have to write a no. on that chit upto 6. Now get you Sprig roll the dice and enjoy with your friends.
How do you play your game?
So basically what you have to do is upload this game to you Sprig console and after that when you are sitting with your friends , each on of you have to press 'S' Button this will show a no of dots on the screen based on the no. of dots you have to give each other the dares
Code
Check off the items that are true.

[x] The game was made using the Sprig editor.

[x] The game is placed in the in the /games directory.

[x] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").

[x] The game runs without errors.

[x] The name of the file/game contains only alphanumeric characters, -s, or _s.

[x] The game name is not the same as the others from gallery

[x] The image is in the /games/img directory.

[ ] The name of the image matches the name of your file.

Thanks for your PR!

Please do check you my game Me and my friends have really worked on this project and also want to run this game on you professional console specially made for it

Please do check you my game Me and my friends have really worked on this project and also want to run this game on you professional console specially made for it

We check all PRs:)

My friends Ansh, Atman and Soumay asked me to contribute as a co-leader in their hack club named "Webventures" where we started our first project with sprig and spread awareness at our school jiva public school. We gave a lecture on how to build game in editor for sprig on 10th june 2023, so a curiosity sparked in back of my mind that i want a sprig

Nice! Out of curiosity, did you use any game for inspiration?

Hey can you please elaborate your question like what do you mean by "inspiration"

so that I can revert you in best way possible

Hey Lucas! I hope you are doing good Please reply to my question

I thought i already answered, sorry. I mean used as a base

No I have not used any game for inspiration but yes my co-leaders while introducing others about the hackclub in our school , we did use some of the games made by us to make more students interested in this program and make them join our club

@LucasHT22 is this PR not passing the originality test or ?

Hey Lucas! I edited the file kindly check

What about add 2-3 words per addText? :) To avoid this:

do check now , is it fine ?

I think it's better to keep the cases empty to who is playing complete... the actual isnt appropriate... if you understand me

Sorry but i am not able to understand , just tell me what i need to do now

I'll be direct, some of these are not appropriate, especially d5, please change

do check now

Right:) Out of curiosity, some friend helped you with code?

nope no one really helped me in making my game but yes I did consulted my teacher who knew about JavaScript


Full height image inside StepOne.

I have a small problem with using Image as a background. So like it goes i have Index:
<View style={styles.container}>      <MultiStep steps={steps} onFinish={this.finish}/> </View>
container: {   	flex: 1,   },
And later StepOne:
<Image style={styles.container} source={require('../../img/signup.png')} >         <Text style={styles.welcome}>           Welcome to React Native!         </Text>         <Text style={styles.instructions}>           To get started, edit index.ios.js         </Text>         <Text style={styles.instructions}>           Press Cmd+R to reload,{'\n'}           Cmd+D or shake for dev menu         </Text>        </Image>
container: {     flex: 1,     width: undefined,     height: undefined,     justifyContent: 'center',     alignItems: 'center',   },   welcome: {     fontSize: 20,     textAlign: 'center',     margin: 10,   },   instructions: {     textAlign: 'center',     color: '#333333',     marginBottom: 5,   },
And the problem is when StepOne is loaded normaly  is full width/height, but when goes by  container takes height only for content (those 4 lines of text).
Realy dunno how to fix it by styles.

Please share all the steps for me to replicate.


No Yahoo support

does it support yahoo?

Does Yahoo have an Oauth API?

https://developer.yahoo.com/oauth/

Adding this for reference:
https://developer.yahoo.com/oauth2/guide/flows_implicitgrant/
When time permits it will be added.

Created PR for this https://github.com/nraboy/ng-cordova-oauth/pull/257

PR updated

This has been merged to development.  It will make its way into the master branch with a version soon.


Let an Edition mark itself as being in history-mode

Adds historic?, based on agreed rule that something should be in
history-mode if it is political and does not belong to the current
Government.
This adds current? on Government and uses Edition.political?,
which already exists. Note,political? isn't fully populated yet.
This was written for https://github.com/alphagov/whitehall/pull/2014 but which only needed Edition.political?
in the end. I've split it out as it'll still be used in future work (like https://trello.com/c/lowbsjJK/80-spike-history-mode-on-document-pages), so getting it reviewed early.

:+1: LGTM


use layer cache when building images

to more closely mimic docker default behavior, the --layers
cli option is set to true by default for podman.  the buildah
environment variable of BUILDAH_LAYERS is still honored and will
override the command line input.
this should be considered in place of PR #1383.
Many thanks for Scott McCarty for inspiring this welcome change.
Signed-off-by: baude<EMAIL_ADDRESS>

@rh-atomic-bot r+


"style" is a reserved attribute

Hello,
I have an issue using polygons with vue-devtools:
[Vue warn]: "style" is a reserved attribute and cannot be used as component prop"
The warning happens with Polygon, LCircle, Polyline and Rectangle components.
Thanks!

@jkermes I think the best way to go is to map a "l-style" property to leaflet style option

You can bind your style to lStyle property (l-style).
Available from version v0.0.47

Perfect! Thank you very much :)


publish token to GH Pages?

This is very neat (an abomination but a neat one) , I'm thinking of using this in my own testing as well... The only tricky part seems to be finding a token: see https://github.com/sigstore/sigstore-conformance/blob/6c958ac7c74146cb694b9bdb3f8111c3e447d6cf/test/conftest.py#L75 for an example
What if the workflow published to GitHub Pages with a static filename?  That way the newest produced token could always be available at https://sigstore-conformance.github.io/extremely-dangerous-public-oidc-beacon/cursed-token.txt. Am I missing anything?
The downloader would still have to deal with the token being too old (unsure how at this hand waving stage) but all of this would go away:

GitHub auth
workflow run json download
workflow run json parsing
unzipping

downloader would still have to deal with the token being too old (unsure how at this hand waving stage)

I think this is easiest:
token=jwt.decode(cursed_token_data[:-1], options={"verify_signature": False})
expiry=datetime.fromtimestamp(token["exp"])

should be trivial to retry if the token is not valid ( or is valid for less than 10 secs or however long we need it for)

What if the workflow published to GitHub Pages with a static filename? That way the newest produced token could always be available at https://sigstore-conformance.github.io/extremely-dangerous-public-oidc-beacon/cursed-token.txt. Am I missing anything?

This is a great idea, and should be way less flaky than the current approach!
If you have the time to work on this I'd be overjoyed to review and merge it, otherwise I'll try and find the time to do it in the near future 🙂

If you have the time to work on this I'd be overjoyed to review and merge it, otherwise I'll try and find the time to do it in the near future 🙂

It's on my list of "when I need a break from actual project, try this"... so no promises but I might. I'll leave a note here if I do anything

So... this is easy to do, I have it running in my fork.
Unfortunately

the existing artifact can't be used to publish to Pages (pages requires a zip that contains a tar that contains a single directory)
sigstore-conformance will break if I add another artifact

Plan:

patch  sigstore-conformance to not break if it sees multiple artifacts, hopefully there are no other users of this?
later merge branch that adds another artifact and GH pages publish to this project
later switch sigstore-conformance to actually use the GH pages published artifact

That plan SGTM -- I think it's safe to assume that sigstore-conformance is the only current user.


Update FAQ page

(misc) Update iOS build files
Replaced ExpansionTile with custom FAQ widget
Adjust padding sizes

LGTM, i noticed that the animation when expanding the dropdown is no longer present which was included in ExpansionTile. Is this something we want?

No, I don't think it's important. The implementation used in ExpansionTile is clunky as well.


Dev

Codecov Report

Merging #633 (edd493c) into master (de32d02) will decrease coverage by 0.01%.
The diff coverage is 66.45%.

@@            Coverage Diff             @@
##           master     #633      +/-   ##
==========================================
- Coverage   42.31%   42.30%   -0.02%
==========================================
  Files         321      329       +8
  Lines       23133    23182      +49
==========================================
+ Hits         9788     9806      +18
- Misses      13345    13376      +31

Flag
Coverage Δ

unittests
42.30% <66.45%> (-0.02%)
:arrow_down:

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

Impacted Files
Coverage Δ

dbaas/tsuru/views/serviceRemove.py
24.56% <24.56%> (ø)

dbaas/tsuru/views/getServiceStatus.py
41.37% <41.37%> (ø)

dbaas/physical/models.py
74.90% <42.85%> (-0.28%)
:arrow_down:

dbaas/tsuru/utils.py
45.00% <45.00%> (ø)

dbaas/tsuru/views/getServiceInfo.py
50.00% <50.00%> (ø)

dbaas/tsuru/views/listPlans.py
64.28% <64.28%> (ø)

dbaas/tsuru/views/serviceUnitBind.py
83.33% <83.33%> (ø)

dbaas/tsuru/views/serviceAppBind.py
84.37% <84.37%> (ø)

dbaas/logical/models.py
53.61% <90.90%> (+0.51%)
:arrow_up:

dbaas/tsuru/views/serviceAdd.py
92.61% <92.61%> (ø)

... and 11 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update de32d02...edd493c. Read the comment docs.


ERROR  Cannot convert a BigInt value to a number

Describe the bug
When entering a few usernames such as the ones listed in this report (this doesnt happen with all usernames) the following error occurs: ERROR  Cannot convert a BigInt value to a number
Profile/Location
https://sky.shiiyu.moe/stats/dreadingg https://sky.shiiyu.moe/stats/Dario_on_IOS__2
Additional Info
N/A
Expected behavior
id usually expect the site to show me the accounts stats (ive searched the names listed in the post in the past with no trouble)
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser: Brave
Version idk

Additional context
This error doent show anything (stats) at all, it isnt some error with profiles or anything. It doesnt even let you access them, you are stuck in the search bar from when you first open the website.

Ive got the same issue for "Banishedddd"

Fixed in #2156


image build issue

Hello,
I created custom image based on v1.2.2. but when I compile the image, I met such error
Configuration file '/etc/hosts'
 ==> File on system created by you or by a script.
 ==> File also in package provided by package maintainer.
   What would you like to do about it ?  Your options are:
    Y or I  : install the package maintainer's version
    N or O  : keep your currently-installed version
      D     : show the differences between the versions
      Z     : start a shell to examine the situation
 The default action is to keep your current version.
*** hosts (Y/I/N/O/D/Z) [default=N] ? dpkg: error processing package customizations-base (--configure):
 end of file on stdin at conffile prompt

The compilation host is Ubuntu 20.04, I checked the fail generated rootfs, I found /etc/hosts in already installed, but not the one created by customizations-base.
Could you help check the problem? thanks a lot.

There can't be two packages owning the same file on the installed rootfs. If you want to set /etc/hosts yourself, replace or append to customization-base in your own layer.

I don't have another package which owns /etc/hosts. It is only operated by costomizations-base package in meta-iot2050. It is fine when the base layer commit is v1.1.1. But when I switch to v1.2.2, this error occured.

Were you building 883c59cbf5be87483810bb8798df6293f49e8a55 or stable/V01.02 HEAD (79f5701104a3f177af62606d8f1f41e78cc7044d)? The latter is required unless you use the package snapshot of V01.02.02 as well (kas/opt/package-lock.yml).

I am using the commit of 883c59c, so if I build this commit, I need to use the package snapshot?

It seems this issue is similar to #262 , and is solved by #263 . Can we merge #263 to stable/V01.02?

Oh, forgot about that, indeed. Let's add it to the queue, but I'm not sure if we will have a V01.02.03.
@BaochengSu

@jan-kiszka I think it's OK to add it to the queue, but same as you I am not sure if V01.02.03 is needed, let's put it undecide.


Added PUMP_CONFIG_CONTROL_CLUSTER to controller and chip-tool

Problem
This PR is intended to add the PUMP_CONFIG_CONTROL_CLUSTER to the chip-tool, both Python and CLI
In order to be able to control and test Pump test applications, we need to have this feature in the chip-tool and the chip-device-trl tools. This PR resolves that.
Summary of Changes
Added the PUMP_CONFIG_CONTROL_CLUSTER to both the controller-clusters.zap and the chip-tool.zap files. Then regenerated the ZAP files using ./scripts/tools/zap_regen_all.py with success. Finally a few files under src/controller and src/app was manually clang-format'et to be able to ease the merge.

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

@saurabhst @msandstedt @mspang @andy31415 ?


How to remove markers?

Hi, how can I remove markers?
I can add markers via: monaco.editor.setModelMarkers(editor.editor.getModel(), 'java', monacoErrors);
But I cannot remove them as passing an empty array or null leads to ' invalid marker service state'.

Hi @tomitrescak, this is caused by a known bug, #230. It's already fixed in the core code base, the fix just hasn't been released yet.
If you want a version where this is fixed, you can just depend on my build here, just add something like the following to your package.json:
"monaco-editor"<EMAIL_ADDRESS>

Thanks @JKillian ! Will wait for release!


advisor: "az advisor recommendation list" command hangs

Python SDK's paging gets into a dead loop as the nextLink is "", while AutoRest runtime only stops at None.
This should be a service end bug, as clearly there are other information in the upcoming pages.

Hi, if that is the case. Can you explain how it was working in the first place.

The real fix would come from https://github.com/Azure/msrest-for-python/issues/147

@Prasanna-Padmanabhan for comment.

This issue was resolved more than a month ago, please let us know if this is still an issue.


docs: Update go-sdk example

i think it needs to be there, otherwise it will error
On Thu, Feb 8 2024 at 6:26 AM, Mark G. < @.*** > wrote:

@.**** commented on this pull request.
In apps/docs/src/pages/en/sdks/go.mdx (
https://github.com/streamdal/streamdal/pull/199#discussion_r1483064831 ) :

  ServiceName:     "billing-svc",

ShutdownCtx:     ctx,

I just copied the example from the readme. I can add it here if you want
it
—
Reply to this email directly, view it on GitHub (
https://github.com/streamdal/streamdal/pull/199#discussion_r1483064831 ) ,
or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AAOZKRY2E2UK3CP2UYTFBILYSTOCJAVCNFSM6AAAAABC6LXAS6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQNZQGM3TCMZVGI
).
You are receiving this because your review was requested. Message ID: <streamdal/streamdal/pull/199/review/1870371352
@ github. com>


Replace if variable != nil {} with ??= operator

When setting the parameters, we currently use...
if variable != nil {
  parameters[key] = variable!
}

...which becomes rather redundant and repetitive after a while; so instead, I added a ??= operator which checks if the rhs is not a nil, if so, it'll set the value of rhs to lhs (aka parameter[key])

Awesome trick! :clap: :clap:


Support new version in Chromedriver asset name

Support new version in asset name starting in 1.7.0
Refs https://github.com/electron/electron/pull/8682

I assume this needs to be backported?

Do you mean to<EMAIL_ADDRESS>line?

I assume this needs to be backported?

Do you mean to electron-download 3.x line?

Yeah, I meant to the version that electron-prebuilt uses.

Yeah, I meant to the version that electron-prebuilt uses.

I could definitely do that, although this won't impact prebuilt since it does not download Chromedriver, the only consumer of the chromedriver download path that I know of is https://github.com/electron/chromedriver which I was already going to upgrade to 4.x

Ah, OK.  Sounds like that's not necessary, then.


plasma lipoprotein particle oxidation

Hi
GO:0034441 plasma lipoprotein particle oxidation should not have the parent GO:0042161 lipoprotein oxidation. In general the proteins in plasma lipoprotein particles are bound together rather than covalently associated therefore this parent is in appropriate. There are currently no annotations using this term (although I would like to add one).
Instead it should have the parent GO:0034369 plasma lipoprotein particle remodeling
The definition should be changed from:
The modification of a lipoprotein by oxidation of one or more amino acids or the lipid group, occurring in the blood plasma.
to
The modification of a lipid or protein within a plasma lipoprotein particle by oxidation of the lipid or one or more amino acids.
Thanks
Ruth
Reported by: RLovering
Original Ticket: geneontology/ontology-requests/8618

Hi @RLovering
This issue looks like it's still valid. However GO:0034441 does not have the label "plasma lipoprotein particle oxidation", but just "plasma lipoprotein oxidation " ('lipoprotein particle is a related synonym.) All annotations to this term if from you group (see below).
Would it solve your pb if I swapped the term label and synonym and changed the hierarchy as you request ?
UniProtKB:P06727	Apolipoprotein A-IV	GO:0034445	BHF-UCL	IDA	PMID:16945374	20080719
UniProtKB:Q13093	Platelet-activating factor acetylhydrolase	GO:0034441	BHF-UCL	IDA	PMID:8624782	20110711
UniProtKB:P27169	Serum paraoxonase/arylesterase 1	GO:0034445	BHF-UCL	IDA	PMID:15342686	20090205
UniProtKB:O95445	Apolipoprotein M	GO:0034445	BHF-UCL	IDA	PMID:16682745	20110309
UniProtKB:O95445	Apolipoprotein M	GO:0034445	BHF-UCL	IDA	PMID:22204862	20120531
MGI:MGI:1930124	apolipoprotein M	GO:0034445	BHF-UCL	IDA	MGI:MGI:5427066|PMID:22204862	20120531
Thanks, Pascale

Hi Pascale
please revise as you/I suggest
Ruth

Changes made:
OLD name: plasma lipoprotein oxidation
NEW name: plasma lipoprotein particle oxidation
namespace: biological_process
OLD def: "The modification of a lipoprotein by oxidation of one or more amino acids or the lipid group, occurring in the blood plasma." [GOC:BHF, GOC:mah]
OLD synonym: "plasma lipoprotein particle oxidation" RELATED [GOC:dph]
OLD is_a: GO:0042161 ! lipoprotein oxidation
NEW def: "The modification of a lipid or protein within a plasma lipoprotein particle by oxidation of the lipid or one or more amino acids." [GOC:BHF, GOC:mah]
NEW synonym: "plasma lipoprotein oxidation" RELATED [GOC:dph]
NEW is_a: GO:0034369 ! plasma lipoprotein particle remodeling
Thanks, Pascale


It's new version (v_1.2.0) not working for linux, may be for windows also but older one works fine.

twocaptcha couldnot be resolved.
result=solver.normal('/path')
Error:
'normal' not found

Could you please provide all the code you use starting from the imports so we can try to reproduce the issue?

from twocaptcha import TwoCaptcha
solver = TwoCaptcha(anticaptcha_key)
solver.normal('/path')

from twocaptcha import TwoCaptcha
solver = TwoCaptcha(anticaptcha_key)
solver.normal('/path')

Just tested on a clean linux installation:
from twocaptcha import TwoCaptcha
solver = TwoCaptcha('KEY')
res = solver.normal('./cap.jpg')
print(res)

Works fine and prints the result.
Could you please run the following code and show the result?
import twocaptcha
print(twocaptcha.__version__)
print(dir(twocaptcha.TwoCaptcha))


To remove the restriction of having this repository cloned within the installation directory.

It would be cleaner to decouple the installer from the installation directory. If a copy of the installer scripts is needed within the installation directory, it can be created at the end of the process.

Done


pl.path requires LuaFileSystem error

first thanks to your work so I could do unit test with lua on Neovim.
Here's a CI pipeline I want to do vusted unit tests on Windows, to support Neovim plugin running on Windows: https://github.com/linrongbin16/commons.nvim/pull/148
Here's the action runs: https://github.com/linrongbin16/commons.nvim/actions/runs/7382472681/job/20082321700

in the CI pipeline, I installed lua-5.1 and luarocks via the hererocks, then install all the vusted dependencies, they all succeed, but vusted failed to run with error:
pre-vimrc command line..script D:\a\commons.nvim\commons.nvim\env\bin/vusted_entry.vim, line 20
Vim(lua):E5108: Error executing lua
...commons.nvim\commons.nvim\env\/share/lua/5.1/pl\path.lua:26: pl.path requires LuaFileSystem
stack traceback:        [C]: in function 'error'
...commons.nvim\commons.nvim\env\/share/lua/5.1/pl\path.lua:26: in main chunk        [C]: in function 'require'
...s.nvim\commons.nvim\env\/share/lua/5.1/busted\runner.lua:3: in main chunk        [C]: in function 'require'
...mons.nvim\commons.nvim\env\/share/lua/5.1/vusted/run.lua:18: in function <...mons.nvim\commons.nvim\env\/share/lua/5.1/vusted/run.lua:1>
[string ":lua"]:10: in main chunk

I'm not sure why that happened, would you please help on that?

Hi! This is LuaFileSystem's limitation.

On Windows, the C runtime used to compile LuaFileSystem must be the same runtime that Lua uses, or some LuaFileSystem functions will not work.

https://lunarmodules.github.io/luafilesystem/manual.html#building
FYI: I use the following workaround.
https://github.com/notomo/action-setup-nvim-lua/blob/cc7bc230b4ed5c9cbb2df9570bb749f560b69001/src/luarocks.js#L57-L65

hi @notomo , really thanks to your reply, I had updated my CI yaml file: https://github.com/linrongbin16/commons.nvim/pull/148/files#diff-b803fcb7f17ed9235f1e5cb1fcd2f5d3b2838429d4368ae4c57ce4436577f03fR63-R85.
It's now:
  windows_unit_test:
    name: Windows Unit Test
    needs:
      - luacheck
    strategy:
      matrix:
        nvim_version: [stable, nightly, v0.6.0, v0.7.0]
        os: [windows-latest]
        lua: [lua 5.1]
    runs-on: ${{ matrix.os }}
    steps:
      - uses: actions/checkout@v4
      - uses: rhysd/action-setup-vim@v1
        id: vim
        with:
          neovim: true
          version: ${{ matrix.nvim_version }}
      - uses: ilammy/msvc-dev-cmd@v1
      - uses: notomo/action-setup-nvim-lua@v1
      - run: |
          luarocks install busted
          luarocks install vusted
          vusted --shuffle ./test

It does following steps:

checkout git repo source code, it's the plugin source code I will test.
setup 'nvim'.
setup MSVC (the cl) VC++ compiler.
use action-setup-nvim-lua@v1 you mentioned above (I think it's going to install luarocks and luajit-2).
setup busted and vusted, then run the unit tests.

But there's 2 more errors in action runs: https://github.com/linrongbin16/commons.nvim/actions/runs/7383346276/job/20084404068
The Error-1: failed to install luasystem library.

The Error-2: failed to find vusted executable.

I guess the error rootcause is failed to install luasystem, would you please help me?

hmm. I reproduced this.
But this is hard to resolve for now.
I'll look into it when I have time.
This may be action-setup-nvim-lua's problem.

hi @notomo , FYI:
cannot remember the exact resource, but I believe I had seem some thing say that, we should compile lua/luajit/luarocks with ilammy/msvc-dev-cmd (the MSVC VC++ compiler, e.g. the cl binary).
Because default mingw/GCC has some issue, that could be the root cause of failure of luasystem.

cc: @linrongbin16
Released workaround!
https://github.com/notomo/vusted/releases/tag/v2.3.4

It's working here: https://github.com/linrongbin16/commons.nvim/actions/runs/7394420875/job/20115783971!!!


C API ownership problems

Version
Latest
Platform
All
Description
The docs for hyper_clientconn_handshake say "Both the io and the options are consumed in this function call." But the code looks like this:
    fn hyper_clientconn_handshake(io: *mut hyper_io, options: *mut hyper_clientconn_options) -> *mut hyper_task {
        let options = non_null! { Box::from_raw(options) ?= ptr::null_mut() };
        let io = non_null! { Box::from_raw(io) ?= ptr::null_mut() };

If options is null the function won't take ownership of io, which has a high chance of causing a memory leak. Probably Box::from_raw should be called on both arguments before they are checked for null?
Similarly, the docs for hyper_executor_push say "The executor takes ownership of the task, it should not be accessed
again unless returned back to the user with hyper_executor_poll." But the code does this:
    fn hyper_executor_push(exec: *const hyper_executor, task: *mut hyper_task) -> hyper_code {
        let exec = non_null!(&*exec ?= hyper_code::HYPERE_INVALID_ARG);
        let task = non_null!(Box::from_raw(task) ?= hyper_code::HYPERE_INVALID_ARG);

If exec is null the function won't take ownership of task, which also is likely to result in memory leaks. In this case the two lines could be swapped to fix the problem.
I'm happy to file a PR fixing these if it seems like I've diagnosed the situation correctly. I looked through all the other C API functions and these were the only two cases like this I could find.

Thanks for the write-up and taking a look. I see what you're saying. Which would be more expected? If the function is called with bad arguments, is it expected that it still took ownership of everything else that wasn't bad? I think this currently assumes that if the function returns an error-ish value, that the caller may want to do something else with the arguments. But if it's more idiomatic, we can just gobble everything up first.
I admit my experience with writing C APIs is limited (I've read and debugged plenty of C, but not really exposed a public API). So I welcome hearing your recommendations :)

I agree with OP - caller can't know exactly where you failed (whether on the first check or second) then can't know which one it should "own" so best behavior is to first own then check.

I also wasn't sure about this. The current behaviour of owning one and not owning the other is definitely bad. Own-on-failure and don't-own-on-failure are both reasonable, but I lean towards own-on-failure:

It matches the existing docs.
It is simpler to describe and implement. You do all the owning up front; if there's an error later in a function you don't have to worry about de-owning things.

hyper_clientconn_options_http2 is an interesting case, it's a rare function that can fail for a reason other than an invalid arg, and it currently takes ownership of (and drops) the opts in the error case.

It is potentially simpler for the C coder, because they don't have to free the input(s) on failure.
The only downside I can see is that it prevents the C coder from doing something else with the input on failure. But the curl code I've seen never does that, it always jumps straight to some error handler.

https://github.com/curl/curl/pull/11745 is relevant: there was a call to hyper_request_set_body(hyperreq, body), which always takes ownership of body. On failure the code then did hyper_body_free(body), which would case a double free. So I removed that hyper_body_free call. (In practice this double free would only happen if hyperreq was null.)
Another example from curl: Curl_http has lots of code like this:
  hyper_io *io = NULL;
  hyper_clientconn_options *options = NULL;
  ...
  io = hyper_io_new();
  if(!io) {
    failf(data, "Couldn't create hyper IO");
    result = CURLE_OUT_OF_MEMORY;
    goto error;
  }
  ...
  options = hyper_clientconn_options_new();
  if(!options) {
    failf(data, "Couldn't create hyper client options");
    result = CURLE_OUT_OF_MEMORY;
    goto error;
  }
  ...
  <more code here that can `goto error`>
  ...
  handshake = hyper_clientconn_handshake(io, options);
  if(!handshake) {
    failf(data, "Couldn't create hyper client handshake");
    result = CURLE_OUT_OF_MEMORY;
    goto error;
  }
  io = NULL;
  options = NULL;
  ...
error:
  if(io)
    hyper_io_free(io);

  if(options)
    hyper_clientconn_options_free(options);

Look at where the null assignments are. They are appropriate if hyper_clientconn_handshake uses the don't-own-on-failure approach. For the own-on-failure approach the null assignments should be moved to before the if(!handshake). (Which, admittedly, is non-idiomatic C, to have something between the handshake assignment and the handshake check.) This pattern is repeated a lot in curl. In practice it doesn't matter right now because almost all the hyper C API functions can only fail if passed a null pointer, and curl does enough checking that this never happens. The examples in the hyper repo don't do much error checking, so they don't provide guidance.
If we do commit to own-on-failure, then (a) the docs should probably be clarified throughout, and (b) curl's code should be changed to move the null assignments earlier.
@bagder may have opinions here.

My opinion is pretty simple and clear:
If there would be documentation for each C function that describes how they work, including when they fail, we would adapt to that no matter what the decision is to do about ownership. I think the lack of the first part is what makes the second part so complicated.

Perhaps using Option<Box<Foo>> should be preferred to Box<Option<Foo>>. After all, Option<Box<Foo>> is the direct safe-Rust equivalent of a nullable smart pointer. Operating on a raw nullable pointer, in C with manual memory management, is analogous to handling an optional boxed value, in Rust with static memory management.
This approach provides an elegant parallel with respect to Rust semantics. It matches up with the Rust compiler's null-pointer optimization (NPO) which makes mem::size_of::<Option<Box<Foo>>>() == mem::size_of::<Box<Foo>>() because the box itself, being a pointer, can never be null. In fact, this will enable you to simply and safely transmute with zero runtime overhead, consistent ownership semantics, and beautiful code style:
fn pointer_to_box<V>(ptr: *mut V) -> Option<Box<V>> {
    // SAFETY
    // As explained above : )
    unsafe { mem::transmute(ptr) }
}

fn box_to_pointer<V>(boxed: Option<Box<P>>) -> *mut V {
    // SAFETY
    // And likewise
    unsafe { mem::transmute(boxed) }
}

fn hyper_clientconn_handshake(io: *mut hyper_io, options: *mut hyper_clientconn_options) -> *mut hyper_task {
    let options = pointer_to_box(options);
    let io = pointer_to_box(io);

    let task: hyper_task = { ... };
    box_to_pointer(Some(hyper_task))
}

The function arguments are consumed regardless of success or failure. One could imagine writing a macro to replace this boilerplate and generate automatic FFI bridges, whereby Rust code uses Option<Box<V>> and C code operates with raw pointers.

@A248 The problem I describe solving in https://github.com/rustls/rustls-ffi/issues/244 is not that C code might pass a NULL pointer, it's that C code might pass a dangling pointer. That is, a pointer to an object that has been moved by Rust code.
For this purpose, Option<Box<Foo>> is not helpful, because it would have to store the "target was moved" information on the caller side. In other words, if the C code has foo *item, it would represent the None variant by setting item = NULL after it knows the Rust code consumed the item (moved it, or turned it into something else). Most C code does do that. But there are two problems:

Control flows are complicated and sometimes the code to NULL out a pointer after use doesn't get called.
The Rust code has to clearly communicate whether it moved the item or not.

Even if we carefully document whether items are moved on error or not, we still run into (1) because the C code has to check an error code before deciding whether to free and NULL the pointer or whether to just NULL it.
So, yes, Box<Option<Foo>> is a bummer because it doesn't take part in the NPO, but it is necessary to achieve the properties I want: Rust knows at runtime whether a given item has been moved, at least until the containing pointer has been freed.

I think I see what you mean. However, I must admit I'm not too familiar with C. I do have one question, though, about your statement here:

because the C code has to check an error code before deciding whether to free and NULL the pointer or whether to just NULL it.

Why does the C code need to decide whether to free the pointer? As I understand it, if the Rust code uses Option<Box<Foo>>, the Rust compiler will drop the parameters in all circumstances. Thus, as soon as the Rust function returns, the arguments have been consumed and, if necessary, dropped and freed. Rust takes ownership so C doesn't have to worry about freeing the passed arguments anymore.
I guess, what I am wondering is, why can't the C code just hand over responsibility for freeing memory to the Rust compiler? Then, C code only has to NULL the pointer, or somewhat stop using it (I'm not really clear on what C programmers do after they have a pointer they know is freed).
Of course, I may be missing some finer semantics here relating to drops, frees, and returning memory to the allocator. So, please do correct me if I'm wrong. I come from higher-level languages and have worked my way down here...

Why does the C code need to decide whether to free the pointer?

Speaking in general, not about the specific case that prompted this discussion:
All objects have a lifecycle. They are created and then they are freed. In Rust, that lifecycle is mostly managed automatically. Objects are freed when they go out of scope without having ownership handed to some other scope.
In C, the lifecycle is managed manually. Every object must be freed once and exactly once, by calling free(3) or by calling an object-specific function like hyper_executor_free(). This is the source of a large proportion of memory safety vulnerabilities.
When writing an FFI API in Rust, the Rust code usually has to hand over ownership of certain values to the C code. That makes the C code responsible for freeing them. The Rust code doesn't necessarily know when the C code is done using them.

I was still curious, so I looked into what others have said about this, because I'm still relatively new to this. The docs for Box seem to suggest using Option<Box<Foo>> for the reason I described: that the C pointer can be null. Even more interestingly, it seems that FFI functions can be declared as simply accepting Option<Box<Foo>>, meaning that manual conversions in source code are unnecessary. From https://doc.rust-lang.org/std/boxed/index.html#memory-layout:

/* C header */

/* Returns ownership to the caller */
struct Foo* foo_new(void);

/* Takes ownership from the caller; no-op when invoked with null */
void foo_delete(struct Foo*);

These two functions might be implemented in Rust as follows. Here, the struct Foo* type from C is translated to Box, which captures the ownership constraints. Note also that the nullable argument to foo_delete is represented in Rust as Option<Box>, since Box cannot be null.

#[repr(C)]
pub struct Foo;

#[no_mangle]
pub extern "C" fn foo_new() -> Box<Foo> {
    Box::new(Foo)
}

#[no_mangle]
pub extern "C" fn foo_delete(_: Option<Box<Foo>>) {}

Notice how Rust can return a non-null Box<Foo> back to C land. However, because this is an externally expoed API, the Rust code cannot make assumptions about the argument to foo_delete and must check for nullability. The implementation of foo_delete is, correctly, an empty function, because it will drop and free the *mut Foo hiding behind the abstractions if and only if that *mut Foo is non-null.
So I think resolving this issue may be as simple as changing function arguments to Option<Box<Foo>>. Provided, of course, that the relevant argument types like hyper_io, hyper_clientconn_options, and company are instantiated in Rust land using the global allcoator -- which seems to hold true based on my reading of the Hyper C API part of the codebase.


Regression Failure: MB13

I'm getting the following error on MB13 regression (from indexing scratch):
2018-08-14 15:45:54,999 INFO  [pool-2-thread-19] index.IndexCollection$IndexerThread (IndexCollection.java:243) - data/statuses.log.2013-02-08-15.gz: 185018 docs added.
2018-08-14 15:56:55,819 WARN  [main] index.IndexCollection (IndexCollection.java:370) - Unexpected difference between number of indexed documents and index maxDoc.
2018-08-14 15:56:55,819 INFO  [main] index.IndexCollection (IndexCollection.java:373) - # Final Counter Values
2018-08-14 15:56:55,820 INFO  [main] index.IndexCollection (IndexCollection.java:374) - indexed:      203,162,923
2018-08-14 15:56:55,821 INFO  [main] index.IndexCollection (IndexCollection.java:375) - empty:            143,890
2018-08-14 15:56:55,821 INFO  [main] index.IndexCollection (IndexCollection.java:376) - unindexed:     55,894,079
2018-08-14 15:56:55,821 INFO  [main] index.IndexCollection (IndexCollection.java:377) - unindexable:            0
2018-08-14 15:56:55,821 INFO  [main] index.IndexCollection (IndexCollection.java:378) - skipped:       88,516,634
2018-08-14 15:56:55,821 INFO  [main] index.IndexCollection (IndexCollection.java:379) - errors:             2,754
2018-08-14 15:56:55,833 INFO  [main] index.IndexCollection (IndexCollection.java:382) - Total 203,145,943 documents indexed in 08:33:45
2018-08-14 15:57:02,736 - regression_test - INFO - ==========Verifying Index==========
2018-08-14 15:57:02,738 - regression_test - INFO - [Index]: lucene-index.mb13.pos+docvectors+rawdocs
2018-08-14 15:57:06,630 - regression_test - INFO - documents:             203145916
Traceback (most recent call last):
  File "src/main/python/run_regression.py", line 203, in <module>
    verify_index(yaml_data, args.index, args.dry_run)
  File "src/main/python/run_regression.py", line 98, in verify_index
    assert value == yaml_data['index_stats'][stat]
AssertionError

I'm at this current commit:
commit d7764a12c21de01492dd7967cfe347186f068746
Author: Boris Lin<EMAIL_ADDRESS>Date:   Tue Aug 14 10:00:10 2018 +0800

    Patch for Issue #381: Refactor iterator (#399)

    Issue #381 - Iterator depending on NoSuchElementException to signal end of iteration.
    This causes exceptions when we try to apply a functional programming style.

This is probably the critical line:
2018-08-14 15:56:55,819 WARN  [main] index.IndexCollection (IndexCollection.java:370) - Unexpected difference between number of indexed documents and index maxDoc.

@borislin it's probably your code above? before, when you hit an error, we bomb out of that segment... now we continue along?

Regressions fail here for mb13 also:
commit 05df1d68fc6bfa26f291a18fbc34c8c1e13c1dd6
Author: Peilin Yang<EMAIL_ADDRESS>Date:   Mon Aug 13 13:23:24 2018 -0700

    fix runtag for TREC submission (#406)

I suppose we could do galloping search through the commits to find out which one broke the regression... but does anyone else have any hypotheses?

Regressions failed here for mb13 also:
commit 6b19d373634d510e8ac5f5b26b1a3d5c321926e4
Author: Peilin Yang<EMAIL_ADDRESS>Date:   Thu Aug 9 13:50:44 2018 -0700

    Remove -optimize option for regression test (#393)

mb13 regressions pass here:
commit fb959bbb5c94c376cebc498a6b95600182e1aaee [PASS]
Author: Jimmy Lin<EMAIL_ADDRESS>Date:   Mon Aug 6 10:25:17 2018 -0400

    Added +rawdocs to name of regression indexes (#389)

    #388 added -storeRawDocs, this patch changes the name of the index to reflect it.

I've isolated the issue to the following patch:
commit 6b19d373634d510e8ac5f5b26b1a3d5c321926e4 [FAIL]
Author: Peilin Yang<EMAIL_ADDRESS>Date:   Thu Aug 9 13:50:44 2018 -0700

    Remove -optimize option for regression test (#393)

commit 81582f82685118911192e97c35f9d494f19d0fd7 [PASS]
Author: Peilin Yang<EMAIL_ADDRESS>Date:   Thu Aug 9 13:20:22 2018 -0700

    fix search runtag arg; make default runtag as Anserini (#392)

commit d4b3272e7f07fa274e5d8ffd50976beb0c08de52 [PASS]
Author: Victor Yang<EMAIL_ADDRESS>Date:   Wed Aug 8 22:33:15 2018 -0400

    Add Regression Test for Car17 (#379)

commit 135d08c7e026c4455ab595f39fbb956ba34afb9f [PASS]
Author: Victor Yang<EMAIL_ADDRESS>Date:   Tue Aug 7 11:13:45 2018 -0400

    Fix zero-length norm bug with RM3 (#376)

    Avoids zero-length feedback documents, which causes division by zero when computing term
    weights. Zero-length feedback documents occur (e.g., with CAR17) when a document has only
    terms that accents (which are indexed, but not selected for feedback).

In other words, it's @Peilin-Yang 's fault :)
I understand how this happened now - in MB13 there are duplicate documents... I assume optimize cleans up deleted docs, so it yields a different index stats.

It appears that the number of docs are non-deterministic! What??
$ target/appassembler/bin/IndexUtils -index lucene-index.mb13.pos+docvectors+rawdocs -stats
Index statistics
----------------
documents:             203145916
documents (non-empty): 203143470
unique terms:          -1
total terms:<PHONE_NUMBER>
stored fields:
  contents (indexOption: DOCS_AND_FREQS_AND_POSITIONS, hasVectors: true)
  id (indexOption: DOCS, hasVectors: false)
  lang (indexOption: DOCS, hasVectors: false)
  screen_name (indexOption: DOCS, hasVectors: false)

Fixed by #415


Side markdown preview doesn't line up when headings are collapsed

Issue Type: Bug

Open a large-ish markdown file.
Collapse some headings
Scroll around

It should keep aligned, but they actually misalign.
 This means that as you scroll past any collapsed section in the editor, the preview it will rapidly jump past it. Is this what you see? If not, please provide a gif or photo of the problem


Tooltip for "multisig transaction" symbol

0.5.4-BETA
... is now "Incoming transaction".
... should be "Incoming multisig transaction".

Not sure if that is necessary. Multisig transactions have a different icon on the left side.

On recipients side:

On issuer side:

On actual multisig account (it's where funds come from, that's why it's red):

Hopefully fixed in 0.5.11

Not yet fixed in 0.5.12-BETA
To be clear: I was just talking about the tooltip. (And I know, this is a minor minor issue ;) I just thought it would be more consistent.)
http://s3.postimg.org/oebvrmfub/NEM_tooltip_multisig.png

The thing is, it IS incoming transaction, to be more precise incoming multisig transaction, but we can't put all possiblities in tooltip.

okay :)


Website is down

Hello, the website is down. And has been for a few days now. Can i use this source to continue the project if you decide to drop it? Thank you

People are free to do whatever they want with any of the projects I had my website.

Data files (e.g. https://www.haizor.net/rotmg/assets/production/xml/equip.xml) are similarly unaccessible and not present in the repository. Before I make a webscraper for RealmEye is there any way I can get these? Thanks.

Data files (e.g. https://www.haizor.net/rotmg/assets/production/xml/equip.xml) are similarly unaccessible and not present in the repository. Before I make a webscraper for RealmEye is there any way I can get these? Thanks.

Have you got any of the data files? We can remake this dps calc if you'd want to.

Somebody else has already rehosted it, but if you still want to remake it:
https://github.com/AssetRipper/AssetRipper
You can rip the assets from the game using this. Most Text Assets are XML files, and you only need the four main atlases (groundTiles, characters, and mapObjects) from the Texture2Ds. The only other major asset is the spritesheet.json, which is a text asset.

Hi, just commenting to ask about the legality of hosting the XML files, etc. , I've made an automatic CI for it, but am unsure whether the Creative Commons License inside those files applies to all the files included in the original repository: https://www.haizor.net/rotmg/assets/production/

I mean technically it's probably not entirely legal to rehost the realm assets, but DECA doesn't seem to care if that's what you're asking.

oh alright thanks!

@Haizor thanks for linking both the tool and the rehost, incredibly helpful + thanks for the work on this project overall.


Isort automatically added extra unknown flags

VS Code version
1.62.1
Extension version
v2021.11.1422169775
OS type
Linux
OS version
Linux x64 5.13.19-2 Manjaro
Python distribution
Operating system
Python version
3.9.7 64-bit
Language server
Pylance
Expected behaviour
As I could and also did, define the flags for python.sortImports.args, I expected that the isort would follow the config.
actually I executed the command for sorting in the terminal (without that extra dash and --dif flag) and it was OK.
Actual behaviour
But it didn't and I reviewed the log for python extension so I found that calls for isort with an extra dash(following no characters), and also the --dif flag that is not what I put there.(I set the settings to format my code on save and it would also call for sorting as well.)
Steps to reproduce

Logs
> ~/.pyenv/bin/python ~/.vscode/extensions/ms-python.python-2021.11.1422169775/pythonFiles/sortImports.py - --diff --atomic --overwrite-in-place --profile=django --trailing-comma --use-parentheses --float-to-top
cwd: path/to/my/python-file

Code of Conduct

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

@mh-firouzjaah We use isort with --diff to generate the text edits for the editor. We do this to allow users to have the ability to undo changes just like you would with any other text edits. The extra - is used to tell isort to read from stdin. To perform the isort operation we pass the file contents to isort via stdin.
You should not need --overwrite-in-place, we will calculate the diff and update the contents of the opened file.
If you wish to run isort with your own flags, you can always create a task using task.json. You can control any aspect of how you run isort there. If you are using this method then you may want to disable isort for the python extension.

Hello and thanks for the information and attention.
My problem was that when vscode called for sorting especially in Python files, the sorting behavior was different from what I expected (I expected it to work like when we call isort). I searched for about 3 hours and finally realized that vscode needs a special flag to behave as I expected. The solution is to add flag "--src=${workspaceRoot}" for "python.sortImports.args"
I love vscode and I spend my time to find some good extensions and settings for it to make more better.
Thanks again and best regards.

And I need a favor to avoid spending another 3 hours for searching the net.
Is there any settings that would help Pylance/Python extension to realize python packages in order to auto-suggest for import.
Actually I spent some time to find out, "python.analysis.diagnosticMode": "workspace", , "python.analysis.extraPaths" and "python.autoComplete.extraPaths" are required, but these are not enough and the is some shortage/deficiency.

@mh-firouzjaah I created this issue https://github.com/microsoft/pylance-release/issues/2069 in pylance. They should be able to help with this.


[sc-536] Add debug mode for lazy evaluation in hvml

An alternative would be for hvm-core to implement partial lazy reduction which both hvm-core and hvm-lang could use for debug mode


Upload to PyPi and make installation hella easier

Its not that tough, here is a simple tutorial:
http://peterdowns.com/posts/first-time-with-pypi.html

I am even emailing you the source package and wheels to make your life easier!

Hi Chris,
thanks for taking the time and checking the 2.7 compatibility! I carefully checked your modifications and merged the pull request, plus some other minor cleanup.
I will closely look at pypi and probably upload soon. Would be cool to just write 'pip install pysut' and be done with it :)
One question: You capitalized pydsm.py to pdDSM.py because of the repository name, right? I accepted that change and tried to to the same for pySUT.py, but my capitalisation on Windows does not translate to Github. Plus, a student of mine had problems installing pydsm previously when it was capitalized for the first time. What's your take on this?
Cheers,Stefan

I just re-uploaded pySUT.py and it is capitalized now. So is the call in init.py.
That should do the job.

There are three approaches to fix this; all three would work, and the one you choose just depends on your personal style:

Change the name of the file to pysut.py (i.e. make it lower-case)
Do what you already did, change the import line to pySUT in __init__.py
Move the code into the __init__.py file - as there is only one file, you don't need anything else.


Use request headers in mock body?

Would it be possible to add to the body Variable Tags something that can extract values from the request headers, similar to the request.path."key" tag?
Hoping to read a user ID from a request header and use it in the response body.

Fixed by @mrgleam


Move actual start and end times to end of form

Gaile requested this a while ago. Flask Admin is weird, but
this seems to work ok.
@xybrnet

That is a bit weird. :+1:


Demo/front end

This is the correct web application frame work.

[ ]  Does the code work?
[ ]  Are functions, methods, and variables adequately named?
[ ]  Any test on the new component?
[ ]  Is it well tested? All components could be rendered and pass the test.
[ ]  Is the code clear and easy to understand?
[ ]  Does it meet the team guidelines and style guides?
[ ]  Is there any duplicated code?
[ ]  Is documentation on functions, methods, classes, contexts, and behaviours adequate?
[ ]  Is the code as modular as possible?
[ ]  Are there better or simpler solutions?
[ ]  Is there any performance issue?

Provided front-end framework.


Add missing exports, add Hash for SubscriberInfo

Add missing core type exports
Fix signature of fetch_subscribers() method
Add Hash implementation for SubscriberInfo

Note: tests fail on this commit because it's based of the current main, which is fundamentally broken as per the recent removal of UPayload from up-spec (#145). #108 fixes all that, this PR should test green once #108 is merged.


Samsung Smart TV Prime Video

https://www.reddit.com/r/pihole/comments/9fmudp/getting_prime_video_to_work_with_samsung_smart_tv/
According to that Reddit post, the following URL needs to be whitelisted for Samsung TV's Amazon Prime Video app.
time.samsungcloudsolution.com

@PurpleHullPeas Thanks :)


[FEATURE] Make (More Composter Variants by LieOnLion) compatible with your mod \ 1.21

Compatibility with this mod would be delightful.

Added with 4.1.0 https://modrinth.com/mod/compost-bag/version/4.1.0+fabric


cannot change/rename the name config-sample.json to config.json

Expected Behavior

Current Behavior

Steps to Reproduce (for bugs)

Versions

Homebridge: homebridge --version
Node:       node --version
OS:

Configuration

Homebridge Log / Command Output

I cannot rename the config-sample.json to config.json
I get a error
error renaming file /usr/lib/node_modules/homebridge/config-sample.json: Toegang geweigerd acces dinied) What is the problem en how can i fix it?
Thanks Chris


C# model no longer generated

I'm trying to generate a REST client with the following command:
autorest --input-file=openapi.yaml --csharp --namespace=SmartControlServices --verbose --public-clients=true --clear-output-folder
SourceMapConsumer.initialize is a no-op when running in node.js
AutoRest code generation utility [cli version: 3.5.1; node: v16.13.0]
(C) 2018 Microsoft Corporation.
https://aka.ms/autorest
info    | AutoRest core version selected from configuration: ^3.7.0.

There is a new version of AutoRest available (3.6.1).
 > You can install the newer version with with npm install -g autorest@latest

info    |    Loading AutoRest core      'C:\Users\stefa\.autorest\@autorestcore@3.8.4\nodemodules\@autorest\core\dist' (3.8.4)
verbose | [1.50 s] No configuration found at 'file:///C:/Users/stefa/source/repos/SmartControl.Manager/SmartControlClient/openapi/'.
verbose | [1.53 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [1.59 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [1.61 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [1.64 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [1.68 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [1.72 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [1.76 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [1.80 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [1.85 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [1.90 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [1.93 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [1.96 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [1.99 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [2.03 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [2.07 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [2.12 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [2.16 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [2.20 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [2.23 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [2.27 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [2.32 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>verbose | [2.37 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcore<EMAIL_ADDRESS>info    |    Loading AutoRest extension '@autorest/csharp' (latest->3.0.0-beta.20220701.3)
verbose | [2.68 s]    Including extension configuration file 'file:///C:/Users/stefa/.autorest/@autorestcsharp<EMAIL_ADDRESS>verbose | [2.69 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestcsharp<EMAIL_ADDRESS>info    |    Loading AutoRest extension '@autorest/modelerfour' (4.23.5->4.23.5)
verbose | [2.92 s]    Including extension configuration file 'file:///C:/Users/stefa/.autorest/@autorestmodelerfour<EMAIL_ADDRESS>verbose | [2.92 s]    Including configuration file 'file:///C:/Users/stefa/.autorest/@autorestmodelerfour<EMAIL_ADDRESS>verbose | [4.09 s] Reading OpenAPI 3.0 file file:///C:/Users/stefa/source/repos/SmartControl.Manager/SmartControlClient/openapi/openapi.yaml
warning | PreCheck/CheckDuplicateSchemas | Checking for duplicate schemas, this could take a (long) while.  Run with --verbose for more detail.
verbose | [4.98 s] Found and removed 0 duplicate schema
verbose | [5.39 s]   ModelerFour/api-version-mode:client
verbose | [5.39 s]   ModelerFour/api-version-parameter:constant
verbose | [5.39 s]   ModelerFour/use-model-namespace:false
warning | output-folder path should be an absolute path
info    | Autorest completed in 10.79s. 3 files generated.

openapi.yaml.txt
Few weeks ago everything worked well, but now the model is no longer generated. Attached you can find the openapi file.
Many thanks for the great support!
Stefan

It works!
Great!

This is really unfortunate, this has cost me hours. How can such fundamental change be made which breaks all users' command lines.
I mean, you call it "beta", but in fact this affect all command lines which do not specify any specific version.

Is there any update about the fix?

This project is no longer useful or usable for anybody outside of MS.
The focus has clearly shifted on addressing MS internal requirements - which is totally fine IMO, but it would have been (would be) much better to continue the (publicly useless) work on 3.x privately and keep the (useful) 2.x branch public and in a working state instead of breaking all existing usages.
("breaking" due to the auto-updating to 3.0 which doesn't work anymore like it did before)

BTW, after having tried dozens(!) of version combinations, here's the magic one that works:
call npx autorest --input-file=api.json --csharp --public-clients --version=3.0.6375 --use=<EMAIL_ADDRESS>

I agree with softworkz, and thank you for his fix, it is indeed the version that worked for me.


run: Add --uts flag

Adds support for specifying the host UTS namespace (the only thing
docker supported). This also consolidates the namespace options
checks for linux on run into a single function.

Unfortunately don't have a decent box to verify this for a day or two, so draft for now

Same strange (potential) rate-limit behavior on the project checks. If anyone can re-kick that off I'd appreciate

Thanks for your PR! In my personal opinion, I think it's unnecessary to change the setPlatformOptions signature in this PR. One PR for one thing, please.
If we need to change all signatures related to internalLabels, maybe we need another issue to discuss it.

It's not worth the hassle then don't worry about it, I'll move the signature change to another PR then. It was impacting the ability to just easily setup all the pid options enough it seemed warranted, and it's not exported or some relied on API here.

@Zheaoli, removed the setPlatformOptions signature change. I'll make an issue/separate PR

@Zheaoli, removed the setPlatformOptions signature change. I'll make an issue/separate PR. Thanks!

Doesn't seem removed

A separate PR is preferred, but a separate commit in the same PR is also ok IMHO

@Zheaoli, removed the setPlatformOptions signature change. I'll make an issue/separate PR. Thanks!

Doesn't seem removed

It's the same signature still, it's just over multiple lines as it was a bit long. The signature change was:
Before
func setPlatformOptions(
 	ctx context.Context,
 	opts []oci.SpecOpts,
 	cmd *cobra.Command,
 	client *containerd.Client,
 	id string,
 	internalLabels internalLabels,
 ) ([]oci.SpecOpts, internalLabels, error) {

After
func setPlatformOptions(
 	ctx context.Context,
 	opts []oci.SpecOpts,
 	cmd *cobra.Command,
 	client *containerd.Client,
 	id string,
 	internalLabels *internalLabels,
 ) ([]oci.SpecOpts, error) {

but I reverted the after variant after the discussion

A separate PR is preferred, but a separate commit in the same PR is also ok IMHO (for this amount of PR)

Yea, this didn't seem too unwieldy to all be in one PR, but I'll just make a separate at this point no worries.

Rebased after https://github.com/containerd/nerdctl/pull/1771 landed.


[AutoPR signalr/resource-manager] Change operationIds of SignalRService from Signalr to SignalR

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2894

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/2413


Use modular versions of lodash functions

@michaelcontento Can we get a new official release when you have a chance? Thanks!

Sure, 1.2.4 published to npm :beers:

:+1: Thanks

At first I thought it was a good idea, but after my project got bigger I realized that most of my 3rd party packages uses monolithic version of Lodash, so I ended up with 2 versions of the same function. For example: lodash.merge and lodash/merge.
Webpack 1 only produces big bundles if importing from Lodash root module (import { merge } from 'lodash'), but importing an specific module (import merge from 'lodash/merge') have the same result as importing the modular version (import merge from 'lodash.merge').
I'd recommend reverting to monolithic version of Lodash to keep in sync with most of other packages.

Quick explanation of how import works on Webpack 1:
import { merge } from 'lodash' imports everything declared by Lodash index file (an object containing all the functions) then apply destructure to get merge function. The bundle increases a lot because all unused functions are still declared in the imported object.
import merge from 'lodash/merge' imports the only the merge function.
This applies to almost all 3rd packages, including redux:
import createStore from 'redux/lib/createStore';
import applyMiddleware from 'redux/lib/applyMiddleware';
import compose from 'redux/lib/compose';
...

On Webpack 2 there will be an tree-shaking optimization allowing ES6 modules imported using destructure notation to be shrieked. More here: http://www.2ality.com/2015/12/webpack-tree-shaking.html


Added phone validation method inside PhoneNumber class which will help in validation before form submission.

Added phone validation method inside PhoneNumber class which will help in validation before form submission. I needed to validate the current phone number before final submission and onSubmit. So I wrote the isValidNumber() method inside the PhoneNumber class so that I can quickly validate the PhoneNumber object.

The function added is not used anywhere, so why even bother


Convert into cli tool (Python Package)

I would love to convert into Python Package
CC @meetmangukiya

Hmm, unfortunately it looks like the best name is already taken by https://github.com/rarcega/instagram-scraper (https://pypi.org/project/instagram-scraper/)


signalbackup-tools: 20230906 -> 20230907

Description of changes
Update to latest version. No changes that effect build.
Diff: https://github.com/bepaald/signalbackup-tools/compare/20230906...20230907
Things done

Built on platform(s)

[ ] x86_64-linux
[x] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.


Color needs to be adjustable so users have full ability to see everything on the screen

The color selection outline of blocks needs to be adjustable as well as the background color of the UI to ensure users are able to have full access to the interface.

There is now a dropdown menu containing several color variations.

Great thanks!  I look forward to seeing that third theme next week.
Steph
Sent from my iPhone

On Jul 8, 2016, at 5:13 PM, MaryS94<EMAIL_ADDRESS>wrote:
There is now a dropdown menu containing several color variations.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Can't compile from latest master branch

Attempted to recompile against the latest master and got the following errors:

# github.com/dmacvicar/terraform-provider-libvirt/libvirt
libvirt/resource_libvirt_domain.go:212: domain.ListAllInterfaceAddresses undefined (type libvirt.VirDomain has no field or method ListAllInterfaceAddresses)
libvirt/resource_libvirt_domain.go:212: undefined: libvirt.VIR_DOMAIN_INTERFACE_ADDRESSES_SRC_LEASE
libvirt/resource_libvirt_domain.go:239: undefined: libvirt.VIR_IP_ADDR_TYPE_IPV4
libvirt/resource_libvirt_domain.go:241: undefined: libvirt.VIR_IP_ADDR_TYPE_IPV6
libvirt/resource_libvirt_domain.go:359: domain.ListAllInterfaceAddresses undefined (type libvirt.VirDomain has no field or method ListAllInterfaceAddresses)
libvirt/resource_libvirt_domain.go:359: undefined: libvirt.VIR_DOMAIN_INTERFACE_ADDRESSES_SRC_LEASE

You need the updates libvirt-go

Yes, I keep my fork of libvirt-go in sync with this project.

Durp... got it. Thank you.


[JavaScript] Incorrect 'new' keyword warning

I'm seeing this warning complaining about me calling a non-void function with the new keyword, when it actually is a void function.

Reproduces with this code:
var BinomialHeap = function (customCompare) {
  this.head = undefined;
  this.nodeCount = 0;

  if (customCompare) {
    this.compare = customCompare;
  }
};

function removeTreeRoot(heap, root, prev) {
  // Remove root from the heap
  if (root === heap.head) {
    heap.head = root.sibling;
  } else {
    prev.sibling = root.sibling;
  }

  // Reverse the order of root's children and make a new heap
  var newHead;
  var child = root.child;
  while (child) {
    var next = child.sibling;
    child.sibling = newHead;
    child.parent = undefined;
    newHead = child;
    child = next;
  }
  var newHeap = new BinomialHeap();
  newHeap.head = newHead;

  heap.union(newHeap);
}

It actually goes away when I add undefined as an argument but that means the warning is complaining about the wrong thing. Also I don't think it's appropriate for the linter to warn on optional arguments in JavaScript like this by default as that's very strict.

Closed with the adoption of Salsa.


[VideoConferenceMute] separate mute/unmute hotkeys

Description of the new feature / enhancement
The mic and video hotkeys work as toggles. It would be handy to have an option to define separate hot keys for mute and unmute for the mic and similar distinct hotkeys for the video. This could be instead of or in addition to the existing toggle hotkey settings.
Scenario when this would be used?
Have you ever been on a call where someone thought they were on mute and toggled the mic to speak? They then accidentally put themselves on mute instead of unmuting. Hilarity ensues. This whole dumb situation could be avoided by having a way to set separate mute/unmute keys. That would just be for those who want that (me! me!) since the default of using toggles seems to be ingrained in all conferencing tools these days.
Supporting information
No response

Duplicate of #7185

Thanks for your suggestion! This issue is a duplicate.
I'm linking this issue to #7185 to centralize the discussion thread, please go to that thread for discussion.
Thanks for your help to make PowerToys a better piece of software!


Use org.junit.jupiter.api.Assertions package instead of org.junit.Assert

Make use of org.junit.jupiter.api.Assertions package instead of org.junit.Assert
and also org.hamcrest.MatcherAssert.assertThat instead of org.junit.Assert.assertThat .
I would like to help to make changes .

Thank you for offering to help.
Support to convert org.junit.Assert and org.junit.Assert.assertThat was merged end of July.

https://github.com/junit-pioneer/convert-junit4-to-junit5/blob/master/convert-junit4-to-junit5/src/main/java/jb/convert/ast/AssertConversion.java
https://github.com/junit-pioneer/convert-junit4-to-junit5/blob/master/convert-junit4-to-junit5/src/main/java/jb/convert/ast/AssertThatConversion.java

Does this fit your needs?

Please re-open in case you are missing features.


margo: aborting QueryCompletions. it did not respond in time on non-Go code

When entering Chinese
[11:44:37] margo: aborting QueryCompletions. it did not respond in time
Can set not execute gosubilme on non-Go code ?

https://github.com/KurokuLabs/margo/blob/master/extension-example/extension-example.go#L19


feat: support optimize router

考虑到后续其他代码优化压缩逻辑（比如 ice 2.x 自动的移除运行时代码）将 removeHistoryDeadCode 扩展为 optimization.router
增加 react-router 和 history 代码移除单测 #180

Codecov Report

:exclamation: No coverage uploaded for pull request base (release-next@3ca8f6e). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff               @@
##             release-next     #456   +/-   ##
===============================================
  Coverage                ?   81.98%
===============================================
  Files                   ?       87
  Lines                   ?     7545
  Branches                ?      952
===============================================
  Hits                    ?     6186
  Misses                  ?     1352
  Partials                ?        7

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


Add Error Prone Annotations dependency to suppress warnings

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
[x] Update the CHANGES log.

Add Error Prone Annotations dependency to suppress warnings, which are not errors.
Fix some issues reported by lift, https://lift.sonatype.com/results/github.com/apache/skywalking/01FZET2E227JTF4679DBEFBVBF

Add Error Prone Annotations dependency to suppress warnings, which are not errors.

Can you elaborate why the dependency can suppress warnings and what warnings it suppresses? I can find related info in the link

Dependency can't, I am using some annotations to do.

I just saw it is in the dependency path, what do you prefer? Remove it?


Adding the ability to specify tmux session name.

:+1: for the functionality here. @guildencrantz are you interested in fixing the rc file per the comment above? @sorin-ionescu Is that all it would take to get this merged?

That should be enough.

Sorry for the delay.

Merged in ddf5a99


config-server do not decrypt properties

Config-server won't decrypt properties server-side (even if it's enabled). Whenever a client asks for a configuration the server will not decrypt {cipher} marked properties. This happens if the spring-boot-starter-parent version is equal or greater than 2.2.0.RELEASE.
The error is caused by the following java method:
https://github.com/spring-cloud/spring-cloud-config/blob/baabff6a60c1a6a0dd7836804b6b076f4b53e471/spring-cloud-config-server/src/main/java/org/springframework/cloud/config/server/encryption/CipherEnvironmentEncryptor.java#L60-L80
seems that since 2.2.0.RELEASE the Map.Entry<Object, Object> entry (line: 65) value is not a String but an Object. This will cause entry.getValue().toString() call to return the full name of the object class instead of the value itself, preventing any decryption.
The solution is to change the way how the entry value is retrieved from entry.getValue().toString() to entry.getValue().getValue().
Details
Working: 2.1.7.RELEASE
Not Working: 2.2.0.RELEASE and above
To reproduce the error it's sufficient to start a config-server instance (with a spring-boot-starter-parent version 2.2.0.RELEASE or above) and another SpringBootApplication with spring-cloud-config-client that fetches the configuration. The config-server configuration must contain at least a property marked with {cipher} placeholder.

related #1490

I'm unable to reproduce this problem. I can successfully decrypt with the proper keystore and properties setup.

I'm unable to reproduce this problem. I can successfully decrypt with the proper keystore and properties setup.

Could you please try with a symmetric  encryption by declaring the encryption key  (encrypt.key) on the config-server bootstrap.yml file?

That works for me as well.

Can you provide a complete, minimal, verifiable sample that reproduces the problem? It should be available as a GitHub (or similar) project or attached to this issue as a zip file.

I've created a sample project that reproduces the problem. Here is the file with the project config-server-test.zip (remember to initialize the git repository inside the configurations folder and commit the yml file). To reproduce the error just run eureka-server first, config-server and then springboot-admin. The last one show report the following error:
[main] ERROR o.s.boot.SpringApplication - Application run failed
java.lang.IllegalStateException: Cannot decrypt: key=spring.security.user.name
	at org.springframework.cloud.bootstrap.encrypt.EnvironmentDecryptApplicationInitializer.decrypt(EnvironmentDecryptApplicationInitializer.java:292)
	at org.springframework.cloud.bootstrap.encrypt.EnvironmentDecryptApplicationInitializer.lambda$decrypt$0(EnvironmentDecryptApplicationInitializer.java:270)
	at java.base/java.util.LinkedHashMap.replaceAll(LinkedHashMap.java:694)
	at org.springframework.cloud.bootstrap.encrypt.EnvironmentDecryptApplicationInitializer.decrypt(EnvironmentDecryptApplicationInitializer.java:265)
	at org.springframework.cloud.bootstrap.encrypt.EnvironmentDecryptApplicationInitializer.decrypt(EnvironmentDecryptApplicationInitializer.java:190)
	at org.springframework.cloud.bootstrap.encrypt.EnvironmentDecryptApplicationInitializer.initialize(EnvironmentDecryptApplicationInitializer.java:124)
	at org.springframework.boot.SpringApplication.applyInitializers(SpringApplication.java:626)
	at org.springframework.boot.SpringApplication.prepareContext(SpringApplication.java:370)
	at org.springframework.boot.SpringApplication.run(SpringApplication.java:314)
	at org.springframework.boot.SpringApplication.run(SpringApplication.java:1226)
	at org.springframework.boot.SpringApplication.run(SpringApplication.java:1215)
	at com.example.SpringBootAdminApplication.main(SpringBootAdminApplication.java:17)
Caused by: java.lang.UnsupportedOperationException: No decryption for FailsafeTextEncryptor. Did you configure the keystore correctly?
	at org.springframework.cloud.bootstrap.encrypt.EncryptionBootstrapConfiguration$FailsafeTextEncryptor.decrypt(EncryptionBootstrapConfiguration.java:165)
	at org.springframework.cloud.bootstrap.encrypt.EnvironmentDecryptApplicationInitializer.decrypt(EnvironmentDecryptApplicationInitializer.java:277)
	... 11 common frames omitted

NOTE
While creating the new sample project I discovered what it was probably causing the problem. Apparently I was using the Spring Cloud Hoxton.M3 when I initially discovered the problem. Now I've changed to Hoxton.RELEASE and the problem is gone.

eureka and springboot-admin should not be needed to replicate the issue, can you remove them and make the project as minimal as possible.

Ok, I've simplified the project. Here it is the new archive config-server-test.zip

I ran the demo, but there was something wrong.I ran config-server,and then ran project test-app.Some error info output to the console.The error info displayed below:
java.lang.IllegalStateException: No .git at file://${user.dir}/configurations
@DaviGia

I ran the demo, but there was something wrong.I ran config-server,and then ran project test-app.Some error info output to the console.The error info displayed below:
java.lang.IllegalStateException: No .git at file://${user.dir}/configurations
@DaviGia

You must initialized the git repository before launching the project (see the comment above)

Well,the new error info was output to the console after I init the git repo.The error info displayed below:
java.lang.IllegalStateException: Could not locate PropertySource and the fail fast property is set, failing: None of labels [] found @DaviGia

Well,the new error info was output to the console of project test-app after I init the git repo.The error info displayed below:
java.lang.IllegalStateException: Could not locate PropertySource and the fail fast property is set, failing: None of labels [] found @DaviGia
You did not commit any file in the repository, please read carefully the whole reply I made.

Oh,Sorry. It is my carelessness. I  reproduced the bug after commit the  yml file. @DaviGia

Now I've changed to Hoxton.RELEASE and the problem is gone.

M3 is a milestone and not supported.


Refactor AttributeChanges and EntityAttributeChanges

There are two similar classes in Jmix:

io.jmix.core.event.AttributeChanges
io.jmix.data.impl.EntityAttributeChanges

Second class does almost the same as first, but also provides extra functionality: e.g. get changes from  eclipselink's AttributeChangeListener, work with dynamic attributes [in progress]
Proposition

Make EntityAttributeChanges extend AttributeChanges to get rid of code duplication
Rename EntityAttributeChanges to something like PersistentAttributeChanges / EclipselinkAttributeChanges

Fixed in Haulmont/jmix-data#46


"Light" version of dev setup / GitHub setup for those not needing a full clone

Description
Not all contributors need a full Git-linked local workspace, or would want to go through the manual command-line based version. The GitHub desktop client has also caught up fairly well and we could use a version of our setup docs based on it.
What sort of stuff can we edit nowadays via GitHub's user interface? That has been expanded a fair bit.
Definiton of 'Done'

[ ] add a wiki page on working with the GitHub desktop client
[ ] add information on "shallow clones" and other ways to manage big repos with long history
[ ] describe possibilities of GitHub web interace in the wiki (add and edit files, e.g., block definitions, create branches and PRs, ...)

Required Skills
Having worked with GitHub before should help you with this. It is more about general project setup/management with GitHub rather than code-related, so no Java required at this point.
Where to Start

this issue might be paired with #2569 to illustrate the dev setup
study the DevSetup page to get a basic understanding of the project layout
information on shallow clones and how to handle big repositories
GitHub Desktop Client and GitHub Workflow in the Browser

Closing as complete long ago - https://github.com/MovingBlocks/Terasology/wiki/Setting-up-a-Light-Environment has been a thing for a while now


components_datadog-private-location-ecs Error: Missing resource instance key

Describe the Bug
Error: Missing resource instance key
on .terraform/modules/components_datadog-private-location-ecs/modules/datadog-private-location-ecs/main.tf line 9, in locals:
  datadog_location_config = jsondecode(datadog_synthetics_private_location.private_location.config)
Because datadog_synthetics_private_location.private_location has "count" set, its attributes must be accessed on specific instances.

For example, to correlate with indices of a referring resource, use:
    datadog_synthetics_private_location.private_location[count.index]

Expected Behavior
The datadog-private-location-ecs module to deploy a private space in datadog and an aws ecs fargate container.
Steps to Reproduce
I have this in my main.tf and am deploying using Terraform Cloud,
terraform {
  required_providers {
    datadog = {
      source = "DataDog/datadog"
    }
  }
}
provider "datadog" {
  api_key = var.datadog_api_key
  app_key = var.datadog_app_key
}
module "components_datadog-private-location-ecs" {
  source  = "cloudposse/components/aws//modules/datadog-private-location-ecs"
  version = "1.293.2"
  region = "us-west-2"
}

Screenshots
No response
Environment
Terraform version 1.4.6 (Terraform Cloud)
cloudposse/components/aws//modules/datadog-private-location-ecs - version 1.295.0
Additional Context
I 100% recognize this may be user error and not a bug at all. I apologize in advance if it is not.
Thank you!

Are you still having this issue? can you retry with the latest component version?


Update READMEs with info on bundled transactions with/without pricing schemas

See https://github.com/oceanprotocol/ocean.py/pull/1297 and https://github.com/oceanprotocol/ocean.py/pull/1303
where we implemented bundled transactions. These are currently only showcased in tests, we should add some information in the readmes as well.

Duplicate. I forgot I already created this issue.


Diseases that share phenotypes with Influenza (MONDO:0005812)

This issue has been created to address https://github.com/NCATSTranslator/testing/issues/55.

Bulk closing testing issues; will open new as problems come up going forward.


Make sqlite3-abi valid nimble package

I have tested only:

nimble install
nimble check (to verify the package)
./update.sh

Please suggest if anything else is needed.

the most important check is to check out nimbus-eth2, run make update then check out the version you're testing in the vendor folder and make sure make nimbus_beacon_node still compiles - if that works, the change is compatible enough ;)

the most important check is to check out nimbus-eth2, run make update then check out the version you're testing in the vendor folder and make sure make nimbus_beacon_node still compiles - if that works, the change is compatible enough ;)

Thank you, I will test it out and report back.

nimbus-eth1 builds fine after https://github.com/arnetheduck/nim-sqlite3-abi/compare/844d81ba1858a4591fe53a34f776008819a7c882..b1bbe0d2d600195ced6b1f9a5b6badb47418323c


Editing policy with form inputs will not complete

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When editing a cluster admission policy from the form page the save will not complete due to having the incorrect spec property values.

However, editing from the yaml page there are no issues.
Expected Behavior
The spec should save and return you to the policies page.
Steps To Reproduce

Navigate to Kubewarden -> ClusterAdmissionPolicy
Select Edit Config from the menu of an existing policy
Try to save new config

Reference
Cause

No action was being called when Save was pressed
The input fields would change the type from an array of strings to just one string, causing the spec to be incorrect

Fix
https://user-images.githubusercontent.com/40806497/156803493-7fe7c0ee-aad2-4dd3-a5b6-eee9bcf9f8c8.mp4


Use 'vagrant' as default username.

Since this is specified in both the preseed and provisioning scripts,
it always needs to be set to 'vagrant'. Fixes plain virtualbox-iso
builds (issue #31).

Awesome - cool approach.
I tweaked this so the default username for vagrant so it matches everything else (i.e. ubuntu) - so sort of the opposite approach that you started with here. I think ubuntu makes more sense for the cloud images - it is sort an established precedent there and the base images are configured with that user already available. Even for virtualbox only and docker images - I think ubuntu makes more sense than vagrant.


Same resolution deblur model

Hey, thanks for such an interesting work. I am wondering is there any 1x deblur model, without any resolution change? I thought super-resolution could also solve the problem of motion deblur, is it the same strategy to deblur the image using 1x upscaling?

You can try prepare dataset that it's 1x size and use finetune option just change scale to 1.


Fix: add missing require for aws_sdk in instrumentation all

Added missing opentelemetry-instrumentation-aws_sdk require for opentelemetry-instrumentation-all
This PR should fix https://github.com/open-telemetry/opentelemetry-ruby/issues/1053

This might need an empty commit to "kick" EasyCLA (if only there was an easy way to do this :trollface: )

kicked the 2.5 tests, couldn't reproduce locally, seems unrelated to this PR anyway (active job test is flaky apparently)


Fix installation of module 'future' in Python 3

Can't go in main dependencies because it breaks python 2.6 due to inclusion of argsparse sigh.
Maintaining dual 2/3 support is ugly.
Nasty hack options:

postinstall script that is run when installed by pip?

It appears to work now, when installed from PyPI main (manually tested by 'pip install pyresttest' on python:3.4, 3.5 docker images).


Offline Installation Instructions is Unclear or Incomplete

Over the past four days, I've tried to install additional packages to the R Server, with no avail.
Our team has tried to install from a folder where copies of packages were located, attempting to use the CMD function, and installing through R itself.
Help would be appreciated.
[Enter feedback here]

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

ID: 40e48762-b66c-949c-4cc4-cf5f55463912
Version Independent ID: 62671bd8-8edb-4276-e855-45284f9f6b32
Content: Get R package information - SQL Server Machine Learning Services
Content Source: docs/advanced-analytics/package-management/r-package-information.md
Product: sql
Technology: machine-learning
GitHub Login: @garyericson
Microsoft Alias: garye

@TeamSwarm - I assume you're following the instructions in the article, Install new R packages - SQL Server Machine Learning Services, specifically the section "Add the package offline". What errors are you getting?

Hi Gary,
Thank you for reaching out.
In the perquisite, the installation of R and/or R Studio does not automatically point to the base R program that R Server uses. Specifically, I’ve noticed that in order to install R Server, a version of R gets installed. In our case 3.3.3 was installed. However, to execute many of the steps to install additional packages offline, R by itself needed to be installed as well. Therefore, after R is installed, it creates another instance of R, which is separate from the instance of R that the R Server uses. The result is that executing many of the instructions on the page do not actually affect R Server, just the separate instance of R.
Hope that makes sense, if not, I can clarify or restate the challenge further.
Thanks,
Sebastian Montagne
From: Gary Ericson<EMAIL_ADDRESS>Sent: Tuesday, November 19, 2019 6:15 PM
To: MicrosoftDocs/sql-docs<EMAIL_ADDRESS>Cc: Montagne Sebastian (RWY1DCV<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [EXTERNAL] Re: [MicrosoftDocs/sql-docs] Offline Installation Instructions is Unclear or Incomplete (#3638)
CAUTION! This email originated outside of the organization. Please do not open attachments or click links from an unknown or suspicious origin.

@TeamSwarmhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_TeamSwarm&d=DwMCaQ&c=3IlhnxorPWbyVbfJC_b1tA&r=urN4D445GVtw6smNZtwPBr4pxB8aToyuxWoH6As7tr4&m=EwC_AmghYjuJWbnpR74ey33L3WpL-bP2yZyqEE-_lcM&s=ghQ6hDZ7WfOpFCokPtX6xm55i-Hb3wWRTihE0UBNhW4&e= - I assume you're following the instructions in the article, Install new R packages - SQL Server Machine Learning Serviceshttps://urldefense.proofpoint.com/v2/url?u=https-3A__docs.microsoft.com_en-2Dus_sql_advanced-2Danalytics_package-2Dmanagement_install-2Dadditional-2Dr-2Dpackages-2Don-2Dsql-2Dserver&d=DwMCaQ&c=3IlhnxorPWbyVbfJC_b1tA&r=urN4D445GVtw6smNZtwPBr4pxB8aToyuxWoH6As7tr4&m=EwC_AmghYjuJWbnpR74ey33L3WpL-bP2yZyqEE-_lcM&s=OHyO5WSfYZskssqO2hbvoFBAbGUKPfThkzamMSdDbbk&e=, specifically the section "Add the package offlinehttps://urldefense.proofpoint.com/v2/url?u=https-3A__docs.microsoft.com_en-2Dus_sql_advanced-2Danalytics_package-2Dmanagement_install-2Dadditional-2Dr-2Dpackages-2Don-2Dsql-2Dserver-23add-2Dthe-2Dpackage-2Doffline&d=DwMCaQ&c=3IlhnxorPWbyVbfJC_b1tA&r=urN4D445GVtw6smNZtwPBr4pxB8aToyuxWoH6As7tr4&m=EwC_AmghYjuJWbnpR74ey33L3WpL-bP2yZyqEE-_lcM&s=vWe-7U57QoQXjt5zY5dra9jRrjyfx5NMxTw9EG1hDM8&e=". What errors are you getting?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_MicrosoftDocs_sql-2Ddocs_issues_3638-3Femail-5Fsource-3Dnotifications-26email-5Ftoken-3DAN2HWSC5VMZRKN5XHURS5T3QURXO3A5CNFSM4JPJGJC2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEQDRVY-23issuecomment-2D555759831&d=DwMCaQ&c=3IlhnxorPWbyVbfJC_b1tA&r=urN4D445GVtw6smNZtwPBr4pxB8aToyuxWoH6As7tr4&m=EwC_AmghYjuJWbnpR74ey33L3WpL-bP2yZyqEE-_lcM&s=gF-7FuvO3-KPcmKorxX0ktZvP7-i2ILJKadcTIis6Gg&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AN2HWSE6VJWC54QBJD2ON4TQURXO3ANCNFSM4JPJGJCQ&d=DwMCaQ&c=3IlhnxorPWbyVbfJC_b1tA&r=urN4D445GVtw6smNZtwPBr4pxB8aToyuxWoH6As7tr4&m=EwC_AmghYjuJWbnpR74ey33L3WpL-bP2yZyqEE-_lcM&s=uNm0loYoez0DEmZKlgLXyXHFAFpZ3Zt44jfoSjSzcmU&e=.

@TeamSwarm - Thanks for the details, I think I understand better now.
If you're using sqlmlutils (recommended for SQL 2019), then those functions basically reach over from the client computer and installing the package into the SQL instance. So the version of R on the client computer doesn't matter.
If you're using 2017 or earlier, then the idea is to either use RevoScaleR which should handle the version issues, or run R from within the folder in the instance so you're using the correct version of R. If you're using RStudio, you can tell it to use that installation of R. I've just updated the articles for 2016/2017 and I hope it's clearer now (will probably be published on Monday). Let me know if it's still not clear or if you see an issue.

#please-close, pending additional comments from @TeamSwarm


Icon not displayed even after setting LocalNotificationsImplementation.NotificationIconId

If this issue is for a new feature, please skip the "Bug" section and fill out the "Feature Request" section at the bottom.
Bug
Icon not displayed even after setting LocalNotificationsImplementation.NotificationIconId
Plugin Version:  2.1.0
Platform (e.g., Android v21):  All
Device/Hardware:  Emulator, also on Android phone with Lolipop 6.0...
Expected behavior
Should show the icon, and it used to work but for some unknown reasons it no longer works
Actual behavior
Showing a generic blank square instead
Steps to reproduce the behavior

Feature Request
Feature description:

+1

It seems that some devices render the icon so that everything that is not transparent is shown as white.

I'm also seeing a blank icon when I set it for Android. Was there any suggestions for this?

Hey, who fixed this issue?

This tool was helpful in making sure my icon was correct:
http://romannurik.github.io/AndroidAssetStudio/icons-notification.html
I had to update the icon to be a single color transparent png.

I tried your tool, but when upload image I have white square on tool :(

It's not my tool, so I can't really speak for it. Did you make sure the icon is a single color transparent PNG before putting it into the icon creator?

It's not my tool, so I can't really speak for it. Did you make sure the icon is a single color transparent PNG before putting it into the icon creator?

Working, thank you very much.


Choose the browser used to make web apps

Id like the option to choose chrome, chromium, opera or edge so i can view 4  web apps at the same time. Or be able to open multiple instances of chrome would be better of course.

@Globetrotter129 I'm going to stick with chrome for now, but I was able to do your second option in a pretty cool way! I will add this fix in the next update

@Globetrotter129 actually bro i just updated the Desktop Version, feel free to delete your pre-existing streaming website shortcuts and re-add the new ones. feel free to try, here is how you use it: basically either use your finger or use the mouse somehow, but bring it to the top of the page and swipe down. you will see an "x" at the top click it, this will enable TABS. from here you can then choose whatever website you want in the same shortcut. You can also now fullscreen any page you want, on the TOP RIGHT of chrome, click the 3 vertical dots. Once clicked, look towards the middle of the chrome options you will see this, "Zoom        - !00% + | []   <--- click that square for full screen any tabs paged you want. i hope that makes sense. lol . ill be adding this to the plugin version right now as well.


The End issues

I entered the End and it acted as if I were in space. I was dying from no oxygen and endermen were dying too, and I spawned no where near the main island. The only thing I edited from the config was edit the space station dimension ID to -11.

which version of minecraft?

I am on version 1.10.2, latest version of FTB Beyond. I made the world before adding the mod, which is version 1.10.2-1.2.1. Here is the file.

endermen suffocating and
myself dying

alright, try removing the "space" planet from the XML config and see if it occurs again.  Or you can use this modified version of the file where I've already done that.
planetDefs.xml.zip

I replaced the file, and the End no longer acts as space. I also picked up a few enderpearl drops on the ground. :p Thank you!


8413 Do not write any secrets in systemSettings

https://github.com/Shippable/pm/issues/8413
remove all passwords, keys, usernames, tokens from systemSettings table
these should either be stored in systemIntegrations or admiral.env.

also need to update the base migate.js file to remove these fields as well.

Usernames and passwords have been removed from the config columns in systemSettings.


Unable to get ferry working "Invalid value: Not in inclusive range"

Unhandled exception:
Crash when compiling file:///Users/WarHorse/Work/appname/appname/lib/services/ferry_graphql/graphql/answer_list.data.gql.g.dart,
at character offset 3899:
RangeError (offset): Invalid value: Not in inclusive range 0..1938: 3943

Been trying to migrate my GraphQL Data Layer to Ferry but I the complier crashes with this error

Did you try running pub run with --delete-conflicting-outputs?
That file is actually generated by built_value, not ferry itself. Can you share the actual failing code?

Actually ran it a couple of times just to be sure.

Also, I'm going to move this to the gql repo since it appears to be an issue with gql_build, not one of the ferry packages.

Here is the link to the full code
https://gist.github.com/arpitjacob/899bb258ff7d663f725b82c4e7aa9111
If excluding this datatype it throws the same error for other generated files as well.


Safe4337Module

We are using this address for BSC-MAINNET : https://bscscan.com/address/0x75cf11467937ce3F2f357CE24ffc3DBF8fD5c226#code
which is already 0.3.0 but we are getting this error :
Safe4337Module and/or AddModulesLib not available for chain 56 and modules version 0.2.0

This could be because you are using v2.2.1 of the safe-modules-deployment package.

then what i should do ?

How we can check the version of safe module?

we dont know our safe module version !

You can use npm list command. Source

Can we connect over a meet cause or prod is down! we are literally stuck!
Please!

Could you tell me what is the issue and any steps to reproduce that error?

Are you able to access the BNB Mainnet Safe 4337 Modules network address through this now that you have updated the version?

Thank you for your response :
Yes , wec can access our safe and version is : 1.4.1+L2 Latest version
This is what we add in the module sourcing from :
https://github.com/safe-global/safe-modules-deployments/commit/b9304bf8c4ec77cbc0831e2f3f7d4d1aae196f1d#diff-b6125fbf927e45f[…]7a3c0bc9c3fccb5ef1ceb

https://bscscan.com/address/0x75cf11467937ce3F2f357CE24ffc3DBF8fD5c226
Error: Safe4337Module and/or AddModulesLib not available for chain 56 and modules version 0.2.0

The address (0x75cf11467937ce3F2f357CE24ffc3DBF8fD5c226) you have shared is Safe4337Module v0.3.0.
The error you shared mentions v0.2.0 which does not have an official deployment made yet in BNB Chain (chain id: 56). If you want to use an older version (v0.2.0), you can go to safe-modules repo, and checkout 4337/v0.2.0 tag to deploy these.
It is recommended to use the latest version v0.3.0 of Safe4337Module with the SafeModuleSetup (instead of AddModulesLib).

It is recommended to use the latest version v0.3.0 of Safe4337Module with the SafeModuleSetup (instead of AddModulesLib).
So in this case what we need to do?

@faisal-anwar825
Have a look please!

@remedcu  if you can please tell what we have to do exactly?
upgrade the v0.3.0 of Safe4337Module so it is compatible with "address version" ?
if yes how we can do it?
please guide

Hey @Malikrehman00107, we can't help you because we cannot understand the issue. @remedcu had asked to provide the complete reproduction steps so we can help you. Please cooperate and kindly provide these, thanks

Sure !
So we deploy a safe on Binance & ETH Mainnet.
Safe BNB :bnb:0xb86b48Cfa46B3Bdf8f584B8A78B4fEEd1B54AC19
Safe ETH : eth:0x2F36fa9D3543bb2fdb3B74fc50cF3ED2269b2328
Version of both safe :  1.4.1
we are using relays using this method : https://docs.safe.global/sdk/relay-kit/guides/4337-safe-sdk
This is what we add in the module sourcing from :
https://github.com/safe-global/safe-modules-deployments/commit/b9304bf8c4ec77cbc0831e2f3f7d4d1aae196f1d#diff-b6125fbf927e45f[…]7a3c0bc9c3fccb5ef1ceb
For BNB for example we add : https://bscscan.com/address/0x75cf11467937ce3F2f357CE24ffc3DBF8fD5c226
Error: Safe4337Module and/or AddModulesLib not available for chain 56 and modules version 0.2.0
how we can solve it ?

@remedcu @mmv08 If you can please guide?

@remedcu  please!

@remedcu  please have a look !

@remedcu  please understand this is a prod issue, we need to fix it ASAP and early attention is highly appreciated please
!

Could you please use the latest release of the relay-kit<EMAIL_ADDRESS>With v0.2.0, currently the SDK does not support v0.3.0.

@faisal-anwar825


perf: improve mapObjectValues by ~50%

Overview

Improves performance of mapObjectValues by ~50% (no proper benchmark was done). This function is used when processing engine's result. For a 1000 records each having 50 related records, we go from ~32-34ms to ~11-14ms (locally). Using Object.fromEntries + Object.entries iterated the dataset twice, which can get costly on large datasets.

I don't have the rights to merge folks, so feel free to click the holy button!