deprecated build-in function - initCustomEvent

Hello,
I just spotted that the initCustomEvent is marked as deprecated.

MDN docs is suggesting usage of CustomEvent ctor.
https://developer.mozilla.org/en-US/docs/Web/API/CustomEvent/CustomEvent
Thank you very much for this script,
its very useful.
Cheers,
Martin.

sorry, didn't saw this is to make it work on IE/Chrome.
I have closed the issue.


DOC: Complete R interface section

Currently, "rpy2  R interface" documentation has blank section. Added descriptions to cover required basic topics.

let's redirect to rpy2 for conversions, see #9602


Test not recognizing correct answer

Challenge http://www.freecodecamp.com/challenges/-using-typeof has an issue. Please describe how to reproduce it, and include links to screenshots if possible.
After it failed on Safari, I tried Firefox. Didn't work either.
https://www.dropbox.com/s/m5jf7tjcx18hwcy/UsingTypeof.tiff?dl=0

Seems to be resolved


Usage of each with included_id is not working

On branch master.
Trying to use this:
required(:skills).each(included_in?: ['some', 'valid', 'skills'])

Throws this:
/dry-validation-846c4de4998c/lib/dry/validation/schema/value.rb:36:in `block in each': [:included_in?, ["some", "valid", "skills"]] is not a symbol nor a string (TypeError)

This is not specific to :included_in? it affects any predicate with options being used in each.

each predicates are applied to each element of an array, so this should work I think. We probably accidantely coerce options to array with a splat operator somewhere.

@solnic it's because {key: "value"}.to_a returns [[:key, "value"]] so when you splat it to args you get [:key, "value"] and that fails in send which expects the first param to always be a symbol.

and you'll have the same issue with when aswell as it works in the same way


ign -> gz Macro Migration : gz-fuel-tools

See: https://github.com/gazebo-tooling/release-tools/issues/737

@osrf-jenkins run tests please!

No new test failures


luggit/react-native-config does not support with swift , it works fine with objective C

Yes, it does. We only use Swift in our app and we use it.

@gsetiaofb please add more details for we can help you

It would be nice to see some examples of it being used with Swift.

@luancurti readme shows how we can access variables from .env in Obj-C classes. Is there any way to access those variables in swift files or classes?

Please could you kindly show how to use this in a Swift class. I have created a bridge and I would like to read ENV file inside the bridge.
Thanks for your help.

Create the bridging header file in your project if it doesn't exist yet (xcode should prompt to create it when you mixin swift and objective-c code) - see https://developer.apple.com/documentation/swift/imported_c_and_objective-c_apis/importing_objective-c_into_swift, https://stackoverflow.com/a/37295090/6685359
It must be called like YourProduct-Bridging-Header.h
Open this header file and add the line #import "ReactNativeConfig.h"
Use ReactNativeConfig in your swift code:

ReactNativeConfig.env(for: "MY_VARIABLE")


Proposal Created but visably Missing after Creation

I created a new proposal. The transaction went thru smoothly.
https://tzkt.io/ooWTNZJG5xazsNXCrpPBiUAyRW7mjyZ7gv8UjdnnzwazWM3NLgz
On the DAO interface the proposal is not visible. It looks like there is no Proposals been made yet. This was the first proposal.
Also,  The proposal fee when creating was reported as 100,000 SEB, but the DAO was created with 1000.00 SEB Deposit for Proposals. I believe the decimals is incorrect. SEB Has 2 decimals.
Proposal in storage viewable on TZKT.io
https://tzkt.io/KT19CF3KKrvdW77ttFomCuin2k4uAVkryYqh/storage/38283
proposals page console log by sebuh tezos-homebase.io-1638057076334.log
here is the console log from my brave browser while trying to look at the active proposals page.

I have a feeling the issue may have being caused by the Cavedelica Tezotop NFT that was deposited into the treasury, as it has caused an issue with the nft treasury page loading.

Fixed in #243


add picture design type

What are you changing?
Adding the Picture article design type
Why?
So that we can start to render this format in DCR

Updating the type probably warrants a changeset and minor version bump?

Updating the type probably warrants a changeset and minor version bump?

Thanks, wasn't sure how this worked :D fixing

minor version bump?

Just a heads up (for the future since this has already gone out 🔮), I think this would be a major version bump because you're changing the existing public API?

Only because we use the magically awkward TypeScript enums, though, right? If this was a string union I don't think it would have warranted a major bump.

Only because we use the magically awkward TypeScript enums, though, right? If this was a string union I don't think it would have warranted a major bump.

I don't think it matters how you construct the union type (via type or enum); the change that we consider breaking is adding a new member to an exported type? You've widened the type and therefore the consumer's code may need to be adapted to handle the additional case.
For example:
// Library
export type Num = 'One' | 'Two';

// Consumer
import type { Num } from 'library';

const numToNumber = (num: Num): number => {
  switch (num) {
    case 'One':
      return 1;
    case 'Two':
      return 2;
  }
}

If the library changes the type of Num to:
export type Num = 'One' | 'Two' | 'Three';

the consumer gets a compiler error:
Function lacks ending return statement and return type does not include 'undefined'.

because numToNumber doesn't have a case for 'Three' and therefore JS implicitly returns undefined, which is not a number.
This is also the case if we construct Num using enum:
enum Num {
  One,
  Two,
  Three,
}


API-Key is not being recognized

Even when using a correct API-key I get the following error:
"Something went wrong. Please verify that your API Key in settings is correct."
I've tried multiple API keys, and checked for white space characters. Everything is working on my side.

I resolved the issue by using "Project API keys" instead of the default API keys.

Awesome, happy to hear!!


[Bug]

Before Reporting

[X] I found no existing issues matching my bug
[X] My issue is not caused by a plugin or theme

Describe the Bug
"0" is displayed on the top left of discord, inside the titlebar area
Expected Behavior
"0" to not be visible
Steps To Reproduce

Run discord with BD

Operating System
Windows
Discord Version
Stable
BetterDiscord Version
v1.10.1
Anything else?
I have removed a significant portion of the dom node and found it is caused by this style sheet

You are ahead of me, I just checked whether this error is a consequence of using any plugin. And.. no, it appears even without any plugin or theme enabled.

This is already fixed in https://github.com/BetterDiscord/BetterDiscord/pull/1731
Wait for merge.

Someone on Reddit figured out a temporary fix while we wait fro the next merge.
https://www.reddit.com/r/BetterDiscord/comments/1btk1oz/wtf_is_this_annoying_0_thing_on_my_screen/ky0m6n1/

병신다 니가 뭐네 할곳이 없어 남한테 테마를 뺏아 먹으냐 쫄라게 년 이네 악마년!!!****

can the above user be removed from the thread please?

Can you rename the issue so that it's more clear about what's going on? I created a new issue because I couldn't find any similar issues, but then I found your issue. Thanks. @ShadowMonster99

Can you rename the issue so that it's more clear about what's going on? I created a new issue because I couldn't find any similar issues, but then I found your issue. Thanks. @ShadowMonster99

I think it's pretty clear. However i'm open to suggestions!
Cheers!

I don't mean to be rude, but.... the issue name is literally [Bug]

I don't mean to be rude, but.... the issue name is literally [Bug]

Oh haha apologies. i'm viewing this on my phone and read the wrong title. I'll definitely fix it!

Thanks.


Interpreter

Right now there are rough signatures for each helper function we will need to implement. I have documented all function stubs. Here are a few notes:

There is a toplevel _eval function that will call eval_lib for primitive components, or eval_c and eval_s (short for eval control and eval structure, respectively) for user-defined component.
There is a State data structure that can store all of the state of a component- currently only allows registers as primitive state components, so it basically stores all register values in the component. Needs some helper functions
The current big task is evaluating a single enable statement. This will likely require a custom structure graph traversal, and also injecting values from the component's inputs and state into the structure graph.
Evaluation creates a new component state, and I think each evaluation step should only depend on the component's state, inputs, and enabled components.

For testing/debugging I added a small repl that I will use to print out some info at different time steps. The repl can be run on the counter example by setting the backend to interpreter as follows:
cargo run -- tests/integration/counter.futil -l primitives/std.lib -b interpreter
For now, it has very limited functionality.

As it turns out we are not actually storing primitive parameters anywhere in a user-defined component. For now we are hacking this together by adding to the Component struct, but we should revisit this and clean it up more.

Some notes that I ran into:

I have two possible semantics for how to read outputs from a structure graph:

I can either take the final state and apply it to a fresh graph and then read the output ports
I can save the results from the last structure graph that was run and read the outputs from that graph.

Debugging was tricky without visualization, so next I think I will focus on debugging tools to make the interpreter clearer.

cargo run --  tests/interpreter/count3.futil -l primitives/std.lib -b interpreter -i tests/interpreter/count3.json runs the count3 example, and it outputs 3 as expected. I am currently using semantics 1 above for reading outputs from an evaluation graph.

Running the interpreter currently just evaluates a component, but I will try to switch it back to using the interactive REPL soon.

Currently std_add, std_reg, and std_const are the only primitives that are implemented.

Next tasks to take care of:

[ ] Implement the rest of the primitives, and change their signatures on our branch so that they don't have the extra signals. (Vivi)

[ ] Implement remaining control statements. (Vivi + Ken pair programming)

[ ] Refactor Interpreter to be able to run one step at a time- this seems useful for debugging and visualizing the code. (Vivi + Ken)

[ ] Set up better visualization for StructureGraph. This will help significantly in debugging any interpreter issues that pop up, and seems generally useful.

Thanks for all these notes; this is cool! I'd be interested to hear more about that first bullet—I'm not 100% clear on what the difference is between just reading the final state vs. reconstructing a graph.
Debugging tools also certainly sounds very useful!

Subsumed by #263.


Update custom-pouchdb.d.ts

Typo:  Line 10 - "F ields".
Fixes : Removed the space.

 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.


[Q]Anti selection function of glob, support ?

Is there a document on what features glob supports

Now look at the documentation and try
I only know how to use ’{test1,test2}‘

Now look at the documentation and try
I only know how to use ’{test1,test2}‘

Hi @zzzzzzzzlllllllll
Current documentation about glob format located in description of target parameter of render API - https://graphite.readthedocs.io/en/stable/render_api.html#paths-and-wildcards

Hi @zzzzzzzzlllllllll
Current documentation about glob format located in description of target parameter of render API - https://graphite.readthedocs.io/en/stable/render_api.html#paths-and-wildcards


ValueError: loaded state dict has a different number of parameter groups

I followed the scripts you gave and pre-trained the model on imagenet22k first, then finetuned on imagenet1k. When evaluating the saved checkpoints, it reported the following error "ValueError: loaded state dict has a different number of parameter groups". I would like to know how to fix it?
My pretrain script and finetune script follow the official documentation, and the evaluation script is as follows,
python main.py \ --model convnext_small \ --resume finetune_convfastSmall_Imagenet1k_224_outputs/checkpoint-best.pth \ --input_size 224 \ --data_path /mnt/datasets/general_datasets/ImageNet \ --eval true

I thought this is because the condition if 'optimizer' in checkpoint and 'epoch' in checkpoint (utils.py 494 line) been met. The code below this conditon was unnecessary for the evaluation process. So I comment it out when doing evaluation. This is not a elegant way to sove this problem, but it could work.
        # if 'optimizer' in checkpoint and 'epoch' in checkpoint:
        #     optimizer.load_state_dict(checkpoint['optimizer'])
        #     if not isinstance(checkpoint['epoch'], str): # does not support resuming with 'best', 'best-ema'
        #         args.start_epoch = checkpoint['epoch'] + 1
        #     else:
        #         assert args.eval, 'Does not support resuming with checkpoint-best'
        #     if hasattr(args, 'model_ema') and args.model_ema:
        #         if 'model_ema' in checkpoint.keys():
        #             model_ema.ema.load_state_dict(checkpoint['model_ema'])
        #         else:
        #             model_ema.ema.load_state_dict(checkpoint['model'])
        #     if 'scaler' in checkpoint:
        #         loss_scaler.load_state_dict(checkpoint['scaler'])
        #     print("With optim & sched!")

This might be some bug related to Does not support resuming with checkpoint-best


Atom Beta gets stuck in auto-update loop.

Description
Atom 1.9.0-beta1 gets stuck in a restart-to-install loop. I know 1.9b1 is not the latest but considering the nature of this bug, I suppose experiencing it on the latest build would be impossible.
Steps to Reproduce

Open Atom menu
Click 'Restart to install update'
Atom restarts
The Atom menu shows that the same update is downloading again.

Expected behavior: Atom should install the update it downloads and see that there are no newer versions.
Actual behavior: In addition to the steps above, this behavior repeats even after downloading the beta disk image from the atom website.
Versions
OS X: 10.11.6 (15G31)

$ atom-beta -v
Atom    : 1.9.0-beta1
Electron: 0.37.8
Chrome  : 49.0.2623.75
Node    : 5.10.0

$ apm-beta -v
apm : 1.10.0
npm:  2.13.3
node: 0.10.40
python: 2.7.11
git: 2.8.1

I experienced this on OS X 10.7, but since it's a work computer where I don't have administrative rights, it's the behaviour I'd expect. However, it would be nice if Atom would request administrative rights before downloading anything – or at least store previously downloaded updates. Now, every time I check whether there's an update, Atom starts downloading it, unaware of the fact that I can't install it.

I believe this might be because Atom 1.9.0-beta1 is not currently published (perhaps it might have been, then got reverted to Draft state).
@idleberg you are experiencing a different bug: #2860.

As pointed out by @50Wliu, the original report is caused by some kind of hiccup/mistake/test that ended up publishing Atom 1.9.0-beta1, which has since been fixed/reverted. The problem can be resolved by reinstalling 1.9.0-beta0.


[BUG] Oak-D-PoE on Mac and LAN, is visible, but produces [X_LINK_BOOTLOADER] error

Check if issue already exists
I checked and nothing looks similar enough to my circumstances.
Describe the bug
I got a new Oak-D-PoE and I wanted to try to connect to it with the demo code. In the past I have managed to get the OAK-D and OAK working on Mac, Windows, and Linux over USB however get this error when trying to do the same over Ethernet. I first tried to connect directly to the PC and I managed to ping it, but wasnt able to find it automatically, but then I tried to do it over LAN and It discovers it, but produces this error.
(luxonis) ppurnyn@JP-KIT-M-232 depthai % python3 depthai_demo.py  Using depthai module from:  /Users/ppurnyn/Library/Python/3.9/lib/python/site-packages/depthai.cpython-39-darwin.so Depthai version installed:  <IP_ADDRESS> Setting up demo... Available devices: [0] <IP_ADDRESS> [X_LINK_BOOTLOADER] Stack trace (most recent call last) in thread<PHONE_NUMBER>2: #11   Object "libsystem_pthread.dylib", at 0x7ff811701513, in _pthread_start + 124 #10   Object "QtCore", at 0x163e556a7, in QTimerInfoList::activateTimers() + 5463 #9    Object "QtQuick", at 0x16a114b27, in QNativeInterface::QSGMetalTexture::fromNative(id<MTLTexture>, QQuickWindow*, QSize const&, QFlags<id<MTLTexture>::CreateTextureOption>) + 28471 #8    Object "QtCore", at 0x163e3c989, in QMacAutoReleasePool::~QMacAutoReleasePool() + 41 #7    Object "Foundation                          0x00007ff812528057", at 0x7ff812528057, in drain] + 132 #6    Object "CoreFoundation", at 0x7ff8117895df, in _CFAutoreleasePoolPop + 21 #5    Object "libobjc.A.dylib", at 0x7ff81159939f, in objc_autoreleasePoolPop + 160 #4    Object "libobjc.A.dylib", at 0x7ff81159c41e, in AutoreleasePoolPage::releaseUntil(objc_object**) + 166 #3    Object "libobjc.A.dylib", at 0x7ff811597e80, in objc_release + 32 #2    Object "libsystem_platform.dylib", at 0x7ff811716e2c, in _sigtramp + 28 #1    Object "depthai.cpython-39-darwin.so", at 0x11b04b5fd, in backward::SignalHandling::sig_handler(int, __siginfo*, void*) + 13 #0    Object "depthai.cpython-39-darwin.so", at 0x11b04b66c, in backward::SignalHandling::sig_handler(int, __siginfo*, void*) + 124 zsh: segmentation fault  python3 depthai_demo.py (luxonis) ppurnyn@JP-KIT-M-232 depthai % python depthai_demo.py    File "depthai_demo.py", line 42     Trackbars.instances[name] = {**Trackbars.instances.get(name, {}), window: defaultVal}                                   ^ SyntaxError: invalid syntax
To Reproduce
Steps to reproduce the behavior:

Run script python3 depthai_demo.py on Mac M1
See error

Expected behavior
demo connects and produces no error

@mpurnyn I just tried running the demo with an OAK-D-PoE on a Mac mini M1 with Big Sur 11.1, Python3.9, with the first command you tried python3 depthai_demo.py and it works fine.
(For the second command, python seems to point to python2 issuing that invalid syntax error.)
I'm installing in parallel Monterey 12.0.1 to test there as well.
The error seems related to our recent Qt6 based GUI addition to the demo. Could you try with this command instead:
QT_QUICK_BACKEND=software python3 depthai_demo.py
If the issue persists, just for test, try downgrading to an older version, before the GUI addition:
git fetch
git checkout old_interface_before_gui

Also, was the dependencies installation on the M1 done using the command mentioned here?
https://docs.luxonis.com/projects/api/en/latest/install/?#macos

Thank you for your quick response.

I did miss the dependencies for m1 from your link, I ran the curl command and installed. It did end up erroring out
`HEAD is now at 3ab140e39 Merge pull request #12470 from Homebrew/dependabot/bundler/Library/Homebrew/sorbet-0.5.9368
==> Tapping homebrew/core
remote: Enumerating objects: 1097944, done.
remote: Counting objects: 100% (47/47), done.
remote: Compressing objects: 100% (29/29), done.
remote: Total 1097944 (delta 27), reused 34 (delta 18), pack-reused 1097897
Receiving objects: 100% (1097944/1097944), 419.61 MiB | 9.04 MiB/s, done.
Resolving deltas: 100% (756237/756237), done.
From https://github.com/Homebrew/homebrew-core

[new branch]              master     -> origin/master
HEAD is now at 5c075f4a626 php: remove glib dependency (#89902)
Error: Failed to link all completions, docs and manpages:
Permission denied @ rb_file_s_symlink - (../../../Homebrew/completions/zsh/_brew, /usr/local/share/zsh/site-functions/_brew)
Failed during: /usr/local/bin/brew update --force --quiet`

I tried to run with the QT parameter provided and it worked after loading the camera.

I then tried to downgrade to the older version and that worked as well.

My Mac is currently running 12.0.1

I guess I should still be able to connect and use the camera, even if QT and the demo do not work, right?

Yes, QT is actually used just with this demo. All other examples should work:
https://github.com/luxonis/depthai-python/tree/main/examples
Good to know that applying the QT_QUICK_BACKEND=software environment variables resolves the problem.
Actually we saw a similar issue on Windows and applied it directly inside the demo: https://github.com/luxonis/depthai/pull/532/files
The easy workaround for us would be to do the same for macOS, but since the above might slightly affect the performance (e.g. increase CPU load on host), it's better to look for a proper solution. I'm checking on a freshly installed Monterey as well now.
About downgrading, thanks for the test as well. Note: don't forget to check out back the main branch for the latest.
Will check as well installing the dependencies. It's possible it failed for you because Homebrew was already installed.

Ran bash -c "$(curl -fL http://docs.luxonis.com/_static/install_dependencies.sh)"
on a freshly installed Monterey 12.0.1 on a Mac mini M1, then cloned this repo, ran install_requirements.py and depthai_demo.py and it worked all good without setting the environment variable (main branch, not on the PR mentioned above).
So it could be that it depends on whether other software packages are (or are not) installed in the system.
Anyway, I checked the CPU load with top, and didn't see a noticeable difference when having the variable set or not. So we'll add a workaround for now: https://github.com/luxonis/depthai/pull/544

Thanks for putting in the fix. I managed to get another error when going back to main though. It does not appear when using the before-gui tag though. I went back to the older tag for now.
env-depthai) ppurnyn@JP-KIT-M-232 depthai % QT_QUICK_BACKEND=software python3 depthai_demo.py -cnn person-detection-retail-0013 Using depthai module from:  /Users/ppurnyn/workspace/kws-vision/kpmgjp-kit-lab-kws-vision/env-depthai/lib/python3.9/site-packages/depthai.cpython-39-darwin.so Depthai version installed:  <IP_ADDRESS> Setting up demo... Available devices: [0] <IP_ADDRESS> [X_LINK_BOOTLOADER]   File "/Users/ppurnyn/workspace/kws-vision/kpmgjp-kit-lab-kws-vision/depthai/depthai_demo.py", line 549, in run     self.instance.run_all(self.conf)   File "/Users/ppurnyn/workspace/kws-vision/kpmgjp-kit-lab-kws-vision/depthai/depthai_demo.py", line 60, in run_all     self.setup(conf)   File "/Users/ppurnyn/workspace/kws-vision/kpmgjp-kit-lab-kws-vision/depthai/depthai_demo.py", line 135, in setup     self.metrics.reportDevice(self._device)   File "/Users/ppurnyn/workspace/kws-vision/kpmgjp-kit-lab-kws-vision/depthai/depthai_helpers/metrics.py", line 53, in reportDevice     self.demo_table.update({   File "/Users/ppurnyn/workspace/kws-vision/kpmgjp-kit-lab-kws-vision/env-depthai/lib/python3.9/site-packages/pyrebase/pyrebase.py", line 309, in update     raise_detailed_error(request_object)   File "/Users/ppurnyn/workspace/kws-vision/kpmgjp-kit-lab-kws-vision/env-depthai/lib/python3.9/site-packages/pyrebase/pyrebase.py", line 448, in raise_detailed_error     raise HTTPError(e, request_object.text) [Errno 400 Client Error: Bad Request for url: https://depthai-data-default-rtdb.firebaseio.com/demo.json] {   "error" : "Invalid data; couldn't parse JSON object. Are you sending a JSON object with valid key names?" }

(reopening automatically closed issue)
@mpurnyn Thanks for testing, I merged the above PR to main, so manually prepending QT_QUICK_BACKEND=software should no longer be needed.
And sorry about the new issue, seems it's introduced by https://github.com/luxonis/depthai/pull/531 (merged yesterday). I get the same failure with an OAK-D-PoE, but works with OAK-D (over USB).
We'll work on fixing it.

I also edited your comment above to enclose the log snippet within triple backticks ``` multi-line snippet ```, as apparently it's not displayed properly by single backticks, line terminators were ignored.

@mpurnyn Should be fixed by https://github.com/luxonis/depthai/pull/548 (merged to main).


Add async Function run support

Why
To support asynchronous Function run functionality

Need to check what this means for users who don't have async Function runs enabled. If async is set to False (default behaviour) will that throw an error?

https://linear.app/evervault/issue/ETR-1609/add-async-call-to-python-sdk


Denote cancelled workshops in AMY

Add tag for cancelled workshops.
This should also result in the workshop being unpublished.
(So published == has date and has location and has workshop page and  tag != "cancelled")

Would it be useful to have reasons (for stats and further analysis):

cancelled due to lack of funds
cancelled due to problems in finding instructors
cancelled due to venue problems [not sure if that is interesting for us, I think the first two are]
other reasons

@pbanaszkiewicz
Since we also have to record the reason behind the cancellation, we can add a CharField with a choice that lists the possible reason to the Event model.
Is this schema change good enough?

@apawlik I think the potential reasons for canceling a workshop are too complex to enumerate.
Also we are thinking of cancelled meaning that a workshop was completely planned - i.e., had instructors, a venue, registered learners, etc. and had to cancel.
If we start talking to a host about doing a workshop and they can't do it because of lack of funds, don't have a venue, etc.  then it is considered "stalled."

@maneesha Yes, I agree that to enumerate all reasons is not possible. I was just wondering (based on the past experience) whether capturing how often it was due to lack of instructor vs lack of funds was interesting for us to note any patterns. Though if we define cancelled as all fully planned, then this is not applicable.
I think this may be a question to the Steering Committees whether we want to capture any stats on how many workshops become stalled due to lack of funds/troubles in finding instructors. The latter used to be a headache in the UK (more than the former) and knowing that was informative for SSI (but whether this is for the SCs now, it's a different question).


Build script fails on errors from Pester and Update-Help

The build script is outdated which presents the following problems:

The script install the latest version of Pester (5.x) but the Pester tests are written for v4.x syntax.
The script runs Update-Help for all installed help. However, there are known issues with help for certain modules. Update-Help throws errors for those modules which causes the build script to exit.

To fix this:

Change the script to install  Pester 4.10.1
Change the Update-Help command to only update help for the modules needed by the Pester tests.

Also, the Pester tests need to be rewritten. See #498 for more detail.

This issue is fixed in #497.

This issue is fixed in #497.

No longer applies with Microsoft.PowerShell.PlatyPS v1.0


EXTRACT should be KSM_EXTRACT in README

Hello,
I found that in the Example and Extracting ACLs, EXTRACT should actually be KSM_EXTRACT according to the configuration setting below.
Reference:
https://github.com/simplesteph/kafka-security-manager/blob/d3a9a60219996e2b70d8a78424b41b8947a0784b/src/main/resources/application.conf#L7

fixed by #48


Modal window "Do you want to save a password for all VMs?" appears even if "No" is set in Settings

Steps to reproduce

Set "Save VM passwords by default" = false in settings
Create new VM (with password) and press Save password

Actual result

A modal window appears and it appears every time when "Save VM passwords by default" - disabled
Expected result

A modal window should appear only once until the system doesn't know user preference (For example new user). If  "Save VM passwords by default" was enabled and you disable it then modal windows "Do you want to save a password for all VMs?" should not appear anymore.
For the new user, there is no difference in the "Save VM passwords by default" checkbox between the state is not set and turned off.
Environment

Browser:
- [ ] Chrome version XX
- [ ] Firefox version XX
- [ ] Safari version XX
- [ ] Opera version XX
- [ ] IE version XX
- [ ] Edge version XX

Others:

Connected feature ID

settings_password_saveall
vm_password_save
Screenshots

TBD in #1014

Please, test this issue on master

Tested on Master

This error wasn't fixed.

Actual result: A modal window appears every time when parameter "Save VM password by default" set as disabled. It does not matter whether the user is new or old.
Expected result: Modal window appears only one time when a new user creates first VM and parameter "Save VM password by default" set as disabled.

The parameter "Save VM password by default" not set to "Enabled" state when confirming the modal window "Save all passwords" after creating VM.

Steps:

Set "Save VM passwords by default" = false in settings
Create new VM (with password) and press Save password
Confirm the modal window

Actual result: Parameter "Save VM password by default" set as disabled in settings
Expected result: Parameter "Save VM password by default" set as enabled in settings

Tested on Master


Animation end listener is missing in the library

It will be great if you add an Animation listener to be a parameter to the Moving Params

Hi. Thank you for contributing. I'll check this possibility and let you know

Fixed in version 1.1.2
Please check


Save good graph positions

We might want to save the positioned graph in a file, preferably when we reach a low minimum (of the physical potential). This can speed up the generation of often-used groups, e. G. gallery groups and make it less random (for now, still sometimes the toroidal groups are entangled with themselves)

Since we only need to save an abstract graph and the positions of the vertices this wouldn't be too much data. It's definitely worth thinking about as soon as other projects are finished.
It's also a task which could theoretically be automated.


usePolling is required by vite watcher, otherwise the server keeps crashing

usePolling is a workaround. Adding this ticket to track the issue and hopefully find + fix the root cause in future.
See here for more context: https://github.com/svelteness/kit-docs/pull/71

I cannot reproduce this as of today. FWIW, this is the most relevant comment.
I changed .../src/routes/docs/[...1]getting-started/[...2]quickstart/+page.md and I only see hmr updates in log:
[vite] hmr update /src/routes/docs/[...1]getting-started/[...2]quickstart/+page.md.
Can take a look if I can reproduce it.


Add ofType property to connection and edge types.

Like GraphQLNonNull and GraphQLList types, Connection type also should have this useful property for getting its internal type. Espessialy via method getNamedType
I'll open cross PR to GraphQL about relaxing getNamedType method for accepting any object with ofType property.
If @leeb found this legal and accept his PR, then you may accept this PR.

Lee founded this changes not legal. And I agree with him.
Anyway, this hack landed in graphql-compose-connection, cause all inner type manipulations and inspections served by graphql-compose and will not break 3rd party tools.
I'm using ofType in connections for proper determining projection from AST, when constructing queries to databases.


Add new countries with different variant types of license plates

How to add new license plates for different cities or states for a specific country?

I've found some config here for other countries, which you can load into the openalpr.conf file in the runtime_data folder
https://github.com/openalpr/openalpr/tree/master/runtime_data/config

Yes I did noticed that too, but is there some documentation that explains each line on the config or some kind of example that someone can use to add custom config for a country that does not exist in the "runtime_data" folder.

you can add a new country , in this link http://doc.openalpr.com/opensource.html#adding-a-new-country you will find how to do it and then you can add the config files after the training to the application.


why not cast dtype in deepspeed zero3 ?

Reminder

[X] I have read the README and searched the existing issues.

System Info

llamafactory version: 0.8.1.dev0
Platform: Linux-5.4.0-125-generic-x86_64-with-glibc2.29
Python version: 3.8.10
PyTorch version: 2.3.0+cu121 (GPU)
Transformers version: 4.41.2
Datasets version: 2.19.1
Accelerate version: 0.30.1
PEFT version: 0.11.1
TRL version: 0.8.6
GPU type: NVIDIA A800-SXM4-80GB
DeepSpeed version: 0.14.0

Reproduction
SFT training with deepspeed zero3,  bf16 and flash attention 2 result in the following warning:
Flash Attention 2.0 only supports torch.float16 and torch.bfloat16 dtypes, but the current dype in LlamaForCausalLM is torch.float32. You should run training or inference using Automatic Mixed-Precision via the `with torch.autocast(device_type='torch_device'):` decorator, or load the model with the `torch_dtype` argument. Example: `model = AutoModel.from_pretrained("openai/whisper-tiny", attn_implementation="flash_attention_2", torch_dtype=torch.float16)`
Flash Attention 2.0 only supports torch.float16 and torch.bfloat16 dtypes, but the current dype in LlamaModel is torch.float32. You should run training or inference using Automatic Mixed-Precision via the `with torch.autocast(device_type='torch_device'):` decorator, or load the model with the `torch_dtype` argument. Example: `model = AutoModel.from_pretrained("openai/whisper-tiny", attn_implementation="flash_attention_2", torch_dtype=torch.float16)`

I found out that torch_dtype  is not assigned to init_kwargs in zero3 from the following code. So the AutoModel.from_pretrained got the default data_type which is  torch.float32.
https://github.com/hiyouga/LLaMA-Factory/blob/577de2fa078ee62d61bca24b75fa59590e2ef66b/src/llamafactory/model/patcher.py#L75-L76
so why not cast dtype in deepspeed zoero3 ?
Expected behavior
use bf16 to train the model
Others
No response

what are your training commends? did you use --bf16 at training?

@hiyouga yes, --bf16
### model
model_name_or_path: ...
flash_attn: fa2
cache_dir:...

### method
stage: sft
do_train: true
finetuning_type: full

### ddp
ddp_timeout: 180000000
deepspeed: 32b_ds.config.json

### dataset
dataset: sft_pro_total,glaive_toolcall_1k
template: qwen
cutoff_len: 8192
preprocessing_num_workers: 16

### output
output_dir: ...
logging_steps: 10
save_strategy: epoch
save_steps: 1000
plot_loss: true

### train
per_device_train_batch_size: 1
gradient_accumulation_steps: 8
learning_rate: 4.0e-6
num_train_epochs: 2.0
lr_scheduler_type: cosine
warmup_ratio: 0.1
bf16: true

### eval

and the kwargs passed to AutoModel.from_pretrained() is
{'trust_remote_code': True, 'cache_dir': '...', 'revision': 'main', 'token': None, 'low_cpu_mem_usage': False, 'config': Qwen2Config {
  "_name_or_path": "...",
  "architectures": [
    "Qwen2ForCausalLM"
  ],
  "attention_dropout": 0.0,
  "bos_token_id": 151643,
  "eos_token_id": 151645,
  "hidden_act": "silu",
  "hidden_size": 3584,
  "initializer_range": 0.02,
  "intermediate_size": 18944,
  "max_position_embeddings": 32768,
  "max_window_layers": 28,
  "model_type": "qwen2",
  "num_attention_heads": 28,
  "num_hidden_layers": 28,
  "num_key_value_heads": 4,
  "rms_norm_eps": 1e-06,
  "rope_theta": 1000000.0,
  "sliding_window": 131072,
  "tie_word_embeddings": false,
  "torch_dtype": "bfloat16",
  "transformers_version": "4.41.2",
  "use_cache": false,
  "use_sliding_window": false,
  "vocab_size": 152064
}
, 'pretrained_model_name_or_path': '...'}

Same problem in training Qwen2 with zero3. Enabling either fp16 or bf16 doesn't work.

Flash Attention 2.0 only supports torch.float16 and torch.bfloat16 dtypes, but the current dype in LlamaForCausalLM is torch.float32. You should run training or inference using Automatic Mixed-Precision via the with torch.autocast(device_type='torch_device'): decorator, or load the model with the torch_dtype argument. Example: model = AutoModel.from_pretrained("openai/whisper-tiny", attn_implementation="flash_attention_2", torch_dtype=torch.float16)

We actually used torch.autocast in bf16/fp16 training, so this warning message can be ignored. And we have validated that the training can start without errors.

Flash Attention 2.0 only supports torch.float16 and torch.bfloat16 dtypes, but the current dype in LlamaForCausalLM is torch.float32. You should run training or inference using Automatic Mixed-Precision via the with torch.autocast(device_type='torch_device'): decorator, or load the model with the torch_dtype argument. Example: model = AutoModel.from_pretrained("openai/whisper-tiny", attn_implementation="flash_attention_2", torch_dtype=torch.float16)

We actually used torch.autocast in bf16/fp16 training, so this warning message can be ignored. And we have validated that the training can start without errors.

@hiyouga thanks for your clarification. Can we assign init_kwargs["torch_dtype"] = model_args.compute_dtype before
https://github.com/hiyouga/LLaMA-Factory/blob/577de2fa078ee62d61bca24b75fa59590e2ef66b/src/llamafactory/model/patcher.py#L75-L76
like
init_kwargs["torch_dtype"] = model_args.compute_dtype
if not is_deepspeed_zero3_enabled() and not is_fsdp_enabled():  # cast dtype and device if not use zero3 or fsdp

I'v tested and it can fix the flash attention 2 warning.

@Huarong I do not recommend to do this


Disable text selection issue #71

Additions:

Disable text selection on the .logo h1 which is a child of .navbar and .content
Files modified: index.html and assets/css/index.css
Tested in the following browsers: Chrome 85, Firefox 80, Opera 70, Edge 85

Issue:
Relates to issue #71, specs:
Add a rule to .logo h1 to disable user selection. Make sure to add the changes to assets/css/index.css

Great thank you!
@all-contributors please add @CodeMode1 for code


Custom Tabular Exporter: add option to quote all strings #1869

@wetneb sure, will however do that on a separate branch as this as you mentions does not modify the backend.
Picking issues that affect me as an user so tomorrow I will aim to do a WD import with OpenRefine to then being able to take on additional issues.


Find a way to keep toolkit and compilecheck in sync

Ideally all of the discovery files we have in testdata should always pass compilecheck. We should find a way to enforce this.

@pongad Is this Discovery-specific?

@bjwatson It is. Added the label. In this case, "compilecheck" refers to the tool we wrote to make sure discovery samples compile.

Ok, thanks.

This is a duplicate of other bugs... and is being implemented


improve: Optimize RoaringBitSet.get(int fromIndex, int toIndex)

SUMMARY

RoaringBitSet was added in 1.1.0 release, PR #725
The get(int fromIndex, int toIndex) method implementation wasn't optimal

It would first make an entire java.util.BitSet instance and then use that for getting the subset

This PR uses RoaringBitmap's methods for the implementation of get(int fromIndex, int toIndex) instead of relying on java.util.BitSet for the implementation.

Automated Checks

[x] I have run ./gradlew test and made sure that my PR does not break any unit test.
[x] I have run ./gradlew checkstyleMain or the equivalent and corrected the formatting warnings reported.

I'll merge this but revisit it. I think we can make it better.


some Icons gets out of its box constraints

Image for the issue

I tried to use flutter_svg package https://pub.dev/packages/flutter_svg and it displayed the same icon well, so i think this is an issue from the flutter_icon website

I have the same problem.
I've been doing some research and it seems like the problem is that the font generated is not completely squared so when Flutter renders the icon (the generated font), the icon overflows, and the overflow is shown by Flutter. You can try this if you add textDirection: TextDirection.rtl in your icon and it will overflow the other way.
The only way I see we could fix this is by making all the characters generated in the font completely squared, so the icons never overflow, being completely centered.

@ElZombieIsra Thank you for the workaround class that is great! However a fix for this would still be very nice from the code owners.


http: fix http/1 client low watermark crash

This is a regression from https://github.com/envoyproxy/envoy/issues/10566.
Fixes https://github.com/envoyproxy/envoy/issues/10655
Risk Level: Low (for this fix, the original change was/is still medium)
Testing: New UT
Docs Changes: N/A
Release Notes: N/A

cc @euroelessar @rgs1

@mattklein123 thanks! I'll give it a spin with real traffic tomorrow (but if you want to merge before, that's good too).

@htuch @asraa I've re-run TSAN 3 times and it keeps failing on:
[2020-04-06 02:23:47.923][156][critical][assert] [source/common/event/file_event_impl.cc:65] assert failure: events.
[2020-04-06 02:23:47.923][156][critical][backtrace] [bazel-out/k8-dbg/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:104] Caught Aborted, suspect faulting address 0xfffe00000011
[2020-04-06 02:23:47.923][156][critical][backtrace] [bazel-out/k8-dbg/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:91] Backtrace (use tools/stack_decode.py to get line numbers):
[2020-04-06 02:23:47.923][156][critical][backtrace] [bazel-out/k8-dbg/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:92] Envoy version: 0/1.14.0-dev/redacted/DEBUG/BoringSSL
[2020-04-06 02:23:47.943][150][info][main] [source/server/server.cc:613] shutting down server instance
[2020-04-06 02:23:47.946][150][info][main] [source/server/server.cc:560] main dispatch loop exited
[2020-04-06 02:23:48.031][156][critical][backtrace] [bazel-out/k8-dbg/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:96] #0: Envoy::SignalAction::sigHandler() [0x5a8e817]
[2020-04-06 02:23:48.132][156][critical][backtrace] [bazel-out/k8-dbg/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:96] #1: __tsan::CallUserSignalHandler() [0x2684350]
external/bazel_tools/tools/test/test-setup.sh: line 310:    17 Aborted                 (core dumped) "${TEST_PATH}" "$@" 2>&1
================================================================================
FAIL: //test/integration:h1_capture_fuzz_test (see /build/tmp/_bazel_bazel/b570b5ccd0454dc9af9f65ab1833764d/execroot/envoy/bazel-out/k8-dbg/testlogs/test/integration/h1_capture_fuzz_test/test.log)
INFO: From Testing //test/integration:h1_capture_fuzz_test:

I tried running this locally in TSAN mode and it passed. I'm at a loss for how my change would effect this test in this way either way. Any ideas here?

Nevermind I did get this to reproduce locally with a non-RBE build. Looking.

Hmm, I think I'm hitting https://github.com/libevent/libevent/issues/984 and https://github.com/envoyproxy/envoy/issues/8199, but I don't understand how my patch would effect this. Also, it seems to fail sporadically so maybe tickling a timing issue. @htuch any suggestions?

You could try flake instructions before and after and see if it's statistically worse?

@alyssawilk updated w/ merged flake fix.


perl: update patches so that they can be applied to perl 5.22.3

Motivation for this change
Perl patches cpp-precomp.patch and no-sys-dirs.patch cannot be applied to perl-5.22.3 source and was giving HUNKS. This commit fixes those patches so that they can be applied cleanly.
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

No idea what the cpp patch is for, but the other patch is Nix specific in that it helps enforce purity (prevent picking up references to/using stuff outside the Nix store).


Install the vgpu driver in the ubuntu virtual machine

Hello everyone，is there a way to disguise the device id in the linux virtual machine and install the corresponding driver?

I shared some of my linux guest findings here in another issue
After changing the PCI IDs -- the Linux guest works great until the official driver goes into limp mode (triggered at 20 minutes uptime and slows the freq and sets a 15 FPS cap). The linux driver seemingly recognizes the host driver is vgpu/grid and the guest is unlicensed --- even when changing the device id .
I want to try rolling back some drivers but havn't yet done so.


Add Rounded Borders to Card Components

I would like to enhance the visual appeal of the profile card by implementing rounded borders. This change will create a softer look and improve the overall user interface.

If you feel there is a need for the above addition, then please assign me this issue

#117


report

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. Linux]
IntelliJ Version: [e.g. PHPStorm 2022.3]
Plugin Version [e.g. 1.9.1]

Additional context
Add any other context about the problem here.

What am I supposed to do with this?


feat: add 'default_branch' attribute

resolves #2029

welcome! how soon could a release be cut? i'd like to get this into the terraform provider.

Just released v0.111.0


Footstep sounds are improperly replaced

Need to sound.Add on both client and server.

Should be fixed by 996801eb9bf8d6259a0fec6027a4357f234e5efb.


new policy issue in the  project

when new policy in the project, we checked new destination checkbox to create a new destination. we find that we can create the same endpoint by different user and destination name,  but it should't be allowed.
@wknet123

You could create duplicate endpoints under same project or different?

we test situation is under the same project

@ywk253100 should we check it on API?

I will check it.


Leonxu

Created a linear model on price with technical data with respect to one single stock using pandas.

Interesting - i think the reason why R-squared is 0.99 is because you're predicting adjusted_close from the technical indicators from the same day. Our model should predict the adjusted close of tomorrow given the technical indicators of today.


Print Specific Columns. How can I get $excludeFromPrint=[] array value dynamically from colvis. I am using InfyOm generator and Yajra Datatable

Summary of problem or feature request
Print Specific Columns. How can I get $excludeFromPrint=[] array value dynamically from colvis. I am using InfyOm generator and Yajra Datatable

Code snippet of problem
protected $excludeFromPrint = [];

protected function getPrintColumnsFromBuilder()
{
    return $this->html()->removeColumn(...$this->excludeFromPrint)->getColumns();
}

System details

Operating System
PHP Version
Laravel Version
Laravel-DataTables Version

Quick answer, not supported as the colvis button does not support server-side implementation afaik.
However, I think you can create a custom button for this. See https://github.com/yajra/laravel-datatables-buttons/blob/master/src/resources/assets/buttons.server-side.js#L160 for reference.


Update clap version and Allow hyphen values for --nth and --with-nth options

Run sk --delimiter ':' --nth -1.. is not vaild, issue: https://github.com/ibhagwan/fzf-lua/issues/1085

Closed in favor of #586


Refs #23097 -- Used new octal format in FILE_UPLOAD_PERMISSIONS docs.

0644 is a syntax error on Python 3. Octal numbers must be specified with the 0o prefix.

@rfleschenberg Thanks :+1:


added hlist_to_list and hlist_map_to_list

Hi Gregory,
What do you think of these two functions? Is this already easily achieved somehow?

They both seem like reasonable functions. Do you think they are useful
enough to include in the library? And should they be (or are they already)
universe polymorphic?
On Thu, May 4, 2017, 7:16 PM Valentin Robert<EMAIL_ADDRESS>wrote:

Hi Gregory,
What do you think of these two functions? Is this already easily achieved
somehow?
You can view, comment on, or merge this pull request online at:
https://github.com/coq-ext-lib/coq-ext-lib/pull/37
Commit Summary

added hlist_to_list and hlist_map_to_list

File Changes

M theories/Data/HList.v
https://github.com/coq-ext-lib/coq-ext-lib/pull/37/files#diff-0 (26)

Patch Links:

https://github.com/coq-ext-lib/coq-ext-lib/pull/37.patch
https://github.com/coq-ext-lib/coq-ext-lib/pull/37.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/coq-ext-lib/coq-ext-lib/pull/37, or mute the thread
https://github.com/notifications/unsubscribe-auth/AASlV2lF6nB3sF0y4LZbO1-t9HoUF4hKks5r2lxKgaJpZM4NRVDW
.

--

gregory malecha
gmalecha.github.io

They are useful when you want to compute some non-dependent value for each element of the heterogeneous list, and use it as a list.
They are not universe-polymorphic and this is creating issues already for me, let me try to fix that and update the PR...


(PUP-6847) Omit whitespace from provider keyword in network_device

Prior to this update, any trailing whitespace left in a device
configuration file will cause puppet device to fail with
a cryptic message because it is using the string to construct
the path for an require statement.  Since require statements
should never have spaces in them, it makes more sense to have the
regex only accept the non-whitespace characters it finds in the
confiugration file.

@abottchen Thanks! Two things:

Puppet tickets needs to start with PUP. You should be able to just move the PA ticket into the PUP project and edit your commit message
I'm personally not a fan of how many really complicated regular expressions we have in Puppet. I think this would be clearer if you leave the regex as-is and strip() it after the fact. Others might disagree with me, so this is really up to you if you want to change it ¯_(ツ)_/¯

Thanks for the PR looks good to me. Re-architecting everything would be great, but this is a necessary fix and a great catch.  👍


feat: Added bulgarian (bg) translations

Description

Related Issues

Checklist:

[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you linted your code locally prior to submission?
[x] Have you successfully ran app with your changes locally?

Thank you for your translation : )

@agalwood Fixed bugs

Sorry, I found one less during the review.

src/shared/locales/index.js

@agalwood
lines 73-76
  {
    value: 'bg',
    label: 'Българският език'
  }

src/shared/locales/index.js
...
{
  value: 'bg',
  label: 'Българският език'
},
{
  value: 'ca',
  label: 'Català'
},
...

I found a missing translation:
https://github.com/agalwood/Motrix/blob/0223e691ff49deb6d183b40ef2edbad03322a119/src/shared/locales/en-US/preferences.js#L23

@agalwood check please


Update ComponentTableSeeder.php

Tags were removed from the component table.

:beers:

I have a question about the usage of this platform, say I want to monitor the status of a certain API or URL how do I get started with that? I see you can add a component with a link for example, but I want the status of that API link depending on what HTTP response that link says (e.g. 200 = ok still running, 500 = oops, not ok).

That's now what Cachet is, for something like that you should look at Uptime Robot or Pingdom. What you can do though is integrate either of those services with Cachet, by using the API to automatically update the statuses or add an incident.

Aha, thanks ^^
On Thu, Jan 22, 2015 at 6:07 PM, James Brooks<EMAIL_ADDRESS>wrote:

That's now what Cachet is, for something like that you should look at
Uptime Robot or Pingdom. What you can do though is integrate either of
those services with Cachet, by using the API to automatically update the
statuses or add an incident.
—
Reply to this email directly or view it on GitHub
https://github.com/cachethq/Cachet/pull/410#issuecomment-71056568.

--
Met vriendelijke groeten,
Jan Vansteenlandt

:+1:


Alias for check

How about adding cargo c alias for cargo check?

I believe that one of the most frequently used subcommands is check.
Adding this alias would save much time.

Currently, there are three aliases: b for build, r for run, and t for test.
I think that adding c alias would cover most of the use cases.
One problem I have come up with is that the clean subcommand also starts with the letter c.
But I believe that running check subcommand by mistake would not hurt developers so much.
I would like to prepare a PR for this change.

You can define your own aliases in ~/.cargo/config.  They are briefly mentioned at the bottom of this page: https://doc.rust-lang.org/nightly/cargo/reference/config.html#configuration-keys
This is the alias that I personally use:
[alias]
c = ["check", "--all-targets"]

I did not know the configuration. Thank you!
Still, I personally want the out-of-the-box c alias for check because the subcommand is frequently used.
I want to know how people feel about this addition.

I would argue it warrants equal status in the single-char alias club.  I'd say send the PR and let's see 🙂


2nd factor authentication turns off 1st factor authentication

When I did "security.pam.services.sshd.oathAuth = true", the resulting pam file was:
auth sufficient pam_unix.so  likeauth try_first_pass
auth sufficient ${pkgs.oathToolkit}/lib/security/pam_oath.so window=5 usersfile=/etc/users.oath digits=6
which allows me to type a wrong password and make up for it by a correct OTP. I haven't reviewed the resulting PAM file carefully after making that change, and I was taken by surprise by that behavior. I would have expected a more conservative "conjunctive" semantic here, as in "OTP" is required in addition to everything else.
Justification:
While the other authentication mechanisms surrounding the "auth" directives in
https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/security/pam.nix#L262
could be argued to have password like qualities and rightfully replace pam_unix.so (pam_ldap, pam_otpw), I would argue that 2nd factor auth methods like xOTP don't have those qualities (2^6^8 for a 8 char password vs 10^6~=2^16 for xOTP). Otherwise, they wouldn't be called 2nd factor :)
Implementation:
Move 2nd factor mechanisms to the top of the auth section and mark them required. Unfortunately that makes all authenticators prompt for 2nd factor first, which is a bit odd. I haven't yet thought about a more complex restructuring of pam.nix.
While at it, we could split up the "pam.text" section into four overridable ones for account/auth/session/password, so that people, that don't agree with my conservative stance and want 2nd factor only, can change their configs easily.

System: 16.03.616.4ac4b35 (Emu)
Nix version: nix-env (Nix) 1.11.2
Nixpkgs version: 16.03.616.4ac4b35

There is PR https://github.com/NixOS/nixpkgs/pull/11886 which tries to fix it...

I assume, I could add something like security.pam.u2f.type = required|sufficient, is that right?

jesus, does this still occur?

@clefru is there some updates ?
cc @vcunat @obadz @spacefrogg

@jpierre03 Yes.[*] https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/security/pam.nix#L271 still has auth sufficient pam_unix.so followed by L275 which generates also auth sufficient pam_oath.so. Why this is problematic is explained in the initial comment.
My fix hard-codes:
# Authentication management.
auth required pam_unix.so  likeauth
auth required ${pkgs.oathToolkit}/lib/security/pam_oath.so window=5 usersfile=/etc/users.oath digits=6

[*] Not tested, but I see no changes in pam.nix since I filed this that could affect the pam file generation.

I have a PR https://github.com/NixOS/nixpkgs/pull/11886 where I refactored it a little (but u2f not oath), I need to polish it a little and I'll try to fix this issue on u2f module.

For those following this at home, I have submitted a tentative fix for this issue in #22393, which seems to work for me.
It would be great if someone who understands PAM could review it, since I'm far from being an expert.

Fixed in master ^


Update router.php

Adds redirects

PhpRouter doesn't overwrite the normal PHP redirect that can done via the header() function and it will stay that way. PhpRouter respects the normal behavior of the language.


where is  imed_models.py

Thank you for your excellent work, but I can't find imed_models.py in your code, where can I find it please?

Sorry for neglection. We have uploaded imed_models.py now.

Thank you for your excellent work, but I can't find imed_models.py in your code, where can I find it please?

Hello, could you please give me the OCTA data set? My e-mail is zyf1787594682@163. com,thank you very much!


[PDI-19689] Fix ETL metadata injection stage tooltip description, doc…

…umentation link, and behavior in search results.

:white_check_mark: Build finished in 7m 7s
Build command:
mvn clean verify -B -e -Daudit -amd -pl plugins/meta-inject/impl

:ok_hand: All tests passed!
Tests run: 46, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 7m 8s
Build command:
mvn clean verify -B -e -Daudit -amd -pl plugins/meta-inject/impl

:ok_hand: All tests passed!
Tests run: 46, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message


Update AliasField.php to fix misspelling of limitting

Fixed the misspelling of limitting to limiting.

Merged, thanks!


Which Open Source license will you apply?

Perhaps MIT?

https://github.com/stems/join-monster/blob/master/LICENSE
MIT

didn't see that for some reason. Thanks!


Generate mkdocstrings docs

This will allow us to import the inventory into the starlite reference docs and cross reference to these docs.
See https://mkdocstrings.github.io/usage/#cross-references-to-other-projects-inventories

We can generate an mkdocs website as well if we want.

Absolutely. Maybe this library would better suit the automated reference docs pattern that I had in starlite before reverting back to the manual approach. Even if they weren't perfect they'd be better than nothing and would keep themselves in check.
Also then might be able to have a landing page on https://starlite-api.github.io/ that has links to all the org project docs - at the moment it just 404s and is something I've been keen to do something about.

Sounds good to me


Setting Git Sync Keybinding Breaks Git View Keybinding

VSCode Version: 1.3.1
OS Version: Windows 7 Professional (SP1)

Steps to Reproduce:

Change keybindings.json to the following:
 // Place your key bindings in this file to overwrite the defaults
 [
     {"key": "ctrl+shift+g ctrl+shift+s",    "command": "workbench.action.git.sync"}
 ]

The new action works, but now when trying to access git with ctrl + shift + g, I am seeing the message:

(Ctrl + Shift + G) was pressed. Waiting for second key of chord...

Even though the default keybindings still contain
{ "key": "ctrl+shift+g",          "command": "workbench.view.git" }

I also tried including "when": "editorFocus" to my Sync keybinding but with no luck.

ctrl+shift+g ctrl+shift+s defines a cord - that means ctrl+shift+g isn't a short cut anymore but only the first part of a double short cut.


Udemy Course Analysis (Hacktoberfest 2023)

PR Description 📜
Machine learning analysis of Udemy courses involves using data-driven techniques to gain insights, improve recommendations, and enhance user experience on the platform.
Fixes #21
Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I have followed the proper naming convention shown in CONTRIBUTING GUIDELINE
[x] I have added the accuracy at the end of the notebook
[x] I have added all README.md files which are specified in the CONTRIBUTING GUIDELINE's file structures section
[x] I have specified the respective issue number

Thank you soo much for contributing to our repository 💗

@sujanrupu Thank you for contributing!!


Camera not working in howdy on Lenovo X1 Carbon 6th Gen on Fedora 38

I've installed howdy via copr on Fedora 38 on my Lenovo X1 Carbon 6th Gen. When I try to add a face model the IR emitters turn on and after a couple of seconds I get the following error:
[ben@laptop ~]$ sudo howdy add
Adding face model for the user ben
Enter a label for this new model [Initial model] (max 24 characters):
[<EMAIL_ADDRESS>global videoio_registry.cpp:232 VideoBackendRegistry VIDEOIO: Enabled backends(9, sorted by priority): FFMPEG(1000); GSTREAMER(990); INTEL_MFX(980); V4L2(970); CV_IMAGES(960); CV_MJPEG(950); FIREWIRE(940); UEYE(930); OBSENSOR(920)
[<EMAIL_ADDRESS>global backend_plugin.cpp:369 getPluginCandidates VideoIO plugin (FFMPEG): glob is 'libopencv_videoio_ffmpeg*.so', 1 location(s)
[<EMAIL_ADDRESS>global backend_plugin.cpp:379 getPluginCandidates     - /lib64: 0
[<EMAIL_ADDRESS>global backend_plugin.cpp:383 getPluginCandidates Found 0 plugin(s) for FFMPEG
[<EMAIL_ADDRESS>global cap_gstreamer.cpp:1403 open OpenCV | GStreamer: /dev/video0
[<EMAIL_ADDRESS>global cap_gstreamer.cpp:1436 open OpenCV | GStreamer: mode - FILE
[<EMAIL_ADDRESS>global cap_gstreamer.cpp:2784 handleMessage OpenCV | GStreamer warning: Embedded video playback halted; module source reported: Could not read from resource.
[<EMAIL_ADDRESS>global cap_gstreamer.cpp:1679 open OpenCV | GStreamer warning: unable to start pipeline
[<EMAIL_ADDRESS>global cap_gstreamer.cpp:1164 isPipelinePlaying OpenCV | GStreamer warning: GStreamer: pipeline have not been created
libva info: VA-API version 1.18.0
libva info: Trying to open /usr/lib64/dri/iHD_drv_video.so
libva info: Found init function __vaDriverInit_1_18
libva info: va_openDriver() returns 0
MFX: Unsupported extension: /dev/video0
[<EMAIL_ADDRESS>global cap_v4l.cpp:1119 tryIoctl VIDEOIO(V4L2:/dev/video0): select() timeout.

Please look straight into the camera
[<EMAIL_ADDRESS>global cap_v4l.cpp:1119 tryIoctl VIDEOIO(V4L2:/dev/video0): select() timeout.
Failed to read camera specified in your 'device_path', aborting

The webcam only supports one resolution
[ben@laptop ~]$ v4l2-ctl -d /dev/video0 --list-formats-ext
ioctl: VIDIOC_ENUM_FMT
	Type: Video Capture

	[0]: 'YUYV' (YUYV 4:2:2)
		Size: Discrete 160x120
			Interval: Discrete 0.033s (30.000 fps)

I have tried setting this resolution in howdy's config and turning on force_mjpeg both to no avail.
I also tried setting the resolution to 400 by 400 as suggested by this wiki article but that didn't change anything either.
If the wiki article is to be believed the hardware worked at one point with howdy... at least on arch linux.
So maybe this is a bug/regression?

I've searched for similar issues already, and my issue has not been reported yet: Yes
Linux distribution (if applicable): Fedora 38
Howdy version (sudo howdy version): 2.6.1

Failed to read camera specified in your 'device_path', aborting

Are you sure camera device in device_path is correct? Does reading from that device from other programs (eg. video player) work?

Not on fedora, but on arch, this configuration works fine.

Make sure you have all the latest firmware updates, as in my experience that makes a big difference with a lot of lenovo devices!
Make sure you have those configurations:device_path = /dev/v4l/by-id/usb-SunplusIT_Inc_Integrated_IR_Camera-video-index0
max_height = 400
force_mjpeg = true

@tokox pretty sure the path I was using was correct yes. When I try to use the device from other programs the red LEDs next to the camera turn on but I don't get a video feed... I don't know if this is to be expected for IR cameras. I set the resolution already.
@boredland This is what 'sudo fwupdmgr update' says:
Devices with no available firmware updates:   • INTEL SSDPEKKF256G8L Devices with the latest available firmware version:  • Embedded Controller  • Intel Management Engine  • System Firmware  • UEFI dbx
So I guess I am at the latest firmware? Also I tried with your exact config and now I get the following error when I run "sudo howdy add" or "sudo howdy test"
Camera path is not configured correctly, please edit the 'device_path' config value. Exception ignored in: <function VideoCapture.__del__ at 0x7f11c728d760> Traceback (most recent call last):   File "/usr/lib64/security/howdy/recorders/video_capture.py", line 55, in __del__     self.internal.release()     ^^^^^^^^^^^^^ AttributeError: 'VideoCapture' object has no attribute 'internal'

Well, I am on arch - perhaps they behave differently. What does /dev/v4l/by-id/ yield?

[ben@laptop ~]$ ls /dev/v4l/by-id
usb-Chicony_Electronics_Co._Ltd._Integrated_Camera-video-index0  usb-SunplusIT_Inc_SPCA2085_PC_Camera_01.00.00-video-index0
usb-Chicony_Electronics_Co._Ltd._Integrated_Camera-video-index1  usb-SunplusIT_Inc_SPCA2085_PC_Camera_01.00.00-video-index1


Improve go.mod support in generator

This improves the go.mod support added in #1636 by adding support for
targets not at the module level.
Imagine you generate code like this:
$ swagger generate client --name my-api --target ./pkg/api

Then it would assume go.mod is under pkg/api and not detect that go
modules are used. The fix is to walk up the directory to find the first
occurance of go.mod and assume that as the module base URL.

I don't understand why the build is failing, I can run the tests fine locally. Any ideas if it's related to my changes?

Looks related to some previous changes I made... I have to investigate there: a bug did sneak in undetected

I suggest as a temporary workaround to pass your PR to comment the "Parallel()" statement here https://github.com/go-swagger/go-swagger/blob/dd867fd63c30269ac217004c102f47a1774d4f5a/generator/moreschemavalidation_test.go#L301
I assume my latest attempt to improve this did introduce some smelly behavior.
You could try this and see if your pr passes. I'll try to fix things up later on.

@fredbi commenting out the parallel config did make the build run successfully.

Found the concurrency bug in unit test. Going to fix it and specifically test for this...


Fixing a typo and some information related to the

Removing the bit about the text rig; only need an identity
Rearranging headings
Fixing typo
Signed-off-by: moxiegirl<EMAIL_ADDRESS>

@larrysalibra can you give this a check and then merge it if it looks ok.  I removed the bit about having a test rig. Not necessary. I add in a section about creating an ID.  While not strictly necessary it is easier for them if they have an ID when they start rather than creating it as part of the mobile walkthrough.


Error: end of central directory record signature not found on Linux Travis tests

I'm seeing Travis failures for vscode-go with this error, but only on Linux, not OS/X.  This is seen building the current master branch of https://github.com/Microsoft/vscode-go.
It looks like a different issue than #2384 though that led to the same error message.
$ npm test --silent
### VS Code Extension Test Run ###
Current working directory: /home/travis/gopath/src/github.com/Microsoft/vscode-go
Downloading VS Code into "/home/travis/gopath/src/github.com/Microsoft/vscode-go/.vscode-test" from: https://vscode-update.azurewebsites.net/1.2.0/linux-x64/stable
stream.js:94
      throw er; // Unhandled stream error in pipe.
            ^
Error: end of central directory record signature not found
    at /home/travis/gopath/src/github.com/Microsoft/vscode-go/node_modules/vscode/node_modules/gulp-vinyl-zip/node_modules/yauzl/index.js:167:14
    at /home/travis/gopath/src/github.com/Microsoft/vscode-go/node_modules/vscode/node_modules/gulp-vinyl-zip/node_modules/yauzl/index.js:475:5
    at Object._onImmediate (/home/travis/gopath/src/github.com/Microsoft/vscode-go/node_modules/vscode/node_modules/gulp-vinyl-zip/node_modules/yauzl/node_modules/fd-slicer/index.js:200:5)
    at processImmediate [as _immediateCallback] (timers.js:354:15)
The command "npm test --silent" exited with 1.

gulp-vinyl-zip

@Tyriar did you change all linux builds to produce tar.xz files instead of zip? We have tools that depend on the fact that they are zip files.

This should be fixed now, it happened because stable was built using some changes in vscode-distro that I wasn't expecting. I created https://github.com/Microsoft/vscode/issues/7308 to follow this up in the June release so this doesn't happen in the real switch over.


Chained onCall and withArgs doesn't work

stub = sinon.stub()
stub.withArgs(42).onFirstCall().returns(1).onSecondCall().returns(2)
stub.returns(0);
assert stub(1) == 0
assert stub(42) == 1 # fails, returns 0
assert stub(1) == 0
assert stub(42) == 2
assert stub(1) == 0
assert stub(42) == 0

Need further investigation.

This is issue is not the same as https://github.com/note35/sinon/issues/22

I am working on this now

This one is difficult to solve, I am still working on it

I finished the first part of a solution. I will clean it up and submit on Monday.


Unit tests disappeared?

Some unit tests, in particular pipelining tests, have disappeared since build 596. I want to develop something and see if it still works. Are they going to come back?
In package eu.freme.broker.integration_tests.pipelines:

EntityLinkTerminologyTest
EntityLinkTest
EntityLinkTranslationTest

Seems like they were lost with 4ce0d3de9b371bede2b254ce8511409d2a387e2e albeit not being in the changelog.
I will readd them

OK, thanks!

There they are. Sorry for the inconvenience


Update DiagnosticSource to 4.4.1

This would solve https://github.com/Microsoft/ApplicationInsights-dotnet-server/issues/613

4.4.1  version does not have a fix yet, closing this issue


Home Page documentation

Fix #554

@smaranjitghose Kindly check :)

Certain changes:

For stuff like path names or even lines of code, use literal blocks like:

print("Hello World")

or ```assets\Images|..````

For selecting color, give a line you can select your color using Google's color picker
and also we need the hex value

Increase spacing between points

For the files, required to edit give external links (i.e. links to the files on this repo's master branch)

@smaranjitghose I have made all the changes which you told. Please review and let me know if something else is required till here :)

@smaranjitghose I have made all the changes which you told. Please review and let me know if something else is required till here :)

docs/_build/html/_sources/index.rst.txt
I think the files are supposed to be in .rst extension instead of plain text. just do a make html changing that. If everything works fine, make those changes otherwise let it be

I will just recheck

I have done it, and it is the same.
within the source directory its extension is. rst.txt only

I have done it, and it is the same.
within the source directory its extension is. rst.txt only


Issue#83 Code Documentation: Explaining Reasoning and Attempts

This issue aims to create detailed documentation that thoroughly explains the reasoning behind the code and outlines the various attempts made during its development. The documentation will serve as a comprehensive guide for understanding the decisions made, the problem-solving process, and the evolution of the codebase. It will cover insights into the logic, methodologies, and iterations undergone in the development cycle to provide a clear and informative resource for current and future contributors.
Objectives:

Explain the reasoning behind design decisions and how the code was built.
Keep track of what worked, what didn't, and why, during development.
Make the decision-making process clear to help contributors grasp the codebase better.

Expected Outcome:
A clear document detailing the thinking behind design choices and the development process. This documentation will help developers understand why certain decisions were made and how the code evolved.

I have an issue about a broken link on the adr-004 file (not related to that), I'll delete it.

@rngadam I see that other folder have mixed case too, do you want me to change them before merging ?

@JolanThomassin let's merge yours with correct names and fix the others in a separate issue/PR

@JolanThomassin is out sick, merging


Doesn't throw errors when there is a compilation error

This is really important. There are times when the page is blatantly just loading and causes too many pain points.

@ktkaushik sorry to hear you're having trouble. Could you please provide a reproduction case so that it's clear what you're referring to?

This one below for an example - Compiling a hbs which has typo in if case should throw an error, right ?
{{#if user.username}}
   -- Do Something here

{{elsee}}

  -- Flash message here
{{/if}}

Shouldn't something like this throw an error of some sort ?
Another great example would be the typo in the partials path
{{> /dashboard/activities}}

instead of
{{> dashboard/activities}}

Is there any solution ?
Thanks

As far as I am aware, this correct and inline with how the underlying handlebars library works:
Template compilation does not throw an error. Rather, errors are thrown when the compiled template is executed with some data. Only when you have the data, i.e. the context in which the template is executed, are you fully able to determine all the possible errors which might occur.
Handlebars has another method - precompile which does throw syntax errors in advance of having data.
Assuming you're using express-hbs in an express app and expecting template errors to occur when the server starts, rather than when a is page rendered, then I'd suggest you build a precompile step into your build process to detect any errors before the server is even started?

@ErisDS I will try the precompile today in a grunt task and get back to you.
Thanks :)

@ktkaushik did that solve it?

@ErisDS no, the precompile didn't really solve it. I wonder why though.

I used grunt-handlebars to precompile but no luck

IS this still an issue with the latest?

Closing out all old issues. If this is still an issue with the latest version please let us know.


ayatana-indicator-bluetooth: init at 24.5.0

Description of changes
Working towards https://github.com/NixOS/nixpkgs/issues/99090.
Ayatana Indicator Bluetooth, for showing & changing bluetooth settings.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] 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/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build nixosTests.lomiri

Result of nixpkgs-review pr 336982 run on aarch64-linux 1

1 package blacklisted:

nixos-install-tools

2 packages built:

ayatana-indicator-bluetooth
disko


sklearn compatibility when n_samples == 1

Hi,
Sorry if my question is a bit out of scope (which is a bit more related to scikit-learn compatibitlity), but I've looked at the code and found this, which intrigued me.
https://github.com/lmcinnes/umap/blob/bf77f919533c9fc33fbcc9f488d48eeb1ac34810/umap/umap_.py#L1336-L1338
I've been looking at scikit-learn source code for some time and I wonder where this restriction comes from.  In comparison, scikit-learn's implementation of TSNE raises an error in that scenario (not directly, but when calling the neighbor algorithm, because the number of neighbors is equal to zero). Could you explain me why you decided to handle this 'irrelevant' scenario this way?
Moreover, if you stick with this choice, shouldn't it return an array of zeros whose length depends on n_components (i.e. np.zeros(self.n_components))?
Obviously it's not very important, but it just intrigued me, so if you have the time to answer, it would be really appreciated.

That's a bug -- I believe I was thinking of fit_transform not fit.
I will get that fixed. Good catch! thanks.
On Fri, Aug 3, 2018 at 2:54 AM Johann Faouzi<EMAIL_ADDRESS>wrote:

Hi,
Sorry if my question is a bit out of scope (which is a bit more related to
scikit-learn compatibitlity), but I've looked at the code and found this,
which intrigued me.
https://github.com/lmcinnes/umap/blob/bf77f919533c9fc33fbcc9f488d48eeb1ac34810/umap/umap_.py#L1336-L1338
I've been looking at scikit-learn source code for some time and I wonder
where this restriction comes from.  In comparison, scikit-learn's implementation of TSNE raises an
error in that scenario (not directly, but when calling the neighbor
algorithm, because the number of neighbors is equal to zero). Could you
explain me why you decided to handle this 'irrelevant' scenario this way?
Moreover, if you stick with this choice, shouldn't it return an array of
zeros whose length depends on n_components (i.e. np.zeros(n_components))?
Obviously it's not very important, but it just intrigued me, so if you
have the time to answer, it would be really appreciated.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lmcinnes/umap/issues/102, or mute the thread
https://github.com/notifications/unsubscribe-auth/ALaKBUIrKpUhDkmr9BrgeU74eST6vU9-ks5uM_OqgaJpZM4VteLv
.

https://github.com/lmcinnes/umap/blob/23e6af696ef715c21655faa7cdefb7d44a5f9d0c/umap/umap_.py#L1336-L1339
With the current code, the docstring for the fit_transform method is not compatible with the way you deal with the "one sample" scenario. The fit_transform method should return a numpy.array with shape (n_samples, n_components), but it will return np.array([0.0, 0.0]). You may want to replace np.array([0.0, 0.0]) with np.zeros((1, self.n_components)). I didn't run the code with the latest version on GitHub so I don't know if it would handle the transform method.
You could also update the docstring of the fit method to say that it returns self.

Good point, thanks.  I definitely appreciate your
corrections!
On Mon, Aug 6, 2018 at 4:15 AM Johann Faouzi<EMAIL_ADDRESS>wrote:

https://github.com/lmcinnes/umap/blob/23e6af696ef715c21655faa7cdefb7d44a5f9d0c/umap/umap_.py#L1336-L1339
With the current code, the docstring for the fit_transform method is not
compatible with the way you deal with the "one sample" scenario. The
fit_transform method should return a numpy.array with shape (n_samples,
n_components), but it will return np.array([0.0, 0.0]). You may want to
replace np.array([0.0, 0.0]) with np.zeros((1, self.n_components)). I
didn't run the code with the latest version on GitHub so I don't know if it
would handle the transform method.
You could also update the docstring of the fit method to say that it
returns self.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/lmcinnes/umap/issues/102#issuecomment-410625898, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ALaKBZXdjxj2yTbA3-gbbytHNP5f-wjXks5uN_sigaJpZM4VteLv
.

I feel like I have a reasonable approach -- it isn't expensive to use
np.zeros((1, self.n_components)), and that is good enough in this odd
circumstance.
On Tue, Aug 7, 2018 at 3:23 AM Johann Faouzi<EMAIL_ADDRESS>wrote:

Maybe raising an error would be the easiest solution.
Currently, if you fit UMAP with one sample, you want the fit_transform
and transform methods to return np.zeros(X.shape). The user may be
confused by those results, so you would probably add a warning when the
instance of UMAP is fitted. To me that looks a bit expensive code-wise, for
a "dumb" scenario that should be avoided.
Imho the solution to this issue should be driven by the theory: if the
theory doesn't handle fitting with one sample, raise an error; otherwise
the output should probably different from an array of zeros (that's my
assumption, I'm not familiar enough with UMAP theory).
In comparison to PCA and TSNE implementations in scikit-learn:

TSNE raises an error (not directly, but when the neighbor algorithm
is called).
PCA raises a warning (true_divide for the explained variance) and
outputs something different from an array of zeros.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/lmcinnes/umap/issues/102#issuecomment-410960103, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ALaKBTHKnYUvAr1LX8IV1jNO6yiin0QSks5uOUBegaJpZM4VteLv
.

It's not enough because the transform method returns an error if you fit with one sample:
AttributeError                            Traceback (most recent call last)
<ipython-input-8-b21dd561e9fc> in <module>()
----> 1 embedding.transform(np.random.randn(40, 20))

c:\users\johann\src\umap\umap\umap_.py in transform(self, X)
   1541         X = check_array(X, dtype=np.float32, accept_sparse="csr")
   1542         x_hash = joblib.hash(X)
-> 1543         if x_hash == self._input_hash:
   1544             return self.embedding_
   1545

AttributeError: 'UMAP' object has no attribute '_input_hash'

When you return np.zeros((1, self.n_components)) early in fit, none of the code below is run. Maybe you could add something like self._fit_with_one_sample = True if it happens, otherwise self._fit_with_one_sample = False.
Then, early in transform, you could add something like:
if self._fit_with_one_sample:
    return np.zeros(X.shape)


Message cannot be send

i get this error by the end of the session
selenium.common.exceptions.StaleElementReferenceException: Message: stale element reference: element is not attached to the page document

Do you get this error when trying to send a message or when you end the script?

i get this error when trying to swipe
raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.ElementClickInterceptedException: Message: element click intercepted: Element ... is not clickable at point (59, 96). Other element would receive the click: ...
(Session info: chrome=89.0.4389.114)
(Driver info: chromedriver=89.0.4389.23 (61b08ee2c50024bab004e48d2b1b083cdbdac579-refs/branch-heads/4389@{#294}),platform=Windows NT 10.0.17763 x86_64)

But this shows up right after the script ends
selenium.common.exceptions.StaleElementReferenceException: Message: stale element reference: element is not attached to the page document

Hi @DarkKni0ht ,
This was being caused by Tinder showing a new pop up, which is now being handled in the newest update.
So pulling the latest version should resolve your problem.  :)
If the problem persists feel free to re-open the issue.
Have a nice day,
Fred