When using Structured Outputs, only strings fields are marked as required

Bug Report
Overview
When using Structured Outputs, only strings fields are marked as required.  According to OpenAI documentation "all fields or function parameters must be specified as required".
When adding number or boolean fields, they are not marked as required and OpenAI API responds with an error.
Screenshots
Part of the input schema

Response from OpenAI:

They recently updated how schemas work on the backend that started breaking existing implementation.

Great!
Additionally, I'm missing the Description field. When I add the [Description("text... bla bla...")] attribute to the class, it doesn't appear in the object's description field. I hope this can also be fixed.

Great! Additionally, I'm missing the Description field. When I add the [Description("text... bla bla...")] attribute to the class, it doesn't appear in the object's description field. I hope this can also be fixed.

it's not an additional attribute, but part of the FunctionAttribute:
https://github.com/RageAgainstThePixel/OpenAI-DotNet/blob/b1aba5d1e8d35c698777519deedb4f9642a92dcd/OpenAI-DotNet/Common/FunctionAttribute.cs#L10
[Function(description: "blah blah")]
private void Function(){}

Look at this example in the OpenAI documentation:
https://platform.openai.com/docs/guides/structured-outputs#all-fields-must-be-required
Properties also contains descriptions to give more context.

Look at this example in the OpenAI documentation: https://platform.openai.com/docs/guides/structured-outputs#all-fields-must-be-required
Properties also contains descriptions to give more context.

I'm well aware.
FunctionPropertyAttribute also supports descriptions, as well as other things:
https://github.com/RageAgainstThePixel/OpenAI-DotNet/blob/b1aba5d1e8d35c698777519deedb4f9642a92dcd/OpenAI-DotNet/Common/FunctionPropertyAttribute.cs#L25
I've already fixed some of this recently just needs to get ported. Recently OpenAI updated their rules on this stuff and broke compatibility.
I simply disabled strict for the tool definitions generated by the built in function attributes, which should fix this issue.
If you provide your own schema definitions, then that is another story.

FWIW if you're using .net 9, then there is a json schema generator that should be compliant:
https://devblogs.microsoft.com/dotnet/system-text-json-in-dotnet-9/#json-schema-exporter

Thank you for the response. So, is there currently any way to provide attributes with descriptions to obtain a structured outputs?

Thank you for the response. So, is there currently any way to provide attributes with descriptions to obtain a structured outputs?

see last comment about .net 9 json schema exporter. Likely I'll remove/depricate the custom impl in this library once .net 8 EOL comes around.

Finally I was able to add my own schema. I did it using the library Newtonsoft.Json.Schema.
JSchemaGenerator generator = new()
{
DefaultRequired = Required.AllowNull,
GenerationProviders =
{
new StringEnumGenerationProvider(),
}
};
JSchema jSChema = generator.Generate(typeof(OpenAIStructuredOutput));
SetAdditionalPropertiesFalse(jSChema); // this method set AllowAdditionalProperties=false to all objects
var schema = jSChema.ToString();
var jsonSchema = new global::OpenAI.JsonSchema("OpenAIStructuredOutput", schema);
var chatRequest = new ChatRequest(
messages: messages,
model: MODEL_NAME,
temperature: TEMPERATURE,
responseFormat: ChatResponseFormat.JsonSchema,
jsonSchema: jsonSchema
);
ChatResponse chatResponse = Task.Run(async () => await OpenAIClient.ChatEndpoint.GetCompletionAsync(chatRequest)).Result;
var structuredOutput = JsonConvert.DeserializeObject(chatResponse.FirstChoice);


[backport-target] Backport test PR

To test backporting:

Update the v1.0-test tag with the changes you want to test (i.e., update the tag based on your topic branch)
Enter the following comment

@gitbot backport backport-target

Check the Actions menu above for the status of the backport GitHub action that is triggered; particularly, if the backport bot does not add status comments to this PR
If successful a new backport PR against the backport-target branch will be created.  Recommend simply closing that PR without merging it to preserve the pending change for a subsequent backport test
When you've completed your testing, please delete all of the comments you've added to this PR to help reduce clutter
Backport of #11

Test PR. Closing


smaller input data file

The input data file is ~19 GB which is large for comfortable development.
It would be great to produce a smaller file, say read the big one and take every 100th event and write the smaller one that we could use to ease the development.
@atrisovic Can you please look at that?

Thanks!  Still it runs "forever" in my modest 8 GB RAM laptop... I stopped it after 4 hours I think.  Could you make it even smaller?

Hi, yes, I created a new file called test_D2PimumuOS.root of only 100KB in the same repository.

Thanks, closing. We have a ~1G and a ~10G input file that are now sufficient for further testing. A box with 16G of RAM recommended to run Optimise.


Update blueprints to use orm-adapter/application.js instead of ormAdapter config property

The application always uses the MemoryAdapter, since that seems to be the default if none are found.

FWIW, I came across this after playing with the recent container refactor and found the same thing. However, after conversation with @davewasmer the intent was to move away from specifying the ormAdapter in config and instead declaring an ApplicationOrmAdapter that extends whichever one you want, which makes it align more with the other first-class concepts in Denali (serializers, parsers)

Makes sense. So the blueprints need updating.

I updated the issue title to reflect @acburdine's comment.


update unstake docstring

Since stack function removed now, this link will 404
I update link to point to Base.stack

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.12%. Comparing base (c826901) to head (2c07d1b).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #176      +/-   ##
==========================================
- Coverage   88.39%   84.12%   -4.27%
==========================================
  Files          15       13       -2
  Lines         603      567      -36
==========================================
- Hits          533      477      -56
- Misses         70       90      +20

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


added LinearSVR_Scale_PowerTransformer.ipynb

Required Changes:

Remove font size from metadescription cell
Preprocessing is wrong, you have utilized target label encoding in regression problem.


Add fast-path Union subclass for unions of two simple types, take 2

Motivation

This is a straight rebase of #4568; I don't think I can push to djudd's branch.  I have been working to get Stripe's codebase green, and assuming the last change or two lands, I think we could safely commit this.
Test plan

See included automated tests.

Stripe build link: https://go/builds/bui_MWAy1BjGsgySuj

(needed to fix up some rebase errors around test conflicts)


General changes

requested by @SmalsJulien
General changes
Intent —> One value set for ServiceRequest (order, proposal) and another for task (only order)
BeReferralPrescriptionNursing —> BeReferralServiceRequestNursing
all period must be BeYearMonthDay except the authoredOn dates.
latestDraft - supprimer car pas encore
device pour l’instant pour 1-care
requisition —> remove must support
reasonCode - 0..1 —> 1..1 ?

BeReferralPrescriptionNursing —> BeReferralServiceRequestNursing was already done in https://github.com/hl7-be/referral/commit/068a71daff0c5c855207adb11b9683d85a864169

reasonCode 0..1 according to https://github.com/hl7-be/referral/issues/219

@SmalsJulien ,
check capitals in VS et Vs.


CSS Injection Vulnerability

When using the DOMPurify library to sanitize HTML from malicious code here, the library doesn't sanitize CSS style tags. These can be used to change the website's appearance and possibly to pull off clickjacking attacks. - https://portswigger.net/web-security/clickjacking
PoC: https://flatgithub.com/makuga01/csv-test?filename=css.csv&sha=9cfecff99206e7a11716d14b48a2edb9c16ce0c1
I suggest adding using a DOMPurify config like this one:
const dompurifyConfig = {
  FORBID_TAGS: ['style', 'form'],
}
const sanitized = DOMPurify.sanitize(displayValue, dompurifyConfig);

I also added HTML forms to the forbidden tags as they can be used for creating a fake login screen

PR here: https://github.com/githubocto/flat-ui/pull/34


Just turn on a vpn and the ads won't load, or alternatively you could fork the project and remove the ads in the index.html


bad marshal data (unknown type code)

The checkpoint of your pre-train model  is not loading for me. There is an error message stating "bad marshal data". I think there is an inconsistency between the versions of my Keras package and yours. I am experimenting with your notbook on Google Colab. If you are able to help me with this error, it would be greatly appreciated!

Yes it's a Keras version error. One solution is to save the model weights and then use the load_weights method on a newly instantiated model with the same architecture. I hope I will get around to doing this soon but if you do let me know!


Files semi freezes up and doesn't show drives as well as shell right click menu

Description
There was some sort of glitch or crash and when files got back, the issues in the title cropped up. Those persisted through full restarts of the files app via taskmanager. Also right clicking in some folders full freezes the manager now.
2023-02-21 12:52:49.2966|WARN|ParseLinkAsync|Error HRESULT E_FAIL has been returned from a call to a COM component. 	System.Runtime.InteropServices.COMException (0x80004005): Error HRESULT E_FAIL has been returned from a call to a COM component.    at Vanara.PInvoke.Shell32.IShellLinkW.Resolve(HWND hwnd, SLR_FLAGS fFlags)    at Vanara.Windows.Shell.ShellLink.LoadAndResolve(String linkFile, SLR_FLAGS resolveFlags, HWND hWin, UInt16 timeOut)    at Vanara.Windows.Shell.ShellLink..ctor(String linkFile, LinkResolution resolveFlags, HWND window, TimeSpan timeOut)    at Files.App.Helpers.FileOperationsHelpers.ParseLinkAsync(String linkPath)
and so on (see debug file)
Steps To Reproduce
Sorry it suddenly happened, no steps available
Steps to fix
haven't been able to fix it.
Files Version
<IP_ADDRESS>
Windows Version
Windows 10 Pro 22H2
Log file
debug.log

Can you try the preview version https://files.community/download

I'm going to try it but the issue appears semi randomly so not sure how fast I could get it again.
Also a full system restart fixed / reset it for now...

We're working on improvements so very likely it's resolved in the preview version.


Resolve warnings about duplicate imports of Microsoft.Common.targets

Resolves https://github.com/aspnet/KoreBuild/issues/216
Depends on https://github.com/aspnet/BuildTools/pull/239

When this merges, will revert https://github.com/aspnet/KoreBuild/pull/227


[VS Code] Not currently hosting a collaboration session.

Error:
Not currently hosting a collaboration session.
Steps to Reproduce:
1.
2.

Version Data

extensionName
VSLS

extensionVersion
1.0.905

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.38.1

platformName
Windows

platformVersion
10.0.17763

@kafriyiegyamfi , thanks for reaching out , could you provide some more information? What were you doing when you received this error message? Do you continue to see this error message?
If you can get Live Share logs with Live Share: Export Logs command from VSCode command palette (ctrl+ship+p to get there), and attach them to this issue, this will help us troubleshoot.

no
On Thu, Oct 3, 2019 at 6:32 PM Selen Taskin<EMAIL_ADDRESS>wrote:

@kafriyiegyamfi https://github.com/kafriyiegyamfi , thanks for reaching
out , could you provide some more information? What were you doing when you
received this error message? Do you continue to see this error message?
If you can get Live Share logs with Live Share: Export Logs command from
VSCode command palette (ctrl+ship+p to get there), and attach them to this
issue, this will help us troubleshoot.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/live-share/issues/2879?email_source=notifications&email_token=AGICH42NLLWGB2VGGHQPRKLQMY3EHA5CNFSM4I5FJX32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAJEZMY#issuecomment-538070195,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGICH42BXLY6RQM4CPIENZLQMY3EHANCNFSM4I5FJX3Q
.

Closing this due to inactivity. Please re-open this issue if you're still encountering this problem. Thanks!


fix 505

Fixes #505 .
Description of the changes:

Changed all the E_CODE_* Variables located in CalcErr.h from Macros to static const expr (inline is implicite)

How changes were validated:

-Manual-Adhoc testing - I ran the program, and did my best to try and break it. I was able to get the "Overflow" to work.

 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.:x: Grommers00 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

I have updated the code to remove the white line space.
Is there anything else I need to do?


CVE-2021-3807 @ Npm-ansi-regex-2.1.1

Vulnerable Package issue exists @ Npm-ansi-regex-2.1.1 in branch master
ansi-regex prior to 5.0.1 and 6.0.x prior to 6.0.1 is vulnerable to Inefficient Regular Expression Complexity
Namespace: RobertMickleCx
Repository: NodeGoat
Repository Url: https://github.com/RobertMickleCx/NodeGoat
CxAST-Project: RobertMickleCx/NodeGoat
CxAST platform scan: 4cad0b9d-cbe1-4acd-bb82-244764df9dbd
Branch: master
Application: NodeGoat
Severity: HIGH
State: NOT_IGNORED
Status: RECURRENT
CWE: CWE-1333

Additional Info
Attack vector: NETWORK
Attack complexity: LOW
Confidentiality impact: NONE
Availability impact: HIGH

References
Advisory
POC/Exploit
Pull request
Commit
Issue

@RobertMickleCx Has some agent misinterpreted the affected ranges of CVE-2021-3807?
See affected ranges in https://github.com/advisories/GHSA-93q8-gq69-wqmw
Confirmed: https://github.com/chalk/ansi-regex/issues/38#issuecomment-925924774

The vulnerability does not exist in versions v2.1.1 and lower.


How to trigger different  snippet in different part ?

Thank you for awesome plugin !
There are many file type such as markdown or vue etc. They has some  Piece of Snippets to trigger,  for example , in markdowns code block with any language such as go or js, can luasnip  trigger only my snippets defined in go.snippets or lua.snippets in that code block?  In vscode , it can do this, but i cant find out how to do this with luasnip, is there any way?

I have found a ft_func option can fix this.
ft_func = require("luasnip.extras.filetype_functions").from_pos_or_filetype

now there is other question, is there some project based snippets can overwrite  default snippets, such as in default  js file, i can use pl to expand console.log,  but in some project, i use log lib and want use pl to expand log.info(),  and i dont want to have 2 pl selection candidate and just want use one. how to do this?


⚠ Mobitopup has degraded performance

In c756b8f, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 3816 ms

Resolved: Mobitopup performance has improved in 44f75cb.


RELEASE 0.4.6

Coverage remained the same at 90.064% when pulling 45dd59db84d7c6db3a945367ba6042b733ec5d52 on develop into e44b9433ff55b74d821a7977b9ed785efe027ebf on master.


接口报 500 错误

https://api.douyin.wtf/tiktok_video_data?video_id=7257925577091386642
这个接口报 500 错误 Internal Server Error

Also affecting by this.

Already fix the issue from the server, this is network issue not about the code:)


packet is not EOF

I've got an error:
  /usr/local/rvm/gems/ruby-1.9.3-p551/gems/mysqltopostgres-0.2.20/bin/mysqltopostgres mysql2pg.yml
  Mysql2psql: Conversion failed: packet is not EOF
  packet is not EOF
  /usr/local/rvm/gems/ruby-2.2.0/gems/mysql-pr-2.9.11/lib/mysql-pr/protocol.rb:562:in `read_eof_packet'
  /usr/local/rvm/gems/ruby-2.2.0/gems/mysql-pr-2.9.11/lib/mysql-pr/protocol.rb:274:in `retr_fields'
  /usr/local/rvm/gems/ruby-2.2.0/gems/mysql-pr-2.9.11/lib/mysql-pr.rb:322:in `query'

the very short dump is appeared:
  -- MySQL 2 PostgreSQL dump

  SET client_encoding = 'UTF8';
  SET standard_conforming_strings = off;
  SET check_function_bodies = false;
  SET client_min_messages = warning;

Could you share a minimal sql file contents and the settings used, that trigger this behaviour?

What kind of settings are you using?

I've been able to reproduce this error on Ruby 2.x. Using Ruby 1.9.3 seems to work without problems.

+1

The fix https://github.com/ajokela/mysql-pr/pull/1

n-rodriguez's patch worked for me (OSX 10.11), thanks

I am getting the same error:
vagrant@jessie:/vagrant/mysql2postgres$ mysqltopostgres config_options.yml
Mysql2psql: Conversion failed: packet is not EOF
packet is not EOF
/home/vagrant/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/mysql-pr-2.9.11/lib/mysql-pr/protocol.rb:562:in `read_eof_packet'
/home/vagrant/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/mysql-pr-2.9.11/lib/mysql-pr/protocol.rb:274:in `retr_fields'
/home/vagrant/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/mysql-pr-2.9.11/lib/mysql-pr.rb:322:in `query'
vagrant@jessie:/vagrant/mysql2postgres$ which ruby
/home/vagrant/.rbenv/shims/ruby
vagrant@jessie:/vagrant/mysql2postgres$ ruby -v
ruby 2.3.0p0 (2015-12-25 revision 53290) [x86_64-linux]

Listing of the config file:
# vagrant@jessie:/vagrant/mysql2postgres$ cat config_options.yml
default: &default
  adapter:  postgresql
  host:     localhost
  encoding: unicode
  pool:     5
  username: rails_user
  password: sekret
  template: template0

development: &development
  <<: *default
  database: stp_dev

test: &test
  <<: *default
  database: stp_test

mysql2psql:
  mysql:
    host: localhost
    socket: /var/run/mysqld/mysqld.sock
    username: root
    database: stp

  destination:
    test:
      <<: *test
    development:
      <<: *development

  tables:
  - certification_types

  dump_file_directory: /tmp
  remove_dump_file: false

Wondering how procommerz applied the patch from @n-rodriguez  to this issue.
Suggested work arounds?

The patch worked for me.
Since it was a one liner I manually applied it. Open packet.rb from gem mysql-pr and insert it as first line.
Thanks @n-rodriguez
At first I was concerned with Postgres saying no COPY in progress but the data does appear to have transferred over.

@bhfailor  :
cd $GEM_HOME
cd gems/mysql-pr-<version>/
vi lib/mysql-pr/packet.rb

Thanks, @n-rodriguez and @onpaws - I got it working!  And I agree that the no COPY in progress message is misleading.

I got same issues, but i can't find out the solution.
valuenet@valuenet-lap:~/Sites/branch/mysql2postgres$ mysqltopostgres default.database.yml
/home/valuenet/.rvm/gems/ruby-2.2.3@rails425/gems/pg-0.18.4/lib/pg/connection.rb:230: warning: already initialized constant PGconn
/home/valuenet/.rvm/gems/ruby-2.2.3@rails425/gems/postgres-pr-0.7.0/lib/postgres-pr/postgres-compat.rb:6: warning: previous definition of PGconn was here
/home/valuenet/.rvm/gems/ruby-2.2.3@rails425/gems/pg-0.18.4/lib/pg/result.rb:26: warning: already initialized constant PGresult
/home/valuenet/.rvm/gems/ruby-2.2.3@rails425/gems/postgres-pr-0.7.0/lib/postgres-pr/postgres-compat.rb:69: warning: previous definition of PGresult was here
Mysql2psql: Conversion failed: packet is not EOF
packet is not EOF
/home/valuenet/.rvm/gems/ruby-2.2.3@rails425/gems/mysql-pr-2.9.11/lib/mysql-pr/protocol.rb:562:in read_eof_packet' /home/valuenet/.rvm/gems/ruby-2.2.3@rails425/gems/mysql-pr-2.9.11/lib/mysql-pr/protocol.rb:274:in retr_fields'
/home/valuenet/.rvm/gems/ruby-2.2.3@rails425/gems/mysql-pr-2.9.11/lib/mysql-pr.rb:322:in `query'
valuenet@valuenet-lap:~/Sites/branch/mysql2postgres$

@jvkiran - I just made sure that the suggested line is inserted at the beginning of the suggested file:
$ bundle show mysql-pr
/home/vagrant/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/mysql-pr-2.9.11
$ cat /home/vagrant/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/mysql-pr-2.9.11/lib/mysql-pr/packet.rb
#encoding: ascii-8bit

class MysqlPR
  class Packet
    # convert Numeric to LengthCodedBinary
# [ remaining lines in file not shown ]

The inserted line is:
#encoding: ascii-8bit
then things worked.


Update ytdl-core please.

Please update to<EMAIL_ADDRESS>

There's a pull request already in for it, just needs accepting :)

The old ytdl-core is causing some errors. I also hope they will update ytdl-core in this project or else I might have to download this whole thing and just change that little bit of stuff.

Issue fixed in 3ee5182a4f201e97eff6d79a7ae63d726cf35afb

Nice


Vickel: Watch smartway2.com

Vickel requests the watch of the watch_keyword smartway2\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

smartway2\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


Generate a CSV file containing 104 information from the IEC 61850 SCL file

Is your feature request related to a problem? Please describe.
As a user of CoMPAS, I would like to be able to to export a CSV file from the IEC 61850 SCL file, containing relevant IEC 104 addresses and associated information. The outlined process in the like below explains how to export a CSV file from the SCL file and details the expected outcome and information within the CSV file.

@Sander3003 I made this ticket for Exporting CSV file feature.

@Sander3003 I just updated the pictures here. I have also sent a request for emoving the picture by the URL that you shared. Now I have to wait for a reply from them.

It is fixed, thanks @DavoodSooran!


chore(deps): only allow semver-patch dependabot updates for k8s deps

Pull request type

[x] :rainbow: Refactoring (no functional changes, no api changes)
[x] :hamster: Trivial/Minor
[x] :robot: @dependabot config

Test Plan

[x] :zap: CI. Github automatically validates the config when the PR is opened.

dependabot.yml syntax validation => https://github.com/kanisterio/kanister/actions/runs/5603501238/jobs/10250212186?pr=2197


aur-sync: Allow reading package names from standard input

For when you want the latest commit of everything.

Maybe you are looking for #283?

Since pkgname cannot contain spaces or tabs, you could use aur-sync --list
aur sync $(aur sync --list | jq .pkgname)

I guess you could add something like this to take packages from stdin (newline separated, as pacman)
for arg in "$@"; do
  case $arg in
    -) stdin=1 ;;
  esac
done

if ((stdin)); then
  mapfile -t pkg_stdin
  set -- "$@" "${pkg_stdin[@]}"
fi

so the above transforms to:
aur sync --repo-list | jq .name | aur sync -

Ahh, thanks! It did seem too easy.

That said, you can already achieve the above with command substitution

That works! Though I think you meant  | jq -r '.[][].pkgname'.

I guess you could add something like this to take packages from stdin

OK, I turned this PR into a change that implements it. How's this?

Nice, I like the cat trick. :+1:

Hmm, I wonder what happens if you specify - twice?

The second cat hits EOF immediately, so it's a no-op.
$ echo test | ( cat ; cat )
test

All right, thanks for clarifying.

Not sure it works completely as intended: https://github.com/jrudess/aurutils-plugins/blob/master/lib/aur-update-devel#L20-L23

@AladW fwiw, you can declare -l the reponse variable to avoid needing to do any case matching.
help declare
...
      -l	to convert NAMEs to lower case on assignment

One possibility is that aurutils could read interactive responses from the TTY directly, like sudo. That would however mean that yes and similar programs become unusable (at least not without being combined with script or such).

Indeed, please don't ever do that.

I guess a better approach is to instead add -a to aur-sync. Then you could do
aur sync --no-ver-shallow -a <(cut -d: -f1 updates)

Not sure if there's still much benefit in - then, though programs like xargs support both -a and -.


Fixed #27704 -- Fixed TypedMultipleChoiceField as base field class for contrib.postgres.ArrayField with choices.

As per suggestions in @ngnpope's comment on #7850 made choices_form_class = TypedMultipleChoiceField also removed support for ArrayField.choices and instead ArrayField.base_field.choices shall be used. Removed ArrayField.clean() method intoduced in https://github.com/django/django/commit/9dd244394236388c3479ab202a0ec31055f7ec09 because SimpleArrayField will no more be used while ArrayField has choices and also shifted the test test_model_field_choices introduced in https://github.com/django/django/commit/9dd244394236388c3479ab202a0ec31055f7ec09 to tests.postgres_tests.test_array.TestChoiceFormField.

@ngnpope Do you want to take a look?

Committed the changes according to @ngnpope's review and added test for it although we still need to look for a fix for some shortcomings that I picked out in this comment.

@felixxm if someone could review 🙇🏻

@ngnpope can you have a look??

@jacobtylerwalls committed the suggested changes although the patch does not throws a warning if choices are not in base field as far as i tested even though we have that in the patch and also the test works fine.

Ah, along the lines of this https://github.com/django/django/pull/15805#discussion_r921342795?
You could consider changing the warning to a system check as Nick suggested.

Ah, along the lines of this #15805 (comment)?
You could consider changing the warning to a system check as Nick suggested.
Got it 👍🏻  will look into it

Ah, along the lines of this #15805 (comment)?
You could consider changing the warning to a system check as Nick suggested.

So we would have to raise an error on running the makemigrations command right?
Also according to our current change that wouldn't be possible cause we would only be able to check it once the ArrayField.formfield() is called.

Have a look at the documentation for field checks. This is what I was referring to.

Closing due to inactivity.


API Endpoint cannot recognise user

Hi,
I am using the starter kit and I must say, It is really awesome and made the startup of the project, absolutely an ease.
Recently, I am facing an issue where the API endpoint is not able to fetch the authenticated user through getToken().
package.json
{
  "name": "saas-starter",
  "version": "0.2.0",
  "private": true,
  "author": {
    "name": "mickasmt",
    "url": "https://twitter.com/mickasmt"
  },
  "scripts": {
    "dev": "next dev",
    "build": "next build",
    "vercel-build": "prisma generate && prisma migrate deploy && next build",
    "turbo": "next dev --turbo",
    "start": "next start",
    "lint": "next lint",
    "preview": "next build && next start",
    "postinstall": "prisma generate",
    "email": "email dev --dir emails --port 3333"
  },
  "dependencies": {
    "@auth/prisma-adapter": "^2.1.0",
    "@hookform/resolvers": "^3.3.4",
    "@next-auth/prisma-adapter": "^1.0.7",
    "@prisma/client": "^5.14.0",
    "@radix-ui/react-accessible-icon": "^1.0.3",
    "@radix-ui/react-accordion": "^1.1.2",
    "@radix-ui/react-alert-dialog": "^1.0.5",
    "@radix-ui/react-aspect-ratio": "^1.0.3",
    "@radix-ui/react-avatar": "^1.0.4",
    "@radix-ui/react-checkbox": "^1.0.4",
    "@radix-ui/react-collapsible": "^1.0.3",
    "@radix-ui/react-context-menu": "^2.1.5",
    "@radix-ui/react-dialog": "^1.0.5",
    "@radix-ui/react-dropdown-menu": "^2.0.6",
    "@radix-ui/react-hover-card": "^1.0.7",
    "@radix-ui/react-label": "^2.0.2",
    "@radix-ui/react-menubar": "^1.0.4",
    "@radix-ui/react-navigation-menu": "^1.1.4",
    "@radix-ui/react-popover": "^1.0.7",
    "@radix-ui/react-progress": "^1.0.3",
    "@radix-ui/react-radio-group": "^1.1.3",
    "@radix-ui/react-scroll-area": "^1.0.5",
    "@radix-ui/react-select": "^2.0.0",
    "@radix-ui/react-separator": "^1.0.3",
    "@radix-ui/react-slider": "^1.1.2",
    "@radix-ui/react-slot": "^1.0.2",
    "@radix-ui/react-switch": "^1.0.3",
    "@radix-ui/react-tabs": "^1.0.4",
    "@radix-ui/react-toast": "^1.1.5",
    "@radix-ui/react-toggle": "^1.0.3",
    "@radix-ui/react-toggle-group": "^1.0.4",
    "@radix-ui/react-tooltip": "^1.0.7",
    "@react-email/button": "0.0.15",
    "@react-email/components": "0.0.17",
    "@react-email/html": "0.0.8",
    "@t3-oss/env-nextjs": "^0.10.1",
    "@types/uuid": "^9.0.8",
    "@typescript-eslint/parser": "^7.8.0",
    "@vercel/analytics": "^1.2.2",
    "@vercel/og": "^0.6.2",
    "axios": "^1.7.2",
    "class-variance-authority": "^0.7.0",
    "clsx": "^2.1.1",
    "cmdk": "^1.0.0",
    "concurrently": "^8.2.2",
    "contentlayer2": "^0.4.6",
    "date-fns": "^3.6.0",
    "html-react-parser": "^5.1.10",
    "lucide-react": "^0.377.0",
    "luxon": "^3.4.4",
    "ms": "^2.1.3",
    "next": "14.2.3",
    "next-auth": "5.0.0-beta.18",
    "next-contentlayer2": "^0.4.6",
    "next-sitemap": "^4.2.3",
    "next-themes": "^0.3.0",
    "nodemailer": "^6.9.13",
    "prop-types": "^15.8.1",
    "react": "18.3.1",
    "react-aiwriter": "^1.0.0",
    "react-day-picker": "^8.10.1",
    "react-dom": "18.3.1",
    "react-email": "2.1.2",
    "react-hook-form": "^7.51.3",
    "react-markdown": "^9.0.1",
    "react-scroll": "^1.9.0",
    "react-textarea-autosize": "^8.5.3",
    "react-toastify": "^10.0.5",
    "react-typed": "^2.0.12",
    "request-ip": "^3.3.0",
    "resend": "^3.2.0",
    "sharp": "^0.33.3",
    "shiki": "^1.4.0",
    "stripe": "^15.4.0",
    "tailwind-merge": "^2.3.0",
    "tailwindcss-animate": "^1.0.7",
    "uuidv4": "^6.2.13",
    "vaul": "^0.9.0",
    "zod": "^3.23.5",
    "zustand": "^4.5.2"
  },
  "devDependencies": {
    "@commitlint/cli": "^19.3.0",
    "@commitlint/config-conventional": "^19.2.2",
    "@ianvs/prettier-plugin-sort-imports": "^4.2.1",
    "@tailwindcss/line-clamp": "^0.4.4",
    "@tailwindcss/typography": "^0.5.13",
    "@types/node": "^20.12.8",
    "@types/react": "18.3.1",
    "@types/react-dom": "18.3.0",
    "autoprefixer": "^10.4.19",
    "eslint": "^8.57.0",
    "eslint-config-next": "14.2.3",
    "eslint-config-prettier": "^9.1.0",
    "eslint-plugin-react": "^7.34.1",
    "eslint-plugin-tailwindcss": "^3.15.1",
    "husky": "^9.0.11",
    "mdast-util-toc": "^7.0.1",
    "postcss": "^8.4.38",
    "prettier": "^3.2.5",
    "prettier-plugin-tailwindcss": "^0.5.14",
    "pretty-quick": "^4.0.0",
    "prisma": "^5.14.0",
    "rehype": "^13.0.1",
    "rehype-autolink-headings": "^7.1.0",
    "rehype-pretty-code": "^0.13.1",
    "rehype-slug": "^6.0.0",
    "remark": "^15.0.1",
    "remark-gfm": "^4.0.0",
    "tailwindcss": "^3.4.3",
    "typescript": "5.4.5",
    "unist-util-visit": "^5.0.0"
  }
}

components/services/service-input-card.tsx
....

 const axResponse = await axios.post(
        "/api/jobs",
        {
          slug: service.slug,
          input: query,
          ip: null,
          token: visit.token,
        },
        {
          headers: {
            "Content-Type": "application/json",
          },
          withCredentials: true,
        },
      );
      const response: {
        message: any;
        response: any;
      } = (await axResponse.data) as any;

....

api/jobs/route.tsx
export async function POST(req: NextRequest) {
  let { slug, input, ip, token } = await req.json();
  ip = getIp(req);
  if (!ip || (!!token && token.length != 36)) {
    return new Response(JSON.stringify({ message: "Invalid request" }), {
      status: 400,
    });
  }

  const secret = process.env.NEXTAUTH_SECRET;
  console.log("🎩 Secret:", secret);
  const bareUser = (await getToken({
    req,
    secret,
    encryption: true,
  } as any)) as any;
  console.log("🎩 Bare User:", bareUser);
  let user: any = null;
  if (bareUser) {
    user = await prisma.user.findUnique({
      where: {
        email: bareUser.email,
      },
    });
  }
  console.log("🎩 User:", user);
}

The bareUser and user both come out to be null.
Note: In development, things seem to be working but on Vercel it is not. Possibly some issue with the transition from next-auth to auth.js.
Please guide.

Hi @AdnanHussainTurki thanks for your interest on the project!
I updated your code to work with next auth v5. The route will be protected and no need to bring back the env variable or the token. I don't know if this is what you are looking for. More information here
import { auth } from "@/auth";
import { prisma } from "@/lib/db";

export const POST = auth(async (req) => {
  if (!req.auth) {
    return Response.json({ message: "Not authenticated" }, { status: 401 });
  }

  const currentUser = req.auth.user;
  let { slug, input, ip } = await req.json();

  ip = getIp(req);
  if (!ip) {
    return Response.json({ message: "Invalid request" }, { status: 400 });
  }

  let user: any = null;
  if (currentUser) {
    user = await prisma.user.findUnique({
      where: {
        id: currentUser.id,
      },
    });
  }

  return Response.json({ message: "ok" }, { status: 200 });
});


edge bug?

Code it your self, its easy.

Code it your self, its easy.

damn if i good at coding ill do my self man

@aljawary

Tho is useful velocity indicator :)

thats from another cheat

thats from another cheat

How do you know its not just Osiris?

@KernCS checkboxes dont need XorStr() in osiris, binds are hotkey() or something like that not bind()

@KernCS checkboxes dont need XorStr() in osiris, binds are hotkey() or something like that not bind()

Maybe a remake of the keybinds? i did the same since i did not like the one that's in the source rn.
XorStr is simple to add so i don't really see a reason not just to add it, again i did exactly the same


Support for recursive go get

I am using dep on a small web app and everything is working great for the actual app. However, when I try to run the tests I receive several errors related to missing packages. One of them being related to gomega:
vendor/github.com/onsi/gomega/ghttp/handlers.go:12:2: cannot find package "github.com/golang/protobuf/proto" in any of: 	/Users/lukashambsch/workspace/go/src/github.com/lukashambsch/gym-all-over/vendor/github.com/golang/protobuf/proto (vendor tree) 	/usr/local/Cellar/go/1.8.1/libexec/src/github.com/golang/protobuf/proto (from $GOROOT) 	/Users/lukashambsch/workspace/go/src/github.com/golang/protobuf/proto (from $GOPATH)
When I install the package with go get github.com/onsi/gomega/... that error goes away. I believe it is related to this as well: https://github.com/onsi/gomega/issues/123
Is there any way to state that the install should be done in the recursive manner?

Hmm, this is surprising. My guess would be that this occurs if you run go test ./... from the root of your project, which will try to run tests for packages under vendor (unless you're running go tip, where that's been changed). That could fail, because dep ignores dependencies needed solely by your dependencies' tests.
If you're only running your project's tests and there are necessary dependencies missing, though, then something else is going on - possibly a bug.

You are correct, I'm running go test ./.... I can't believe I missed that. Thanks!


Multiple instances of a type can be declared in the using statement

The code in the example produces error: CS0819: Implicitly-typed variables cannot have multiple declarators.
using (var font3 = new Font("Arial", 10.0f),
font4 = new Font("Arial", 10.0f))
{
// Use font3 and font4.
}

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

ID: 042902cc-1598-5df1-34cc-caa0fad63e02
Version Independent ID: e186d0c4-523e-38f7-2cde-d00109ea784b
Content: using statement - C# Reference
Content Source: docs/csharp/language-reference/keywords/using-statement.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Oh, Embarassing. I've reviewed the file history and this bug was produced by me in https://github.com/dotnet/samples/pull/1042.
I've opened https://github.com/dotnet/samples/pull/1472 to fix that.


Chip Hot

Motor circuit should be rewired such that it uses two different boards

Rewired motors in commit 5846393


sys/openbsd: neutralize yet another pf ioctl command

Disarms the latest reproducer[1] for the discovered "no output from test
machine" panic.
While here, tidy up and improve comments.
[1] https://syzkaller.appspot.com/bug?id=4c3b1ac22d6d7f1d9ebcad5469a28cb4562debda

@blackgnezdo I trust you to merge such PRs in future ;)


Empty cpe produce in csv report

Describe the bug
A clear and concise description of what the bug is.
Null/empty cpe in the csv report
** Version of dependency-check used **
The problem occurs using version X.X.X of the the ____ (cli, gradle plugin, maven plugin, etc.)
5.1.1 maven plugin
** Log file **
When reporting errors, 99% of the time log file output is required. Please post the log file as a gist and provide a link in the new issue.
Csv sample
Project,ScanDate,DependencyName,DependencyPath,Description,License,Md5,Sha1,Identifiers,CPE,CVE,CWE,Vulnerability,Source,CVSSv2_Severity,CVSSv2_Score,CVSSv2,CVSSv3_BaseSeverity,CVSSv3_BaseScore,CVSSv3,CPE Confidence,Evidence CountPNC Assets,"Wed, 17 Jul 2019 06:16:19 +0100",libs-<IP_ADDRESS>.tar.gz: libs-<IP_ADDRESS>.tar: dozer-5.5.1.jar,C:.maven\com\baesystems\libs\<IP_ADDRESS>\libs-<IP_ADDRESS>.tar.gz\libs-<IP_ADDRESS>.tar\libs\dozer\dozer-5.5.1.jar,,,54eec02e942f50ffea7e147f33cd85f7,f2e52ef54166788ae29b5b4eec6fdb9cd580acab,pkg:maven/net.sf.dozer/dozer@5.5.1,,CVE-2014-9515,null,"Dozer improperly uses a reflection-based approach to type conversion, which might allow remote attackers to execute arbitrary code via a crafted serialized object.",OSSINDEX,,,,CRITICAL,9.8,/AV:N/AC:L/PR:N/UI:N/S:U/C:H/I:H/A:H,HIGH,15
PN,"Wed, 17 Jul 2019 06:16:19 +0100",libs-<IP_ADDRESS>.tar.gz: libs-<IP_ADDRESS>.tar: guava-11.0.2.jar,C:.maven\com\sys\libs\<IP_ADDRESS>\libs-<IP_ADDRESS>.tar.gz\libs-<IP_ADDRESS>.tar\libs\google\guava\11.0.2\guava-11.0.2.jar,"Guava is a suite of core and expanded libraries that include     utility classes, google's collections, io classes, and much     much more.      This project is a complete packaging of all the Guava libraries     into a single jar.  Individual portions of Guava can be used     by downloading the appropriate module and its dependencies.      Guava (complete) has only one code dependency - javax.annotation,     per the JSR-305 spec.",,bed5977336ea1279d2bad3bb258dc8c3,35a3c69e19d72743cac83778aecbee68680f63eb,pkg:maven/com.google.guava/guava@11.0.2,,CVE-2018-10237,null,"Unbounded memory allocation in Google Guava 11.0 through 24.x before 24.1.1 allows remote attackers to conduct denial of service attacks against servers that depend on this library and deserialize attacker-provided data, because the AtomicDoubleArray class (when serialized with Java serialization) and the CompoundOrdering class (when serialized with GWT serialization) perform eager allocation without appropriate checks on what a client has sent and whether the data size is reasonable.",OSSINDEX,,,,MEDIUM,5.9,/AV:N/AC:H/PR:N/UI:N/S:U/C:N/I:N/A:H,HIGH,21,,,,,,,,,,,,,,,,,,,,,
PN,"Wed, 17 Jul 2019 06:16:19 +0100",libs-<IP_ADDRESS>.tar.gz: libs-<IP_ADDRESS>.tar: hadoop-auth-2.2.0.jar,C:.maven\com\sys\libs\<IP_ADDRESS>\libs-<IP_ADDRESS>.tar.gz\libs-<IP_ADDRESS>.tar\libs\hadoop\2.2.0\hadoop-auth-2.2.0.jar,Apache Hadoop Auth - Java HTTP SPNEGO,,8bb0f03bea387738b61642a2502b3289,74e5f8b2134be51312c004d29e33a7bf4377ce20,pkg:maven/org.apache.hadoop/hadoop-auth@2.2.0,,CVE-2014-0229,null,"Apache Hadoop 0.23.x before 0.23.11 and 2.x before 2.4.1, as used in Cloudera CDH 5.0.x before 5.0.2, do not check authorization for the (1) refreshNamenodes, (2) deleteBlockPool, and (3) shutdownDatanode HDFS admin commands, which allows remote authenticated users to cause a denial of service (DataNodes shutdown) or perform unnecessary operations by issuing a command.",OSSINDEX,,,,MEDIUM,6.5,/AV:N/AC:L/PR:L/UI:N/S:U/C:N/I:N/A:H,HIGH,24,,,,,,,,,,,,,,,,,,,,,
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.
suppression with cpe regex with filePath will not work as the cpe is empty or null
Additional context
Add any other context about the problem here.

dependency-check-report.zip

This was resolved with #2071 and will be included in the next release.


Zoom activation key remains pressed

Describe the Bug
While using figma like controls, if i press cmd (meta key on mac) inside of an input,  board seams to keep a state of meta key pressed and allways does zoom on scroll wheel instead of panning,
Your Example Website or App
https://codesandbox.io/s/busy-benji-fyenzx?file=/Board.js
Steps to Reproduce the Bug or Issue

Use scroll wheel on the board to pan
Use scroll wheel on the board with cmd pressed to zoom
Focus input
Click on cmd
Use scroll wheel, it starts to zoom instead of panning

Expected behavior
Clicking cmd on inputs does not affect board cmd for zoom behaviour
Screenshots or Videos
No response
Platform

OS: [macOS, Windows, Linux]
Browser: [Chrome]
Version: [e.g. 109.0.5414.87]

Additional context
No response

@moklick i think i figure out what is happening here
On useKeyPress: https://github.com/wbkd/react-flow/blob/4963aec71a9b6160afbb60e8bc1457d1b1657a0e/packages/core/src/hooks/useKeyPress.ts#L46
This code is checking if the event comes from an input/textarea/contenteditable for that it calls isInputDOMNode
But on isInputDOMNode there's this condition:  https://github.com/wbkd/react-flow/blob/4963aec71a9b6160afbb60e8bc1457d1b1657a0e/packages/core/src/utils/index.ts#L107 the comment above this condition states the following:
// we want to be able to do a multi selection event if we are in an input fiel
And indeed it makes sense for the case where we do not have figma like controls, but when we do have figma like controls it is unnecessary
I'm more than happy to summit a pr to fix this, i'm just not sure how to approach it, since even the useKeyPress does not have access to the props of ReactFlow

From what I can tell, the keyUp event does not contain the modifier anymore, so if (event.ctrlKey || event.metaKey || event.shiftKey) can never be true when checking it in the keyUp event handler.
What we're trying to check for should be:

If we're focusing an input element
And no modifier key was pressed
--> return false

Which works for the keyDown event, that one contains the modifier flags as expected.
On keyUp however these modifiers are missing, thus the above condition will return true, making it impossible to release these keys as long as you focus an input element (if you click somewhere else, press the same key and release it, it actually registers the release "correctly").
Seems like a way to solve this might be storing a modifierPressed flag on keyDown and check against that stored flag in keyUp.
// useKeyPress.ts

// ...

const modifierPressed = useRef(false);

// ...

const downHandler = (event: KeyboardEvent) => {
	  modifierPressed.current = wasModifierPressed(event);

	  if (!modifierPressed && isInputDOMNode(event)) {
	    return false;
	  }
	  const keyOrCode = useKeyOrCode(event.code, keysToWatch);
	  pressedKeys.current.add(event[keyOrCode]);

	  if (isMatchingKey(keyCodes, pressedKeys.current, false)) {
	    event.preventDefault();
	    setKeyPressed(true);
	  }
};

const upHandler = (event: KeyboardEvent) => {
	  if (!modifierPressed.current && isInputDOMNode(event)) {
	    return false;
	  }
	  const keyOrCode = useKeyOrCode(event.code, keysToWatch);

	  if (isMatchingKey(keyCodes, pressedKeys.current, true)) {
	    setKeyPressed(false);
	    pressedKeys.current.clear();
	  } else {
	    pressedKeys.current.delete(event[keyOrCode]);
	  }

	  modifierPressed.current = false;
};

These are just my thoughts on the issue though.


Not working anymore

Hi,
Im using XCDYouTubeClient in my iOS and tvOS but from some days i noticed that i can’t get direct url to the video.
Could you please check, thanks :)

Hello @Red-Mak,
Did you test with the latest version 2.9.0 ?
Can you please share the URLs that don't work ?

Thank you for the quick answer
here's alink that doesn't work : https://www.youtube.com/watch?v=JDcAlo8i2y8

Video plays fine on my end, please confirm you've updated to v2.9.0 and that you've cleaned you build folder.

not sure what happened, but i switch to use SPM instead of cocoapods and it works ..


Plugin "OpenStreeMap" map icon missing in albums

Hello,
I've searched on closed issues, ... and the only thing i found was a resolved issue regarding the map icon for the plugin RV Map & Earth. I'm using the plugin OpenStreetMap and on the compatibility matrix i can see this : OpenStreetMap | Yes | requires template/plugin_fixes_js.tpl
Does this fix the issue about the map icon not displayed in the album ? I used the "Elegant" theme before, and figure it now that this icon is missing with the Darkroom theme (which it really nice, really great job you've done).
Fyi, I'm using the "piwigo" cloud version, so i don't have a direct access to the local files, i can't verify that this *.tpl is present. The theme version installed is the 1.5.20.
Thank you,

I've done some more tests with my local installation of piwigo.
I have the world map icon on the mail page and album page. this is a bug only with a previous version of your theme. Forget about this issue.


Test

Coverage remained the same at 99.636% when pulling 1a68d3d3daeef6e778035146514bf0f5bb1127b2 on murbans1:feature/test into 32128a96bdf876dbe46c15357f213e166ecbe523 on agschwender:master.


generate the virtual points

Hello tianweiy,
when I generate the points with gpu RTX 3090 , I get this by train set:
100%|█████████████████████████| 271572/271572 [110:45:35,<00:00,  1.46s/it]
and this by val set:
100%|█████████████████████████| 58066/58066 [25:44:38<00:00,  1.60s/it]
1- What does these numbers 271572,58066  mean? Is it the number of segmentation masks in the images?
num of images in val set is: 16445!
2- Why does it take so long to generate virtual points? Is this logical?

it is the number of frames. in nuscenes, you get 1 key frame + 10 non-key frame, so it is much larger.

I am not sure. you can profile the program and see which part is the most time consuming in your setup.


Demo repeats first animation

On the demo page, when you click on one of the buttons for the first time, the animation gets repeated after the alert finishes it's first transition.

Hey @rapito,
thank you for your feedback. I didn't find that happening anymore on version 2. Could you please let me know if you still see this happening. Closing for now.

@RoopeHakulinen Its not happening anymore, thanks!


Assorted UI fixes

Some padding has been increased, some text is now left-aligned, etc.

I'll send this to someone in NYC.


[Issue]: hipp-runtime-nvidia and rocm-hip-sdk canntot be both installed - EL9

Problem Description
Starting on RHEL9 and repository yum repo: https://repo.radeon.com/rocm/el9/6.2.2/main
As per the instruction for installing on an nvidia node both hip-devel and hip-runtime-nvidia can be installed.
This pulls in hipcc-nvidia which is probably correct.
If you then try to install rocm-hip-sdk this fails due to
  file /opt/rocm-6.2.2/bin/hipcc.bin from install of hipcc-<IP_ADDRESS>202-116.el9.x86_64 conflicts with file from package hipcc-nvidia-<IP_ADDRESS>202-116.el9.x86_64
  file /opt/rocm-6.2.2/bin/hipconfig.bin from install of hipcc-<IP_ADDRESS>202-116.el9.x86_64 conflicts with file from package hipcc-nvidia-<IP_ADDRESS>202-116.el9.x86_64

In particular rocm-hip-sdk requires rocm-hip-runtime-devel which in turn requires hipcc resulting in the conflict.
Operating System
Red Hat Enterprise Linux 9.4
CPU
Intel(R) Xeon(R) Silver 4216 CPU @ 2.10GHz
GPU
Tesla T4
ROCm Version
ROCm 6.2.2
ROCm Component
HIPCC
Steps to Reproduce
Configure https://repo.radeon.com/rocm/el9/6.2.2/main
dnf install -y hipp-runtime-nvidia hipd-devel
dnf install -y rocm-hip-sdk

(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
No response
Additional Information
No response

Hi @traylenator. Internal ticket has been created to investigate your issue. Thanks!

@traylenator rocm-hip-sdk is meant for AMD platforms (see our documentation), which is why installing it is likely going to result in conflicts with the the NVIDIA hip packages. A potential work-around I can think of is to install these in separate docker containers. Out of curiosity, would you mind letting us know your use case which requires both to be installed?
Hope this helps.
Thanks!

@tcgu-amd thanks for the comments.
We are running mostly NVIDIA cards today but wanted to try out HIP on those machines.  We are hoping to avoid vendor lock-in and  (cross-)compile to both AMD and NVIDIA hardware platform from the same build machine at the same time so any hardware migration in the future would be  hopefully easier.

@traylenator Ah I see. That makes sense. Unfortunately I don't think it is possible to achieve this on bare metal at the moment. But, as I mentioned, I think it would be possible to achieve this through installing ROCm in two separate docker containers, given that the drivers and hardware are configured properly on the host system. There will be some redundancy for sure, but unfortunately that is unavoidable at the moment because a lot of our libraries in our runtime is either compiled for NVIDIA or AMD.
Thanks!

@traylenator I believe the key difference is that hipcc depends on rocm-llvm, whereas hipcc-nvidia doesn't. The source code of the two versions hipcc themselves are virtually the same; however, as hipcc are just perl wrappers, there might still be discrepancies due to different backends.

@traylenator, to follow up, it might be possible to resolve the hipcc conflict by uninstalling hipcc-nvidia and then try installing rocm-hip-sdk, since the hipcc installed by rocm-hip-sdk should work for both NVIDIA and AMD runtimes. That being said, it is still strongly recommend to use a containerized environment to avoid further conflicts. Thanks!

Yes for sure you can install hipcc rather than hipcc-nvidia and all is "good" as I say the results even run on NV.
This can probably be closed.
Thanks for all the responses, much appreciated.


chore(release): release v2.3.4

Types

[ ] 🎉 New Features
[ ] 🐛 Bug Fixes
[ ] 📚 Documentation Changes
[ ] 💄 Code Style Changes
[ ] 💄 Style Changes
[ ] 🪚 Refactors
[ ] 🚀 Performance Improvements
[ ] 🏗 Build System
[ ] ⏱ Tests
[ ] 🧹 Chores
[ ] Other Changes

Background or solution
ChangeLog

/merge-this


Update kf-deployment-ui-version.html

Now KF v0.7.0 can be deployed using UI.

This change is

/assign
/ok-to-test

Hallo @rky0930 a hint in case you're using the GitHub UI to make this change: Unfortunately, the GitHub UI always puts the new line back. You need to use the command line. You can find a quick guide here: https://github.com/kubeflow/website/blob/master/quick-github-guide.md#using-the-command-line

Thanks!
/lgtm
/approve

Thank you for helping me. I didn't know github UI always puts the new line back.
I remove the newline at end of file using cli and checked deploy preview. It seems OK now.
If you find anything else, I will contribute again.


specter doesn't respawn + error when using /s attack

[Server thread/CRITICAL]: Error: "Undefined class constant 'ACTION_RIGHT_CLICK'" (EXCEPTION) in "plugins/Specter-master/src/specter/Specter" at line 123
pocketmine: 3.11.1

I can confirm this, this happened to me recently. I am yet unsure why specter used the right click action, and what it does. I will check if it is for arm swinging and fix it asap

Fixed arm swing and attacking in https://github.com/falkirks/Specter/commit/b39d20d11933d4e746cf3a1a488cdc7de9c39cd9
I will check respawn asap

Respawn fixed in https://github.com/falkirks/Specter/commit/10c118e64d2a1e9b86be1a2969cdf04e318127d0

also some plugins can't fetch the specter device model and then the server crashes

can't fetch the specter device mode

Then those plugins should fix it - there is no official implementation of device model/device OS in pmmp. I can not set a specific OS by default, this would be inaccurate

^ hmm, maybe i can set the OS as -1 (undefined)
@falkirks as SOFe told me, i need admin repo access to schedule updates to poggit, can you do that?

Yup I can do that, just tag a release on GitHub and I can put it on poggit. Is there a way to give you admin access though?

I have no idea how to do it, i know there is "transfer ownership", but i dont think thats what we want

Perhaps I should change Poggit config to allow anyone with write access to submit the plugin


Edit suggested: Escape tab characters to resolve BWA error

https://github.com/CPTR-ReSeqTB/UVP/blob/55fd74cd5436d36cb6e7ec64c5368f1671af1529/scripts/UVP.py#L251
That line seems to be giving me some issue in the BWA step. Not sure if I'm the only one.
MY command line interprets the statement with tab characters when calling, and that throws up the " [E::bwa_set_rg] the read group line contained literal characters -- replace with escaped tabs: \t" error.
I fixed my version by doubling up the escape characters, so the line is this instead:
"@RG\\tID:" + self.name + "\\tSM:" + self.name + "\\tPL:ILLUMINA",

I encounter this problem too, and your method helps by adding the backslash escape character. Thanks.


doc: Effective Go doc misuses word "anew", resulting in confusing usage rule

In Effective Go, in the section on "Redeclaration and reassignment," it says that an existing variable may appear in a short declaration if "there is at least one other variable in the declaration that is being declared anew."
Doesn't it really mean being declared for the first time? That's not what "anew" means; if anything, it means "again."

@robpike

The word can go either way but it is the wrong choice, I agree. I will fix.


feat: transaction_index

PR Type
Enhancement

Description

Added transaction_index field to AccountTransfers struct to store the index of the transaction in the block
Updated serialization logic to include the new transaction_index field
Modified transaction_to_events function to populate the transaction_index from TransactionMined
Updated test case ledger_events_serde_account_transfers to include transaction_index
Improved documentation for existing fields, including format specifications
Minor formatting changes (e.g., "Number" to "Integer" in some comments)

Changes walkthrough 📝
Relevant filesEnhancement

events.rsAdd transaction index to AccountTransfers

src/ledger/events.rs
Added transaction_index field to AccountTransfers struct  Updated serialization to include transaction_index  Modified transaction_to_events function to populate transaction_index  Updated test case to include transaction_index  Minor formatting and documentation improvements

+14/-3

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

Final benchmark:
RPS Stats: Max: 913.00, Min: 779.00, Avg: 847.88, StdDev: 20.28
TPS Stats: Max: 896.00, Min: 779.00, Avg: 847.55, StdDev: 19.61
Plot: View Plot


Graph Mode QAT / Static Quantization

This PR adds graph mode QAT and static quantization using the experimental symbolic tracing feature from the transformers library . This will allow more flexible model editing to apply the changes needed to target some device / inference engine.
For now, it is not integrated to the rest of the library (will come in later PRs), and export to TorchScript is supported.

Nice work ! Can't wait for the next steps !


File names with brackets cause incorrect error on first symbol

[Bracketed Filename].fsx
open System
Console.WriteLine("Error above"

A program as simple as the above causes the following error message to appear
file: '[Bracketed Filename].fsx'
severity: 'Error'
message: 'Invalid namespace, module, type or union case name'
at: '1,1'
source: ''
code: 'undefined'

You do not even need code in the editor to cause this issue. It will appear even if the file is blank, so long as it has brackets in the filename.

Upon further inspection, this appears to be an issue with F# itself, as I can reproduce the same error in Visual Studio 2017.


Add dispatch for provider_ehr_launch based on queryparams

The body of the function remains unchanged just added an additional dispatch on queryparams dict

We noticed that it's only the parameters that are actually used in the function body and it would be simpler for us to just pass the parameters. I'm wondering if it would be even better to just pass iss and launch_token directly as arguments instead of passing the Dict.

I'm wondering if it would be even better to just pass iss and launch_token directly as arguments instead of passing the Dict.

#23


Reworked setuptools and add github build process

Fixes #11 and #26 .
This removes setuptools as part of main project dependencies, the use of setup.py. It adds a new way to run from the CLI using the standard way to run a module with Python. It updates the related README file sections. It adds ci-build workflow to build release assets using the Github workflow and the Github runners. The ci-build can only be started once it's in the main branch of the repo. In order to test this, you can merge it into your personal main branch and start the workflow manually by going into actions, selecting the ci-build workflow and clicking on Run workflow as detailed in https://docs.github.com/en/actions/using-workflows/manually-running-a-workflow .
There might be more work to be done for a full build and release workflow like a runner for Linux arm64, GPG signing, running tests against the resulting bundled binaries (the tests in test_binary_btec_script.py, test_binary_deposit_script.py, test_btec_script.py and test_deposit_script.py for instance). These efforts can be completed in other PRs.

I also removed the use of setup.py as part of the Dockerfile and the Makefile. I've tested the Docker image and the Makefile and they are still working for me after those changes.


[Feature]: add the content to tensor-flow page

Is your feature request related to a problem? Please describe.
https://www.nsccbpit.tech/tensorflow

add the content and resources to this page
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

Please assign this to me

go ahead @prakartigoel24

resolved


SW-4731 Move permanent plot selection to PlantingZoneModel

In preparation for supporting on-demand creation of monitoring plots, move the
logic for selecting permanent clusters for an observation to PlantingZoneModel.
Cluster selection is now done completely in the application code rather than via
a database query.
No change in behavior is introduced here; the plot selection logic is already
covered by ObservationServiceTest and PlotAssignmentTest which continue to
pass.

Current dependencies on/for this PR:

main

PR #1637   👈

This stack of pull requests is managed by Graphite.


distro: update feed defaults to actual nightly location

Set the FEEDDOMAIN and FEEDURIPREFIX to what is used for nightly builds.
This allows up using these in the SRC_URI of binary only packages, and
then only need to update/override these for releases instead of updating
the SRC_URI in the recipes.
Signed-off-by: Jonas Gorski<EMAIL_ADDRESS>

lgtm


Last week my youtube-dl was working perfectly, now I can't download anything and it keeps on giving me the message "is not recognized as an internal or external command, operable program or batch file". I can't even get through "Downloads" without it giving me the message

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

Why aren't my commands going through? I've noticed ever since I opened up cmd.exe that the line was in ">" form rather than the usual slash. It wasn't like that in the past week. Could that be the reason why I'm unable to download videos?

C:\Users\Sam> chdir Downloads


sass variable for previous month dates on hover

for days that are in the current month, there are variables to change the foreground color on hover. this is useful if my hover background is a dark color and I need to switch the foreground to a light color.
however, there's no way to style the foreground color on hover for days that are not in the current months. by these, I mean the dates that would excluded if {includeDays: 'none'}. however, these days are still affected by the hover background. so I get dark on dark colors.

@jonathan-chin I have updated the scss to handle the hover state on out of range month days, you can find the change in the latest version. Let me know if this works for you.


Crash when cache file has no permissions

When running the program to get the token, it crashes with sigsegv in case when
the cache file exists, but there are no permissions to read its parent
directory:
$ AWS_PROFILE=xxx aws-iam-authenticator token --region yyy --cluster-id xxx-eks --cache
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x28 pc=0x15f1fbe]

goroutine 1 [running]:
sigs.k8s.io/aws-iam-authenticator/pkg/token.NewFileCacheProvider(0x7fffffffe84a, 0xe, 0xc0000442ec, 0xa, 0x0, 0x0, 0xc000414c00, 0x0, 0x0, 0x0, ...)
        /tmp/aws-iam-authenticator/pkg/token/filecache.go:189 +0x3fe
sigs.k8s.io/aws-iam-authenticator/pkg/token.generator.GetWithOptions(0x7ffff7d70100, 0xc00007c900, 0x0, 0x0, 0x0, 0x0, 0x0, 0x40d1af, 0xc00019fc50)
        /tmp/aws-iam-authenticator/pkg/token/token.go:253 +0x911
main.glob..func3(0x2797620, 0xc00007e820, 0x0, 0x5)
        /tmp/aws-iam-authenticator/cmd/aws-iam-authenticator/token.go:64 +0x366
github.com/spf13/cobra.(*Command).execute(0x2797620, 0xc00007e7d0, 0x5, 0x5, 0x2797620, 0xc00007e7d0)
        /home/wolf/go/pkg/mod/github.com/spf13/cobra@v0.0.5/command.go:830 +0x2c2
github.com/spf13/cobra.(*Command).ExecuteC(0x2797120, 0x1a5870b, 0x4, 0x1a866fd)
        /home/wolf/go/pkg/mod/github.com/spf13/cobra@v0.0.5/command.go:914 +0x30b
github.com/spf13/cobra.(*Command).Execute(...)
        /home/wolf/go/pkg/mod/github.com/spf13/cobra@v0.0.5/command.go:864
main.Execute()
        /tmp/aws-iam-authenticator/cmd/aws-iam-authenticator/root.go:50 +0x31
main.main()
        /tmp/aws-iam-authenticator/cmd/aws-iam-authenticator/root.go:45 +0x25

That is bit subpar user experience and without looking into source code it's
imposible to figure out what is going on.
Fix seems to be fairly trivial, diff is
$ g d --stat
 pkg/token/filecache.go | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

Since I do not want to accept CLA, I cannot provide the diff itself.

/remove-lifecycle stale

/remove-lifecycle stale

How exactly would the issue "solve itself"?

How exactly would the issue "solve itself"?

/remove-lifecycle stale

It still did not resolve itself.

/remove-lifecycle stale


Is there a way to give a different color for the maximum point in a line chart?

This is a question related to using Chart.js in the real world. We recommend you to also post this question to stackoverflow.com and tag with CHARTJS in order to get the most attention to your question.
If you do, please link from here to there and vice versa to get the most eyes on this.

closing, unresponsive, no SO link, no JSBIN


feat: added navigation bar with hyperlinks

Pull Request Type
What kind of change does this PR introduce?
Feature
What is the current behavior?
Absence of a navigation bar with hyperlinks
Issue Number: #1541
What is the new behavior?
Added a navigation bar with hyperlinks
Other information

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out our forkcommitmerge.io website repository. The repository contain real issues that you can help to resolve or create some of your own if you see any bugs or want to add new features to the site, etc. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Specifying the iframe element directly

The iFrameResize([{options}],[selector]) call only accepts a string selector as its second parameter. I've got code that is creating iframes dynamically with no unique ID or selector. I'd be great if this function could also accept simply the iframe Element as its second parameter.

Released in v2.8.1.


Either Create an about page or remove it from the index page navbar

Currently, there is a dead link to an /about page. in the top navbar of the index. This issue could be extended to generally restyling the top bar on the index page.

I believe that @jeanniedavis mentioned something about the about page before but I don't know if she actually ended up working on it at all.

this is complete!

I linked it to your pull request so this issue will be closed when that branch is merged in.


Plugin disables itself - Call  to a member function add_hint() on null in file

2020/03/24 07:44:45 - CMDPHP ERRORS DETECTED - DISABLING PLUGIN 'weathermap'
--
2020/03/24 07:44:45 - ERROR PHP ERROR in  Plugin 'weathermap': Call  to a member function add_hint() on null in file:  /var/lib/cacti/plugins/weathermap/lib/Weathermap.class.php on line: 2260
2020/03/24 07:44:45 - WEATHERMAP WARNING: [Map 3] bpce.conf: Unrecognised config on line 195: BANDWIDTH 10G
2020/03/24 07:44:45 - WEATHERMAP WARNING: [Map 3] bpce.conf: Unrecognised config on line 194: VIA 294 474

This may have something to do with the bpce.conf file.   Can you make sure it does not contain any errors or temporarly disabled/move it and test with a plain config file ?

@thurban I started digging in when I didn't hear on this. There are quite a few errors actually, but it was a working map in the original 0.98a (IIRC) version of Weathermap. Do you want the errors it's throwing?

@misterjmw, please note below...
This issue will be resolved once this pull request is merged: https://github.com/Cacti/plugin_weathermap/pull/39


remember the size of the screen

i find the default window size to be "too large". Can we have a setting that lets me save the size of the screen so the next time it opens it's not 'ginormous'?

For some reason this particular app always opens with an extremely large and extra wide shape on my primary display which has a geometry of 21:9 ultrawide. It makes no difference what was the most recent shape when I closed the app or which display it was on when I closed it. Observed with 3.0.2 on Linux.

I will add a "Max Size" for open right now it does like 80% of the screen

Size is maintained between sessions now


No useless index in import paths

As discussed in #394, this PR adds an option noUselessIndex to detect (and fix) paths containing unnecessary index parts (with or without file extension):
Given the following file structure:
my-project
├── foo.js
└── foo
    └── index.js
└── bar
    └── index.js

Before:
import 'foo/index' // No warning
import 'bar/index' // No warning

After:
import './foo/index.js' // Warning (fixable to `./foo/`)
import './foo/index' // Warning (fixable to `./foo/`)
import './bar/index' // Warning (fixable to `./bar`)

Implementation detail:
I've replaced the sumBy function with reduce to get rid of Lodash (not necessary in this case IMO). A benchmark which compares these 2 alternatives shows no difference between them or a slight advantage for reduce: https://www.measurethat.net/Benchmarks/ShowResult/46787
Fixes #394.

@ljharb I've updated the PR to use an option detectUselessIndex to detect these issue. The rule tries to find ambiguous imports for all extensions provided in:
    'import/resolver': {
      'node': {
        'extensions': [
          // some extensions
        ],
      },
    },

If this setting is not set, ['.js'] will be assumed.
Let me know if that looks like a reasonable approach to you. I'm totally open for different ideas.

Coverage decreased (-0.5%) to 93.651% when pulling 29ad42cd5ecea97a627b3be30eea108ef5fbc57f on timkraut:no-useless-index-in-import-paths into bdc05aa1d029b70125ae415e5ca5dca22250858b on benmosher:master.

Coverage decreased (-0.5%) to 93.651% when pulling 29ad42cd5ecea97a627b3be30eea108ef5fbc57f on timkraut:no-useless-index-in-import-paths into bdc05aa1d029b70125ae415e5ca5dca22250858b on benmosher:master.

Coverage decreased (-0.5%) to 93.651% when pulling 29ad42cd5ecea97a627b3be30eea108ef5fbc57f on timkraut:no-useless-index-in-import-paths into bdc05aa1d029b70125ae415e5ca5dca22250858b on benmosher:master.

I've changed the name of the new option from detectUselessIndex to noUselessIndex and rebased the branch to the latest master commit.

I've updated the PR to include tests for getFileExtensions. As this function returns a Set, I've updated chai.js to the latest version to test for that. This resulted in failing tests which I've fixed as well. See https://github.com/chaijs/chai/issues/781 for the breaking changes (in the case of this plugin, I had to rename .deep to .nested)

I'm struggling a little bit with the support for ESLint 2 and 3. I've fixed the issue that message is missing which resulted in failing tests. ESLint 4 und 5 seem to understand the messageId property. It looks like the failing tests are mostly due to the TypeScript support which isn't working in ESLint 2 and 3 apparently.
I get messages like the following:
errors:
   [ TypeError: traverser.traverse is not a function
         at Object.parseForESLint (/Users/timkraut/Development/eslint-plugin-import/node_modules/typescript-eslint-parser/parser.js:41:15)
         at Object.exports.parse (/Users/timkraut/Development/eslint-plugin-import/node_modules/typescript-eslint-parser/parser.js:79:17)
         at parse (/Users/timkraut/Development/eslint-plugin-import/utils/parse.js:38:17)
         at Function.ExportMap.parse (/Users/timkraut/Development/eslint-plugin-import/src/ExportMap.js:339:15)
         at Function.ExportMap.for (/Users/timkraut/Development/eslint-plugin-import/src/ExportMap.js:323:25)
         at Function.ExportMap.get (/Users/timkraut/Development/eslint-plugin-import/src/ExportMap.js:286:10)
         at Context.<anonymous> (/Users/timkraut/Development/eslint-plugin-import/tests/src/core/getExports.js:332:31)
         at callFn (/Users/timkraut/Development/eslint-plugin-import/node_modules/mocha/lib/runnable.js:348:21)
         at Hook.Runnable.run (/Users/timkraut/Development/eslint-plugin-import/node_modules/mocha/lib/runnable.js:340:7)
         at next (/Users/timkraut/Development/eslint-plugin-import/node_modules/mocha/lib/runner.js:309:10)
         at Immediate.<anonymous> (/Users/timkraut/Development/eslint-plugin-import/node_modules/mocha/lib/runner.js:339:5)
         at processImmediate (timers.js:632:19) ],

Do you have an idea how I might fix those?

@timkraut i've pulled out your chai change into a separate commit on master (it'll be pushed shortly)
If older eslints don't support messageId, let's not use it yet.

Is there anything else I can do to get this merged/released?

@ljharb Could you release this, too? Currently the documentation states something which isn't possible with the release from january.

Nope, only one person can release this package.


list required (gene coordinate change, new genes)

Hi Kim,
At some point could we get a list of
warning, new gene with PMID:21511999
and a list of "warning, gene coordinate change" with PMID:21511999
...we need to put them in these tables
http://www.pombase.org/status/gene-coordinate-changes
http://www.pombase.org/status/new-and-removed-genes
....or we could wait and just make these lists available  as "canned queries", which is the preferred long term option
I think maybe we'd better update the lists in the meantime.....
Original comment by: ValWood

We can manage this  until  we are through the "changes"

@manulera work addressed this 2012 ticket!


Missing images

Bug Report
Expected behaviour
Should see appropriate images for "My workflows" and in registration wizard.
Actual behaviour
See the text "menu" and "refresh" oddly aligned.
Looks like "Done" in the "Register Workflow" wizard.
Likely other places using material as well.

Works on staging


Prebuilt CRUD table actions & soft delete filter

Introduce actions and filters for SoftDeletion functionality:
Pages\Actions\ForceDeleteAction
Pages\Actions\RestoreAction
Tables\Actions\ForceDeleteAction
Tables\Actions\RestoreAction
Tables\Filters\SoftDeletesFilter
Todo

[x] ForceDeleteBulkAction
[x] RestoreBulkAction
[x] Fix PhpStan
[x] Add @deprecated notices everywhere
[ ] Write docs
[ ] Implement SoftDelete in Filament Demo

I'll wait for you to sort the PHPStan errors out and then give this a review, the diff is a bit hard to look at with all the comments/notices.

@ryangjchandler PHPStan issues should be resolved now.

Replaced by #2706.


an empty folder causes a panic

Hitting this in parse.go when a directory doesn't contain any go files (may contain other types of files). This error bubbles up and causes a panic to occur.
	if len(fnames) == 0 && len(pkgs) == 0 {
		return Package{}, EmptyErr
	}

Would suggest checking for this error on the function that calls parse and ignoring it?
Difficult to tell what directory is causing this to happen since that isn't included in the error. The project this was ran on seems to have some subdirectories that only contain png or html files. Some other directories do contain .go files.

Seems to be happening on projects that don't have any .go files at the root but have subdirectories containing go code instead. A typical server app

cmd

main.go

internal

server

server.go

docs

docs.go

Dockerfile
go.mod
go.sum

^ This type of directory layout

How do you call gdmd? It doesn’t go to each directory recursively without ‘-r’ flag. The error you see happens when it didn’t find any .go file in a directory without the flag and all subdirectories with it.

Thank you for that information. I did not see that option described anywhere so I assumed it recursed. The gdmd ./ -r command works with -r on the end.


Invalid with unc0ver6.0 A14 iOS14.3

Thanks for this tweak, the only thing to my jailbroken
After installation with unc0ver6.0, iOS 14.3, A14(iPhone12 mini), Only  original Carplay, It looks like the tweak is invalid
And error in settings

same thing here with iOS14.1 and A13.

Same here, iOS 14.2.1, iPhone 12 pro, Unc0ver 6 JB

Same with iphone 11 pro & 14.1

Same here, iOS 14.2 iPhone 12 Pro Max, Unc0ver 6 JB

no issues with ipX 14.3 UC 6 JB..but I cant hide apps

Same issue here with iPhone 11 uncover 6.0 ios 14. Thanks for ur help

Same issue here with iPhone XS uncover 6.0 ios 14.3  Thanks for ur help

Same thin with iphone 11, iOS 14.0.1.
need your help 🥺

Alguien supo cómo se arregla ese problema ?

https://drive.google.com/file/d/106HjlUoQXC3ovPUq_I3Ei3kvcm5jbjGN/view?usp=sharing
Try this deb

Please try the latest build from https://repo.ghostbin.co. Version 1.0.1-1

Please try the latest build from https://repo.ghostbin.co. Version 1.0.1-1

I just did that on my iPhone 12 , iOS 14.3 uncover 6, it caused my phone to reboot every time I try to open the settings page.
Also didn’t let me enter jailbreak mode again , had to disable tweaks injection and then jailbreak from uncover

Please try the latest build from https://repo.ghostbin.co. Version 1.0.1-1

I just did that on my iPhone 12 , iOS 14.3 uncover 6, it caused my phone to reboot every time I try to open the settings page.
Also didn’t let me enter jailbreak mode again , had to disable tweaks injection and then jailbreak from uncover

The only other tweaks I have installed are Plus for Instagram , AudioRecorderXS .
I don’t have apple file conduit installed, could that be it ?

Pruebe la última compilación de https://repo.ghostbin.co. Versión 1.0.1-1

thanks !!

The tweak caused my phone to restart every time I tried opening the settings page.
Problem was solved after installing apple file conduit 2 (afc2). Thanks a lot.

@hungdaddyyy afc2 should not be required. I will try to reproduce this issue and fix. Thanks for reporting


[TASK] update elasticsearch client to be compatible with >2.0

See https://www.elastic.co/guide/en/elasticsearch/reference/current/breaking_20_mapping_changes.html

Hi that's good but I think it's best to organise some version branches here for v1.x and v2.x before merging. What do you think?

Depends if you wan't to continue supporting ES 1.* versions

I think this is handled in the new 0.2 branch. could you confirm?


move std.{outbuffer,signals,socket,xml} to undeaD

I'd be in favor of finally deprecating all sub-standard phobos module (moving them to https://github.com/dlang/undeaD) to make room for good implementations. Having a vacant place might be more encouraging than to wait for something better to come along.
Of course discussing which modules should be obsoleted is a tricky question.
std.signals
std.xml
std.outbuffer
std.socket
std.json // maybe ;)

<EMAIL_ADDRESS>I think this is a good list to start from :+1:
Users should have more than enough time to switch to undeaD and it's really as simple as sed -E 's/std[.](outbuffer|signals|socket|xml)/undead.\1/' -i **/*.d
Sub PRS

[ ] Hide modules from the sidenav on dlang.org
[ ] Move to undeaD

CC @MartinNowak @andralex

I'm against this as is, here's why:

std.signals can be saved IMO
Is there something wrong with outbuffer? I've literally never heard anyone complain about it
std.socket, std.json and std.xml shouldn't be deprecated until we have semi-stable replacements in experimental

We have good opportunities to improve the obsolescence/deprecation process.
The fundamental rule should be: deprecation is accompanied by a better option. There should be a clear, simple, and well-documented migration path from the deprecated facilities to the new ones. There would be reasonable exceptions if e.g. the deprecated facility was hardly used etc.
I've said a number of times (sorry for repeating myself) that we should predate a deprecation cycle by a marginalization process, meaning the documentation has special interior design and UX for features that are still working, but are not the preferred way to do things. Documentation should clarify which better facilities are preferable. That's always the right step. This really does need to happen, can I please summon @MartinNowak and @CyberShadow to help with that.
So let's get better things available before deprecating the not-so-nice things. Thanks.

We should wait until we have some semi-decent replacements for these, otherwise a lot of code will be just broken.
To @lodo1995 : Please, hurry up with your project, a faster XML parser would be a pretty good thing for the whole D language.

Having a vacant place might be more encouraging than to wait for something better to come along.

Pretty sure that's fallacious, and might contribute to the replacement being rushed.

std.socket

This will break essentially all D programs that do anything network-related, including even Vibe.d.

and it's really as simple as

Not unless undeaD becomes included with the language - otherwise, there is package management configurations to deal with. If your project is not using Dub, now it either must, or deal with undeaD's directory structure that precludes using it as a package submodule. Many projects which depended only on things in Phobos will now require additional dependencies to build. Etc.
I could see projects being updated with a reasonable amount of effort to get by without std.signals (which hails from the D1 days, before closures) or std.outbuffer (same, predates appender), but there's no easy way to replace std.xml (without new third-party dependencies) or std.socket (at all).

Related: we may be able to help the deprecation of the string pattern matching by means of changelog and documentation examples.

@andralex Yes, that's on my todo list this weekend.

Point was we're not seeing decent replacements, making space would hopefully be encouraging to contributors. As long as new people keep using std.xml and std.socket, we're causing more grief and accumulate more crappy D legacy code in our ecosystem.
Yes, marking modules unstable/outdated/deprecated as is done in NodeJS's documentation is simple and effective to inform people.

Point was we're not seeing decent replacements, making space would hopefully be encouraging to contributors.

That's a hypothesis. What evidence would substantiate it?

Yes, marking modules unstable/outdated/deprecated as is done in NodeJS's documentation is simple and effective to inform people.

Great, I think starting with documentation should be the official way of implementing all deprecations.

Yes, marking modules unstable/outdated/deprecated as is done in NodeJS's documentation is simple and effective to inform people.
Great, I think starting with documentation should be the official way of implementing all deprecations.

Okay, I gave this a try: https://github.com/dlang/dlang.org/pull/1778
Once agreed / merged, we can slap a simple Ddoc macro like $(COMPATIBILITY_BOX_0) on all modules.


Basic sandbox refinement

The basic layout is here but need some refinements for later:

[x] Category limit in settings does not match anymore the category list layout which is now on 4 category by row, so it must be multiple of 4, i recommend a limit to 8 at least or 12 at maximum;
[x] Migration to Bootstrap 5.1.x ?
[x] Category detail content is not really styled, it's a little bit rude. It need basic styling;
[x] Article states should be reflected with icon(accumulated? or keeping the most important one when there is multiple states?) (using Bootstrap icons);
[x] Category and Article images are very basic blue squares, this never match any of available space. Article cover should be a little taller, Category cover may keep its square size, Article large image need something more large like an image for a Hero component. Finally, either each image have a different color (but will be a little ugly) or we create something more like common placeholder (grey image with its size written in or a text like "Article cover", etc..);
[x] There is no breadcrumb #23;

Originally posted by @sveetch in https://github.com/emencia/django-blog-lotus/issues/21#issuecomment-878577862

Closed with commit 1c56a5d (commit message was closing the issue 29 instead of this one).

Closed with commit 1c56a5d (commit message was closing the issue 29 instead of this one).


global connection is not released on interrupted requests

I am not sure if I should open this bug here or on Quart. Here is the problem: when a request is interrupted by the client (resulting in HTTP 499 if you have nginx in front of Quart for instance) the after_request_funcs are not called in Quart, so after_request is not called in Quart-DB, so it leaks a connection. Ultimately the connection pool is saturated and Quart will stop processing requests (acquire will block).
To reproduce, create a route that returns megabytes of data, call it with cURL and kill cURL while it is running 10 times in a row. After that Quart should block.
A workaround is to disable QUART_DB_AUTO_REQUEST_CONNECTION.

Should be fixed in 94a68e10b74176c966ff2741d68dcc42ea69a362

Thanks, I tried the fix and can confirm it works.


Set node labels for all supported versions of k8s (fixes #750).

this PR needs to be rebased to pass CI (needs to pick up e2e CI notifications)


how to create without class for Manually create and use a volume

Do you have a tech article for ARO/OCP to use without storage class? So with Manually create and use a volume with Azure Files share.
comparable like this for AKS https://docs.microsoft.com/en-us/azure/aks/azure-files-volume

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

ID: 3baa66d8-0696-9a88-dafd-567dab6d2eb5
Version Independent ID: d8e47064-b974-982b-b01b-638dac4c6831
Content: Create an Azure Files StorageClass on Azure Red Hat OpenShift 4
Content Source: articles/openshift/howto-create-a-storageclass.md
Service: azure-redhat-openshift
GitHub Login: @georgewallace
Microsoft Alias: gwallace

@TiloGit Thanks for reaching out. We will investigate and update the thread accordingly.

@TiloGit I'd recommend posting your question in https://azure.microsoft.com/en-us/support/community/ since it's not directly related to the documentation. We'll follow up there. We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@SumanthMarigowda i was using the AKS link. Glad to hear that this is also the official MS way of doing it in ARO/OCP.


IOU - In distance request paid transaction,receipt delete option is shown.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.3.83-1
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Go to https://staging.new.expensify.com/
Tap on any Workspace chat
Tap request money-- distance
Enter start and finish waypoints
Tap next
Tap request amount
Tap on the transaction
Tap pay elsewhere and pay it
Tap on the receipt and open 3dots
Tap delete receipt

Expected Result:
In paid transaction, delete option must not be shown or if shown, receipt must be deleted if tapped on delete
Actual Result:
In paid transaction,receipt delete option is shown. But, tapping on delete, neither receipt is not deleted nor does not trigger any action
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Android: Native

Android: mWeb Chrome

https://github.com/Expensify/App/assets/78819774/89a4525d-3ae7-4ac6-8b2e-cfc6850f70b6

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

@ishpaul777 are you available to submit a PR right now?

@ishpaul777 Give me a second while I get another set of eyes on your proposal

CPed to staging

Not 100% sure if I'm best manually updating this to hold-for-payment 2023-10-20, or adding it to this one. Checked in with @joekaufmanexpensify , switching to daily so I keep it up to date.

@ishpaul777 - the root cause here as the same as https://github.com/Expensify/App/issues/29452 right?

Thanks for the quick clarification.
Summarizing payouts for this issue:
Contributor: @ishpaul777 $750 (inc. urgency bonus)
Upwork job: https://www.upwork.com/nx/wm/workroom/35123916/overview

Ah shoot, I was confused about the step-ordering here across the two issues - I should not have paid this out until this BZ Checklist was completed. Keeping this open with an eye on that in the meantime.

(I think it's fine as that checklist is waiting on the C+ - I'm just leaving a clear paper trail to save my poor, poor brain from having to remember things)


Use NextJS for better web performance

React docs itself suggesting to use frameworks so NextJS would be great for this project.
Why NextJS?

SEO friendly
server side rendering
file based router helps in developing
static site generation

And more...

@RakeshSangem while I agree with your points, Bloom is a CFP management tool which means it involves:

Lots of dynamic content generation
User interaction

For above cases, SSR actually slows down because it'll take time to fetch the compiled JS files from server and render an empty HTML page which is bad UX. If it were a static site, then Next.js would be perfect. I hope that answers your question. 😃

@RakeshSangem I'm assuming you have no further questions so I'm closing this issue. In the future, you can raise your questions in Discussions. Have a great day ahead  🙌🏻


Ability to Add a Horizontal Separator in Profile List

I suggest to include ability to add a horizontal separator in profile list, similar to the existing separator line just above Settings option. This can be done via a { "lineBreak": "horizontalLine" } or similar entry in settings.json. This would be helpful to especially separate types of profiles such as terminal, ssh, etc.

Thanks for the suggestion! Please make sure to search for duplicates when filing new issues. This is already being tracked by another issue on our repo - please refer to #1571 for more discussion.
/dup #1571


Implement Luhn

Closes #237
This mostly follows the current test suite
(https://github.com/exercism/x-common/issues/491), with one change.
The final canonical test adds an alphabetical character to a known
invalid string and then asserts that it's still invalid.
Instead what the test should do is alter a known-valid string and show
that it is now invalid.
That's what I've done here. I'll also propose the change to canonical
test suite.

I haven't placed the problem yet, so this can not be merged.

Thinking about placement. I don't think my solution is that far from what people will do. The existing crate uses another crate to handle digits > 10 after doubling. If people go down that path their solutions will vary a bit from mine, but not by a bunch.
So the solution mostly requires a good handle on

Converting a string to digits
Working with iteration

I would put it between Sum of Multiples and Scrabble Score.

This works, but I've been thinking about some other, more Rusty ways of approaching it.
Option 1: Use From
This way we could validate strings and different types of unsized ints. The test API would look like:
let luhn = Luhn::from("012345");
assert!(!luhn.is_valid());
let luhn = Luhn::from(12345);
assert(!luhn.is_valid());

Option 2: Use Traits
Students would create a LuhnValidator trait and then implement it for different types. Test API would look like:
let string_luhn = String::from("012345")
assert!(!string_luhn.valid_luhn());

let u8_luhn: u8 = 8;
assert!(!u8_luhn.valid_luhn());
//and so on

Option 3
Do all of them. Have 3 Luhn exercises that show different ways of solving the problem, so that students can see each approach.

I know you already placed, but I'm going to make my placement comment without looking at where you placed to avoid bias, and if our ideas match up then great.
Place no later than robot-simulator.
Place no earlier than hamming.
Fantastic, we agree.

EBWODP:

Yeah, interesting way to add functions to existing types. Do we have any exercise that already does this?

Looking at our git grep "impl.*for", I guess not.

Just this week I was trying to figure out how to monkey-patch Ruby to add methods to the number 2. Not that this is a good idea, but it was interesting. For the same reason I think show traits on primitives could be interesting, though the documentation of the exercise should say that it is not recommended.
I'm going to experiment with Option 3 and will either update this PR or create a new one.


Fix klog.V(x) references for klog v2

klog.V(x) is no longer a boolean. We need to use the .Enabled() method

/assign @bowei @spencerhance
cc @tanjunchen @serathius

get it

This is not quite sufficient -- you need to fix references in the code generator.

https://github.com/GoogleCloudPlatform/k8s-cloud-provider/compare/master...bowei:fix-klog?expand=1

@bowei thanks! pulled in your two commits.


Added getChatDetails

@adamlui nothing (it has default values). it's asynchronous, so you need to use await or then()

Ok I'm making it more user friendly

sure, make the detail parameter be one of these:

@madkarmaa ok it's user-friendly now, the next logical step is to make it also allow supplying id (long string from detail) or title as the 1st arg. also i'm not merging yet because I have to setup a workflow to auto-update kudoai/chatgpt.js-chrome-starter's lib code + version on all edits to chatgpt.js

@madkarmaa you could also add it to userguide.md for when I finally style chatgpt.js.org to display it pretty and begin linking to it from main readme

sure, I'll add a first argument

@madkarmaa you mean to the function? it's a lot harder than it sounds, it has to allow user to supply index like it currently does, but also loop thru all conversations fetched to parse for 'title' match if title was supplied (or long string id if that's supplied)

but if that's what you meant you were gonna do, to make it even more user friendly you should use partial matches via regex

well it's not really that difficult. by destructuring what we need to do, I can check if idx is a number and use it as a list index, else I do a for-loop to match the title or id with the respective title or id in the list items

but doing it with as few lines as possible is the hard part, if you can include partial matching case insensitively within a few lines then it would be perfect

also regex should be used to auto-detect if it's a long string id (they all follow a certain format with hyphens)

if (idx === item.[title/id]) is not possible? 😂

it certainly is if you don't make it user friendly, again i said making it partial match case insensitively for max user experience with few lines and regex as possible is the challenge (not that challenging for me but if you want to do it will take you much more time than you anticipate i predict)

=== is an exact match

actually you can do the === and i can add the flexible matching

I can use a ternary operator (condition ? true : false) like this:
toCheck = idx.test(idRegexp) ? 'id' : 'title'

...

if (idx === item[toCheck])

i renamed idx to i btw but it should be renamed to chat since the arg is expanding, also item[toCheck] is not the right sub-obj it's actually nested one more (items.i.details)

@madkarmaa it's updated in your fork

(any changes made in your fork by anyone show up as commits in this pr)

@madkarmaa I just wrote an epic workflow https://github.com/kudoai/chatgpt.js/blob/main/.github/workflows/auto-sync-chatgpt.js-to-chrome-starter.yml I'm aching to test so i'ma merge now and you can create new pr


fix: fix the CODEOWNERS format issue

Fix the format issue of CODEOWNERS file

The owners should be on one line

Signed-off-by: Yi Zha<EMAIL_ADDRESS>

Codecov Report

Merging #564 (3e7a880) into main (5e2fa4a) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #564   +/-   ##
=======================================
  Coverage   34.36%   34.36%
=======================================
  Files          32       32
  Lines        1848     1848
=======================================
  Hits          635      635
  Misses       1192     1192
  Partials       21       21

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


Thank you for making this

We are trying to do our own swiper react component and it's working ok, but it will be much better to have a singular one we can all work on. Thank you!!!

Hi @timkindberg , thank you for your support. I hope we can make this library better for community .


Honeypot added to Youtube section

Honeypot produces monthly documentaries exploring tech culture, influential technologies and untold developer stories from around the world.

Hi Zlatan. I think we should concentrate more on coding related channels. If you know any other channels that fit more in this group, I encourage you to add them.
Thanks.

Hello @MarkoDenic this sounds fine. I know few more great channels. Will add them soon.


Document how to invoke embedded scripts from custom scripts

PTAL @bparees
https://trello.com/c/FJ7ZVLhR/1016-3-document-and-investigate-s2i-exit-hooks

@php-coder one more round of changes and then i think it's good.

@bparees Thank you for your feedback! PTAL again.

@php-coder squash and lgtm

@bparees Squashed and ready to merge.

[merge]


Fix TrelloAPI requiring Token

By sending a http request to https://trello.com/1/members/me, it would return 400 if there was no token.
Changing this to some constant like https://trello.com/1/members/trello will work with a valid app key. With an invalid app key it returns an error. No token is required.
This PR also adds in a check for httpenabled before running the trello validity check.
I am open to criticism with this.

What confuses me about some of the Trello codebase is the fact there is multiple HTTP checks, one inside the Update function in HTTP and inside the TrelloAPI module too? (there probably should be a check in HTTP.lua before requiring TrelloAPI but I'm pretty sure the API is supposed to be versatile for other developers wanting to use it in their modules too that aren't Adonis)

Yes there are multiple HTTP checks. This check was added because previously it would fail the http check and just say "Please make sure Http requests are enabled", even if the trello test errored.


RoyLFine.WinHTTPVPNNetworkExercisor version <IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@ItzLevvie, The install mode prompt is expected behavior. The install mode allows the user/installer to install for current user or install with machine level access

changing installer mode - will resubmit


No module named 'pygslib.drillhole' on 'import pygslib'

I followed the install from source directions. The installation finishes with no errors or warnings.
Did I miss something?
opengeostats and pygslib are not found when using conda install -c opengeostats pygslib.
Thanks

Hi leonfoks,
I am closing this issue becasue we will be moving to a new conda distribution based on wheels. This will allows installation of compiled code with pip.