surfaceform make-unspottable is not working for me

e.g.
sh model-editor surfaceform candidates path/to/en/model automated returns results.
If I run:
sh model-editor surfaceform make-unspottable path/to/en/model automated' or sh model-editor surfaceform make-unspottable path/to/en/model make-unspottable.txt -f' where make-unspottable.txt has one line in it: automated
I see the output: the counts for:automated,2660903 has been reduced.
However, if I run sh model-editor surfaceform candidates path/to/en/model automated again, automated is still 'spottable'. Further, the results - scores and support - are exactly the same.
I was hoping this surfaceform would be removed completely.
I'd be grateful if someone could let me know what I am doing wrong.

You're doing nothing wrong.
make-unspottable reduces the annotated count of the surface form to make it less spottable. If you want to make surface forms completely unspottable, you can edit this function.

Thank you, I'll try that.
Separately, some surfaceforms that I see in results, are not found as candidates. Is this related to "types"?
e.g. if I have 'ul' in the annotation text I get the following result:
<Resource URI="http://dbpedia.org/resource/University_of_Limerick" support="394" types="DUL:Agent,DBpedia:Agent,Wikidata:Q43229,DUL:SocialPerson,Schema:Organization,DBpedia:Organisation,Schema:EducationalOrganization,DBpedia:EducationalInstitution,Wikidata:Q3918,Schema:CollegeOrUniversity,DBpedia:University" surfaceForm="ul" offset="1" similarityScore="0.9999999999263878" percentageOfSecondRank="7.146778453439215E-11"/>
But: sh model-editor surfaceform candidates path/to/model/en/model ul raises a SurfaceFormNotFoundException.
Do you know what is happening here?
Thank you.

Are you using the same model when annotating with spotlight and exploring with spotlight-model-editor? That's with only explanation I can see.

Thanks for your prompt response. Yes, I am. I have only one model in this environment. In addition, when running spotlight I can confirm that other edits I have made (using make-unspottable or association remove) have taken effect. Strange.

Closing as part of archiving process.


stage_inputs format

Just wondering how to get the stride of a vertex input so I can create my buffer descriptor correctly. Calling get_type(input.type_id) on layout(location = 0) in vec4 a_Pos; appears to return a Float { array: [] }. There doesn't seem to be any way to get the actual vecsize which I was expecting. Am I missing something here?

Hi @erlendp! Were you able to figure this out?

@grovesNL I've put this down for a while, while I work on other projects, but I'm still keen to find a solution!

Looks like the required fields hadn't been mapped through yet, so I've added a PR to expose them:
https://github.com/grovesNL/spirv_cross/pull/142

Looks like this was fixed by #142, thanks @sir-earl!


Add new Coding Train Live! track

maybe this doesn't need credits? 😅

Also, in the future it might be useful to have a "Live" section on the site, or feature this track on the homepage; if there ends up happening multiple project live streams.

I agree it doesn't need the credits, but it fails the tests w/o them.  Also, I am under the impression that there are a lot of videos on the site that haven't been edited that have the credits.
I am not sure where the best place to put the livestream videos is, but this was an easy way to add the video and allow people to start contributing  their own versions.

For the credits, I think providing an empty array will pass the tests.

I think this is a great start! Ideally, I would love to annotate and document every single live stream and have a separate section of hte website like challenges, tracks, but I think that's too big of an undertaking and unrealistic to do right now. So I track for "select" livestreams makes a lot of sense!
I don't feel too strongly about the credits, it's true they don't apply for livestreams so maybe it makes sense to leave a blank array. @kfahn22 let me know when this is ready to merge! I'll review and see if I want to make any small edits too.

@shiffman I have tried passing a blank array and it is failing the content tests for me.  Other than that, it is ready for review.

@shiffman I have tried passing a blank array and it is failing the content tests for me. Other than that, it is ready for review.

You can remove the min(1) and/or required() rules on the credits field of the schema if you want to loosen the validation rule. Make sure to test that the site is able to handle an undefined or empty array value before merging!
https://github.com/CodingTrain/thecodingtrain.com/blob/3f59b50d8c3137a11c1f9083fa49c832145dc263/content-testing/schemas.js#L121-L129

I have changed the tests to allow for empty credits.

Apologies for the delay, this is wonderful, thank you everyone! I'm going to go ahead and merge and we can always refine more later!


feat: implement 0x33 - CALLER

Since
Frontier
Group
Environmental Information
Description
0x33 - CALLER:  Get caller address
Index 1 is top of the stack. See PUSH.Stack outputaddress: the 20-byte address of the caller account. This is the account that did the last call (except delegate call).Example
| Input | Output
-- | -- | --
1 |   | 0xbe862ad9abfe6f22bcb087716c7d89a26051f74c
Reproduce in playground.Error casesThe state changes done by the current context are reverted in those cases:Not enough gas.Stack overflow.

I can do this one

Can be closed now as #112 implements it


Pass Extensions not working(pass-import, pass-update, pass-tomb ...)

Describe the bug
$ pass import
Error: import is not in the password store.
To Reproduce
Steps to reproduce the behavior:

Install pass globally through configuration.nix

  environment.systemPackages = with pkgs; [
    ### Pass
    pass
    passExtensions.pass-audit
    passExtensions.pass-genphrase
    passExtensions.pass-import
    passExtensions.pass-otp
    passExtensions.pass-tomb
    passExtensions.pass-update
    (pass.withExtensions (ext: with ext; [ pass-audit pass-otp pass-import pass-genphrase pass-update pass-tomb]))
  ];

Expected behavior
Pass import and other extensions working without error.
pass import keepassxml keepass.xml
(*) Importing passwords from keepass
 .  File: keepass.xml
 .  Number of password imported: 6
 .  Passwords imported:
       Social/mastodon.social
       Social/twitter.com

Additional context
Other pass extensions are also not working
Metadata

system: "x86_64-linux"
host os: Linux 4.19.96, NixOS, 19.09.1155.bae4d7daa01 (Loris)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3
channels(quark): "nixpkgs-20.03pre207249.7e8454fb856"
nixpkgs: /home/sathish/.nix-defexpr/channels/nixpkgs

The order of your packages matters. You have both pass and pass.withExtensions in your global packages defined. The order of packages is important, because who comes first - wins. See an example:
$ nix-shell -E 'with import ./.{}; mkShell { buildInputs = [
    pass
    (pass.withExtensions (ext: with ext; [pass-import]))
]; }'

[nix-shell]$ pass import
Error: import is not in the password store.

$ nix-shell -E 'with import ./.{}; mkShell { buildInputs = [
      (pass.withExtensions (ext: with ext; [pass-import]))
      pass
]; }'

[nix-shell]$ pass import
 [x] Error: password manager not present. See 'pass import -h'

(but there are also hiPrio and lowPrio modifiers, which can change priority of a package)


PowerShell - errors not handled

If I add a terminating error to my PowerShell script (write-error "snakkes -erroraction Stop) there is no indication of the error in the console/logs, and the function still reports "Success" (although script execution seems to halt). We need a much better visibility into the actual health of the script, and all streams (output/warning/verbose/error) before PowerShell in Functions in really useable.

Thanks. We're revamping our PowerShell support as we speak (see PR here). In addition to adding custom module support and other features, hopefully that work will address these issues as well.
Feel free to check out that PR and comment on it.

@tohling Do your PowerShell changes address this issue?

Yes, with the next release, if the customer's script executes the following
Write-Error "Test error message!" -ErrorAction Stop
the customer should see 2 trace messages in the logs:

"Function completed (Failure, Id="
"Exception while executing function: Functions.. System.Management.Automation: The running command stopped because the preference variable "ErrorActionPreference" or common parameter is set to Stop: Test error message!."

Sample output in logs is shown below:
2016-06-02T15:17:34.512 Function started (Id=26fa7834-5cfb-4407-ac06-ea2acfccff29)
2016-06-02T15:17:35.198 Function completed (Failure, Id=26fa7834-5cfb-4407-ac06-ea2acfccff29)
2016-06-02T15:17:35.232 Exception while executing function: Functions.HttpTrigger-Powershell-Modules. System.Management.Automation: The running command stopped because the preference variable "ErrorActionPreference" or common parameter is set to Stop: Test error message!.

Closing this based on Ling's comments above. The next imminent release will include these changes.


feat(connect): add df.filter

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#3346  👈 (View in Graphite)
#3342 : 2 other dependent PRs (#3344 , #3345 )
#3343 : 2 other dependent PRs (#3309 , #3326 )
#3341
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Merge activity

Dec 11, 4:54 PM EST: A user merged this pull request with Graphite.


spl: update spl_token_2022 dependency to v0.7

There are a couple new features in 0.7 related to Token2022 extensions that would be nice to have in Anchor as well.

Having multiple versions of spl-token-2022 crate has caused dependency issues in the past(e.g. https://github.com/coral-xyz/anchor/issues/2502) and it will most likely cause issues again in the current form because Solana 1.16 crates depend on spl-token-2022 0.6.1.
To minimize the possibility of dependency issues, we'll bump it before the next release if the conditions allow for it(depends on Solana crates bump it too) but generally it's best avoid PRs that bump a breaking version of a dependency.


import if lib directory exists

#234

Why would serving import tm if tm is not required?

Why would serving import tm if tm is not required?

Serving with gradio supports two ways:

serving with tritonserver
serving with turbomind directly

Why not import turbomind in the function call that serving turbomind?

Lets say. If I want to import Tokenizer or other modules from turbomind while turbomind was not build. The error will be triggered.

All module inside turbomind should be depended on pythonbind, if it is not, move it out.


Prettier quotemark

Allow quotemark to use single quotes without clashing with Prettier rule about using double quotes on strings with ' int it. Ex.: "can't be used with single quotemark".

I believe this is about the TSLint quotemark rule, not about tslint-react. That said, there's an "allow-escape" option that should do what you need!


bugfix DeletedEntitesQuery

I see. The fix more complicated than I thought...
So far so good, but I'm getting still older entries in the same day when using the after parameter
> yarn fetch DeletedEntitiesQuery '{"first": 100, "after": "2014-03-24T14:23:28+01:00"}'
...
    {
      "dateOfDeletion": "2014-03-27T13:49:47+01:00",
      "id": 20985
    },
    {
      "dateOfDeletion": "2014-03-24T14:23:28+01:00",
      "id": 19595
    },
    {
      "dateOfDeletion": "2014-03-24T14:23:16+01:00", # it should have started after this date
      "id": 19628
    },
    {
      "dateOfDeletion": "2014-03-24T14:22:45+01:00",
      "id": 10493
    },
    {
      "dateOfDeletion": "2014-03-24T14:21:58+01:00",
      "id": 17114
    }
  ]
}


How can I generate from trained model?

I don't understand how I can generate Audio from my trained model, other than the samples automatically saved during the training process.

`import torch
import os
import soundfile as sf
DEVICE = 'cuda'
SAMPLING_RATE = 22050
BATCH_SIZE = 32
AUDIO_OUTPUT_DIR = 'generated_samples'
if not os.path.exists(AUDIO_OUTPUT_DIR):
os.makedirs(AUDIO_OUTPUT_DIR)
model_path = '/path/to/saved/model'
model = torch.load(model_path)
model.load_state_dict(model['generator'])
noise_vector = torch.FloatTensor(BATCH_SIZE, 100).to(DEVICE)
noise_vector.data.normal_()
with torch.no_grad():
fake = generator(noise_vector).detach().cpu().numpy()
for idx, sample in enumerate(fake):
sample = sample[0]
sf.write(os.path.join(AUDIO_OUTPUT_DIR, '{}.wav'.format(idx)), sample, SAMPLING_RATE)
`

model.load_state_dict(model['generator'])
AttributeError: 'dict' object has no attribute 'load_state_dict'

How to solve this error


Nanohttpd has some feature to log request in some file?

I would like to know if the project has some feature to store some kind of log like Apache:
http://httpd.apache.org/docs/1.3/logs.html
Many thanks in advance.

concure, you can do this in your subclass if you need it


Ueye

Currently we are trying to use an Ensenso n35. Up untill recently it worked just fine.
Today the camera started throwing errors. We are not using custom code simply the ensenso_camera_node and request_data. When trying to start the camera nodes we sometimes get the following error, which we have not run into up until now:
[ERROR] [1555504175.323337616]: NxLibException 17 (ExecutionFailed) for item /Execute
[ERROR] [1555504175.323476283]: UEyeApiException: UEye API error 17(The driver could not be loaded) at </common/uEye/uEyeCamera.cpp:472>.

At times the camera does manage to start up and data is received however after some time Ros will lose the connection and a similar error message is shown. Could this pertain to the ueye driver or the SDK?

This sounds like an issue with the uEye driver. Can ueyedemo use the camera reliably?

The ueyedemo seems to working fine. The camera has not given the error the last couple of times that we tested with it. However, this seems kind of unreliable in regard to deploying the system on site. It should not have errors that come and go with the weather in my opinion

I guess the problem also occurs in NxView, not only with the ROS node, right? If so, there is probably an issue with the uEye driver. You could try to reinstall it. If that doesn't help, you should contact the IDS support. They also provide support for our cameras and SDK.


Support json marshalling for structs

Proposal
Use case. Why is this important?
Multiple Yaml marshalling library started to use json tags as the main source for marshalling to yaml which means we need to use specific yaml marshalling libraries as the main marshalling format for prometheus structs.
For more context, see http://web.archive.org/web/20190603050330/http://ghodss.com/2014/the-right-way-to-handle-yaml-in-golang/

Can you explain more what the feature is that you wish to add to the Prometheus binary?
The code in this repo is internal, and as far as I'm aware we already have appropriate json tags where we are using JSON.

I am in need of serializing the structs in https://github.com/prometheus/prometheus/tree/master/discovery and those do not really seem internal as they are in use in multiple projects like prometheus-operator or promxy.
My use case is that I need to be able to marshal and unmarshal kubernetes_sd_configs as I need to generate scrape configs whenever we create a new cluster in my operator but our yaml library is using what is outlined in the link I posted previously so this is not getting marshalled properly.
The change would be minimal and I can do it but I wanted to propose my idea first

That's all internal code for internal data structures, and others import them entirely at their own risk.
If you need this I'd suggest maintaining your own structs, as none of the marshalling libraries we use have this issue.

In that case, should it clearly not be marked as internal then (e.g. moved into an internal directory) to avoid confusion for users?

That'd involve moving ~everything into an internal directory, which we're not willing to do. We're generally willing to let others import our code at their own risk, but that's very different from taking on code and features which we ourselves do not need which would bloat the codebase, set incorrect expectations, and make maintenance more difficult.

Sure, I understand your reasoning


Fails to load more than one plugin

I'm trying to run the most recent version of node-host with some fairly recent neovim, and I have pathogen.vim and clj-refactor.nvim and nvim-parinfer.js plugins in my bundle directory.  Tried on node 6.7.0 and 7.0.0.
If I move either plugin out of the way, :UpdateRemotePlugins works.  Both together fail with the following:
function remote#host#UpdateRemotePlugins[6]..<SNR>14_RegistrationCommands, line 20
Vim(let):Channel was closed by the client
remote/host: generated rplugin manifest: /Users/jfelice/.local/share/nvim/rplugin.vim
Press ENTER or type command to continue

I can get both working together manually by splicing together the lines in ~/.local/share/nvim/rplugin.vim produced by running it with each plugin installed by itself.

seems like a bug in UpdateRemotePlugins

Seeing this as well. Interestingly only with js plugins and not multiple python plugins

Thanks @eraserhd and @mhartington for isolating the issue! I'll be able to take a look at this over the weekend. If it's blocking your work, however, I'll prioritize a fix.

Not to be annoying and "that person", but did you get a chance to look at things @nhynes?

Hey @mhartington, I think that it is I who is being "that person" by not having taken a serious look at this yet. I'm quite busy until the 16th,  but if I have a few free moments between now and then, I'll fix this instead of something else!
If, after the 16th, this isn't yet fixed, you're encouraged to call me a bad maintainer :)
Sorry for not being able to fix this faster!

hehe, no worries!
Happy to dive in and take a look, just not sure where to poke around first.

I was without internet for a while yesterday, so I went and took a look at his bug. The issue is actually in node-client.
I fixed the multi-plugin bug by modifying msgpack5rpc, but that introduced a new issue seems to prevent the host from attaching to nvim. Debugging is complicated by the lack of a print channel, but I'll keep at it in my down time (and maybe clear a path for print statements, too)!

For the record, it used to be possible to run multiple plugins. Something changed between ~August and November.

As promised, I've prepared a fix. Now we must wait until @tarruda gets some time to look at the corresponding fix to msgpack5rpc!
Thanks again for your patience.


Initial D878 CSV generation

This PR now generates a D878 CSV file for all the (appropriate) channels.
I also fixed the download links on the Programming Files page so they download from same origin (and don't just open the CSV
file in the browser).

Thanks for the review.  I think I addressed all of them.


Test Issue created at Mon, 15 Mar 2021 16:31:24 GMT

Test issue body  Mon, 15 Mar 2021 16:31:24 GMT

Comment on issue at Mon, 15 Mar 2021 16:31:28 GMT


Rsync copies of files in Ceph to google storage.

This has been implemented in the backup server. With the help of rclone, rsync is happening every 3 hours to sync nmrxiv files in ceph to the google cloud storage.


Update SiteMiddleware import for Wagtail 2.11 compat

The import location of SiteMiddleware was changed in the 2.11 release of Wagtail to wagtail.contrib.legacy. The import would need to be changed in this package or the use of SiteMiddleware would need to be removed to allow use/development of the package with later versions of Wagtail.
See also Wagtail 2.11 release notes

Also wondering, is this added to the MIDDLEWARES list on purpose or is this just a default added by an older Wagtail starter? I did not find any use of request.site in the test app.
In that case, the reference could just be removed from the  MIDDLEWARES list.

I removed that middleware because I don't think it was used in the test app -- thanks for bringing it up!


3DS QR code using wrong IP address

I ran into an issue with the /opt/retronas/scripts/3ds_qr.sh file. I assign addresses via DHCP and usually have PfSense set up with a Static Assignment. At some point during testing I think I used the WiFi on my Pi4 instead of the LAN and it got the address <IP_ADDRESS> it wrote that into the 3ds_qr.sh file and never updated it (LAN uses <IP_ADDRESS>). Could some way be implemented RetroNAS to force an update either periodically or during reboot or as part of one of the update scripts?
TLDR: RetroNAS got an IP address during testing, that address has changed but the old one is still used by scripts.
Edit replaced * in IP address with 0

Not really an issue since the script worked as defined, am converting to discussion to continue with enhancement request conversation


Video Rebuffering on orientation change and playing in 16:9

As i requested orientation change my activity is recreated and again the buffering of the video takes place, if i use youtubeplayerview.engerfullscreen in my code then it enters full screen but in potrait mode only.
I want video in landscape mode without loosing the buffering.
https://drive.google.com/file/d/0B_F1L93eAIB6cVhfTUdOSVptUTFVTkZXeXZmdzZnTk55dGlr/view?usp=sharing

Hi, the library is not responsible for changing the orientation of your app, that should be done in your app.
The video is re buffered because when the orientation changes the Activity is recreated.
You should add this to the activity element in your manifest: android:configChanges="orientation|screenSize|keyboardHidden|smallestScreenSize|screenLayout" to manually handle orientation changes. As @ambiguous1991 suggested you can refer to the other issue for more information.
Let us know if you need help.

Documentation updated.

I added the following code to the manifest file,but now my video goes in
the full screen and comes back again.
below i share the screen recording of the application.
On Fri, Jul 6, 2018 at 10:37 AM, Pierfrancesco Soffritti <
<EMAIL_ADDRESS>wrote:

Closed #156
https://github.com/PierfrancescoSoffritti/android-youtube-player/issues/156
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PierfrancescoSoffritti/android-youtube-player/issues/156#event-1719493301,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Am-ZYSf_j7apRd-RawoVVh5PuUJuPD0Dks5uDvCggaJpZM4VETIf
.

I suggest you read this page from the Android doc.

https://drive.google.com/file/d/1y_DhLwFsquRBqU7Pm64i2cSeftdpPlG_/view?usp=sharing
On Fri, Jul 6, 2018 at 10:57 AM, Pierfrancesco Soffritti <
<EMAIL_ADDRESS>wrote:

I can see only the video in your first comment.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PierfrancescoSoffritti/android-youtube-player/issues/156#issuecomment-402929635,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Am-ZYZgaf6ou1dduaU4pYoMboWd41lT-ks5uDvU0gaJpZM4VETIf
.

It looks like you're changing the orientation programmatically to landscape but the physical orientation is still portrait.
This problem is Android-related, it's not due to the library.

okay, i removed the code of set requested orientation portrait but now this happens
please help
https://drive.google.com/file/d/1W2TVXEdFZnr4SYQPpTKDUWASlYckf9Db/view?usp=sharing

this is the same problem you had at the beginning, right? you need to manually handle orientation changes.


How to make disappear overflowY & borders when using SwatchesPicker

Hello,
I am looking for a solution in order to remove the css properties overflow-y and border but also box-shadow from SwatchesPicker but I cannot find a way to do this.
Below what I am always getting:

Is there a way to do it through props? Do I have to implement a custom color picker?
Anyway, thanks a bunch for this awesome component!

Probably best to just make your own custom picker. You can copy the entire picker from src/components/swatches and then tweak any of the styles there!


react-native

Hi,
I got react-native-pouchdb working (i.e. react-native-sqlite-storage as a websql adapter), and now I'm trying to get pouchdb-find working.  The system is quite unstable -- sometimes I can get insert and then a find to work, but other times the database gets locked up somehow.  The only clue I have before just giving up is the following:
error in _pouch_Tasks
sqlite.core.js:562 Object {code: 6, message: "UNIQUE constraint failed: local-store.id"}

I logged all the transactions and found two suspicious looking transactions:
BEGIN
INSERT INTO 'local-store' (id, rev, json) VALUES (?,?,?)
p1: "_local/indexes"
p2: "0-1"
p3: "{"views":{"idx-Tasks_day/idx-Tasks_day":{"Tasks-mrview-23295376d72329945b5bc85d197260a7":true}}}"

and:
BEGIN
INSERT INTO 'local-store' (id, rev, json) VALUES (?,?,?)
p1: "_local/indexes"
p2: "0-1"
p3:
"{"views":{"idx-Tasks_dueDate/idx-Tasks_dueDate":{"Tasks-mrview-a7a172c943daf3a928b3fa7fb9fbba74":true}}}"

These are two indexes I'm creating like this:
    const fields = ["dueDate"];
    db.createIndex({
      index: {
        fields,
        name: `idx-${dbName}_${fields.join("-")}`,
        ddoc: `idx-${dbName}_${fields.join("-")}`,
      },
    });

(I had the same problem when I let the index and ddoc name be chosen automatically)
I did some grepping and realized that this might be a pouchdb issue and not a pouchdb-find issue.  But I'm hoping you can at least point me in the right direction!

Have you used pouchdb-find with react-native?
Do any of the above transactions look familiar?

Thank you!  I'm a bit sad now because pouchdb and pouchdb-find have been working so great for me in the web version of my product, and I'm really hoping to not have to switch databases!!!

I think @nolanlawson has some experience with all of this.

OK with further testing I am still getting this error, but it doesn't seem to affect the stability.  Rather it seems like having more than one database is causing the issue (even with pouchdb-find disabled)!

Sounds like you have uncovered a bug in the SQLite engine for React Native (whatever is under the hood in react-native-sqlite-storage). This is a pretty common occurrence we're run into with SQLite adapters; they're usually very buggy and it isn't until you have a big heavy database like PouchDB that people uncover this stuff.
I'm working on porting node-websql to React Native. When that's ready, please switch over. Until then, consider React Native support very, very experimental. 😅

BTW the reason that I can confidently say it's not our bug is because PouchDB's WebSQL engine has been confirmed working on an enormous test suite on multiple browsers (PhantomJS, Chrome, Safari) as well as node-websql and SQLite Plugin 2. So if another WebSQL engine fails it's almost undoubtedly a bug in that engine.

@nolanlawson Thanks for the insight!  I am excited for node-websql in React Native :).

what is the version of sqlite-storage you are using? Can you send more information or file a bug with cordova-sqlite-storage which is the baseline for the react-native impl? Will try to fix it...


In Contact Us Form

Describe the bug
In the Contact us Form when user try to submit data then it is asking for the email address and message but it is accepting the form without writing the Name of user
To Reproduce
Steps to reproduce the behavior:

Go to Contact Us form
Add email
Add message
Submit it and it will be submitted
see the error

Expected behavior
The form should not be submitted without giving the name.
Screenshots
Here the form will be submitted but should not.

Desktop (please complete the following information):

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

Smartphone (please complete the following information):

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

Additional context
Add any other context about the problem here.

@akshitagupta15june please assign these issue to me.

I would like to work on this project. Please assign it to me.
Thank you

@akshitagupta15june please assign these issue to me.


average price v.s. best price

The algorithm is using the best bid and best ask as input to the graph. However, for most coins, their price is much lower than BTC/ETH, thus the actual filled price will differ from the best bid/ask.
Any plan to support average price rather than best price?

I had been busy the past few months with work but I will be adding this feature to the project. It will be first developed in the dev branch. To clarify, you do mean the average of the best bid and ask, correct?

@wardbradt do add to his question, does the script, for example use the ask or bid price to find arbitrage opportunities for single exchange multi-currencies?


Add support for the NATS Key-Value Store

Hi
Ive been looking at the suitability of the Quarkus NATS reactive messaging extension and noticed that the NATS Key-Value store API is not supported.  Is this something you could consider adding?
NATS has a Key-Value store API that is used by the NATS community and would be a great addition for the Quarkus Reactive Message Extension for NATS.
See here,   here & here
Thanks in advance
Declan

Sorry the late answer. It will probably take some time before I can look at implementing a bean to call the key-value store. You are welcome to make a PR if it's urgent:)

no problems just wanted to know if it was on a roadmap.

I have now started implementing support for Key-Value store :) Hopefully I have a release ready next week

Brilliant, thats a great addition IMO. Look forward to see it go live
On Fri, May 31, 2024 at 11:04 AM Paw Figgé Kjeldgaard <
@.***> wrote:

I have now started implementing support for Key-Value store :) Hopefully I
have a release ready next week
—
Reply to this email directly, view it on GitHub
https://github.com/quarkiverse/quarkus-reactive-messaging-nats-jetstream/issues/86#issuecomment-2141686823,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGY6FOGLB2OJUO7V3YAS3TZFBDMTAVCNFSM6AAAAABGIWT2WSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBRGY4DMOBSGM
.
You are receiving this because you authored the thread.Message ID:
<quarkiverse/quarkus-reactive-messaging-nats-jetstream/issues/86/2141686823
@github.com>

The latest release contains support for key / value store. I'm still missing to update documentation. But try look at the tests that see how to use it


Question about applying this method across multiple images

Hello, cool method! I would like to know how to apply this method across multiple images. For example, if I have $N$ images (samples), or an AnnData object with a sample categorical column in obs, how can I use this method across all of them to get similar clusters across multiple images correctly? Currently if I run the method across multiple images at once, each image gets essentially gets labeled with one cluster. I would like to see the same "subpopulations" appearing across multiple field of views.

My code for reference, maybe I did something wrong here?
nbl_adata: ad.AnnData
sc.pp.neighbors(
    nbl_adata,
    n_neighbors=15,
    key_added="sc_neighbors",
    transformer=AnnoyTransformer(n_neighbors=15, n_trees=100),
)
sq.gr.spatial_neighbors(
    nbl_adata, spatial_key="spatial", library_key="region", coord_type="generic", key_added="sq_spatial", n_neighs=15
)

nbl_adata.obsm["X_msPCA"] = MultispatiPCA(n_components=2, connectivity=nbl_adata.obsp["sq_spatial_connectivities"]).fit_transform(nbl_adata.X)

sc.pp.neighbors(
    nbl_adata,
    n_neighbors=15,
    key_added="msPCA_neighbors",
    use_rep="X_msPCA",
    transformer=AnnoyTransformer(n_neighbors=6, n_trees=100),
)

sl.spatialleiden(
    nbl_adata, # All fields of view
    layer_ratio=1.8,
    directed=(True, False),
    latent_distance_key="msPCA_neighbors_connectivities",
    spatial_distance_key="sq_spatial_connectivities",
    key_added="spatialleiden",
)

...

sl.spatialleiden(
    nbl_fov, # subset the AnnData on a single Field of View
    layer_ratio=3,
    directed=(True, False),
    latent_distance_key="sc_neighbors_connectivities",
    spatial_distance_key="sq_spatial_connectivities",
    key_added="spatialleiden",
)

For the following 2 figures I have subset on a cell type and ran the clustering algorithm on just those cells. I've plotted the FOV with the numerical spatialleiden clusters, and the other cell types present in the FOV.
One FOV after running SpatialLeiden across all FOVs

One FOV after running SpatialLeiden on just this FOV

Hi, thanks for interest in our package!
Multi-sample integration/clustering is definitely an interesting application. Unfortunately, I haven't had the time to look into that so far, but we will investigate it during SpaceHack in december. So maybe there will be more info after that.

Some comments:

that region you are analyzing looks really small. We have only really tested the SpatialLeiden package on larger datasets
it could be that there might be strong batch effects across your samples. Check the distribution of transcripts in each FOV to make sure they are comparable, and maybe remove any that seem to be FOV specific
the second image looks like the layer_ratio is too high. It reminds me when we set the value to very high in our Figure S4 of the biorxiv

as Niklas says, watch this space! We will be looking into multi-sample support next week!

@naveedishaque Hi, thanks for your feedback!

that region you are analyzing looks really small. We have only really tested the SpatialLeiden package on larger datasets

Yeah my images are rather small, I have a set of 107 FOVs with an X, Y size of 1024 $\times$ 1024.

it could be that there might be strong batch effects across your samples. Check the distribution of transcripts in each FOV to make sure they are comparable, and maybe remove any that seem to be FOV specific

I"ll give this a try.

the second image looks like the layer_ratio is too high.

I'll try lowering the layer_ratio down.


Changed next() approach to iterate DataLoader

First of all, thank you for your amazing book (and repository). Lot of concepts are extremely well explained and easy to understand.
Previous chapter used next(iter(DL)), however, chapter 5 uses iter(DL).next() which throws the following error:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
Cell In[149], line 1
----> 1 images_batch, labels_batch = iter(val_loader).next()
      2 logits = sbs_cnn1.predict(images_batch)

AttributeError: '_SingleProcessDataLoaderIter' object has no attribute 'next'

I don't know if that function was implemented in previous PyTorch versions, however with PyTorch 1.13, it doesn't work anymore.

Note: Python 3.8.10, jupyterlab 3.5.0

Hi @scmanjarrez
Thank you for your kind words, and for spotting and fixing this issue :-)
Best,
Daniel


configuring via environment variables does not work

regarding #73 i configured the container with an environment variable
URL=https://example.com/

After startup (removed old container) the generated links on the page (tags, rss) still show http://localhost:2368.
Also the startup messages say
Your blog is now available on http://localhost:2368/

executing a shell in the container shows the environment variable though
# env
GHOST_INSTALL=/var/lib/ghost
NODE_VERSION=6.11.2
HOSTNAME=39c60a570f9e
TERM=xterm
VIRTUAL_PORT=2368
NPM_CONFIG_LOGLEVEL=warn
GHOST_VERSION=1.5.0
GHOST_CONTENT=/var/lib/ghost/content
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
PWD=/var/lib/ghost
NODE_ENV=production
URL=https://example.com/

@smoebody It does work - it's a bit weird though, the environment variables have to be lowercased, e.g. url=https://example.com. Sorry for the confusion!

Whas about to type the same thing! The env stuff can be found here:
https://docs.ghost.org/docs/config#section-running-ghost-with-config-env-variables
(All lower-case everywhere...)

thanks for clarifying. i now can confirm that it works. One should correct the ghost docs at https://docs.ghost.org/docs/config#section-running-ghost-with-config-env-variables. The examples given there are misleading.


refactor version metadata fetching

motivation: move the version metadata fetching closer to download of archive so we can reuse this data to explose publisher information more easily
changes:

refactor TOFO and signature validation utilities to take a provider of metadata instead of dependending directly on the client
refactor archive downloading to first fecth the version metadata (cached)
update call sites and tests

cc @yim-lee

@swift-ci smoke test

@swift-ci smoke test

I guess self-hosted are now expected to fail?

Correct, until we have 5.7 or newer swift.


Upgrade Resource & Storage SDK to T2

A couple of changes to implement this:

list() now uses an PagedAsyncIterableIterator instead of Promise<IPartialList<T>.  This means I can't just do an await list() anymore, which is why I have the helper function to help load all of the resources.  Might be worth to do a loadByPage(numOfResources) helper function later too.
All T2 packages use a new type of credential called coreAuth.TokenCredential.  This uses getToken rather than signRequest so I made a type compatible with both so createAzureClient could be used with both T1 and T2 ManagementClients.
They changed how to import the Models now to import individual models rather than the entire Model namespace.

Tested this with Functions Create (Advanced) and Container Apps.  It still lists and creates resource groups and storage accounts, so I think everything is working?  Container Apps uses the new T2 appservice package and those seem to be working as well, meaning the credentials are compatible with T1 and T2 azure sdk packages.

Changed quite a bit so I won't honor these previous approvals 😅
Relies on this PR https://github.com/microsoft/vscode-azure-account/pull/418/files


#592 feat(container-details): container details docker attach

This PR adds the possibility to attach to a running container.
Closes #592

Hi @mrdrogdrog

Hi!

Thanks a lot for the contribution, good job ! My first review will only be related to functional feedback and UI/UX.

:blush:

First, I've discovered a few issues with the console/attach view:

The breadcrumb and the user profile/logout actions are not displayed anymore

fixed!

The attach console behaves inconsistently with some containers, see an example of a failing wordpress container below:

Well.. The problem is, that the container's TTY is sized only on term init, but the terminal uses the full width for writing.
We could:

Set a fixed width for TTY and terminal
Only resize the terminal on init and keep the size
Resize the TTY after a terminal resize

Obviously 3 would be the best choice and I try to fix this but I still have trouble with the terminal size.
The bug already existed before I made my changes. You can replicate it on the current master-branch.
Maybe it would be better to open another issue for this.

Another point regarding UX/UI, we're trying to keep Portainer UX/UI really simple to use and by having these two features (exec/attach) in the same view I think that it can be confusing for new Docker users hence I'd rather have these 2 features separated (but leveraging the same view/controller) as pointed in #592 (comment) (see mockups).

I split the console view into two views.

Obviously 3 would be the best choice and I try to fix this but I still have trouble with the terminal size.

The bug already existed before I made my changes. You can replicate it on the current master-branch.
Maybe it would be better to open another issue for this.

Agree, let's work on that in another issue.
Thanks for the update, i'll review it soon.

I'm aware of both points.
For the first one I wasn't sure if this extra click is okay or not. (extra function: create a container that is not interactive or has no TTY and then try to attach.)
I'm still looking for a solution for the second one. Docker decideds when to send text for the terminal. I don't want to send an keystroke on connect because this could lead to unexpected behaviour in the attached application.

I checked the docker cli code but can't find anything that seems like a "redraw the last line" command. Any idea?

(extra function: create a container that is not interactive or has no TTY and then try to attach.)

If the container is not attachable, the attach request should fail and that's ok behavior.
In that case we can simply display an error notification in the view with an empty view.
Regarding your second point, could be related to attach size?
Hint: https://github.com/docker/cli/blob/master/cli/command/container/attach.go#L173

You CAN attach to a container that is not interactive (no input) or has no tty (no output). It's not very smart, but possible.
The current code shows a warning.

(extra function: create a container that is not interactive or has no TTY and then try to attach.)

If the container is not attachable, the attach request should fail and that's ok behavior.
In that case we can simply display an error notification in the view with an empty view.
Regarding your second point, could be related to attach size?
Hint: https://github.com/docker/cli/blob/master/cli/command/container/attach.go#L173

I had the same idea. Let me check this.

You CAN attach to a container that is not interactive (no input) or has no tty (no output). It's not very smart, but possible. (Tested with my local docker cli)
The current code shows a warning.

Right, then let's display this panel with a single warning text if the container is not attachable or the red error message if the container is not running. Do not display the attach button.

You CAN attach to a container that is not interactive (no input) or has no tty (no output). It's not very smart, but possible. (Tested with my local docker cli)
The current code shows a warning.

Right, then let's display this panel with a single warning text if the container is not attachable or the red error message if the container is not running. Do not display the attach button.

Am I getting this right? You want to deny the attach feature if the tty or interactive flag is missing? Even if the cli allows that?

Actually, you're right, we want to let the user attach to the process even if not interactive/no tty set. Just display these messages inside the attach panel.
My main point here is to automatically attach to the console to avoid an extra click.

Auto-Attach if container is running: check

@mrdrogdrog awesome, looks a lot better. We'll discuss the UI/UX internally to see if everyone is ok with this so far.
Regarding the "redraw last line" problem, I believe it happens after attaching/disattaching and re-attaching again so the resize could be the problem here.

I just fixed the resize problem and will commit it soon.

IIRC the $timeout was used to wait for a specific amount of time before creating the terminal or resisizing the TTY otherwise it was causing an error.
It might not be required anymore though. I'll do a backend review soon.

Beautiful work @mrdrogdrog
Thanks :+1:


Dependency on OculusSampleFramework in many classes

Dependency on "using OculusSampleFramework" causes errors if a project sets up an assembly structure where the only dependency is on the Oculus.VR dll.
It would be ideal to remove a dependency from Oculus' example code, and instead depend directly on the SDK.

Thanks @provencher - this initial setup is build on top of OculusSampleFramework - so might not be easiest to reuse yet. I'll get rid of that as I move forward with other project.

@provencher that's now also closed and merged into master, the setup is slightly different - you can find details in updated readme


Cannot install Magento 2.1.4 with plugin enabled

With the cloudflare magento plugin required, a fresh installation of Magento 2.1.4 fails.
Error message: (vagrant/puppet encoded)
==> default: Notice: /Stage[main]/Magento/Exec[Install Magento]/returns: Running schema recurring...PHP Fatal error:  Uncaught Error: Cannot instantiate interface CF\Integration\IntegrationInterface in /var/www/Magento/vendor/magento/framework/ObjectManager/Factory/Dynamic/Developer.php:73
==> default: Notice: /Stage[main]/Magento/Exec[Install Magento]/returns: Stack trace:
==> default: Notice: /Stage[main]/Magento/Exec[Install Magento]/returns: #0 /var/www/Magento/vendor/magento/framework/ObjectManager/ObjectManager.php(71): Magento\Framework\ObjectManager\Factory\Dynamic\Developer->create('CF\\Integration\\...')
==> default: Notice: /Stage[main]/Magento/Exec[Install Magento]/returns: #1 /var/www/Magento/vendor/magento/framework/ObjectManager/Factory/AbstractFactory.php(126): Magento\Framework\ObjectManager\ObjectManager->get('CF\\Integration\\...')
==> default: Notice: /Stage[main]/Magento/Exec[Install Magento]/returns: #2 /var/www/Magento/vendor/magento/framework/ObjectManager/Factory/Dynamic/Developer.php(53): Magento\Framework\ObjectManager\Factory\AbstractFactory->resolveArgument(Array, 'CF\\Integration\\...', NULL, 'integration', 'CloudFlare\\Plug...')
==> default: Notice: /Stage[main]/Magento/Exec[Install Magento]/returns: #3 /var/www/Magento/vendor/magento/framework/ObjectManager/Factory/Dynamic/Developer.php(82): Magento\Framework\ObjectManager\Factory\Dynamic\Developer->_resolveArguments('CloudFlare\\Plug...', Array, Array)
==> default: Notice: /Stage[main]/Magento/Exec[Install Magento]/returns: #4 /var/www/Magento/vendor/magento/framework/Obj in /var/www/Magento/vendor/magento/framework/ObjectManager/Factory/Dynamic/Developer.php on line 73

Hi,
Are you in development or production mode?
What bin/magento commands did you run beforehand?

So this is happening as part of our deployment process.  So no Magento commands have been run, there's no var files, etc.
The error occurs during a setup:install.
MariaDB 10, PHP 5.6
It was also occurring during a setup:upgrade on the server, but that's harder to get a good reproducible case for.

I think this is a dupe or at least related to https://github.com/cloudflare/Cloudflare-Magento/issues/34
I'm looking into these issues next week.  I'll leave this open and update when I know more.
I'm able to reproduce though.

I'm having this same issue as well. I can't get cloudflare to work with magento 2.1.4

Fixed in https://github.com/cloudflare/Cloudflare-Magento/pull/47
Can you upgrade to 1.1.1 and let me know if this is resolved please?


Changes in Romajization between 0.2.4 and 1.0.0

With Kuroshiro 1.0.0, this script
const Kuroshiro = require( "kuroshiro" );
const KuromojiAnalyzer = require("kuroshiro-analyzer-kuromoji")
async function main() {
	const kuroshiro = new Kuroshiro();
	await kuroshiro.init(new KuromojiAnalyzer());
	console.log(await kuroshiro.convert('樋口, 直人', {to: 'romaji'}))
}
main()

gets me higuti, naoto, but under 0.2.4, this
const kuroshiro = require("kuroshiro");
kuroshiro.init(function (err) {
    // kuroshiro is ready
    var result = kuroshiro.convert('樋口, 直人', {to: 'romaji'})
    console.log(result);
});

gets me higuchi, naoto.

if I'd have to guess, it may have something to do with wanakana being replaced.

The canonical romanization system was not imported in current version. In the upcoming 1.1.x version of kuroshiro, three kinds of romanization systems will be added, which are nippon-shiki, passport-shiki, hepburn-shiki. You could choose one of them you like.

Is there a time line for 1.1? And which one of these would give me higuchi?

It will be released this week if all goes well. And you could use hepburn-shiki to get higuchi. Here is a reference webpage to check the difference between various romanization systems.

That was fast! Works as expected!


HTTP section - Privoxy install instructions

Hi,
I'm the OS X maintainer for Privoxy. The Homebrew recipe for Privoxy installation is weak in a number of regards, chief among which is that it leaves Privoxy running as root which is of course an unnecessary security risk. The supported installer (available at http://sourceforge.net/projects/ijbswa/files/Macintosh (OS X)/) does not suffer the weaknesses of the Homebrew recipe (it is also as easy to uninstall and includes complete instructions for configuration and obtaining support). Please could you consider altering the HTTP section of your instructions to point readers to this installer instead of using Homebrew?
Separately, many thanks for creating this guide - I've been reading through it with great interest!
Regards,
Ian Silvester

Are you sure Homebrew leaves privoxy running as root? A quick test shows the launch agent is loaded and job is run as the primary user. This point was brought up in https://github.com/drduh/OS-X-Security-and-Privacy-Guide/issues/38. Anyway, I'm glad to add instructions for a manual installation of privoxy as well.
Thank you

You're right, it runs as the installing user not root.
Still and all, my installer creates a non-privileged, non-interative user and group to run as, offering much better security isolation and audit; should that account be compromised the attacker has access to neither an end user account nor the UI. I figured since that was the driving aim of your guide that it'd be a worthwhile improvement to use the packaged installer instead of the Homebrew recipe.
It has the side benefit of course that I can offer better support to end users since I'm intimately familiar with the supported installation package.
I hope you'll consider these good reasons to update the guide.
Cheers,
Ian
On October 29, 2015 10:56:16 AM EDT, drduh<EMAIL_ADDRESS>wrote:

Are you sure Homebrew leaves privoxy running as root? A quick test
shows the launch agent is loaded and job is run as the primary user.
This point was brought up in
https://github.com/drduh/OS-X-Security-and-Privacy-Guide/issues/38.
Anyway, I'm glad to add instructions for a manual installation of
privoxy as well.
Thank you

Reply to this email directly or view it on GitHub:
https://github.com/drduh/OS-X-Security-and-Privacy-Guide/issues/65#issuecomment-152205430

Sourceforge is blocked by uBlock's default ruleset due to their recent badware policy, and the guide recommends uBlock, so it might be a bit contradictory to link to SF. Any alternative host for the installer?

Ha, ironic! How the mighty are fallen! I don't have an alternate location as yet; we have been considering a move to Github as it goes. I'll tell the other maintainers about uBlock's new rule and see if we can accelerate the process.
Thanks for the tip,
Ian
On October 29, 2015 11:30:44 AM EDT, Dominic Evans<EMAIL_ADDRESS>wrote:

Sourceforge is blocked by uBlock's default ruleset due to their recent
badware
policy, and the guide recommends uBlock, so it might be a bit
contradictory to link to SF. Any alternative host for the installer?

Reply to this email directly or view it on GitHub:
https://github.com/drduh/OS-X-Security-and-Privacy-Guide/issues/65#issuecomment-152215738

Thank you for the suggestion, Ian. Let me know if you spot any other issues.

Hi again Dominic,
So there is now an alternative download URL for the official Privoxy OS
X installation packages. Please find them here:
http://silvester.org.uk/privoxy_installers/OSX/
Cheers,
Ian
On 2015-10-29 11:30, Dominic Evans wrote:

Sourceforge is blocked by uBlock's default ruleset due to their recent
badware https://duckduckgo.com/?q=sourceforge+installers+malware+mac
policy, and the guide recommends uBlock, so it might be a bit
contradictory to link to SF. Any alternative host for the installer?
—
Reply to this email directly or view it on GitHub
https://github.com/drduh/OS-X-Security-and-Privacy-Guide/issues/65#issuecomment-152215738.

--
My PGP public key
http://diem.serveftp.net:8080/IanSilvesterPGPPublicKey.asc.

Hi once again!
I've now modified that URL to:
http://silvester.org.uk/privoxy/OSX/
Also, we're planning to start offering this as an official
non-Sourceforge mirror, so the official project page at www.privoxy.org
will shortly also point here, so you can point folks in this direction
with no worries ;o)
I do hope you decide to update your guide - like I say the supported
installer is a more secure option that Homebrew, and that's the guide's
aim after all!
Cheers,
Ian
On 2015-10-31 18:18, Ian Silvester wrote:

Hi again Dominic,
So there is now an alternative download URL for the official Privoxy
OS X installation packages. Please find them here:
http://silvester.org.uk/privoxy_installers/OSX/
Cheers,
Ian
On 2015-10-29 11:30, Dominic Evans wrote:

Sourceforge is blocked by uBlock's default ruleset due to their
recent badware
https://duckduckgo.com/?q=sourceforge+installers+malware+mac
policy, and the guide recommends uBlock, so it might be a bit
contradictory to link to SF. Any alternative host for the installer?
—
Reply to this email directly or view it on GitHub
https://github.com/drduh/OS-X-Security-and-Privacy-Guide/issues/65#issuecomment-152215738.

--
My PGP public key
http://diem.serveftp.net:8080/IanSilvesterPGPPublicKey.asc.

--
My PGP public key
http://diem.serveftp.net:8080/IanSilvesterPGPPublicKey.asc.

Please have a look at the updated instructions at https://github.com/drduh/OS-X-Security-and-Privacy-Guide#http and let me know if there's any issues.

Hi Dominic,
Thanks for going ahead with the change, it's much appreciated. The only
alteration I'd suggest, for the sake of full disclosure, is to state
that the installation created by the signed package is more secure than
the Homebrew one and attracts full support from the Privoxy project.
Cheers,
Ian
On 2015-11-04 16:42, drduh wrote:

Please have a look at the updated instructions at
https://github.com/drduh/OS-X-Security-and-Privacy-Guide#http and let
me know if there's any issues.
—
Reply to this email directly or view it on GitHub
https://github.com/drduh/OS-X-Security-and-Privacy-Guide/issues/65#issuecomment-153875368.

--
My PGP public key
http://diem.serveftp.net:8080/IanSilvesterPGPPublicKey.asc.

Sorry! Didn't realise there were multiple folks involved on the project.
The clarification is perfect.
Thanks again,
Ian
On 2015-11-04 16:52, drduh wrote:

I'm not Dominic, but have gone ahead and made your suggested
clarification.
—
Reply to this email directly or view it on GitHub
https://github.com/drduh/OS-X-Security-and-Privacy-Guide/issues/65#issuecomment-153879912.

--
My PGP public key
http://diem.serveftp.net:8080/IanSilvesterPGPPublicKey.asc.


[Bug] If enterprise is specified, it shouldn't set a default value for organization

What happened?
When I specify the enterprise input, I am expecting to see enterprise level metrics. So organization should be "" unless otherwise specified.
Currently, organization defaults to ${{ github.repository_owner }}
Workaround is to set
with:
  enterprise: <enterprise-slug>
  organization: ""

Relevant log output
No response
Contact Details
No response

Today
enterprise > team > organization
https://github.com/austenstone/copilot-usage/blob/9c819bed3de4a3abca6a34ddba2f86b065ab50f9/src/run.ts#L92-L116
I do this so we can use default values
https://github.com/austenstone/copilot-usage/blob/9c819bed3de4a3abca6a34ddba2f86b065ab50f9/action.yml#L15-L18
If you're running the action in the organization you're trying to get metrics from you simply can call it as
      - uses: austenstone/copilot-usage@v4
        with:
          github-token: ${{ secrets.TOKEN }}

If you want enterprise level metrics pass the enterprise
      - uses: austenstone/copilot-usage@v4
        with:
          github-token: ${{ secrets.TOKEN }}
          enterprise: avocado-corp

If you want team metrics pass team
      - uses: austenstone/copilot-usage@v4
        with:
          github-token: ${{ secrets.TOKEN }}
          team: avocado-team

If you want org metrics from a seperate org from the one you're running in pass the org
      - uses: austenstone/copilot-usage@v4
        with:
          github-token: ${{ secrets.TOKEN }}
          organization: callmegreg-demo-org

HI @austenstone, I'm basically running this GHA in my personal account/repo but pulling metrics that belong to a different enterprise and organization.
So every assumption to use ${{ github.repository_owner }} fails for me, as ${{ github.repository_owner }} does not belong to the enterprise_slug that I'm using.
So with this code
      - uses: austenstone/copilot-usage@v4
        with:
          github-token: ${{ secrets.TOKEN }}
          enterprise: avocado-corp

I am getting this error:
Run austenstone/copilot-usage@v4
Fetching Copilot usage for enterprise ***
Fetched Copilot usage data for 1 days (2024-[10](https://github.com/raffertyuy/github-copilot-metrics/actions/runs/11338106911/job/31530784868#step:2:11)-14 to 2024-10-14)
Fetching Copilot details for organization raffertyuy
HttpError: Not Found - https://docs.github.com/rest/copilot/copilot-user-management#get-copilot-seat-information-and-settings-for-an-organization
    at /home/runner/work/_actions/austenstone/copilot-usage/v4/node_modules/@actions/github/node_modules/@octokit/request/dist-node/index.js:[12](https://github.com/raffertyuy/github-copilot-metrics/actions/runs/11338106911/job/31530784868#step:2:13)5:1
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at run (/home/runner/work/_actions/austenstone/copilot-usage/v4/src/run.ts:128:1)

But this workaround works:
      - uses: austenstone/copilot-usage@v4
        with:
          github-token: ${{ secrets.TOKEN }}
          enterprise: avocado-corp
          organization: ""

You should use either the enterprise input or the organization input but not both at the same time.


fix: Move functions to admin operations

What kind of change does this PR introduce?
Move functions to admin operations

But these are not admin functions.

not applicable


IDENTITY-6487- UI Improvement related to Main/Identity Provider area

Section - Add New Identity Provider

Help text for the description. Instead of saying "A meaningful description about the identity provider" say something like "Provide a meaningful description about the identity provider
Description for the Alias ("If the resident identity provider is known by an alias at the federated identity provider specify it") there should be a comma after the word "provider"

Section - Claim Configuration ->Advanced Claim Configuration

When you click the plus sign to add the claim without selecting a claim, it gives the error "Add Valid Attribute". A more meaningful error message is better. Ex: Select a valid claim URI
Select a claim URI for provisioning and click Add (+ sign). The "Delete" is not accompanied by a "Bin" image.

Section - Role Configuration

Identity Provider OutBound Provisioning Roles: The term "OutBound" should be "Outbound"
The description "Enter the identity provider's comma separated provisioning roles name" . Is should be "role names"

Section - Federated Authenticators: SAML 2 Web SSO Configuration

If selecting Enable SAML2 Web SSO makes the default one clicked, it should not look selectable. Can we make it with lighter font?
Select Mode description question mark should a space backward.
Descriptions Enable SAML2 Web SSO and Enable Assertion Signing, better make the term consistent "specify vs specifies"
Description "Specifies if logout/single Logout is enabled for this identity provider", "Logout" should be "logout" (Lower case "L")
"Url" should be "URL" everywhere
The question marks in front of the descriptions should be aligned correctly.

Area - WS-Federation C

"STS" should be uppercase in the description

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


📎  Implement media-feature-name-no-unknown

Description
Implement media-feature-name-no-unknown

[!IMPORTANT]

Please skip implementing options for now since we will evaluate users actually want them later.
Please ignore handling extended CSS language cases such as sass and less.
Please skip custom function since we haven't syntax model yet.

Want to contribute? Lets you know you are interested! We will assign you to the issue to prevent several people to work on the same issue. Don't worry, we can unassign you later if you are no longer interested in the issue! Read our contributing guide and analyzer contributing guide.

I'm interested in tackling this issue 🦀

I'd like to reference media features like https://github.com/stylelint/stylelint/blob/main/lib/reference/mediaFeatures.mjs#L56
I searched project directories but I couldn't find.
Is there a reference? Or, do I have to make?

You should define them here.
https://github.com/biomejs/biome/blob/main/crates/biome_css_analyze/src/keywords.rs


avahi-alias service fails

On reboot:
●<EMAIL_ADDRESS>- Publish pioreactor.local as alias for pioreactor1.local via mdns
   Loaded: loaded<EMAIL_ADDRESS>enabled; vendor preset: enabled)
   Active: failed (Result: exit-code) since Mon 2021-02-08 20:37:03 GMT; 1h 4min ago
  Process: 354 ExecStart=/bin/bash -c /usr/bin/avahi-publish -a -R pioreactor.local $(avahi-resolve -4 -n pioreactor1.local | cut -f 2) (code=exited, status=1
 Main PID: 354 (code=exited, status=1/FAILURE)

Feb 08 20:36:57 pioreactor1 systemd[1]: Started Publish pioreactor.local as alias for pioreactor1.local via mdns.
Feb 08 20:37:03 pioreactor1 bash[354]: Failed to resolve host name 'pioreactor1.local': Timeout reached
Feb 08 20:37:03 pioreactor1 bash[354]: Bad number of arguments
Feb 08 20:37:03 pioreactor1 systemd[1]<EMAIL_ADDRESS>Main process exited, code=exited, status=1/FAILURE
Feb 08 20:37:03 pioreactor1 systemd[1]<EMAIL_ADDRESS>Failed with result 'exit-code'.

I added a retry block, will succeed eventually

I put an example for someone who has the same trouble:
[Unit]
Description=Publish %I as alias for %H.local via mdns

[Service]
Type=simple
ExecStart=/bin/bash -c "/usr/bin/avahi-publish -a -R %I $(hostname -I | cut -f1 -d' ')"
Restart=on-failure
RestartSec=5s

[Install]
WantedBy=multi-user.target

and enable service with instance name like;
sudo systemctl enable --now<EMAIL_ADDRESS>

Nice suggestion @HiroTamori!


Clarify PathFingerprintCompareStrategy's includeAdded parameter

During https://github.com/gradle/gradle/pull/9407/files#diff-63a979673a1151b2ff2bab58f4f97857R69 we saw that includeAdded doesn't always make sense for every strategy, e.g. for Normalized we've hardcoded it to true.
We should investigate how to model the two paths in a meaningful way, without confusing boolean parameters that alter the client code's behavior (maybe we're hiding two independent features that should be split instead).

includeAdded == false is for detecting output changes between builds. Maybe we should do this separately completely, and detect added outputs between builds, and use these changes to detect stale output files.

If we implement https://github.com/gradle/gradle/issues/9034, we can remove the flag completely since we won't use fingerprints for the outputs any more.

Fixed by removing the parameter altogether in https://github.com/gradle/gradle/pull/9860.

That's the best explanation!


New HowTo calling the search API from bash

Added a new markdown file and updated the main HowTo index.

 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.


Update tests and upload documentation for JOSS

Resuming what was left off on #18

Extensive tests for each function will be added in another PR


Append Moti chris_culvert_id to PSCIS spreadsheet comments

May as well tie it together.
Be sure to distinguish between the major structures by appending column name at same time
(ex. chris_culvert_id: 122342 or chris_hwy_structure_road_id = 2434).
Looks like all the chris_culvert_id are high (greater than 1,000,000 and chris_hwy_structure_road_id are less than 10,000 to inform left_join case_when (can query db with select max(chris_hwy_structure_road_id) and select min from whse_whatever.chris_hwy_structure_road_id` to confirm though)

What happens if there is more than one moti culvert? If there is more than one pipe at a crossing we only use one modelled id, whereas they have a point for each pipe. Is it ok to just have one moti id in the pscis spreadsheet comments field? Even if there is more than one pipe. But in our reporting we can reference all moti chris culvert ids that are present in our site memos.
I QAd the moti culvert fields in Q. There was one or two that were incorrect or missing an id. And there were two crossings (modelled id 3701558 and 3700010) that were actually highway structures so I changed the chris_culvert_id to a chris_hwy_structure_road_id

If there are more than one it could go in another mutated column and added to the comments programmatically. We can make a note to add another colum to the form in dff too…
min the comments it should be easy for the reader to understand what a chris_culvert_id refers to so it could say something more like “Ministry of Transportation chris_culvert_id: 12345”

There is actually only two crossings in peace that might have more than one moti culvert.

modelled crossing 3700007 : there are two pipes right next to each other seen in assessment, but in Q there are two moti culverts that are far apart so not sure if it's the same one

modelled crossing 3702044 : only one culvert found during assessment but apparently there is supposed to be 2

I'm just going to leave it as is. But I will update the comments so it says "Ministry of Transportation".


Add Cycle.Express

Dear @cycle-express,
In order to potentially merge your code in this open-source repository and therefore proceed with your contribution, we need to have your approval on DFINITY's CLA.
If you decide to agree with it, please visit this issue and read the instructions there. Once you have signed it, re-trigger the workflow on this PR to see if your code can be merged.
— The DFINITY Foundation


cleaning up dictionaries in timeseriesinst and supplyinst

This issue can be closed when this is addressed: https://github.com/ergs/d3ploy/pull/157#issuecomment-445984651

closed with https://github.com/ergs/d3ploy/pull/157


Please invite me to the community

Name
Santiago Urrego Botero
Additional Context
I heard Infraform Community from my friend, I'd love to be a core contributor of repositories on Infraform Community.
Thank you.

Invitation sent to join the GitHub Organization. Welcome to Infraform 🎉Please check your mail to accept the invitation sent by GitHub! 🤖About usInfraform is a remarkable open-source solution tailored for serverless platforms, aimed at empowering creators to craft their infrastructure solutions. 💻 We're thrilled to welcome you into our vibrant community! ✨ We eagerly anticipate your valuable contributions! 🤓 Don't forget to explore our community on Github and connect with us on LinkedIn 💜 Feel free to spread the word about our community to like-minded individuals interested in development, DevOps, technology, and collaborative contributions. 👉💜👈


Create base client/server Rust projects

oda-api-ri-rust version
latest
OS version
All
Description

Motivation
In Rust RI should be implemented client/server projects.
Proposal
Create base client/server projects over cargo:

$ cargo new client
$ cargo new server

Current behavior
Missed client/server projects.
Expected behavior
The client/server projects are added.
Config
Default configuration which implemented by cargo.

Client and server builds implemented.


adds missing abide features without splitting inputs into types (code…

… reduction)
This should fix #7424 #7406 #7357 #7345 #7304 #7230 #7228 #7226 #7209 #7196 #7127
adds support for html5 input types, selects (multi), textareas
adds support for html5 elements through patternLib
adds support for pattern and data-pattern through patternLib and own patterns
everything is tested in a testcase, for me it works without splitting the types.
For those input elements (email..) which return a el.type for example email and a pattern exists in the patternLib it will be validated with this.
what have i done?

fix the default case of requiredCheck because is(':emtpy') checks for innerhtml elements in inputs and there aren't
remove not used

textInput = $form.find('input[type="text"]'),
checkInput = $form.find('input[type="checkbox"]'),

inputcount improvement
submit form if valid
pattern support
in html5 type overrides pattern
in my abide i handle it like this

Data-pattern overrides
Pattern overrides
type

so you can flexible choose whatyou would like to use
even a input and for example type email, or pattern, without required, can be validated if there is something typed in.
i tested the following elements
input types and elements
hidden
text
password
color
date
datetime
datetime-local
email
month
number
range
search
tel
time
url
week
select
select multiple
textarea
radio
checkbox
reset
submit
if you want to i cann paste my test html
please have a look maybe this fixes the issues without splitting the input types and bloat the code

validator can also be added later

See #7407
We want to push out Abide with the 6.1 release, I could use some community support in making sure all the pieces are there beforehand.

maybe for 6.1 see #7481


Copy over new version of Milo's geo-based locales

See https://github.com/adobecom/milo/pull/915 and https://github.com/adobecom/milo/pull/938
Test URLs:

Before: https://main--dc--adobecom.hlx.page/?martech=off
After: https://hparra-patch-4--dc--adobecom.hlx.page/?martech=off

Codecov Report

Merging #279 (7943e11) into stage (f201892) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##            stage     #279      +/-   ##
==========================================
- Coverage   99.46%   99.46%   -0.01%
==========================================
  Files          26       26
  Lines        2229     2224       -5
==========================================
- Hits         2217     2212       -5
  Misses         12       12

Impacted Files
Coverage Δ

acrobat/scripts/scripts.js
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@hparra  we have a minor conflict
want to resolve and then we can merge this. thank you!


handling of 'error' event for subscriptions

NodeJS's EventEmitter treats emitting 'error' as a special case - when
there are no listeners, an Error object is thrown on behalf of the
EventEmitter.  Due to this behavior, an unhandled exception was being
thrown from blpapi-wrapper.ts that wasn't allowing the
SessionTerminated event being propagated properly to the users of the
API.
This change now has subscriptions listen for the 'error' event to
properly update the internal state of outstanding subscriptions.
Closes #39

Updated PR based on latest comments from @yupinghu and @ericlu88.

PR updated based on out-of-band discussion with @ericlu88 about logging and sending an error back to the client.

LGTM.

Squashed commits


[TF-TRT] Introduce _remove_native_segments function to remove native segments in TF-TRT Graph

This function parses through the TF-TRT graph and deletes native segment nodes constructed for falling back. Changes have also been made to TRTEngineOp to recognize when the loaded graph doesn't have _native_segment nodes and disallow loading the graph associated with the native segments.
To do: When native segment execution is disabled, we don't generate the native_segment nodes in the first place.
@tfeher and @DEKHTIARJonathan  please review.

@pavanimajety @tfeher instead of introducing an API, could we use for now: is_experimental_feature_activated defined here: https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/compiler/tensorrt/utils.py#L79.
I would like to avoid introducing an API and later changing it because it's not a fortunate design.
For now, let's keep it simple and avoid any API change. Once the feature is complete and properly tested, we'll think about what's the best way(s) to expose the feature to the end-user.
@pavanimajety what do you think about TF_TRT_EXPERIMENTAL_FEATURES=remove_native_segments ? If you have an idea that you prefer, it's fine with me. It doesn't really matter :)


Fix ScreenSpaceEventHandler when Cesium does not take up the entire browser window

A bug was introduced in #2408 that breaks mouse events completely when the Cesium canvas is not in the upper-left of the browser window.
The problem is that clientX and clientY are relative to the client area of the overall browser window, not the DOM element, even for listeners attached to a specific element.
https://developer.mozilla.org/en-US/docs/Web/API/MouseEvent.clientX

@bagnell can you review please?

I found a related bug in CameraEventAggregator.  Fix coming soon.

Additional discussion: https://groups.google.com/forum/#!topic/cesium-dev/Emt-CrGYulY

The related bug I mentioned above is separate from the issue mentioned on the forum, and is now fixed in this pull request.  CameraEventAggregator was receiving events from two different ScreenSpaceEventHandlers, and not accounting for the fact that they provide coordinates in two different coordinate systems (document versus canvas).

I know very little about the mouse handling code, but isn't canvas.getBoundingClientRect(); a performance bottleneck?  I'm pretty sure some browsers reflow/recalculate it every time it's called.  I think we cache results like this in other places and only compute it once at the beginning of the frame.  In this case it might not be an issue, but I just wanted to bring it up.
@shunter any thoughts here?

Any update?  We'll want to merge this this week so it makes 1.6.

This looks good to me. Just waiting on a response to @mramato's comment.

I found various references here and there that suggest a full layout is triggered by getBoundingClientRect.  We're already getting the size in viewer.resize() every frame, is it reasonable to pass that size down into the handlers somehow?

We're already getting the size in viewer.resize() every frame

Size, yes, but we're not calling getBoundingClientRect every frame.  That's good because we only need it on input, which happens less often than rendering.

is it reasonable to pass that size down into the handlers somehow?

Not really.  The top of the call stack is an input event, so we can't pass it down from there.  The best we could do is compute and cache the rectangle if it's undefined, and then reset it to undefined each frame.  I started trying to do this but it's ugly and likely to lead to bugs.  In particular, ScreenSpaceEventHandler has no notion of render frames and no dependence on Scene.  The scene could potentially reset its ScreenSpaceEventHandler(s) each frame, but what if the user creates their own?  I think that's somewhat common because a single ScreenSpaceEventHandler can only have a single handler per input event.  The user would need to remember to reset it each frame (or at least anytime the size/location of the canvas changes).  If they don't, they get weird bugs.
I just committed a change to reduce the number of calls to getBoundingClientRect a bit, so maybe that makes it more acceptable to everyone.
By the way, this pull request brings Cesium's input handling from "horribly broken" to "working but possibly not quite as optimal as it could be" so merge it already!  6 days, sheesh. :)

Agreed we should merge this before the release.

input, which happens less often than rendering.

With one important caveat: When you're moving the mouse, dozens of mousemove messages come in per frame.  That's the reason for the CameraEventAggregator, is to gather up those messages and limit activity to once per frame. In theory we could make less calls by moving the logic there, but in practice I don't know if additional re-layouts are actually caused by multiple calls in the same frame.
Anyway, I did a little performance testing of this vs master, and I couldn't spot the difference.  So @bagnell I think you should merge.

As @emackey stated, profiling doesn't show any performance issues (I tried IE, Firefox, and Chrome).


Properly size profile images

Describe the bug
The profile pictures of users as served by Github are bigger than what they appear i.e. 460x460
To Reproduce
Steps to reproduce the behavior:

Go to the list of users
Notice the size i.e. height and width of the profile pic.
Compare the size of the pic with the original one as served.
See error

Expected behavior
The profile pictures height and width should be as rendered on the page (i.e. 80x80)
Screenshots

This issue can not be fixed as the GitHub API does not support image transformations


Sent

Error transaction. Sent 5 $ . When checked transaction on Stellar said transaction not found

Error transaction. Sent 5 $ . When checked transaction on Stellar said transaction not found

Hey @MIKEKO48 ,
I hope your issue is resolved, but if not, I apologize for the delay. It looks like you're facing a transcript error, likely due to a system glitch.
I've created a support ticket (ID: STR7638). Please visit Steller Network Dapps Support and use the live chat icon at the bottom right to connect with a support agent for help.


jax.experimental.loops as a decorator

What do you think about a decorator that transforms code to use jax.experimental.loops?
Today you have to write this:
from jax.experimental import loops
def f(arr):
  with loops.Scope() as s:
    s.arr = arr
    for i in s.range(s.arr.shape[0]):
      s.arr = ops.index_update(s.arr, i, s.arr[i] + 2.)
      for _ in s.cond_range(i % 2 == 0):  # Conditionals as loops with 0 or 1 iterations
        s.arr = ops.index_update(s.arr, i, s.arr[i] + 1.)
    return s.arr

This would be more readable:
from jax.experimental import loops2
# This lets you run this code:
# ret = f.with_scope(loops.Scope())(*args, **kwargs)
@loops2.enable
def f(arr):
  for i in loops2.dynamic_loop(range(arr.shape[0])):
    arr = ops.index_update(arr, i, arr[i] + 2.)
    if loops2.dynamic_branch(i % 2 == 0):
      arr = ops.index_update(arr, i, arr[i] + 1.)
  return arr

The semantics of this transform are:

All local variable access is mirrored to $scope (or we can make it anonymous). Since there is a $ symbol in the name, it won't conflict with a user-defined variable.

On entry, we call: $scope.__dict__.update(locals())
Getting arr goes to $scope.x.
Setting/deleting arr goes to both x and $scope.x.
Variables referenced in another function are not stored at all:# Either of these would disable updating $scope.arr
send_to_other_thread(lambda: arr)
send_to_other_thread(lambda value: (arr := value))

We know when captured variables are modified by the decorated function, so we could also just instrument those.

All control flow becomes a giant switch statement that returns the next branch to run. Alternatively, we could translate for loops into something nice and make everything else a switch statement.

Questions:

Does loops.Scope() need to be eager? A lazy interface (locals() and set_locals(l) instead of set_local, get_local, and del_local), would be much easier to implement.
Should we default all loops to dynamic?
What should this code do?

for i in range(n):
  if not is_valid(i):
    break
else:
  x += n

try-break-finally, try-continue-finally, nested try-finally
Which should be captured? What if the functions can update the locals? Or globals?

f = loops2.enable((lambda x: lambda y: lambda z: x + y + z)(0))

I personally prefer the current API because it is more explicit, and there is somewhat less "magic" happening. I think it is useful for the user to have the mental model that only fields of the Scope object are being tracked.
We are also preparing a change to JAX to make it easier to keep track of state. I would rather wait for that to see how it is going to improve the ergonomics of loops.

Closing, as jax.experimental.loops is deprecated and will soon be removed.