Legal fund

Is there a place where we can donate to your legal fund when they hulk smash over the use of their mark?  Your tool seems too useful and functional to claim parody although obviously I'm no lawyer.

I am not sure if I should mark it as issue. I think there is a typo in the name of this project. It's name should have been BSNL but it is misspelled as comcast.

There are a lot of other ISPs who I'm sure would be happy to have a
namesake.
On Wed, Jan 28, 2015, 11:58 AM Jeremiah Gowdy<EMAIL_ADDRESS>wrote:

Is there a place where we can donate to your legal fund when they hulk
smash over the use of their mark? Your tool seems too useful and functional
to claim parody although obviously I'm no lawyer.
—
Reply to this email directly or view it on GitHub
https://github.com/tylertreat/Comcast/issues/9.

Plot twist: @jgowdy is the person at Comcast whose job it is to reduce the cost of lawsuits.

lulz @planetmcd :+1:

It's just an abbreviation of "Communications Castrator"

@up You Sir are da real MVP

I believe this issue is related to #10. If that were implemented, I believe it would turn this project into political speech, to which the First Amendment would apply. (Disclaimer: I'm not a lawyer.)

Agreed that #10 reinforces [what I see as] all aspects this valuable piece of culture and technology.  This is a distinct tool from ChaosMonkey, and provides a valuable service that reflects the real challenges that are faced in delivering superior customer experiences.

Simulating the quality of the Comcast Internet Connection may also qualify as patent infringement as I'm sure Comcast has patented everything they could.

The name of this project is hysterical.
There shouldn't be any legal issues in the US given the nature of this product and the explicit protections on parody in trademark law. I'm not a lawyer, but been around the block with IP.


some of files disappeared after download

Hi
Some of my users are reporting   files disappeared  after hours or days after downloading and they download those files again
Why this happened?

Can you share the dir path? Can Provide additional information or app URL link. To help.


Align the encryption flags

Changes

Align the encryption flags according to https://github.com/knative/serving/issues/14368
Adds reverse compatibility for existing flags
Drop dataplane-trust and controlplane-trust

I'd argue that its ok to drop the dataplane-& controlplane-trust flags, as they are clearly marked as ALPHA. IMHO, Introducing backward compatibility for those would make it really unread and unmaintainable. Also we have not implemented this feature and various values of those, so it's basically not possible to be reverse compatible.
/kind cleanup
Partially fixes https://github.com/knative/serving/issues/14368
Release Note
Renames the flags that control encryption
- `auto-tls` is now named `external-domain-tls`
- `internal-encryption` is now named `knative-internal-tls`
- `cluster-local-domain-tls` is introduced as a new alpha state flag to control TLS certificates for cluster-local domains

Docs
PRs for docs will follow separately in https://github.com/knative/serving/issues/14368

/assign @dprotaso
/assign @KauzClay
/assign @nak3

About the expected CI errors:
I see the following options

Keep the dropped flags here (without handling the compatibility stuff), rework all repos, then drop them here in a new PR
YOLO - ignore it for now, fix it after follup-PRs on Serving + net-contour are through

We'd need to do both approaches our configuration would be broken for a short time period. And we'd need to do the fixes in a short time period and before the release.
Opinions?

I think I could go for

ignore it for now, fix it after follup-PRs on Serving + net-contour are through

I can get some quick PRs for serving and net-contour to use your new stuff ready

I'd argue that its ok to drop the dataplane-& controlplane-trust flags, as they are clearly marked as ALPHA. IMHO, Introducing backward compatibility for those would make it really unread and unmaintainable. Also we have not implemented this feature and various values of those, so it's basically not possible to be reverse compatible.

I would like to get approval from TOC just in case. Otherwise LGTM.

@knative/technical-oversight-committee @dprotaso  Could you please take a look: https://github.com/knative/networking/pull/858#issuecomment-1724720424

I would like to get approval from TOC just in case.

TOC wouldn't make this call - just leads of the WG. It's an alpha feature so I don't we should worry about the renaming.

/lgtm
/approve
/hold
@ReToCode will let you unhold

/unhold
@dprotaso can you override the failed checks?

/override "Unit Test (knative/serving)"
/override "Unit Test (knative-extensions/net-contour)"

/override "style / Golang / Lint"


deallocate all locals on function exit and transitively freeze constants through pointers

This should be pretty cool for const eval, since at the end all one has to do is traverse the allocation list and create llvm allocs for all frozen allocations. As long as something is reachable, it gets frozen, everything else is gone.
fixes #83

addressed comments


'Could not find /fsevents' - failure to cope with (validly) missing modules

For some reason my application turns out to be 64mb - so thought I'd use this tool to try and work our which dependancy is the problem.
 analyze-module-size
dependencies found: 687 ...done
Loading dependency details: =============================================================================================== /yargs
Connecting dependency graph
Error: Could not find "/fsevents"

Not sure if this analyze-module-size or something else that requires fsevents.

The problem is clearer, https://www.npmjs.com/package/fsevents is the module in question which is OSX specific. As I'm on Linux, this isn't installed - so therefore it isn't going to be found.
So not a bug as such but an enhancement to cope with modules that aren't present validly so.

The fact that fsevetns' is not enough  to get this error message. The error is thrown [in src/Package.js](https://github.com/nknapp/analyze-module-size/blob/master/src/Package.js#L43) when a dependency claims to be _requiredBy` this package.
There must be a dependency that has _requiredBy: [ '/fsevents' ] in its package.json. Is your package  using lerna like the composer?

I you like, you can submit a PR that does not throw an exception but logs a warning instead.
But, it feels like there should be a warning logged (like xxxx is required by fsevents, but there is no such package.


Stale proposals alert in email for managers - hierarchical

User Story
stale proposals alert manager in email

Epic Title
Hierarchical

T-shirt size: SML

User Story
"As a manager of approvers, I want an email alert as proposals grow stale so I know I have actions to take to keep the business moving"

Description
Lead time to trigger this alert is a configurable setting.

Acceptance Criteria/Tests
Verify that the email alert correctly triggers for the manager when a direct report has stale pending proposals.  Verify that alert indicates which approver(s) has the actions -- can list more than one.

migrated to new tmobile/sawtooth-next-directory repo


asDecimal delimiter inconsistency

What steps will reproduce the problem?

Make sure php intl is installed
Set your app's language to a language which uses comma as decimal delimiter (e.g. German)

echo Yii::$app->formatter->asDecimal("0.026199999999999998", 4);
result: 0.0262
echo Yii::$app->formatter->asDecimal("0.026199999999999", 4);
result: 0,0262
As you can see "asDecimal" gets inconsistent with the delimiter as soon the decimal places exceed 15.
What is the expected result?
0,0262
What do you get instead?
0.0262
Additional info

Q
A

Yii version
2.0.21

PHP version
7.1

Operating system
Debian Stretch

Maybe #17369 is related but I still had no time to go deep in the issue.

This is expected behavior since your first argument after normalization to float changes its value so it falls into asDecimalStringFallback which clearly states it doesn't use intl and not explicitly set decimalSeparator becomes there a dot.
To get comma set formatter's decimalSeparator explicitly to ,.

Hm is there any reason behind this? Yii 2.0.15 did that very well. It feels a little bit strange to explicitly set the separators after defining the app's language (which should do the job anyway?).

See #14950

Just in case someone has the same issue, add the following to a bootstrap component:
`
$numberFormatter = new \NumberFormatter($language, \NumberFormatter::DECIMAL);
        $app->formatter->decimalSeparator = $numberFormatter->getSymbol(\NumberFormatter::DECIMAL_SEPARATOR_SYMBOL);
        $app->formatter->thousandSeparator = $numberFormatter->getSymbol(\NumberFormatter::GROUPING_SEPARATOR_SYMBOL);

`
where $language is the locale string (e.g. de-DE).


[ stable/concourse ] HostPath can now be used as working dir for workers

What this PR does / why we need it:
Amazon supports NVME drives, which now can be mapped as concourse working dir using hostPath volumes.
Special notes for your reviewer: The goal is to be able to use NVME for baggage claim, if you have a better implementation, I'm all ears. This just seemed the fasted/less complicated way of doing it since baggage claim is set within the working directory.

/assign @william-tran

Found a better implementation to get what I need. Will re-open PR later.


Probrably doing something wrong

Traceback (most recent call last):
File "C:\Users\Diobo\Free-Games\Free_Games.py", line 308, in
browser = start_firefox_browser(user_agent)
File "C:\Users\Diobo\Free-Games\Free_Games.py", line 155, in start_firefox_browser
browser = webdriver.Firefox(profile) # Setup the browser object to use our modified profile
File "C:\Users\Diobo\AppData\Local\Programs\Python\Python38\lib\site-packages\selenium\webdriver\firefox\webdriver.py", line 170, in init
RemoteWebDriver.init(
File "C:\Users\Diobo\AppData\Local\Programs\Python\Python38\lib\site-packages\selenium\webdriver\remote\webdriver.py", line 157, in init
self.start_session(capabilities, browser_profile)
File "C:\Users\Diobo\AppData\Local\Programs\Python\Python38\lib\site-packages\selenium\webdriver\remote\webdriver.py", line 252, in start_session
response = self.execute(Command.NEW_SESSION, parameters)
File "C:\Users\Diobo\AppData\Local\Programs\Python\Python38\lib\site-packages\selenium\webdriver\remote\webdriver.py", line 321, in execute
self.error_handler.check_response(response)
File "C:\Users\Diobo\AppData\Local\Programs\Python\Python38\lib\site-packages\selenium\webdriver\remote\errorhandler.py", line 242, in check_response
raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.SessionNotCreatedException: Message: Expected browser binary location, but unable to find binary in default location, no 'moz:firefoxOptions.binary' capability provided, and no binary flag set on the command line

I'm no longer going to update this repository. Please use this new and improved one that makes use of the Epic Games desktop app rather than logging in via the browser:
https://github.com/MasonStooksbury/Free-Games-V2


Ensure all Key Vault models align correctly with the Swagger specification

Issue #14513 showed that the Key Vault Keys library had issues when deserializing a couple models (fixed by #15375), so it would be a worthwhile effort to further look into the rest of the models in Keys, Certificates and Secrets to ensure everything conforms to the service's Swagger specification.

Closing as KeyVault now uses code generation


file renamed; README updated accordingly. closes #5

instantce is confusing to read; that's alike instant and instance in one word.
quick_app or fast_app would also be an option.

It is indeed a grammatical error. Thank you for your contribute.


corrected --web-page option

Corrected the --web-page option and fixed some minor typo's along the way.
awesome work porting this to Windows🤜

How do I merge this w/o messing up my local repo? I'm still a n00b when it comes to git.
I've now added your patches here locally and thinking of doing git commit --author smholvoet README.md.
But I failed to find a proper email  address for you!

You could accept these changes right here without the need to first pull them in locally, take a look at Merging a pull request

You can then git pull or git fetch to update your local repo.
If you'd like to preview my changes the easiest way would be to probably to clone my fork which holds the changes.


Bad guidance regarding global.json usage

The current version of this page reads like a general recommendation not to use global.json files. This is harmful imo. This may be due to the history of the global.json; once upon a sad time we were not able to set relaxation rules regarding enforcement, so someone who installed a patch to the specified version was prompted to install a lower version, which made the whole thing an annoyance.
With rollForward being available common sense has always been to pin down versions as much as possible.
I think there should be a strong recommendation to use global.json (just like every big dotnet codebase on github is doing already). Tooling has caught up (github actions reads them, azdo reads them) and i find no earthly reason why these files aren't automatically generated by dotnet new in clean workspaces - it is generally considered bad practice to leave implied dependencies dangling like this in CICD/DevOps circles.

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

ID: a94c335b-0b65-96cc-1c25-65777fbe4bb5
Version Independent ID: 6d80613f-42dc-35d7-aeff-3187a36dbbca
Content: global.json overview - .NET CLI
Content Source: docs/core/tools/global-json.md
Product: dotnet-fundamentals
Technology: dotnet-cli
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

Thanks for the suggestion. #29060 mentions CI and eliminates the text saying that global.json is only for advanced scenarios.


Promote all peers from lobby when a peer joins with the PROMOTE_PEER …

Promote all peers from lobby when a peer joins with the PROMOTE_PEER permission and activateOnHostJoin is true in config #376

Thanks @Astagor


max_batches value

This is more a question than an issue.
Since the crowdhuman608x608 dataset contains around 15000 photos for training, why is the max_batches set to 6000 only?
In the instructions of yolov4 there is a guideline saying that the max_batches shall be at least equal to the number of your training images.

Refer to yolov4-crowdhuman-608x608.cfg:

batch=64  (stack 64 images as a batch/iteration during training)
max_batches=6000
So the total number of images used during training = 64 * 6,000 = 384,000

In fact, I determined the number 6,000 by looking at the "loss curve".   Training the model longer probably doesn't help (lowering the loss) much.

thanks!


Port all Halogen examples to Halogen Hooks recipes

Doing these one at a time isn't worth the overhead. I think I'll use this PR to add all of the remaining recipes in #138.
Fixes #138 (WIP)

This builds. The only example not ported in this PR is #120, which is currently blocked until ace is in the package set.
Note: I didn't explicitly state in each recipe in this PR that it was a port of the examples folder in Halogen. Not sure whether we want to port those examples to this cookbook or not...


test: adding tests for getEnvironment().

Sample PR for https://github.com/colynb/serverless-dotenv-plugin/pull/70. This same PR should be made against the original repo after https://github.com/colynb/serverless-dotenv-plugin/pull/70 has been merged.

https://github.com/colynb/serverless-dotenv-plugin/pull/75


docs: add example error message for standalone

I missed the standalone option. Had to learn the hard way.  By adding this error message to the docs I hope search engine indexing will help other people who did not read the docs correctly.

Thank you - but honestly I don't think this is the right way to make it clear. It only describes the fact and the consequence - for people that doesn't read the docs carefully they still get confused by the error message they would receive. I personally think the docs should be clear enough, but not sure how I can improve it further

My main goal was for people who get the error, to easily arrive on this section of the page when they google is (or get the info after they chatgpt it, is that's a verb already)
Some ideas:

mention it in the features list that there is a mode where you can use it in combination with custom config preset
add some more info to Quick setup


Solution of Special Number

Issue Id you have worked upon -
#34
Briefly explain your program logic -
In SpecialNo.java User Input a number then Find the factorial of its digits then Add all the factorials then Check if the computed sum is equal to the original number and If so then the number is special, otherwise not.
Screenshots(Attach 2 screenshots of sample input and output) -
Attach here

Checklist:
Eg - If your code follow the below guidelines. Kindly change [] to [x]
All the conditions should be fulfilled for considering your code for merging -
[x] I have mentioned the question as comment in my solution file.
[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code.
[x] My code gives the correct output.
[x] I confirm that I have not copied the code from anywhere. In case its found that I have copied I can be banned from the repository.
[x

] I affirm that I strictly follow contributing guidelines and code of conduct.

@dineshparekh11,
Kindly mention the question as comment in my solution file.

I mention the question as comment in solution file


Feature/#december

@ProbablePrime, thanks for your PR! By analyzing the history of the files in this pull request, we identified @connor4312 and @jackcook to be potential reviewers.


Can anyone tell when official 2.206 geode will be realesed

Your Question
I want to know the rlease date

Release*

when it's ready


Compatibility issue with vim-processing

Hello and thanks for this great distrib.
I installed vim-processing (https://github.com/sophacles/vim-processing) by adding it to my ~/.vimrc.local but spf13-vim seems to prevent it from working.
Syntax is not colored and :make just returns:
make: *** No targets specified and no makefile found. Stop.
Vundle's :BundleList correctly shows Plugin 'sophacles/vim-processing' among the installed bundles.
Before I installed spf13, I had vim-processing installed with Vundle which worked fine.
Corresponding issue on the vim-processing repo: https://github.com/sophacles/vim-processing/issues/24

Did you solve this issue? I'm considering just installing plugins one by one on a clean vim. Easier to debug :)

Hi Abe. I didn't solve it. I went on to use vintage mode in Sublime. I'll get back to actual Vim some time.


WRQ- 30259: Replaced deprecated platform.touchscreen with platform.touchScreen

Checklist

[x] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
"platform.touchscreen" is now deprecated.
Resolution
Replaced deprecated platform.touchscreen with platform.touchScreen available in @enact/core/platform.
Additional Considerations
Links
WRQ- 30259
Comments
Enact-DCO-1.0-Signed-off-by: Stanca Pop<EMAIL_ADDRESS>

Overall coverage has not changed.


The Swift pod YouTubeSdk depends upon React, which does not define modules

Got an error when running pod install

The Swift pod `YouTubeSdk` depends upon `React`, which does not define modules

To opt into those targets generating module maps (which is necessary to import them from Swift when building as static libraries), you may set `use_modular_headers!` globally in your Podfile, or specify `:modular_headers => true` for particular dependencies.

After creating a blank Swift File and creating a bridge header, then running pod install gave me the error above.

React Native version ?

same issue facing on
react native 0.59.10.
react: 16.8.3
react-native-youtube-sdk 1.0.4


Add issue templates

Resolved #29

Hi @rachlin:
I'm not entirely sure, as I haven't set this up before. I think these are forms that are put into the "create an issue" textbox, but which are freeform. Just starter templates, for people to work off of. This is what Github calls issue templates, as far as I know


Return disposer from register APIs

New Feature Proposal
Description
The idea is, that all register APIs return an unregister function instead of void.
export function setup(app: PiletApi) {
    const unregisterMenu = app.registerMenu(() => (
        <a href="https://docs.piral.io" target="_blank">
            Documentation
        </a>
    ));

    app.on("someEvent", () => {
        unregisterMenu();
    });
}

Background
While this feature does not give you any additional flexibility, it reminds you that you probably should unregister at some point, advocating best practices.
Discussion
This could be also applied to the event handler registration via app.on. The downside would be, that it wouldn't be possible to chain handler registrations like this, making it a breaking change:
app.on("foo", () => {}).on("bar", () => {});

it reminds you that you probably should unregister at some point, advocating best practices.

That is only true in edge cases. In the usual (common) case you will never unregister. Your registrations are valid for the entire lifetime of the pilet, which is active for the lifetime of the application.

This could be also applied to the event handler registration via app.on.

Yes, we will never break the pilet API. We could introduce another function for this, but since its so straight forward
function appendFluent(api, name, handler) {
  api.on(name, handler);
  return () => api.off(name, handler);
}

I'd rather leave it out. The current on, off is taking the Node.js EventEmitter approach, which is sufficiently fine imho.
In general this is great and we should introduce this for all registerX calls.


Seems to be issues with the bot running over lengthy periods of time

I have the bot running on my VPS and decided to leave it on while I went to sleep. I woke up to the bot just scanning the area and then walking to the next area. No pokestops, no pokemon, just walking around. Judging from the level of my character and bag changes, this happened not very long after I left the bot alone (perhaps 20-30 minutes), so I would estimate that this kicks in around an hour or so of uptime total if not less.
This is the second time this has happened, and while not a huge issue to just force restart the script after 30 minutes, perhaps this should be looked into further.
Behaviour
http://pastebin.com/raw/vjZMDAFZ

Have the same problem.

Yeah, same here, I have to restart the process once in a while.

Same problem here! I have to restart every 30-60 mins.

Have the similar problem. keep printout unknown pokestop and do nothing.
[2016-07-24 21:10:43] [#] Finished walking
[2016-07-24 21:10:43] [#] Arrived at Pokestop
2016-07-24 21:10:47 [#] Now at Pokestop: Unknown - Spinning...
[2016-07-24 21:10:57] [#] Found fort 14b615d60dee42bb9e1a83041adc232a.11 at distance 0.90km

Yep, same here - can run the bot for ~2 hours before getting stuck

Same issue, starts not finding just moving around about the 30-45 minute mark as well.

same here

It seems to continue working for me if i press enter. Maybe it doesn't wake up from sleep.
Can you say what OS you are on, maybe it's scheduler issue?

@JohanJu OS seems to be irrelevant on this. I've run it on CentOS/OSX/Windows and all yield the same result..
Also note that some people are claiming duplicates of this, but I dont think they are quite the same. Alot of the "duplicates" are people actually finding the pokestops and it just walking between them without collecting anything. This one it doesnt even find the pokestops, it just searches cell blocks and finds nothing indefinitely

@vanZeben You are right, I got another bug where the bot just stops and a enter press woke it up.

maybe fixed in #701 , please confirm

tested on my server

55adefeeb580ed8bb51d097e56a506e2973377c9

I get the same things as @hklcf after about a hour with google. Seems to go longer with PTC, but I need to verify that.

The issue is with the authentication token not getting refreshed. We're aware and it will be fixed. :)

#532

Is this problem fixed already? I got it three times the last 3 Hours :-(

@L3nderSson seem work fine for me.

You changed something? Or just an Update?

@L3nderSson im using fb2cca3b1f701653d58424f49a2ba7fae5cd97e9

A change corrected a wrong behavior (teleporting to pokestops causing softban), that now should be fixed.

work fine on lastest master and dev branch

Hm okay I updated after this three times one hour ago, So normally It must work

@L3nderSson the change that fixed it was made 20-30mins ago

Oh damn thanks! So I will update later when I'll come home. Thanks guys for your help, I'm absolutly new ✌🏾

Just comment here once you update so we know if it's fixed :) @L3nderSson

Yes I will! Ehm and by the way, the right way for Update ist git pull and then the Python install require or Is that the wrong way?

I ask cause my cmd said: " please commit your changes or stash them before you can merge. Aborting"

@L3nderSson most of the times you won't have to install again, just commit your changes and then pull from the branch. Then it will list the merging issues

Running latest dev branch. Problem persists. After 20-30 minutes this happens.

While we wait for a commit on this to come through, maybe a batch file (for windows at least) that closes/reruns the script every hour would work?

I didnt have it no more, now I have the problem with scanning area for objects :D

pullrequest #735


Help request to reproduce the Table 2 results

Hi @AdamDHines,
I was trying to reproduce Table 2 results in your paper. Would you share the code that generated the VPRSNN results? I previously asked Somayeh's help here a month ago. However, I have not heard back from her yet.
I would like to ask your help for reproducing VPRTempo's results as well. I started with working on the Nordland data first. I downloaded the data and organized the directories accordingly. This is my input to the terminal.
python main.py --train_new_model --max_module 3300 --database_places 3300
It does not throw any errors during the training. However, it looks like it is terminating the training process prematurely.

When I try to test the trained model by using the following command in the terminal
python main.py --max_module 3300 --database_places 3300
I get the error below

Click to expand
Initializing modules: 100%|█████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 1/1 [00:00<00:00,  2.48it/s]
Model name: springfall_VPRTempo_IN3136_FN6272_DB3300.pth
Running the test network:  70%|█████████████████████████████████████████████████████████████████████████████████████████████████▍                                          | 348/500 [00:01<00:00, 212.59it/s]
Traceback (most recent call last):
  File "main.py", line 269, in
    parse_network(use_quantize=False,
  File "main.py", line 265, in parse_network
    initialize_and_run_model(args,dims)
  File "main.py", line 200, in initialize_and_run_model
    run_inference(models, model_name)
  File "/home/bertha/Documents/VPRTempo/vprtempo/VPRTempo.py", line 320, in run_inference
    model.evaluate(models, test_loader)
  File "/home/bertha/Documents/VPRTempo/vprtempo/VPRTempo.py", line 144, in evaluate
    for spikes, label in test_loader:
  File "/home/bertha/.pyenv/versions/vprtempo_env/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 631, in __next__
    data = self._next_data()
  File "/home/bertha/.pyenv/versions/vprtempo_env/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1326, in _next_data
    return self._process_data(data)
  File "/home/bertha/.pyenv/versions/vprtempo_env/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1372, in _process_data
    data.reraise()
  File "/home/bertha/.pyenv/versions/vprtempo_env/lib/python3.8/site-packages/torch/_utils.py", line 705, in reraise
    raise exception
IndexError: Caught IndexError in DataLoader worker process 5.
Original Traceback (most recent call last):
  File "/home/bertha/.pyenv/versions/vprtempo_env/lib/python3.8/site-packages/torch/utils/data/_utils/worker.py", line 308, in _worker_loop
    data = fetcher.fetch(index)  # type: ignore[possibly-undefined]
  File "/home/bertha/.pyenv/versions/vprtempo_env/lib/python3.8/site-packages/torch/utils/data/_utils/fetch.py", line 49, in fetch
    data = self.dataset.__getitems__(possibly_batched_index)
  File "/home/bertha/.pyenv/versions/vprtempo_env/lib/python3.8/site-packages/torch/utils/data/dataset.py", line 419, in __getitems__
    return [self.dataset[self.indices[idx]] for idx in indices]
  File "/home/bertha/.pyenv/versions/vprtempo_env/lib/python3.8/site-packages/torch/utils/data/dataset.py", line 419, in
    return [self.dataset[self.indices[idx]] for idx in indices]
  File "/home/bertha/Documents/VPRTempo/vprtempo/src/dataset.py", line 195, in __getitem__
    img_path = self.img_labels.iloc[idx]['file_path']
  File "/home/bertha/.pyenv/versions/vprtempo_env/lib/python3.8/site-packages/pandas/core/indexing.py", line 1103, in __getitem__
    return self._getitem_axis(maybe_callable, axis=axis)
  File "/home/bertha/.pyenv/versions/vprtempo_env/lib/python3.8/site-packages/pandas/core/indexing.py", line 1656, in _getitem_axis
    self._validate_integer(key, axis)
  File "/home/bertha/.pyenv/versions/vprtempo_env/lib/python3.8/site-packages/pandas/core/indexing.py", line 1589, in _validate_integer
    raise IndexError("single positional indexer is out-of-bounds")
IndexError: single positional indexer is out-of-bounds
Running the test network:  71%|███████████████████████████                                     | 357/500 [00:01<00:00, 191.34it/s]

I would appreciate it if you could help me.
Thanks in advance.
Best,
Bertha

Hi Betha,
Thanks for your interest in our work. Before debugging, I'll need a little clarification on some things. First, which version of the network are you using (v1.1.5 being the latest, noting that v1.0.0 is the version used in our paper so the results will differ if using the newer version). Second, which Nordland dataset did you download, as there are two? The paper used the full resolution images, not the downsampled ones.
Training or inferencing terminating before the end usually indicates that the number of database places and the filter (or skip value) exceeds the available number of images in your directory. So with a default filter value of 8 and 3300 database places, you need at least 26,400 images in your folder. The full resolution Nordland set contains more images than the downsampled ones, so that's why I ask. This would also likely explain your error for out of bounds indexing.
Thanks,
Adam

Hello @AdamDHines,
I am using v.1.1.5. I downloaded the Nordland data via the link shared by Somayeh at here. It contains 35,768 full-resolution images for each season.
I followed your Nordland data preparation instructions as well. I downloaded the original .zip files (of the full-resolution images) via the link you shared in your repository. I then used vprtempo/src/nordland.py to unzip and process the images. However, I encountered the following error. I noticed that each folder (spring, summer, winter, and fall folders organized by nordland.py) contains 24,570 files.

Click to expand
Initializing modules: 100%|█████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 1/1 [00:01<00:00,  1.69s/it]
Model name: springfall_VPRTempo_IN3136_FN6272_DB3300.pth
Training layer: feature_layer
Module 1:   0%|                                                                                | 0/26400
Traceback (most recent call last):
File "main.py", line 269, in
parse_network(use_quantize=False,
File "main.py", line 265, in parse_network
initialize_and_run_model(args,dims)
File "main.py", line 144, in initialize_and_run_model
train_new_model(models, model_name)
File "/home/bertha/Documents/VPRTempo/vprtempo/VPRTempoTrain.py", line 292, in train_new_model
model.train_model(train_loader, layer, model, i, prev_layers=trained_layers)
File "/home/bertha/Documents/VPRTempo/vprtempo/VPRTempoTrain.py", line 165, in train_model
for spikes, labels in train_loader:
File "/home/bertha/.pyenv/versions/3.8.5/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 631, in next
data = self._next_data()
File "/home/bertha/.pyenv/versions/3.8.5/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1346, in _next_data
return self._process_data(data)
File "/home/bertha/.pyenv/versions/3.8.5/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1372, in _process_data
data.reraise()
File "/home/bertha/.pyenv/versions/3.8.5/lib/python3.8/site-packages/torch/_utils.py", line 705, in reraise
raise exception
FileNotFoundError: Caught FileNotFoundError in DataLoader worker process 0.
Original Traceback (most recent call last):
File "/home/bertha/.pyenv/versions/3.8.5/lib/python3.8/site-packages/torch/utils/data/_utils/worker.py", line 308, in _worker_loop
data = fetcher.fetch(index)  # type: ignore[possibly-undefined]
File "/home/bertha/.pyenv/versions/3.8.5/lib/python3.8/site-packages/torch/utils/data/_utils/fetch.py", line 51, in fetch
data = [self.dataset[idx] for idx in possibly_batched_index]
File "/home/bertha/.pyenv/versions/3.8.5/lib/python3.8/site-packages/torch/utils/data/_utils/fetch.py", line 51, in
data = [self.dataset[idx] for idx in possibly_batched_index]
File "/home/bertha/Documents/VPRTempo/vprtempo/src/dataset.py", line 197, in getitem
raise FileNotFoundError(f"No file found for index {idx} at {img_path}.")
FileNotFoundError: No file found for index 2087 at ./vprtempo/dataset/spring/images-27264.png.
Module 1:   0%|                                                                                | 0/26400 [00:00

Are you planning to release a generalized framework or more detailed instructions for importing custom datasets? That would be very useful for the SNN-VPR community.
I noticed minor typos in the code causing other errors. I will not list them here, but you can find them easily by running the code from a fresh clone of your GitHub repository.
I am also particularly curious about how you obtained the R@1 values for Somayeh's VPRSNN. The results I am getting are nowhere near 53% for Nordland and 40% for Oxford RobotCar data. I would appreciate it if you could share your implementation of VPRSNN or help me find out what I am doing wrong. Feel free to check the issue I posted in Somayeh's repository at here.
Best,
Bertha

Thanks for the clarification, @berthaSZ.
I believe I found the source of the issue. The ./vprtempo/dataset/nordland.csv file is missing the first ~7000 image names in the index. This was because we only compared the precision/recall for 2700 places to VPRSNN, since the first 20% of the 3300 images in that model are used for spike calibration and are discarded from precision metrics. This is the same for the Oxford Robot Car. For now I will keep it the way it is, just to keep in line with what the paper presents but I have added some clarification in the README about this.
The error posted means that the image is not found in the directory, if there only 24,570 files then images-27264.png won't be in the folder. I'm not sure what's happening with the Nordland data from the link in the README, I'll have to investigate later as I'm a bit time poor at the moment. For now please just use the dataset downloaded from the link Somayeh shared.
I have updated the repository to include a script ./vprtempo/src/create_data_csv.py which allows users to create the .csv file required to train new networks and inference them on new datasets. Please take a look at the udpated README for usage. You can use this to generate a new nordland.csv if you wish to train and test the full 3300 images.
What errors are you referring to from "minor typos"? I have downloaded a fresh copy of the repo and cannot find any errors, except for a missing ./vprtempo/output folder which I have now corrected to create the folder if it doesn't exist first. I have no issue training or inferencing new models or using the pretrained weights.
We used pretrained weights for our comparison to VPRSNN, as we were more interested in the speed of training and inferencing than precision/recall metrics. I won't be much help in training new models for VPRSNN I'm afraid.
Thanks,
Adam

Hello @AdamDHines,
Thanks to your help, I am able to run your code.
In my case, it was much faster to train VPRTempo on the Nordland data. It took approximately ~15 minutes to train, and I achieved 64% R@1 on Nordland at 150-200 Hz inference frequency. However, the place recognition performance of VPRTempo was much worse on the Oxford data, where I obtained 14% R@1 at the same inference frequency. Additionally, it took 68 hours to train VPRTempo on the Oxford data due to using a larger set (over 60k training images and over 10k test images covering 300 places) of 2014 trajectory images. For both training and inference, I used an NVIDIA RTX 2080 GPU.
Training VPRSNN on the larger Oxford data was not feasible, as it is not as compute-efficient as your method. Therefore, I tried training on a smaller subset of Oxford data (approximately 2000 training and 1000 test images) covering 200 places. VPRSNN achieved ~12% R@1. Despite using parallel CPU computing, it took 1.5 days to train. On the other hand, VPRTempo took 9 minutes (with the RTX 2080 GPU) to train on this smaller Oxford data and achieved ~29% R@1 at 150-200 Hz inference frequency. Overall, if I did not misimplement anything, VPRTempo consistently demonstrates better performance than VPRSNN.
If you can share the original image names of the Nordland and Oxford data you used, it would be helpful for better reproducibility.
Thank you for taking the time to help me understand your work better!
Best,
Berta

My apologies for closing the issue before receiving your response. I am posting to reopen it.
Best,
Bertha

My apologies for closing the issue before receiving your response. I am posting to reopen it.
Best,
Bertha

Hi @berthaSZ,
Good to hear you've been able to get it to work. Just note, the improved performance you're seeing is likely due to the change in image downsampling and model architecture from what is presented in the paper. In v1.1.5, we use a 56x56 size input image with a patch size of 15 which is double what we used in the paper and what VPRSNN uses. We use a larger model because we can afford it with the low training and inferencing times. For the paper, we wanted an apples to apples comparison so just FYI.
The image names we used for ORC can be found in ./vprtempo/dataset/orc.csv, again this list has a truncated amount of places to account for the 20% used for VPRSNN's calibration. I would say your loss of accuracy is because you're training it on the full 60k training images. Without modifying the learning rate to be significantly smaller, you'll be overfitting the model a lot. VPRTempo works best on a single training example repeated for a 4-5 epochs rather than a variety of examples. It's more important that the reference and query is as close in viewpoint than anything, so please try ORC again with just one example dataset of the images you wish to train. Keep the filter to 8 and use the .csv file in the repo and you should be good.
Thanks,
Adam

Hello @AdamDHines ,
I can confirm the numbers Bertha provided. I also obtained significantly lower performance results for VPRTempo and particularly for VPRSNN on the Oxford data.
I think @berthaSZ  is referring to the Oxford RobotCar's image names in the original dataset naming convention, which follows the format "2015-10-29-12-18-17/stereo/right/UNIXTIMESTAMP.png". I also asked Somayeh about this here, but she ignored my requests for both the Oxford processing script and the original image names. The image names in ./vprtempo/dataset/orc.csv are not useful, as you haven’t shared your version of the aligned Oxford data. This is understandable due to redistribution restrictions. That is why I previously requested either the processing script or the image names in the original naming convention. I believe @berthaSZ is also asking for the original names for the same reason.
Matching the input image resolution does not necessarily ensure an apples-to-apples comparison. In my opinion, it is just another important model parameter. If a model performs worse with decreasing input resolution, it is simply a characteristic of the model.
I disagree with your comment about overfitting. The full training images are of unique instances and should actually help prevent overfitting. You mentioned that "It's more important that the reference and query are as close in viewpoint as possible." Wouldn't this lead to overfitting to such cherry-picked images? Wouldn't one want variety in the training and test data to train models that are more robust to real-world challenges? Of course, such closeness between the reference and query will yield significantly better results.
Best,
Robert

@berthaSZ @bobhagen Timestamped image names are now available in the ./vprtempo/dataset/ folder which contains the corresponding timestamps for each rain, dusk, and sun traversal. Please see the updated README.md for Oxford RobotCar to process raw images for training and testing. VPRTempo has a new release for v1.1.6, please download the latest code.
Following the protocol from the README, I have been able to replicate the results in the paper (see below):

You can more easily train and test like the paper, with the addition of a --skip argument that allows you to ignore the first n images in the .csv file. In this case, we train on 450 places and query 360. To train and test your own, use this:
# Train the ORC model
python main.py --train_new_model --data_dir /media/adam/vprdatasets/data/orc/ --database_places 450 --database_dirs sun,rain --skip 0 --max_module 450 --dataset orc --dims 28,28 --patches 7 --filter 7

# Test the ORC model
python main.py --data_dir /media/adam/vprdatasets/data/orc/ --database_places 450 --database_dirs sun,rain --skip 630 --dataset orc --dims 28,28 --patches 7 --filter 7 --query_dir dusk --query_places 360 --sim_mat --max_module 450


Structure change

Used standard setting keys for plugin, for better automatic configuration.
Added scripted testing support and tests.
Changed to multiproject structure as basis for future class2Avro generation task.
This branch would break compatability with old versions, don't know if such a structural change is accepted.

This is all great thanks.


Enables support to specify the region.

By default the gem fog-aws is using the region 'us-east-1' as we could see here:
https://github.com/fog/fog-aws/blob/master/lib/fog/aws/compute.rb#L302
If I specify the endpoint: https://ec2.eu-west-1.amazonaws.com, (the region is 'us-east-1', as we have seen) then the connection returns OK but when I try to list the route table ruby throws an exception because the endpoint and region are not matching.
If the user didn't specify the region I will let him know that by default he is using 'us-east-1'
Comments are more than welcome.
Thanks.

@marcgascon thanks for the contribution! I’ll let @eherot chime in here on this, as I haven‘t mucked about with this repo much at all.

In principle I like the idea of adding region support but I would rather not break backwards compatibility. How about instead we do something like:
options[:region] = @conf['region'] if @conf['region']

That way people can specify the region if they want but otherwise it defaults to us-east-1 (which would mimic most other Amazon services.

Ok @eherot it sounds good, but then I would like to update the Readme to clarify some things and to make easier to use this gem.
I will do another commit doing these changes and I will let you know.

New commit with your changes proposal and update of the documentation is added.
Thanks for the comments. It seems that the gem could be very useful for me. Can you please review it? Also, if there is something more that I could do please tell me.
Thanks!

Ok I will do all the changes you are proposing.

By the way, kyle-house-sp/nat-monitor@4dfeaa75030b317b2820b4ba76aece93b17d9da8 seems to have gone a similar route (cc @kyle-house-sp). I wager they might be interested to see this feature land here.

sorry about that, I should've pushed my PR back up...happy to do so, but this looks like it's on the same track!

Hello @eherot PR is updated with your corrections. How do you see it now?
@jeffbyrnes I was sawing the @kyle-house-sp fork and I found very interesting his first commit allowing to work with only 2 instances in the quorum. And about the region I think that we can merge this PR and he can do a rebase to continue working with the first commit. What do you think?
For the future work @eherot it could be very useful if you can describe an example (in the README) of the minimum AWS set up (Instances, AZ's, etc). I am experiencing a lot of troubleshooting using the gem.
Thanks for all your comments guys.
Appreciated!

Thanks for merging @eherot ! When do you think that I will be able to download this new version from rubygems?
And @jeffbyrnes, yes it could be very useful a basic diagram or explanation of the use case to see how many instances we need to use this gem and the communication between them because from now on I wasn't able to work with the gem as expected, but I will keep investigating.
Thank you all!

@marcgascon sorry it took so long. The new release (v1.0.11) should be available on RubyGems now.


Release 0.1.2

### v0.1.2

* Preserve chunking in Fourier filters. #33
* Pull in our fftfreq simplifications from upstream. #33, #34
* Drop unused imports. #35

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Remove debug trace

@Kartones please CR. I don't think this needs to wait for tests

Frontend tests were OK :+1: (details)

:+1:


Fanart widgets are not working.

It keeps on buffering where I use it too.

I know, there is a problem that makes requests extremely slow, but now I can't figure out why
PR are welcome


Implement the functionality of EntityLoadInfo from CUBA

In CUBA, there is the EntityLoadInfo class that stores information about the entity class and its id. The EntityLoadInfo can be serialized into string. We should have something similar in Jmix. There is a io.jmix.core.Id class but it doesn't support serialization/deserialization into string.

Use io.jmix.core.IdSerialization bean.


prepare for new telemetry

Describe your changes
Issue number
#000
Checklist before requesting a review

[ ] The pr title can be used to describe what this pr did in CHANGELOG.md file
[ ] I have executed pre-commit on my machine
[ ] I have passed pr-check on my machine

Thanks for your cooperation!

Thanks @lonegunmanb  , LGTM!


Alignment issue

Alignment or media queries not responding for medium-sized screen devices.

It is optimised only for mobile and Laptop..


Question: what about the others?

Currently I am looking at ChatGPT4All (Nomic AI) for dervative tools like Khoj (PKM or "second brain"), and it seems that this list is currently a bit short.
There is this table that circulates around for more pedestrian applications but it gets the point across for what could be included into this list https://rentry.org/ayumi_erp_rating
Also there are researchers on Reddit doing comparisons https://www.reddit.com/r/LocalLLaMA/comments/15ogc60/new_model_rp_comparisontest_7_models_tested/ https://www.reddit.com/r/LocalLLaMA/comments/15lihmq/big_model_comparisontest_13_models_tested/
Some names include MythoMax, OpenChat, Chronos, Airoboros, Hermes (by Nous), and Falcon (by TII)

See also this repo, but probably they might not be all FOSS https://github.com/MLNLP-World/Awesome-LLM


input tensor lost in --mode eightbit quantization

Quantizing with --mode eightbit somehow loses the input tensor, whereas other methods like weights_rounded don't. If I do this:
python quantize_graph.py --input ~/proc/frozen_inference_optimized.pb --output ~/proc/frozen_inference_optimized_quantized.pb --output_node_names on_logits --mode eightbit
the following error pops up in the iOS tensorflow runtime:
Running model failed:Not found: FeedInputs: unable to find feed output preprocess/centered_bgr
Here preprocess/centered_bgr is the node I want to use as the input. If instead I use something like --mode weights_rounded no error is raised and the network functions as expected.
Environment info
Operating System: ubuntu / iOS
Installed version of CUDA and cuDNN:
root@0f1fd91e29a1:~/w/tensorflow/tensorflow/contrib/quantization/tools# ls -l /usr/local/cuda/lib64/libcud*
-rw-r--r-- 1 root root 322936 Aug 15  2015 /usr/local/cuda/lib64/libcudadevrt.a
lrwxrwxrwx 1 root root     16 Aug 15  2015 /usr/local/cuda/lib64/libcudart.so -> libcudart.so.7.5
lrwxrwxrwx 1 root root     19 Aug 15  2015 /usr/local/cuda/lib64/libcudart.so.7.5 -> libcudart.so.7.5.18
-rwxr-xr-x 1 root root 383336 Aug 15  2015 /usr/local/cuda/lib64/libcudart.so.7.5.18
-rw-r--r-- 1 root root 720192 Aug 15  2015 /usr/local/cuda/lib64/libcudart_static.a

Commit I'm using: 8915f0f8072c406ae3fe0dff888f51b4cad02d7d. bazel version 0.3.0.
cc @petewarden

Does this information help solve your problem?
### I am getting APP_ERROR(5) during inference, how do I fix that?

Here's a typical error:

`"APP_ERROR(5) FeedInputs: unable to find feed output x"`

This typically means that your model was not exported correctly, or that you've
not formatted the request properly. Double check your [signatures](#signatures)
and request format.

Hi @jart, no, it doesn't because it works with a different quantization of the same frozen model file and the same runtime code on iOS.

Thank you for your patience @tachim.
@petewarden and @cwhipkey would be the best ones to help out with this.

As a temporary workaround, can you try the following to see if it fixes the issue?
python quantize_graph.py --input ~/proc/frozen_inference_optimized.pb --output ~/proc/frozen_inference_optimized_quantized.pb --output_node_names on_logits,preprocess/centered_bgr --mode eightbit
Here I've added the input node to the list of outputs. The issue is that it's deciding that the input node 'preprocess/centered_bgr' isn't needed, so I'm forcing it to include it by listing it as an output. If that fixes the immediate issue, it will give more information about what's going wrong.

Is this still an open issue?

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Add 2 more Website for Project Management and Notes

Evernote: Evernote is a popular note-taking app that lets you organize notes, ideas, and documents. While it's primarily a note-taking tool, it can also be used for project-related notes and planning.
Asana: Asana is a comprehensive project management tool that allows you to plan, track, and manage tasks and projects. It offers a variety of features for team collaboration.

@DhanushNehru anything changes required ??


Add new config property to install chrome extensions

Coverage: 89.175% (+0.02%) from 89.157% when pulling 9adf0c25424bed68ab778a90794432d08979dd29 on feat/chrome_extensions into d6b8cf5e08132b9fafc73db45a81c8c96fa27001 on master.


Add an option to banphrase emotes

If you currently banphrase an emote, anyone that types it will get timed out, even if they don't have that emote, what would be better is to check if the irc message contains the emote.
So it would be a new banphrase operator that matches the banned phrase with the emotes parameter in the PRIVMSG
something like this: !add banphrase (forsenE|forsenC) --operator emote
it would be nice if it supported regex also so we can ban multiple emotes with a single phrase or a whole emote set by banning the prefix.

NOIDONTTHINKSO

!add banphrase (forsenE|forsenC) --operator emote

So would the operator also have to support regex? Because that seems to be crossing lines with the regex operator.


How do I import this search component to use on a custom index/home page?

Love the search, but would ideally be able to import it in to my custom home page. Is there any way to do this?

I don't have any idea that either,
You can try running
npm run swizzle docusaurus-lunr-search SearchBar -- --eject --danger
this will copy the search component file to src/theme/SearchBar.
You can make changes to it or try importing it in other pages.


[Feature request] Use a different huggingface model for llama, not requiring sign-in

eg
llava-v1.5
llava-v1.6
or
llava-llama-3-8b
may we need the
mmproj  file ???
let us use it like in kobold
https://github.com/LostRuins/koboldcpp

eg llava-v1.5 llava-v1.6 or llava-llama-3-8b
may we need the mmproj file ???
let us use it like in kobold https://github.com/LostRuins/koboldcpp

Umm, I don't quite understand what you are saying.
But yes, being able to change the CLIP and LLM is the intent of this feature request.
I don't understand what you linked with kobold.

Added support for 2 models. It can easily be configured. One is for Low VRAM use, and one is for high.

kobold is usual an LLM gui .. but it has the ability to load any model with the mmproject file to recognize images
so now we can choose models without any huggingface account ?

kobold is usual an LLM gui .. but it has the ability to load any model with the mmproject file to recognize images
so now we can choose models without any huggingface account ?

I see, okay.
I don't know how to do that setup, but if you want to add it, feel free!
You can change to any huggingface model you wish. This was always the case, but I exposed it a bit higher up now:
# Clip path
CLIP_PATH = "google/siglip-so400m-patch14-384"

# Model paths based on VRAM usage
if LOW_VRAM_MODE:
    MODEL_PATH = "unsloth/llama-3-8b-bnb-4bit"
else:
    MODEL_PATH = "unsloth/Meta-Llama-3.1-8B"

I also added a low VRAM model option that can be used for faster speeds or lower VRAM GPUs.


Inconsistent/confusing handling of cursor position

It seems that it can toggle the word to the left or to the right of the cursor, which can be handy I guess, but can also be confusing and cause inconsistent behavior, especially combined with the fact that the cursor can end up in a different position after a toggle causing multiple toggles in a row to not always circle back to the original value.
For example:
Use case 1:
The text: "getCustomInput"
Cursor position: before the word "get"
Toggle-configuration for the word "custom": <not set>
Toggle result: "getCustomInput" -> "setCustomInput" -> "getCustomInput"
Cursor movement: before the word "get" -> after the word "set" -> after the word "get"
Expected: same toggle result, but with the cursor remaining before the word "get"/"set"
Use case 2:
The text: "getFirstInput"
Cursor position: before the word "get"
Toggle-configuration for the word "first": "last"
Toggle result: "getFirstInput" -> "setFirstInput" -> "setLastInput" -> "setLastOutput" -> "setLastInput" -> "setLastOutput" ...
Cursor movement: before the word "get" -> after the word "set" -> after the word "Last" -> after the word "In" -> after the word "Out" -> after the word "In" ...
Expected toggle result: "getFirstInput" -> "setFirstInput" -> "getFirstInput"
Expected cursor movement: staying before the word "get"/"set"

Thanks for the feedback.
This is not something that has been bothering me as I guess it's the output I expect altho I can see how some people may expect the behavior you describe.
When I have the time to finish the next version, I'll add an option for this, or maybe a different shortcut or maybe even both, not sure yet.

Finally got back to this issue, turns out implementing this is a bit annoying.
The cursor moving is actually the expected behavior for the code we're currently using. Instead of replacing a word when we don't have a selection, we're removing it and inserting the new word, which cause the cursor to move to the end of the word.
The reason behind this particular behavior is due to this issue in VS Code.
I just tried implementing a proof-of-concept where I manually move back the cursor to its initial position after an edit. Unfortunately, on top of being  a pain for multiple cursor as edits are asynchronous, it also adds a very annoying visual glitch where you see the cursor moving back and forth after an edit.
I think I'll just wait for the forceMoveMarkers flag described in the linked issue to become part of the public API before implementing this option.


Correctly importing ZFS pools, and nixos-generate-config perl-ness

Issue description
It's a long story, so bear with me for a minute. There's the ZFS pool issue; and then there's what I did in an attempt to fix it.
ZFS pools first.
These, like most volume managers, provide a level of indirection between the underlying device and the filesystem being mounted. nixos-generate-config already has a degree of understanding of this, even if just through robust parsing; if you use it, it'll generate segments looking like this:
  fileSystems."/home" =
    { device = "rpool/home";
      fsType = "zfs";
    };

--which are correctly handled by the rest of NixOS, generating the appropriate systemd units for mounting the filesystem.
Well, in principle. There's one problem: The mapping from device to filesystem is lost, which means systemd won't wait until the device nodes exist. There's an attempt to work around that with udevadm settle, but this doesn't actually work, as on modern systems init may be executed before the devices have even joined the bus. udevadm settle only waits until any pending udev events are processed, it doesn't wait for devices it doesn't know about.
I have on multiple occasions (https://github.com/NixOS/nixpkgs/pull/16901 and https://github.com/NixOS/nixpkgs/pull/42087) attempted to fix this, by hacky methods such as sleep loops.
Unfortunately, as I was coding the newest hack, I realized there's a problem with that approach: ZFS will happily import a pool with some missing devices, so long as there's sufficient redundancy for the pool to function. In layman's terms, if you have a simple two-disk mirror, it will still import the pool and let you mount it even if one disk is missing.
This is only natural. Mirroring would be pointless if that wasn't possible. Unfortunately, ZFS will not afterwards re-join the second disk to the pool, unless if the user explicitly asks it to by way of calling zpool online.
So, in summary: The practical effect of the wait-loop code is to make it possible that ZFS pools are imported in degraded mode, opening users up to data loss that they weren't expecting. This is terrible, and I've spent the last few hours trying to think of a quick fix... without much success. I want to say it's unlikely in practice, but all I can say is I haven't observed the behaviour myself.
I'm not sure how to quickly fix that, in a safe manner that can be backported to -stable. Help?
===
Now, the second issue: Fixing it properly requires generating systemd dependencies for the backing devices, in the form of After= dependencies on device units like dev-disk-by\x2did-nvme\x2deui.0025385871b09294\x2dpart2.device.
These cannot be generated at build-time, since the build might not be run on the machine the pool exists on. (And it would be a breach of hermeticity.) It could be done at bootloader install time, and that's an option I've considered, though it's also one I feel rather uncomfortable with. Ideally, I'd like the devices to be defined in hardware-configuration.nix -- that is what we already do for all non-pooled filesystems, in practice.
Which means that nixos-generate-config should be generating this, and I've spent the last two hours trying to make it so. It's not particularly complicated.
...except, nixos-generate-config is written in Perl. I've never learned Perl, and it seems a bad choice for everything but historical reasons.
So would anyone mind terribly if I start the process of converting it to Rust, by way of making at least this particular case use a Rust binary?
It won't expand the runtime closure. Rust binaries are statically linked; nothing but glibc is needed at runtime.

Conversation on IRC starting at https://botbot.me/freenode/nixos/2018-06-18/?msg=101240239&page=8

There is also a prototype for zfs upstream, that does this at boot time: https://github.com/zfsonlinux/zfs/pull/4943

I noticed this referenced in zfsonlinux/zfs#4943, and I wanted to point to zfs-import.target merged in zfsonlinux/zfs#6764 (and the yet to be merged integration of this target into zfs dracut integration, zfsonlinux/zfs#6964).
I was actually unaware of some of the complexities of getting all devices to become available---udevadm settle has always been sufficient in the cases I've seen. Ideally, all this wait logic should be hidden behind zfs-import.target (and possibly later zfs-import@pool.service). However, at the moment I don't believe it addresses your concerns over partially available pools.
I've rebased the zready patch, and half-integrated it with the zfs-mount-generator, but there were many comments in that PR that need to be addressed. The zready patch would allow for much faster boots on systems that have nonessential pools that take a long time to import, so merging it would certainly be preferred.

For non-root pools this is definitely the better choice.

The 'big hammer' I've got in #42269 is a better approach than requiring static configuration of the pool members, I think; it's only slightly slower than having the correct systemd dependencies would be, so we can wait for upstream.
I'm still going to try to rewrite nixos-generate-config, on the principle that probably more people can contribute if it isn't Perl. There are other changes I might like to make to it, but for now I'll be sticking to a 1:1 rewrite.
We'll see how the community feels about that in the corresponding PR, I suppose.

How'd that rewrite go? Been reading nixos-generate-config.pl lately 😅

You have to be quick otherwise it might be python3: https://github.com/NixOS/nixpkgs/pull/48378

@jcrben Real life got in the way. Suffice to say that I'm only just starting to look at it.
I don't really mind if it's Python, I just don't like Perl very much. :D
That said, thinking about rewriting the zfs-pool-import script in rust now. The shell scripting is getting over the top, and this is something that has to work.

Closing this issue in favor of https://github.com/NixOS/nixpkgs/issues/89396
If we use this generator the problem should no longer appear.


ARROW-8651: [Python][Dataset] Support pickling of Dataset objects

I opened https://issues.apache.org/jira/browse/ARROW-9332 for the parquet statistics


WebFlux tracing should trace all URLs by default.

The urlPatterns field in WebTracingProperties has different semantics for servlet and reactive web server technologies. (This naturally comes from the fact that the semantics are defined in the servlet spec for the former, and by the implementation in WebFlux for the latter.)
The default value of urlPatterns is set to /*, which enables tracing of all URLs in case of servlet stack, but it means "only single path component URLs" for reactive/WebFlux (e.g. /foo would be traced but not /foo/bar).
This PR ensures that by default all URLs will be traced by both servlet and reactive stacks, and it also improves on the documentation of the urlPatterns field to make the difference in semantics clear.

@csabakos Thanks, this looks good!
Can we have a couple tests also where the urlPatterns are not set in properties to ensure that the new defaults work as expected?

@csabakos Thanks, this looks good!
Can we have a couple tests also where the urlPatterns are not set in properties to ensure that the new defaults work as expected?

Hi @geoand and thanks for the review! I added ServerTracingAutoConfigurationDefaultsTest.java and WebFluxTracingAutoConfigurationDefaultsTest.java with that intention. Do you think more test cases are needed, or should I add some documentation about what those tests are trying to check?

@csabakos Thanks, this looks good!
Can we have a couple tests also where the urlPatterns are not set in properties to ensure that the new defaults work as expected?

Hi @geoand and thanks for the review! I added ServerTracingAutoConfigurationDefaultsTest.java and WebFluxTracingAutoConfigurationDefaultsTest.java with that intention. Do you think more test cases are needed, or should I add some documentation about what those tests are trying to check?

I personally think that some documentation would be just fine. @pavolloffay what are your thoughts on this?

I leave it to you @geoand. I think the PR looks fine.

I will take another look tomorrow, but my first impression is that this is good :)

@csabakos This LGTM! I take it that you have completed the work and we can merge?

@geoand Thank you! Yes, please feel free to merge any time.

Thank you for the PR @csabakos!


Improve usage instructions for novices

Love this project, but I no longer understand how to use it in my simple app. I used to simply add a script tag to my html:
<script type="text/javascript" src="https://rawgit.com/spencermountain/nlp_compromise/master/client_side/nlp.min.js"> </script>
But for the new version, I'm not yet familiar with node and npm. Where should I install the module, inside my project or outside of it?
I tried npm install nlp_compromise, then copied nlp_compromise.es5.min.js into my project, and adjusted the recommended <script src="./builds/nlp_compromise.es5.min.js"> </script> to point to it.
But that got me an error:
mutating the [[Prototype]] of an object will cause your code to run very slowly; instead create the object with the correct initial [[Prototype]] value using Object.create nlp_compromise.es5.min.js:5:11112 TypeError: a.match is not a function  nlp_compromise.es5.min.js:5:5697
What am I missing? Thanks

hi Julian, thanks for the kind words, and sorry for the delay.
I completely agree, I've done some big changes the past few weeks and the documentation & build have suffered.
That prototype thing sounds like a crazy bug with the build process. give me a few days, I'll try to turn it around.
that rawgit cdn is a little goofy, and i nominated this project on cdnjs, and haven't heard anything. The best way to get a cdn going is probably just do put it on an amazon cdn myself. I should probably do this.

Ok sounds great.

hey, how's this?
https://github.com/nlp-compromise/nlp_compromise/blob/master/demo/index.html
view it at:
http://rawgit.com/nlp-compromise/nlp_compromise/master/demo/index.html

The dependency is now available for me again (at http://rawgit.com/nlp-compromise/nlp_compromise/master/builds/nlp-compromise.es5.min.js), but I still get the error:
mutating the [[Prototype]] of an object will cause your code to run very slowly; instead create the object with the correct initial [[Prototype]] value using Object.create nlp-compromise.es5.min.js:5:11112 TypeError: a.match is not a function
Any ideas? I'm using it with Scala.js and building with sbt, which was working before the recent changes.
Thanks for the efforts, no rush!

hey Julian, think this is working now.
i don't see any errors when i load the simple demo
I have seen issues like this with webpack, various compilers/loaders. Please re-open this issue if it persists. cheers


LCFS - Director records a transfer - Backend REST API.

Description:
Develop the ability for the Director to record (i.e., approve) a transfer:

The backend should expose an API endpoint to handle the recording of transfers.
Upon successful recording of a transfer, the backend should update the transfer status to 'Recorded.'
The compliance balances of the two organizations should be updated accordingly in the database.
A transaction history record should be saved to the database with details of the recording, including the timestamp and the user (Director) who recorded the transfer.

Acceptance Criteria:

[ ] The backend API endpoint for recording transfers is implemented.
[ ] The transfer status is updated to 'Recorded' in the database.
[ ] Compliance balances of the transferor and transferee organizations are updated appropriately.
[ ] Transaction history records are saved to the database with accurate details.
[ ] The API endpoint is secured to ensure only users with the Director role can access it.
[ ] Backend unit tests cover the recording functionality.
[ ] Backend tests ensure accurate updating of transfer status and compliance balances along with Transaction history.

Development Checklist:

[ ] Implement backend API endpoint for recording transfers.
[ ] Update transfer status in the database.
[ ] Update compliance balances for the involved organizations.
[ ] Save transaction history records in the database.
[ ] Implement role-based access control for the recording endpoint.

Duplicate


Service is not accessible

Kubernetes service is not accessible after pod created, it is related with a hyperd bug introduced recently https://github.com/hyperhq/hyperd/issues/468.

Already fixed in #67.


Compatibility issue with IP2Location DB which uses int format for ip address range start and end

Command: mmdbctl import -c --in IP2LOCATION-LITE-DB3.CSV --out IP2LOCATION-LITE-DB3.mmdb
Output: err: couldn't parse cidr "16777216/32": invalid CIDR address: 16777216/32
IP2LOCATION-LITE-DB3.CSV file format:
"0","16777215","-","-","-","-"
"16777216","16777471","US","United States of America","California","San Jose"
"16777472","16778239","CN","China","Fujian","Fuzhou"
"16778240","16778495","AU","Australia","Tasmania","Glebe"
"16778496","16779263","AU","Australia","Victoria","Melbourne"
"16779264","16781311","CN","China","Guangdong","Guangzhou"
"16781312","16785407","JP","Japan","Tokyo","Tokyo"

@hicasper is this still an issue?

@hicasper is this still an issue?

Problem solved, thanks.


Node version is duplicated, which is brittle

At the moment the node version is duplicated in both:

the Dockerfile
the .circleci/config.yml

Renovate is only updating the version in the Dockerfile, so we have to remember to update the config.yml in the same PR manually, which is brittle.

Could use a Renovate custom manager to specify the dependency just in one place, and pass it e.g. as an argument or environment variable to the Dockerfile and the config.yml?

Not only that, the hardcoded version has also been incorrect since we moved to the go:x.y.z-node base image


Bug when used with Warptorio

First of all, thank you for this mod. I really like it :)
I tried using it with Warptorio, which I expected it to might not work, since it changes the surface the platform is on.
After warping, the item tank broke and I got this exception, when I tried to open it:
The mod Item Network (0.3.3) caused a non-recoverable error.
Please report this error to the mod author.
Error while running event item-network::on_gui_opened (ID 88)
__item-network__/src/NetworkChest.lua:511: assertion failed!
stack traceback:
	[C]: in function 'assert'
	__item-network__/src/NetworkChest.lua:511: in function <__item-network__/src/NetworkChest.lua:498>

The item tank was requesting water. There were no other item tanks. (No other fluid providers or requesters)
After this crash, I tried opening an item chest, which worked strangely enough. I expected them both to break, but only the fluid one broke.
In warptorio, the lower floors stay and the items don't get moved.
Items on the platform, that is warping from planet to planet, get affected by the warp. I don't know if it moves them or if it destroys them and tries to recreate them on the other planet.
It would be great if it could work, but otherwise I could work around the issue of not being able to use the item network on the moving platforms.

I reproduced on my end and fixed in b13cd738eed011b1966ba052d9ac995841fad9b2.

Thank you for the quick fix!


Pythran fails to perform unchecked access on the following code:

import numpy as np

def _clip(x, low, high):
    if x > high:
        return high
    if x < low:
        return low
    return x

def _integ(img, r, c, rl, cl):
    r = _clip(r, 0, img.shape[0] - 1)
    c = _clip(c, 0, img.shape[1] - 1)

    r2 = _clip(r + rl, 0, img.shape[0] - 1)
    c2 = _clip(c + cl, 0, img.shape[1] - 1)

    ans = img[r, c] + img[r2, c2] - img[r, c2] - img[r2, c]
    return max(0, ans)

Thanks to #1517, the access are now unchecked for
import numpy as np

def _clip(x, low, high):
    if x > high:
        return high
    if x < low:
        return low
    return x

def _integ(img, r, c, rl, cl):
    r = _clip(r, 0, img.shape[0] - 1)
    c = _clip(c, 0, img.shape[1] - 1)

    r2 = _clip(r + rl, 0, img.shape[0] - 1)
    c2 = _clip(c + cl, 0, img.shape[1] - 1)

    assert r >= 0 and c >= 0 and r2 >= 0 and c2 >= 0

    ans = img[r, c] + img[r2, c2] - img[r, c2] - img[r2, c]
    return max(0, ans)


Release 0.0.31.

Time for another release so that we can pull the latest goodness into Cranelift! In particular, we need #100 in order to handle an AArch64 ABI fix.

Published to crates.io.


refactor: 升级到vue2.7.x，并改用setup写法

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Subreddit Search

I created the option to search subreddits, and it's completely working but, The WIP banner stands, because I'm not sure if I made the completely best decisions regarding the API structure, would really appreciate input about this PR, Thanks.
Edit:
The credit about the reddit api documentation goes to here https://snuze.shaunc.com/docs/api-reference/subreddit/.

Thanks for writing this. I'm not too fussed on API design, but it might be worth to create a separate Subreddits struct for that endpoint if it starts getting used more. It would also make it cleaner for when a subreddit post search is added (eventually). So you'd search for subreddits with Subreddits::search("rust") and you'd search posts in a subreddit with Subredddit::new("rust").search("crabs").
And is there a specific reason you need to use the either crate? I think I've looked at it before and decided that just an enum would work fine in most cases, but I could have missed something.
Other than that it looks pretty good. Great job on the documentation of the responses as well.

Whoops sorry mate, didn't realise you made changes to this and I forgot about it. Looks like you did my suggestions though and it works well. Had to change a few things, but it's merged.


Error on launch IOS

Hi, i have a problem with giphy-react-native-sdk in IOS. Build works fine but when launching my app, it stucks on launchscreen and i got log error : "Fatal error: Unexpectedly found nil while unwrapping an Optional value: file giphy_react_native_sdk/RNGiphyDialog.swift, line 60"
Can you help me please ?

Hi @Sondago, thanks for the report!
Should be fixed in version v1.0.2

Thank's you very much, it's work now


##[error]New-PSSession : Unable to load DLL 'vmcompute.dll': The specified module could not be found. (Exception from HRESULT:  0x8007007E)

Hello Kamil,
we ran into the next problem during "Prepare environment". Could you give us some more informationen on how you did setup the VM for the agent? We only edited the suggested settings within the repository.
Please see the attached log file if you're looking for some more information.
logs_2649.zip
Do you have any idea/suggestions on where to look for answers? Looking for the message we found this issue and this issue. Both suggested to reinstall docker and hyper-v, but this did not solve our issue.
Thank you very much

I assume that running x64 version of the agent helped. Because no communication for longer period. I am closing this (even this issue is not related to the template).


Add Statistics Calculator

This calculator takes input as values separated by commas and when the user click on the click button , it will calculate the following:

No. of items
Mean
Median
Mode
Range
Standard Deviation
Draw the frequency distribution table.

@Srijita-Mandal , I would like to work on this issue under Jwoc. Can you assign it to me?

@Laxmankohar Mean. median. mode, range, standard deviation these calculators are already added. You can make frequency distribution table.

ok.

@Laxmankohar okay. go for it.


pot: 0.5.0 -> 1.0.0

Description of changes
https://github.com/pot-app/pot-desktop/releases/tag/1.0.0
https://github.com/pot-app/pot-desktop/compare/0.5.0...1.0.0

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

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

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

[ ] Fits CONTRIBUTING.md.

I forgot to remove them. Fixed.

Thanks!


Add Adabas access method

There is a GVBMR95 read exit already that can be used to access Adabas data during extract processing (GVBXRA).
However, this is a read exit and not an access method, so uses OP/RD/CL calls like other read exits. Also it contains some "customized" information relating to Adabas access which cannot yet be defined through the Workbench.
This suggestion is to provide an access method for Adabas, implemented in a similar way to GVBMRBS/GVBMRSQ and allow the necessary Adabas information to be defined in the Physical File definition through the workbench.

The existing read exit for Adabas works fine but requires the user exit to contain certain Adabas information needed to read the database. For example:
DBID and file number
Returned record length
Adabas search and value buffers (i.e. selection criteria)
Adabas format buffer (i.e. description of fields, formats and lengths returned to the LR).
Optionally (multi-fetch) blocking factor, if not the default 32K / LRECL
It would be more consistent with other types of file access methods to put this logic into a defined SAFR IO module for Adabas, e.g. GVBMRAD and allow the required parameters to be defined through the workbench.
Notes:
The returned record length should match the LR -- provided the Adabas format buffer is correctly defined
The Adabas value buffer may be a mute point as we normally start at the beginning of a file -- but this does allow to start at a particular value with Adabas
The Adabas format buffer is the fields and optionally the formats and lengths used to populate the contents of the returned records.


TOTP_VALID_WINDOW setting

This pull request adds a setting for the valid_window value passed to PyTOTP’s verify method.
In my tests users with a certain authenticator app couldn’t log in unless I changed the code to call verify with valid_window=1 (or higher). I added a setting instead of just changing the value as increasing the value has security implications and not everyone would want to have this change.
Please let me know if I should add a test or some additional documentation.

Thanks for the contribution! The code looks good to me, and I think exposing a lower level parameter sounds reasonable.
But just to be sure: What is causing this issue? Is this a bug in the authenticator app?

The app in question is Google Authenticator for Android. I think it’s rather particular behaviour than a bug. As far as I can tell it only supports generating codes every 30 seconds and seems to expect that a code remains valid for 60 seconds.

The spec discusses this issue:

A larger acceptable delay window would expose a larger window for attacks.  We RECOMMEND that at most one time step is allowed as the network delay.

Yes, you probably shouldn’t set valid_window to anything lager than 1.
The next section of the spec says:

Because of possible clock drifts between a client and a validation server, we RECOMMEND that the validator be set with a specific limit to the number of time steps a prover can be "out of synch" before being rejected.

And it suggests to do automatic resynchronisation, but PyOTP doesn’t support that.

Thanks for the quick feedback. That really made solving this a joy!


readme: add future plans and blog link

What problem does this PR solve?

Issue Number: close none
Problem Summary:
Add future plans and the blog link for reference.
What is changed and how it works:
Add future plans and the blog link for reference.
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[x] No code

Notable changes

[ ] Has configuration change
[ ] Has HTTP API interfaces change
[ ] Has tiproxyctl change
[ ] Other user behavior changes

Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

/retest-required