Simplify keymaps

<C-M-h> does not work in gvim (<88> is displayed -- don't know whether that code is universal across keyboard layouts and locales).

Hey, can you please better describe the problem you're having and how to reproduce it?
Also, a lot of the changes were made, can you please explain this line?
cnoremap <nowait> <expr> <Esc> stridx(&cpo, 'x') < 0 ? "\<C-c>" : "\<Esc>"

Oh sorry! The <C-M-h> in Gvim is a problem that is not solved by this PR...

can you please explain this line?

Simply cmaping <Esc>d to <M-d> (and so on) will make Vim wait for a keypress when simply <Esc> is pressed, therefore, we should tell Vim not to wait when it's just <Esc>. The simple solution would have been:
" cmaps here...

cnoremap <nowait> <Esc> <Esc>

But, the above will execute the current command regardless of whether &cpo contains x or not! Therefore, we check its presence explicitly...
See :h c_Esc:

                                                    *c_<Esc>* *c_Esc*

           When typed and 'x' not present in 'cpoptions', quit
Command-line mode without executing.  In macros or when 'x'
present in 'cpoptions', start entered command.

Oh! I forgot about this... Sorry for the trouble!

Hmmm... But I don't understand your silence!

Hey @critiqjo,
sorry for not responding anytime sooner. I was super busy for the last 2 weeks.
Unfortunately that often happens on OS projects (maintainers not responding). It's not due to ignorance but because ppl are just too busy. If you want your work reviewed faster, a good strategy is to periodically post updates or "gentle" reminders.

The patch in this PR was applied in https://github.com/vim-utils/vim-husk/commit/ad89559ff8a6c4b1e94a18b2d494a69a1f701d08. I tried to give credit to you @critiqjo by mentioning you in the commit msg.
Btw. how did you find about the x in cpoptions feature? Even after reading vim's docs I still wouldn't be able to figure it out..

Oh... Coool!

Btw. how did you find about the x in cpoptions feature?

It was after I tried cnoremap <Esc> <Esc>, and suddenly all my <Esc> started executing the command... Now that's strange, and too easily found to be a bug... That's when I explored the docs...


mAdapter.notifyDataSetChanged()  invalid

List dbList= new ArrayList<>();
when I used  dbList = Bean.find(Bean.class, "sql") ;And Then  I  setAdapter to listview. then  I change the database. but when I used mAdapter.notifyDataSetChanged()  to refresh data  found  it's   invalid?
Why???

This is because you don't obtain the new results after the database has changed.
The code of the situation looks like this:
List dbList = Bean.find(Bean.class, "sql");
Adapter adapter = new Adapter(this, dbList);

//This comes magically from whatever..
ListView list = getListView();
list.setAdapter(adapter);

changeDataInDbList();

adapter.notifyDataSetChanged();

So, you make a change in the db, but you don't update the list referenced on the adapter, that's why you won't see the changes.
The correct approach would be something like this:
List dbList = Bean.find(Bean.class, "sql");
Adapter adapter = new Adapter(this, dbList);

//This comes magically from whatever..
ListView list = getListView();
list.setAdapter(adapter);

changeDataInDbList();

//You make the query again to get the dbList updated
dbList = Bean.find(Bean.class, "sql");

//If your adapter is custom, you can add a method to set the items
adapter.setItems(dbList);

//Then, this method will work as expected because you updated the list
adapter.notifyDataSetChanged();

Another approach could be the following:
List dbList = Bean.find(Bean.class, "sql");
Adapter adapter = new Adapter(this, dbList);

//This comes magically from whatever..
ListView list = getListView();
list.setAdapter(adapter);

changeDataInDbList();

//You make the query again to get the dbList updated
dbList = Bean.find(Bean.class, "sql");

//If your adapter it's not custom, you can recreate the instance, set again to the listView and trigger update
adapter = new Adapter(this, dbList);
list.setAdapter(adapter);

//Then, this method will work as expected because you updated the list
adapter.notifyDataSetChanged();

Well, that's all, hope it helps! :smile:


Pass information between step files

I have 2 single-step JSON files both of which make API calls to an external service. I have included both the files as steps in a test JSON file. I wanted to use a response field of the first API as a request field of the second API.
Is this possible?

@r5k5  yes its possible. lets say request field name is reqParamName, you could write smth like this:   reqParamNam: "${$.first_step_name.response.body.id}"    where first_step_name is the name you named your first step, and id is the parameter  that you want value from ( you should change that according to your needs, in your case it most certainly  will have  a different name).  Give it a try

Thanks, @sparrowV.


Keep track of training stats

Since this is a quick project, would be nice to skip a hefty tensorboard output. Realistically, I need the plots. However, I don't have to save anything except the final model (as full pth, or weights).

Just use the plot protocol in the dqn tutorial here, but save the plots rather than display.


upgrade to Terraform 0.14

https://trello.com/c/zZEis2fo/860-3-upgrade-to-terraform-014
Note this version introduces a lock file which fixes plugin versions, it is recommended to check this into version control see: https://www.terraform.io/upgrade-guides/0-14.html#opting-out-of-dependency-locking

I was still reviewing but up till now I only had minor suggestions around commit messages so I'll just finish combing the files, that I was in the middle of, but, as far as I can see 👍 from me.


SCC: add {Default,}AllowPrivilegeEscalation options

PTAL @simo5 @enj
Required for https://github.com/openshift/origin/pull/18398

/assign @liggitt

@liggitt PTAL

/lgtm
/approve

sudo !!

/lgtm

I assume we are now just waiting on 3.11 branches.

I've head that 3.11 is open. Can we merge this PR then?

Ping. master is open, can we proceed?

this is ok to merge. getting it into origin will require picking it into a branch in this repo pre-1.11 api-machinery, then updating origin from that branch. it looks like @bparees is already working through that process in https://github.com/openshift/api/tree/temporary-pre-3.11... I'd recommend coordinating with him.

If you want to go through that process [...] to come in before the rebase, this can merge now.

Yes, please, merge it.
I've changed the target branch from master to temporary-pre-3.11.
/retest

I've changed the target branch from master to temporary-pre-3.11.

you need to merge to master too.  The pattern is merge to master, then cherry-pick to pre-3.11.

@bparees Thanks! Rebased/updated back :)

The pattern is merge to master, then cherry-pick to pre-3.11.

Will the /cherry-pick command work for that or I have to submit another PR?

Will the /cherry-pick command work for that or I have to submit another PR?

I don't think prow is enabled for this repo but not sure.  i did mine manually.

I've also submitted https://github.com/openshift/api/pull/62


Expose search result page as RSS feed

If we expose the search result page, moderators using private chat channels could use the link to the rss feed, seeing the latest posts, without causing any information leakage compared to the alternative solution of inviting the smokey account.
Exposing the search page instead of the posts page also has its advantages that moderators could filter the information in a way they want to use it, instead of sticking with predefined filters we provide them.
This has been suggested by me and quartata before.

Doing this as part of API v2.0.

626c16f0dbf5478f61b1498b06cc0f577aa6c734


'Model' object has no attribute 'lm_head'

When I try to initiate the model, I got the error AttributeError: 'XLNetLMHeadModel' object has no attribute 'lm_head'. I tried with Xlnet-base and bert-base model from huggingface. From the readme file it seems the code will work with any model but I failed to do so. Could you please help with that?
Example Code:
tokenizer = AutoTokenizer.from_pretrained('xlnet-base-cased')
model = AutoModelWithLMHead.from_pretrained('xlnet-base-cased')

this repo is for controllable generation, since Xlnet-base and bert-base are not used for sequence generation, so it may not able to work.
You can try bart or gpt instead~

Thanks for your reply. The code worked for bart model. However, I could also make it work for xlnet-base by changing line 16 of actor.py
self.converter.weight = copy.deepcopy(torch.nn.Parameter(self.model.state_dict()['lm_loss.weight']))


Extract method not working when invoking it on a member inside a class

 Environment data

Language Server version: 2023.12.1
OS and version: win32 x64
Python version (and distribution if applicable, e.g. Anaconda):
python.analysis.indexing: true
python.analysis.typeCheckingMode: basic

 Code Snippet

class foo:
    x = 1

 Repro Steps

Create a Python file with the content above
Select x = 1
Trigger "Extract method"

It results in the following:

 Expected behavior
Maybe a failure that this can't be done at the class level?
 Logs

XXX

cc @heejaechang

There's also another example that maybe the result should be a failure message:
x = 1

If you select  x = 1 and do "extract variable", we do the following:
new_var = x = 1
new_var

for this case - https://github.com/microsoft/pylance-release/issues/5277#issuecomment-1861531993

x = 1

I think problem is extract variable is considering x = 1 as expression and not statement. extract variable should only be offered for expression.

should we create a new issue for that case since https://github.com/microsoft/pyrx/pull/4476 will fix the first one in the opening comment?

This issue has been fixed in prerelease version 2023.12.103, which we've just released. You can find the changelog here: CHANGELOG.md


[FEATURE] More tooltips in Scene Builder

Scope
I tend to forget what the different stages in the story circle mean, or what 'exciting' means. A few more tooltips explaining these terms would help a lot. Encounter types are already being explained in the scene builder, something like that would already help a lot.
Describe the solution you'd like
Tooltips and descriptions for the different stages in scene builder, and for the 'Exciting?' toggle.
Alternatives

Additional context

I second this.

Hello!
Version 4.0 of RPG Manager has just been released. This is a complete re-writing of the plugin, and will impact all opened issues.
I am therefore closing this issue as it was related to v3.4 (now unsupported), but please feel free to re-open the task if the issue persists in the new version.
Ciao
Carlo


This is a preview for the System Integration Tests.

run integration tests

@mesosphere-mergebot test-it

Closed in favour of #1994


Including Azure ML compute with existing examples. Adding network-isolated examples.

With this PR we bring multiple changes to the Azure Machine Learning examples:

We add compute and bug fixes to the 101 example
We introduce compute to the 201 examples
We add a brownfield 202 example

@grayzu @dapolloxp @ryhud @djr1991

Thanks @grayzu ! Tagging @ryhud @dapolloxp as well to look a the feedback.

@denniseik I am going to go ahead and merge this as the code looks good. I would suggest updating the quickstart/202-machine-learning-moderately-secure-existing-VNet/readme.md to show all of the required variables but will leave that up to you.

Thanks @grayzu !


android version 7.0 leads to crash ?

Steps to reproduce the behaviour

Lot of crashes, often when I tried to save a modification.

Expected behaviour no crashes
Actual behaviour crashing, not saving modifications.
Software specifications

GnuCash Android version:2.1.3
System Android version:7.0
Device type: sony xperia xz

This seems to be fixed in v2.1.4

Awesome, then I'll close this issue.
Feel free to reopen if you notice it again.


The About Category

Remove the  "About" on Nav Bar replace with
"Tools" as a category: Tool will house infrastructure, tools etx

Can you detail on what exactly will go into the tools page so I can create it.


Can't cache just the system prompt CachedContent must have at least one content

When you try to create cache with just system prompt:
    await vertex.preview.cachedContents.create({
      displayName: key,
      model,
      contents: []; // either [] or undefined, does not change the result
      systemInstruction: systemInstructions,
      ttl: `${ttlSeconds}s`
    });

You will get:
[VertexAI.ClientError]: got status: 400 Bad Request. {"error":{"code":400,"message":"CachedContent must have at least one content.","status":"INVALID_ARGUMENT"}}

Could be related?
https://github.com/firebase/genkit/pull/1297

Are there any further updates?
I'm experiencing the same issue.
I'm using a long system prompt, and I only want to set the system instructions when creating the cache, but it seems impossible with the current structure.
Thanks!


unix: simplify uv/posix.h include logic

This commit includes uv/posix.h in one #elif instead of four. Noticed while reviewing https://github.com/libuv/libuv/pull/2301.

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/1400/


[AutoPR datafactory/resource-manager] Add Dataset and CopySource for SAP HANA

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

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


Nothing has been happening recently and I am wondering how stable this is right now.

So basically a bump

Hello JuanPotato. Life took over for a bit. I have a bunch of code to push this weekend so keep an eye out :) For this exact moment, not stable at all. But again, keep an eye out over the next few days I'll close this ticket when I get something up.

Great! Glad to know that you still are here :)
I'll be waiting, may try to help out once you push. I know some python and am really waiting for this project to be finished so that I can finally move away from the bug infested tg-cli ;-;

It's been 3 weeks. I just want the latest code with some description so that I can continue this.

I might join into here aswell, see if I find the time.

Also interested from 2016.

Dang, 7 months,  this isn't getting up, but I'm slowly making a new library, github.com/libmtproto

The botapi has become the priority as I'm more interested in the bot than
the library right now. We have had some discussion how to move this forward
however. And a C++ library is not very relevant to this project.
On Sunday, February 28, 2016, Juan Potato<EMAIL_ADDRESS>wrote:

Dang, 7 months, this isn't getting up, but I'm slowly making a new
library, github.com/libmtproto
—
Reply to this email directly or view it on GitHub
https://github.com/datamachine/twx.mtproto/issues/1#issuecomment-189918562
.

Telegram also has not posted protocol docs in over a year despite they
claiming an open API so I am not happy about that. They have added dozens
of features and many layers that are completely undocumented.
On Sunday, February 28, 2016, Juan Potato<EMAIL_ADDRESS>wrote:

Dang, 7 months, this isn't getting up, but I'm slowly making a new
library, github.com/libmtproto
—
Reply to this email directly or view it on GitHub
https://github.com/datamachine/twx.mtproto/issues/1#issuecomment-189918562
.

Fun thing, I just was browsing through this at this earlier today.

@JuanPotato there is no useful Python Telegram API lib. C++ is great, but I don't want to write a wrapper for now :)
@Surye wow, you're damn right, I didn't mentioned that last update for API is at 2014.
@luckydonald how's your lib refactoring branch? :) I'm seaching for lib.

@vanch there will be bindings ;)

@vanch  I don't understand

@luckydonald oh, sorry, I made a muddle of https://github.com/griganton/telepy/tree/TL-refactor/
@JuanPotato https://imgflip.com/i/100au2

@vanch patience my child

@vanch switched to just improving an old go mtproto library, everything is working. Need to make it into an actual library though. https://github.com/JuanPotato/mtproto It works, it exists. It is being worked on. Bindings will come

Does that mean this project is back up?

It means this isn't an issue, and it's not a forum to discuss other libraries, and so it has no reason to be open.


[Bug]: CLI unable to authorize

Please confirm that you have:

[x] Searched existing issues to see if your issue is a duplicate. (If you’ve found a duplicate issue, feel free to add additional information in a comment on it.)
[x] Reproduced the issue in the latest CLI version.

In which of these areas are you experiencing a problem?
Theme
Expected behavior
when running shopify theme pull --store foo.myshopify.com
OR
when running shopify theme pull --store foo.myshopify.com --password mysecret
I would expect successful authorization and pull the current theme from client acct.
Actual behavior
actual behavior for either either cmd and cli versions is the same message.
To run this command, log in to Shopify.
╭─ error ──────────────────────────────────────────────────────────────────────╮
│                                                                              │
│  Authorization is required to continue, but the current environment does     │
│  not support interactive prompts.                                            │
│                                                                              │
│  To resolve this, specify credentials in your environment, or run the        │
│  command in an interactive environment such as your local terminal.          │
│                                                                              │
╰──────────────────────────────────────────────────────────────────────────────╯
╭─ error ──────────────────────────────────────────────────────────────────────╮
│                                                                              │
│  getaddrinfo ENOENT notify.bugsnag.com                                       │
│                                                                              │
│  To investigate the issue, examine this stack trace:                         │
│    at onlookupall [as oncomplete] (node:dns:120)                             │
│                                                                              │
╰──────────────────────────────────────────────────────────────────────────────╯
Verbose output
2024-10-24T20:49:25.052Z: Running command theme pull
2024-10-24T20:49:25.068Z: Ensuring that the user is authenticated with the Theme API with the following scopes:
[]
2024-10-24T20:49:25.068Z: Ensuring that the user is authenticated with the Admin API with the following scopes for the store foo.myshopify.com:
[]
2024-10-24T20:49:25.069Z: Getting session store...
2024-10-24T20:49:25.070Z: Validating existing session against the scopes:
[
"openid",
"https://api.shopify.com/auth/shop.admin.graphql",
"https://api.shopify.com/auth/shop.admin.themes",
"https://api.shopify.com/auth/partners.collaborator-relationships.readonly",
"https://api.shopify.com/auth/shop.storefront-renderer.devtools",
"https://api.shopify.com/auth/partners.app.cli.access",
"https://api.shopify.com/auth/destinations.readonly"
]
For applications:
{
"adminApi": {
"scopes": [],
"storeFqdn": "foo.myshopify.com"
}
}
2024-10-24T20:49:25.070Z: Initiating the full authentication flow...
2024-10-24T20:49:25.070Z: Requesting device authorization code...
2024-10-24T20:49:25.071Z: Sending POST request to URL https://accounts.shopify.com/oauth/device_authorization
With request headers:

User-Agent: Shopify CLI; v=3.69.2
Keep-Alive: timeout=30
Sec-CH-UA-PLATFORM: win32
Content-Type: application/json
Content-type: application/x-www-form-urlencoded

2024-10-24T20:49:26.363Z: Request to https://accounts.shopify.com/oauth/device_authorization completed in 1291 ms
With response headers:

cache-control: no-cache, no-store, private, must-revalidate, max-age=0
content-type: application/json; charset=utf-8
etag: W/"bdcc4cecf03f697d8dfc0dc3374c2296"
server-timing: processing;dur=129, socket_queue;dur=1.954, edge;dur=1.022, util;dur=0.05, cfRequestDuration;dur=289.000034
x-request-id: 41ccfd25-35ab-471c-9aad-a800ac6fbb57-1729802964

2024-10-24T20:49:26.372Z: Received device authorization code: {
"verification_uri": "https://shopify.com/activate",
"verification_uri_complete": "https://accounts.shopify.com/activate-with-code?device_code[user_code]=QHTQ-JQCH",
"expires_in": 599,
"interval": 5,
"device_code": "e097cbc0-d36f-4648-9691-b5a5fbb4e43d",
"user_code": "QHTQ-JQCH"
}
To run this command, log in to Shopify.
╭─ error ──────────────────────────────────────────────────────────────────────╮
│                                                                              │
│  Authorization is required to continue, but the current environment does     │
│  not support interactive prompts.                                            │
│                                                                              │
│  To resolve this, specify credentials in your environment, or run the        │
│  command in an interactive environment such as your local terminal.          │
│                                                                              │
╰──────────────────────────────────────────────────────────────────────────────╯
2024-10-24T20:49:26.447Z:
Running system process:
· Command: npm prefix
· Working directory: C:/Users/dave
2024-10-24T20:49:26.735Z: Obtaining the dependency manager in directory C:\Users\dave...
2024-10-24T20:49:27.078Z: Getting session store...
2024-10-24T20:49:27.109Z:
Running system process:
· Command: ruby -v
· Working directory: C:/Users/dave
2024-10-24T20:49:27.117Z: Getting session store...
2024-10-24T20:49:27.141Z: Request to https://monorail-edge.shopifysvc.com/v1/produce completed in 18 ms
With response headers:
2024-10-24T20:49:27.141Z: Failed to report usage analytics: request to https://monorail-edge.shopifysvc.com/v1/produce failed, reason: getaddrinfo ENOENT monorail-edge.shopifysvc.com
2024-10-24T20:49:27.141Z: Failed to report usage analytics: request to https://monorail-edge.shopifysvc.com/v1/produce failed, reason: getaddrinfo ENOENT monorail-edge.shopifysvc.com
2024-10-24T20:49:27.165Z: Reporting handled error to Bugsnag: Authorization is required to continue, but the current environment does not support interactive prompts.
2024-10-24T20:49:27.702Z:
Running system process:
· Command: npm prefix
· Working directory: C:/Users/dave
2024-10-24T20:49:27.982Z: Obtaining the dependency manager in directory C:\Users\dave...
2024-10-24T20:49:27.982Z: Getting session store...
╭─ error ──────────────────────────────────────────────────────────────────────╮
│                                                                              │
│  getaddrinfo ENOENT notify.bugsnag.com                                       │
│                                                                              │
│  To investigate the issue, examine this stack trace:                         │
│    at onlookupall [as oncomplete] (node:dns:120)                             │
│                                                                              │
╰──────────────────────────────────────────────────────────────────────────────╯
2024-10-24T20:49:28.025Z: Reporting unhandled error to Bugsnag: getaddrinfo ENOENT notify.bugsnag.com
2024-10-24T20:49:28.055Z:
Running system process:
· Command: npm prefix
· Working directory: C:/Users/dave
2024-10-24T20:49:28.368Z: Obtaining the dependency manager in directory C:\Users\dave...
2024-10-24T20:49:28.369Z: Getting session store...
Error: getaddrinfo ENOENT notify.bugsnag.com
at GetAddrInfoReqWrap.onlookupall [as oncomplete] (node:dns:120:26)
Reproduction steps

get invited as a staff member with theme and dev permissions enabled
install cli
try to pull theme from store.

Operating System
windows 11
Shopify CLI version (check your project's package.json if you're not sure)
3.69.2, 3.68.0
Shell
bash
Node version (run node -v if you're not sure)
v20.18.0
What language and version are you using in your application?
Node?

I was able to find a work around using powershell. the cli doesn't seem to work with git bash in windows.

Hi @dgordon, I can confirm the there are some known issues with git bash not being detected as a TTY, which makes it incompatible with the CLI. For windows we recommend using Powershell yes.


[WIP] Retry Bucket#objects() when invalid XML returned

Bucket#objects may throw Aws::Xml::Parser::ParsingError when S3 API returned invalid XML.
ref. http://docs.aws.amazon.com/AmazonS3/latest/API/RESTBucketGET.html
I added retry block since this exception can be retried.

対応しました


Enable splitting transcripts in chunks

This fixes infinite looping when splitting a transcript into chunks.
I asked ChatGPT for help with this one because I'm bad at off-by-one errors and this bug had the distinct smell of an
off-by-one error.
It might be good to add some tests, but that would require separating the main program from the utility functions, which is work we better save for a later commit.

Hey, sorry, I didn't really mean to submit this just yet.


nats-server: 2.2.0 -> 2.2.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/nats-io/nats-server/releases.
meta.description for nats-server is: "High-Performance server for NATS"
meta.homepage for nats-server is: "https://nats.io/"
meta.changelog for nats-server is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/drxxyvb7aw5ms982rhw6fxpirzzi62y8-nats-server-2.2.1/bin/nats-server passed the binary check.

1 of 1 passed binary check by having a zero exit code.

1 of 1 passed binary check by having the new version present in output.

found 2.2.1 with grep in /nix/store/drxxyvb7aw5ms982rhw6fxpirzzi62y8-nats-server-2.2.1

found 2.2.1 in filename of file in /nix/store/drxxyvb7aw5ms982rhw6fxpirzzi62y8-nats-server-2.2.1

directory tree listing: https://gist.github.com/41dea3f265d9f70c4ffcdb65d25d344e

du listing: https://gist.github.com/e32ad204c4d1e51ce98cbdf095f91b33

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

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
nats-server

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

gnatsd (nats-server)

Maintainer pings
cc @swdunlop for testing.

Result of nixpkgs-review pr 118524 run on x86_64-linux 1

1 package built:

nats-server


CreateStatefulContract does not work

The CreateStatefulContract does not work. I made the following changes which fixes some of the issues but still leaves the issue that the set_message call fails to update the message.

changed hello_world.json to stateful.json
changed var byteCode = ExampleHelper.parseHex(byteCodeHex); to var byteCode = byteCodeHex.getBytes();
added the following to the new ContractCreateTranscation() call: .setAutoRenewPeriod(Duration.ofHours(1)).setGas(100_000_000)
added .setGas(100_000_000) to the other transaction and query calls.

After making the above changes, it runs but still returns "hello from hedera!" as shown below:
contract bytecode file: 0.0.10566
new contract ID: 0.0.10567
contract returned message: hello from hedera!
contract returned message: hello from hedera!

I have the same issues although the fixes above helped a lot.
I tested Stateful.sol in Remix-IDE just to be sure and it was able to change the message without issue. I removed the owner requirement to see if was calling from the wrong account but that didn't solve it either. It's likely something to do with the SDK's ContractExecuteTransaction function.

Thanks Paul, this is now fixed as a result of PR #128


and some more test failures

For completeness and to check the system of #5541, #5540 and #5539, these tests fail for both scipy 0.16.1 superpack installers and my homemade build:
======================================================================
FAIL: test_correlate (test_filters.TestThreading)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\Python27\lib\site-packages\scipy\ndimage\tests\test_filters.py", line 156, in test_correlate
    assert_array_equal(os, ot)
  File "C:\Python27\lib\site-packages\numpy\testing\utils.py", line 719, in assert_array_equal
    verbose=verbose, header='Arrays are not equal')
  File "C:\Python27\lib\site-packages\numpy\testing\utils.py", line 645, in assert_array_compare
    raise AssertionError(msg)
AssertionError:
Arrays are not equal

(mismatch 84.4332%)
 x: array([[[ -0.47639956,   3.1331553 ,  10.35781802, ...,   9.18432613,
          -2.55174618,  -4.26523813],
        [ 12.99837238,  13.49555098,  -5.12679432, ...,  -8.76131474,...
 y: array([[[ -0.47639956,   3.1331553 ,  10.35781802, ...,   9.18432613,
          -2.55174618,  -4.26523813],
        [ 12.99837238,  13.49555098,  -5.12679432, ...,  -8.76131474,...

======================================================================
FAIL: test_correlate1d (test_filters.TestThreading)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\Python27\lib\site-packages\scipy\ndimage\tests\test_filters.py", line 147, in test_correlate1d
    assert_array_equal(os, ot)
  File "C:\Python27\lib\site-packages\numpy\testing\utils.py", line 719, in assert_array_equal
    verbose=verbose, header='Arrays are not equal')
  File "C:\Python27\lib\site-packages\numpy\testing\utils.py", line 645, in assert_array_compare
    raise AssertionError(msg)
AssertionError:
Arrays are not equal

(mismatch 41.12%)
 x: array([[  3.21790167,  -0.93421372,  -2.55252719, ...,  -8.14940241,
        -14.68842249, -14.25442978],
       [  3.21790167,  -0.93421372,  -2.55252719, ...,  -8.14940241,...
 y: array([[  3.21790167,  -0.93421372,  -2.55252719, ...,  -8.14940241,
        -14.68842249, -14.25442978],
       [  3.21790167,  -0.93421372,  -2.55252719, ...,  -8.14940241,...

======================================================================
FAIL: test_uniform_filter1d (test_filters.TestThreading)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\Python27\lib\site-packages\scipy\ndimage\tests\test_filters.py", line 172, in test_uniform_filter1d
    assert_array_equal(os, ot)
  File "C:\Python27\lib\site-packages\numpy\testing\utils.py", line 719, in assert_array_equal
    verbose=verbose, header='Arrays are not equal')
  File "C:\Python27\lib\site-packages\numpy\testing\utils.py", line 645, in assert_array_compare
    raise AssertionError(msg)
AssertionError:
Arrays are not equal

(mismatch 98.54%)
 x: array([[-0.63712208, -0.53668664, -0.25268181, ..., -0.74188017,
        -0.85684177, -1.13660049],
       [-0.63712208, -0.53668664, -0.25268181, ..., -0.74188017,...
 y: array([[-0.63712208, -0.53668664, -0.25268181, ..., -0.74188017,
        -0.85684177, -1.13660049],
       [-0.63712208, -0.53668664, -0.25268181, ..., -0.74188017,...

======================================================================
FAIL: test_qhull.TestUtilities.test_more_barycentric_transforms
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\Python27\lib\site-packages\nose\case.py", line 197, in runTest
    self.test(*self.arg)
  File "C:\Python27\lib\site-packages\scipy\spatial\tests\test_qhull.py", line 384, in test_more_barycentric_transforms
    unit_cube_tol=1e7*eps)
  File "C:\Python27\lib\site-packages\scipy\spatial\tests\test_qhull.py", line 311, in _check_barycentric_transforms
    assert_(ok.all(), "%s %s" % (err_msg, np.where(~ok)))
  File "C:\Python27\lib\site-packages\numpy\testing\utils.py", line 35, in assert_
    raise AssertionError(msg)
AssertionError: ndim=5 (array([7345, 7346]),)

======================================================================
FAIL: test_data.test_boost(<Data for btdtri_comp: ibeta_inv_data_ipp-ibeta_inv_data>,)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\Python27\lib\site-packages\nose\case.py", line 197, in runTest
    self.test(*self.arg)
  File "C:\Python27\lib\site-packages\scipy\special\tests\test_data.py", line 481, in _test_factory
    test.check(dtype=dtype)
  File "C:\Python27\lib\site-packages\scipy\special\_testutils.py", line 292, in check
    assert_(False, "\n".join(msg))
  File "C:\Python27\lib\site-packages\numpy\testing\utils.py", line 35, in assert_
    raise AssertionError(msg)
AssertionError:
Max |adiff|: 7.90146e-13
Max |rdiff|: 8.34159e-07
Bad results (1 out of 1210) for the following points (in output 0):
        0.00022049853578209877<PHONE_NUMBER>5               0.135563462972641 =>          6.19326407745025e-293 !=         6.193269243624209e-293  (rdiff           8.34159432765113e-07)

======================================================================
FAIL: test_continuous_basic.test_cont_basic(<scipy.stats._continuous_distns.nct_gen object at 0x02F69710>, (14, 0.24045031331198066), 'nct')
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\Python27\lib\site-packages\nose\case.py", line 197, in runTest
    self.test(*self.arg)
  File "C:\Python27\lib\site-packages\numpy\testing\decorators.py", line 215, in knownfailer
    return f(*args, **kwargs)
  File "C:\Python27\lib\site-packages\scipy\stats\tests\test_continuous_basic.py", line 380, in check_ppf_private
    npt.assert_(not np.any(np.isnan(ppfs)), msg + 'ppf private is nan')
  File "C:\Python27\lib\site-packages\numpy\testing\utils.py", line 35, in assert_
    raise AssertionError(msg)
AssertionError: nctppf private is nan

The qhull and boost failures I also see (not always reproducibly) on 32-bit Linux. I think there are other issues for them.

And the other failures I cannot reproduce with the numpy-vendor setup. Probably depends on CPU. Do you see them on your old XP system, or also under Wine?

Not on Wine, no.

good, then this is all low prio I would think.

yup, bumped the milestone

Superpack installers not produced anymore, so closing this as wontfix. If we encounter these test failures also with MingwPy, let's open new issues for them.


Tracking issue for yew 0.8.0

Required properties
Until now, there was no way to ensure that a component would receive certain properties at compile time. Components were required to handle cases where properties were not set. Furthermore, the Properties associated type must implement the Default trait.
For example, if a component had a struct as a property, you were forced to pick a default value for it or wrap it in an Option:
#[derive(PartialEq, Clone)]
pub struct GameStatus {
    pub difficulty: GameDifficulty,
    pub start: Option<Date>,
    pub end: Option<Date>,
}

#[derive(PartialEq, Clone, Default)]
pub struct Props {
    pub game_status: Option<GameStatus>,
}

This pattern forced a component to handle a default case that could have been avoided. Of course, you could always risk a runtime panic by using unwrap.
In yew 0.8, these workaround are no longer necessary since properties can be annotated as required. Yew will add a compile time guarantee that these props are present, avoiding the risk of a runtime panic. Properties must now implement the yew::Properties trait which can be derived with a macro like so:
use yew::Properties;

#[derive(PartialEq, Properties)]
pub struct Props {
    #[props(required)]
    pub game_status: GameStatus,
}

SVG support
It's now possible to render <svg> tags with Yew! This has been requested for a long time 😅. This change helped polish the html! {} macro as well since the SVG namespace uses a whole bunch of new attributes that need to be supported.

Dashed attribute and tag names are supported!
All rust keywords can now be used as attributes!

Here's a snippet of a complex svg image that works now with Yew:
html! {
    <svg width="120" height="120" viewBox="0 0 149 147" fill="none" xmlns="http://www.w3.org/2000/svg">
        <circle cx="71" cy="99" r="5" fill="white" fill-opacity="0.75" stroke="black" stroke-width="3"/>
        <defs>
            <filter id="filter0_d" x="16.3326" y="18.4918" width="118" height="118" filterUnits="userSpaceOnUse" color-interpolation-filters="sRGB">
                <feFlood flood-opacity="0" result="BackgroundImageFix"/>
                <feColorMatrix in="SourceAlpha" type="matrix" values="0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 127 0"/>
                <feOffset dy="4"/>
                <feGaussianBlur stdDeviation="2"/>
                <feColorMatrix type="matrix" values="0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0.25 0"/>
                <feBlend mode="normal" in2="BackgroundImageFix" result="effect1_dropShadow"/>
                <feBlend mode="normal" in="SourceGraphic" in2="effect1_dropShadow" result="shape"/>
            </filter>
        </defs>
    </svg>
}

Community Updates
Huge thanks to those who helped since the last release!

@totorigolo fixed self-closing tags https://github.com/DenisKolodin/yew/pull/523
It's now possible to write any tag as self-closing for brevity:
html! {

}

@tiziano88 added input event support for content editable elements https://github.com/DenisKolodin/yew/pull/549

@totorigolo also submitted a perf improvement https://github.com/DenisKolodin/yew/pull/539
There is a tracking issue for tracking benchmarking here: https://github.com/DenisKolodin/yew/issues/5 which will be completed soon. Hoping that will unlock more opportunities for those who are interested in boosting yew's performance!

@charvp added persistent class ordering which unlocks frameworks like Semantic UI https://github.com/DenisKolodin/yew/pull/424

@kellytk helped allow mounting directly to the  tag https://github.com/DenisKolodin/yew/pull/544

Breaking changes

Component::Properties associated type must implement the new Properties trait
This change enables required properties and can be derived with #[derive(Properties)]. Additionally, the Clone + PartialEq + Default traits are no longer necessary but PartialEq is recommended for efficient updates.
use yew::Properties;

#[derive(PartialEq, Properties)]
pub struct Props {
    #[props(required)]
    pub game_status: GameStatus,
 }

Callbacks no longer transform into Option types
html! {
    <Counter initial=x onclick=Msg::ChildClicked />
}

Props before:
#[derive(PartialEq, Clone, Default)]
pub struct CounterProps {
    value: u32,
    onclick: Option<Callback<u32>>,
}

Props after (note the props(required) attribute):
#[derive(PartialEq, Properties)]
pub struct CounterProps {
    value: u32,
    #[props(required)]
    onclick: Callback<u32>,
}

Changelog has been updated here: https://github.com/yewstack/yew/pull/575

Released 🎉


CAMISIM: Package up and go

Generate scripts that set the scene on Mjölnir.
I will check and see if CAMISIM exists already on Mjölnir, otherwise, i will request it to be installed.      (requested Mars 12)
Best approach: Clone the GitHub repo to mjölnir and then run housekeeper.sh to set up the data for a clean CAMISIM run.

Clone GitHub repo to Mjölnir
Run the script that downloads, unpacks, and sorts the data, so the scene is set for CAMISIM.

For example, the scripts i have created as of today (March 12th) can be combined to housekeep the data.


[Removal]: Request Removal from Denylist

Hotspot Name
Handsome Sapphire Anteater
Hotspot b58 Address
112Qqd9SJbB2jagBF4oGDCEDkR7nVUJfCRQCN1Ftyhp93UXqeUiq
Discord Handle
No response
Hotspot Manufacturer
Heltec Automation
Removal Reason
My hot spot is on the denylist. Now they have been adjusted, corrected previous mistakes, and redeployed on the spot. Hope to be removed from the denylist. Thank you very much!
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information

We do not have sufficient evidence to remove these hotspots from the denylist. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.

We have taken into account this removal request in the next iteration of the denylist. It may take some time before it's processed. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.


Job using "X-CloudTasks-TaskExecutionCount" instead of "x-cloudtasks-taskretrycount" for attempts

In v3.x of this package, the task handler (TaskHandler.php:98) sets the header X-CloudTasks-TaskExecutionCount as the jobs attempt count. But according to the documentation, the execution count is only incremented when the task was processed, returned a response in time, and the response code is not in the 5xx code range.
In cases where the task execution returns 5xx, the retry count is incremented, but the execution count is not.
Those cases might lead to failed tasks not being stored in the failed jobs table, because the job still has attempts < maxAttempts.
Task not being stored in the failed jobs table can be reproduced by creating a job which simply does
'abort(500)' vs a job that does 'abort(404)'

Thanks a lot for the bug reports! I have fixed both in v3.1.4.


Poor tooltip name and description for Distance and Direction addin (ArcMap)

When I hover over the Distance and Direction addin the following tooltip is displayed:

"Show Distance and Direction" - there is no reason to use the verb "Show". Users know that the tool will open or do something.
"Add-in command to show Distance and Direction." - This doesn't help the user at all. We should be describing what the user can do with the addin. Look at other tools and see how they describe the tool.

Here is the description part of the tootip that we can use:
"Interactively create geodetic lines, circles, ellipses, range rings, and distance calculations based on time and rate. Created graphics can be saved to feature classes, shapefiles, and KMZ files."

PR merged, waiting to see if it gets into the newest builds.

@pHill5136
Not seeing the change in 10.4.1 from today's build 1067.

@mfunk @pHill5136 @nfeuerstein @lfunkhouser - the tool tips for all three should have similar style & content. Right now Visibility's is the same, and Coordinate Conversion's tip is "Displays Coordinate Conversion" - maybe a little better, but not as descriptive as @elinz's suggestion.

Checked again and still seeing above screenshot in 10.4.1 as of today's build.

Verified.


Add rustfmt.toml with formatting disabled

Currently, Cargo does not use rustfmt for its source code. By adding a rustfmt.toml that simply disables all formatting rules, editors which use rustfmt by default for all Rust code will do the right thing and leave the source code unchanged,.
This makes life a little bit easier for developers who no longer need to explicitly disable automatic rustfmt formatting when working on Cargo to avoid code unrelated to a change being reformatted.

r? @matklad
(rust_highfive has picked a reviewer for you, use r? to override)

I know a PR with a bare-bones rustfmt.toml file (it set the maximum line size) was rejected previously. I figured this might be acceptable since it disables rustfmt formatting, which means running rustfmt on Cargo will do the right thing, as opposed to the current situation where it will reformat a lot of code.

@bors r+
Thanks!

:pushpin: Commit 0c7d975 has been approved by matklad

:hourglass: Testing commit 0c7d975ceb668915116c99259c7bb19875824fba with merge a924c5a3ead658666f9a81c3804ddeaedb3a67ce...

:sunny: Test successful - status-appveyor, status-travis
Approved by: matklad
Pushing a924c5a3ead658666f9a81c3804ddeaedb3a67ce to master...


Allow limiting the history

Is your feature request related to a problem? Please describe.
I want to automatically create the changelog for a github release. As information, I do have the current tag only.
Describe the solution you'd like
I would like to have a way to limit the generation up until (excluding) the previous tag.
So assuming I am running as the release, and have the current release in $TAG. I would like to run e.g:
convco changelog ..$TAG --max-tags 1

Which would limit the output to one tag, starting with the tag provided.
Describe alternatives you've considered

Actually query the GitHub releases API: that might be more correct than just checking for the "previous" tag, but also more complicated and limited
Additionally there could be an "ignore refs" pattern, which could allow ignoring tags in the process of counting tags.

Additional context
None

Thanks for the suggestion. For checking we have a --max-count, but it counts the commits, not the tags.
For the changelog it could indeed be good to limit the number of versions, but probably it should be --max-majors --max-minors --max-versions.
For now if you would limit on a tag boundary you could use:
convco changelog $(git tag --sort v:refname -l 'v*' | tail -n5 | head -n1)..$TAG

which would create the changelog for $TAG and 3 other versions.
Ignoring a version is more complicated, e.g if you would ignore v2.0.1 should those changes be visible under v2.0.2 or not at all?

I like the proposal, and thanks for the alternative to try out.

Ignoring a version is more complicated, e.g if you would ignore v2.0.1 should those changes be visible under v2.0.2 or not at all?

A good question. I was simply assuming it would indeed end up under the 2.0.2 in this case. The case I had in mind was, when the some other mechanism (like helm charts) create tags on the repository as well. Those would have a different format though. So you could say v.* is included in counting, all others are not (like charts-2.0.2). So it would not stop, but add more commits to the processing. Until it reaches e.g. v2.0.1.

convco is only walking between tags with the prefix (v by default).
so the other tags are ignored. I'll try to add a --max-versions flag this week as that would be closest to your initial request. the other flags can be added later

Would you mind testing the branch max-versions?

Would you mind testing the branch max-versions?

Works like a charm!

I'll close this issue for now.
To hide a version, i think it is better just to untag it.
for the max-majors, max-minors, I think it is better to add a flag to convco version to select major -<n> or minor -<n> where it would print out that version. This way it could be used to create the lower limit convco changelog "$(convco version --major -1)..". A [version req](https://docs.rs/semver/latest/semver/struct.VersionReq.html) filter could be added to changelog`.


Bump to lts 20.19

I'll take a look when I'm home next week, I think I have merge rights

I note that my previous review comment hasn't been addressed. What is it that people actually need? Are some of our bounds too restrictive? There's a whole bunch going on in this PR and it would be good to unpick

Folq folks (@agevelt @hanshenrik): do you want to keep these packages (elm-syntax, haskell-to-elm, servant-to-elm) under @folq?
If not, I suggest we create a new 'haskell-to-elm' Github organization with some volunteers (preferably active users of the libraries) and move development there.
If you would like to keep them, can you please maintain them (keep track of and merge PRs etc) or ensure that someone who would like to maintain them has commit access on Github?
I am a maintainer for these packages on Hackage, and I can hand out maintainer bits to whoever ends up maintaining these libraries in the end (Folq or others). Notably haskell-to-elm is missing maintainers other than myself.

This PR slipped my mind, sorry about that.
Moving these to a different org is a good idea. Would you like to create the org and then I can transfer the repos @ollef?
Meanwhile I'll bump the prettyprinter bounds and merge this. I think I'll drop the version bump as well, and leave that to a separate PR.

Thanks @agevelt! Development has now moved to https://github.com/haskell-to-elm and I've invited past contributors there. There's still some setup to get sorted, but these libraries should hopefully soon be on track again.

That would be great @vladimirlogachev! Do you have the permissions to do it or is there anything I need to change?


Does not work with jsRender

I tried using this color picker with jsRender but it doesn't work.
I have the input tag in a jsRender template, but the color picker doesn't show. Is it because the content is loaded dynamically?

I don't know exactly how jsRender works but there may be some way to make it work with jQuery plugins that are rendered on the DOMContentLoaded event, like this one.
Isn't there a way to initialize this plugin after the template is rendered and the HTML is available on the DOM?

You also could provide an example using JSFiddle or similar, as it is specified here (and when you create a new issue) https://github.com/mjolnic/bootstrap-colorpicker/blob/master/CONTRIBUTING.md


[UNDERTOW-1935] - buffer leak on incoming websocket PONG message

Just a backport of 2.2.x and 2.0.x branches https://github.com/undertow-io/undertow/commit/c7e84a0b7efced38506d7d1dfea5902366973877

@bruno-durand  hi! Thanks for your PR! We are no longer mantaining 2.1.x branch, though.


[stm32] ADC channel selection wrong

Found something that indicates that all the channel selections of the ADCs are completly wrong.
In the PAC of my L0:
impl_adc_pin!(ADC1, PA4, 4u8);

The implementation of the adc pin:
macro_rules! impl_adc_pin {
    ($inst:ident, $pin:ident, $ch:expr) => {
        impl crate::adc::AdcChannel<peripherals::$inst> for crate::peripherals::$pin {}
        impl crate::adc::SealedAdcChannel<peripherals::$inst> for crate::peripherals::$pin {
            #[cfg(any(adc_v1, adc_l0, adc_v2, adc_g4, adc_v4))]
            fn setup(&mut self) {
                <Self as crate::gpio::SealedPin>::set_as_analog(self);
            }

            fn channel(&self) -> u8 {
                $ch
            }
        }
    };
}

the read function in mod.rs
    pub async fn read(&mut self, channel: &mut impl AdcChannel<T>) -> u16 {
        let ch_num = channel.channel();
        channel.setup();

        // A.7.5 Single conversion sequence code example - Software trigger
        T::regs().chselr().write(|reg| reg.set_chselx(ch_num as usize, true));

        self.convert().await
    }

So when passing &mut PA4 to the read function, the channel written will be 4 - but it should be 1 << 4  = 16
But the datasheet says:

This is so off I can't believe that it's really wrong and I'm expecting I made a mistake.
What have I done wrong? :thinking:
edit: the STM32 HAL Header file containes:
#define ADC_CHSELR_CHSEL4_Pos      (4U)
#define ADC_CHSELR_CHSEL4_Msk      (0x1UL << ADC_CHSELR_CHSEL4_Pos)             /*!< 0x00000010 */
#define ADC_CHSELR_CHSEL4          ADC_CHSELR_CHSEL4_Msk                       /*!< Channel 4 selection */

The ADC examples use e.g. ADC_CHSELR_CHSEL4 for chsel which is not the pin number but the _Msk.

So this
 fn channel(&self) -> u8 {
                $ch
}

should actually be:
 fn channel(&self) -> u8 {
                1 << $ch
}

Ah found it! :man_facepalming:
        pub fn set_chselx(&mut self, n: usize, val: bool) {
            assert!(n < 19usize);
            let offs = 0usize + n * 1usize;
            self.0 = (self.0 & !(0x01 << offs)) | (((val as u32) & 0x01) << offs);
        }


Update form_builder_field.dart

Connection with issue(s)
Close #1348
Connected to
Solution description
To avoid breaking changes, implemented valueIsValid and valueHasError getters on FormBuilderFieldState which perform the behavior of the overridden isValid and hasError getters of FormFieldState.
To Do

[x]  Read contributing guide
[x]  Check the original issue to confirm it is fully satisfied
[x]  Add solution description to help guide reviewers
[x]  Add unit test to verify new or fixed behavior
[x]  If apply, add documentation to code properties and package readme

@deandreamatias This issue relates to the overrides of isValid and hasError that you implemented 8 months ago. In doing so, the package seems to have lost the ability to assess the validity of an input's value independent of the errorText.
This pull request reintroduces the ability to assess the validity of an input's value independent of the errorText while preserving the overrides you wrote to ensure there are no breaking changes.

Ran dart format .


Fix e2e tests by following up on breaking change in lookup step

Checklist

[x] I've read and followed the Contribution Guidelines

Context
Breaking change has been introduced in the steps-lookup-xcode-simulator

Triumphed by https://github.com/bitrise-steplib/steps-xcode-test/pull/185


New feature: login.

The app has been updated to provide a working set of features to allow
the user to log in into Emotly.
No persistance has been implemented yet, though.

LGTM.


Move the charts on Artifact Hub to Camunda org

Currently, Camunda Platform Helm charts on Artifact Hub are under @Zelldon user.
For better structure, we need to move the charts on Artifact Hub to Camunda org/repo.
We need to do the following:

[x] Create camunda org (it was already created for Camunda Platform v7 chart).
[ ] Invite @Zelldon to the Camunda org.
[ ] Transfer camunda-platform to Camunda org.
[ ] We could also rename the repo to camunda.

And to avoid confusion, we also need to do some changes in the camunda-community-hub/camunda-helm:

[ ] Rename the camunda repo to camunda-community-hub.
[ ] Rename the community hub sub domain helm.camunda.cloud to helm-cch.camunda.cloud.

Nice to have:

[ ] Add the Official Badge to our Helm chart page in Artifact Hub.

We could also rename the repo to camunda.

Which repo you want to rename?

We could also rename the repo to camunda.

Which repo you want to rename?

This one camunda-platform.
The chart URL in AH shows as:

https://artifacthub.io/packages/helm/[REPO]/[CHART]

So instead of:

https://artifacthub.io/packages/helm/camunda-platform/camunda-platform

by renaming the repo to camunda we will have:

https://artifacthub.io/packages/helm/camunda/camunda-platform


Add missing notebooks to notebook CI stage

A notebook test CI stage was added in #308, but not every notebook in the repo is being tested yet. The ones still missing involved larger models or datasets, but to prevent them from going stale we should still add them, if not to CI builds for every PR then at least to a cron job:

[x] dynamical_intro.ipynb (blocked by #396)
[ ] deepscm.ipynb (may be blocked by #179)
[ ] sciplex.ipynb
[ ] automated_dr_learner.ipynb
[x] instrumental_var.ipynb #555

Adding checkmark for #555


Lava buckets not leaving behind bucket

When Lava Buckets are used in forge controllers as fuel, they consume the whole bucket instead of leaving behind the bucket like the furnaces

Hi, can you provide more details?
For example what version of the mod are you using, and are you using any other mods?

Hi Noaaan, I have been trying to reproduce the bug but it doesn't seem to reoccur. My server has been a bit glitchy and have been trying to make it more stable, I highly apologise for the inconvenience. I will report again if I am able to reproduce it.


[Discussion]: Duplication / overlap of efforts with atomate2

Email (Optional)
No response
Problem
Some of the workflows / tasks defined in matcalc and atomate2 for ML interatomic potentials are nearly identical. Part of this is because the initial implementation of the relaxation forcefield class was based on matcalc / matgl. However, our efforts diverged after that, but led to similarly structured workflows (forcefield-agnostic requiring only a common implementation of a calculator-like object).
In particular, I see overlap with relaxation, EOS, and quasi-/harmonic phonon flows. atomate2 has a few extra jobs/workflows (MD, Grüneiesen) and will eventually support NEB as matcalc currently does.
Obviously the intended purposes of the two codes are different: atomate2 is meant for modular high-throughput atomistic workflows, and matcalc is more for lower-throughput calculation of specific properties with ML forcefields.
Still, I think it's good to make sure we're not duplicating effort between projects where there's no need to (e.g., workflow schemas / data schemas). Copying @shyuep and @utf in case you have thoughts on this
Proposed Solution
Further discussion between the two teams
Alternatives
No response
Code of Conduct

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

I think this is somewhat similar to the VASP input sets problem - there is the actual logic for computing a property (e.g., phonons, EOS, elastic constants) and those should be in matcalc. And there is the workflow stuff, which is in atomate2. Atomate2 can just use matcalc as a dependency and call the appropriate property calculators to get the properties. I should add that matcalc has far more properties than atomate2 at the moment and more will be added.

I'd be very open to this if it can be done cleanly. Having only skimmed the source for matcalc I'm not clear how that would actually work in practice. If you have any suggestions @esoteric-ephemera I'd be open to that.
I might have missed something but from what I can tell atomate2 has basically the same number of properties as matcalc (the NEB workflow isn't merged yet but we also have anharmonic phonons, klatt etc).

As far as I know, atomate takes an input structure, perform a calculation and then returns the result. That is precisely what matcalc does. Also, almost all those properties were on matcalc since the 0.0.4 release on Jan 1 2024. Only a few things have been added since.


fix expected return code for block / unblock user

According to the API documentation, a 201 OK will be returned
upon success for the block / unblock user call.
See https://docs.gitlab.com/ce/api/users.html#block-user

Thanks @michaellihs! LGTM!


Crashes Atom

Very nice plugin but I am experiencing crashes/hangs and I have to close the current atom editor/window.
Let me know if there is anything I can do to track this down.

Please update to the latest version of atom and let me know if things are better now.

I've been using it for the last few days without issues. Closing.


How to use the tydiqa dataset

I'm confused with this because it doesn't seem to have a test set and the paper doesn't say which other dataset was used as a test set. The dev set also does not seem to be labelled. Furthermore, I don't see how to use tydiqa-goldp-v1.1-train.json together with the other training set files. Can someone help please?

Hi Marc, the Gold Passage version of TyDiQA does not have separate dev and test sets, so the dev sets are used for testing. This is also why they are not labelled as we remove their annotations.
When using the download_data.sh script, this line calls the utils_preprocess.py script, which is responsible for splitting the tydiqa-goldp-v1.1-train.json file, which consist data in all languages into language-specific files. If you check your download folder, you should find a tydiqa-goldp-v1.1-train folder within the tydiqa folder that contains the language-specific train files.
Let us know if you have any further questions.


🐞 [Bug]: Deposit TFT not working properly

Is there an existing issue for this?

[X] I have searched the existing issues

which package/s did you face the problem with?
Dashboard
What happened?
I tried to deposit a TFT from my tf connect app via the QR code. Although the amount got deducted, the balance was not reflected in the Dashboard.
Steps To Reproduce
No response
which network/s did you face the problem on?
Main
version
2.5.2
Twin ID/s
No response
Node ID/s
No response
Farm ID/s
No response
Contract ID/s
No response
Relevant screenshots/screen records

Relevant log output
-

works fine with me on test and main net

Verified on Mainnet
version: 2.5.2
The transaction has been done successfully

Testcase


arm64 via raspberry pi 3 does not display memory values

arm64 via raspberry pi 3, ubuntu 20.04. Does not display container memory values. I thought that this problem would be solved by a complete reinstallation of ubuntu, but everything remained the same.

Please confirm you have memory cgroups enabled; see https://github.com/bcicen/ctop/issues/45

It was my fault. I fixed it and now everything works fine.


Pools: add missing platforms, checksum addresses, MDex -> Mdex etc

validatePools.js: add check for missing platform or tokenDescription in *_pools.js
validatePools.js: add check for checksummed addresses in *_pools.js ('tokenAddress', 'earnedTokenAddress', 'earnContractAddress')
validatePools.js: do not skip checking the rest of the pool fields for venus-wbnb
validatePools.js: add --platform-summary cli argument (see below)
remove zero-width spaces from DRUGS token in bsc_pools.js
use 'Mdex' rather than 'MDex' in bsc_pools.js
use checksummed addresses (avax: 5, bsc: 35, heco: 1, polygon: 5) in *_pools.js
add missing platform: keys (bsc: 97, heco: 1, fantom: 1) in *_pools.js

yarn run validate --platform-summary will do usual checks but also output a list of platforms per chain (useful for checking they are consistently named)
e.g.
Validating 22 pools in fantomPools...
Platforms:
        Beefy.Finance (1)
        Ester (2)
        Froyo (2)
        SpookySwap (17)
Active pools: 22/22

adds @ethersproject/address as dev dependency

can we revert all bsc_pools platforms please? 😅
i intentionally removed most of them (all degens) where we have only 1 token + bnb/busd to not show in filter.
i dont think we need there ramen, salt, yieldbay etc, they can be filtered by token name.
also all new 'Pancake' are not really Pancake platform (they just use Pancake LPs which we set in token description), only PCS farms are Pancake platform

@roman-monk Is there a reason we don't want some platforms appearing in the UI? (Surely if we farm the platform is should appear?)
If we really do need a way to exclude some platforms from the filter (/only include a set list); can we come up with another way to do so other than omitting the platform: key? I'd like the validatePools script to force this to be filled in as some pools that definitely should be in the filter have been added without one.
Re: Pancake - I'll go through them and try figure out what the platform should actually be.

@roman-monk Is there a reason we don't want some platforms appearing in the UI? (Surely if we farm the platform is should appear?)

yes, it doesn't help users when they see long list of platforms most of them they've never heard about and very likely not interested.
i suggest to just remove all platform that you've added and do not validate them.
there was only 1 case when we lost platform - single Mdex.
except this we didn't have any issues, that's why i think we can resolve them manually

can suggest "Degen" platform as a compromise 😄

@roman-monk Maybe just 'Other' ?
BillionHappiness (2)

@roman-monk Maybe just 'Other'?
Can we confirm we don't want to keep any of the following? Happy to replace them all with 'Other'.
BillionHappiness (2)
CafeSwap (3)
Complus (1)
ForTube (5)
FryWorld (2)
Marshmallow (6)
Narwhal (1)
Nyanswop (1)
PantherSwap (2)
Pumpy (1)
RamenSwap (2)
SaltSwap (2)
SlimeFinance (2)
Sponge (3)
Squirrel (1)
ThunderSwap (2)
Traphouse (18)
YieldBay (3)

Can we confirm we don't want to keep any of the following? Happy to replace them all with 'Other'.

would be great 👍

Global platform: changes
As discussed above
PantherSwap -> Other
BillionHappiness -> Other
Marshmallow -> Other
YieldBay -> Other
ThunderSwap -> Other
Pumpy -> Other
Complus -> Other
SlimeFinance -> Other
Sponge -> Other
SaltSwap -> Other
RamenSwap -> Other
CafeSwap -> Other
Nyanswop -> Other
Traphouse -> Other
Narwhal -> Other
ForTube -> Other
FryWorld -> Other
Vault platform: changes
hps-hps-bnb: JulSwap -> Other (JulSwap LP but farm is on BillionHappiness)
Vault tokenDescription: changes
Changed to follow the LP Provider (Farm Provider) format used elsewhere e.g. Mdex (Auto)
garuda-garuda-usdc: Pancake -> Pancake (Garuda)
grand-grand: Pancake -> Grand
grand-grand-bnb: Pancake -> Pancake (Grand)
dumpling-sdump-bnb: Pancake -> Pancake (Dumpling)
dumpling-sdump-busd: Pancake -> Pancake (Dumpling)
satis-xsat-busd: Pancake -> Pancake (Statis)
satis-xsat-bnb: Pancake -> Pancake (Statis)
garuda-garuda-usdt: Pancake -> Pancake (Garuda)
ica-ica-btcb-eol: Pancake -> Pancake (Icarus)
ica-ica-eth-eol: Pancake -> Pancake (Icarus)
zefiv2-zefi-bnb: Pancake -> Pancake (ZCore)
zefiv2-zefi-busd: Pancake -> Pancake (ZCore)
satis-sat-bnb-eol: Pancake -> Pancake (Statis)
satis-sat-busd-eol: Pancake -> Pancake (Statis)
typhv2-typh-bnb: Pancake -> Pancake (Typhoon)
typh-typh-bnb: Pancake -> Pancake (Typhoon)
biti-biti-bnb: Pancake -> Pancake (Biti)
bingo-bgo-busd: Pancake -> Pancake (Bingo)
bingo-sbgo-busd: Pancake -> Pancake (Bingo)
swirl-swirl-bnb: Pancake -> Pancake (Swirl)
zefi-zefi-bnb: Pancake -> Pancake (ZCore)
zefi-zefi-busd: Pancake -> Pancake (ZCore)
hps-hps-bnb: JulSwap -> JulSwap (BillionHappiness)
squirrel-nuts-bnb-eol: Pancake -> Pancake (Squirrel)
memefarm-ape-bnb: Pancake -> Pancake (MemeFarm)
memefarm-mfrm-bnb: Pancake -> Pancake (MemeFarm)
soup-soup-bnb: Pancake -> Pancake (Soup)
soup-soups-bnb: Pancake -> Pancake (Soup)
crow-crow-busd: Pancake -> Pancake (Crow)
crow-crow-bnb: Pancake -> Pancake (Crow)
monster-msc-busd: Pancake -> Pancake (Monster)
monster-msc-bnb: Pancake -> Pancake (Monster)
cake-bts-busd: Pancake -> Pancake (Bolt)
cake-btd-busd: Pancake -> Pancake (Bolt)
monster-mss-busd: Pancake -> Pancake (Monster)
cake-bdo-bnb-sbdo: Pancake -> Pancake (bDollar)
cake-bdo-bnb-old: Pancake -> Pancake (bDollar)
cake-sbdo-busd: Pancake -> Pancake (bDollar)
cake-bdo-busd: Pancake -> Pancake (bDollar)

Via Discord:
Pancake -> PancakeSwap in tokenDescription and platform
Auto -> Autofarm in tokenDescription


Native review for Fresheyeball/elm-http-server

Simple bindings to Node.js's Http.Server as a Signal (Request, Response)
Basically this package is a starting place where low level api may expand. Ideally people build routing libraries and such on-top. Personally, this is a step to isomorphic templating in Elm.

The native review pipeline is stalled. That's probably not news to you, but I wanted to leave a note on all the open reviews. I cannot speak for everyone, but as one of the reviewers I just want to speak up and say that I will not be doing more reviews. The warning about publishing native packages has changed, and Evan has mentioned multiple times in different places that one of his priorities for 0.17 is improving this process. So I'm going to wait for that. I'm sorry we cannot move forward with publishing your library, but my experience is that even if we review and approve it, it will become stuck waiting for Evan to whitelist it.

Sorry to keep this in limbo for so long. I am moving towards the plan outlined here and discussed a bit further in the 0.17 announcement.
I don't really want to go towards node or anything in that ecosystem really. I say a bit more about that at http://package.elm-lang.org/packages/elm-lang/core/latest/Process


Can not use browsers to browse TestServer

I use the following code to start a TestServer and keep it running:
TestServer server = TestServer.Create(app => app.UseFileServer());
Thread.Sleep(20000);
Then I enter "http://localhost/index.html".
The browser shows "can not connect to server".
server.BaseAddress.Port is 80.
How to fix it?

This is by design. TestServer is for unit testing components in memory, it does not open any real network connections. HttpRequestMessages (HttpClient) are converted directly to HttpContext objects   (Asp.Net) without being sent over a socket.
Here is a usage example:
https://github.com/aspnet/BasicMiddleware/blob/dev/test/Microsoft.AspNet.HttpOverrides.Tests/OverrideHeaderMiddlewareTest.cs#L21
If you would like to test using the browser then you need to use a real server like WebListener, Kestrel, or IIS instead.

I want to build unit tests to test a JavaScript library.
First I start a local server, then use Selenium to do web browser automation, then do some assertion.
I want all these code within an xUnit testcase.
My question is: Can I start a "real" server within an [Fact]? And how?

@Tratcher Will get you an example, AFAIK it is possible to do what you want.

You can do that with WebListener or Kestrel. Try this:
using System;
using System.Collections.Generic;
using System.Net;
using System.Net.Http;
using System.Threading.Tasks;
using Microsoft.AspNet.Builder;
using Microsoft.AspNet.Hosting;
using Microsoft.AspNet.Http;
using Microsoft.Dnx.Runtime.Infrastructure;
using Microsoft.Framework.Configuration;
using Xunit;

namespace ClassLibrary1
{
    public class TestClass
    {
        [Fact]
        public async Task MyTest()
        {
            string address = "http://localhost:8080";
            using (CreateServer(address, app =>
            {
                app.Run(ctx =>
                {
                    return ctx.Response.WriteAsync("Hello World");
                });
            }))
            {
                var response = await SendRequestAsync(address);
                Assert.Equal(HttpStatusCode.OK, response.StatusCode);
                Assert.Equal("Hello World", await response.Content.ReadAsStringAsync());
            }
        }

        public async Task<HttpResponseMessage> SendRequestAsync(string uri)
        {
            using (HttpClient client = new HttpClient())
            {
                return await client.GetAsync(uri);
            }
        }

        public IDisposable CreateServer(string address, Action<IApplicationBuilder> app)
        {
            var builder = new ConfigurationBuilder();
            builder.AddInMemoryCollection(new[]
            {
                new KeyValuePair<string, string>("server.urls", address)
            });
            var config = builder.Build();

            var hostBuilder = new WebHostBuilder(CallContextServiceLocator.Locator.ServiceProvider, config);
            hostBuilder.UseStartup(app);
            hostBuilder.UseServer("Microsoft.AspNet.Server.WebListener");
            return hostBuilder.Build().Start();
        }
    }
}

And project.json:
{
  "version": "1.0.0-*",
  "commands": {
    "test": "xunit.runner.aspnet"
  },

  "dependencies": {
    "Microsoft.AspNet.Server.WebListener": "1.0.0-*",
    "xunit.runner.aspnet": "2.0.0-aspnet-*",
    "System.Net.Http": "4.0.0"
  },

  "frameworks": {
    "dnx451": { },
    "dnxcore50": { }
  }
}

Thanks. I tried this code and got a Runtime Error at CreateServer.
After debugging, I find that CallContextServiceLocator.Locator is null.
Could you help me to fix that?

Hmm, which version of DNX are you running it on? It worked fine on beta6.

@JiajieBao are you on mono?

Fixed. Looks like I used some incompatible references. Thanks.

Hello, I've read the Sep 10th webdev blog. It says,

Kestrel and DNX are the only server and host processes that will be shipped.
WebListener will not be shipped.

and

Question: What if I wanted to host kestrel in my own process?
There is no self-host model.  Dnx is a hosted model, where you pass in the application that you want dnx to host.  This is similar to node where you execute node and pass in the application that you want node to run.

So here is my question: since WebListener will not be 'shipped', and kestrel doesn't have a self-host model, how do we achieve an real test server demonstrated by @Tratcher above? Thanks.

Change this line:
hostBuilder.UseServer("Microsoft.AspNet.Server.Kestrel");

@davidfowl
Thanks. What happens when changing to hostBuilder.UseServer("Microsoft.AspNet.Server.Kestrel");? Will it spin up a new process for kestrel? If yes,when will the external process will be shut down?

No, it starts kestrel in process just like WebListener. Try it.

That is what I want :)
But what does the blog mean by There is no self-host model for kestrel? A bit confused, as the code here seems to be a self-hosted model.

The blog might be wrong :smile: I'll check the wording.

It has been a year since this answer. Is there any new information on how someone can do browser testing with tools like Selenium? This is what I currently have, and it works, but I don't know how correct it is or if there is a better way.
public abstract class SeleniumTests : nspec
{
    private IWebHost host;
    protected IWebDriver driver;

    void before_each()
    {
        host = new WebHostBuilder()
                        .UseKestrel()
                        .UseContentRoot(GetContentRoot())
                        .UseStartup<Startup>()
                        .Build();

        host.Start();

        var ieService = InternetExplorerDriverService.CreateDefaultService();
        ieService.SuppressInitialDiagnosticInformation = true;

        driver = new InternetExplorerDriver(ieService);
    }

    void after_each()
    {
        driver.Dispose();
        host.Dispose();
    }

    private static string GetContentRoot()
    {
        var solutionRoot = new DirectoryInfo(PlatformServices.Default.Application.ApplicationBasePath) // netcoreapp1.1 folder
            .Parent // Debug or Release folder
            .Parent // bin folder
            .Parent // TSC.Services.Authentication.AcceptanceTests folder
            .Parent // test folder
            .Parent.FullName; // root folder

        Console.WriteLine(PlatformServices.Default.Application.ApplicationBasePath);

        return Path.GetFullPath(Path.Combine(solutionRoot, @"src\TSC.Services.Authentication.Api"));
    }
}

FWIW we have created a simple browser-like user agent to use when testing with the TestServer: https://brockallen.com/2016/01/01/browsr-helper-for-integration-testing-in-asp-net-5/, and here's the current version in use: https://github.com/IdentityServer/IdentityServer4/blob/dev/test/IdentityServer.IntegrationTests/Common/BrowserClient.cs

@brockallen Thank you for the links. Its actually IdentityServer4 that I am currently testing against. That looks like it will be helpful for API calls, but does it allow for actual browser viewing of the HTML? I'm using Selenium for functional/user acceptance testing to show the UI is connected up.

Correct -- there's not real "viewing" or the HTML. It's mostly just to make requests that act like a browser in terms of cookie container and following redirects.


Multipart handling is very confusing when files are involved; can the docs be clarified?

Description
Multipart handling is very confusing in Connexion 3.x.  I found that:

If a client passes a non-file object to something which is expected to be a file, then the object winds up in the body parameter as a string.
If a developer declares a file-like parameter to be required, and makes it a positional argument (makes sense?!?), but the client passes a non-file under that parameter name, then the code will fail, because it will fail to supply a positional parameter.

Expected behaviour
The following code illustrates what seems like strange (or at least, not-documented) behavior:
openapi: "3.0.1"

info:
  title: Form Data
  version: "1.0"
servers:
  - url: /openapi

paths:
  /greeting:
    post:
      summary: Generate greeting
      operationId: hello.post_greeting
      requestBody:
        content:
          multipart/form-data:
            schema:
              type: object
              properties:
                name:
                  type: string
                selfie:
                  type: string
                  format: binary
              required:
                - name
                - selfie
      responses:
        200:
          description: OK

from pathlib import Path
import connexion
from starlette.responses import PlainTextResponse, Response

async def post_greeting(body, selfie=None) -> Response:
    return PlainTextResponse(f"Hello body={body} selfie={selfie}!", status_code=201)

app = connexion.AsyncApp(__name__, specification_dir="spec")
app.add_api("openapi.yaml", arguments={"title": "Hello World Example"})

if __name__ == "__main__":
    app.run(f"{Path(__file__).stem}:app", port=8080)

Actual behaviour
# pass non-file for selfie; required param validation still passes, but selfie param is "None"
$ curl -F name=dave -F selfie=nah http://localhost:8080/openapi/greeting
Hello body={'name': 'dave', 'selfie': 'nah'} selfie=None!

$ curl -F name=dave -F<EMAIL_ADDRESS>http://localhost:8080/openapi/greeting
Hello body={'name': 'dave'} selfie=[<starlette.datastructures.UploadFile object at 0x7f4587243df0>]!

# pass file for both, now body is empty, but required param validation still passes
$ curl -F<EMAIL_ADDRESS>-F<EMAIL_ADDRESS>http://localhost:8080/openapi/greeting
Hello body={} selfie=[<starlette.datastructures.UploadFile object at 0x7f4587243df0>]!

This makes for a very confusing development experience.  I had mistakenly assumed that either:

The selfie param would have type starlette.datastructures.UploadFile | str OR
body.selfie would have type starlette.datastructures.UploadFile | str

I'm not saying that the current behavior is wrong, and, now that I have completely worked it out, I can program against it, but it feels non-obvious to me.  Some greater guidance in the documentation would really help.  I had to write about 6 test programs to sort out the behavior here.  The documentation doesn't really say what the semantics of "body" is.
It also seems really confusing that string-type form parameters like name only seem to show up in the "body" parameter, but are not expanded into function params.  This disparate treatment of these two kinds of things is really not clearly explained.
Steps to reproduce
See above
Additional info:
Output of the commands:

python --version: Python 3.10.13
pip show connexion | grep "^Version\:" 3.0.1

One more thing for the docs: When a file is uploaded, it seems that this shows up as a list of FileUpload objects.  No where is this documented (that I could find, apologies if I missed it).  So the correct function signature for a handler is something like:
async def foobar(image: Optional[List[UploadFile]])

Hi @danielbprice, multiple file upload was not working correctly which led to this confusion. See #1806. I just opened a PR with a fix and accompanying documentation.


Incident Analysis - Incident Table to Point model creates duplicate fields (ArcMap)

After running the model, the output point feature class will have all the fields duplicated.
For example, if source table has fields A, B, and C, the output feature class will have A, B, C, A_1, B_1, C_1.
Expected Behavior

I would expect that the output feature class would not have these duplicate fields.
Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Version used:
Environment name and version (e.g. Chrome 39, node.js 5.4):
Operating System and version (desktop or mobile):
Link to your project:

Talked with @mfunk and learned that the issue is caused by newer versions of the Convert Coordinate Notation gp tool copy the fields from the source table into the output feature class. Older versions of the Convert Coordinate Notation tool did not copy the fields, so the Incident Table to Point gp tool model performed a "join" so that the output feature class would contain the fields from the input table. Based on the behavior of the newer versions of Convert Coordinate Notation gp tool, I believe there is no reason for the Incident Table to Point model to exist. Deleting this model would also fix issue #484.

@elinz Going to drop this one as a known issue with previous version of the template, since we are removing Incident Table To Point entirely.


Intercept FSM messages in terminated state

A changelog item should probably also be added (maybe just something like 'general bugfixes')


Invalid swagger parameters generation

if we have some param in the route, but we don't catch this parameter in action's parameters
    [Route("api/users/{userId}/[controller]")]
    public class OrdersController : Controller
    {
        [HttpGet]
        public async Task<IActionResult> Get(int orderId)
        {
            //// action code
        }
    }

we will get wrong swagger data for this parameter (without type)
parameters: [
	{
		name: "orderId",
		in: "path",
		required: true,
		type: "integer",
		format: "int32"
	},
	{
		name: "userId",
		in: "path",
		required: true
	}
],

and swagger-ui 2.1.0 and greater couldn't render this file

What's wrong about the data for userId? How do you expect us to infer the type?
This will show up in ApiDescription coming from MVC as untyped because we don't know what type to treat it as unless there's a declared parameter.

The parameter is missing the "type" which is a required Swagger property. Defaulting to "string" in this case seems appropriate.


diffs not lining up

It finds the diffs correctly but the gutter does not display properly and the highlighting are off.  I believe it is caused by the line height.

Whoah! Yes possibly so. You may actually need to resolve that yourself by adding some CSS to override the line height for your environment. The default CSS defines the right height but it's probably being overridden by something on your end.

Have you tried this with the latest version of ACE?  I am pretty sure line 87, assigning the line height, does not work. It returns zero which cause all of the diffs to be at the top.

Ah, good to know! I'll try to take a look tonight.

[bump for me...]

Hey @tinba12, I updated ace to the latest version but everything still looks okay to me. If you still have the page handy that illustrates the problem, mind posting it here?

It looks like the updated version fixed the issue.

Great, thanks for the heads up!

It looks like that what was happening is that when the divs for the editors are display:none; the line height is screwed up. When I the divs are shown the line heights are correct.


For a correct Mime encoded Message you need two Line Breaks

Currently on some mail clients a single line break after the content type of text/html or text/plain would break the message apart.

Thanks!


De-munchify

I was always a bit embarrassed about using munch for the query object. SimpleNamespace is standard py3 and fits our usage better besides.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@fc05e18). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master      #45   +/-   ##
=========================================
  Coverage          ?   96.96%
=========================================
  Files             ?        9
  Lines             ?      132
  Branches          ?        0
=========================================
  Hits              ?      128
  Misses            ?        4
  Partials          ?        0

Impacted Files
Coverage Δ

nerium/app.py
96.42% <100%> (ø)

nerium/commit.py
75% <100%> (ø)

nerium/query.py
100% <100%> (ø)

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 fc05e18...662314a. Read the comment docs.


Squirrel triggering virustotal "PUA.Shopper"?

Based on this comment:
https://github.com/dustinblackman/Championify/issues/244#issuecomment-269629375
Is this true?  Trying to track down a flag in a recent Virustotal hit on the above project.
Is there a way for that project to avoid the flag?

See Squirrel.Windows https://github.com/Squirrel/Squirrel.Windows/issues/786#issuecomment-237299772


Fix numpy CVE issue

Signed-off-by: Harsha Vamsi Kalluri<EMAIL_ADDRESS>Description
Describe what this change achieves.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Closing because we already use the latest version of numpy in dependencies.


SD: SumPrint line in input file had name of SSSum

If there was a file parsing error in JOINT ADDITIONAL CONCENTRATED MASSES section, the SumPrint line in the input file could have a read error.  This results in an error printed to the screen about SSSum not found -- but the input files all say SumPrint.
Another option instead of changing this is to update all the SD input files to have SSSum.  However, this would be nonstandard with the rest of our input files
This is ready for merging.
Issue reported by @shashankNREL

Done.  Updated docs and input files in the regression tests.


feat(services) - Add option to specify target on service create and u…

…pdate
Closes #1254

Did a quick review for this, I don't really like the UX and rather see this:

You can find my changes inside this branch https://github.com/portainer/portainer/tree/patch-pr1365.
Note: It only affects the UX for the service creation view and should be applied to service update as well.

Looks like there is a small display issue with the service details view:

Seems OK to me

The service details view gives the ability to override the target when connected to an engine with Docker API version < 1.30, that should not be possible.
On my Docker 17-03 setup:

Oh yeah my bad, I just pushed a patch

Thanks @WTFKr0 :+1:


SSL certificate validation failed

Can't use https://www.portfoliorebalancer.com/ anymore. Cert seems invalid.

Would be nice to see this fixed.

Seems the cert has been fixed. thanks.


Document CC behavior when storing providers

Fixes #26363
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test BD

I've triggered the following builds:

Build Distributions (Pull Request Feedback)

Build Scan

The following builds have passed:

Build Distributions (Pull Request Feedback)

Build Scan


Evaluate https://github.com/andreypopp/validated for config validation

https://github.com/andreypopp/validated

Is there any reason you'd prefer this over https://github.com/hapijs/joi?

It looks simpler to me. That's why I'd like to play with a bit before I can say,.


how to get option quote?

I can use "q SPY", but how can I do "q SPY call 270 6/29".
thanks.

it isn't implemented - would appreciate a pull request

I can work on it

I can help for testing.
But how can you reverse engineering the symbol format for quote options? from sniff robinhood app?
I thought it is all HTTPS.

No, you can use Firefox and Robinhood web

I see.
I thought robinhood web does not support option quote yet though.

Yeah but you can figure out the endpoints and poke around

:)
Nice.

Fixed by #11

works great


Breaking change in 2.4.1: Keys are now symbolized

#97 breaks our app. When receiving a hash in a mailer method, the keys used to be strings. Now they are symbols.

Thanks for reporting this -- surprised it hasn't bitten anyone else! I'll see if I can get an update out this week. In the meantime, PRs always welcome :). Seems that the support for block syntax added by @jogaco (requiring symbolized keys) inadvertently broke this -- should add some tests around that requirement.
https://github.com/zapnap/resque_mailer/commit/3feb4a462892a8a3520420caded63443fe9f0f28

Sorry about this. I've submitted #101 as a quick fix

Big thanks to @jogaco for sorting this so quickly. v2.4.2 has been released.


ffmpeg executable not found on machine

ffmpeg executable not found on machine

if you're WINDOWS USER download it from here :
https://www.gyan.dev/ffmpeg/builds/packages/ffmpeg-2023-04-26-git-e3143703e9-full_build.7z

then extract that ffmpeg.exe file into C:\Windows\system32\ folder
if you're LINUX USER :
install it using your package manager, e.g. if you're on DEBIAN or UBUNTU you can just type :
apt install -y ffmpeg

I'VE WROTE ALL OF THESE ON THE FIRST PAGE OF MY REPO, PLEASE READ THAT :)

Yes, I have saved it in the C:\Windows\system32\ folder, but still get an error


[fix] highlight 04-forms / 02-named-form-actions

Highlighting the comma in the Task's readme to avoid simple error when copy-pasting.

This is quite disappointing. A few characters worth of change should make the tutorial more readable, and nobody in the team had the 30s to approve the PR. And there are a few others like this in the issue list...


[Travis CI] don't push container image for PR builds

ISSUE TYPE

Bug fix Pull Request

SUMMARY

Makes sure that image on Docker hub is updated only for commits to
develop branch and not for PRs
ref:
https://docs.travis-ci.com/user/pull-requests/#pull-requests-and-security-restrictions
As PRs from contributors don't have access to secrets, builds are failing with error like this one, https://travis-ci.org/infracloudio/botkube/builds/533893897#L580

LGTM.Please go ahead and merge. Thanks for quick fix.

Thanks @bhavin192 for catching and fixing this


move parse execution to main.py

I think with my change to using relative imports it broke using entry point. This was an idea to fix it, open to other ideas.

Change LGTM!

LGTM, just fix red DCO and this can be merged


Meet OSAF conformance targets on cvrf-csaf-converter

The official conformance target states the following requirements:

satisfies the "CSAF producer" conformance profile
takes only CVRF documents as input.
additionally satisfies the normative requirements given below.

Secondly, the program fulfills the following for all items of:

[ ] type /$defs/version_t: If any element doesn't match the semantic versioning, replace the all elements of type /$defs/version_t with the corresponding integer version. For that, CVRF CSAF converter sorts the items of /document/tracking/revision_history by number ascending according to the rules of CVRF. Then, it replaces the value of number with the index number in the array (starting with 1). The value of /document/tracking/version is replaced by value of number of the corresponding revision item. The match must be calculated by the original values used in the CVRF document.
[x] /document/acknowledgments[]/organization and /vulnerabilities[]/acknowledgments[]/organization: If more than one cvrf:Organization instance is given, the CVRF CSAF converter converts the first one into the organization. In addition the converter outputs a warning that information might be lost during conversion of document or vulnerability acknowledgment.
[x] /document/publisher/name and /document/publisher/namespace: Sets the value as given in the configuration of the program or the corresponding argument the program was invoked with. If values from both sources are present, the program should prefer the latter one. The program SHALL NOT use hard-coded values.
[ ] /vulnerabilities[]/scores[]: If no product_id is given, the CVRF CSAF converter appends all Product IDs which are listed under ../product_status in the arrays known_affected, first_affected and last_affected.
[ ] /vulnerabilities[]/scores[]: If there are CVSS v3.0 and CVSS v3.1 Vectors available for the same product, the CVRF CSAF converter discards the CVSS v3.0 information and provide in CSAF only the CVSS v3.1 information.
[x] /product_tree/relationships[]: If more than one prod:FullProductName instance is given, the CVRF CSAF converter converts the first one into the full_product_name. In addition, the converter outputs a warning that information might be lost during conversion of product relationships.

@zpevma: Due to the end of the project is reached soon, please use the three open points of the description to check that the general conformance targets for vulnerability section is fulfilled.
@sustefil: As you worked the most with document tracking section (#21), same this here for Q/A.


Check permanent redirects

This is more of a suggestion or an idea for future, and a https://github.com/apify/apify-docs/pull/944 spin-off. We already check links. Sometimes the target changes and we link through unnecessary redirects. There are several types of redirects, but it boils down to:

Temporary redirects,
permanent redirects, when the content moves.

In case of the permanent ones, we can just update our links so they point to new places, saving readers' time and unnecessary hops, and also lowering the chance that the link breaks completely.
I think it's okay to check permanent redirects only from time to time, as a matter of regular seasonal gardening, rather than having the check under each PR.
As @TC-MO pointed out, this could be done by disallowing HTTP status codes for permanent redirects in Lychee configuration. I'm creating this issue as a note for myself that I want to do this locally and send a PR with updated links.

Running Lychee locally with --max-redirects 0 is probably the best approximation of what I had in mind, but it's quite a manual process to go through the links and fix them. It does detect a lot of interesting issues though:

Some links are just broken, but the target site returns a redirect instead of error.
Some links redirect to existing pages, but those are nonsense in the original context, and need to be fixed.
A lot of old links within the Apify realm itself.
Some links redirect to new places, but it's clear that they're midway to become broken. Updating them or the context of the link prolongs their life.

I'll be sending a one-off PR to demonstrate usefulness of doing this regularly, but I'll leave up to the rest of the team if they wish to build a respective infrastructure. I feel like I'm stepping outside of my Academy garden here.


Python AWS Lambda SDK: Truncate payload if exceeds 256KB

Description

Related issue https://linear.app/serverless/issue/SC-1166/ensure-sdk-payloads-are-never-over-256kb#comment-9723da2b
Truncate payload if exceeds 256KB

Testing done
Unit/integration tested

@medikoo ready for review 🙏


added settings.xml for PRs

Please add it also here: https://github.com/kiegroup/kie-jenkins-scripts/blob/master/job-dsls/jobs/springboot_pr_job.groovy

@MarianMacik DONE
I will go on around merge once you give your +1 :)


struct passed by value OK if function is not called

Copied on request from https://github.com/zig-lang/zig/issues/548#issuecomment-338088080.
Docs states:

In Zig, structs, unions, and enums with payloads cannot be passed by value to a function.

However this compiles:
const foo = struct
{
  x : i32,
  y : i32,
};

fn bar(f : foo) {}

pub fn main() -> %void {

  var f : foo = undefined;
  //bar(f);  // <<== it stops to compile when I uncomment this
}

This feels like a bug, the offending function has to be called for error to show.

Zig has lazy analysis of top level declarations.
It's so you can do something like this:
if (is_windows) { // assume `is_windows` is compile-time known
    functionThatOnlyWorksOnWindows();
}

functionThatOnlyWorksOnWindows is only analyzed if it is called.
This also helps us compile faster; we don't waste effort on dead code.


[BEAM-3204] Clean-up Coder to only contain FunctionSpec removing its usage of SdkFunctionSpec

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Run Java PreCommit

@kennknowles Yes, I need to import the code into Google and make some minor internal changes that match this. I'm going to wait on merging till that is done.

Cool. It is nice to see these cleanups happening. It'll make it easier for new folks to work on portability.

LGTM for the Go change. Yay reduced stuttering :D


Bottling allay with full inventory

When you trying to pick allay in a bottle with full inventory - item does not appearing in your inventory so you vanished allay lol
That's it
https://github.com/RocKing1001/AllayInABottle/assets/131537305/e9085663-a3f5-4582-9dcd-60ecc89420ba

Thank you for creating an issue.
I have noted the issue, I was busy with other things so sorry for the delay. I will fix it  as soon as I am available. Should not be long.

#7 I added it to my roadmap


AXP code not working for the Tbeam SX1262 chip.

Describe the bug
The current code for to get battery usage for the LoRa SX126x chip not working.
To reproduce
Upload the TBeam-AXP-Example in the sx126s chip board. When the packets are sent, the battery voltage, board temperature is -3.
Expected behavior
It should provide the battery voltage, and board temperature
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context (please complete)

MCU: T-Beam
Wireless module type - LoRa SX1262

As noted in #433 , this  issue is no longer relevant