Removing ARChon

What files remain in Chrome (Windows 7) after removing the unpacked extension? I visit play store and I still get this app is installed message.

No files will remain in chrome unless you have loaded an app which was given permission to write to the hard drive. In that case there could be a few empty folders + whatever files that specific app added. Removing Archon should give you a clean uninstall.
The only reason the playstore would say that you have the app is installed on your android, installed the app on your Android in the past, or if you used an apk downloader exstention that worked by tricking Google Play Store into thinking that your browser was you android phone/tablet downloading the App.
In other words, don't worry about it. It just means that at one point in time, Goolge thought you downloaded the app from the Playstore, it is not checking your computer.

Haven't. The only playstore interaction in Chrome that I've done was only through ARChon. Removing it did not revert whatever it did to make playstore think I'm on android and even now I get install app choice or app is installed. So somewhere there's still code that interferes.

And it's not just on the main profile, I get this throughout Chrome, all I have to do is to login to a Gmail account.

I visit play store and I still get this app is installed message.

Could you provide a screenshot?
You might have to clear your extension cache. Chrome copies / caches some extension sources

Will this remove Tampermonkey's installed scripts, ABP's custom filters or other extension's settings?

@ViperCode7 So, what exactly is the problem then?
Did you actually install the Google Play store app on your computer and it thinks it still exists, is that the problem?
If so, you should know that to Google, your device is simply offline. They don't know that it doesn't exists anymore.

As I've said enough times, nothing else is installed. No Play store app, no other Android emulator app or anything which could trick the site to see Chrome as an Android Chrome instead of the usual PC Chrome.

Based on your screenshots, it doesn't look like anything is wrong. I do not see anything indicating that Google is seeing anything other than usual PC Chrome.
In case you are unfamiliar, the Play Store always gives you an install button to remotely install on devices connected to your account when you are signed in on the computer.
You can rest assured, nothing is installed on or wrong with your computer :smile_cat: :smile: :smiley:

OK, I see... so ARChon emulated a device, so that apps got installed, but now that it's gone, the google accounts still keeps track of the emulated devices by reference even if it's inactive. Apps just get abstractly installed, when in fact they're not installed. That seems confusing... but now I'm left with an inactive device which I can't remove from my account, so I'm left with the play store the way it is. Tried to hide it, I still get the install options and it's still listed at devices on which to install.

Exactly.
BTW, How did you get the play store app onto Archon? I'd really like to know how you achieved that.
I think you can remove it the device from the list.

Does it put your PC name into this list:

?

Don't remember how I did it... but I loaded another Chrome profile in which I never installed this sort of apps or extensions, in there I signed in with a different gmail account and I got the same options to install sans a registered device or the play store app.
On my main account I get only the device, but not my PC. Now I've hidden the devices and I get This app is incompatible with your device under the app title but when I click Install I get No eligible devices in the list. On the clean profile I just get the message You don't have any devices and it points me to the the white shopping bag icon.


deamon fails after starting with 'getpwnam(): name not found: newrelic'

I run the agent with:
newrelic_plugin_agent -c /etc/newrelic/newrelic_plugin_agent.cfg
And immediately get this result:
Traceback (most recent call last):
File "/usr/lib64/python2.7/site-packages/helper/unix.py", line 87, in start
self._daemonize()
File "/usr/lib64/python2.7/site-packages/helper/unix.py", line 160, in _daemonize
uid = self._get_uid()
File "/usr/lib64/python2.7/site-packages/helper/unix.py", line 265, in _get_uid
return pwd.getpwnam(self.controller.config.daemon.user).pw_uid
KeyError: 'getpwnam(): name not found: newrelic'
If i run with -f it seems to run ok.

+1, thanks for this.

Thanks


add docker compose file

Fixes #875
Proposed Changes

Add compose.yaml file.  It is another way to manage and run commands in the docker.

You can check ref
you can run
docker compose up

to build and run your container
Checklist

[ ] CONTRIBUTING guide has been followed.
[ ] PR is based on the current GaNDLF master .
[ ] Non-breaking change (does not break existing functionality): provide as many details as possible for any breaking change.
[ ] Function/class source code documentation added/updated (ensure typing is used to provide type hints, including and not limited to using Optional if a variable has a pre-defined value).
[ ] Code has been blacked for style consistency and linting.
[ ] If applicable, version information has been updated in GANDLF/version.py.
[ ] If adding a git submodule, add to list of exceptions for black styling in pyproject.toml file.
[ ] Usage documentation has been updated, if appropriate.
[ ] Tests added or modified to cover the changes; if coverage is reduced, please give explanation.
[ ] If customized dependency installation is required (i.e., a separate pip install step is needed for PR to be functional), please ensure it is reflected in all the files that control the CI, namely: python-test.yml, and all docker files [1,2,3].
[ ] The logging library is being used and no print statements are left.

Looks good, but should we not be using this for all Docker builds?

It's the same with this, IMO but in an easier way.
You can modify the compose.yaml. You can choose the dockerfile, the volume, the commands that you want  to execute.

Looks good, but should we not be using this for all Docker builds?

It's the same with this, IMO but in an easier way. You can modify the compose.yaml. You can choose the dockerfile, the volume, the commands that you want to execute.

Okay, then please provide clear examples of how this improves the existing docker workflow.

I have created an example to demonstrate how to use it, but I have noticed that extra effort is required from the user to modify the compose.yaml file.
Using the docker command line
docker run -it --rm --name dataprep \
  --volume /home/researcher/gandlf_input:/input:ro \ # input data is mounted as read-only
  --volume /home/researcher/gandlf_output:/output \ # output data is mounted as read-write
  cbica/gandlf:latest-cpu \ # change to appropriate docker image tag
  construct-csv \ # standard construct CSV API starts
  --input-dir /input/data \
  --output-file /output/data.csv \
  --channels-id _t1.nii.gz \
  --label-id _seg.nii.gz

Using the docker compose file.
version: '1'

services:
  gandlf:
    build:
      context: .
      dockerfile: Dockerfile-CPU #choose the dockerfile
#    volumes:
#     - /home/researcher/gandlf_input:/input:to
       - /home/researcher/gandlf_output:/output
    entrypoint: ["gandlf"]
    command: ["construct-csv",
"--input-dir ","/input/data",
"--output-file ","/output/data.csv",
" --channels-id"," _t1.nii.gz ",
" --label-id", "_seg.nii.gz" ]

You can run  docker compose up  to run the container and execute the command.

@hasan7n since you are far more familiar with docker than I am, what are you views on this?


TypeError: value.matchAll is not a function

When using numerical values in the application.yml file and restarting the application, getting the following error
database:
  uri: some_uri
  port: 27017

error log
[Nest] 26114  - 07/18/2024, 10:46:18 PM     LOG [NestFactory] Starting Nest application...
[Nest] 26114  - 07/18/2024, 10:46:18 PM   ERROR [ExceptionHandler] value.matchAll is not a function
TypeError: value.matchAll is not a function
    at Function.lastIndexOf (/Users/my-api/node_modules/@itgorillaz/configify/src/interpolation/variables.ts:98:38)
    at Function.interpolate (/Users/my-api/node_modules/@itgorillaz/configify/src/interpolation/variables.ts:59:47)
    at Function.expand (/Users/my-api/node_modules/@itgorillaz/configify/src/interpolation/variables.ts:43:49)
    at Function.forRootAsync (/Users/my-api/node_modules/@itgorillaz/configify/src/configify.module.ts:93:34)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
Waiting for the debugger to disconnect...

When i convert the number to a string like below, its working fine
database:
  uri: some_uri
  port: '27017'

What is the reason behind the error  value.matchAll is not a function
Note: You need to restart the application to get the above error while using a numerical value

@sandeepsuvit thanks for reporting the error!
I was able to reproduce and indeed the yaml parser is converting the value to a number so it fails when interpolating the variables.
The fix will be in the next release.

@sandeepsuvit fixed on v1.2.3, note that since we assign the configuration to process.env the values are converted to string, it is a good idea to use the parse option if your config attribute is not a string, e.g.:
@Configuration
export class DatabaseConfiguration {
  @Value('database.host')
  host: string;

  @Value('database.port', { parse: parseInt })
  port: number;
}


Create DotNetNuke Library Package for MailKit

Description of problem
A new library package should be created to contain the MailKit, MimeKit, and associated references so they can be installed during installation, rather than included during the build process.
Additionally, the Cake build script should also exclude these files from the Upgrade package
Description of solution
Creating a package will prevent upgrade issues for those that have existing versions of these dependencies.
Additional context
This solution was discussed during the TAG meeting as a solution for the current upgrade issues impacting those with certain users of third-party modules

shouldn't both providers be moved into separate packages, allowing admins to select which one to install?

@sleupold At this point, we are working to solve the immediate concern, without breaking existing.


correctly return the browser response + fix the handle step nonesense

Reverting the behavior agentBrowser to remove handle step. There is some type weirdness going on and I don't want to deal with it now.

I need to fix that one test since it keeps giving us issues.


Convert four INSIDE_WINDOWS blocks to til features

This pull request converts four of our existing #ifdef (or #ifndef)
INSIDE_WINDOWS blocks to til::features:

Attempting to establish a handoff session (inside Windows only)
The ability to receive a handoff session (outside Windows only)
The DX engine (outside Windows only) and shaders (also outside only)
Whether we use numpad event synthesis for clipboard/conpty (inside
Windows only)

Most of these are using the preprocessor verison of til::feature, only
because it is more difficult to gate the inclusion of headers on
constant expressions. I'd love to prefer the compile time version.

@msftbot merge this in 1 minute


New cell types - olfactory

from https://github.com/openMetadataInitiative/openMINDS_controlledTerms/pull/459
I tried to incorporate @lzehl feedback as good as possible.

Please wait with the review until after I revised this again. I just found https://github.com/openMetadataInitiative/openMINDS_controlledTerms/issues/240 and will add the missing ones for the accessory olf. bulb and revise the existing ones.

I've updated the new cell types to be the ones from the main olfactory bulb. Since @apdavison said that there were no use cases for the accessory olfactory bulb, I would like to wait with those until after we did the clean-up for the cell types. @lzehl could you please review the PR?


error cargo run --example hello_world

cargo run --example hello_world
warning: unused manifest key: badges.travis-ci.repository
warning: unused manifest key: package.categories
warning: unused manifest key: package.categories
warning: unused manifest key: package.edition
warning: unused manifest key: workspace.exclude
warning: unused manifest key: workspace.exclude
warning: unused manifest key: workspace.exclude
error: no example target named hello_world
linux arm
second trouble 'cc' library
cargo install cc
and next
cargo run --example hello_world
error: current package believes it's in a workspace when it's not:
current:   /home/itt/imgui-rs/imgui-examples/Cargo.toml
workspace: /home/itt/imgui-rs/Cargo.toml
this may be fixable by adding imgui-examples to the workspace.members array of the manifest located at: /home/itt/imgui-rs/Cargo.toml

What's the Rust version?
rustc --version

Note that imgui-rs requires at least 1.33

Closing old issue - sounds like problem was a too-old version of Rust toolchain


[Mentorship Diary] Ian and Maria

[Mentorship Diary] Ian and Maria
(Mentorship Round 2) @iansu will be mentoring @mpaktiti about tooling in Node.js

Kickoff meeting (9 July 2019)
We met with @iansu over Zoom. We got to know each other a bit, and chatted about tooling in Node.js.
Scope & Expectations
During the mentorship period we will be working together on some ongoing projects around tooling. We haven't yet decided on exactly what but there are some interesting stuff going on, like incorporating rimraf and quik in core.
Neither of us have extended time off planned so the next 10 weeks should be productive. We will probably both be out for a few days mid-August.
Communication
We will use Slack for daily check-ins and questions. Once @mpaktiti starts working on issues we will plan for Zoom syncs/pairings as needed (weekly or biweekly).
Next Steps
Identify the first issue that @mpaktiti can start working on!

First Contribution (14 October 2019)
Recently the rimraf library was incorporated in core. The core included some basic tests but the original package included more than that, testing multiply nested directories and different file types like symlinks. My contribution was about adding more tests to the core by copying the logic of the original package (see PR https://github.com/nodejs/node/pull/29815).
Next Steps
Identify more areas for @mpaktiti to contribute.


LIMIT results in Syntax Error

Version
1.13.0
What happened?
Hello!
I tried to create a query with dynamic query/offset. Unfortunately, it returned an error.
Tickets I discovered:
https://github.com/kyleconroy/sqlc/issues?q=is%3Aissue+is%3Aopen+limit+label%3A"%3Abooks%3A+mysql"
Things I've tried:
https://github.com/kyleconroy/sqlc/issues/1439 - it appears changing the reserve words did not change the error.
Comparing the output of PG and MySQL

This works for PG and not for MySQL

PG: https://play.sqlc.dev/p/a5a03961f9a447cf433773e462ef1508f677ebc263e533fcf53f481b9224a650
MySQL: https://play.sqlc.dev/p/d6232b25b11799f71f0bae2951b41ab359c0c299c6a1287a54567696f1a5e650

Relevant log output
sqlc generate failed.
# package db
query.sql:16:11: syntax error near "sqlc.arg(row_offset), sqlc.arg(row_limit);"

Database schema
-- Example queries for sqlc
CREATE TABLE authors (
  id  varchar(255) NOT NULL,
  name text      NOT NULL,
  bio  text
);

SQL queries
-- name: ListAuthors :many
SELECT * FROM authors
ORDER BY name
LIMIT sqlc.arg(row_offset), sqlc.arg(row_limit);

Configuration
{
  "version": "1",
  "packages": [
    {
      "path": "db",
      "engine": "mysql",
      "schema": "query.sql",
      "queries": "query.sql"
    }
  ]
}

Playground URL
https://play.sqlc.dev/p/d6232b25b11799f71f0bae2951b41ab359c0c299c6a1287a54567696f1a5e650
What operating system are you using?
macOS
What database engines are you using?
MySQL
What type of code are you generating?
Go

I got this to work by using regular ? parameters inside of named parameters. The MySQL parser we're using is not happy with function calls after LIMIT and OFFSET
https://play.sqlc.dev/p/ac63085049628c6b20fd85479e03ea071df9e42f5224a025863f7b6adba84ab4
-- name: ListAuthors :many
SELECT * FROM authors
ORDER BY name
LIMIT ? OFFSET ?;

Noted. I think this can be closed. I was hoping to use the sqlc.arg method but this is fine. Thank you @kyleconroy

This is a rare case. I think fixing this issue is very low priority.
When using UNION ALL in mysql, a problem occurred when LIMIT ? OFFSET ? was specified.
Specifically, Limit is not added to the Params field.
The version I checked is "sqlc v1.15.0"
-- name: ListAuthors :many
SELECT * FROM authors
WHERE country = 'DE'
UNION ALL
SELECT * FROM authors2
WHERE country = 'US'
LIMIT ? OFFSET ?;

Closing since this is very old and the workaround seems acceptable until and unless the upstream parser is changed.
I created a new issue (https://github.com/sqlc-dev/sqlc/issues/2611) for the UNION ALL bug.

Closing since this is very old and the workaround seems acceptable until and unless the upstream parser is changed.
I created a new issue (https://github.com/sqlc-dev/sqlc/issues/2611) for the UNION ALL bug.


POST requests

Hello,
I have a POST request that is not being redirected correctly because it seems that the redirection is always using GET. Weird right?
I'm using Rails 3.2 and enforcer 0.2.8.
11:17:38 web.1      | <IP_ADDRESS> - - [27/Mar/2015 11:17:38] "POST /api/method HTTP/1.1" 301 - 0.2573
11:17:44 web.1      | <IP_ADDRESS> - - [27/Mar/2015 11:17:44] "GET /api/method HTTP/1.0" 404 781 6.2435

Thanks

Hey,
would you mind posting the configuration you're using?

Thanks for being fast.
My application.rb has:
    if Rails.env.production? || Rails.env.thredtest?
      config.middleware.use Rack::SslEnforcer, only_hosts: ["www.thredup.com", "thredup.com", "thredtest.com"]
    end

I may be wrong but I'm having errors on production right now on some rerouting and I'm suspecting this to cause errors. I've got different routes having the same issues, all are POSTs.
Let me know if I can help more.

Seems like your assumption is correct, see http://www.faqs.org/rfcs/rfc2616.html (Section 10.3.2 Note):
  Note: When automatically redirecting a POST request after
  receiving a 301 status code, some existing HTTP/1.0 user agents
  will erroneously change it into a GET request.

It might be an option to use a different status code (307) for POST redirection, but i have to think about this for a while...

Ooh it all makes sense now. It's a debatable decision though.
I don't think you can really do anything about that but I'll stay tuned.
Thanks for the support!

I would say it's an application/endpoint group specific decision to make
How the RFC declares it and how the browsers handles it could be different, especially about 302, 303
I think it's fine to use 302 for API if you have a clear doc
But for website part... :S


Release 2.0.0

draft because waiting on https://github.com/voxpupuli/beaker-hostgenerator/pull/301

should we add dependencies to the ignore list?
https://github.com/voxpupuli/beaker-hostgenerator/blob/74cb5d64a6e9772328f686db73f0311d06b91756/Rakefile#L59

yes we should ignore dependencies


No configurable or customizable

I can't understand why we are obligated to change the AppDelegate, there are no better solution for this?
I mean, most applications need a first login storyboard and only after opens a main view with probably a UITabController.
Your solution looks no much flexible for most of us.

Hey @jbarros35
It's not necessary to change it in AppDelegate. You can use it from main storyboard as well. I'm planning to add few examples on how to use FAPanels in different scenarios. I'll keep you posting here. Not sure how long its gonna take but will be out soon.


Fix macos hashing for O/s and availability of md5sum

This should fix bootstrap errors on mac.

Thanks!


find stores only last walked path

I have not found consequences of that yet, however find only writes the last walked path to out:
        for path, dirs, files in self.walk(path, maxdepth, detail=True, **kwargs):
            if withdirs:
                files.update(dirs)
            out.update({info["name"]: info for name, info in files.items()})
        if self.isfile(path) and path not in out:
            # walk works on directories, but find should also return [path]
            # when path happens to be a file
            out[path] = {}

I guess either _path or an indent was intended.

Sorry, I don't follow what you're saying here. Since this is a recursive process, all the subdirectories will end up being included in the output. The final condition is for the rare case of filesystems where the same path can be both a file and a directory.

The final condition, I'm unclear which path you'd maybe want in out. It seems wrong. Three cases:

path as passed into the function, would need for _path, dirs, files since path becomes the last walked path if any
every path returned by walk: then the last condition needs to go into the for block
last path if any returned by self.walk (now): would be OK, but I couldn't find any spec on the ordering of walked paths

Which one is right?


Inventory of Utah state, county, local, etc. government AGOL orgs

Benefit

Show a list and/or link to them from our website

"Knock on effect" 😉

Build the case for AGOL as a core DTS low-code, no-code platform

Acceptance Criteria
Find all the AGOL orgs we can:

[ ] State gov
[ ] Counties
[ ] Local govs
[ ] University/higher ed
[ ] etc.

Notes
No response
Risks
No response
Issue Reference
refs #

State

UDAF: https://udaf.maps.arcgis.com/home/index.html


Keep the records IDs when migrating

The Record class has a @GeneratedValue annotation that makes EclispeLink to ignore the manually set ID. We used to remove this annotation with Javassist, which might work with Hibernate, but obviously won’t do the trick with EclipseLink.
So now, this annotation is removed with an orm.xml file.

Resolves #58.


Added a requirement for PR approvals to the contributing file

Updated the contributing.md file with the requirement: each PR must have at least 2 team members approvals in order for it to be merged.
Closes #143

Tests resolved


Compatibility with Laravel

I have a problem because you are using "{{var}}" and the Laravel framework interpret these characters.
How can i change that ?
Thanks, sorry for my bad english.

Where do you want to use this variables?

I use them in the initial js function (.filter()) and i wanted to know if it's possible to change the {{ tags by another tag like @@ or something else. It's just that i don't understand where the {{ are interpreted by your script.
Thanks a lot and good job for your jquery filer

.filer*

#L779


make it possible to pass socket options as named args

Thanks for this package
as flutter dev I was confused with lack of possibility to pass options as named args. Consider adding this to the API 😏

You might have to give me a few days and I'm changing laptop and haven't got my access keys for pub setup yet

No problem, thanks

Actually, I published it just now.

Kudos!


Do enumerate all metadata objects when catalog is wrong

Do enumerate all metadata objects when catalog is wrong

Do enumerate all metadata objects when catalog is wrong

@kokosing
Do not enumerate all metadata objects when catalog is wrong or
Enumerate all metadata objects when catalog is wrong ?

@findepi
Sorry about this should be: Do not enumerate all metadata objects when catalog is wrong

@findepi comments addressed


Installation help with Drupal 9

Hi, I'm trying to install this on Drupal 9.1.10 and getting the following. Can you think of a way around this? Thanks.
$ composer require rpkamp/mailhog-mink-behat-context -W

Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - drupal/core-recommended is locked to version 9.1.10 and an update of this package was not requested.
    - drupal/core-recommended 9.1.10 requires symfony/http-client-contracts v2.3.1 -> found symfony/http-client-contracts[v2.3.1] but it conflicts with another require.

Is this the full output of the error?
It seems to be complaining about symfony/http-client-contracts but AFAIK this package doesn't depend on that (certainly not directly). So it feels like there is more going on and there should be more output.
If not, could you create a minimal reproducible example?

I'm still working on this when I have time. I updated my composer and I can install this using
$ composer require rpkamp/mailhog-mink-behat-context -W
However, on the same site,
$ lando composer require rpkamp/mailhog-mink-behat-context -W fails with the error above. That is the complete output.
I'll ask the Lando people.

For future reference, did you ever figure this out?

I didn't figure it out. I may come back to it when I have more time.


The latest and historical versions of portainer have incorrect access control vulnerabilities.

The key point of exploiting the vulnerability is to add the portainer.JWT parameter of any address after a successful login in the local storage of the browser. Portainer.JWT is the session credential generated after the login is successful, but the credential can be used in any other request address to obtain the admin User rights.

Please email<EMAIL_ADDRESS>as we have responsible disclosure policies.

Case


Could not read from file: /errori18n/node_modules/dist/vue-i18n.mjs

Environment

Operating System: Linux
Node Version:     v16.18.0
Nuxt Version:     3.0.0
Nitro Version:    1.0.0
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      ssr, css, build, vite, modules
Runtime Modules<EMAIL_ADDRESS>()
Build Modules:    -

Reproduction
link to a repo:
Reproduction repo
Describe the bug
When running: npm run dev error accrued.
node_modules/@nuxtjs/i18n/node_modules/vue-i18n-routing/node_modules/@intlify/vue-i18n-bridge/lib/index.mjs:10:14:
      10 │ export * from 'vue-i18n'
         ╵               ~~~~~~~~~

Additional context
No response
Logs
No response

Putting handy link to Stackblitz with the provided repo: https://stackblitz.com/github/artmarydotir/nuxt-i18n-error

Hi @artmarydotir, thanks for reporting your issue. I was investigating the repository through stackblitz with the link above, and may I ask if you had restricted installing peer dependencies? I deleted package-lock.json and reinstalled and it works on stackblitz!

I also added  "vue-i18n": "^9.2.2", but still have this error.

@ineshbose
Hi, thank you. yes i did.
i removed .nuxt/ and package-lock.json  and node_modules.  but nothing change. :(

*but if you removed package-lock.json , did you saw the translated message at index.vue page?

The Nuxt app was working without any errors or warnings, but I did not see the content in index.vue since I see that there's no app.vue with <NuxtPage /> component (to make use of pages/ and vue-router).

i removed .nuxt/ and package-lock.json and node_modules. but nothing change. :(

Could I know what steps did you perform exactly to reinstall and run the app again please?

app.vue

oh, i see. i removed app.vue by mistake. but still have error.
The steps:
i just clone the repo.
remove package-lock.json
remove node_modules/
npm install
npm run dev
and get the error.

I started to remove all packages one by one, and in every step removed package-lock.json,  and run the nuxt dev.
and i figured out when i install "vuetify": "^3.0" , i get this error.
is it possible that there is some conflict or overwritten routing by installing vuetify?

I also added  "vue-i18n": "^9.2.2", but still have this error.

nuxtjs/i18n will install vue-i18n latest version (currently v9.3.0-beta.10)
so, you wouldn't need to add manually it!

I also added  "vue-i18n": "^9.2.2", but still have this error.

nuxtjs/i18n will install vue-i18n latest version (currently v9.3.0-beta.10) so, you wouldn't need to add manually it!

Yes, i removed vue-i18n.
i find when i  installing vuetify after i18n will change some packages version. i checked package-lock git diff.
could it be a problem with this package(vuetify)?
1/

2/

3/

Im closing this issue, i update node version from 16 to 18. its works now.

I have the same issue with the error:
[ERROR] Could not read from file: ../nuxt-loco/playground/node_modules/dist/vue-i18n.mjs
I need to use node 16 because of my hosting.

Operating System: Darwin
Node Version:     v16.16.0
Nuxt Version:     3.0.0
Nitro Version:    1.0.0
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      -
Runtime Modules:  -
Build Modules:    -

Here the reproduction link in the playground : https://github.com/gaetansenn/nuxt-loco


External database gets recreated upon pod restart

Trying to use external db via NOMINATIM_DATABASE_DSN env variables. However it appears that db is dropped and recreated upon every restart of nominatim container. Obviously it defeats the purpose. Please advise.

Looks like /var/lib/postgresql/14/main/import-finished must be persisted between restarts.
Does it mean that even when using external postrgre db we still need to fuss around with persistent volumes for /var/lib/postgresql/14/main ?

Unfortunately, there is no advice to give. What you're doing just happens to work but it's actually not something that the core contributors use and therefore it's unsupported.

Thank you for clarifying. I was coming with k8s bias where workers are easily disposable and managing stateful disks attached to particular nodes disk seems anti-pattern. Your answer made me realize the intended purpose is most likely a full VM with attached disks.
Thank you.

Indeed. We are aware that our setup is not adhering to principles of more complex deployments but we value the ease of setup and operation over pretty much anything else.
Maybe https://github.com/smithmicro/n7m meets your requirements.

Good insights. Thanks again!


Update UBI to 8.4

Signed-off-by: Chmouel Boudjnah<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@a409a42). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #144   +/-   ##
=======================================
  Coverage        ?   76.73%
=======================================
  Files           ?       13
  Lines           ?      834
  Branches        ?        0
=======================================
  Hits            ?      640
  Misses          ?      122
  Partials        ?       72

Continue to review full report at Codecov.

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


add(game):新增XR的api文档内容

Merge pull request #148 from swan-team/4game

Merge pull request #148 from swan-team/4game

Merge pull request #148 from swan-team/4game

Merge pull request #148 from swan-team/4game

Merge pull request #148 from swan-team/4game


Hypershift: Integrate new create cluster catalog to current wizard

Issue:
https://github.com/stolostron/backlog/issues/24372

Hook up new catalog to wizard
Moved AI warning to cluster details in wizard

Not sure why the code smell is based on the older versions of the changed files. Looking at the comparisons, the code that its complaining about is not there anymore.


Setup analytics dashboard for the BC Wallet Demo

Basic analysts dashboard

duplicates #88

I will clarify the details of the analytics requirements with Dan Pollock.

we received an estimation fro GDX, requested budget approval


Provide beta artifacts for upcoming release

fixes #103

Codecov Report

Merging #104 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #104   +/-   ##
=========================================
  Coverage     71.97%   71.97%
  Complexity      238      238
=========================================
  Files            48       48
  Lines           842      842
  Branches         40       40
=========================================
  Hits            606      606
  Misses          213      213
  Partials         23       23

Continue to review full report at Codecov.

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

wrong branch


How to repair failed setup?

I had an issue during the first setup where it got "stuck" during the tarball extraction.  I don't have AVAST installed, but do have BitDefender installed.  Could that be an issue?
Either way, I closed the window after trying to hit enter to see if I'd get a username creation prompt like one thread I'd seen in here suggested.  I saw nothing new, nothing printed...so I closed the terminal.
I have an Ubuntu on Windows shortcut, but when I run it I get a root@<computer_name> prompt.  I assume more setup should have happened because my network doesn't seem to work.  I tried to "apt-get update" and it seemed to be stuck trying to access the network (stays at 0%).  I even tried that in an administrator run terminal.
I also don't know how to uninstall/reinstall or I'd have tried.  Any suggestions or URLs are appreciated.

Hi @OneOfMany07 -- well, that sounds frustrating.  Hopefully easy to fix, though:
Regarding BitDefender -- I would recommend that you read the "No Internet with 3rd party AV/Firewall" ticket ( #475 ).  It discusses problems and workarounds for many firewalls, including BitDefender.  Also, make sure you're using the latest version, as many security programs had to be updated when WSL was released in order to recognize and correctly handle Linux binaries and network requests.
My guess is that BitDefender is causing your network issues, and that your network issues are causing problems with the script that installs WSL.
For uninstalling/reinstalling, there is a command line tool for doing that:
https://msdn.microsoft.com/en-us/commandline/wsl/reference

@oneofmany07 - Can you please clarify what build you're running?  If you'd like to create a non-root user you can use the below command:
lxrun.exe /setdefaultuser
I'd also recommend deleting your /etc/hosts file so WSL regenerates one.


Scroll Error on template page

Describe the bug
small UI bug. when scrolling on content page, when you get to the bottom. both section 3 and 4 will be highlighted
To Reproduce
Steps to reproduce the behavior:

Go to template page.
Scroll to bottom
See error

Expected behavior
section 4 on navbar should be highlighted
Screenshots
.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Thanks good catch I’ll fix it in the nex mt version It


Cannot interpret feed_dict as Tensor

Hi! I'm trying to run a model that predicts node attributes based on global and edge inputs.
I've been largely following the shortest_path.ipynb demo to write my code, and my code at the moment looks as follows (happy to include more if need be!):
# train_input, train_target, test_input etc. are all lists containing nxgraphs

train_input_ph, train_target_ph = create_placeholders(train_input, train_target)
test_input_ph, test_target_ph = create_placeholders(test_input, test_target)

output_train_graphs = graph_net_module(train_input_ph)
output_test_graphs = graph_net_module(test_input_ph)

loss_train = create_loss_ops(train_target_ph, output_train_graphs)
loss_test = create_loss_ops(test_target_ph, output_test_graphs)
....

train_input_ph, train_target_ph, output_train_graphs, output_test_graphs = make_all_runnable_in_session(train_input_ph, train_target_ph, output_train_graphs, output_test_graphs)

In the running training section of code, I then have:
for iteration in range(last_iteration, num_training_iterations):
    last_iteration = iteration
    train_values = sess.run({
        "step": step_op,
        "target": train_target_ph,
        "train_loss": loss_train,
        "outputs": output_train_graphs
    },
    feed_dict={train_input_ph: gn.utils_np.networkxs_to_graphs_tuple(train_input),
               train_target_ph: gn.utils_np.networkxs_to_graphs_tuple(train_target)}
    )

However, when I try to run the second set of code, I get the following error:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
/miniconda3/lib/python3.7/site-packages/tensorflow/python/client/session.py in _run(self, handle, fetches, feed_dict, options, run_metadata)
   1091             subfeed_t = self.graph.as_graph_element(
-> 1092                 subfeed, allow_tensor=True, allow_operation=False)
   1093           except Exception as e:

/miniconda3/lib/python3.7/site-packages/tensorflow/python/framework/ops.py in as_graph_element(self, obj, allow_tensor, allow_operation)
   3477     with self._lock:
-> 3478       return self._as_graph_element_locked(obj, allow_tensor, allow_operation)
   3479

/miniconda3/lib/python3.7/site-packages/tensorflow/python/framework/ops.py in _as_graph_element_locked(self, obj, allow_tensor, allow_operation)
   3566       raise TypeError("Can not convert a %s into a %s." % (type(obj).__name__,
-> 3567                                                            types_str))
   3568

TypeError: Can not convert a Operation into a Tensor.

During handling of the above exception, another exception occurred:

TypeError                                 Traceback (most recent call last)
<ipython-input-1103-fddee2f34548> in <module>
     16     },
     17     feed_dict={train_input_ph: gn.utils_np.networkxs_to_graphs_tuple(train_input),
---> 18                train_target_ph: gn.utils_np.networkxs_to_graphs_tuple(train_target)}
     19     )
     20     the_time = time.time()

/miniconda3/lib/python3.7/site-packages/tensorflow/python/client/session.py in run(self, fetches, feed_dict, options, run_metadata)
    927     try:
    928       result = self._run(None, fetches, feed_dict, options_ptr,
--> 929                          run_metadata_ptr)
    930       if run_metadata:
    931         proto_data = tf_session.TF_GetBuffer(run_metadata_ptr)

/miniconda3/lib/python3.7/site-packages/tensorflow/python/client/session.py in _run(self, handle, fetches, feed_dict, options, run_metadata)
   1093           except Exception as e:
   1094             raise TypeError(
-> 1095                 'Cannot interpret feed_dict key as Tensor: ' + e.args[0])
   1096
   1097           if isinstance(subfeed_val, ops.Tensor):

TypeError: Cannot interpret feed_dict key as Tensor: Can not convert a Operation into a Tensor.

I notice this is a similar thing to #24 but when I tried the solution there of reducing make_all_runnable_in_session to only act on output_train_graphs and output_test_graphs, I get the following error instead:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
/miniconda3/lib/python3.7/site-packages/tensorflow/python/client/session.py in _run(self, handle, fetches, feed_dict, options, run_metadata)
   1091             subfeed_t = self.graph.as_graph_element(
-> 1092                 subfeed, allow_tensor=True, allow_operation=False)
   1093           except Exception as e:

/miniconda3/lib/python3.7/site-packages/tensorflow/python/framework/ops.py in as_graph_element(self, obj, allow_tensor, allow_operation)
   3477     with self._lock:
-> 3478       return self._as_graph_element_locked(obj, allow_tensor, allow_operation)
   3479

/miniconda3/lib/python3.7/site-packages/tensorflow/python/framework/ops.py in _as_graph_element_locked(self, obj, allow_tensor, allow_operation)
   3566       raise TypeError("Can not convert a %s into a %s." % (type(obj).__name__,
-> 3567                                                            types_str))
   3568

TypeError: Can not convert a NoneType into a Tensor.

During handling of the above exception, another exception occurred:

TypeError                                 Traceback (most recent call last)
<ipython-input-1106-fddee2f34548> in <module>
     16     },
     17     feed_dict={train_input_ph: gn.utils_np.networkxs_to_graphs_tuple(train_input),
---> 18                train_target_ph: gn.utils_np.networkxs_to_graphs_tuple(train_target)}
     19     )
     20     the_time = time.time()

/miniconda3/lib/python3.7/site-packages/tensorflow/python/client/session.py in run(self, fetches, feed_dict, options, run_metadata)
    927     try:
    928       result = self._run(None, fetches, feed_dict, options_ptr,
--> 929                          run_metadata_ptr)
    930       if run_metadata:
    931         proto_data = tf_session.TF_GetBuffer(run_metadata_ptr)

/miniconda3/lib/python3.7/site-packages/tensorflow/python/client/session.py in _run(self, handle, fetches, feed_dict, options, run_metadata)
   1093           except Exception as e:
   1094             raise TypeError(
-> 1095                 'Cannot interpret feed_dict key as Tensor: ' + e.args[0])
   1096
   1097           if isinstance(subfeed_val, ops.Tensor):

TypeError: Cannot interpret feed_dict key as Tensor: Can not convert a NoneType into a Tensor.

What can I do with the feed_dict and session variables to make this all work?

The problem is that your graphs have Nones fields in them, so you cannot directly create the feed_dict like that.
Instead you should use the helper function gn.utils_tf.get_feed_dict:
feed_dict = gn.utils_tf.get_feed_dict(
    train_input_ph,
    gn.utils_np.networkxs_to_graphs_tuple(train_input))
feed_dict.update(utils_tf.get_feed_dict(
    train_target_ph,
    gn.utils_np.networkxs_to_graphs_tuple(train_target)))

Hope this works!

Hi,
I think you have Nones in your input. Can you try:
from graph_nets import utils_tf

feed_dict=utils_tf.get_feed_dict(train_input_ph: gn.utils_np.networkxs_to_graphs_tuple(train_input))
feed_dict.update(utils_tf.get_feed_dict(train_target_ph:gn.utils_np.networkxs_to_graphs_tuple(train_target))

Hi, thanks very much for getting back to me! So I'm now running:
for iteration in range(last_iteration, num_training_iterations):
    last_iteration = iteration
    feed_dict = utils_tf.get_feed_dict(
        train_input_ph,
        utils_np.networkxs_to_graphs_tuple(train_input))
    feed_dict.update(utils_tf.get_feed_dict(
        train_target_ph,
        utils_np.networkxs_to_graphs_tuple(train_target)
    ))
    train_values = sess.run({
        "step": step_op,
        "target": train_target_ph,
        "train_loss": loss_train,
        "outputs": output_train_graphs
    },
    feed_dict=feed_dict
    )

And the error is now:
---------------------------------------------------------------------------
InternalError                             Traceback (most recent call last)
/miniconda3/lib/python3.7/site-packages/tensorflow/python/client/session.py in _do_call(self, fn, *args)
   1333     try:
-> 1334       return fn(*args)
   1335     except errors.OpError as e:

/miniconda3/lib/python3.7/site-packages/tensorflow/python/client/session.py in _run_fn(feed_dict, fetch_list, target_list, options, run_metadata)
   1318       return self._call_tf_sessionrun(
-> 1319           options, feed_dict, fetch_list, target_list, run_metadata)
   1320

/miniconda3/lib/python3.7/site-packages/tensorflow/python/client/session.py in _call_tf_sessionrun(self, options, feed_dict, fetch_list, target_list, run_metadata)
   1406         self._session, options, feed_dict, fetch_list, target_list,
-> 1407         run_metadata)
   1408

InternalError: Unable to get element as bytes.

During handling of the above exception, another exception occurred:

InternalError                             Traceback (most recent call last)
<ipython-input-1140-2017fe71ea6b> in <module>
     24         "outputs": output_train_graphs
     25     },
---> 26     feed_dict=feed_dict
     27     )
     28     the_time = time.time()

/miniconda3/lib/python3.7/site-packages/tensorflow/python/client/session.py in run(self, fetches, feed_dict, options, run_metadata)
    927     try:
    928       result = self._run(None, fetches, feed_dict, options_ptr,
--> 929                          run_metadata_ptr)
    930       if run_metadata:
    931         proto_data = tf_session.TF_GetBuffer(run_metadata_ptr)

/miniconda3/lib/python3.7/site-packages/tensorflow/python/client/session.py in _run(self, handle, fetches, feed_dict, options, run_metadata)
   1150     if final_fetches or final_targets or (handle and feed_dict_tensor):
   1151       results = self._do_run(handle, final_targets, final_fetches,
-> 1152                              feed_dict_tensor, options, run_metadata)
   1153     else:
   1154       results = []

/miniconda3/lib/python3.7/site-packages/tensorflow/python/client/session.py in _do_run(self, handle, target_list, fetch_list, feed_dict, options, run_metadata)
   1326     if handle is None:
   1327       return self._do_call(_run_fn, feeds, fetches, targets, options,
-> 1328                            run_metadata)
   1329     else:
   1330       return self._do_call(_prun_fn, handle, feeds, fetches)

/miniconda3/lib/python3.7/site-packages/tensorflow/python/client/session.py in _do_call(self, fn, *args)
   1346           pass
   1347       message = error_interpolation.interpolate(message, self._graph)
-> 1348       raise type(e)(node_def, op, message)
   1349
   1350   def _extend_graph(self):

InternalError: Unable to get element as bytes.

Googling this seems to suggest there can be all kinds of reasons for this error, so I thought I'd bring it back here just in case.

Can you try to see which output causes this?
For instance I think you wont be able to run train_target_ph but should use utils_tf.make_runnable_in_session(train_target_ph)

I think you may need to use the graphs placeholders to call get_feed_dict before you pass them through by make_runnable_in_session.
But you seem to be losing the reference to those when you do:
train_input_ph, train_target_ph, output_train_graphs, output_test_graphs = make_all_runnable_in_session(train_input_ph, train_target_ph, output_train_graphs, output_test_graphs)
Just try:
`runnable_train_input_ph, runnable_train_target_ph, runnable_output_train_graphs, runnable_output_test_graphs = make_all_runnable_in_session(train_input_ph, train_target_ph, output_train_graphs, output_test_graphs)
And use the runnable_* ones only for feeding them directly in the first argument of sess.run, but not for anything else (not even to create the feed_dict).

Thanks so much! The model is now training, really appreciate the help


Node dragging animations

Hi,
I started using your package in my product and I felt something is missing with the animations.
If you consider the d3-force-directed graph it's providing nice animation for nodes drag, if I drag one node then automatically all the nodes are moving with respect to the node.
Demo link for d3-force-directed
I tried with automaticRearrangeAfterDropNode: true also by disabling nodeHighlightBehavior but still, I can't get that effect.
Did I missed anything, can you please help me out on this.
Thanks,
Mahesh Reddy

Hi @onlyreddy ,
I think that you could use update callbacks like onNodePositionChange to change the position of connected nodes according to one node position change. I don't really think that this is something that react-d3-graph should provide out of the box since it seems really specific.
What do you think?

It would be a nice experience enhancement tough to provide this force animations out of the box when clients set nodeHighlightBehavior to true.


Fix link for a bag with intensities.

Merge requested by authorized user SirVer. Merge queue now has a length of 1.


[WFLY-10986] Clone the connection definitions in a synchronized block

https://issues.jboss.org/browse/WFLY-10986

Thank you for this fix , but some clients are running under Wildfly 14 .
Is it possible to fix it in 14 version ?
Thanks,


Remove square in face detection or add a custom widget

I need to change the shape of square detection, could you add that?

Hello there! @WilliamBz12  I wanted to let you know that I've made some updates to the plugin. Developers have more control over the square detection shape with the new features I've added. You can use the indicatorShapes and indicatorAssetImage parameters to choose the face indicator that suits your needs. Additionally, I've introduced an indicatorBuilder function which allows developers to create a custom face indicator by providing the detected face information and image size. I hope these enhancements make the plugin even more versatile for your use case. Feel free to check out the latest version and give it a try! If you have any feedback or questions, please don't hesitate to reach out.


Remove docopt

This PR completely removes docopt from _lib since we're no longer using it at all.

Coverage increased (+2.4%) to 71.76% when pulling d0d6451bf8cd1fcf33a8dfca8e81a85800e9c1fb on remove-docopt into ad5be44deeb9f1df5b4c5488d4046c9e9b5993c3 on master.

Since this passes all tests, I think that's a pretty good indication that there's not any dependencies on docopt that I missed. Should be ready to merge in once #121 is merged.

Would you mind just triggering a travis rebuild to be safe? I usually just do `git commit --allow-empty -m ..", but apparently the travis website also has a refresh button for those with admin access.


Handle youtube errors better

[x] Completed PR - Has all features or fixes that it was meant to and isn't half done
[x] Follows the style guide - Style guide found here
[x] Tested changes

What is added
3 tries to connect to YT
New exception to distinguish between a failure and a non-existent video/playlist
What is fixed
Update apache commons lang so everything works
Update JDA

Also why is this in data-restructure?

I explained the apache commons to walshy. the built in one in JDa doesn't work for us so I had to declare it in our POM.
This is in data-restructure to avoid merging conflicts from dev

But how does it not work??????????????????????????????????

I don't know, I tested it and it just doesn't. It makes no difference really. It doesn't make our effective POM bigger or anything

Elaborate

Basically, When I tried to run the test bot, it wouldn't run due to it's own commons-lang being overridden by our outdated one. SO I removed the outdated on and that side worked but then I couldn't use commons-lang in out project. So I added lang to our pom and both JDA and us used it and it worked.


Documenting subtasks?

Does the plugin support this?
The screenshot in your documentation shows:
clean -> Clean files and folders. (dist | tmp | templates)

Is the text in brackets documentation of the available sub tasks ? ie clean:dist, clean:tmp and clean:templates
If so, how do I configure the plugin to include information about these sub tasks?
Thanks.

Yes, that is for showing the multitask targets. There isn't any additional information about these that we can get from Grunt. But we could introduce a multiTasks option to add descriptions for them.
Would be open to a PR for this. I'm pretty busy on other things right now so this isn't a high priority for me.


Add GraphQL support for monitoring dashboard widgets.

Description:
Add GraphQL support for monitoring dashboard widgets.

This needs to be updated in "TRAFFIC" page and fixed in other pages.

This is fixed now. Hence closing.


Setting standing gains from yaml, setting target height from lcm message

Updating the OSC standing controller from the experimental branch. New features include:

Setting gains from a yaml file
Receiving the target center of mass height from an LCM message. Currently sent from an updated pd_panel script in the drake director

This change is

The walking controller also has gains set by a yaml file but I want to get the walking controller working on hardware before merging the many updates into master.


Loading pretrained model

I wanted to load the pretrained model which is in the form of a .model file. However as far as I know saved models in tensorflow are in the form of .meta files and can be loaded by tf.train.save_meta_files(). Can someone please let me know how can I load the graph from this model?

@gandhirushabh2311 have you ever solved the problem?

@geroge-gao download the .model files as per the instructions in readme file.

@geroge-gao no I didn't have that issue.

@geroge-gao I did follow the instructions as per readme file.

@JulioEndara sorry to brother you,but this problem has troubled me for a few days, could I ask you a few questions on the email? my email is<EMAIL_ADDRESS>

@geroge-gao github is better than email because other people see the issue and they can help out rather than just one person, btw I'm not an expert on this so you are better off raising your issue to the community.
Also this is also a closed issue.
I suggest you open a new issue and explain in detail what you are doing, what script you are executing and include full dump of your error... also confirm if you have modified the code or use it straight from the repo.


Wireguard: use singe port for multi-tunnel

Description
Wireguard uses UDP for communication.
Currently during creation it checks to see if a udp-port is already taken and returns an error if it is. Then it stores unique port for every interface.
Problem
NSM needs to be able to support multiple independent Wireguard tunnels between the same pair of hosts.
It is often attaching to the single K8s interface with AF_PACKET. It needs to block any packets not intended for it for its various tunnels.
Therefore, Wireguard for NSM needs to use a single Port for all of its Wireguard interfaces.
Solution
Store multiple interfaces per port:

Change index_t *wg_if_index_by_port -> index_t **wg_if_indexes_by_port.
Check for port is in use if and only if vec_len(wg_if_indexes_by_port[port]) == 0. Otherwise, simply append the wg_interface to the list of interfaces for the port.
Change wg_process_handshake to iterate over the interfaces for the port when processing a handshake.

Estimation: 3d

Merged https://gerrit.fd.io/r/c/vpp/+/32443


Expose Linux kernel configuration in pure Nix

Currently, we seem to have a giant string we interpolate values into inside common-config.nix. We furthermore have ad-hoc optionalString in there to expose certain options only for certain kernel versions.
It seems like it wouldn't be hard to generate that giant string automatically with Nix, along with a simple way to define new options scoped to particular kernel versions. It could even use the module system outside of the standard NixOS configuration mechanism!

@aszlig has done something like this here
Example config, imported here

Will close in favor of #12158.


Add cf-rabbitmq-smoke-tests

This bosh release is a pre-req of https://github.com/pivotal-cf/cf-rabbitmq-multitenant-broker-release, and it does not make sense to release the dependent without the dependency, as pointed out in https://github.com/pivotal-cf/cf-rabbitmq-smoke-tests-release/issues/2
Resolves https://github.com/pivotal-cf/cf-rabbitmq-smoke-tests-release/issues/2

Conceptually, it seems a bit odd for bosh.io to need to support test assets like this, but it looks like we already have a couple other smoke test releases. We might follow up with your team this at some point once/if we have alternative recommendations.
I'll merge for now since it does seem like a pre-req, helpful, and there's already precedent.


time bar in pshych ( yes i have alredy the source)

Describe your problem here.
in the psych engine there is a time bar (dhu) I would like to know how to modify it in the middle of a song, I would like to know how to extend the time bar in the middle of the song and with sudden movements (something like bob & bosip ex but with movements ).
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
no, i don't edit this biuld

I think it's better you put this in discussions instead of issues. Most people only look at discussion posts.
Also, it's impossible to extend the time because it would have to refresh the whole play state all over again and possibly lead to a crash. Idk I haven't tried or figured out the code to this.

I think you misunderstood me, by extending the time bar I meant that the progress bar goes backwards like in bob and bosip AND EXTEND THE TIME, not the time bar because that is impossible

a friend told me that you can make a fake time bar for that and he didn't tell me how to do it and so far he doesn't want to do it xd

Why you close my question?

Why is this closed + What properties are in time bar?


Network: Add option for wildcard identification of interface

This would be helpful when having multiple, similar interfaces or when using the same configuration on different machines.
Example:
"network#wlan": {
    "interface": "wlp*", // Wildcard for all interfaces starting with wlp
    "format-wifi": "{essid} ({signalStrength}%) ",
    "format-disconnected": "⚠",
    "on-click": "nm-connection-editor"
},

Nice thanks!


Clearer readme for configuring a rule

It took me a few minutes to figure out how to configure a rule, given that there are no complete examples of a sass-lint.yml (something that perhaps should be addressed?).
In particular, it wasn't clear to me that the rule configuration needs to be nested under rules:, or what the example config shown was doing. I've clarified both of these in this updated readme.
Hope it helps, thanks for the great project.
DCO 1.1 Signed-off-by: James Riley<EMAIL_ADDRESS>

Coverage remained the same at 97.527% when pulling 987e096f9781aabb53b5927315eb757100afef17 on mrjamesriley:develop into 47ae2e79e827470f2bcb222bc67a3282afcc2544 on sasstools:develop.

Thanks, there are actually links in the block above about all of this see here including sample configs but I agree this should follow the pattern of the other examples. Can you do me a quick favour though and make sure you include a DCO with your PR.
DCO - https://github.com/sasstools/sass-lint/blob/develop/CONTRIBUTING.md#developer-certificate-of-origin as per our contribution guidelines https://github.com/sasstools/sass-lint/blob/develop/CONTRIBUTING.md#pull-requests
Thanks! 👍

Prompt response there! I've added the DCO, hope that suffices.
(was already added, but with angle brackets either side that had it not showing out of edit view 😷)

(completely overlooked that sample config, thanks for pointing it out)

Coverage remained the same at 97.529% when pulling d64b2b716c2bfc53a1ff08f7247a50a6bc748e13 on mrjamesriley:develop into 6fd03417c818935e49e63e41da346310f0320a41 on sasstools:develop.

Coverage remained the same at 97.529% when pulling d64b2b716c2bfc53a1ff08f7247a50a6bc748e13 on mrjamesriley:develop into 6fd03417c818935e49e63e41da346310f0320a41 on sasstools:develop.

corrected 😁

Coverage remained the same at 97.529% when pulling c448971ab59e7a73ab7146f581bed67d0ab44690 on mrjamesriley:develop into 6fd03417c818935e49e63e41da346310f0320a41 on sasstools:develop.

Thanks 👍


Online demo brings me to a scam

help i cannot play the online demo it brings me to a scam site

This is not a hard fix, though it's still not done after 8 months


test: more Unified runner operations

Adds a number of operations, makes use of chai expect instead of boolean
for assertions to track stack traces, adds session and bucket entity
types. Passes utilClient through to operation functions which are"
inlined into the operations map to get type checking to work.
Modifies url helper function to make multipleMongoses detection work.

Working on the few failures left, but shouldn't be drastic so I think this is ready for a look, thanks!


Write some unit tests

Write some unit tests Author in Domain layer

Please resolve merge conflicts. @Ershad95


bug: redundant footer

Description
If we go to:

https://docs.keploy.io/docs/keploy-explained/introduction
https://docs.keploy.io/docs/devtools/sdk-contrib-guide
https://docs.keploy.io/docs/gsoc/contribution-guide
https://docs.keploy.io/docs/go/installation

...and almost every section of the website, we can see a redundant footer like in the screenshot floating at the bottom unnecessarily.

I have fixed the unnecessary import of this footer in the issue #568
Fixes #568
Fixes #549
Type of change
UI - Website

[x] Bug fix (non-breaking change which fixes an issue).

How Has This Been Tested?
Test locally as there is only one component that is used unnecessarily in one of the files.
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.

I have read the CLA Document and I hereby sign the CLA

@nehagup I have fixed the linting issue.


Added install targets to cmake.

The install targets were added for each of the libraries and executable. I created separate per-library config export files. Let me know if you think this needs adjustment, or if you are not confortable with the names I used (there is an ambiguity with the dash and underscore separators).
See the tree below, which is the result of a Release install on Windows:
./bin/spirv-cross.exe
./include/spirv_cross/GLSL.std.450.h
./include/spirv_cross/spirv.hpp
./include/spirv_cross/spirv_cfg.hpp
./include/spirv_cross/spirv_common.hpp
./include/spirv_cross/spirv_cpp.hpp
./include/spirv_cross/spirv_cross.hpp
./include/spirv_cross/spirv_glsl.hpp
./include/spirv_cross/spirv_msl.hpp
./lib/spirv-cross-core.lib
./lib/spirv-cross-cpp.lib
./lib/spirv-cross-glsl.lib
./lib/spirv-cross-msl.lib
./share/spirv_cross_core/cmake/spirv_crossConfig-release.cmake
./share/spirv_cross_core/cmake/spirv_crossConfig.cmake
./share/spirv_cross_cpp/cmake/spirv_cross_cppConfig-release.cmake
./share/spirv_cross_cpp/cmake/spirv_cross_cppConfig.cmake
./share/spirv_cross_glsl/cmake/spirv_cross_glslConfig-release.cmake
./share/spirv_cross_glsl/cmake/spirv_cross_glslConfig.cmake
./share/spirv_cross_msl/cmake/spirv_cross_mslConfig-release.cmake
./share/spirv_cross_msl/cmake/spirv_cross_mslConfig.cmake

 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.Joao Paulo Magalhaes 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.

Ping, the CLA needs to be signed before this can be merged.

Sorry for the delay. Kept putting this off as I had to rebase my commits because the email was wrong. CLA now is working; if the tests pass all should be fine.

Can you rebase this branch on current master? This base branch is really old.

@HansKristian-ARM As it is the PR is merged with current master. Could you clarify?

It is merged, but the merge commit is against a very old master, which I would like to avoid if possible. I would prefer if you could run git rebase <remote>/master and push that up to your branch to keep a more linear history when it is merged.

I rebased manually on a different PR and merged it. Sorry for the long delay.


ree-1.8.7-* fails to built on Yosemite

I haven't seen an issue opened for this yet but ree-1.8.7 is failing to build on Yosemite. Specifying CC=/usr/local/bin/gcc-4.2 yields the same result. Failure message below and log output: https://gist.github.com/paulbeattie/3b6931c997ee40393c06
Configured with: --prefix=/Library/Developer/CommandLineTools/usr --with-gxx-include-dir=/usr/include/c++/4.2.1
couldn't understand kern.osversion `14.0.0'
Downloading ruby-enterprise-1.8.7-2012.02.tar.gz...
-> http://dqw8nmjcqpjn7.cloudfront.net/ecf4a6d4c96b547b3bf4b6be14e082ddaa781e83ad7f69437cd3169fb7576e42
Installing ruby-enterprise-1.8.7-2012.02...

BUILD FAILED (OS X 10.10 using ruby-build 20141016)

Inspect or clean up the working tree at /var/folders/75/_yb744wd5jd1y2gf4cb4dmg80000gn/T/ruby-build.20141018115238.52997
Results logged to /var/folders/75/_yb744wd5jd1y2gf4cb4dmg80000gn/T/ruby-build.20141018115238.52997.log

Last 10 log lines:
couldn't understand kern.osversion `14.0.0'
/usr/local/bin/gcc-4.2 -g -O2  -pipe -fno-common    -DRUBY_EXPORT  -I. -I. -I/Users/paulbeattie/.rbenv/versions/ree-1.8.7-2012.02/include  -D_XOPEN_SOURCE -D_DARWIN_C_SOURCE   -c dmydln.c
couldn't understand kern.osversion `14.0.0'
/usr/local/bin/gcc-4.2 -g -O2  -pipe -fno-common    -DRUBY_EXPORT  -L. -L/Users/paulbeattie/.rbenv/versions/ree-1.8.7-2012.02/lib    main.o dmydln.o libruby-static.a -L/opt/local/lib -Wl,-rpath,/Users/paulbeattie/.rbenv/versions/ree-1.8.7-2012.02/lib -L/Users/paulbeattie/.rbenv/versions/ree-1.8.7-2012.02/lib -lsystem_allocator -ldl -lobjc   -o miniruby
couldn't understand kern.osversion `14.0.0'
ld: warning: directory not found for option '-L/opt/local/lib'
ld: -rpath can only be used when targeting Mac OS X 10.5 or later
collect2: ld returned 1 exit status
make: *** [miniruby] Error 1

this worked for me:
MACOSX_DEPLOYMENT_TARGET=10.9 CC=/usr/bin/gcc rvm install ruby-1.8.7

Setting the deployment target to 10.9 worked for me too.
MACOSX_DEPLOYMENT_TARGET=10.9 CC=/usr/bin/gcc rbenv install <ruby_version>

@juanpaulo Which version of ruby-build do you have? This shouldn't be needed since v20141028 10fd04909d3bf77bb26bf6610dbdd10940e80e09
@bliaxiong You seem to be using RVM not rbenv. You're commenting on the wrong project.

@mislav I was using ruby-build 20140702-158-g95373b4.


Consider implementing h.Scalasig.toLowlevel

I'm more or less happy about the scalasig API so far, except for one thing. For the sake of symmetry, we've added h.Scalasig.toLowlevel, but have never found a use case for it so far, so it's been kept unimplemented. We may consider fixing this.

I think this is out of scope of the project at this point. Maybe if we move scalasig to Scalameta (https://github.com/scalameta/scalameta/issues/1683), we'll have more motivation to implement this, but for now this will be a wontfix.


🐛 BUG: default <IP_ADDRESS> wrangler dev address not routable in Windows

What version of Wrangler are you using?
2.0.25
What operating system are you using?
Windows 10
Describe the Bug
When using wrangler dev or wrangler pages dev, you'll get output like this:
[pages:inf] Listening on <IP_ADDRESS>:8788
[pages:inf] - http://<IP_ADDRESS>:8788

Previously, you could click on this and open up your browser without issue, as well as of course b opens this address too.
Unfortunately, at least within Windows, <IP_ADDRESS> isn't actually a routable address, and the browser throws a ERR_ADDRESS_INVALID error, and something like this:
It seems like this address may be routable in some OSes, but not Windows. This seems to be due to the changes in #1605

Can confirm it's not routable in Windows, it's better to give <IP_ADDRESS> or localhost as a default.

Another solution here might be to print all of the available network adapters, like serve does:
   │   Serving!                                         │
   │                                                    │
   │   - Local:            http://localhost:3000        │
   │   - On Your Network:  http://<IP_ADDRESS>:3000   |

Windows probably isn't the only place that <IP_ADDRESS> won't be routable. This information could be determined from os.networkInterfaces().

@Skye-31 Since you are picking this up here is a reference to the recent PR that caused this issue in Windows, we need to be able to support the VM/Container environments as well which prefer the <IP_ADDRESS>
reference: https://github.com/cloudflare/wrangler2/pull/1605

My plan is to list all available addresses, so this should cover that 🙂
Will probably get around to this tomorrow

@JacobMGEvans I filed the original issue you mentioned, and just wanted to give my two cents and can confirm that keeping the <IP_ADDRESS> as default while advertising all the available addresses locally in console will suit both local and remote users, including VM users. 👍

Created 2 PRs. One solves the b button opening <IP_ADDRESS>, in wrangler. The other solves the console log issue. Once those are both merged, will close this

Re-opening this as the issue of what is displayed to the user is only fixed in local mode (i.e. in Miniflare) and not in remote mode (i.e. when using the proxy.ts code).

@petebacondarwin, have also made a PR at #1881 to resolve this 🙂


Type error during callback

/app/app.js:259
ip_address: req.connection.remoteAddress.toString(),
^
TypeError: Cannot read properties of undefined (reading 'toString')
at /app/app.js:259:45

Need to figure out how to get the IP from the cloud run environment in express


Docs page route added

Please describe the changes this PR makes and why it should be merged:
A main docs file was added in order to fix the navbar navigation issue
Status

[x] Code changes have been tested against prettier, or there are no code changes
[x] I know how to update typings and have done so, or typings don't need updating

Semantic versioning classification:

[x] This PR changes the codebase

[ ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)
[ ] This PR changes the internal workings with no modifications to the external API (bug fixes, performance improvements)

[x] This PR only includes non-code changes, like changes to documentation, README, etc.

 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.

great work @oliverloops :)


tiago_navigation: 4.0.1-1 in 'humble/distribution.yaml' [bloom]

Increasing version of package(s) in repository tiago_navigation to 4.0.1-1:

upstream repository: https://github.com/pal-robotics/tiago_navigation.git
release repository: https://github.com/pal-gbp/tiago_navigation-release.git
distro file: humble/distribution.yaml
bloom version: 0.11.2
previous version for package: null

tiago_2dnav
* Merge branch 'mlu/fix/tiago_2dnav_buildtool' into 'humble-devel'
  Fix  buildtool_depend of tiago_2dnav
  See merge request robots/tiago_navigation!68
* Fix  buildtool_depend of tiago_2dnav
  Must be ament_cmake_auto
* Contributors: Noel Jimenez, mathiasluedtke

tiago_navigation

No changes

New package review:

[x] At least one of the following must be present

[x] Top level license file: https://github.com/pal-robotics/tiago_navigation/blob/humble-devel/LICENSE
[ ] Per package license files

[x] License is OSI-approved: Apache-2
[x] License correctly listed in package.xmls

[x] tiago_2dnav: https://github.com/pal-robotics/tiago_navigation/blob/humble-devel/tiago_2dnav/package.xml#L8
[x] tiago_navigation: https://github.com/pal-robotics/tiago_navigation/blob/humble-devel/tiago_navigation/package.xml#L8

[x] Public source repo: https://github.com/pal-robotics/tiago_navigation/tree/humble-devel/
[x] ROS packages

tiago_2dnav
tiago_navigation


Update staging role with expiration and snapshot/timestamp

Signed-off-by: Asra Ali<EMAIL_ADDRESS>
Summary

Ticket Link

Fixes
Release Note

The staging role had an early expiration causing snapshot/timestamp to fail.
Not entirely sure how that happened since the other delegation, rekor.json has the correct expiration (uniform with root/targets). So updated, which requires re-snapshotting and timestamping.

Do we have a process which regenerates staging?

Not ad-hoc, besides the ./scripts/step-4.sh that happens on root-signing events. Since the expiraiton didn't line up with those it expired early


about indentation

The default indentation is 2, how to change to 4?thank you

Set the indent.value to 4 spaces instead of 2.
https://github.com/millermedeiros/esformatter/blob/master/doc/config.md#indent


widgetplugin: reduce some of the locking, simplify and also fix the deadlock on s8

doResume must be called before a widget is initialized; the code though was written in such a way that doResume may end up waiting for widget initialization
Addresses the deadlock from https://github.com/Samsung/GearVRf/issues/1139

GearVRf-DCO-1.0-Signed-off-by: Mihail Marinov<EMAIL_ADDRESS>

Verified with all backends on S8 and Note5.

Will merge Thursday after lunch


Feature request: Anime-proof Japanese Morphology

To be used along side https://github.com/zcyzcy88/TagTable and https://github.com/evazion/translate-pixiv-tags there should be a feature to add custom words into Janome to be more accurate for certain tasks

A tool that guides people on creating user defined dictionaries...

User defined dictionary is just simple CSV file, I'm not sure what kind of tool is needed (except for ordinary text editors) for that. If you have any thoughts on your mind, can you open an issue for that? Please be more specific.


Fix build and librosa numpy dependency errors

Updated librosa function signatures (added missing positional arguments according to https://github.com/librosa/librosa/pull/1521), so librosa and numpy dependency can be flawlessly updated. Also fixed scikit issues according to #1068.
Updating librosa fixes #473, fixes #620, fixes #1016.
Removing Dora fixes  #765, fixes  #1046, fixes #522, fixes #725, fixes #919.
I also suggest installing into Python virtual environment instead of breaking Arch system Python, so installed as follows:
cd /path/to/UVR
python -m venv venv
source venv/bin/activate
pip install -r requirements.txt
python UVR.py

Tested on Manjaro 23.1.4 and Python 3.11.

good job. why not the author merge it? strange


Unify the constructor and build method names for all builders

We use a lot of builders in Jodd. They are created using the new keyword:
new FooBuilder().setAaa(a).setBbb(b).build();

Let's add a method that replaces the constructor and to unify the build method, eg.:
FooBuilder.newFoo().setAaa(a).setBbb(b).build();

In other words:

constructor method would be called new<Name> (e.g. newThreadFactory) - similar to what one would be written (new ThreadFactory).
builder method (the last one) is just build

@neroux @moh-sushi wdyt?

Now, we deeply delve into design patterns :)
From what I understand you'd like to replace the instantiation of a new builder object with a static method, right? At the first glance I would have "yelled" Factory :) and I actually had to refresh my memory on that subject and found http://stackoverflow.com/questions/757743/what-is-the-difference-between-builder-design-pattern-and-factory-design-pattern, which suggests a builder class should actually use instantiation versus the factory's common static method approach.
Do you have a particular reason in mind why you'd like to introduce that change?
On a different but related subject, are we sure we need builders/factories in all that cases anyhow or would a regular class instantiation maybe suffice? Chaining setters would be still possible with the latter anyhow.
As for build(), I believe I missed something here. Both examples above refer to a zero-parameter method.

Here is the background story - it is really not much, neither a big change, it's more about proper and unify naming, which I always personally have problem with :)
In Jodd we are already using Builder pattern on several places, in order to simplify complex creation of some objects, and to make it more visual and clear to user-developer. Factories are not used much (if any), as we don't provide different implementations of the same things, hence no reason to have them in the first place.
Factories, where needed, will be inside the interface, as an interface static method. As a library, we don't need to provide more then that... as factories are easy to build, and yet everyone see it configured differently, depending on the needs on the projects.
Builders, on the other side, goes well with making things developer-friendly. I love when code gives users hints on how it something has to be used. For now we use them in Jodd on several places to make object creation simpler.
This issue is all about naming the builders. We have two types of builders:

'real' builders, like ThreadFactoryBuilder - that simply creates ThreadFactory in an fluent way. It has newThreadFactory() static method for creating the builder and finally build() method to construct the ThreadFactory and returns it.
'bean' builders, like Email - where we just need to create and populate a bean. It has create() static method for creating the the bean - that might be called createEmail().

All I want is to have the unified naming across libraries :)

ThreadFactoryBuilder is a good example, as I grepped earlier today for Builder and came across it. I am not quite sure why there are two build methods. The instance method is calling the static method and passes this (which basically also happens on a compiler level with the instance method anyhow). If the static method wasnt private I could even imagine some other use case, but in that way I cant and would really feel tempted to unify/simplify that particular class (unless of course you have an objection).
My question about the purpose of builder classes was mostly because I dont think it would be necessary to have additional classes to achieve something that can be easily done with a constructor. There'd be virtually no difference between
Dummy dummy = new DummyBuilder().setString('').setInt(123).setBoolean(true).build();
and
Dummy dummy = new Dummy().setString('').setInt(123).setBoolean(true);
If I blatantly missed something please let me know.
But as for a unified naming schema, that's definitely something desirable 👍

Absolutely! This is that second example I was talking about :) Let me use your example and add all the variants that i have in mind:
/A with constructor, default usage:
Dummy dummy = new Dummy().setString('').setInt(123).setBoolean(true);

/B with static construction method (i like this as in the future you can change the way how class is created, i.e. we might prepopulate etc):
Dummy dummy = Dummy.create().setString('').setInt(123).setBoolean(true);

or:
Dummy dummy = newDummy().setString('').setInt(123).setBoolean(true); // with static import

/C Finally, there is a third case of builders for objects that we dont control, like ThreadFactoryBuilder:
Mummy mummy = new MummyBuilder().setString('').setInt(123).build();

or:
Mummy mummy = newMummyBuilder().setString('').setInt(123).build(); // with static import

--
Questions:

case B & C: should static constructor method can be called create() - which is ugly with static import; OR newDummy()/newMummyBuilder()?
case C: should terminal method be called build()?

I would dismiss B in general. There is little point in emulating a constructor via a static method.
As for C and the builder context, I'd still go for the constructor and I guess build would be a fitting name (and seems to be pretty standard) for the method eventually doing the hard work.

Nice, thanx for sharing thoughts, make sense!

I would also prefer C with its final method name build.
Additional remark:
What do you think about naming the methods with prefix with and name of property?
example:
Mummy mummy = newMummyBuilder().withLabel('jodd').withAge(123).build();

@moh-sushi Yes, with is today commonly used for builders. The only thing with this is that when we have a bean (with get & set), we will use setters to return this so a bean becomes fluent.
Closing this one as resolved


[DOC]update depreciated brew commands for installing Java and Spark

The current brew install commands automatically downloads Java 9 but there were version conflicts with Scala. Also, "brew tap homebrew/versions" is depreciated and there is no way to set version 2.1 for Apache Spark. The current change simply downloads the newest version but perhaps it can be included in the Maven dependency if version specification is necessary.

Can an Admin verify this patch?

Hi @Matsunaga-D , I believe we are not supporting Java 9 anytime soon.
ping @dusenberrymw .

LGTM. @j143 can you merge this?

I will merge this.

Thank you, Daiki. Congratulations for your first commit to SystemML.

Thank you @j143 and @dusenberrymw!


Token limit change after apollo-parser 0.5.3

Description
Apollo Router exposes the token limit in its own configuration. The test for this config fails when upgrading apollo-router from 0.5.3.
Steps to reproduce
Modify the existing token_limit test in crates/apollo-parser/src/lexer/mod.rs:
    #[test]
    fn token_limit() {
        const TEST_INPUT: &str = "type Query { a a a a a a a a a }";
        const TEST_INPUT_TOKEN_COUNT: usize = 25;

        let lexer = Lexer::new(TEST_INPUT);
        let (tokens, errors) = lexer.lex();
        // +1 for the EOF pseudo-token
        assert_eq!(errors, &[]);
        assert_eq!(tokens.len(), TEST_INPUT_TOKEN_COUNT + 1);

        // EOF is not counted in the limit
        let lexer = Lexer::new(TEST_INPUT).with_limit(TEST_INPUT_TOKEN_COUNT);
        let (tokens, errors) = lexer.lex();
        assert_eq!(errors, &[]);
        assert_eq!(tokens.len(), TEST_INPUT_TOKEN_COUNT + 1);

        let lexer = Lexer::new(TEST_INPUT).with_limit(TEST_INPUT_TOKEN_COUNT - 1);
        let (tokens, errors) = lexer.lex();
        assert_eq!(
            errors,
            &[Error::limit("token limit reached, aborting lexing", 31)]
        );
        assert_eq!(tokens.len(), TEST_INPUT_TOKEN_COUNT - 1);

        let lexer = Lexer::new(TEST_INPUT).with_limit(10);
        let (tokens, errors) = lexer.lex();
        // Error happens earlier, then lexing stops
        assert_eq!(
            errors,
            &[Error::limit("token limit reached, aborting lexing", 17)]
        );
        assert_eq!(tokens.len(), 10);
    }

Expected result
Passes in both 0.5.3 and current main 126816fc6717bf5df429dac48cd71995e5538a7d.
Actual result
Passes in 0.5.3. On main:
thread 'lexer::test::token_limit' panicked at 'assertion failed: `(left == right)`
  left: `[ERROR@31:31 "token limit reached, aborting lexing" ]`,
 right: `[]`', crates/apollo-parser/src/lexer/mod.rs:646:9

The panic location points to the .with_limit(TEST_INPUT_TOKEN_COUNT) case. If commenting out the failing assert produces:
thread 'lexer::test::token_limit' panicked at 'assertion failed: `(left == right)`
  left: `25`,
 right: `26`', crates/apollo-parser/src/lexer/mod.rs:647:9

Still in the the .with_limit(TEST_INPUT_TOKEN_COUNT) case. Also commenting out that one makes the test pass. So it doesn’t look like an off-by-one change, but rather something that happens specifically around the EOF token. I don’t fully understand it just by looking at the code though, since TokenKind::Eof is (surprisingly) now created in two places.
Environment

apollo-rs crate: apollo-parser
Crate version: 126816fc6717bf5df429dac48cd71995e5538a7d

I might call this intentional as previously you would get one more token than the limit, as the eof token was not part of the limit. Now when you .collect() the output of the lexer in a Vec it really will not exceed token_limit. Will call it out in the changelog.

Admittedly it does feel a bit odd to have to abort just because the EOF pseudo-token doesn't fit. But you're not really meant to be close to the limit in normal use.


Added password toggle and improved UI

Issue: #279
Description

Password Toggle: Add a clickable eye icon beside the password fields and use JavaScript to toggle the type attribute between "password" and "text" to show or hide the password.

Validation Feedback: Add an asterisk () after each field label (Full Name, Email*, etc.) using HTML/CSS for required fields.

Update Contrast: Modify the CSS color property of the "Sign in" link to a darker shade (e.g., #000 or a color with higher contrast) to meet accessibility standards for better readability.

Type of PR

[x] Feature enhancement

Screenshots / videos (if ap

Before:

applicable)

After:

Checklist:

[x] I have made this change from my own.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

@PriyaGhosal Kindly review and merge my PR

@PriyaGhosal Kindly review and merge my PR

@kushal7201 Your pull request has been successfully merged! Don't stop here—continue contributing by addressing more issues and aiming to become a top contributor. Exclusive swag awaits for those who go the extra mile!
Great contribution to GSSOC'24 EXTENDED! 🎉


Does Tag really need checkable?

Maybe we should think about tag's main use cases, seems duplicate with checkbox's

I think I find the use case for checkable
see: https://joshuajohnson.co.uk/Choices/


Chi-Square Test

The Chi-Square Test is a statistical method used to determine if there is a significant association between categorical variables. It compares the observed frequencies of categorical data with the frequencies that would be expected if the variables were independent. The test calculates a Chi-Square statistic, which measures the difference between observed and expected frequencies under the null hypothesis of independence. It is widely used in fields such as biology, social sciences, and market research to assess relationships between variables without assuming a specific distribution of the data.
Used for categorical data to assess how likely it is that an observed distribution is due to chance.

Chi-Square Goodness of Fit Test: Tests if a sample matches a population.
Chi-Square Test for Independence: Tests if there is a significant association between two categorical variables.

Can you assign it to me? @Niketkumardheeryan
Full name: Manish Rana
GitHub profile link: https://github.com/ranamanish674zu
Email ID<EMAIL_ADDRESS>Approach for this : python
What is your participant role?: GSSoC-2024 contributor
Can you add the label for GSSoC, i want to work on it
Thanks.


Merkle tree API does not support r1cs + no_std

Seems that I forgot to handle R1CS in no-std context. Result is that r1cs would not work with no-std.
I will fix that shortly. Sorry for the inconvenience.
(I think in the future, we need to update the CI to ensure r1cs compiles with no-default-features. )

Current workaround: ensure std feature is on when writing the constraints. I will fix that soon


Stop building llvm with modules enabled.

Stop building LLVM with C++ modules until they are more robust.

@swift-ci please smoke test

@adrian-prantl @vedantk Adrian and Vedant, do you have any objections to this? It's causing some problems for Swift CI (rdar://problem/29627138)

No, this makes sense! Thanks, and sorry for the disruption.


Issue with synch with time of the video and comments (mostly in live)

There's an issue with times, for example, when you see an live, and it is on 5:00, when you post a comment in minute 5:00 and other people are in 4:30, they can actually see the comment, and can be a potentially spoiler for people

I have no solution for it, but I think websocket danmaku is a better way for live.

There's no "time" in live, I think all people should watch the same danmaku at the first time. Does there any live service have an exception?


Generate PKI for watcher chart in Kyma reconcile loop

Description
The SKR watcher (web-hook server + watcher http client) need PKI (public key, private key, ca certificate) to authenticate with the SKR API server and with the KCP Gateway.
In the current implementation, the Kyma control loop is reconciling the installation of watcher helm chart using helm to generate the needed PKI.
In this backlog item, we implement the PKI generation using the Gardner cert-management capabilities.

PoC documented here

ACs

[ ] SKR watcher uses the same PKI for authenticating with the k8s API server in SKR and for establishing the mTLS connection to the KCP cluster endpoint.
[ ] PKI management and rotation leverages Gardner's cert-management capabilities.
[ ] "local mode" should also be supported for k3d (BYOC and local clusters will not run in the Gardener)

Developer notes

[ ] check Issuer  by type
[ ] wait for Issuer to be ready
[ ] check Gateway certificate by type is ready
[ ] create SKR certificate gardener
[ ] wait for certificate to be ready
[ ] take secret from certificate and include in SKR watcher installation

Why is this in blocked and not refined again? I thought we aligned here that this is the most important ticket of Phase 2a. @janmedrek @khlifi411


Add missing documentation on arguement retry_exceptions

Closes #269

Successful PR Checklist:

[ ] Tests

[x] (not applicable)

[x] Documentation

[ ] (not applicable?)

[x] Had a good time contributing?

Great job ! Thanks !


Misleading comment in docker toolbox points users for expensive alternative

File: toolbox/toolbox_install_windows.md
This is about the statement "Legacy desktop solution. Docker Toolbox is for older Mac and Windows systems...."
A tool like docker is provided for free and used as a standard used for open source and free projects as well as self training for individuals.
Docker for windows requires windows 10 professional for hyper V, this is not only a very specific version of the operating system but also the most expensive flavour, which costs an extra 100 dollars.
My brand new computer comes with windows 10 home included, and i had no plan on buying a new operating system, I was mislead by that statement in many ways, first Docker for windows would not work for my system even if my computer is not "an older Windows system", but also if i develop my project and my scripts on top of Docker Windows rather than Docker Toolbox I would prevent many of my users from using easily my project and they would have to adapt it. I want to make life easier for my users rather than more difficult.
Having say that i understand that the docker windows is the new project which contains the new features which you don't offer with the toolbox, and that your brand new project is the exciting one for you guys and that is the one you want to promote. All fine and very understandable. But I would like you guys to provide a neutral statement which has the best thing for all kinds of users in mind.

No longer applicable as Docker Toolbox has been deprecated.


ssh_public_key_file Variable in instance-with-ssh example

Hi Stephen,
I was creating "instance-with-ssh" and got error in "ssh_public_key_file" :
1 error(s) occurred:

opc_compute_ssh_key.ssh_key: 1 error(s) occurred:

opc_compute_ssh_key.ssh_key: file: open ./id_rsa.pub: no such file or directory in:

${file(var.ssh_public_key)}
################
I changed the variables file:
--OLD VALUE
variable ssh_public_key_file {
description = "ssh public key"
default     = "/.id_rsa.pub"
}
--NEW VALUE
variable ssh_public_key_file {
description = "ssh public key"
default     = "~/.ssh/id_rsa.pub" --to get my default id_rsa.pub file
}
Thanks,
Portugal

An alternative to modifying the default value would be to create a terraform.tfvars file with the path to your ssh key file
ssh_public_key_file="~/.ssh/id_rsa.pub"


Linux releases contain only x64 build

I think the Github build script adds the x64 code to aarch64 and armv7 releases. I copied all 3 linux releases to different folders and get the same message on running it in Docker (on macOS):
root@602a0ba1c943:/data/aarch64/piper# ./piper
OrbStack ERROR: Dynamic loader not found: /lib64/ld-linux-x86-64.so.2

This usually means that you're running an x86 program on an arm64 OS without multi-arch libraries.

Checking with the file command gives me this:
root@602a0ba1c943:/data/aarch64/piper# file ./piper
./piper: ELF 64-bit LSB pie executable, x86-64, version 1 (GNU/Linux), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, BuildID[sha1]=4b66130bd84000a8da56440929f82b181fcad608, for GNU/Linux 3.2.0, not stripped

I never used GitHub builds before, but as I read the main.yml script, I personally think it copies the same binary for all 3 platforms:
      - name: package
        run: |
          cd _install && \
          tar -czf piper_linux_${{ matrix.arch }}.tar.gz piper/

There is no {{ matrix.arch }} specified on the build command. I have no idea how to add this, to be honest.

Fixed in the latest release: https://github.com/rhasspy/piper/releases/tag/2023.11.14-2
Apparently Github runners don't actually run using the architecture you specify :/


fahcontrol,fahviewer: fix variable substitution for shortcuts

Motivation for this change
Because single quotes are used in the sed command, the application
shortcut had the literal $out instead of the package path.
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/710


release 3.1.1

PR for release 3.1.1. This is a bugfix release, so feature or breaking PRs like #3405, #3515, and #3581 should not be merged into it.
I created this on a LightGBM branch so any maintainer can push to it.
Release checklist:
The PRs added to the checklist below are based on https://github.com/microsoft/LightGBM/pull/3586#issuecomment-735417087

[x] Update VERSION.txt number.
[x] Update version in Appveyor config file.
[x] Update version in configure file of R-package.
[x] merge #3598
[x] merge #3592
[x] fix #3629
[x] run R valgrind checks after all PRs are merged
[x] run R Solaris checks after all PRs are merged
[x] Push commit with v* tag to trigger GitHubRelease action at Azure Pipelines.
[x] Convert automatic GitHub release from Draft to normal one.
[x] Update stable tag at GitHub.
[x] Upload release to PyPI.
[x] Upload release to CRAN.
[x] Update cran-comments.md when 3.1.1 is accepted on CRAN
[x] Upload release to NuGet.
[x] Update version and hash in Homebrew formula.

I just updated this branch with the changes from #3592. Will run the Solaris and valgrind checks on the R package right now.

Thank you! I will draft the new release

Thank you! I will draft the new release

ok great, I'll submit to CRAN shortly! They'll send you an email with a link to click. It should go a lot faster this time.

Do anyone know why v3.1.2 is auto released?

@guolinke can you check LightGBM slack? I made a mistake tonight, and explained it there.

@guolinke I removed 3.1.2 release and tag. Should be good now (it is a Draft that is seen only by maintainers again now).

@jameslamb I've noticed that release drafter re-visites all PRs when triggered by a new commit in master. So there is no problem if you forgot to label PR. Just add a label later after a merge and release drafter will pick it (and categorise in appropriate changes group) after any next PR will be merged.

got it, thank you. Sorry again :grimacing:


docs: update the development branch docs

[x] instructions to spin up the platform should be in line with current development branches
[x] rework the "First Steps" section for new CLI and UI

closed via #141


the map is not displayed anymore

The map that should be displayed by Xarrayleaflet is not displayed anymore I shloud investigate and maybe return to a normal gee display (as everything is gee based)

Corrected with sepal_ui v1.1.5


Docs: Usage examples and sponsoring

Add example uses and request for maintainers and sponsors to readme.

Codecov Report

Merging #865 (56f2521) into main (4d2c74a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #865   +/-   ##
=======================================
  Coverage   99.39%   99.39%
=======================================
  Files         115      115
  Lines         823      823
  Branches      230      230
=======================================
  Hits          818      818
  Misses          5        5

Continue to review full report at Codecov.

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