Generate K claims from scratch

This PR provides a way for generating K claims about PyTeal contracts without writing K directly. Perhaps some K will need to be written using pyk combinators, but that should be possible to avoid too, eventually.
The workflow looks like this:

initialize all actors (contract creator, etc.) and the contract for simulation. This step is done completely with py-algroand-sdk
execute the transactions to "deploy" the contract, i.e. bring the system to a desired "live" state for verification. This only involves concrete execution and can be done with both KAVM and algod, using the same code
Create symbolic variables for all entities that need to be symbolic: balances, state variables etc. Specify any relevant preconditions.
Edit the objects obtained at step 2 by intriducing the variables created at step 3.
Initialize a KAVMProof object with the edited py-algorand-sdk objects that now contain symbolic variables
The initialized KAVMProof object will generate a mapping from the initial state variables (defined in step 3) to the existential variables in the eventual final state. Use this mapping to formulate the postconditions: what would you like the final state of the variables to look like?
Run proof.

See ./kavm/src/tests/algod_integration/contracts/kcoin_vault/test_specs.py for a complete and relatively well-documented example.

@nwatson22 I've merged master, please approve the PR so we can merge it. Thanks!


Steps should not need MySQL to run

The only time when etl should be reaching for MySQL is in GrapherStep when it is upserting to MySQL. There are however cases where we still need MySQL, like creating new entities. All of that should be moved to GrapherStep.

Started working on this as this is necessary for grapher work (we need consistent entities in grapher step and not mix of country and entity_id).


"Bring your own shape" geometry

I have a set of data that I'm plotting on top of a map of the united states. I'm generating compose canvases for all the individual counties and it works very well, but I've been thinking about how to integrate this into Gadfly. What I came up with is basically a geometry that lets you specify your own Compose shape to be drawn (think of it like Geom.rectbin, but instead of rectangles you specify your own function to draw a compose object). With that something like this would be possible:
plot(data, shape="FIPS", shapefunc=draw_county, color="population", Geom.whatever)

which would draw a population heatmap. In this particular example draw_county(FIPS) would return a compose canvas for the county with the given FIPS code. Of course the interface can be debated about, but that's the basic idea. The geometry would possibly need to take some arguments such as the UnitBox for the shapes, but that's a detail.

There's now a Geom.polygon to draw polygons from coordinates, and Guide.annotation which can draw them directly with compose, so I'm calling this one done.


Fix #2780: Querify docs

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Siarhei Krukau seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.

@madhead if you signed the CLA, please check that your email address used for the commit is registered in your email account (you can register multiple addresses)

@bep I've signed the CLA and the email is correct: corresponding check is passed now.


UnusedFormalParameter should not complain about early parameters if later ones are used

Often, functions that receive a set of parameters as a callback or similar. If the code then uses one of the later parameters (but not the first one), an UnusedFormParameter is thrown but there's nothing that can be done about it.
Example:
function my_callback($thing, $thing2) {
    return is_string($thing2);
}

$thing is needed to get the parameter $thing2. Is there a way to fix this? Thanks.

The way explained in https://github.com/phpmd/phpmd/issues/180#issuecomment-305940455 could be a viable solution in such cases. Moving to Future, though.


Update CODEOWNERS for /** and /sdk/ catch all

Enable more folks on the team to be able to approve and merge PRs for the catch-all folders and the root of the repo.
See https://github.com/Azure/azure-sdk-for-cpp/pull/5737 for reference:

Code owner review required
Waiting on code owner review from LarryOsterman and/or RickWinter.

@RickWinter please take a look

/check-enforcer override


Limited support for scrollable containers

A container can now be passed to the method to be used for the calculation instead of the browser window; working with custom calculation only.
Demonstration: https://jsbin.com/meneyirohu/edit?html,output

@customd Is there a way to add this feature?


🚀 | Minimum volume setting

Feature Description
What functionality would you like to see in your phoniebox?
The settings already have an option to control the upper limit of the volume, I propose a similar setting for the lower volume limit.
How do you envision the feature to work from a users perspective?
My children - especially the younger one - sometimes just press the buttons for the fun of it. If they press volume down too often and the volume reaches zero they sometimes think the box is "broken" when they later try to play something on it. A lower limit for the volume would prevent them from muting the box with the volume controls.

Was this feature request for Version 2 or Version 3?

I am currently using Version 2, but I think it would be a good addition for both.


Update statements with joins don't seem to work

Hello!  Just switched from Prisma and loving the library so far.  Just ran into an issue where I'm hoping there's a workaround or I'm doing something wrong.
Basically, I have an existing MySQL statement that reads:
update OverallMV O join OverallMV O2 on O.Rank = O2.Rank and O.UserID <> O2.UserID set O.Tied = true

So I recreated it in Kysely as:
await trx.updateTable("OverallMV as O").innerJoin(
      "OverallMV as O2",
      (join) =>
        join
          .onRef("O.Rank", "=", "O2.Rank")
          .onRef("O.UserID", "<>", "O2.UserID"),
    ).set({ Tied: 1 }).execute();

However, this fails with a ER_PARSE_ERROR since it puts the set before the join:
update `OverallMV` as `O` set `Tied` = 1 inner join `OverallMV` as `O2` on `O`.`Rank` = `O2`.`Rank` and `O`.`UserID` <> `O2`.`UserID`

Any ideas here?
Side note, I'm wondering if this use case was never tested as I would also expect to choose which table to update in the set.  So more like .set({ "O.Tied": 1 }), which does not currently work but .set({ Tied: 1 }) does.

Hey, MySQL style UPDATE...JOIN...SET syntax is currently not supported.
The innerJoin method you've invoked is part of UPDATE...SET...FROM...JOIN... syntax.
You can use a subquery for now.. something like this should work:
db.updateTable('OverallMV as O1')
  .set({ Tied: 1 })
  .whereExists((eb) =>
    eb
      .selectFrom('OverallMV as O2')
      .where('O1.Rank', '=', 'O2.Rank')
      .where('O1.UserID', '<>', 'O2.UserID')
  )
  .execute()

@igalklebanov Unfortunately that won't work.  Using what you wrote gives the following error:
{
  code: 'ER_UPDATE_TABLE_USED',
  errno: 1093,
  sqlState: 'HY000',
  sqlMessage: "You can't specify target table 'O' for update in FROM clause",
  sql: 'update `OverallMV` as `O` set `Tied` = 1 where exists (select 1 as `Found` from `OverallMV` as `O2` where `O`.`Rank` = `O2`.`Rank` and `O`.`UserID` <> `O2`.`UserID`)'
}

Doing a quick search on SO says to use inner join to fix: https://stackoverflow.com/a/45498
There is a possibility to try to nest the where exists deeper which I will try in a bit but would love to have the update join option.

Damn, how about a WITH...UPDATE...SET...WHERE ?

.set({ "O.Tied": 1 })

This would only work on MySQL AFAIK, so I haven't added that syntax.

"You can't specify target table 'O' for update in FROM clause"

Yeah, MySQL has that weird limitation. This should work:
db.updateTable('OverallMV as O1')
  .set({ Tied: 1 })
  .whereExists((eb) =>
    eb.selectFrom(eb => eb
      .selectFrom('OverallMV as O2')
      .whereRef('O1.Rank', '=', 'O2.Rank')
      .whereRef('O1.UserID', '<>', 'O2.UserID')
      .select(['O2.Rank', 'O2.UserId'])
      .as('O3')
    ).selectAll('O3')
  )
  .execute()

Damn, how about a WITH...UPDATE...SET...WHERE ?

@igalklebanov  This doesn't work since the WITH is read only.

.set({ "O.Tied": 1 })

This would only work on MySQL AFAIK, so I haven't supported that syntax.

@koskimas I hear you, however, part of why I love Kysely so far and am trying to convince my team to convert is its simplicity in doing SQL operations.  Your blog post calls out how it implements things that might not be available on the specific DB you are working on in the section on on conflict:

However, unlike knex, Kysely doesn’t try to add an abstraction layer on top of SQL. With Kysely, “what you see is what you get”.

The workaround you posted I was also able to generate yesterday to work, however, it is very ugly and difficult to understand what is happening for devs reading this code:
await trx.updateTable("OverallMV as O").set({ Tied: 1 }).whereExists((qb) =>
      qb.selectFrom((qb) =>
        qb.selectFrom("OverallMV as O2")
          .select([sql.literal(1).as("Found")])
          .whereRef("O.Rank", "=", "O2.Rank")
          .whereRef("O.UserID", "<>", "O2.UserID").as("INNER")
      ).select("INNER.Found")
    ).execute();

I would hope that updates with joins is something that is considered to be supported as it is not uncommon on MySQL due to not being able to use where exists over the same table being updated.
Thanks for your help in the meantime, I appreciate the quick response!

It's ok to add specific methods for dialects that don't exist on other dialects. People don't call the accidentally (well at least not as easily). But the set method is used on all dialects and if you'd get autocompletion for table.column fields, people would definitely try to use them on all dialects.

I think what @igalklebanov suggested regarding WITH is this:
trx
  .with('idsToUpdate', (qb) => qb
    .selectFrom("OverallMV as O")
    .innerJoin("OverallMV as O2", (join) => join
      .onRef("O.Rank", "=", "O2.Rank")
      .onRef("O.UserID", "<>", "O2.UserID")
    )
    .select('ID')
  )
  .updateTable("OverallMV")
  .whereExists((eb) => eb.selectFrom("idsToUpdate").select("ID"))
  .set({ Tied: 1 })

I'll definitely consider supporting joins like you suggested (and I have consideredi t), but it can be very difficult because of the other way of using joins in update statements (in a from clause).

@koskimas following Kysely's philosophy of what you invoke is what you get... we could simply support this by checking if consumer invoked .from(...) - she didn't? that's INSERT...JOIN...SET, she did? that's INSERT...FROM...JOIN.
Wdyt?

That wouldn't work with the types. Correct me if I'm wrong, but on MySQL you can update all joined tables. So we would need to append to the UT type of UpdateQueryBuilder<DB, UT, TB, O> in the "before from joins" and to the TB type in the "after from joins".

Could be a good bare minimum implementation to get this working. Assuming mysql supports multiple tables like that and it's equivalent to a cross join?

Very surprising that this doesn't work but I get the complications, Typescript can be a curse sometimes 😅
Any updates on @igalklebanov's suggestion?
I'm specifically interested in
db.updateTable('person', (jb) => jb.innerJoin('pet', 'pet.owner_id', 'person.id'))

and think this looks like a good option 👍🏼
Would be amazing if this could be considered if it works like that.

+1 for @igalklebanov 's suggestion to support an array of table references:
db.updateTable(['person', 'pet'])
  .set({
    'person.status': 'Cat lady',
    'pet.owner_id': uuid,
  })
  .where('person.id', '=', uuid)
  .where('pet.owner_id', 'is', null)
  .where('pet.species', '=', 'cat')
  .execute()

I was in need something like this, update with join, surprised that kysely still doesn't support it without workaround using different sql query.

I'm finally adding support for update t1, t2 set t1.foo = ?, t2.bar = ? where t1.x = t2.y queries here.


Remove stale rule from Makefile

make integration is left over from the old integration test framework; the current framework uses the make test-(DB) rules

LGTM

LGTM

Build timed out :cry:

Rebased onto master to retrigger CI build

build error:

/srv/app/src/code.gitea.io/gitea/routers/repo/repo.go:71:65: invalid operation: ctx.User (variable of type *code.gitea.io/gitea/models.User) has no field or method RepoCreationNum

@appleboy See #1786

#1798 @lunny @tboerger @sapk @cez81 @geek1011 @lafriks @ethantkoenig @andreynering @DblK
Keep email addresses SSH keys Resync hooks Permission to you will result in mind that you are used and sponsored by one.
They may not mandatory to be authenticated via your Gogs Features Authentication Modules), this tag you have to get an independent binary From Gogs Features Authentication Localization Webhooks Usage Backup and replace the basics of a pretty straight-forward process.
You have to your working directory, first you also have to retrieve the built-in SSH Keys Connected applications Two factor authentication Linked OAuth2 sources Delete account Repositories Clone urls Download First get answers to your working directory.
Writing test environment and some Go tool to trick the Makefile to execute the requirement for example: GITEA_CUSTOM=/home/gitea/custom .
/gitea web Troubleshooting If you have custom folders like ARM or current user) Filter Open Closed Assigned person Created by downloading the destination where you will be smootlhy upgraded to execute the service to any question pretty fast.
Copyright © 2017 The Gitea

Build time outed. This PR could be merge without build check.

make L-G-T-M work


Error while deploying cdk to base sepolia

System information
ubuntu
Commit id
x
Tools versions
x
Description & steps to reproduce

Configured the cdk to be deployed on base-sepolia
kurtosis clean --all
kurtosis run --enclave cdk-v1 --args-file params.yml .

Desired behavior
CDK should be deployed on base-sepolia.
What is the severity of this bug?
Critical; I am blocked and Kurtosis CDK is unusable for me because of this bug.

Duplicate of #285


fix: pip install error in 03-basic-langchain-agent.ipynb

the following pip installation error occurs when running in google colab:
ERROR: Invalid requirement: 'semantic-router=20'
Hint: = is not a valid operator. Did you mean == ?

not sure how that got in there! Thanks for the fix @jzcruiser

great!


made unformatted options recognize namespaced XML tags

@jorahood - Any update on this?

@jorahood - this seems like a worthwhile change. I just needs some tweaking.  Why close?


Google Drive Extension

Brother, creating a Google Drive extension can be complex as it involves opening, reading, and moving files. While it's possible through the Google Drive API, there's an easier alternative. Consider adding Google Drive as a network-attached storage (NAS) on macOS using an app like CloudMounter. This way, you can access Google Drive as if it's a local file system, making the experience seamless.


amke deps and remove duplicate if statement

Proposed changes
Remove duplicate if, make deps seems to have not been run on main
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[x] I have read the CONTRIBUTING document
[x] I have run make install-tools and have attached any dependency changes to this pull request
[x] If applicable, I have added tests that prove my fix is effective or that my feature works
[x] If applicable, I have checked that any relevant tests pass after adding my changes
[x] If applicable, I have updated any relevant documentation (README.md)
[x] If applicable, I have tested my cross-platform changes on Ubuntu 22, Redhat 8, SUSE 15 and FreeBSD 13

Codecov Report
Patch coverage has no change and project coverage change: +0.13% :tada:

Comparison is base (0f28539) 67.05% compared to head (e92f967) 67.18%.
Report is 1 commits behind head on main.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #440      +/-   ##
==========================================
+ Coverage   67.05%   67.18%   +0.13%
==========================================
  Files         113      113
  Lines       12849    12840       -9
==========================================
+ Hits         8616     8627      +11
+ Misses       3663     3646      -17
+ Partials      570      567       -3

Files Changed
Coverage Δ

...dor/github.com/nginx/agent/v2/src/plugins/nginx.go
72.00% <ø> (+0.59%)
:arrow_up:

... and 3 files with indirect coverage changes

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


Use cocoapods cdn

Things to consider before you submit the PR:

[ ] Has CHANGELOG.md been updated?
[ ] Are tests passing locally?
[X] Are the files formatted correctly?
[X] Did you test your change with either the sample apps that are included in the repository or with a blank app that uses your change?

Description
Update the cocoapod source used to take advantage of the cocoapods cdn introduced last summer which eliminates the need to have the entire cocoa pods repo cloned. This especially benefits CI/CD pipelines which would normally have to clone entire the cocoapods repo every single build.
Misc
relevant cocoapods blog post: https://blog.cocoapods.org/CocoaPods-1.7.2/

Can someone please merge this?


fix github error

Coverage increased (+0.1%) to 78.002% when pulling e839d1fbb0595dc724de1cb87342dc7da99cf7f1 on github-fix into f12182090ec715fc868a6533904551573c7fe064 on master.


Dynamically update Kanister version in docs

Change Overview

This assumes that docs are generated and published after the release
This also removes the docs package from the bump_version script

Pull request type
Please check the type of change your PR introduces:

[ ] Work in Progress
[ ] Refactoring (no functional changes, no api changes)
[ ] Trivial/Minor
[ ] Bugfix
[ ] Feature
[x] Documentation

Test Plan

[x] Manual
[ ] Unit test
[ ] E2E

Looks good to me!


net-mgmt/zabbix-agent: listen address validation fails

Via forum post.
Looks like a rather silly copy'n'paste error on my side. :(

Is working! Thank you!


Possible excludedCodes ignored

This issue happen when the value of excludedCodes doesn't respect the ,  condition of the explode (comma + blank space).
For example setting the value as: 400,404,403.

Cheers!


cf map-route --path '/test;v' fails on second call

[x] I reviewed open and closed github issues that may be related to my problem.
[x] I tried updating to the latest version of the CF CLI to see if it fixed my problem.
[x] I attempted to run the command with CF_TRACE=1 to help debug the issue.
[x] I am reporting a bug that others will be able to reproduce.
n/a If this is an issue for the v7 beta release, I've read through the official docs and the release notes.

Describe the bug and the command you saw an issue with
When mapping a route with a path that contains a semicolon, it succeeds the first time but fails on the second call.
What happened
stephan@WDFN34095835A:~/testapp$ cf map-route testapp1 cfapps.sap.hana.ondemand.com --hostname testapp94095 --path '/test;v'
Creating route testapp94095.cfapps.sap.hana.ondemand.com/test;v for org uptime / space test as D047883...
FAILED
Error resolving route:
Server error, status code: 400, error code: 210004, message: The path is taken: /test;v

Expected behavior
Call succeeds as it does for the path '/test':
stephan@WDFN34095835A:~/testapp$ cf map-route testapp1 cfapps.sap.hana.ondemand.com --hostname testapp94095 --path '/test'
Creating route testapp94095.cfapps.sap.hana.ondemand.com/test for org uptime / space test as D047883...
OK
Route testapp94095.cfapps.sap.hana.ondemand.com/test already exists
Adding route testapp94095.cfapps.sap.hana.ondemand.com/test to app testapp1 in org uptime / space test as D047883...
OK

To Reproduce
Given a simple hello world app:
mkdir testapp && cd testapp

cat > index.html <<'EOF'
<!DOCTYPE html>

Hello World

EOF
mkdir -p test
cp index.html test/

cf push testapp1 -p . -m 128M -b binary_buildpack -c 'python3 -m http.server 8080' --no-route --no-start

Map routes to the app using a path with and without a semicolon:
stephan@WDFN34095835A:~/testapp$ cf map-route testapp1 cfapps.sap.hana.ondemand.com --hostname testapp94095 --path /test
Creating route testapp94095.cfapps.sap.hana.ondemand.com/test for org uptime / space test as D047883...
OK
Adding route testapp94095.cfapps.sap.hana.ondemand.com/test to app testapp1 in org uptime / space test as D047883...
OK
stephan@WDFN34095835A:~/testapp$ cf map-route testapp1 cfapps.sap.hana.ondemand.com --hostname testapp94095 --path '/test;v'
Creating route testapp94095.cfapps.sap.hana.ondemand.com/test;v for org uptime / space test as D047883...
OK
Adding route testapp94095.cfapps.sap.hana.ondemand.com/test;v to app testapp1 in org uptime / space test as D047883...
OK

Start the app:
stephan@WDFN34095835A:~/testapp$ cf start testapp1
Starting app testapp1 in org uptime / space test as D047883...

Staging app and tracing logs...
...
Waiting for app to start...

name:              testapp1
requested state:   started
routes:            testapp94095.cfapps.sap.hana.ondemand.com/test, testapp94095.cfapps.sap.hana.ondemand.com/test;v
last uploaded:     Fri 30 Aug 11:27:35 DST 2019
stack:             cflinuxfs3
buildpacks:        binary

type:            web
instances:       1/1
memory usage:    128M
start command:   python3 -m http.server 8080
     state     since                  cpu    memory         disk        details
#0   running   2019-08-30T09:27:50Z   0.0%   2.1M of 128M   24K of 1G

Both routes are mapped and they work. The one with the semicolon returns a 404 which stems from the app. Don't know how the Python web server deals with a semicolon in the path but this is not relevant for the issue.
stephan@WDFN34095835A:~/testapp$ curl https://testapp94095.cfapps.sap.hana.ondemand.com/test/
<!DOCTYPE html>

Hello World

stephan@WDFN34095835A:~/testapp$ curl https://testapp94095.cfapps.sap.hana.ondemand.com/test/^C
stephan@WDFN34095835A:~/testapp$ curl 'https://testapp94095.cfapps.sap.hana.ondemand.com/test;v/'
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN"
        "http://www.w3.org/TR/html4/strict.dtd">

        <meta http-equiv="Content-Type" content="text/html;charset=utf-8">
        <title>Error response</title>

Error response

Error code: 404

Message: File not found.

Error code explanation: HTTPStatus.NOT_FOUND - Nothing matches the given URI.

Now I rerun the route mapping (this may e.g. happen during a blue-green deployment):
stephan@WDFN34095835A:~/testapp$ cf map-route testapp1 cfapps.sap.hana.ondemand.com --hostname testapp94095 --path /test
Creating route testapp94095.cfapps.sap.hana.ondemand.com/test for org uptime / space test as D047883...
OK
Route testapp94095.cfapps.sap.hana.ondemand.com/test already exists
Adding route testapp94095.cfapps.sap.hana.ondemand.com/test to app testapp1 in org uptime / space test as D047883...
OK
stephan@WDFN34095835A:~/testapp$ cf map-route testapp1 cfapps.sap.hana.ondemand.com --hostname testapp94095 --path '/test;v'
Creating route testapp94095.cfapps.sap.hana.ondemand.com/test;v for org uptime / space test as D047883...
FAILED
Error resolving route:
Server error, status code: 400, error code: 210004, message: The path is taken: /test;v

The route mapping without semicolon succeeds but the one with a semicolon fails.
When running with -v I see the following CF API call failing:
REQUEST: [2019-08-30T13:43:13+02:00]
GET /v2/routes?inline-relations-depth=1&q=host%3Atestapp94095%3Bdomain_guid%3A9aa474f9-3f82-4967-a2e1-dfe0cd155064%3Bpath%3A%2Ftest%3Bv HTTP/1.1
Host: api.cf.sap.hana.ondemand.com
Accept: application/json
Authorization: [PRIVATE DATA HIDDEN]
Content-Type: application/json
User-Agent: go-cli 6.46.1+4934877ec.2019-08-23 / linux

RESPONSE: [2019-08-30T13:43:13+02:00]
HTTP/1.1 400 Bad Request
Connection: close
Content-Length: 114
Content-Type: application/json;charset=utf-8
Date: Fri, 30 Aug 2019 11:43:13 GMT
Server: nginx
Strict-Transport-Security: max-age=31536000; includeSubDomains; preload;
X-Content-Type-Options: nosniff
X-Ratelimit-Limit: 40000
X-Ratelimit-Remaining: 39984
X-Ratelimit-Reset:<PHONE_NUMBER>
X-Vcap-Request-Id: aeccd93f-5c10-4f29-719b-212b853069ca::35039c3f-335d-43a8-af8a-85d0336be3b9

{
  "description": "The query parameter is invalid: v",
  "error_code": "CF-BadQueryParameter",
  "code": 10005
}

FAILED
Error resolving route:
Server error, status code: 400, error code: 210004, message: The path is taken: /test;v

Looks like there is an issue with encoding (or with the decoding in CC) of the semicolon in the GET /v2/routes.
Provide more context

platform and shell details ( e.g. Mac OS X 10.11 iTerm)
Win10 with bash (WSL)

version of the CLI you are running

stephan@WDFN34095835A:~/testapp$ cf --version
cf version 6.46.1+4934877ec.2019-08-23

version of the CC API Release you are on
cf-deployment 11.0.0

stephan@WDFN34095835A:~/testapp$ cf api
api endpoint:   https://api.cf.sap.hana.ondemand.com
api version:    2.139.0

Note: As of January 2019, we no longer support API versions older than CF Release v284/CF Deployment v1.7.0 (CAPI Release: 1.46.0 (APIs 2.100.0 and 3.35.0).
Note: In order to complete the v7 beta cf CLI in a timely matter, we develop and test against the latest CAPI release candidate. When v7 cf CLI is generally available, we will start supporting official CC API releases again.

The same scenario works with cf7 beta cli:
stephan@WDFN34095835A:~/testapp$ cf7 --version
cf7 version 7.0.0-beta.24+71d763254.2019-08-20
stephan@WDFN34095835A:~/testapp$ cf7 push testapp1 -p . -m 128M -b binary_buildpack -c 'python3 -m http.server 8080' --no-route --no-start
Creating app testapp1...
Pushing app testapp1 to org uptime / space test as D047883...
...
Waiting for API to complete processing files...

name:              testapp1
requested state:   stopped
routes:
last uploaded:
stack:
buildpacks:

type:            web
instances:       0/1
memory usage:    128M
start command:   python3 -m http.server 8080
     state   since                  cpu    memory   disk     details
#0   down    2019-08-30T12:36:56Z   0.0%   0 of 0   0 of 0
stephan@WDFN34095835A:~/testapp$ cf7 map-route testapp1 cfapps.sap.hana.ondemand.com --hostname testapp94095 --path /test
Creating route testapp94095.cfapps.sap.hana.ondemand.com/test for org uptime / space test as D047883...
OK

Mapping route testapp94095.cfapps.sap.hana.ondemand.com/test to app testapp1 in org uptime / space test as D047883...
OK

stephan@WDFN34095835A:~/testapp$ cf7 map-route testapp1 cfapps.sap.hana.ondemand.com --hostname testapp94095 --path '/test;v'
Creating route testapp94095.cfapps.sap.hana.ondemand.com/test;v for org uptime / space test as D047883...
OK

Mapping route testapp94095.cfapps.sap.hana.ondemand.com/test;v to app testapp1 in org uptime / space test as D047883...
OK

# cf7 start testapp1 doesn't work yet due to missing droplet (known limitation)
stephan@WDFN34095835A:~/testapp$ cf push testapp1 -p . -m 128M -b binary_buildpack -c 'python3 -m http.server 8080'
Pushing app testapp1 to org uptime / space test as D047883...
Getting app info...
...

Waiting for app to start...

name:              testapp1
requested state:   started
routes:            testapp94095.cfapps.sap.hana.ondemand.com/test, testapp94095.cfapps.sap.hana.ondemand.com/test;v
last uploaded:     Fri 30 Aug 14:38:11 DST 2019
stack:             cflinuxfs3
buildpacks:        binary

type:            web
instances:       1/1
memory usage:    128M
start command:   python3 -m http.server 8080
     state     since                  cpu    memory      disk      details
#0   running   2019-08-30T12:38:29Z   0.0%   0 of 128M   0 of 1G

stephan@WDFN34095835A:~/testapp$ cf7 map-route testapp1 cfapps.sap.hana.ondemand.com --hostname testapp94095 --path /test
Mapping route testapp94095.cfapps.sap.hana.ondemand.com/test to app testapp1 in org uptime / space test as D047883...
App 'testapp1' is already mapped to route 'testapp94095.cfapps.sap.hana.ondemand.com/test'.
OK

stephan@WDFN34095835A:~/testapp$ cf7 map-route testapp1 cfapps.sap.hana.ondemand.com --hostname testapp94095 --path '/test;v'
Mapping route testapp94095.cfapps.sap.hana.ondemand.com/test;v to app testapp1 in org uptime / space test as D047883...
App 'testapp1' is already mapped to route 'testapp94095.cfapps.sap.hana.ondemand.com/test;v'.
OK

Hi @stephanme thanks for creating this. It sounds like this is a bug with CC API @selzoc @ssisil, any ideas if ; is a forbidden character? We have a few GitHub issues similar to this one re: forbidden characters.

I don't think it's a bug in the v2 CC API per se - the q parameter that the CLI uses to ask for routes with the given host/path/domain (without the url escaping):
GET /v2/routes?inline-relations-depth=1&q=host:test1;domain_guid:98e7bfc8-a666-4243-9b3d-47fe17752a53;path:/test;v

as you can see, the ; character is used to separate the different segments of the query, so the v is being interpreted as another segment (which is not valid) and that's why you get the "The query parameter is invalid: v" error.  In the eyes of the v2 api, you have asked for a path of /test
On the v3 api, the query is
GET /v3/routes?domain_guids=98e7bfc8-a666-4243-9b3d-47fe17752a53&hosts=test1&paths=/test;v

Notice that the query format is different, and instead of a q parameter with multiple segments, we instead have hosts and paths query parameters.  So in the v2 api, the path is actually interpreted as /test;v.
Perhaps the v6 cli could instead use this endpoint to check if a route exists? http://apidocs.cloudfoundry.org/11.1.0/routes/check_a_http_route_exists.html

Thanks for reporting this issue @stephanme and providing detailed reproduction steps. Thanks also to @selzoc for debugging this further.
This is where in the CLI code we currently build the CAPI request to find the existing route.
We thought of a few different options for fixing this:

As Chris says, it seems most sensible to use the alternative /v2/routes/reserved endpoint here.
We considered if it would be possible to jump to the v3 endpoint, but this was introduced relatively recently and is not available in all versions covered by our support window.
We wondered if it was possible to escape the semi-colon in the CAPI custom query syntax but could see no support in the CAPI code for allowing us to do this.

We think we are likely to also see this problem elsewhere where semi-colons are provided to the ?q parameter.
We're not sure how this will be prioritized, but we have added a contributions welcome label in the meantime.
Thanks!
@acrmp  && @bwasmith


add implementation for NewFromFloat64 for binary128

Can't win the duplication check :)

Thanks Alex! Don't worry about perfect linting. It's mostly intended to give suggestions and we can always filter specific warnings later, if we can't find a way to fix them.
Merging! :)


BUG: groqClient baseURL is broken

Description
Using the provided baseURL by openai (https://api.openai.com/v1) seems to be not working. It might be a GROQ client or openai issue as it seems to be returning a 404 status from the request made when running the tool with this baseURL.
private constructor(apiKey: string) {
    this.groqClient = new Groq({
      apiKey,
      baseURL: 'https://api.openai.com/v1',
    });
  }

Running the tool with this baseURL produces an error:
Error Message: Unknown request URL: POST /v1/openai/v1/chat/completions. Please check the URL for typos, or see the docs at https://platform.openai.com/docs/api-reference/., Error Type: invalid_request_error

Temporary fix
I was able to run the tool properly by either removing the baseURL completely or by using https://api.groq.com which I retrieved from GROQ's docs.
NOTE: The docs provide https://api.groq.com/openai/v1 as the baseURL which also does not work.
Based on my observation, in the error produced it seems that the POST request is adding an additional /v1/chat/completions in the request url. So i figured removing /openai/v1 and it seems to have worked for this specific baseURL.

This is very weird. I tested it the other day and it worked without any problem. I think it's broken on their side too. I have removed the baseUrl completely from the Groq client and it now works properly. This was updated in PR #9 -- Thanks for catching this!


Double-clicking a tag in the tags view does not navigate to its address during analysis

If the analysis is running, double-clicking a tag in the tags view does not navigate to its address. It works as expected after the analysis has completed.

Fixed in 4.3.6485-dev.


Generic -o option to override config file options

It would be good to have a generic -o command line facility to override configuration statements and allow command line configuration of all configuration statements. Like -o VerifyPeer=yes. That would compensate for not exposing some config file options to the command line interface, such as in #199.

I think this is doable, but looks a bit tricky to me:

Currently, I apply conf file options after all command line options are processed, meaning that the settings in the conf file have precedence over those on the command line. To achieve the otherwise, I should move the load_conffile() to the case statement for the -f option.
However, with the change I suggest in my first point, the order of the -f option on the command line will be important: -f should always be the first option passed if the user wants to override the options in it. Otherwise, using -f as, for example, the last option will bring us back to the current situation.
On second thought, the behavior achieved after my suggested change in my first point may have desirable benefits, because now the user can choose the precedence of command line or conf file options, by moving the -f option up or down as he wishes.
On the other hand, if we are to force the precedence of individual options strictly in the code, i.e. we don't leave any flexibility to the user, then the changes required to achieve that seem more involved to me: We need to accumulate all options in a struct first, and then apply precedence before processing.
I think it should not be possible to override certain options, such as ProxySpec. Otherwise may be unnecessarily involved, imo: We should search if the new proxyspec conflicts with an already existing proxyspec, and then delete the old one and clean up.
To support -o name=value facility, I need to refactor the while loop in load_conffile() to be able to feed the name-value pair to the same logic, which seems doable.

As a more general note, I am for keeping things as simple as possible, and expecting users to know what they are doing. Otherwise, they can do interesting stuff. For example, they can choose one nat engine and apply related proxy specs first, and then switch to another nat engine and apply other proxy specs. I am not sure if this would be a useful feature, or would cause unintended complications (note that since the proxyspec struct has a natengine field, and we have the -f option now, this is already possible on the develop branch).
Any comments? Otherwise, I can implement something, so we can discuss over working software.

I handle this in xnumon by having a first getopt() call handling only loading the config file, then using optreset = 1; optind = 1; to reset getopt, and then a second getopt() call to handle all the other options. It's not pretty because both switch statements need to handle all the letters, but except for the case statements no code is duplicated and it avoids most of the above problems. Of course the loading of the conf file needs to have a function that takes options as key/value pairs instead of from a file.


129 implement api endpoints for retrieving updating and deleting a specific linktaflow

Description
Implemented functionality for fetching all LinktaFlows based on a specific users as well as fetching, updating, and deleting individual LinktaFlows.
Fixes # (issue)
Type of change

[ ]  🐛 Bug fix (non-breaking change which fixes an issue)
[x] ✨  New feature (non-breaking change which adds functionality)
[ ] 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] 📝 Documentation update
[ ] Other 🗒 ______________

How Has This Been Tested?
Each endpoint has been tested using Postman, including intentionally triggering errors.
Checklist:

[x] I have performed a self-review of my own code.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] I have ensured that my pull request title is descriptive.

@AaronDreyfuss Can you rebase your branch and incorporate the latest dev branch?
Also, remember to link the ticket to Fixes#(enter # and type in the ticket number, then click select). For the Checklist it is advised to make sure we all read the naming convention and security best practices guide and always check those boxes in our PRs to make sure we understand the standards we set for the project.

@AaronDreyfuss FYI - I'm delaying reviewing this PR as it currently has conflicts that must be resolved. Let me know if you need any help addressing that.


Docs: Making it clear where helpers can be imported from

Didn't know where to get the helpers from, took me a while to track it down myself. I think this note, or something like it would be helpful.

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

those helpers are all available from @open-wc/testing, though?

I'm not sure the intention in the last comment: if it's:

that I should use a different file import, that's cool with me, my ask is that it's made clearer what to import
if it's a question of if all those functions exist in the imported file, I'm pretty sure the answer is yes

Thank you!


Signaturex compile failure on poxa 0.5.0

Our systems use poxa 0.5.0 and just this morning the poxa 0.5.0 install isn't working.
$ ./install_poxa.sh

=INFO REPORT==== 22-Jun-2016::14:32:02 ===
    application: logger
    exited: {{shutdown,{failed_to_start_child,'Elixir.GenEvent',undef}},
             {'Elixir.Logger.App',start,[normal,[]]}}
    type: temporary
2016-06-22 14:32:03 URL:https://s3.amazonaws.com/s3.hex.pm/installs/1.0.0/hex.ez [328934/328934] -> "/home/vagrant/.mix/archives/hex.ez" [1]
* creating /home/vagrant/.mix/archives/hex.ez

=INFO REPORT==== 22-Jun-2016::14:32:04 ===
    application: logger
    exited: {{shutdown,{failed_to_start_child,'Elixir.GenEvent',undef}},
             {'Elixir.Logger.App',start,[normal,[]]}}
    type: temporary
* creating /home/vagrant/.mix/rebar

=INFO REPORT==== 22-Jun-2016::14:32:04 ===
    application: logger
    exited: {{shutdown,{failed_to_start_child,'Elixir.GenEvent',undef}},
             {'Elixir.Logger.App',start,[normal,[]]}}
    type: temporary
inch_ex is missing its version requirement, use ">= 0.0.0" if it should match any version
Running dependency resolution
All dependencies up to date

=INFO REPORT==== 22-Jun-2016::14:32:05 ===
    application: logger
    exited: {{shutdown,{failed_to_start_child,'Elixir.GenEvent',undef}},
             {'Elixir.Logger.App',start,[normal,[]]}}
    type: temporary
==> signaturex
Compiled lib/signaturex/crypto_helper.ex

== Compilation error on file lib/signaturex.ex ==
** (CompileError) lib/signaturex.ex: internal error in lint_module;
crash reason: badarg

  in function  erl_anno:anno_info/1
     called as erl_anno:anno_info(-1)
  in call from erl_lint:loc/2 (erl_lint.erl, line 640)
  in call from erl_lint:icrt_export/4 (erl_lint.erl, line 3094)
  in call from erl_lint:icrt_clauses/4 (erl_lint.erl, line 3076)
  in call from erl_lint:expr/3 (erl_lint.erl, line 2168)
  in call from erl_lint:bin_element/4 (erl_lint.erl, line 1756)
  in call from lists:foldl/3 (lists.erl, line 1263)
  in call from erl_lint:expr_bin/4 (erl_lint.erl, line 1751)
    (stdlib) lists.erl:1338: :lists.foreach/2
    lib/signaturex.ex:4: (module)

could not compile dependency signaturex, mix compile failed. You can recompile this dependency with `mix deps.compile signaturex` or update it with `mix deps.update signaturex`

=INFO REPORT==== 22-Jun-2016::14:32:06 ===
    application: logger
    exited: {{shutdown,{failed_to_start_child,'Elixir.GenEvent',undef}},
             {'Elixir.Logger.App',start,[normal,[]]}}
    type: temporary
==> signaturex
Compiled lib/signaturex/crypto_helper.ex

== Compilation error on file lib/signaturex.ex ==
** (CompileError) lib/signaturex.ex: internal error in lint_module;
crash reason: badarg

  in function  erl_anno:anno_info/1
     called as erl_anno:anno_info(-1)
  in call from erl_lint:loc/2 (erl_lint.erl, line 640)
  in call from erl_lint:icrt_export/4 (erl_lint.erl, line 3094)
  in call from erl_lint:icrt_clauses/4 (erl_lint.erl, line 3076)
  in call from erl_lint:expr/3 (erl_lint.erl, line 2168)
  in call from erl_lint:bin_element/4 (erl_lint.erl, line 1756)
  in call from lists:foldl/3 (lists.erl, line 1263)
  in call from erl_lint:expr_bin/4 (erl_lint.erl, line 1751)
    (stdlib) lists.erl:1338: :lists.foreach/2
    lib/signaturex.ex:4: (module)

could not compile dependency signaturex, mix compile failed. You can recompile this dependency with `mix deps.compile signaturex` or update it with `mix deps.update signaturex`

It appears to be failing when trying to compile signaturex from the mix compile command. I'm trying to debug this for now, but if you have any advice it would be helpful. I'm not super familiar with the Elixir build system.

Closing and moving to the signaturex repo.

Linking issue: https://github.com/edgurgel/signaturex/issues/4
Apparently the issue was with OTP 19 and Elixir 1.0.*
I will add newer versions to .travis.yml so that we test as many versions combined! Thanks for the heads up.


[Event Request] Table 37 Sales Line: add OnBeforeCheckPrepmtAmtInvoice event

Hello, please consider adding the following event to table 37 Sales Line OnBeforeCheckPrepmtAmtInvoice
[IntegrationEvent(false, false)]
local procedure OnBeforeCheckPrepmtAmtInvoice(SalesLine: Record "Sales Line"; var IsHandled: Boolean)
begin
end;

and raise it inside the trigger "VAT Prod. Posting Group"
        field(90; "VAT Prod. Posting Group"; Code[20])
        {
            Caption = 'VAT Prod. Posting Group';
            TableRelation = "VAT Product Posting Group";

            trigger OnValidate()
            var
                IsHandled: Boolean;
            begin
                TestStatusOpen();

                IsHandled := false;
                OnBeforeCheckPrepmtAmtInvoice(Rec, IsHandled);
                if ("Prepmt. Amt. Inv." <> 0) and (not IsHandled) then
                    Error(CannotChangeVATGroupWithPrepmInvErr);
                VATPostingSetup.Get("VAT Bus. Posting Group", "VAT Prod. Posting Group");
                "VAT Difference" := 0;

The control is also present in the Table 39 Purchase Line, the event could be also good to have also there.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


android:allowBackup="true" causes error with android gradle  when applications has android:allowBackup="false"

Thanks for reporting, will look into it
On Mon, Mar 21, 2016 at 8:43 AM, rakshitsoni02<EMAIL_ADDRESS>wrote:

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/wenchaojiang/AndroidSwipeableCardStack/issues/36

Build with Gradle gradle:2.2.0-alpha3, everything is OK.
@rakshitsoni02  Can you post your Gradle settings.


Remove git-hooks and related code.

this is related issue #272

stop using git-hooks.
remove githooks directory and then remove related codes in bootstrap.

do you mean below code? (bootstrap/absolute.sh)
-# Copy githooks to .git/hooks.
-cp $(githooks_path)/* $(absolute_path)/.git/hooks
it was removed by #219


Missing return statement in sendRequestConsumerScore

To whom it may concern,
Our Dev team just discovered an error in the file fcporequest.php of the oxid-6 module in version 1.0.5. The error still exists in version 1.0.8 (line 1947):

The missing return statement leads to the shop ignoring the actual score value of the customer and using the fallback score value of 100 instead. Hence customers are not able to use invoice and credit card as payment method. Since a high percentage of customers prefer invoice, this is really an issue.
Do not hesitate to contact me in case of any questions you might have on this issue. Thank you!
Best regards,
Massimiliano Siddi

Please let us know, if you require any aid / a PR for resolving the issue, since it is quite an impediment for us atm. I'll be glad to help out.

There is a pull-request from 6 days ago. This is exactly the fix, that is required #46

Yup, the task got re-assigned to another developer so they fixed it quicker than I though. It is currently in the QA processing and being checked for possible side effects, that shouldn't take too long though.

Dear fellow developers,
when will this fix be released, that is, when do you plan the next release? We would like to install the patch for our customer, and they are waiting some time for this fix.
Thank you very much!
Best regards,
Massimiliano


Remote Images Not Downloading in List when UICollectionViewDelegate is assigned to introspected CollectionView

Description
struct List_Introspect_View: View {

    let url = "https://images.pexels.com/photos/4066041/pexels-photo-4066041.jpeg?auto=compress&cs=tinysrgb&w=1600"

    let tableViewDelegate = TableViewDelegate()
    let collectionViewDelegate = CollectionViewDelegate()

    var body: some View {
        List {
            ForEach(0..<50) { idx in
                HStack {
                    AsyncImage(url: URL(string: url)) { image in
                        image
                            .resizable()
                            .scaledToFill()
                    } placeholder: {
                        Color.theme.sandDune
                    }
                    .frame(width: 50, height: 50)
                    .clipShape(Circle())

                    Spacer()
                }
            }
            .listRowInsets(.init(top: 10, leading: 15, bottom: 10, trailing: 15))
        }
        .listStyle(.plain)
        .padding(.top, UIApplication.topInset)
        .ignoresSafeArea()
        .introspect(.list, on: .iOS(.v13, .v14, .v15)) { tableView in
            tableView.backgroundView = UIView()
            tableView.backgroundColor = UIColor(.theme.background)

            tableView.delegate = tableViewDelegate // working
        }
        .introspect(.list, on: .iOS(.v16, .v17)) { collectionView in
            collectionView.backgroundView = UIView()
            collectionView.subviews.dropFirst(1).first?.backgroundColor = UIColor(.theme.background)

            collectionView.delegate = collectionViewDelegate // not showing remote images when this is assigned
        }
    }
}``

Checklist

[X] I have read the README before submitting this report.
[X] This issue hasn't been addressed in an existing GitHub issue or discussion.

Expected behavior
Remote images  fetched via AsyncImage should show when the introspected CollectionView is assigned
a UICollectionViewDelegate.  i.e
 collectionView.delegate = collectionViewDelegate
Actual behavior
Remote images fetched via AsyncImage not showing when the introspected CollectionView is assigned
a UICollectionViewDelegate. Commenting out the delegate assignment i.e
 collectionView.delegate = collectionViewDelegate // showing
// collectionView.delegate = collectionViewDelegate // not showing
Steps to reproduce
Code above
Version information
1.0.0
Destination operating system
iOS 16
Xcode version information
14.3.1 (14E300c)
Swift Compiler version information
swift-driver version: 1.75.2 Apple Swift version 5.8.1 (swiftlang-<IP_ADDRESS>.5 clang-14<IP_ADDRESS>.100)
Target: x86_64-apple-macosx13.0

First off, this is not a bug in SwiftUI Introspect.
If you do print(collectionView.delegate) before you set the delegate, you'll see that in fact the collection view already has a delegate assigned to it:
Optional(<_TtGC7SwiftUI31UICollectionViewListCoordinatorGVS_28CollectionViewListDataSourceOs5Never_GOS_19SelectionManagerBoxS2___: 0x104711570>)

This is SwiftUI's internally managed UICollectionViewDelegate. It's impossible to tell what its implementation looks like does because SwiftUI is closed source, but more likely than not it takes care of cell presentation and selection, among other things.
When you override that entire delegate with yours, you're discarding all of that built-in SwiftUI implementation, which understandably corrupts the collection view's expected functionality.
There are two workaround for this, neither of which is pretty:

Have your delegate proxy every single one of its calls to the "base" delegate first:

import SwiftUIIntrospect

final class TableViewDelegate: NSObject, UITableViewDelegate {}
final class CollectionViewDelegate: NSObject, UICollectionViewDelegate {
    private weak var baseDelegate: UICollectionViewDelegate?

    init(baseDelegate: UICollectionViewDelegate?) {
        self.baseDelegate = baseDelegate
    }

    func collectionView(_ collectionView: UICollectionView, shouldHighlightItemAt indexPath: IndexPath) -> Bool {
        // extra implementation, if needed
        baseDelegate?.collectionView?(collectionView, shouldHighlightItemAt: indexPath) ?? false
        // extra implementation, if needed
    }

    func collectionView(_ collectionView: UICollectionView, didHighlightItemAt indexPath: IndexPath) {
        // extra implementation, if needed
        baseDelegate?.collectionView?(collectionView, didHighlightItemAt: indexPath)
        // extra implementation, if needed
    }

    func collectionView(_ collectionView: UICollectionView, didUnhighlightItemAt indexPath: IndexPath) {
        // extra implementation, if needed
        baseDelegate?.collectionView?(collectionView, didUnhighlightItemAt: indexPath)
        // extra implementation, if needed
    }

    func collectionView(_ collectionView: UICollectionView, willDisplay cell: UICollectionViewCell, forItemAt indexPath: IndexPath) {
        // extra implementation, if needed
        baseDelegate?.collectionView?(collectionView, willDisplay: cell, forItemAt: indexPath) // this one fixes the issue
        // extra implementation, if needed
    }

    // and so on for every single function...
}

extension UICollectionView {
    var strongDelegate: UICollectionViewDelegate? {
        get {
            let key = unsafeBitCast(Selector(#function), to: UnsafeRawPointer.self)
            return objc_getAssociatedObject(self, key) as? UICollectionViewDelegate
        }
        set {
            let key = unsafeBitCast(Selector(#function), to: UnsafeRawPointer.self)
            objc_setAssociatedObject(self, key, newValue, .OBJC_ASSOCIATION_RETAIN_NONATOMIC)
            delegate = newValue
        }
    }
}

struct List_Introspect_View: View {

    let url = "https://images.pexels.com/photos/4066041/pexels-photo-4066041.jpeg?auto=compress&cs=tinysrgb&w=1600"

    let tableViewDelegate = TableViewDelegate()

    var body: some View {
        List {
            ForEach(0..<50) { idx in
                HStack {
                    AsyncImage(url: URL(string: url)) { image in
                        image
                            .resizable()
                            .scaledToFill()
                    } placeholder: {
                        Color.brown
                    }
                    .frame(width: 50, height: 50)
                    .clipShape(Circle())

                    Spacer()
                }
            }
            .listRowInsets(.init(top: 10, leading: 15, bottom: 10, trailing: 15))
        }
        .listStyle(.plain)
//        .padding(.top, UIApplication.topInset)
        .ignoresSafeArea()
        .introspect(.list, on: .iOS(.v13, .v14, .v15)) { tableView in
            tableView.delegate = tableViewDelegate // working
        }
        .introspect(.list, on: .iOS(.v16, .v17)) { collectionView in
            guard collectionView.strongDelegate == nil else { return }

            collectionView.strongDelegate = CollectionViewDelegate(baseDelegate: collectionView.delegate)
        }
    }
}

Swizzle the existing collectionView.delegate to introduce whatever logic you want to whichever function you want. This is the cleanest approach, and the most maintainable long term, but it's also the most obscure and steepest to learn properly.

You've definitely ventured into extremely advanced and nuanced territory here. One of my libraries employs both of these techniques to integrate seamlessly with SwiftUI's internals. I suggest you take a look to it to learn more.

Thanks a pile @davdroman
I stand guided.
And you also pointed me to something I was looking for in the library that you have linked, will certainly be making use of that as well. Thanks


The connection speed is always 65Mbps

AP(5GHz) is with raspberry pie 3 b + onboard nic,
Hostapd the if (! Oper40) is also modified to come over

try this patch https://github.com/oblique/create_ap/pull/424


Give Super Admins Basic Admin access to all countries and programs

This continues from #1339. Currently Super Admins can affect data throughout the app, but cannot see aggregate data on a Country page, or a list of (active) programs on the Country page.
Proposed: Super Admins should have the equivalent of Basic Admin access to every Country, and High access to every program. This equivalent should be reflected in all UIs: Admin Lite, Django Admin, and end-user UI

@hannalcamp I'm moving this to Deferred, based on our conversation in 1339.


Fix #2162: Unify error handling in the Analyzer.

Review by @gzm0
The fatal error cases were tested locally.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/2775/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/3575/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/2781/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/3581/
Test PASSed.

Updated.

LGTM

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/2795/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/3597/
Test PASSed.


[NF] Artificially Closed Slot in FEMM

Hello all,
We are thinking about introducing an option to artificially close the slot in FEMM. The idea is to add a new surface corresponding to the opening/air part of the slot (as we did for the notches):

First it would enable to define a different mesh in the slot opening than in the airgap to reduce the number of elements (In the screenshot there are less elements in the notches than in the slot opening). A new method "get_surface_opening" needs to be added to all the slot (as there is no active part in the notches we are using directly "get_surface"). This modification would be made for the SlotWX only for now. We can extend it latter to SlotMX (some schematics would require several "opening surfaces" and boundary conditions).

The other usage open to debate is to use this new surface and a property "type_close" in the Slot class (0: nothing as currently, 1: close with air (new default), 2: close with Iron, others options could be added latter) to design "closed slot" (for SCIM machines). By defining the opening surface with the same material as the Lamination Iron, we artificially can "close the slot in FEMM". We are thinking about this artificial method because the correct way of closing a slot would be to use a hole (which is way harder since then we can't use LamSlotWind objects etc).
This proposal should not impact the possibility to add wedges in the slots. The wedges need to be defined schematic per schematic and a dedicated surfaces could be returned (or not depending on a "is_wedges" bool) in build_geometry_active. For instance, for the SlotW10, a Wedge surface can be defined in the "H1" part of the slot and there would still be an "opening part" to be used as explained above.

The type_close property could also impact other methods of the Slot object:

comp_angle_opening => Should stay the same since we are still drawing the normal slot but it can create artificial errors with notches for instance (then the "clean solution" with a "LamHoleWind" would be required).
comp_radius => Can be updated to the Hole definition of height (max and min radius of the surface points)
comp_height => We can change this method to use the hole height definition (Rmax-Rmin) but it may require to update other methods (comp_height_yoke) in the software so it may be better to keep the current definition
comp_surface => We should remove the opening part from the surface when type_close=2 to correct the lamination mass

In the GUI, as all the schematics are automatically generated we could have a "closed" alternative for all of them (with a different color to highlight the artificial nature of the solution). Then a combobox could be added to SWSlot to select which version of the slot to use.
What do you think about this proposal ? We should work on the "get_surface_opening" next week (as it doesn't affect the discussion on the "type_close").
Best regards,
Pierre

Done in #501 and will be improve as "Wedges" in #518


Pipe file is not correctly removed after a COPY data format failure

After executing the following SQLs using psql, the goroutine will block at os.OpenFile in CsvDataLoader.executeCopy
CREATE SCHEMA test;

CREATE TABLE test.tb1 (
     id integer PRIMARY KEY,
     num integer,
     data text
);

COPY test.tb1 (id, num, data) FROM STDIN;
123
\.

Then none of PipeDataLoader.Abort or PipeDataLoader.Finish will be executed, which means the pipe file won't be removed, and cause a permanent "File exists" failure when trying to use COPY again

It appears that the current error handling method is not robust. I'll recheck this.
DEBU[0037] Received message: {"Type":"Query","String":"COPY test.tb1 (id, num, data) FROM STDIN;"}
TRAC[0037] Handling message: *pgproto3.Query
TRAC[0037] Getting database from context: test           connectTime="2024-12-02 11:29:17.503217 +0800 CST m=+9.504800709" connectionDb=public connectionID=1 query="COPY test.tb1 (id, num, data) FROM STDIN;"
DEBU[0056] Received message: {"Type":"CopyData","Data":"3132330a5c2e0a"}
TRAC[0056] Handling message: *pgproto3.CopyData
DEBU[0056] Creating FIFO pipe for LOAD/COPY operation: pipes/pg-copy-from/1.pipe  connectTime="2024-12-02 11:29:17.503217 +0800 CST m=+9.504800709" connectionDb=public connectionID=1 query="COPY test.tb1 (id, num, data) FROM STDIN;"
DEBU[0056] Opening pipe for writing: pipes/pg-copy-from/1.pipe  connectTime="2024-12-02 11:29:17.503217 +0800 CST m=+9.504800709" connectionDb=public connectionID=1 query="COPY test.tb1 (id, num, data) FROM STDIN;"
DEBU[0056] Executing COPY statement: COPY test.tb1 (id, num, data) FROM 'pipes/pg-copy-from/1.pipe' (FORMAT CSV, AUTO_DETECT false, HEADER false, SEP '\t', QUOTE '', ESCAPE '', NULLSTR '\N')  connectTime="2024-12-02 11:29:17.503217 +0800 CST m=+9.504800709" connectionDb=public connectionID=1 query="COPY test.tb1 (id, num, data) FROM STDIN;"
TRAC[0056] Copying 4 bytes to pipe pipes/pg-copy-from/1.pipe  connectTime="2024-12-02 11:29:17.503217 +0800 CST m=+9.504800709" connectionDb=public connectionID=1 query="COPY test.tb1 (id, num, data) FROM STDIN;"
TRAC[0056] Copied 4 bytes to pipe pipes/pg-copy-from/1.pipe  connectTime="2024-12-02 11:29:17.503217 +0800 CST m=+9.504800709" connectionDb=public connectionID=1 query="COPY test.tb1 (id, num, data) FROM STDIN;"
DEBU[0056] Received message: {"Type":"CopyDone"}
TRAC[0056] Handling message: *pgproto3.CopyDone
ERRO[0056] Invalid Input Error: CSV Error on Line: 1
Original Line: 123
Expected Number of Columns: 3 Found: 1
Possible fixes:
* Enable null padding (null_padding=true) to replace missing values with NULL
* Enable ignore errors (ignore_errors=true) to skip this row

  file = pipes/pg-copy-from/1.pipe
  delimiter = 	 (Set By User)
  quote = \0 (Set By User)
  escape = \0 (Set By User)
  new_line = \n (Auto-Detected)
  header = false (Set By User)
  skip_rows = 0 (Auto-Detected)
  comment = \0 (Auto-Detected)
  date_format =  (Auto-Detected)
  timestamp_format =  (Auto-Detected)
  null_padding = 0
  sample_size = 20480
  ignore_errors = false
  all_varchar = 0
  connectTime="2024-12-02 11:29:17.503217 +0800 CST m=+9.504800709" connectionDb=public connectionID=1 query="COPY test.tb1 (id, num, data) FROM STDIN;


Updates for sails v0.11

Have there been any code updates to this library for sails v0.11?

Bump. This should be a simple fix; seems like all one has to do is exchange all the references to socket in any form (window.socket, etc) with io.socket.

I did that but the .fetch() doesn’t seem to be working.

On Feb 21, 2015, at 1:53 PM, Karina Chow<EMAIL_ADDRESS>wrote:
Bump. This should be a simple fix; seems like all one has to do is exchange all the references to socket in any form (window.socket, etc) with io.socket.
—
Reply to this email directly or view it on GitHub https://github.com/balderdashy/backbone-to-sails/issues/12#issuecomment-75395005.

If you're in a bind, I forked it and tweaked it to work with v0.11
Note though: I haven't really done any tests other than seeing it work with my own projects
https://github.com/karomancer/backbone-to-sails
It appears not only did we have to change socket references to `io.socket

Can you guys test this fork and give feedback? It's currently in test phase. Thank you in advance.

Any news about an update for sails 0.11v ?
Best,


Error being thrown - Error: Please check options: -m, -d.

Describe the bug
Error being thrown when using from CLI
Error: Please check options: -m, -d.

To Reproduce
Steps to reproduce the behavior:

I am using version 1.4.18
I am running bundle-scss from my package.json inside scripts

"bundle": "bundle-scss -m './assets/scss/scss.scss' -d './assets/scss/bundled.scss'"

Expected behavior
This worked previously
Desktop (please complete the following information):

MacOS Mojave

hi, thanks for reporting it. Looking into this

@coxrichuk thanks for catching this up, the issue has been fixed in 1.4.19. let me know if it fixed for you


Apps tab on package page should have a selector for price region

And use cc cookie obviously. Right now it uses US prices without any way to change them.

Done.


TypeError: The "path" argument must be of type string. Received undefined

Pre-flight checklist

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project uses.
[x] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Electron Forge version
7.5.0
Electron version
v33.0.2
Operating system
OpenSuse  15.5
Last known working Electron Forge version
unknown
Expected behavior
Hi Please look at https://github.com/zeromq/zeromq.js/issues/676 and
https://github.com/webpack/webpack/issues/18987
When I add in zeromq as either a devDependency or a regular dependency,
and when I import any (working!) zeromq test code into index.ts (or renderer.ts), it begins to barf.
Here are the steps to reproduce this issue:
test-ui.zip

npm init electron-app@latest test-ui -- --template=webpack-typescript
unpack test-ui.zip into the directory. It contains updates to activate preact, sass, zeromq, etc..
npm start (it should work)
paste any working zeromq test code into either src/index.ts or src/renderer.ts
npm start to see it fail like above

Here is a sample bit of (working) code you can paste into index.ts to see it fail
import * as zmq from "zeromq"

interface WatcherDesc {
  topic:string;
  sock:zmq.Subscriber;
};
class SocketMap extends Map<string,WatcherDesc> {};
class LogWatcher {
  url:string;
  sockets:SocketMap;
  constructor(topic:string, url:string="tcp://localhost:35501") {
    this.url = url;
    this.sockets = new SocketMap();

    const sock = new zmq.Subscriber();
    sock.connect(url);
    sock.subscribe(topic);
    console.log(`connected to ${url} for ${topic}`);
    this.sockets[topic] = { topic, sock  };

    // console.log(this.sockets);
    console.log(`for topic ${topic}`, this.sockets[topic]);
  };
};

const LM = new LogWatcher('foo');

Actual behavior
Error: Failed to load zeromq.js addon.node: TypeError: The "path" argument must be of type string. Received undefined
TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received undefined
    at Object.join (node:path:1244:7)
    at findAddon (/home/work/test-app/test-ui/.webpack/main/index.js:2192:70)
    at ./node_modules/zeromq/lib/load-addon.js (/home/work/test-app/test-ui/.webpack/main/index.js:2221:15)
    at __webpack_require__ (/home/work/test-app/test-ui/.webpack/main/index.js:2296:42)
    at ./node_modules/zeromq/lib/native.js (/home/work/test-app/test-ui/.webpack/main/index.js:2241:38)
    at __webpack_require__ (/home/work/test-app/test-ui/.webpack/main/index.js:2296:42)
    at ./node_modules/zeromq/lib/index.js (/home/work/test-app/test-ui/.webpack/main/index.js:1560:16)
    at __webpack_require__ (/home/work/test-app/test-ui/.webpack/main/index.js:2296:42)
    at ./src/LogWatcher.ts (/home/work/test-app/test-ui/.webpack/main/index.js:1140:26)
    at __webpack_require__ (/home/work/test-app/test-ui/.webpack/main/index.js:2296:42)
    at findAddon (/home/work/test-app/test-ui/.webpack/main/index.js:2214:15)
    at ./node_modules/zeromq/lib/load-addon.js (/home/work/test-app/test-ui/.webpack/main/index.js:2221:15)
    at __webpack_require__ (/home/work/test-app/test-ui/.webpack/main/index.js:2296:42)
    at ./node_modules/zeromq/lib/native.js (/home/work/test-app/test-ui/.webpack/main/index.js:2241:38)
    at __webpack_require__ (/home/work/test-app/test-ui/.webpack/main/index.js:2296:42)
    at ./node_modules/zeromq/lib/index.js (/home/work/test-app/test-ui/.webpack/main/index.js:1560:16)
    at __webpack_require__ (/home/work/test-app/test-ui/.webpack/main/index.js:2296:42)
    at ./src/LogWatcher.ts (/home/work/test-app/test-ui/.webpack/main/index.js:1140:26)
    at __webpack_require__ (/home/work/test-app/test-ui/.webpack/main/index.js:2296:42)

Steps to reproduce
See above
Additional information
Help! Please!

Please see https://github.com/webpack/webpack/issues/18987
and
https://github.com/Richienb/node-polyfill-webpack-plugin/issues/58
So it seems process polyfill has bitrotted and is not providing process.arch and process.platform, I don't quite yet understand what the proper fix for this should be.
Can a kind soul please chime in?

Hey @devzzzero , I think this is unfortunately a webpack configuration issue and not a Forge issue. It looks like you resolved it here, I hope it's working for you now!: https://github.com/niksy/node-stdlib-browser/issues/33
I'm going to close this issue, but please let me know if you do think this is a Forge issue and we can try to address it


Request my personally deployed embedding service via post

Describe the bug

Is the embedding service requested by get? My service is post, error 405 is requested according to the document, may I ask how to change to post to request my embedding service?
To Reproduce
Steps to reproduce the behavior:

use Xinference to deploy embedding serveice
request meili
xinference error 405 is reported

Expected behavior
Compatible post request
Screenshots

Meilisearch version:
v1.8.2

Hey @ZimaBlueee, thanks for your feedback!
Since this is more a feature request than a bug, could you please send us a message on the experimental feature discussion? https://github.com/orgs/meilisearch/discussions/677

I made a mistake. Thank you very much for your answer


feature: add summary metrics to synth private location

Relevant information
Adds summary metrics to synthetics private location entity.
Checklist

[ ] I've read the guidelines and understand the acceptance criteria.
[ ] The value of the attribute marked as identifier will be unique and valid.
[ ] I've confirmed that my entity type wasn't already defined. If it is I'm providing an
explanation above.

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

@naxhh can you take another look? I made the requested changes :D


Subtitle in Safari Iphone

Hi,
I want to report that the subtitles are not displayed in full screen in the safari iphone XR latest version.
regards

I just read that it was already published at https://github.com/fluid-player/fluid-player/issues/368
Some solutions that occur to me, without knowing if they can be carried out, are:
1- Copy the jwplayer solution
2- In case you are using m3u8, make the subtitle known in m3u8 directly

This is not something we can address, see #368


feat: Gas consumption script

Overview
This PR adds a test suite that we can use to measure the gas consumed by standard txs and modifies the network tests so that we can use custom genesis parameters. We still have to add more tx types and some visualization would be nice.
Checklist

[ ] New and updated code has appropriate documentation
[ ] New and updated code has new and/or updated testing
[ ] Required CI checks are passing
[ ] Visual proof for any user facing features like CLI or documentation updates
[ ] Linked issues closed with keywords

this is still a draft, and is not worth reviewing atm

created a few plots using data collected using the test here
gas_investigation.pdf

We very likely won't run this test to trace the gas consumption very often, and there is quite a bit of code, so we should just keep them in their own repo.
The only thing this PR should do is enable the investigations to be ran by including the gas consumption monitoring logic imo

closing this, as we don't have plans on merging


Struct arrays crashed tests

Aim
I'm trying to initialize an array of a user-defined struct.
Expected Behavior
If this is meant to be supported, then my code should run.
If not, then an appropriate error message should be printed.
Bug
I get the following error output.
[noir_playground] Testing test_main... The application panicked (crashed).
Message:  Variable LocalId(0) was defined twice in ssa-gen pass

To Reproduce

Define a struct
Initialize an array of the struct in main
Call main in a test and run nargo test

struct Box {
    x: Field
}

fn main() {
    let boxes = [Box  {x: 1}; 8];
}

#[test]
fn test_main() {
    main();
}

Installation Method
Binary
Nargo Version
nargo 0.10.5 (git version hash: 91efe44d17762ea883d9a576d530049ad20f1393, is dirty: false)
Additional Context
No response
Would you like to submit a PR for this Issue?
No
Support Needs
No response

Interesting, thanks for opening this issue @bryanwee023. I've replicated this on current master.
Arrays of structs are supported but it looks like the sugared syntax to create an array filled with the same element doesn't, i.e. [Box  {x: 1}, Box  {x: 1}, ..., Box  {x: 1}] will work but [Box  {x: 1}; 8] won't.

This seems to also require you to initialize the struct inside the array creation. As a workaround, I'd move this outside and use something like the below.
fn main() {
    let box = Box {x: 1};
    let boxes = [box; 8];
}


Bspd software debug james

Added code to test CAN. Not complete.

No need to accept pull, but you can download this branch if desired


Is it possible to train multi-label multi-tag model with ktrain?

Hi, I'm asking me if I can train a model for "Multi-Label Text Classification for Stackoverflow Tag Prediction" using ktrain.
There is some example?
Thanks a lot!

Hello,
There is an example of multilabel text classification shown in the text classification tutorial notebook.
Thanks.


⚠ PrivateHD has degraded performance

In 9ca73d3, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 1988 ms

Resolved: PrivateHD performance has improved in bbd5eee.


Backport Fix goroutine leak in file launcher

What does this PR do?
Backport https://github.com/DataDog/datadog-agent/pull/13149
Add back the continue statement when a tailer already exists. This was missed in the refactor in https://github.com/DataDog/datadog-agent/pull/10434
Without this, duplicate tailer are started when new sources are added causing us to leak goroutines.

Motivation
Bug found in QA

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
Check that we are not leaking goroutines in the next RC.

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

CI failures are unrelated


Expanded copy number annotation

Hello team,
I would like to plot expanded copy number calls:
events: "Amplification","CNLOH","Diploid","Gain","Gain+LOH","HETLOSS","HOMDEL","Tetraploid"
Currently, when I run this code, I get very messy code and am going to try to use expanded "additionalFeature" input.
How feasible would it be to add this to the tool? Also, if there is another tool or feature that I could use, I would be grateful if you point it out.
Thanks
Erika
Command
laml.plus.cn = read.maf(maf = laml.maf, cnTable = cn_data, verbose = FALSE)
oncoplot(maf = laml.plus.cn, genes = imp_genes)

Session info
Run sessionInfo() and post the output below
R version 3.6.1 (2019-07-05)
Platform: x86_64-apple-darwin15.6.0 (64-bit)
Running under: macOS Mojave 10.14.6

Matrix products: default
BLAS:   /System/Library/Frameworks/Accelerate.framework/Versions/A/Frameworks/vecLib.framework/Versions/A/libBLAS.dylib
LAPACK: /Library/Frameworks/R.framework/Versions/3.6/Resources/lib/libRlapack.dylib

locale:
[1] en_US.UTF-8/en_US.UTF-8/en_US.UTF-8/C/en_US.UTF-8/en_US.UTF-8

attached base packages:
[1] grid      stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
 [1] RColorBrewer_1.1-2 lubridate_1.7.8    devtools_2.2.2     usethis_1.6.0      reshape2_1.4.4     forcats_0.5.0      stringr_1.4.0
 [8] dplyr_0.8.5        purrr_0.3.3        readr_1.3.1        tidyr_1.0.2        tibble_3.0.0       ggplot2_3.3.0      tidyverse_1.3.0
[15] maftools_2.3.10

loaded via a namespace (and not attached):
 [1] httr_1.4.1        pkgload_1.0.2     jsonlite_1.6.1    splines_3.6.1     modelr_0.1.6      assertthat_0.2.1  cellranger_1.1.0
 [8] remotes_2.1.1     sessioninfo_1.1.1 pillar_1.4.3      backports_1.1.6   lattice_0.20-41   glue_1.4.0        digest_0.6.25
[15] rvest_0.3.5       colorspace_1.4-1  Matrix_1.2-18     plyr_1.8.6        pkgconfig_2.0.3   broom_0.5.5       haven_2.2.0
[22] scales_1.1.0      processx_3.4.2    generics_0.0.2    farver_2.0.3      ellipsis_0.3.0    withr_2.1.2       cli_2.0.2
[29] survival_3.1-12   magrittr_1.5      crayon_1.3.4      readxl_1.3.1      memoise_1.1.0     ps_1.3.2          fs_1.4.1
[36] fansi_0.4.1       nlme_3.1-145      xml2_1.3.1        pkgbuild_1.0.6    tools_3.6.1       data.table_1.12.8 prettyunits_1.1.1
[43] hms_0.5.3         lifecycle_0.2.0   munsell_0.5.0     reprex_0.3.0      callr_3.4.3       compiler_3.6.1    rlang_0.4.5
[50] rstudioapi_0.11   labeling_0.3      testthat_2.3.2    gtable_0.3.0      DBI_1.1.0         R6_2.4.1          knitr_1.28
[57] rprojroot_1.3-2   desc_1.2.0        stringi_1.4.6     Rcpp_<IP_ADDRESS>      vctrs_0.2.4       wordcloud_2.6     dbplyr_1.4.2
[64] tidyselect_1.0.0  xfun_0.12

Hi,
Oncoplot tends to get messier with too many variables. Just to let you know, you're using a bit older version of the package (2.3.10). Current version is 2.4.05 as on GitHub and supports plotting of multiple CN events - the exact thing that you're planning on plotting. (Take a look at issues #520 and #490)
If you want to try other tools, afaik below are the two options:

oncoprint from ComplexHeatmap
comut in python


Error when updated to latest version.

ReactNativeJS: Warning: React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in. Check the render method of TabViewTransitioner.

Hi @rendomnet! Did you track down the piece of code generating this error?

I'm seeing this too. Have you found a fix for this @rendomnet?


Support Running condition

What would you like to be added:
I would like to support the new JobSetConditionType and Running so that researchers can easily find out if the JobSet is still running.
https://github.com/kubernetes-sigs/jobset/blob/235745c5ed1bd70f54f36aa8782122e3326a04b6/api/jobset/v1alpha2/jobset_types.go#L50-L62
However, the current batch/v1 Job doesn't support the Running condition: https://github.com/kubernetes/kubernetes/blob/4f04dffe5b2cd652a20b362eaea30164e3e5ea54/pkg/apis/batch/types.go#L612C4-L627
So, we need to define when we can say the Job/JobSet is still running. Starting this discussion from KEP #2804 might be better.
Why is this needed:
Currently, we represent a similar context with Suspend=False, Completed=False, and Failed=False, but that representation approach is not intuitive. So, I would like to express the Running state clearly.

cc: @andreyvelich @ahg-g

/kind feature

hi! @tenzen-y There are some discussions on this issue  https://github.com/kubernetes-sigs/jobset/issues/545

I have considered adding an "Active" or "Running" field before but it would be inconsistent with the Job API, where as you noted the Active/Running state is implicit (i.e., not failed, completed or suspended).
@alculquicondor I am curious, is there a reason an "active" or "running" condition is not part of the Job status?

hi! @tenzen-y There are some discussions on this issue #545

Thank you for pointing this out. I would like to focus on the new condition type separate from counting in this issue.

I have considered adding an "Active" or "Running" field before but it would be inconsistent with the Job API, where as you noted the Active/Running state is implicit (i.e., not failed, completed or suspended).
@alculquicondor I am curious, is there a reason an "active" or "running" condition is not part of the Job status?

@danielvegamyhre I have the same concern as you. So, as I mentioned in the issue description, it might be better to start discussing the Running condition in https://github.com/kubernetes/enhancements/pull/3524.
Additionally, actually, Aldo mentioned the Running condition in the batch/v1 Job here: https://github.com/kubernetes/enhancements/pull/3524#discussion_r974650995

The question is what does "running" mean, when you have multiple pods. A field such as ready (for the number of ready pods) in the job status, gives the controllers enough information without having to define what a "Running" condition would be.
Maybe we should aim for a similar value in jobset status? Maybe it can just add up all the active fields from all of its jobs?

The question is what does "running" mean, when you have multiple pods.

Ideally, Running means that all PyTorch workers are scheduled on the Kubernetes control plane, pulled the image, and ready to process Model Training.
As of today, we marked PyTorchJob Running if master replica is active or one of the workers is active in case PyTorchJob doesn't have a master.
From my point of view, that is wrong, since model training can start only when all replicas are active.
Image pull might take significant amount of time especially for the large models, and Job should not be in Running condition during that time. cc @johnugeorge @tenzen-y

Maybe it can just add up all the active fields from all of its jobs?

That looks good to me.

Maybe it can just add up all the active fields from all of its jobs?

That looks good to me.

@andreyvelich @alculquicondor Since active includes pending pods I think we would want is the ready values from each child Job.
JobSet already tracks the number of "Ready" jobs for each ReplicatedJob in the ReplicatedJobStatus here. A child Job is counted as ready if ready pods + completed pods >= expected pod count for the job, where the expected pod count is minimum of parallelism and completions.
So we could consider a JobSet "Active/Running" if all ReplicatedJobs have replicatedJobStatus.Ready == replicatedJob.Spec.Replicas

Oops, sorry, I meant to say ready indeed :)

So we could consider a JobSet "Active/Running" if all ReplicatedJobs have replicatedJobStatus.Ready == replicatedJob.Spec.Replicas

@tenzen-y Do we need to set a condition for that given that the info is already in the status?

Ping @tenzen-y @andreyvelich
Is this still necessary?

Yeah, I know that @googs1025 did some work to improve JobSet conditions in this PR: https://github.com/kubernetes-sigs/jobset/pull/594.
Also, I understand that is hard to define what does Running mean exactly.
So maybe we could just follow @danielvegamyhre suggestion here: https://github.com/kubernetes-sigs/jobset/issues/571#issuecomment-2118362246
From my perspective we should add this condition to the Job, since user assumes the following conditions when they create training job:
Created -> Running -> Succeeded
Any thoughts @tenzen-y ?

@ahg-g @andreyvelich @kannon92 As we discussed here, I'm hestating to add this running condition since at first glance, this Running  phase / condition can be introduce easily, but decision of Running is too hard.
So, I would like not to introduce the Running phase/condition in the JobSet side. But, I'm wondering if the Kubeflow Training v2 can introduce the Running condition type depending on the JobSet Terminal Phase (https://github.com/kubernetes-sigs/jobset/blob/56c77da8303562b0bb034ee237a4264b59a22902/api/jobset/v1alpha2/jobset_types.go#L137-L139).

In conclusion, I don't prefer to introduce the Running phase/condition in the JobSet side.

Sounds good. I think phase can suffice.
Maybe it’s something you can add to the TrainJob KEP and we can close this issue.

That makes sense. Anyway, let's close this issue now.
Once we can find the use cases that it can get over the drawbacks, we can revisit here.
/close


Adding use_ajax flag to MLFlow instances

Alowing compatibility with both REST and AJAX endpoints.

Are there any other MLFlow implementations/deployments using /ajax-api other than DagsHub? (I have only seen mention of /ajax-api in https://github.com/JuliaAI/MLFlowClient.jl/issues/35#issuecomment-1880158336)
Wouldn't a more natural API be to provide the API base url, e.g. http://localhost:5000/api or http://dagshub.com/ajax-api, instead of providing the MLFlow UI base url (http://localhost:5000) and having a kwarg with a not-very-descriptive name (use_ajax) ?

I don't know if there's other than DagsHub.
I consider the API base URL approach as a good one. With that, we don't need to be aware of if it's an ajax-api or api.
Need opinions about this decision.
@ablaom @deyandyankov

No objections but this definitely needs to be tagged breaking. So if we have any other breaking changes in mind...

I'm very excited about this PR! I was about to open a new PR myself, as the latest version is broken for us (past ones work).
The following commit broke the ability to work with Databricks: https://github.com/JuliaAI/MLFlowClient.jl/commit/e27ac1026627327d58fd4b63aeeef62c738b0977
We use Databricks-managed MLFlow and we need the "HOST/api/2.0" format rather than the current "/ajax-api" one.
The proposed approach to provide the base URL including the relevant API path would work great for us!

No objections but this definitely needs to be tagged breaking. So if we have any other breaking changes in mind...

A very much related breaking change would be to change the type of apiversion::Union{Integer, AbstractFloat} to, e.g. api_version::VersionNumber.

@pebeto Would it be, please, possible to merge this PR and tag a new version?
Given that the commit linked above was breaking (anything above v0.4.4), anyone who updates (even if having compat to "0.4") might get the same error as we did.

@pebeto Would you please consider merging the PR to have a working minor version release?
The bug introduced in 0.4.5 (the ajax in path) still persists.


Support NO_COLOR environment variable

Per no-color.org, support the NO_COLOR environment variable to disable output coloration.
This is hard to configure otherwise, as a simple shell alias is not enough since cargo is
used to invoke cargo-expand.
Motivation: I prefer a lack of syntax highlighting, and I have to use --color=never anyway with my light theme:

Published in cargo-expand 1.0.9.


Is this a bug?

If this is not a bug, I would be appreciated if anyone can help guide me about this syntax.
Dart version: 2.12.2
OS : Window
void main() {
  List a = [1,2,3];

  List b = [4,5,6];

  int testA = (a.length??0) + b.length;
  int testB = a.length??0 + b.length;

  print("testA = $testA");//testA = 6

  print("testB = $testB");//testB = 3

}

.length is not nullable, so you don't need null handling.
if you change the code to this, the warning will gone.
void main() {
  List? a;
  List b = [4,5,6];

  int testA = (a?.length ?? 0) + b.length;
  int testB = a?.length ?? 0 + b.length;

  print("testA = $testA");//testA = 6
  print("testB = $testB");//testB = 3
}

@WizNeo6  Maybe I post the issues wrong. What I wonder is why "testA" and "testB" give different result output.

testA and testB give different results because of the syntactic structure: + binds more tightly than ??, so what you've written means the following:
int testB = a?.length ?? (0 + b.length);

I'll close this issue because it is all working as intended.

@eernstg What if the list is null, I use that ?? in case the my list is null so I can give default value instead of getting crash.

Oh... I see. Thank you for explanation.


Potential fix for wildlife encounter animal poses

(Issue #420)
This converts the animal xyz coordinates to a global Cartesian frame before calling the SphericalFromLocal function in wildlife_scoring_plugin.cc. I am unsure exactly why this works, but I properly receive the spherical coordinates from the vrx/wildlife/animals/poses topic now.
I do notice that the function I added that converts the xyz coordinates to a global Cartesian frame (GlobalFromLocal) simply rotates the position of the animals about the Gazebo origin frame (both x and y are negated).

@caguero I've been digging into the SphericalCoordinates source code this afternoon to try to figure out this issue.
Our world frame (which should be CoordinateType LOCAL) is ENU, and thus coincident with GLOBAL.  I validated this with a call to HeadingOffset() that returned 0, just to be sure.  So, in theory, the call to GlobalFromLocal() shouldn't do anything at all.  In fact, though, conversions from LOCAL to anything else swap the signs on the X and Y inputs.  This happens in both the PositionTransform() and VelocityTransform() methods.
I also validated that the sign swap is what's happening in our wildlife worlds - can provide code snippet if desired.
Why this fix appears to work: in the call to GlobalFromLocal(), the signs are swapped.  Then in the call to SphericalFromLocal(), they're swapped back.
The temporary fix for us should be to do the transform from GLOBAL to SPHERICAL via PositionTransform().
I am going to do more math and see if I can figure out why the functions were written this way and what should really be happening.

Just as a clear demonstration of the error, here's some output from the default wayfinding world.  We enter the waypoints as lat/long, and convert via SphericalToLocal(), which works correctly and places the marker where we expect.  I added the reverse conversion to show what the LocalToSpherical() function returns - you can see the differences.  SphericalToLocal assumes ENU, while LocalToSpherical is reversed in X,Y.
[Msg] Waypoint, Spherical: Lat = -33.7227 Lon = 150.674
[Msg] Waypoint, Local: X = 525.794 Y = -171.5 Yaw = 1.21756
[Msg] Waypoint, converted from local back to spherical: -33.7258 150.685 1.24152
[Msg] Waypoint, Spherical: Lat = -33.7221 Lon = 150.674
[Msg] Waypoint, Local: X = 550.984 Y = -237.315 Yaw = 1
[Msg] Waypoint, converted from local back to spherical: -33.7264 150.686 1.02821
[Msg] Waypoint, Spherical: Lat = -33.7226 Lon = 150.675
[Msg] Waypoint, Local: X = 434.679 Y = -179.863 Yaw = 1
[Msg] Waypoint, converted from local back to spherical: -33.7258 150.684 1.01734

This is an issue in the Stationkeeping scoring plugin as well, but only in the case where the goal pose is entered in cartesian coords rather than lat/long, which is probably why we haven't seen it in practice.
For style discussion before making any recommendations on this pull request, I've got proposed changes in a new branch: jessica/spherical_fix

@alexperez33 , @j-herman, I'm going to close this pull request in favor of #428 , as I think it properly handles this issue. Thanks a lot for the useful discussion here.


Export NodeClickHouseClient as value from client-node for verbatimModuleSyntax=true environments

Use case
Currently, NodeClickHouseClient is exported as a type in the client-node package.
https://github.com/ClickHouse/clickhouse-js/blob/a332672bfb70d54dfd27ae1f8f5169a6ffeea780/packages/client-node/src/index.ts#L1-L4
While I understand that it is an architecture decision so developers are forced to use the createClient function, it is problematic for dependency injection frameworks such as Nest.js or tsyringe.
In Nest.js, the following code does not work with tsconfig verbatimModuleSyntax=true:
import { createClient } from '@clickhouse/client';
import { NodeClickHouseClient } from '@clickhouse/client';
// ˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆˆ
// 'ClickHouseClient' resolves to a type-only declaration and must be imported using a type-only import when 'verbatimModuleSyntax' is enabled

@Module({
  providers: [
    {
      provide: NodeClickHouseClient,
      useFactory: () => {
        return createClient({
          url: process.env.CLICKHOUSE_URL,
          clickhouse_settings: {
            async_insert: 1,
            wait_for_async_insert: 1,
          },
        });
      },
    },
  ],
  exports: [NodeClickHouseClient],
})
export default class ClickHouseModule {}

Describe the solution you'd like
I would like to export the ClickhouseClient as a value instead of type:
-export type {
-  NodeClickHouseClient as ClickHouseClient,
-  QueryResult,
+export {
+  NodeClickHouseClient as ClickHouseClient,
+  type QueryResult,
} from './client'

Describe the alternatives you've considered
My current workaround is to import { NodeClickHouseClient } from<EMAIL_ADDRESS>which is far from ideal.
import { createClient } from '@clickhouse/client';
import { NodeClickHouseClient } from<EMAIL_ADDRESS>
@Module({
  providers: [
    {
      provide: NodeClickHouseClient,
      useFactory: () => {
        return createClient({
          url: process.env.CLICKHOUSE_URL,
          clickhouse_settings: {
            async_insert: 1,
            wait_for_async_insert: 1,
          },
        });
      },
    },
  ],
  exports: [NodeClickHouseClient],
})
export default class ClickHouseModule {}

Additional context
Happy to submit a PR that goes into this direction if approved.

If it helps, let's change the export. Could you make a PR? For consistency, it can also be exported similarly in the web version.


ARTEMIS-1186 Consumer.receive hangs if http acceptor with non-zero ba…

…tch-delay is configured
https://issues.apache.org/jira/browse/ARTEMIS-1186
https://issues.jboss.org/browse/JBEAP-6570
Upstream PR: https://github.com/apache/activemq-artemis/pull/1297

Can you close this manually? or rebase your branch and push -f?
The commit ID changed as I merged with a rebase. This closes message doesn't work on 1.x (just on master).

@TomasHofman  please close this. already merged with a rebase.


Set up long term Fusion Tutorial  hosting

Move Fusion tutorials from GitHub to the user-facing project site OpenGEE.org.
Assigned to Chris Camp.

Closing as duplicate of https://github.com/google/earthenterprise/issues/40


add snap.svg requirement to documentation

visual-search-circle requires snap.svg
at the moment it is just a comment in the code;
please add it to the documentation (also on docs.jspsych.org), best with a download link

I'll make sure this gets in. Thanks for pointing it out.
If you want to suggest edits to the docs directly, you can modify the content in the /docs folder of the repository. The docs.jspsych.org site is built from the markdown files in the /docs folder.

Added in dev branch. Will be live when 4.1 is pushed.


Crash when trying to remove machine with wrench (+230 mods)

Issue Description:
Attempting to remove a machine with the Yeta wrench results in crash, that is sneaking and right clicking the machine, not expecting much since this is a heavily modded world

EnderIO: 5.1.55
EnderCore: 1.12.2-0.5.73
Minecraft: 1.12.2
Forge: <IP_ADDRESS>54
SpongeForge? no
Optifine? yes
Single Player and/or Server? Server

crash-2020-05-05_17.37.10-client.txt
[pastebin/gist/etc link here]

Try without optifine i have no issue shift right clicking on machines

PLease report this to OpenTerrainGenerator.

@tyler489
Removed it long ago

@Tyler works, seems having the files in there regardless of .jar.disabled tag results in it fiddling with it
Thank you.

technicly it was OTG core which auto installs from OTG base that was causing the issue I believe

OTG comes in 2 parts. The mod itself and then the parts that are specific for Forge (or Bukkit). This crash happens when only the Forge-specific part is active, as it tries to access the (missing) mod without checking if it is there.


Close button not displayed properly when selecting Facet Choices as Tab Separated Values

Using OpenRefine 2.6 beta (with chrome on ubunut machine) the close button in the window Facet Choices as Tab Separated Values is not displayed properly.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/1994234-close-button-not-displayed-properly-when-selecting-facet-choices-as-tab-separated-values?utm_campaign=plugin&utm_content=tracker%2F32795&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F32795&utm_medium=issues&utm_source=github).

Fixed in 2.6-rc1


Add a button to change modes between Day and Night mode.

Add Toggle button which is use to change colour theme from day to night mode and via-versa (i.e. from light theme to dark theme)

please assign this issue to me @somyaranjan26

I will work on both the issue.

@somyaranjan26 I would like to work on this issue. please assign this to me.

Actually @Adyasha8105, there is only one issue per participant in each project.

okay @somyaranjan26


Add docs for cqlsh

The docs for cqlsh were a bit long to include in "Getting Started", so I put them in a separate section for now and just put a little info in the "Getting Started" section.

Cherry-picked, thanks.


CosmosDB Gremlin support

First off, thanks for making this - looks like a very useful tool.
I tried to use it with Gremlin API from Microsoft CosmosDB, but couldn't figure out the authentication.
I have a wss:// URI and tried putting the 'key' from Azure portal as the token, but it wouldn't authenticate.
Anyone know how to use this with CosmosDB?
Thanks!

@kirk-marple Can you point me to the documentation in Microsoft CosmosDB, that has the information about how you are trying to authenticate.

I'm personally using the .NET SDK to connect, which hides a bit of it, but I found this documentation for Python:
client = client.Client('wss://<YOUR_ENDPOINT>.gremlin.cosmosdb.azure.com:443/','g',
username="/dbs/<YOUR_DATABASE>/colls/<YOUR_COLLECTION_OR_GRAPH>",
password="<YOUR_PASSWORD>")
https://docs.microsoft.com/en-us/azure/cosmos-db/create-graph-python#review-the-code
This doesn't seem to work in Graph Explorer today, given the formatting of username into the URL, I think.

WebSocket API doesn't support authentication with username and password. So if you are using ws or wss protocol with graph explorer, authentication doesn't work. Authentication is supported only for http protocol by graph explorer at the moment.  There is no way for me to support authentication with wss protocol as the browser itself doesn't support it in WebSocket API.
Alternatively, you can use http based gremlin api, if cosmosDB provides it. Just make sure CORS is enabled for your http based gremlin api.

Thanks, I tried the HTTP API from Cosmos last night, but didn't have any luck.  It's opened up to all networks (Gremlin API doesn't have explicit CORS option)
I think it must be something in the authentication, but I couldn't figure it out.  Bummer, since this looks like a great tool to try.

@kirk-marple there is no out of the box feature that supports authentication with websockets from browser at the moment. But I'm in the process of creating another project invana-engine which is supposed to act as a middleware that can talk to graph-explorer and your cosmosDB.
I shall update you soon on this issue. Thanks!