Templates.ParseText returns an error for legit lg content

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Version
What package version of the SDK are you using.
<IP_ADDRESS>
Describe the bug
Templates.ParseText method returns an error for the following template:
# example()
> To learn more Adaptive Cards format, read the documentation at
> https://docs.microsoft.com/en-us/adaptive-cards/getting-started/bots
- ```
${json({
  "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
  "version": "1.2",
  "type": "AdaptiveCard",
  "body": [
    {
      "type": "TextBlock",
      "text": "default text",
      "weight": "bolder"
    }
  ]
})}
{replace this with triple `}

To Reproduce

Call Templates.ParseText() and inspect Diagnostics list

Expected behavior
No errors

Hi, @Robulane. I use your content but could not reproduce this issue. And the diagnostic list is empty. Could you provide the full LG file, lest I miss some special non-printable characters? Thanks.

Hm... strange, this is what I am getting:

This is the value of content:
"\r\n\r\n# example()\r\n> To learn more Adaptive Cards format, read the documentation at\r\n> https://docs.microsoft.com/en-us/adaptive-cards/getting-started/bots\r\n- \r\n${json({\r\n  \"$schema\": \"http://adaptivecards.io/schemas/adaptive-card.json\",\r\n  \"version\": \"1.2\",\r\n  \"type\": \"AdaptiveCard\",\r\n  \"body\": [\r\n    {\r\n      \"type\": \"TextBlock\",\r\n      \"text\": \"default text\",\r\n      \"weight\": \"bolder\"\r\n    }\r\n  ]\r\n})}\r\n"

hi @Robulane I found that there is a backslash after schema url:
 "$schema": "http://adaptivecards.io/schemas/adaptive-card.json\",

That would break the json.
Could you please remove it and try again?

Another tips: there is no need to write json(), because the parameter itself is json.
json function originally aims to convert json string into a json structure.
So, your template could be
# getJson
- ${json(example())}

# example()
- ```
{
  "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
  "version": "1.2",
  "type": "AdaptiveCard",
  "body": [
    {
      "type": "TextBlock",
      "text": "default text",
      "weight": "bolder"
    }
  ]
}```

or just simplified with:
# example()
- ${{
  "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
  "version": "1.2",
  "type": "AdaptiveCard",
  "body": [
    {
      "type": "TextBlock",
      "text": "default text",
      "weight": "bolder"
    }
  ]
}}

Close it. @Robulane If the solution has not solved your problem or need further discussion, please leave a message under this issue. Thanks！


Improved flow type

hardening up the flow types so that the memoized function is the same type as the function passed in

Codecov Report

Merging #12 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #12   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          15     16    +1
=====================================
+ Hits           15     16    +1

Impacted Files
Coverage Δ

src/index.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

This PR brings the flow coverage up from 64% to 90.6%


Adds after-clean and before-clean events to the tapable system

Coverage decreased (-4.0%) to 89.796% when pulling 43b462737e37fc4efe2cd4dab32c2efa50a8a58b on pastelsky:master into b4c1ab1bc49293e8e928bac72601ad73bdfee7a7 on johnagan:master.


Connecting to remote needs user feedback msgs

Located  successfully... done.
Initializing Remote R process... done.
Closing previous R session... done.

@jflam , @MikhailArkhipov
Do you have any input on this task? I want to estimate it

The idea behind this task is to remove some of the uncertainty behind what the service is doing while you are waiting for the connection. It would also be helpful to use this as a way to report errors as well. It would be helpful if you could provide a list of the distinct "events" that comprise the time taken to connect to the service. Here's some examples without knowing the specifics:

Looking up the name of the remote server
Establishing HTTPS connection to the remote server
Creating user profile on the remote server
Creating R session on the remote server
Connected

For example, if it fails on #1, we could say " -- could resolve the name of the remote server". If it fails on #2, we could say " -- could not establish connection to the remote server".
So I would say, let's come up with a concrete list of the event names. Next, we should figure out what we do in the event of an error: Here's a strawman: the connection dialog halts, display an error message next to the name, along with an OK button to acknowledge the error. When the user clicks OK, we dump a detailed log of what was done and what the last known state was to the console window to make it easier for the user to copy / paste into a report about the error - keep in mind that those reports may be sent to their local admin vs. us, the RTVS team.

what john said.  +1.
On Mon, Nov 28, 2016 at 11:17 AM, John Lam<EMAIL_ADDRESS>wrote:

The idea behind this task is to remove some of the uncertainty behind what
the service is doing while you are waiting for the connection. It would
also be helpful to use this as a way to report errors as well. It would be
helpful if you could provide a list of the distinct "events" that comprise
the time taken to connect to the service. Here's some examples without
knowing the specifics:

Looking up the name of the remote server
Establishing HTTPS connection to the remote server
Creating user profile on the remote server
Creating R session on the remote server
Connected

For example, if it fails on #1 https://github.com/Microsoft/RTVS/pull/1,
we could say " -- could resolve the name of the remote server". If it fails
on #2 https://github.com/Microsoft/RTVS/pull/2, we could say " -- could
not establish connection to the remote server".
So I would say, let's come up with a concrete list of the event names.
Next, we should figure out what we do in the event of an error: Here's a
strawman: the connection dialog halts, display an error message next to the
name, along with an OK button to acknowledge the error. When the user
clicks OK, we dump a detailed log of what was done and what the last known
state was to the console window to make it easier for the user to copy /
paste into a report about the error - keep in mind that those reports may
be sent to their local admin vs. us, the RTVS team.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/RTVS/issues/2683#issuecomment-263366065,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAtUVP1wbTTkBx0-DCVNYvhoKZkNqjrqks5rCyjggaJpZM4KvCQE
.

Right now, we have 2 separate steps:

Start RHost for the new broker
Restart RSession

We can split them further, but I'm not sure that will give any valuable information to the user.

Correct, we don't really have all those stages. We don't look for machine first, it is part of Http request and is internal to the stack. There is no feedback from the server on stages it is coming through.
Also,
https://github.com/Microsoft/RTVS/pull/2766
https://github.com/Microsoft/RTVS/issues/2749
make connection switching much faster

I'm mostly looking at this through the lens of providing useful information for the user. So I don't mean literally translating what we do internally into a sequence of steps, but rather providing the user with a logical set of things that we do that can fail, and showing that to the user. i.e., we can say "looking up server" as part of the initial HTTP connection so that we can fail with message 1 below. Assuming that the connection succeeded but the server doesn't have a cert installed, we can fail with message 2 below. etc.

So, is there a distinguished exception that is returned from the HTTP stack if the user mis-typed the DNS/NETBOIS name of the server? If so, we should have a clear message that tells them that.

If the server is misconfigured somehow such that HTTPS is not enabled, what error message do we provide the user?

If the user profile fails to be created on the server (thereby rendering the entire environment more or less useless), what error message do we report to the user, and how is this actionable for them or their server admin?

1 - yes, it should output exception message similar to 404
2 - yes, most probably returns 'server is online but Remote R service is not running. I don't think we can tell from the client side that certificate is misconfigured as broker won't even start if there is no certificate (logged at the server side).
3. @karthiknadig - is there any diagnostics or does user just get 401 Unauthorized?

3 is currently reported as an authentication failure to the User. The ETL events, for the R user profile service, contains details of what happened.

User probably has little knowledge or means what has to be done on the server. However, admin can look at logs and system events per complaint from the user.

So, bottom line
Looking up the name of the remote server
Establishing HTTPS connection to the remote server

Are part of the same call. We do not manually look up DNS.
Creating user profile on the remote server

We don't know if it is being created. Broker has no means to report this.
So basically there are only two steps:

Establish connection
Create session.

We've added more information to errors, shiny messages etc. We don't have more information at the moment.


Use SIMD where possible?

SIMD instructions are going to be available in ECMAScript 7, and are already supported in Firefox and Crosswalk. Plus, there's a Typed Arrays based polyfill available at https://github.com/johnmccutchan/ecmascript_simd. For projects using convnetjs, especially on mobile, SIMD might make everything a bit faster.

interesting! thanks for pointing this out, I wasn't previously aware of it. I think the ultimate frontier of JS DeepLearning is still on GPU. It would be ideal if there was some Tensor object in JS with various operations that can happen on GPU or CPU. The CPU part could maybe use the SIMD instruction set for additional speed when on CPU.
Cool stuff!

Perhaps Typed Objects (again, Firefox only for now) and Parallel JavaScript might be worth a look? Typed Objects could make going between CPU and GPU pretty easy.
I might fork the repo and see if I can get SIMD and Typed Objects working in Firefox.

@karpathy I've been playing around with Typed Objects and to me it seems to best way to use Typed Objects would be to have a VolType class that extends StructType. This would allow Vols to be very memory efficient and performant, and allow efficient arrays of Vols. Sounds pretty good to me :smile:.
This is a (broken) implementation of VolType:
class VolType extends TypedObject.StructType {
  constructor(sx = 1, sy = 1, depth = 1){
    super({
      sx : TypedObject.uint32,
      sy : TypedObject.uint32,
      depth : TypedObject.uint32,
      w : TypedObject.float64.array(sx).array(sy).array(depth),
      dw : TypedObject.float64.array(sx).array(sy).array(depth)
    });
  }
}

The VolType class could be used like this:
// For example, 32x32 RGBA images from a canvas
let MyVolType = new VolType(32, 32, 4);
let aVol = MyVolType.from(randomImageData);

The above would be equivalent to calling:
let aVol = new Vol(32, 32, 4, randomImageData);

Typed Objects allows multidimensional typed arrays, so the add(), set() and get() methods wouldn't be needed, same goes for addGrad(), setGrad() and getGrad().
cloneAndZero() is also not needed, because another instance of a VolType is used instead. clone() is replaced as well, as passing a Vol to an instance of VolType would perform a clone.
The only huge downside to this is the Typed Objects are only available in Firefox beta builds, and as they are a part of the ECMAScript 7 standard, they won't be standardised until as late as September 2016. I'm sure Typed Objects could be polyfilled (just as SIMD and Typed Arrays can, just with lesser performance), but one would have to be written from scratch.

Nice, thanks for looking into this.
Maybe it's possible to check if the user has TypedObject.float64 arrays available and if so, use them under the hood. We already do this with the implementation of zeros(), using typed arrays when possible.
Unfortunately, I think it's not very clean to put this in with the other code base because I traded speed for modularity, and have all kinds of math implemented inline, all over the place, and in hardcoded ways.
Maybe I should restart ConvNetJS from scratch... :)

I'd agree that using Typed Objects won't go very well with the current codebase.
I ended up actually sort of rewriting ConvNetJS anyway (I turned the concatenation of files into ES6 modules, and turned the pseudo-classes used into ES6 Classes), so you if you take a look at thomasfoster96/convnetjs the /src directory looks completely different. Not to mention using Typed Objects requires incompatible changes to the way Vols work.


Add optimized implementation for string indexes

The new index is meant for opaque identifiers that can only be retrieved via equality lookups. If a string type contains the #index=hash attribute, the index factory chooses this new hash-based implementation.

I haven't had the time yet, but it would be great to have some numbers on the performance delta. Ideally for both space and time.

There's a fundamental issue I overlooked in the design: the index does not support inequality lookups. It can't because it hashes the input. We'll have to reconsider the usefulness of a space-efficient equality-only string index before proceeding.

The commit ede727e seems to have triggered an ADL correctness issue, causing the wrong operator to be called and thus the wrong type name to be displayed. That should likely be fixed in a separate PR.

@dominiklohmann Sorry, I fixed the issue already before seeing your request to do this separately.


cube.js example is broken.

The cube.js example is broken because the lena texture can't be found.

We should replace the lena image with peppers or the baboon.


BN-LINK BNC-60/U133TJ-2P Power Monitoring Plug

Before opening a new issue, please read the following
Are you reporting a successful flash?
No firmware block
BN-LINK BNC-60 just purchased but appears to come with updated firmware (tuya convert ......, device blinks and goes solid but never finishes). The device is difficult to serial flash because the rx/tx lines need to be removed from the board connections (maybe the energy mon runs over those lines)

Happy to share that support for HTTPS firmware is in the works #279


GPIO low

how to make nodemcu esp8266 GPIO LOW

Hey there ...
got the same problem ... i set the output D1 and D2 to low, but when i boot my NodeMCU
this GPIO Pins are high. I can reproduce that also with D3 and D4 ... when i edit the GPIOs in
my config file.
What can i do?

The readme file is really not describing it well, please fix.
Basically when the esp boots up i.e. power on or reset, then it reads the GPIO-00 pin.  If the pin is LOW then the esp goes into listening mode awaiting a firmware update.
That all happens before any of your programs run.  This can only be done with a wire connected to the GPIO-00 pin bridging it to Ground on boot or reset.

Connect a wire bridging GPIO-00 to ground (on a wemos board this is pin D3)
While the wire is connected power the esp on on or reset it, either way.
Now click the Flash button in the nodemcu-flasher and watch the progress bar as your esp is getting its firmware updated.

Hope this helps.


Upgrade to CakePHP 3.5.14 and layout from cakephp-app quickstart

Adopted the standard CakePHP app layout
Divided up the sample content between layout
Pinned a couple dependencies at PHP 7.0 compatible versions

@hhorak @pkubatrh ptal

I guess this is a candidate for testing the staging branch (something we talked about when discussing how to not break things when changes in example applications are done). @pvalena, can you look at it, please?

There is a known (non-blocking) break in origin, I have a chaser PR there:
https://github.com/openshift/origin/pull/19110

I've tested this manually with released images and it works fine, so merging.


not working on/with servers

as the title says i installed the mod on client and on a forge server, both are forge version 40.1.19 (minecraft 1.18.2) and the mod didnt worked at all. going into a singleplayer world did show the mob-drops of alex's mobs and create ores-spawns, but were missing the cobalt ore spawns from tinkers construct.
going on server (once again: server has mod installed too) did show neither of any of the above
checked the servers logs and debug-logs and both said the mod initialised/loaded without any issue
can provide more info like modlist or logfiles when requested
used mod-version was 3.2.1

Can confirm this!

The version 3.3.0 (in review on curseforge) is fixing this issues, i hope (is working on my dev env). ^^'


saveAs with random name

Hello,
is somehow possible to set for method saveAs use name + postfix(random) or just random name?
Thank You

Generating random names is definitely not in the scope of this library. You can use whatever filename you want? So just generate a random name and use that as file name.

so I can create a variable contains string and pass the variable into saveAs ...?
$pdf->saveAs('$pdfName');

Yes - but without single quotes around the variable.
Sorry, these are really basic PHP questions. I can not tutor you to learn PHP. Please consult a PHP tutorial or something.


[QA] Enhancing UX: Reconsidering adding Cookies Blocked filter

In the latest update, we have made some changes to the interface. The 'Cookie Blocked/Accepted' column and its corresponding filter have been removed. Now, the only option to identify blocked cookies is through the 'Cookie Blocked Reasons' filter. While this filter allows users to pinpoint specific cookies based on their reasons, it lacks the convenience of directly showing the total number of blocked cookies in the selected frame.
To enhance user experience, I recommend retaining the 'Cookies Blocked' filter, which enables users to easily filter out blocked cookies without the need to navigate through each table and scroll. Additionally, the 'Cookies Blocked Reasons' filter currently displays all possible reasons for blocking cookies, causing potential confusion. It would be more user-friendly to show only the reasons relevant to the cookies present, eliminating the need for users to check every filter to determine the category of a particular cookie.
https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/94d1ff65-b12f-4858-a12c-7f66079de6be

Now, an option has been added to the cookie landing page to display the total number of blocked cookies.

cc: @amovar18


Enable replayer tests on CI

Replayer test is enabled in nightly. Linked Improvement to use artificial data instead of recorded one

https://buildkite.com/o-1-labs-2/mina-end-to-end-nightlies/builds/1915#018f2871-bf1a-49a7-ae19-484f6d36673e


Overlay sometimes triggers with only one finger

Sometimes the overlay is triggered with one finger only, which leads to weird scrolling behavior.
This can be fixed by adding
if (details.pointerCount != 2) return;

to onInteractionStart

Amazing, thanks! Will test it  :)

Which phone did you use for this test?

It was an iPhone X and an iPhone 12 Pro


ci: do not try to publish external PRs

GH action should not try to build and publish docker image with external PRs e.g. #51 #52 #63

stale


NUTCH-2633 Fix deprecation warnings when building Nutch master branch…

Suggested fix for https://issues.apache.org/jira/browse/NUTCH-2633
Some persisting deprecation warnings
    [javac] /usr/local/nutch/src/java/org/apache/nutch/plugin/PluginRepository.java:307: warning: [deprecation] finalize() in Object has been deprecated
    [javac]   public void finalize() throws Throwable {
    [javac]               ^
    [javac] /usr/local/nutch/src/java/org/apache/nutch/indexer/IndexWriters.java:209: warning: [deprecation] open(Configuration,String) in IndexWriter has been deprecated
    [javac]       entry.getValue().getIndexWriter().open(conf, name);
    [javac]                                        ^

    [javac] /usr/local/nutch/src/plugin/index-geoip/src/java/org/apache/nutch/indexer/geoip/GeoIPDocumentCreator.java:116: warning: [deprecation] isAnonymousProxy() in Traits has been deprecated
    [javac]     doc.add("isAnonProxy", traits.isAnonymousProxy());    [javac]

/usr/local/nutch/src/plugin/indexer-cloudsearch/src/java/org/apache/nutch/indexwriter/cloudsearch/CloudSearchIndexWriter.java:89: warning: [deprecation] open(Configuration,String) in IndexWriter has been deprecated
    [javac]   public void open(Configuration conf, String name) throws IOException {
    [javac]               ^
    [javac] 2 warnings

    [javac] /usr/local/nutch/src/plugin/indexer-csv/src/java/org/apache/nutch/indexwriter/csv/CSVIndexWriter.java:217: warning: [deprecation] open(Configuration,String) in IndexWriter has been deprecated
    [javac]   public void open(Configuration conf, String name) throws IOException {
    [javac]               ^
    [javac] 2 warnings

    [javac] /usr/local/nutch/src/plugin/indexer-dummy/src/java/org/apache/nutch/indexwriter/dummy/DummyIndexWriter.java:44: warning: [deprecation] open(Configuration,String) in IndexWriter has been deprecated
    [javac]   public void open(Configuration conf, String name) throws IOException {
    [javac]               ^
    [javac] 2 warnings

    [javac] /usr/local/nutch/src/plugin/indexer-elastic/src/java/org/apache/nutch/indexwriter/elastic/ElasticIndexWriter.java:78: warning: [deprecation] open(Configuration,String) in IndexWriter has been deprecated
    [javac]   public void open(Configuration conf, String name) throws IOException {
    [javac]               ^
    [javac] 2 warnings

    [javac] /usr/local/nutch/src/plugin/indexer-elastic-rest/src/java/org/apache/nutch/indexwriter/elasticrest/ElasticRestIndexWriter.java:100: warning: [deprecation] open(Configuration,String) in IndexWriter has been deprecated
    [javac]   public void open(Configuration conf, String name) throws IOException {
    [javac]               ^
    [javac] 2 warnings

    [javac] /usr/local/nutch/src/plugin/indexer-rabbit/src/java/org/apache/nutch/indexwriter/rabbit/RabbitIndexWriter.java:68: warning: [deprecation] open(Configuration,String) in IndexWriter has been deprecated
    [javac]   public void open(Configuration conf, String name) throws IOException {
    [javac]               ^
    [javac] 2 warnings

    [javac] /usr/local/nutch/src/plugin/indexer-solr/src/java/org/apache/nutch/indexwriter/solr/SolrIndexWriter.java:71: warning: [deprecation] open(Configuration,String) in IndexWriter has been deprecated
    [javac]   public void open(Configuration conf, String name) {
    [javac]               ^
    [javac] 2 warnings

    [javac] /usr/local/nutch/src/plugin/language-identifier/src/java/org/apache/nutch/analysis/lang/HTMLLanguageParser.java:34: warning: [deprecation] LanguageIdentifier in org.apache.tika.language has been deprecated
    [javac] import org.apache.tika.language.LanguageIdentifier;
    [javac]                                ^
    [javac] /usr/local/nutch/src/plugin/language-identifier/src/java/org/apache/nutch/analysis/lang/HTMLLanguageParser.java:161: warning: [deprecation] LanguageIdentifier in org.apache.tika.language has been deprecated
    [javac]     LanguageIdentifier identifier = new LanguageIdentifier(titleandcontent);
    [javac]     ^
    [javac] /usr/local/nutch/src/plugin/language-identifier/src/java/org/apache/nutch/analysis/lang/HTMLLanguageParser.java:161: warning: [deprecation] LanguageIdentifier in org.apache.tika.language has been deprecated
    [javac]     LanguageIdentifier identifier = new LanguageIdentifier(titleandcontent);

    [javac] /usr/local/nutch/src/plugin/protocol-ftp/src/java/org/apache/nutch/protocol/ftp/Ftp.java:170: warning: [deprecation] finalize() in Object has been deprecated
    [javac]   protected void finalize() {
    [javac]                  ^

Also, I've gone ahead and implemented a code cleanup where I saw it necessary. Some error handing needs to be checked. We can discuss that on the PR once submitted.
Also note that not all file permissions were identical... you will see this with the changes from chmod 755 --> 644

@sebastian-nagel any comments about our javac.version in default.properties?

Work to make Hadoop ready for Java 9/10/11 is still ongoing (HADOOP-11123). After that and when the Hadoop distributions integrated support for Java 9, users need to upgrade their Hadoop clusters. Just, as an estimate, we probably have to make sure that Nutch runs and compiles on Java 8 for the next 2-3 years. For this time I would prefer to keep the javac.version as is. It avoids that a patch introduces usage of new Java features which do not work on Java 8.

@sebastian-nagel got it, thanks for the detail.


Type '{}' is not assignable to type 'Settings'.     Property 'total' is missing in type '{}'

TypeScript Version:  2.4.2 / nightly (2.5.0-dev.201xxxxx)
There is a problem when I'm using default props with a typed properties interface.
Code
export interface Settings { total: number, activePage: number, visiblePages?: number };

import React, { SFC } from 'react';
import pagination, { Settings } from './utils'

//import 'style.css';

export { Settings };

const Pagination: SFC<Settings> = ({ total, activePage }) => (
    <h1>{pagination({ total, activePage })}</h1>
);

Pagination.defaultProps = { total: 1, activePage: 1 };

export default Pagination;

import { storiesOf  } from '@kadira/storybook';
import { action } from '@storybook/addon-actions';

import React from 'react';
import Pagination from './index';

storiesOf('<Pagination />', module)
    .add('without props', () => (<Pagination />)); // error there in Pagination

Expected behavior:
Use the default properties
Actual behavior:
Type '{}' is not assignable to type 'IntrinsicAttributes & Settings & { children?: ReactNode; }'.
Type '{}' is not assignable to type 'Settings'.
Property 'total' is missing in type '{}'

This is not a support forum.
Questions should be asked at StackOverflow or on Gitter.im.
But, you should try Partial<Settings> instead of Settings.  Right now, TypeScript does not have a good way of modelling default properties for something like React (see #16173).

Ok, thank you


bug: prettier plugin for tailwind write credits and sometime tailwind lsp stopped on vim

What version of daisyUI are you using?
latest
Which browsers are you seeing the problem on?
No response
Reproduction URL
https://play.tailwindcss.com/RGJUjgS2RK
Describe your issue
{
  "bracketSameLine": true,
  "plugins": ["prettier-plugin-jinja-template", "prettier-plugin-tailwindcss"],
  "overrides": [
    {
      "files": ["*.html"],
      "options": {
        "parser": "jinja-template"
      }
    }
  ]
}

that is in the .prettierrc.json config, so when the formatting with prettier run in vim, it will write credits on topline of code like below.

sometime the lsp stopped working.
In other times, i had try it in vscode and no problems with that.

Probably the problem is Vim, using the raw terminal output as the result code. That can be problematic even without daisyUI (for example when Tailwind CSS returns a warning or error to the console log)
As a workaround, you can disable the daisyUI logs from the config file:
https://daisyui.com/docs/config/#logs
Let me know if you have any questions

Thank you, I think this will work. I will try when possible.

Probably the problem is Vim, using the raw terminal output as the result code. That can be problematic even without daisyUI (for example when Tailwind CSS returns a warning or error to the console log)
As a workaround, you can disable the daisyUI logs from the config file: https://daisyui.com/docs/config/#logs
Let me know if you have any questions

Not work :(

Probably the problem is Vim, using the raw terminal output as the result code. That can be problematic even without daisyUI (for example when Tailwind CSS returns a warning or error to the console log)
As a workaround, you can disable the daisyUI logs from the config file: https://daisyui.com/docs/config/#logs
Let me know if you have any questions

Not work :(

mine is work

Probably the problem is Vim, using the raw terminal output as the result code. That can be problematic even without daisyUI (for example when Tailwind CSS returns a warning or error to the console log)
As a workaround, you can disable the daisyUI logs from the config file: https://daisyui.com/docs/config/#logs
Let me know if you have any questions

Not work :(

mine is work

Pake neovim apa vim biasa?

must be placed at the top in daisyui object
import type { Config } from "tailwindcss";
import { Config as DaisyUiConfig } from "daisyui";
const config: Config & { daisyui: DaisyUiConfig } = {
  content: [
    "./src/pages/**/*.{js,ts,jsx,tsx,mdx}",
    "./src/components/**/*.{js,ts,jsx,tsx,mdx}",
    "./src/app/**/*.{js,ts,jsx,tsx,mdx}",
  ],
  theme: {},
  plugins: [require("daisyui")],
  daisyui: {
    logs: false, // <-- place here
    themes: [
      "light",
      "dark",
      "cupcake",
      "bumblebee",
      "emerald",
      "corporate",
      "synthwave",
      "retro",
      "cyberpunk",
      "valentine",
      "halloween",
      "garden",
      "forest",
      "aqua",
      "lofi",
      "pastel",
      "fantasy",
      "wireframe",
      "black",
      "luxury",
      "dracula",
      "cmyk",
      "autumn",
      "business",
      "acid",
      "lemonade",
      "night",
      "coffee",
      "winter",
      "dim",
      "nord",
      "sunset",
    ],
  },
};
export default config;


els-blueprints: When destroying the whole network resources (VPC, Subnet, Routetable, NACL, SG) are left

Describe the bug
When calling cdk destroy all or most of the networking resourcing was left un-destroyed.
Expected Behavior
All the resources where destroyed
Current Behavior
Resources like (VPC, Subnet, Routetable, NACL, IGW, NetworkInterfaces, SG). I had to find a Gist that I found to identify and then destroy them manually.
Reproduction Steps
Added a list of Addons
       // AddOns for the cluster.
        const addOns: Array<blueprints.ClusterAddOn> = [
            new blueprints.addons.FluxCDAddOn,
            new blueprints.addons.SSMAgentAddOn,
            new blueprints.addons.ClusterAutoScalerAddOn,
            new blueprints.addons.AwsLoadBalancerControllerAddOn(),
            //new blueprints.addons.VpcCniAddOn(),
            new blueprints.addons.CertManagerAddOn(),
            new blueprints.addons.ExternalDnsAddOn({
                hostedZoneResources: [blueprints.GlobalResources.HostedZone]
            }),
            new blueprints.addons.EfsCsiDriverAddOn({kmsKeys: [kmsKey]}),
            new blueprints.addons.EbsCsiDriverAddOn(),
            new blueprints.addons.IngressNginxAddOn()
        ];

Then created the cluster:
        const stack = blueprints.EksBlueprint.builder()
            .version('auto')
            .account(account)
            .region(region)
            .clusterProvider(clusterProvider)
            .resourceProvider(blueprints.GlobalResources.Vpc, new blueprints.VpcProvider(undefined, { primaryCidr: envContext.vpcCidr }))
            .resourceProvider(blueprints.GlobalResources.HostedZone, new blueprints.ImportHostedZoneProvider(r53HostedZone.hostedZoneId, hostedZoneName))
            .resourceProvider(blueprints.GlobalResources.KmsKey, new blueprints.CreateKmsKeyProvider())
            .resourceProvider("s3-bucket", new blueprints.CreateS3BucketProvider({
                name: envContext.s3BucketName+'.'+account+'.'+region,
                id: envContext.s3BucketName,
                s3BucketProps: { removalPolicy: RemovalPolicy.DESTROY },
            }))
            .addOns(...addOns)
            .build(this, 'my-eks-blueprint');

Possible Solution
No response
Additional Information/Context
No response
CDK CLI Version
2.147.3 (build 32f0fdb)
EKS Blueprints Version
1.15.1
Node.js Version
v22.2.0
Environment details (OS name and version, etc.)

sw_vers ProductName: macOS ProductVersion:         14.5 BuildVersion:           23F79

Other information
No response

@jesperalmstrom what is the status of the stack in cloudformation after you destroy it? Sometimes destroy command won't finish leaving resources behind, that happens for example if any of the resources are modified outside of the stack. In that case the CFN detects drift and stops.

I did a faulty deploy (wrong region) so i destroyed almost immediately. There should not have been any drift.

@shapirov103 do you have any ideas or tricks?

This is not the expected behavior, however, before qualifying it as a defect, please share what were the dependencies that you discovered?
For example, I see that you used Flux. Flux can in turn provision apps that will fail to be removed if flux controller is destroyed, this is especially true for any CRDs from flux which then fail to be cleaned up because no controller is available.
CFN is expected to remove all resources that were provisioned, provided there was no change to the resources.

Thanks for the response I will try to remove Flux and see if the delete becomes more smooth.


Print Airware AWS acct info along with role

When choosing an aws saml role to assume print the respective airware aws
account name

Example output (with midgard account removed from mapping to show how it'll look)
1: Developer-ReadOnly airware-engdev 2: Billing<PHONE_NUMBER>50 3: Administrator airware-org 4: Billing airware-org 5: EngDev-Admin airware-engdev 6: midgard-admin<PHONE_NUMBER>50 7: Developer-ReadOnly<PHONE_NUMBER>50 8: Developer-ReadOnly airware-prod 9: Administrator airware-engdev 10: Administrator<PHONE_NUMBER>50 11: Limited-Admin airware-engdev 12: Billing airware-engdev Please select the AWS role: ^C Aborted!


[Gate] Add array data management for angular speed on TinyJson, close #78

Add the possibility to deal with angular speed array on  Gate TinyJson.

Force merge.
Github Actions projects building are KO because version has been updated in platformio.ini
Building are OK on local PC so, merge is accepted


Associations sync (post-import sync)

Use case
While importing data between the environments the relatedUid fields should be re-initialized depending on Polymorphic relation changes.
Solution
Implement "Sync associations" button on the Settings page which is going to trigger global initization.

Released as v1.0.0-beta.4


Reconsider "votes" threshold for the "severity underestimated" script

The only tool which is using votes is the "severity underestimated" one, so I'm going to assume that's what you meant.
We can surely reconsider the threshold and make it larger than it currently is.

How about ignore votes altogether?

That's an option, we'll check some statistics before deciding the actual thresholds (how many bugs we would touch with each threshold).


linux_rpi1: fix Raspberry Pi Zero device tree file name

Motivation for this change
With the latest update to the Raspberry Pi kernel, the device tree for the Raspberry Pi Zero was renamed, but the package was not updated to account for this.
Things done

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

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @samueldr @tkerber

@GrahamcOfBorg build pkgsCross.raspberryPi.linux_rpi1

Ah, of course. Didn't show up in testing due to it only affecting armv6l builds, which weren't tested. I did enouter the same issue in device-tree-rpi, but missed it here. Thanks!


Not compatible with NextJs12

Hey there 👋 !
I've been trying to use the official integration with my project in NextJs12, but it gives an error.
I'm pretty sure it's cause the panelbear plugin expects previous versions of NextJs.
Here's the error:
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving: bk-korona-rate@undefined
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/next
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer next@"^10 || ^11" from<EMAIL_ADDRESS>npm ERR! node_modules/@panelbear/panelbear-nextjs
npm ERR!   @panelbear/panelbear-nextjs@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See /home/gitpod/.npm/eresolve-report.txt for a full report.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/gitpod/.npm/_logs/2021-11-04T19_06_34_859Z-debug.log

P.S.
Thank you for your product 👍

I also ran into this. Adding the  --legacy-peer-deps flag when installing the package is a workaround.

Hi @tomelliot - thanks for reporting this!
Just released a fix in 1.0.3. Feel free to check it out!

Thanks for the quick fix @anthonynsimon !


Big Source Changes, Much Improvements, Wow.

This PR contains a whole boatload of improvements. In talking with @dchiller about this we thought we would do a monolithic PR to start with, and then see if anything needed to be changed before merging.
At its core this PR changes the way Source records are captured. With the introduction of the Institution model, the siglum and the institution name are now found in that table, and the Source identifier (the shelfmark) is replacing the former title and siglum fields on the Source model.
Along the way, a number of other changes were made.

The 'base.html' template gained two new blocks, 'title' and 'script', and all the <title> and <script> tags were moved to their appropriate blocks
A new experimental approach to Private Collectors is introduced, by marking any given institution as a private collector. This still needs to be fleshed out, though.
The shelfmark field was added to Sources
A hacky migration script was added to convert most of the Source records to the new structure. This still needs adjusting for Private collectors
The forms and templates for the source edit and create were altered, including a new autocomplete for assigning the holding institution
Optimized a bunch of queries and rewrote a couple to remove some code
Removed the siglum column from the source chant browse

The biggest change, however, is the rework of the Feast detail page. The previous code was very slow to load the page, but this is now pretty fast through the use of two raw SQL queries to fetch the necessary data for the view. I've compared it with the numbers from the existing setup, and it seems to give the same results, but the page load time has gone from 3+ seconds to ~250ms, and 1,500 queries (in one case) to a pretty constant 8 (in total for the page load). It also fixed a few problems with this page, and displays all the genres associated with a given chant, instead of just one.
How to review this PR
Normally I don't like large PRs, but it seemed appropriate that, since I was touching so many places to rework sources that I could also improve these places as I went.
I've tried to explain the changes thoroughly in every commit message. I'm also posting a few screenshots to compare the old and new changes.
Outstanding issues
Still to-do is to figure out how to deal with Private Collections. You may notice this in some places where a source will have None -- this just means there is a source, but it hasn't been migrated to the Institution / Shelfmark model, so it's not displaying the right data. This should be fixed before this PR is merged.
Also not tackled is any search interface, specifically how it deals with looking up sources by "siglum." Still TBD, particularly if we want to not break any existing searches... Feedback is useful here.
Screenshots
Changes on the left, Current Production on the right.
Feast detail. Note the genre column, and the removal of the chant with no cantus ID.

Browse Source chants, with removed siglum column

Edit Source, with Holding institution and shelfmark

oo, shiny!
Comments on the first screenshot (Github is not letting me embiggen the other two right now)

The "chants"/"incipit" columns seem to be flipped in the "Most frequent chants" table
I gather that the source "None None" is something to still be dealt with,  but where has the Source Formerly Known as P-Pm 1151 gone?  And why did the HR-Zmk and PL-WRu seemingly lose a chant each?

further screenshot commentary:
-removing the siglum from the browse chant table looks great! excellent change.
-Edit source page: I see it is now optional to give a source a siglum+shelfmark.  If somebody leaves this blank, what happens? Could they create a whole bunch of sources all named "None None"?
-Relatedly, what happens if the editor can't find their institution in the dropdown list (is this all the institutions in the database? in RISM?)  Do they have to contact somebody to add it?

The "chants"/"incipit" columns seem to be flipped in the "Most frequent chants" table

Good catch! I've fixed it.

I gather that the source "None None" is something to still be dealt with, but where has the Source Formerly Known as P-Pm 1151 gone? And why did the HR-Zmk and PL-WRu seemingly lose a chant each?

I'm working from a slightly older DB export at the moment, so it may not be completely up-to-date. P-Pm has the title "Porto 1511", which didn't match the "city, institution, shelfmark" pattern, so it got skipped in the conversion. It shows up as None None because of this.

Edit source page: I see it is now optional to give a source a siglum+shelfmark. If somebody leaves this blank, what happens? Could they create a whole bunch of sources all named "None None"?

It's only optional during the transition, since we can't automatically move everything and leave nothing blank. Once we've migrated everything and it "goes live" the option to leave it blank will be removed.

Relatedly, what happens if the editor can't find their institution in the dropdown list (is this all the institutions in the database? in RISM?) Do they have to contact somebody to add it?

That's kinda up to you guys. Do you want to allow people to add their own institutions, or would you rather create them on request. (I suspect the latter.) The only weird bit is Private Collectors, but I'm still working on a proposal for that.
Thanks for the review!

OK, @dchiller @lucasmarchd01, I think this is ready for review.
All tests pass, except for a bunch in the Feast Detail View that I had to mark as 'skip' because there are problems with testing it with transactions (I explain this in the commit message).
Once this is available and merged to staging, then the "migrate_records" management command should move all the source records to the new regime.
Some more changes will need to be made to the source records to deal with the optional / required fields, but for now I've left them as-is.
Looking forward to your comments.

@dchiller @lucasmarchd01 pinging for review. This PR touches a lot so it would be good to get it merged or reviewed soon so that the conflicts don’t get out of control

Sorry, I'll get to this today! Between vacation and wanting to have #1548 move through the pipeline before and separately from this, I've been avoiding...

Is this a standard approach? I understand that this can improve performance for complex queries and offer more control, but I think that they might be more difficult to maintain and understand compared to Django’s ORM. That, along with all the tests that still need to be skipped.

Yes, it is a standard approach. There's a whole page in the docs about when you need to do raw queries:
https://docs.djangoproject.com/en/5.0/topics/db/sql/
I did try alternate methods with annotate() but the ORM query got incredibly complicated that way. SQL itself is very stable (it has to be!). It requires knowing a bit of SQL, but I personally think the direct approach is easier than a very convoluted ORM query. I worked on this particular problem for a good 2-3 days, and in the end this was what I came up with.
The problem is that this doesn't actually map on to any real "Model" query. It's a feast, sure, but there is so much other information that gets pulled in from the sources and chants, and also statistics about those queries, that it doesn't really map onto a good ORM call.
I tried for quite a while to get the tests to work, but because the test and the actual query are in two different transactions, they don't see each other.

Fixing some migrations fixed the failing tests!

I did try alternate methods with annotate() but the ORM query got incredibly complicated that way. SQL itself is very stable (it has to be!). It requires knowing a bit of SQL, but I personally think the direct approach is easier than a very convoluted ORM query. I worked on this particular problem for a good 2-3 days, and in the end this was what I came up with.
The problem is that this doesn't actually map on to any real "Model" query. It's a feast, sure, but there is so much other information that gets pulled in from the sources and chants, and also statistics about those queries, that it doesn't really map onto a good ORM call.

Makes sense, thanks for the clarification!

The provenance and century detail pages have odd orderings now when changes to templates and views are combined. I'll change them here to match what we now do with notation views.

The preferred ordering was unclear to me. In many places it was done by siglum, so the Austrian (A-) sources would sort first, then the Belgian (B-) etc.
This is fine, but the format for the full heading doesn't actually contain the siglum, only the full institution name and the shelfmark.
I could imagine something like Cividale del Friuli, Archivio Capitolare (I-CF), LVI where the RISM siglum is in parentheses at the end of the institution name. This is what we do in RISM Online, e.g., https://rism.online/institutions/30000053, "Leipziger Stadtbibliothek - Musikbibliothek, Leipzig (D-LEm)".
Then the sorting of the list would make sense, even though the siglum isn't the first component of the title. If that is what is wanted, then "(I-CF) Cividale del Friuli, Archivio Capitolare" is probably best.

The preferred ordering was unclear to me.

To me too, which is why I think it makes sense to separate. As it is now, it can just look totally random which probably doesn't make sense.
The second option you list (eg. (I-CF) Cividale del Friuli, Archivio Capitolare) is how it now looks on the notation detail page which I think makes sense.

The preferred ordering was unclear to me.

To me too, which is why I think it makes sense to separate. As it is now, it can just look totally random which probably doesn't make sense.
The second option you list (eg. (I-CF) Cividale del Friuli, Archivio Capitolare) is how it now looks on the notation detail page which I think makes sense.

The preferred ordering was unclear to me.

i believe this was discussed a bit here? https://github.com/DDMAL/CantusDB/issues/1227

@ahankinson One last question, I think:
In the description above you write that there is still work to be done to figure out "private collections" and that this will lead to some sources showing up as None. Locally, I'm not actually running into any trouble with private collections at the source level...I'm running into them at the "Institution" level.
I'm getting institutions that look like this:

A “title” doesn’t really make sense when talking about MSS… at least it’s not a universally recognized concept. The identifiers we can sort on are:

shelfmark (but we don’t want all the “MS 1” to sort together so that’s not useful
Institution name
City (but may not be known for private collectors)
Siglum

Personally I find sorting by country code, then city, and then institution name to be the most useful, since it groups all the mss from the same institution, and groups all institutions in one city. As long as it’s understood how they’re sorted then that’s probably the best way.

Thanks @ahankinson for all of this!!
From my pov, we should merge this if you are good and this makes sense https://github.com/DDMAL/CantusDB/pull/1545#issuecomment-2228959105

I just started afresh and re-ran the migration. Here's what I get:

Did you run python manage.py migrate_records? The other one, migrate_institution_source_records, got so complicated that I rewrote it but kept the old version for reference.

I just started afresh and re-ran the migration.

Interesting...I thought I did but let me try again.

The same thing happened to me again locally...

I don't see any of those institutions in my data...

I don't see any of those institutions in my data...

Looks like all are recent (past month) additions. I'm gonna merge, we'll fix them in the data later.

I did a fresh export from the cantus database today (the export is currently in /home/fedora on the server) and dropped / re-created the database on my local machine. So I'm not sure how they got there...

Looks like all are recent (past month) additions. I'm gonna merge, we'll fix them in the data later.

I think they are all part of a dump of private collection fragments I'm working on that's still very much in draft form! So that's me.

But... if they're not in the database dump I literally took from the production server today, where did they come from?

I did a fresh export from the cantus database today (the export is currently in /home/fedora on the server) and dropped / re-created the database on my local machine.

Looks like we have this figured out, but for reference, a new database dump is created every day in /home/cantusdb/backups/ so you don't need to do it yourself if you want the whole database.

OK!


Add conda installation instructions

With Anaconda installed, you just execute 'conda install' (from Anaconda Prompt on Windows). I think this should be added first since most people who use conda do so through Anaconda.


Add outline support for Markdown files

This PR adds outline support for Markdown files.

Introduces a MDDocumentSymbolProvider which is able to provide symbols based on the same markdown-it parser.
Optimizes the Front Matter regex to /^---\s*(.|\s)*?---\s*/.
First attempt was #13238
Fixes #3493

Pushed changes.

Is there a reason this cannot be merged? Would be useful for upcoming release notes writing...

@bpasero Change looks good to me. I also want to see this in 1.9. Feel free to merge when ready

@joaomoreno should do the merge, I am just a happy customer ;)


ntr: pyruvylate galactose-beta-1,3-galactose biosynt

pyruvylgalactose biosynthesis
synonym:
PvGal biosynthesis
actually need a child term:
pyruvylated
galactose-beta-1,3-galactose-alpha-1,2-pyruvylgalactose
as in
http://www.genedb.org/genedb/Products?organism=pombe&action=search&search=deletion+mutant+galactomannans+deficient+for+pyruvylated+galactose-beta-1%2C3-galactose-alpha-1%2C2-pyruvylgalactose+(PvGal)
pyruvylated
galactose-beta-1,3-galactose-alpha-1,2-pyruvate
synonym:
Pv
as in
http://www.genedb.org/genedb/Search?name=SPBC317.01&organism=pombe
see
PMID: 15173185
Reported by: ValWood
Original Ticket: "geneontology/ontology-requests/1777":https://sourceforge.net/p/geneontology/ontology-requests/1777

thanks www.movied.org


Adding the Support of Retrieving Actions Persisted via an API

Fixes #: Adding the Support of Retrieving Actions Persisted via an API
Description of changes: Currently only last action persisted in the PersistedActions table is returned. Any action which is persisted in the table would have a different time stamp so getting last action set based on the same timestamp is not possible.
Tests:
[root@b203f70e9257 tmp]# curl --url "localhost:9600/_opendistro/_performanceanalyzer/actions" -XGET
{"LastSuggestedAction":"{\"nodeIps\":\"{<IP_ADDRESS>,<IP_ADDRESS>}\",\"muted\":false,\"actionName\":\"MockAction\",\"timestamp\":1601950003287,\"nodeIds\":\"{1,2}\",\"summary\":\"MockSummary\",\"actionable\":false,\"coolOffPeriod\":0}"}[root@b203f70e9257 tmp]#

Build: ITs succeeding Locally. Failing on Github. Should be a intermittent failure.
SUCCESS: Executed 429 tests in 28m 4s (1 skipped)

> Task :spotbugsMain
Warning at xsl:variable on line 348 column 57 of default.xsl:
  SXWN9001: A variable with no following sibling instructions has no effect
Warning at xsl:variable on line 351 column 57 of default.xsl:
  SXWN9001: A variable with no following sibling instructions has no effect

Deprecated Gradle features were used in this build, making it incompatible with Gradle 7.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/6.5/userguide/command_line_interface.html#sec:command_line_warnings

BUILD SUCCESSFUL in 29m 7s
19 actionable tasks: 19 executed

If new tests are added, how long do the new ones take to complete
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Currently only last action persisted in the PersistedActions table is returned. Any action which is persisted in the table would have a different time stamp so getting last action set based on the same timestamp is not possible.

We definitely need the last set of actions persisted. All actions persisted in one publisher cycle should have the same timestamp and the API should return all actions with the same latest timestamp.
It doesn't look like a difficult change to support this. We'll need to remove picking a per action timestamp in PublisherEventsPersistor, and instead pick it once for all actions in Publisher::compute() and pass it to persistAction.
After that the changes are same as discussed earlier - persistor changes that read all actions with latest timestamp via the sql query. Let's please address those changes in this PR.

Currently only last action persisted in the PersistedActions table is returned. Any action which is persisted in the table would have a different time stamp so getting last action set based on the same timestamp is not possible.

We definitely need the last set of actions persisted. All actions persisted in one publisher cycle should have the same timestamp and the API should return all actions with the same latest timestamp.
It doesn't look like a difficult change to support this. We'll need to remove picking a per action timestamp in PublisherEventsPersistor, and instead pick it once for all actions in Publisher::compute() and pass it to persistAction.
After that the changes are same as discussed earlier - persistor changes that read all actions with latest timestamp via the sql query. Let's please address those changes in this PR.

Added the changes

We will also need a writer side change that can proxy the call over to this rest handler, right ?

We will also need a writer side change that can proxy the call over to this rest handler, right ?

Right! Those changes are on the PA side. Will raise a PR for that Separately.

We will also need a writer side change that can proxy the call over to this rest handler, right ?

https://github.com/opendistro-for-elasticsearch/performance-analyzer/pull/215


Error when running main_bind.py

Hi, thanks for the great work on this!
I had an error running main_bind.py in the code pl_sd = torch.load(ckpt, map_location="cpu"), which is: _pickl. UnpicklingError: invalid load key, '<'.
I checked the sd21-unclip-h.ckpt file and found the html content in the file. I think the error should be related to this. I tried to solve this problem, but failed. If possible, could you please answer it for me?
Thanks!

Hi, thanks for your interest. I guess you should re-download the checkpoint. This error may caused by your fragmentary model.  The md5sum of sd21-unclip-h.ckpt is 72fa8487b729d638a9f4e16b9fb2091a and maybe you can check it.

Hi，according to your suggestion, the problem has been solved. Thank you for your answer！


Circular/Automorphic number

Description
The square number of a number ends with the same digits as the number itself.
That number is known as circular number or automorphic number.
Example:
5^2 = 25
6^2 = 36
376^2 = 141376
Domain
Algorithm
Type of Contribution
Addition
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of LearnCPP.

/assign


Update budget-narrative.md

Added statement about header view as multiple people couldn't find initially
There are no permissions on budget plan process as it is all role/privilege based
Added form name in metadata

:white_check_mark: Organization membership check passed
Requester is not a member of the MicrosoftDocs organization.


Bug/prevent google manage address

all language about "Country" has been removed.  google autodetects this and imposes the forms


fix: addressing assembly text build mechanism

Description

Centralized build mechanism to make the implementation easy to maintain

@xermicus

Just a few minor things, otherwise LGTM!

Just a few minor things, otherwise LGTM!

Ok, commit pushed


stdlib log.SetOutput does not work with the *Middleware's

Suggested change: Default to make a function call to log.Print instead of creating new logger if *AccessLogMiddleware.Logger is nil.
It seems like it's a very uncommon pattern to create new loggers if there isn't a very specific reason for it.

That would be an API change, and that won't happen before the next major version for compatibility reason. Filed under "v4 idea"


Candle tag not showing

The candle tag doesn't show in JEI, which possibly mean it doesn't work
To Reproduce
1: open inventory
2: search "$candle" in jei
3: nothing is showing
Expected behavior
The tag should work
Screenshots
If applicable, add screenshots to help explain your problem.
https://cdn.discordapp.com/attachments/688410719611256852/912758390181212180/Minecraft__1.16.5_-_Singleplayer_23_11_2021_18_35_52.png
System (please complete the following information):

Occultism Version: 1.19.1
OS: Windows
Minecraft Version: 1.16.5
Modpack Link and Version, or list of mods: AI improvements; Akashic tome; Appleskin; Aquaculture; Archers paradox; Architectury; Attribute Fix; Auto reg lib; Better biome blend; Better burning; Yung's better caves; Yung's better dungeons; Yung's better stronghold; Bookshelf; Caelus; Cakechomp; Catalogue; Ceramics; Chunk pregenerator; Clumps; Cofh core; Common capabilities; Configured; Connected glass; Create additions; Create addon; Create gears; Create; Create plus; CTM; Curio of undying; Curios; Curious elytra; Cyclops core; Dark painting; Decorative blocks; Diet; Eidolon; Enchantment descriptions; End reborn; Ensorcellation; Entity culling; Equipement compare; Experience bug fix; Farmers delight; Fast bench minus replacement; Fast furnace minus replacement; Fast leaf decay; Fast suite; Ferrite core; Flywheel; Framed blocks; Framed compact drawers; Friendly Fire; FTB library; Glassentials; Go prone; Hwyla; Iceberg; Immersive engeneering; Immersive petroleum; Industrial foregoing; Iron chest; Iron shulker box; Jeed; JEI; Journey Map; Just enough beacons; Just enough professions; Just enough ressources; Klee slab; Lazy dfu; Mantle; Masonry; Materialis; Max health fix; Mekanism; Mekanism generators; Mekanism tools; Mgui; Mod name tooltip; More create stuff; Mouse tweaks; Neat; Netherite horse armor; Nether portal fix; Occultism; Overloaded armor bar; Pass through signs; Patchouli; Placebo; Polymorph; Pretty pipes; Pretty pipes fluid; Psi; Quark; Save your pets; Scaffolding power; SoL carrot; Spark; Storage drawers; Swing through grass; Tcon planner; TiC; Tetra; Undergarden; Thermal expansion; Thermal foundation; Thermal innovation; Thermal locomotion; TiC rapier; Tinkers delight; Titanium;  Torch slab mod; Toughness bar; Wyrmroost; Yung's API; Yung's bridges

Cleared this up in discord... It is a block tag which does not show up with advanced tooltips on and IE installed(it only shows item tags)... There are other mods that do but no clue which lol.. instructed them to use f3 screen and look at the block placed in world

Thanks for investigating!


[Issue]: Must define exactly one of HIP_PLATFORM_AMD or HIP_PLATFORM_NVIDIA

Problem Description
If I hipify a file already contains hip c++, for example:
#ifndef __HIP_PLATFORM_AMD__
#include <cub/cub.cuh>
#else
#include <hipcub/hipcub.hpp>
#endif

When I hipify this file, I will pass the -D__HIP_PLATFORM_AMD__ to hipify-clang. But error happens:
In file included from /opt/rocm/include/hipcub/hipcub.hpp:31:
In file included from /opt/rocm/include/hipcub/config.hpp:47:
In file included from /opt/rocm/include/rocprim/device/config_types.hpp:31:
In file included from /opt/rocm/include/rocprim/device/../config.hpp:34:
/opt/rocm/include/hip/hip_bfloat16.h:41:2: error: ("Must define exactly one of __HIP_PLATFORM_AMD__ or __HIP_PLATFORM_NVIDIA__");
   41 | #error("Must define exactly one of __HIP_PLATFORM_AMD__ or __HIP_PLATFORM_NVIDIA__");

and:
In file included from /opt/rocm/include/hipcub/hipcub.hpp:31:
In file included from /opt/rocm/include/hipcub/config.hpp:47:
/opt/rocm/include/rocprim/device/config_types.hpp:308:31: error: use of undeclared identifier 'hipGetDevicePropertiesR0600'; did you mean 'cudaGetDeviceProperties_v2'?
  308 |     const hipError_t result = hipGetDeviceProperties(&device_props, device_id);
      |                               ^~~~~~~~~~~~~~~~~~~~~~
      |                               cudaGetDeviceProperties_v2
/opt/rocm/include/hip/hip_runtime_api.h:91:32: note: expanded from macro 'hipGetDeviceProperties'
   91 | #define hipGetDeviceProperties hipGetDevicePropertiesR0600
      |                                ^
/usr/local/cuda/include/cuda_runtime_api.h:1722:58: note: 'cudaGetDeviceProperties_v2' declared here
 1722 | extern __host__ __cudart_builtin__ cudaError_t CUDARTAPI cudaGetDeviceProperties(struct cudaDeviceProp *prop, int device);
      |                                                          ^
/usr/local/cuda/include/cuda_runtime_api.h:170:33: note: expanded from macro 'cudaGetDeviceProperties'

Seems this predefined macro __HIP_PLATFORM_AMD__ was lost when parsing hip core libs?
Operating System
Ubuntu 20.04.6 LTS (Focal Fossa)
CPU
AMD EPYC 7643 48-Core Processor
ROCm Version
ROCm 6.0.0
ROCm Component
HIPIFY
Steps to Reproduce
No response
(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
No response
Additional Information
No response

Hello @foreverlms,
May I ask why do you need to hipify the code already containing HIP code?

It's mixed with cuda and hip. controlled by macros. But most of them are cuda. So for the cuda part, I still need to hipify them.

Hipifying of the mixed code is not what HIPIFY tools are designed for, for at least hipify-clang, which is being a compiler, in fact, gets a CUDA code as a source and compiles it always with -x cu. When you are passing -D__HIP_PLATFORM_AMD__, you try to force the compiler to compile HIP and get the error Must define exactly one of __HIP_PLATFORM_AMD__ or __HIP_PLATFORM_NVIDIA__
Btw, there is still a strategy for the opposite thing on the table to implement - hipifying to the "mixed" code by making the CUDA source codes CUDA or HIP in a single source depending on the preprocessor define (#536).
So, I have two options for you.
Option 1. Please share your code snippet with such a mix, and I'll try to think about how to deal with it by hipify-clang.
Option 2. Try to use hipify-perl instead.

Hi @foreverlms,
So what about the option you chose? Anyway, it would be great to deal with your snippet.

Thanks for your suggestion! I have bypassed this by removing the mix part. Keepping only the cuda way and let hipify-clang to handle it completely.
So my problem is solved, for the mixed HIP and CUDA sources, I think we could close this issue, if anyone else has the same problem, this issue could be reopened.


Editor highlights are displayed incorrectly

Where there are multiple occurrences of the same highlight term, the second and subsequent terms are highlighted incorrectly.

Fixed in >7.0.17


smart select should stop before and after brackets

re https://github.com/Microsoft/vscode/issues/67662
In JSON, JS, et al we stop before and after bracket blocks and I think we should do the same of CSS and its friends. Alternatively we align the other was around and change JSON and the default provider.
@aeschli options?

@octref Also just added before/after for CSS. I think we should go with that.
Let's avoid any options unless we get a major number of requests for it.

The change I did add inner bracket range but removed the range including brackets. In the context of CSS it's less useful.
That's why I like Vim :wink: You can do di{ and da{. Same strokes with options.
Together with the proposal of first expand to whole word instead of hump word, I want to keep the ranges list short. I don't want to have to press Expand 5 times to get what I want every time.

Why do you think the selection with brackets is not useful? You might want to copy the block.
We should try to be consistent across languages.

Because in CSS, it's much more common to do:
.foo (add another selector here) {
  // rules
}

I have never had the need to copy the whole block. Even if I do, copying inner block is good enough. I just need to type another { before pasting.
I don't think we should add another range that will stay in the way of selecting the complete RuleSet.

won't fix


Update: Mozilla.Firefox.DeveloperEdition version 127.0 (127.0b2)

Created by 🥟 Dumplings in workflow run #5145.
Logs
Updated: 127.0b1 → 127.0b2
Submitting WinGet manifests
Existing pull request found: New version: Mozilla.Firefox.DeveloperEdition version 127.0 (127.0b1) - https://github.com/microsoft/winget-pkgs/pull/153555
Cannot process argument transformation on parameter 'MessageID'. Cannot convert the "System.Int32[]" value of type "System.Int32[]" to type "System.Int32".

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


SpaceCharacter: Show check name in message

Right now the message looks like this:
.env:2 The line has spaces around equal sign

Need to add the check name to the message:
.env:2 SpaceCharacter: The line has spaces around equal sign

I'll take this!

@Am3ra Hi! Is there any progress with this issue?

I'll take it

@pineapplethief Thank you! I've assigned you!


Compatibility with PHP 7.4

We should allow use this package with projects with PHP version lower than 8.0. I would like to support PHP version from 7.4.
The package should be full functionality in the projects with PHP 7.4 and above.

I'm closing this issue. PHP 7.4 lost its support on November 2021 (6 months ago!) and will lose security fixes in 5 months. We should not support old versions.


Specified clang-tidy version 8 for contributing guidelines

Motivation and Context
Specified clang-tidy version 8 for contributing guidelines, as our clang config isn't backward compatible.

How Has This Been Tested

Types of changes

[X] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Codecov Report

Merging #408 into master will decrease coverage by 0.78%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #408      +/-   ##
==========================================
- Coverage   56.74%   55.96%   -0.79%
==========================================
  Files         175      171       -4
  Lines        8094     7950     -144
  Branches       84       84
==========================================
- Hits         4593     4449     -144
  Misses       3501     3501

Flag
Coverage Δ

#CPP
52.35% <ø> (ø)
:arrow_up:

#JavaScript
10% <ø> (ø)
:arrow_up:

#Python
77.42% <ø> (-1.41%)
:arrow_down:

Impacted Files
Coverage Δ

tests/test_sensors.py
100% <0%> (ø)
:arrow_up:

habitat_sim/sensors/noise_models/__init__.py
100% <0%> (ø)
:arrow_up:

...t_sim/sensors/noise_models/gaussian_noise_model.py

...ensors/noise_models/salt_and_pepper_noise_model.py

...at_sim/sensors/noise_models/speckle_noise_model.py

...at_sim/sensors/noise_models/poisson_noise_model.py

Continue to review full report at Codecov.

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


fix: render to dataSource, not service, when values.dataSource is specified

Simple fix - service seems to be specified accidentally here. It should be dataSource .

Closing since this was fixed with #95.


Use the mouse wheel to adjust the system volume.

Description of the new feature / enhancement
Use the mouse wheel to adjust the system volume when mouse on the top/bottom/left/right border of screen.
Scenario when this would be used?

When I use a full-screen application and need to adjust the volume
Or when I'm lazy and don't want to open the taskbar
I don't have a multimedia keyboard

Supporting information
This feature can be added to mouse utilities.

Take a look on Volume2, it have a function to adjust volume by scrolling wheel on the taskbar and some other options with wheel.

Take a look on Volume², it have a function to adjust volume by scrolling wheel on the taskbar and some other options with wheel.

Thanks! It did solve my problem!


Update enhanced-http.md

Updated Prerequisites section to specify correct tab and roles for management point and distribution point configuration.
Summarize the change in the pull request title
The previous text specifies the incorrect role to select when configuring http settings for the management point and distribution point.
Fixes #Issue_Number (if necessary)

@mrmiller1972 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off
Once merged, this should be live with the next daily publishing process. Right now that should be tomorrow (Tuesday, 13 January) around 11:00 Pacific time. No further action needed from you.


screenshots are really spiking CPU.

Tons of firefox-esr processes all open at once and it makes kswapd go crazy.
I have an idea for a solution i am trying now.  Rather than iterating over the list of URLs calling take_screenshot once at a time for each URL (which loads and quits the selenium driver each time), I'm going to try to pass the list of urls to take_screenshot, and then for each url in the list, I'll open selenium driver, get the url, take the screenshot, move to next url, take screenshot, etc...  and then finally quit the driver.
i might need a delay in between each, or i might not.   we'll see.

The error was because of a try/except block.  I was not quitting the Firefox instances i started to take a screenshot if there was an error, which meant that the you would get more and more Firefox-ESR process just hanging around until it exhausted memory and crashed the box.
added better exceptions and a finally block which makes sure to close the driver regardless of the result.

tested and closed!


Add support for file owner

This feature adds support for changing the file owner like suggested in #7

Coverage increased (+0.2%) to 97.892% when pulling a8ee1f3986735cc39f431021adb6f347e46d848c on feature/add_file_owner_support into ba46dab16c578ac64bd8b4a01e5586d21a96b9a7 on master.

Coverage increased (+0.2%) to 97.892% when pulling 44ef80a2646a261d9c52f42b642c25d2da106f5e on feature/add_file_owner_support into cffbb1f646d22511b4a9e1b1582e465b288b16c4 on master.

Coverage increased (+0.2%) to 97.892% when pulling 44ef80a2646a261d9c52f42b642c25d2da106f5e on feature/add_file_owner_support into cffbb1f646d22511b4a9e1b1582e465b288b16c4 on master.

Coverage increased (+0.2%) to 97.892% when pulling 44ef80a2646a261d9c52f42b642c25d2da106f5e on feature/add_file_owner_support into cffbb1f646d22511b4a9e1b1582e465b288b16c4 on master.

Coverage decreased (-0.2%) to 97.545% when pulling b996350e69b8fd2babb962fcf541e76f0d22b393 on feature/add_file_owner_support into cffbb1f646d22511b4a9e1b1582e465b288b16c4 on master.


FLAnimatedImage.h not found

I had some issues submitting my build to the app store with Carthage, so I decided to give the pod install a shot. Now the FLAnimatedImage.h is throwing an error in RNFLAnimatedImage.h.  Any suggestions?

Turns out even if your header is in the ios/Pods folder, you have to set the search path.


Integrate Odyssey within editor

Closing since this is already done. Now, splitted in a lot of smaller issues.


improving typescript error message formatting: message contains [object Object]

Does this issue occur when all extensions are disabled?: Yes/No
Does this issue occur when all extensions are disabled?: Yes/No
VS Code Version: 1.77.1
OS Version: Ubuntu 22.10

Steps to Reproduce:
When an error in tsserver occurs, the console output contains [object Object]

[[object Object]]<main> TypeScript Server Error (5.0.2)
Debug Failure. False expression: Non-string value passed to `ts.resolveTypeReferenceDirective`, likely by a wrapping package working with an outdated `resolveTypeReferenceDirectives` signature. This is probably not a problem in TS itself.
Error: Debug Failure. False expression: Non-string value passed to `ts.resolveTypeReferenceDirective`, likely by a wrapping package working with an outdated `resolveTypeReferenceDirectives` signature. This is probably not a problem in TS itself.

Improvement: Printing the extension id as string
[vscode.typescript-language-features]<main> TypeScript Server Error (5.0.2)
Debug Failure. False expression: Non-string value passed to `ts.resolveTypeReferenceDirective`, likely by a wrapping package working with an outdated `resolveTypeReferenceDirectives` signature. This is probably not a problem in TS itself.
Error: Debug Failure. False expression: Non-string value passed to `ts.resolveTypeReferenceDirective`, likely by a wrapping package working with an outdated `resolveTypeReferenceDirectives` signature. This is probably not a problem in TS itself.

The bot never kicked in, @SimonSiefke can you check that it is fixed in the latest Insiders build of VS Code?


Support for ServiceNow London release

Would like to know when the ITSM connector will support ServiceNow London release

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

ID: 82e5fdf6-4687-0a17-9a80-07703684b78f
Version Independent ID: 42ed8d94-6250-54ba-5e8b-de62c0181e82
Content: Supported connections with IT Service Management Connector in Azure Log Analytics
Content Source: articles/log-analytics/log-analytics-itsmc-connections.md
Service: log-analytics
GitHub Login: @JYOTHIRMAISURI
Microsoft Alias: v-jysur

@PaulSells
Thanks for the feedback! We are currently investigating and will update you shortly.

Hi @PaulSells,
We are working on this. At this point, mid to end of November is the ETA.


Support Envoy Proxy automountServiceAccountToken configuration

Description:

Add support to configure automountServiceAccountToken for Proxy Pods

This parameter is set to false at the moment:
https://github.com/envoyproxy/gateway/blob/384577334dea2c0ba28800643a2b94fea4eed244/internal/infrastructure/kubernetes/proxy_deployment.go#L86
In our case, we need the possibility to set automountServiceAccountToken to true in order to integrate Envoy Proxy with Linkerd service mesh.
automountServiceAccountToken=true is the hard requirement of older versions of Linkerd (< 2.12.0). We need Envoy Proxy works with older Linkerd versions as well.

mounting the service account token can be used to abuse the entire data plane so im a -1 on this.
since newer versions of Linkerd already support this, imo this is not needed now, but prefer to keep this issue opened to gather more use cases

closing as not planned


Cannot open file 'python310.lib'

Description
Hello guys, I get following error running said command:

Steps to Reproduce

Run python main.py -O --image <image>.png --workspace trial_image --iters 5000

Expected Behavior
Afaik, this action should start the training of some kind
Environment
Python 3.10.11, CUDA version 11.8, Windows 11

If anyone has the same issue, ram upgrade did it for me (16 to 32gb)


Resnet backbone in ImageNet experiments

Dear James,
Thanks for sharing the code of your method!
A quick question regarding the ResNet architecture used in
experiments/imnet-fivetask.sh
and
experiments/tinyimnet-twentytask.sh
You have "MODELNAME=resnet32" in both scripts.
Shouldn't this be resnet18 as stated in the paper?
Thanks!

Thank you, I have pushed the change. As mentioned in the paper, ImageNet is a weakness of our method and only recorded for transparency. In order to achieve better results, either more compute or an updated model inversion technique would be required.


CRYPTO_AES_CTR define/enum compatibility with FreeBSD, OpenBSD and cryptodev-linux

Hello,
I've noticed that in https://github.com/openssl/openssl/blob/OpenSSL_1_0_2-stable/crypto/engine/eng_cryptodev.c the CRYPTO_AES_CTR is used for a preprocessor check.
In OpenBSD https://github.com/openbsd/src/blob/master/sys/crypto/cryptodev.h is indeed declared with a #define.
In cryptdev-linux in instead declared in an enum, which makes it not defined for the preprocessor.
Looking in more recent version of OpenSSL:
https://github.com/openssl/openssl/blob/master/engines/e_devcrypto.c
if a CRYPTO_ALGORITHM_MIN define is declared CRYPTO_AES_CTR is then handled as a define.
CRYPTO_ALGORITHM_MIN is defined on FreeBSD:
https://github.com/freebsd/freebsd/blob/master/sys/opencrypto/cryptodev.h#L167
But it isn't on OpenBSD:
https://github.com/openbsd/src/blob/master/sys/crypto/cryptodev.h
What do you think about changing:
#ifdef CRYPTO_ALGORITHM_MIN
# define CHECK_BSD_STYLE_MACROS
#endif

in
#if CRYPTO_ALGORITHM_MIN < CRYPTO_ALGORITHM_MAX
# define CHECK_BSD_STYLE_MACROS
#endif

This will work on FreeBSD where both values are known to the preprocessor, on OpenBSD where CRYPTO_ALGORITHM_MIN will be automatically set to 0, lower than the defined CRYPTO_ALGORITHM_MAX, and with cryptodev-linux where CRYPTO_ALGORITHM_MAX for the preprocessor will fall back to (0 - 1).
What about adding the above, and the check below also to the 1.0.2 OpenSSL branches?
#if !defined(CHECK_BSD_STYLE_MACROS) || defined(CRYPTO_AES_CTR)

Thanks,
Vincenzo

I like your ideas for the guard around the definition of CHECK_BSD_STYLE_MACROS, another idea would be to just:
#ifdef CRYPTO_ALGORITHM_MAX
# define CHECK_BSD_STYLE_MACROS
#endif

Regarding 1.0.2, it's on its last year of support, which means we only make directly security related fixes on that branch...  and this is rather a building problem as far as I can see.

I thought about that, but CRYPTO_ALGORITHM_MAX is defined in cryptodev-linux (and not part of the enums):
https://github.com/cryptodev-linux/cryptodev-linux/blob/f971e0cd4a0ebe59fb2e8e17240399bf6901b09b/crypto/cryptodev.h#L57

Argh, right...


unexpected join primary key

Describe the bug
通过sql生成model的时候不支持联合主键吗？
packageinfo.sql: unexpected join primary key
To Reproduce
任意带有联合主键的建表语句都会报错

The code is

create table packageinfo
(
   name            char(50) default ''      not null,
   version         char(50) default '0.0.0' not null,
   productplatform int(2)                   not null comment ' 产品型号:etsme1,etsme1s',
   ipktype         char                     not null,
   ipkversion      char(50)                 not null,
   ipkname         char(50)                 null,
   filename        char(50)                 not null,
   filesize        char(30) default '0'     null,
   dirname         char(60)                 not null,
   downloadurl     varchar(125)             not null,
   createtime      char(30) default ''      null,
   primary key (name, version, ipkversion, ipktype, productplatform)
)
   charset = utf8;

The error is

packageinfo.sql: unexpected join primary key

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

OS: Linux  ubuntu20.04
go-zero version 1.5.6
goctl version 1.5.6

More description
这个怎么解决

goctl doesn't support joint primary keys.


Eliminate Orphaned Files

As discussed on Slack, code that's not used in multiple lectures should be moved out of the _static/code folder or whatever it's called. Also, per @jlperla, files that are not used at all should be nuked.

Closed by #54 If needed re-open it.


feat: support custom marshaling and unmarshaling

WIP
This PR attempts support for custom marshaling and unmarshaling.
This may prove useful for a related PR: https://github.com/hashicorp/go-tfe/pull/786
cc @nfagerlund @brandonc
To do

Add tests

Testing
To demonstrate usage within the /examples folder:
func (c *Comment) UnmarshalJSON(data []byte) error {
       *c = Comment{0, 0, "custom body"}
       return nil
}

func (c *Comment) MarshalJSON() (*jsonapi.Node, error) {
       return &jsonapi.Node{
               Type: "custom_comment",
               ID:   strconv.Itoa(c.ID),
               Attributes: map[string]interface{}{
                       "body": fmt.Sprintf("%s!", c.Body),
               },
       }, nil
}

Well!! 👀
Ok, so I spent some time trying to get my example scrap to work with this patch. It didn't quite make it across the finish line!
jsonapi: Can't unmarshal It'S HAPPNEING (string) to struct field `Problem`, which is a pointer to `ProblematicString (struct)`

The issue here seems to be something like:

The really interesting use case here is for struct-typed attributes where the type implements custom un/marshal.
This draft only puts custom unmarshal logic in the top-level unmarshalNode func, but in order to handle attributes with custom serialization, we need an alternate dispatch path down in unmarshalAttribute.

At this point, I took a stab at implementing that, and immediately got lost in a spiky thicket of reflect chaos. Then eventually I started doubting everything I thought I knew (for example, how in the world does handleStruct work? It looks like it's treating a struct-typed attribute as... a whole-ass jsonapi node? That can't be right, surely I'm misreading something).
Anyway, hopefully that's useful despite me not having a complete answer to deliver. If it'd be helpful, I can try and clean up that gist to make a more legible test case; it's descended from my initial research into this space, so it's a bit of a mess.

WAIT. Actually, I think I figured it out.
func handleCustom(attribute interface{}, fieldValue reflect.Value) (reflect.Value, error) {
	// Re-create the raw json representation of whatever we got over the wire --
	// could be string, bool, Object, etc. etc. etc., but we want json bytes
	// so we can take another crack at it.
	data, err := json.Marshal(attribute)
	if err != nil {
		return reflect.Value{}, err
	}
	// Get something to serialize into, based on what the field is intended to
	// be. (I think fieldValue is a wrapped instance of the field type's zero
	// value... which, if the field's a pointer, is pointing to nil instead of
	// allocated memory, so we're better off starting from scratch than trying
	// to actually use it.)
	var model reflect.Value
	if fieldValue.Kind() == reflect.Pointer {
		// We want the value type.
		model = reflect.New(fieldValue.Type().Elem())
	} else {
		model = reflect.New(fieldValue.Type())
	}

	ref := model.Interface() // a type-erased pointer to the inner value, IIUC
	if err := json.Unmarshal(data, ref); err != nil {
		return reflect.Value{}, err
	}
	// Interestingly, we don't have to worry about whether the destination field
	// is a pointer or a value; something downstream of here just handles that
	// for us.
	return model, nil
}

And then in unmarshalAttribute, you need to do this before all the rest of the dispatch options:
	// Handle field with arbitrary custom unmarshal logic
	if _, ok := fieldValue.Interface().(Unmarshaler); ok {
		// Actually, handleStruct already does what we were about to do
		value, err = handleCustom(attribute, fieldValue)
		return
	}

I'm pretty sure this isn't quite the direction we want to go, at least for the moment. A couple of thoughts:

this implementation doesn't work for nested jsonapi structs (correct? I haven't actually tested, so this point is just from reading the code)
this implementation forces the user to define their own marshal strategy for each struct that wants some degree of customizability

To the latter point: we don't need to grant users complete marshal customizability to support the intent of https://github.com/hashicorp/go-tfe/pull/786 in an alternative way. Essentially all we want to do there is have some conditional mechanism for marshaling a struct field when nil.

Right — I guess I didn't say so explicitly, but I don't think defining custom marshaling for an entire jsonapi resource makes sense for go-tfe's use case. For that, we just want optional custom logic at the attribute level. (But, I think the thing I pasted is pretty close to getting there! It works in my experimental code.)

superceded by https://github.com/hashicorp/jsonapi/pull/21


Add Kafka Source & Target Auth optional

Closes #1248

@odacremolbap @FranBarrera I get the impression that the autogenerated CRD reference docs for KafkaSource have not been updated to reflect the fact that auth is now optional:

On the other hand, the KafkaTarget looks OK:

Also, when you look into the details of the auth object's spec, some things are optional and some aren't:

Any chance we can fix this before release?


Sound replace bug.

I replace all radio sounds with clear soundcache file, and got this error.
https://youtu.be/cE-MoWY37HU

If you followed this tutorial https://github.com/Ochii/cso2-launcher/issues/21#issuecomment-460213422 then there is a typo. You should create empty not 'allsound.cache' but 'allsounds.cache'. Then it will work

it works. Thanks


Why does the cdn serve sass files

Which looking for sources in chrome dev tools, I noticed that the cdn loads scss files as well. Why is it so ? I think its unnecessary and will be a negative effect on the loading time.

it's not. chrome devtools is showing you the sourcemap references


accessibility issues

Gitea version (or commit ref): 1.1.0
Git version: 2.12.0
Operating system: arch linux
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist: not needed

Description
Gitea, as with gogs which it was forked from, has many accessibility issues with screen readers including:

The dropdown menus are clickables, so many screen readers do not know what to do with them as they do not use the mouse cursor.
The license selection/.gitignore selection (and most other menus of that type) are inaccessible, as they all use mouse oriented actions.
Possibly others I haven't seen yet.

Unfortunately, this is caused by the toolkit you use, which does not seem interested in making their toolkit accessible. It can be worked around with aria markup, or by changing  toolkits (the first of which is easier to begin with but requires more maintenance over time, the second of which is of course much harder, but once you get it done it's done pretty much forever).
See the original issue on gogs for more info: https://github.com/gogits/gogs/issues/3340
Looking forward to seeing if anything can be done about this.
-Michael.

Hi,
I was just revisiting this issue as I was trying to find a git manager that had some of the features of gitlab enterprise (which gitea does). Any news on this?
-Michael.

I did a simple try of adding role="listbox" to all items that act as listboxes in gitea, and role="option" to every option in those menues, just to see if it would even help at all. Turns out it won't. So the long and short of it is... semantic UI will need to go if this project is to become accessible. I'd honestly be willing to give converting the project over to another framework my best try, though I'm honestly not super familiar with css, js, or go, but I don't want to do the work if the prospect won't be accepted anyway. So the questions:

Is changing framework (even if I do the brunt of the work) even on the table?
Are there prefered frameworks if so?

At the worst I can always fork gitea and make my changes even if they won't be merged back in -- that's the beauty of open source. But I'd like to get my changes merged into gitea if possible, if only because it already has a following and so everyone can benefit.
-Michael.

It is possible for switching framework but the problem is that for functionality we need simple dropdown list wont help much anyway for most of the dropdowns as we need them searchable and more advanced.
For me preferable would be bulma + vue but that's just for me :)

I like bootstrap3/4 + vue :)

It will be quite big task as quite much all html markdown and javascript functionality needs to be changed


Python: Add support for yaml prompt template

Motivation and Context
SK Python doesn't have a way to create a prompt template config from a yaml file. This is neeed to further achieve parity with dotnet and java.

Description
This PR introduces:

some kernel methods to load yaml prompts from a directory as well as create a kernel function from a string that contains yaml. Closes #4640
Unit tests and a syntax example showing how to create a yaml prompt and load it

Contribution Checklist

[X] The code builds clean without any errors or warnings
[X] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[X] All unit tests pass, and I have added new tests where possible
[X] I didn't break anyone :smile:

Python 3.8 Test Coverage Report • FileStmtsMissCoverMissingsemantic_kernel   kernel.py3242991%194–198, 200–204, 206–210, 218–220, 239, 245, 247–251, 253–257, 259–264, 266, 311–315, 332–337, 344, 348–352, 365, 626TOTAL5368100481%
Python 3.8 Unit Test Overview

Tests
Skipped
Failures
Errors
Time

1200
11 :zzz:
0 :x:
0 :fire:
24.106s :stopwatch:


React Server Components: Server Components

Page: https://id.react.dev/reference/rsc/server-components

Hi @resir014, I want to contribute to this page

@AlexMuhammad all yours!


kasa.exceptions.SmartDeviceException: Communication error on system:set_relay_state

kasa, version <IP_ADDRESS>
Exception: Communication error on system:set_relay_state
Python 3.9.7
MacOS 12.6.3
Example code:
import asyncio
from kasa import SmartPlug
from pprint import pformat as pf

charger = "<IP_ADDRESS>"
plug = SmartPlug(charger)
# plug.timeout = 20
asyncio.run(plug.update())
# print("Hardware: %s" % pf(plug.hw_info))
# print("Full sysinfo: %s" % pf(plug.sys_info))
if (plug.is_on):
    asyncio.run(plug.turn_off())
else:
    print("Plug was already OFF")

If the plug is off then it simply displays the print() statement as expected. Uncommenting out the two print() statements before this works as expected. If the plug is off then the turn_off() method throws the exception as indicated. Adding a timeout of 20 seconds doesn't change this condition.
Things work fine using the kasa CLI.
I note that this has been an issue in the past and it was closed. I'm using the latest code as of 6/9/2024.

Sorry, I've accidentally created this in the wrong repository. My bad.


change font size of messages, etc. with setting

No content

Issue created by BlueBubbles Bot in the BlueBubbles commnity on behalf of @Intellijence in #android-bugs.

We can already do this


Remote Data Section - Documentation

Wanted
Add documentation and examples on how to use the remote data section.

Was closed with PR #3180


Turn off "Send notification to old topic" option by default

When moving messages, users can choose to send a notification to the old and/or new topic, which helps others understand what happened, e.g.:

A good default is to send a notification to the new topic (for clarification), but not to the old topic (where it is more likely to be unnecessary clutter). At present, moving messages defaults to sending a notification to both the new and old topic.  Instead, we should change the default to only send a notification to the new topic. We should do this for moves via:

[ ] the message three-dot menu
[ ] the topic three-dot menu

@zulipbot claim

When moving messages, users can choose to send a notification to the old and/or new topic, which helps others understand what happened, e.g.:
A good default is to send a notification to the new topic (for clarification), but not to the old topic (where it is more likely to be unnecessary clutter).
At present, moving messages via the message three-dot menu defaults to sending a notification to both the new and old topic. Instead, we should change the default to only send a notification to the new topic. This will match the default setting for moves via the topic three-dot menu.

Hi, I'm really new to this project and I was wondering how you can recreate this behavior (moving messages to another topic)  in the dev env. Thanks!

Welcome @djoo3 ! I recommend referring to the help center documentation: https://zulip.com/help/move-content-to-another-topic, https://zulip.com/help/move-content-to-another-stream.
Please also take a look at the Zulip contributor guide, and keep in mind the following guideline:

Before you claim an issue, you should be confident that you will be able to tackle it effectively.

I will go ahead and unassign this issue, and you should feel free to re-claim it once you have figured out how to approach it (or pick a different one if you prefer).

@zulipbot claim

@zulipbot claim

Hi. I'd love to work on this issue. I've recreated this scenario in my local development environment, but I need some help to locate where this feature is implemented.

I tried looking at the console to see where requests are sent, but still it doesn't help me to locate the code.
In general, how is the code for a particular feature normally located? Also, is it possible to run the server in a debugger?

Any help will be great!

@zulipbot claim

@zulipbot claim

@zulipbot abandon


Window Controller Buttons: Light theme inappropriateness

Microsoft PowerToys version
0.78.0
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
General, Settings
Steps to reproduce
By switching to light app theme in the settings, the control buttons at the above of the windows, do not adjust accomodatingly -Getting dark in light background.
Although, the is an issue, when the window is in active state.

✔ Expected Behavior
Simply and Obviously, I expected those three internationally-known buttons "- , o . x" to go dark in that light background to be possible to be seen.
❌ Actual Behavior
No contrast between the buttons and the background... and so, It's too hard to work with app in light theme, then, of course, shifting to dark is the only solution.
Other Software
No response

/dup #27199


Private class properties prevent Readonly type inference

🔎 Search Terms
"type inference readonly private class property", "generic type inference"
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about known bugs and type inference.

⏯ Playground Link
https://www.typescriptlang.org/play/?ts=5.5.0-dev.20240428#code/MYGwhgzhAEBiD28A8AVAfNA3gKGtADgE4CWAbmAC4Cm0A+kfPgPwBc0KA3NgL7bYUBPfDQCSAOwBmVQkljQqAD2piAJjATIwYgWgwBeOPKVVV6xEmKTp7DE3bQ2EsCAhUu2EFQp1qECm3EpGQ0kCkIAVypdLk9vWl8KACYAqxkAJSowFXgxEAFZczDI3TQOaAB6cugkAFpoAAN4qj9E+uhiGHrwsQBreAB3MTbiCThEADp6QkZ2mCIySipxoA
💻 Code
class Foo<T> {
  private _prop?: T;
}

type Infer<F extends Foo<any>> = F extends Foo<infer T> ? T : false;

let _test: Infer<Foo<true>>;
let _test2: Infer<Readonly<Foo<true>>>; // <- `_test2` is `unkown` if Foo._prop is private.

🙁 Actual behavior
The type of the variable _test2 is unknown.
Since Readonly<Foo<T>> extends Foo<T> there is no reason that TypeScript cannot infer T.
🙂 Expected behavior
The type of _test2 should be true or whatever T is (inferred).
Additional information about the issue
If the private modifier from _prop is removed, _test2 correctly shows true or whatever T is. A property modifier should not influence type inference.

nvm Readonly removes private types (???)

See #50668 and others.


Default XLA/GPU memory allocator is synchronous/slow.

This is a continuation of https://github.com/google/jax/issues/304 , in which @joschu observed that TensorFlow GPU outperforms JAX GPU for a Transformer benchmark.
I did a little digging in to the performance difference, and the thing that sticks out most is the memory allocator. The XLA Python client simply calls cudaMalloc/cudaFree on GPU, but NVidia's allocator is synchronous between host and device and apparently slow. Allocating dozens of small tensors can add up.
The obvious fix would be to use a suballocator, as TF and PyTorch both do. TF allocates a large chunk of GPU memory and suballocates it using a best-fit with coalescing (BFC) allocator, whereas PyTorch uses a caching allocator that lazily returns blocks to the CUDA allocator on OOM.
The easiest thing to do would be to use TF's allocator on GPU, although there is merit to the PyTorch approach too (better at sharing GPU memory).

I would be cool if there was an option to use Nvidia's native allocator as well as a custom allocator.  The native allocator makes it possible to detect memory leaks in kernel development (with cuda-memcheck).

Resolved via https://github.com/tensorflow/tensorflow/commit/73ae0b9d065e6fe39b90ef0db5b96adc220990fb (included in jaxlib 0.1.15). We expect multi-GPU workloads to be performant enough for real usage now!
@scott-gray you can set the environment variable XLA_PYTHON_CLIENT_ALLOCATOR=platform to use the nvidia allocator (I forgot to update the commit message, so this is incorrectly documented there :disappointed:).

@hawkinsp @skye should we close this now, or wait for full GPU async features to be switched on too?

Ah meant to close this.


Handling __dbt_tmp tables in incremental models

When I run an incremental model in an incremental run, dbt is creating __dbt_tmp tables and never cleaning them up.
In the SQL file:
{{
    config(
        materialized = "incremental",
    )
}}

Warehouse:

@yaragheb , which version of the adapter are you using?

@prdpsvs dbt-fabric==1.7.4

Same for me with dbt-fabric==1.8.6