Docx files generated by the library(v1.4.7) are corrupt

When generating a .docx file from a template, the Word is not able to open it.
It's weird that the LibreOffice or even Preview on macOS is able to open it just fine.
I'm suspecting some sort of metadata is missing from the file that Microsoft Word is not happy about.
I know there are new versions of the docx-stamper out there, but those have the Jakarta EE 9 migration in it. This migration prevents our project to use those new versions because other dependencies in our codebase use old version of JAXB that effectively causes clash between javax.xml.bind.JAXBElement and jakarta.xml.bind.JAXBElement.

I was not able to find a solution how to mitigate this issue, hence usage of 1.4.7 version.
I would be happy to help if I can, but unfortunately I'm not fully familiar with the codebase.
Nothing particularly interesting is reported in the logs(to my eyes, there are a couple of warnings but nothing else):
[http-nio-8080-exec-2] INFO  [user=] org.docx4j.jaxb.Context - java.version=17.0.3
[http-nio-8080-exec-2] WARN  [user=] o.d.jaxb.NamespacePrefixMapperUtils - com.sun.xml.internal.bind.marshaller.NamespacePrefixMapper .. trying RI.
[http-nio-8080-exec-2] INFO  [user=] o.d.jaxb.NamespacePrefixMapperUtils - Using NamespacePrefixMapper, which is suitable for the JAXB RI
[http-nio-8080-exec-2] INFO  [user=] org.docx4j.jaxb.Context - MOXy JAXB implementation is in use!
[http-nio-8080-exec-1] WARN  [user=] org.docx4j.utils.ResourceUtils - Couldn't get resource: docx4j.properties
[http-nio-8080-exec-1] WARN  [user=] org.docx4j.Docx4jProperties - Couldn't find/read docx4j.properties; docx4j.properties not found via classloader.
[http-nio-8080-exec-2] WARN  [user=] org.docx4j.utils.ResourceUtils - Couldn't get resource: docx4j.properties
[http-nio-8080-exec-2] WARN  [user=] org.docx4j.Docx4jProperties - Couldn't find/read docx4j.properties; docx4j.properties not found via classloader.
[http-nio-8080-exec-2] WARN  [user=] org.docx4j.XmlUtils - default SAXParserFactory property : null
        Please consider using Xerces.
[http-nio-8080-exec-2] INFO  [user=] org.docx4j.XmlUtils - actual: com.sun.org.apache.xerces.internal.jaxp.SAXParserFactoryImpl
[http-nio-8080-exec-2] WARN  [user=] org.docx4j.XmlUtils - default DocumentBuilderFactory property: null
        Please consider using Xerces.
[http-nio-8080-exec-2] INFO  [user=] org.docx4j.XmlUtils - actual: com.sun.org.apache.xerces.internal.jaxp.DocumentBuilderFactoryImpl
[http-nio-8080-exec-2] INFO  [user=] o.d.o.contenttype.ContentTypeManager - Detected WordProcessingML package
[http-nio-8080-exec-1] INFO  [user=] o.d.o.contenttype.ContentTypeManager - Detected WordProcessingML package
[http-nio-8080-exec-2] INFO  [user=] o.d.o.contenttype.ContentTypeManager - Detected WordProcessingML package
[http-nio-8080-exec-1] INFO  [user=] o.d.o.contenttype.ContentTypeManager - Detected WordProcessingML package
[http-nio-8080-exec-2] INFO  [user=] org.docx4j.openpackaging.io3.Load3 - Instantiated package of type org.docx4j.openpackaging.packages.WordprocessingMLPackage
[http-nio-8080-exec-1] INFO  [user=] org.docx4j.openpackaging.io3.Load3 - Instantiated package of type org.docx4j.openpackaging.packages.WordprocessingMLPackage
[http-nio-8080-exec-2] INFO  [user=] org.docx4j.utils.XPathFactoryUtil - xpath implementation: org.apache.xpath.jaxp.XPathFactoryImpl
[http-nio-8080-exec-1] WARN  [user=] o.d.o.contenttype.ContentTypeManager - DefaultPart used for part '/word/media/image2.svg' of content type 'image/svg+xml'
[http-nio-8080-exec-2] WARN  [user=] o.d.o.contenttype.ContentTypeManager - DefaultPart used for part '/word/media/image2.svg' of content type 'image/svg+xml'
[http-nio-8080-exec-2] WARN  [user=] o.d.o.contenttype.ContentTypeManager - DefaultPart used for part '/word/commentsExtensible.xml' of content type 'application/vnd.openxmlformats-officedocument.wordprocessingml.commentsExtensible+xml'
[http-nio-8080-exec-1] WARN  [user=] o.d.o.contenttype.ContentTypeManager - DefaultPart used for part '/word/commentsExtensible.xml' of content type 'application/vnd.openxmlformats-officedocument.wordprocessingml.commentsExtensible+xml'
[http-nio-8080-exec-2] WARN  [user=] o.d.o.contenttype.ContentTypeManager - DefaultPart used for part '/word/commentsIds.xml' of content type 'application/vnd.openxmlformats-officedocument.wordprocessingml.commentsIds+xml'
[http-nio-8080-exec-1] WARN  [user=] o.d.o.contenttype.ContentTypeManager - DefaultPart used for part '/word/commentsIds.xml' of content type 'application/vnd.openxmlformats-officedocument.wordprocessingml.commentsIds+xml'
[http-nio-8080-exec-1] INFO  [user=] org.docx4j.openpackaging.io3.Load3 - package read;  elapsed time: 167 ms
[http-nio-8080-exec-2] INFO  [user=] org.docx4j.openpackaging.io3.Load3 - package read;  elapsed time: 167 ms
[http-nio-8080-exec-1] WARN  [user=] o.d.jaxb.NamespacePrefixMapperUtils - com.sun.xml.internal.bind.marshaller.NamespacePrefixMapper .. trying RI.
[http-nio-8080-exec-2] WARN  [user=] o.d.jaxb.NamespacePrefixMapperUtils - com.sun.xml.internal.bind.marshaller.NamespacePrefixMapper .. trying RI.
[http-nio-8080-exec-1] INFO  [user=] o.d.jaxb.NamespacePrefixMapperUtils - Using NamespacePrefixMapperRelationshipsPart, which is suitable for the JAXB RI
[http-nio-8080-exec-2] INFO  [user=] o.d.jaxb.NamespacePrefixMapperUtils - Using NamespacePrefixMapperRelationshipsPart, which is suitable for the JAXB RI

Screenshots

Desktop (please complete the following information):

OS: macOS Monterey & Windows 10
Java version: 17
docx4j version: 6.1.2
docx-stamper version: 1.4.7 (Can't go higher than that due to Jakarta EE 9)

Ok I just discovered that docx4j 6.1.2 still using javax.xml.bind packages, so I'm guessing if I drop org.docx4j.docx4j:6.1.2 and use org.docx4j:docx4j-core:11.4.8 I wouldn't get java.lang.ClassCastException: javax.xml.bind.JAXBElement incompatible with jakarta.xml.bind.JAXBElement anymore with 1.5.0.
Yes this worked, and the files are not corrupt anymore with the newest versions.
Closing this.

Thx for the feedback.
I'm trying to move the lib to the latest docx4j that supports jakarta binding (the 1.5.0 should works with it)


Empty Mountpoints Are Not Backed Up

When backing up a system with, say, restic backup -x /, I expect for /dev/* or /proc/* to not be backed up, since it's a different filesystem. However, I do expect the base directory, /dev, to be backed up as an empty directory, since it resides on the parent filesystem. I realize you can't descend into covered mounts, in case an unmounted /dev isn't empty, somehow, but at the very least you can back it up as empty?

Huh, you're right, these directories are not saved. This behavior is caused by the fact that calling lstat() on a mount point returns the stat for the root node of the fs mounted there, so it is excluded. This needs to be handled as a special case, unfortunately. Thanks for the report!

FYI, there's a PR which will fix this issue: #3119


Add GPT* compatibility to torchhub

I'll add GPT2 for torchhub later.

Amazing, thanks a lot @VictorSanh!


Shall we include a normalized timestep in the state info for only one or all components?

https://github.com/moratodpg/marl_struct/blob/831de86f5b4ed703f83dae67cc39d31e9849c29a/struct_env/pymarl_ma_struct.py#L136
Let's confirm this and, if necessary, adapt it.

If results are way better, I think we can :)


Parchment does not work with navigation bars

I'm sorry to bother you yet again but I found this problem.
I had a viewcontroller embed in a uinavigationcontroller and in a uitabbarcontroller.
I compiled the example project you listed and it worked. Then I embed it within a navigation bar but it becomes hidden under the bar. I'm confused because your examples render it correctly.

Also, constrain to edges doesn't work in Swift 4.

Hi! The default behaviour on iOS is that the content appear underneath the navigation bar. You can set edgesForExtendedLayout = [] in your view controller to prevent this, or if you're using Storyboard you can disable the Extend edges: Under Top Bars option. With iOS 11, you can constrain the view to the safeAreaLayoutGuide instead, which is recommended by Apple:
https://developer.apple.com/documentation/uikit/uiviewcontroller/1621515-edgesforextendedlayout
The constrainToEdges is not part of UIKit, but rather an extension that I made to make it easier to setup the constraints. You copy over the source from UIView+constraints.swift if you want the same function in your app.

Thank you so much. I won't bother you anymore. :)

My actual solution to the problem was to manually constrain the childViewController below the statusBar and navBar. This retained the translucent effect


Render WebView on Top of Native GPU Rendered Content [feat]

It would be pretty amazing if we could use the power of native GPU frameworks (Vulkan, DirectX, OpenGL, Metal, etc.) to render content with a WebView based UI on top (all in the same window).
While I realize we can use WebGL to achieve this, it's much more limited and often slower than native GPU frameworks. This would be another really nice win over Electron if it was supported and would be the best of both worlds in terms of the power of a native app with the ease of development for UI interfaces that the web gives us.
This would be pretty nice for games to render the game content using the native GPU and render UI with a web based approach.

This is more or less actively being worked on, for example: https://github.com/tauri-apps/tauri/issues/3662#issuecomment-1807219070
(Also, I'm 90% sure there already is some kind of tracking issue in this repo but I can't find it on my phone)

Is there a simple example? Or when an example becomes available can someone share a link here please.

+1

v2 beta has unstable cargo feature, when enabled, will allow you to create a window and multiple webviews as children, so you can render OpenGL (or similar) on the window and place a webview on top of it.


pug: Improved class and id in tag detection

pug: Improved class(named "attr-class") and id(named "attr-id") in tag detection, and added corresponding unit test.
In order to distinguish the id, class and pure tag declared by the tag in the pug, two regular expressions judgment are added to the corresponding code, and analogous unit testings.
Similarly, the next PR is to add judgments on id, class, pesudo (pesudo-class and pesudo-element), ... in Stylus, it is comming soon.

Thank you for making this PR @dev-itsheng!
To make the build pass, you have to install Prism's dependencies using npm ci and then run the npx gulp command to rebuild all generated files (all .min.js files and some more).
Please also take a look at my comment, and please stop lewding Chino.

Yes, your solution is better than mine.
At first, I find that /#[\w\-]+/ will match #hash of a(href="example.html#hash"), so I add ^[\t ]*(?!-) to match the white-space front the tag, [\w\-.]* to match the beginning of the tag or attr-class. It seems to solve the problem.
If insert it after 'punctuation', the problem will not be occurred.
I will make a new PR by correct method soon. Thanks you for this reply.

Glad, I can help.
Also, you don't need to make a new PR, you can just commit and push updates in this one.

I rewrite code, run npm i, npx gulp, try to commit and push it.

My previous avatar is just for fun, but I changed it immediately.

In fact, I has already considered it.
In my opinion, prism supported many languages, it's possible that another language has similar id or class definition, to avoid it, I finally use attr- prefix.
On the other hand, I reviewed some theme CSS files, they use .token.boolean selector rather than [class*="language-..."] .token.boolean, I think that id in pug is different from id in CSS and CSS extension language.
Final, I choose attr-id and attr-class.

Sound reasoning. Then let's keep it as is.

Thank you for contributing @dev-itsheng!

By the way, my GitHub experience is very poor, if I want to create a new PR to solve the another problem, should I delete my fork and create a new fork from this project? or continue to commit and push in my repo?
The next PR will fix the Stylus bug:

You could delete your fork (I see some people do this, don't be like them) but you can also just use the features git has for this:
I won't go into the details because git can be complicated at first but this is what you have to do:

Open a terminal in your prism folder.
Since we added a new commit to the master branch of PrismJS/prism, you have to get these changes in your fork. So first, we have to teach git about the PrismJS/prism repo. Run git remote add upstream https://github.com/PrismJS/prism.git in your terminal. upstream will now refer to the PrismJS/prism repo.
Now git knows that the PrismJS/prism repo exists, but it didn't actually download anything yet. Run git fetch upstream.
Now we merge the changes from upstream (PrismJS/prism) into your fork. First, switch to the master branch of your fork with git checkout master, and then merge the changes from upstream (PrismJS/prism) with git merge upstream/master.
Push to your fork with git push.

Your fork is now up to date.
For a new PR, make a branch from the master branch of your fork. The IDE you use probably has some nice way of doing this. Make your commits into that new branch. To make a PR, publish your branch to your fork.
I hope this helps.
For more details, just google stuff, I guess? There are a lot of resources for git.

Thanks for your help.
In fact, I has already try to use Google to search like 'GitHub new PR after PR' and other keywords, but don't get any useful information, and GitHub always gives me a suggest of deleting this fork, like this:

So I'm afraid of previous commit and push will have an influence on next PR, but after reading your answer(my English is poor, too, hope my understand is correct), my worry is unnecessary. I can continue to commit and push code in this fork(after up to date, if my commit is latest, I couldn't do something about merge).

Yeah, I can image that you won't find anything useful there. From my experience, when I started with git, I only found one guy who actually explained how you do this fork updating (but only that). Git isn't easy to learn but once you get it, it's super useful because literally everything you might want to do with git, you can do. Git is super powerful.
GitHub is really just a server of git repos with a nice web page. It doesn't do a whole lot when you actually use git.
My recommendation: Take your time to really learn git (not just GitHub). It doesn't have to be all at once.
As a start, try to understand why you had to do everything I described in my previous comment. This article (GitHub focused) should explain it. Focus on the concepts not the commands.


How to Pass the wit.ai json such as entities and outcome to modules?

Beacause i need to get the data of outcomes and entities of the json output from wit.ai and then i need it to be passed to the modules. Need help
def transcribe(self, fp):
        data = fp.read()
        r = requests.post('https://api.wit.ai/speech?v=20150101',
                          data=data,
                          headers=self.headers)
        try:
            r.raise_for_status()
            #parsed_json = r.json()
            #for intent in parsed_json["outcomes"]:
            #   global output
                #output = intent ["intent"]
            #print output

Whats the issue here ?

how to pass the parsed json from the transcribe method to handle method in the modules

@sathish09 Did you find a solution which worked for you?


use set instead of setHeader

Not sure why setHeader is used at all. The block that was added to index.js was taken from https://github.com/expressjs/cors/commit/fee800f0dc13a1df5b696035f09c43b8c714371b. I'm not sure why it was pulled out of that file with that PR as the comment that caused that PR (https://github.com/expressjs/cors/commit/165f35e8) has nothing to do with that block.
The problem stems from this...
When you use setHeader with the mock response that Sails uses the headers that are set are set in res._headers and not res.headers, so our specs fail.

LGTM.  As we discussed earlier, we should probably PR this against the cors library to see if we can get it back into the main codebase.

Pish.  Whatever.


Amazon Clone

Project title
Amazon Clone
Project Description
This is an E-COMMERCE website, tech stack to used is HTML, CSS, Js
Doc Domain

[] Solidity , Html, Css , JavaScript , Web3.js / Ether.js
[] Solidity , ReactJs /NextJs , Web3.js /Ether.js
[] Solidity , Python3 , Web3.py
[] Other , feel free to mention below

Are you contributing under any open-source program ? Yes

Please @Vikash-8090-Yadav assign this to me so that ill be start working on it, to add an ecommerce clone to our front-end project.

Is anyone intrested to work on this issue to make an add to card interface please tag me.

Hey , It would be great if i got this issue assigned

Hey , It would be great if i got this issue assigned

Tag @Vikash-8090-Yadav to assign as how i did.
Will you make shop cart to this clone

@Vikash-8090-Yadav Hey, It would be great if i got assigned with this issue.

@TanishqSiva Yeah I can make shopping cart for this clone .
We can discuss about it on discord here is my discord i'd: Mohd Sultan#9593

@TanishqSiva Yeah I can make shopping cart for this clone . We can discuss more about this project on discord. Here is my discord i'd: Mohd Sultan#9593

@TanishqSiva why you cloaed this issue ?

@Vikash-8090-Yadav which repo should I have to explore can you please be more specific

@Vikash-8090-Yadav which repo should I have to explore can you please be more specific

All. Find some good web3 project

@Vikash-8090-Yadav I want to say that where I can find that repo can you please provide me a link of it please

@Vikash-8090-Yadav I want to say that where I can find that repo can you please provide me a link of it please
https://github.com/Vikash-8090-Yadav/Future.WebNet


Postgres migrations broken in beta 115

Describe the Bug
After upgrading to beta 115, I am getting this error:
src/migrations/20241016_175315.ts:646:36 - error TS2345: Argument of type 'SQL<unknown>' is not assignable to parameter of type 'string | SQLWrapper'.
  Type 'SQL<unknown>' is not assignable to type 'SQLWrapper'.
    The types returned by 'getSQL()' are incompatible between these types.
      Type 'import("/..../node_modules/.pnpm/drizzle-orm@0.35.1_@types+pg@8.10.2_pg@8.11.3_react@19.0.0-rc-3edc000d-20240926_types-react@19.0.0-rc.1/node_modules/drizzle-orm/sql/sql").SQL<unknown>' is not assignable to type 'import("/..../node_modules/.pnpm/drizzle-orm@0.34.1-1f15bfd_@types+pg@8.10.2_pg@8.11.3_react@19.0.0-rc-3edc000d-20240926_types-react@19.0.0-rc.1/node_modules/drizzle-orm/sql/sql").SQL<unknown>'.
        Types have separate declarations of a private property 'shouldInlineParams'.

646   await payload.db.drizzle.execute(sql`

Link to the code that reproduces this issue
https://github.com/payloadcms/payload/blob/beta/packages/drizzle/package.json#L49
Reproduction Steps
The breaking change introduced in https://github.com/payloadcms/payload/pull/8617, mentions upgrading the libsql package, which comes with drizzle >0.35. However, in the drizzle package from Payload, it is not updated.
See also Discord; https://discord.com/channels/967097582721572934/1296361666765656138
Which area(s) are affected? (Select all that apply)
db-postgres
Environment Info
Binaries:
Node: 20.12.1
npm: 10.5.0
Yarn: 1.22.22
pnpm: 8.15.6
Relevant Packages:
payload: 3.0.0-beta.115
next: 15.0.0-canary.173
@payloadcms/db-postgres: 3.0.0-beta.115
@payloadcms/email-nodemailer: 3.0.0-beta.115
@payloadcms/graphql: 3.0.0-beta.115
@payloadcms/live-preview: 3.0.0-beta.115
@payloadcms/live-preview-react: 3.0.0-beta.115
@payloadcms/next/utilities: 3.0.0-beta.115
@payloadcms/plugin-cloud: 3.0.0-beta.115
@payloadcms/plugin-cloud-storage: 3.0.0-beta.115
@payloadcms/plugin-form-builder: 3.0.0-beta.115
@payloadcms/plugin-nested-docs: 3.0.0-beta.115
@payloadcms/plugin-redirects: 3.0.0-beta.115
@payloadcms/plugin-seo: 3.0.0-beta.115
@payloadcms/richtext-lexical: 3.0.0-beta.115
@payloadcms/translations: 3.0.0-beta.115
@payloadcms/ui/shared: 3.0.0-beta.115
react: 19.0.0-rc-3edc000d-20240926
react-dom: 19.0.0-rc-3edc000d-20240926
Operating System:
Platform: darwin
Arch: arm64
Version: Darwin Kernel Version 24.0.0: Tue Sep 24 23:39:07 PDT 2024; root:xnu-11215.1.12~1/RELEASE_ARM64_T6000
Available memory (MB): 32768
Available CPU cores: 10

Cheers @r1tsuu, appreciate the fast response and feedback! Looking forward to the fix! 🚀

Patching the<EMAIL_ADDRESS>helped:
index 6ac1e1a446f1b205938f80eaa5014be0c32c69b5..1c379ded6e79b7b5e99b57b1597e1a360693d752 100644
--- a/package.json
+++ b/package.json
@@ -36,7 +36,7 @@
   ],
   "dependencies": {
     "console-table-printer": "2.11.2",
-    "drizzle-orm": "0.34.1-1f15bfd",
+    "drizzle-orm": "0.35.1",
     "prompts": "2.4.2",
     "to-snake-case": "1.0.0",
     "uuid": "9.0.0"

Fixed in https://github.com/payloadcms/payload/pull/8759, will be in the next release


update (plugins/udp-log, tcp-log, http-log, syslog, loggly, file-log)

Two new Customer's fields username  and custom_id  added to  authenticated_entity section of serialized request/response data

Merge only after https://github.com/Mashape/kong/pull/2090 get merged.


Why the move away from npm registry?

I noticed from this ed18acd63d7a5cd31527a36a9af54542cdfcfd30 that you moved away from publishing to the npm registry and recommend people download using tarballs on the CDN. Why did you move away? The npm registry is an extremely common way to download packages.

The package on the npm registry is surprisingly popular (one of the top-500 by dependents).  npm invalidated the old publish token and is forcing 2FA on the publishing account.
Due to ongoing legal matters between SheetJS LLC and npm, Inc. (which will not be discussed here), it did not make sense to continue using the public npm registry for distribution.
With GitHub (the owner of NPM) sunsetting the git.io domain with only 4 days notice, we are reminded of the ephemerality of the Internet and the inherent risks of relying on platforms.

we are reminded of the ephemerality of the Internet and the inherent risks of relying on platforms

https://sheetjs.com/careers

familiar with the tumult of open source and remote collaboration
not prepared to collaborate with the JavaScript and data communities

Top 500 NPM package list, 1.26 million downloads a week, over 3000 dependent packages and not even a mention in the README regarding the fact ongoing development for this package won't exist any longer in the single largest JS library platform on the planet (after 8 years).
🤔

Mandatory 2FA should be a noop, as a responsible maintainer would already have it enabled.
npm publish tokens remain exempt from 2FA, so i'm not clear on why that would be an obstacle.

Yeah, 2FA should be a no brainer. An tokens are indeed except if specified.

This is bizarre

Full disclosure, I happened upon this issue by chance and am not a SheetJS user, but this is... really strange.

The package on the npm registry is surprisingly popular (one of the top-500 by dependents). npm invalidated the old publish token and is forcing 2FA on the publishing account.

What possible justification could you have for taking issue with npm's 2FA requirement for maintainers of popular packages?

With GitHub (the owner of NPM) sunsetting the git.io domain with only 4 days notice, we are reminded of the ephemerality of the Internet and the inherent risks of relying on platforms.

I don't see how this is a reason for silently dropping support for npm. If anything, from the perspective of your users it's an argument for the opposite, because npm is statistically way more likely to exist 5 years from now than your personal CDN.
Maintainers of OSS projects don't owe people anything, of course, but all I can say is you shouldn't be surprised when people (including your paying customers) look at this whole thing and decide to either fork the project or switch to a competing library not maintained by someone who makes decisions like this.

I imagine the conversation going somewhere along the lines of:

phone rings.

Hey, I just got an email saying that our MPM account didn't have 2FA enabled. I think that's like really important right? Why didn't you have it enabled?

Moment of terror. My predacessor setup the account... I never thought to check if that 2FA was enabled! But I probably shouldn't say that to the CTO because I definitely should noticed. Um what to say.. uh.. um. come on think! think AHA I got it!

Really sir? That is very concerning. They must of uh like um deleted our settings.
Deleted our settings?! That's outrageous! They can't do that! Those are OUR settings. Just go ahead and only post it on our site from now on.
Post it on our site? Like a CDN? Well, I'll just reset the security settings no problem.
No. I doubt these MPM guys will be around a lot longer anyways. You will post it on our site only from now on, am I understood?
uh.. yes...
Good work, I'm going to go call the lawyers about this.

CTO hangs up. Oh God what did I just do?!

JavaScript hippies back at it again with the tri weekly cdn outages

Not sure why one would want to use the sheetjs CDN for npm installs instead of just doing something like this(github based install):
npm install SheetJS/sheetjs#semver:^0.18.6

Hilarious.

First, thanks for your amazing work on this library.
Because of security concerns, I prefer relying on npm up-to-date package with proper version management, immutable release binaries and npm audit facilities. I'm sure your users would appreciate an update of the well-known npm registry.
If the issue is 2FA related, other popular packages solved the issue. Without you explaining the other reasons, it is hard to understand the withdraw.
Have a nice day.

This has got to be the most bizarre OSS move I've seen since 2016 when an angry developer unpublished all his 250 npm packages and broke builds all over the planet.

There are over two million weekly downloads on npm pulling in an outdated version of this library. This number is growing meaning an increasing number of people are using the wrong version.
There are no warnings whatsoever to let developers know they're using the wrong mechanism to pull in this dependency. The brief mention on the fourth paragraph in the README is not sufficient. This needs to be a bright flashing red banner at the very top. The NPM package entry should be updated to reflect this is no longer the latest version and to encourage developers to move over to the CDN.
The motivation for this change is flimsy. I understand it's either "I don't want to use MFA" or "I'm in some legal trouble with npm". I don't get it.

Whatever your beef with npm is, please work it out and move on. Don't let it ruin what is otherwise an exceptionally well-designed and well-maintained project.

Just wanted to second the motion for a big update to the README and a warning when installing from the public registry. This is very abnormal and I (like many others, most likely) did not realize I was running on an old version of this package.

Why the move away from GitHub?

@marracuene how flexible is it?
I had to do some complex Excel manipulations at work and SheetJS was the only package I found that's relatively maintained and can do complicated stuff...

@marracuene how flexible is it?
I had to do some complex Excel manipulations at work and SheetJS was the only package I found that's relatively maintained and can do complicated stuff...

@YogliB I suspect it would not handle your use case. In our use case we already have, in-memory, an array of Objects that represent the information to be exported via Excel. All the work to prepare this information has already been done.
The only additional work done at time of export (and which the new package allows us to do), is custom-formatting on specific fields.

I know, I know,
nobody should be installing stuff GPT-4 recommends without proper assessment first.
But damn, GPT and any search engine out there straight up recommending to just npm install xslx (like you would do in any reasonable, well maintained package).
After finally finding relevant docs (you really have to search for it), reading through a bunch of almost irrelevant demos, still couldn't understand why the npm package hasn't been updated in over a year.
Is it so good that it doesn't have bugs or improvements to make?
Nah, I get it now.
This was a fantastic opportunity to switch from work to my 8pm routine of dinner and watching comedy.

@SheetJSDev

it did not make sense to continue using the public npm registry for distribution.

Hi, it's been a year since you wrote that. Please consider updating your npm entry to reflect the vulnerability status of the last posted version and to point people in the right direction? I think it would be the right thing to do for the community. Thanks!

Hey, folks! I made a little tool that allows you to continue using xlsx in your projects.
More details are described here https://github.com/SheetJS/sheetjs/issues/2822#issuecomment-1668303942
I'm writing here as well, so that more people interested in solving the problem will get notified 🙂

Maybe this package should be marked as deprecated on NPM? 🤔
That doesn't make sense to keep it available like that with an outdated and with vulnerabilities in it?

I suggested that.
Do the responsible thing, SheetJS!
El El lun, oct. 2, 2023 a la(s) 5:44 a.m., Vincent Giersch <
@.***> escribió:

Maybe this package should be marked as deprecated on NPM? 🤔
That doesn't make sense to keep it available like that with an outdated
and with vulnerabilities in it?
—
Reply to this email directly, view it on GitHub
https://github.com/SheetJS/sheetjs/issues/2667#issuecomment-1742788412,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGQZM2FU47CELSE7LJI2UTX5KLJNANCNFSM5UNOGEVA
.
You are receiving this because you commented.Message ID:
@.***>

Pretty please with sugar on top, please release on NPM, You'll fix millions of problems. You remember that song from Mariah Carey? "And then a hero comes along, with the strength to carry on". That will be playing in the background as you do it.

oof.

Top 500 NPM package list, 1.26 million downloads a week, over 3000 dependent packages and not even a mention in the README regarding the fact ongoing development for this package won't exist any longer in the single largest JS library platform on the planet (after 8 years).

Is sheetjs sold or will it be?

Since the CE edition is licensed under Apache which allows republishing it with proper attribution I have made a small automated script that periodically checks the CDN and republishes the latest version to NPM if needed.
The NPM package can be found here https://www.npmjs.com/package/xlsx-republish.

Since the CE edition is licensed under Apache which allows republishing it with proper attribution I have made a small automated script that periodically checks the CDN and republishes the latest version to NPM if needed.
The NPM package can be found here https://www.npmjs.com/package/xlsx-republish.

What a great public contribution.  Well done.   We actually moved a way from sheetjs a while ago due to this whole weird soap opera, but you will surely save lots of effort for lots of peeps.

looks like there's two republished npm packages:

@e965/xlsx (most downloads- see comment above)
xlsx-republish (recently created- see comment above)

They couldn't figure out how to charge money for the freemium version with NPM. Hard pass.

So you are saying that the undisclosed legal challenges with NPM that have been mentioned here, have to do with trying to charge for something that is published? That really puts a very different spin on things and doesn't look good for the authors…

As I noted above, the responsible thing to do would be to mark the versions that are in NPM as deprecated, with a warning.


webview error 'Encountered an error loading page @RN 0.34.1

E/ReactNativeJS(27798):   'Encountered an error loading page', { canGoForward: false,
E/ReactNativeJS(27798):   code: -2,
E/ReactNativeJS(27798):   canGoBack: false,
E/ReactNativeJS(27798):   description: 'net::ERR_NAME_NOT_RESOLVED',
E/ReactNativeJS(27798):   loading: false,
E/ReactNativeJS(27798):   title: '',
E/ReactNativeJS(27798):   url: 'https://www.baidu.com/',
E/ReactNativeJS(27798):   target: 122 }

following my code:// component is custom.
                <WebView
                    scalesPageToFit={true}
                    renderLoading={()=><Loading value="XXX"/>}
                    renderError={()=><Loading value="XXX"/>}
                    onError={()=><Loading/>}
                    source={{uri: "https://www.XXX.com"}}
                    style={{padding: 10}}
                />

Who can help me?

I meet this error when I close network.

Are you set up a proxy?

l have de same problem
Encountered an error loading page Object {domain: "WebKitErrorDomain", canGoBack: true, target: 10, code: 101, loading: true…}canGoBack: truecanGoForward: falsecode: 101description: "The URL can’t be shown"domain: "WebKitErrorDomain"loading: truetarget: 10title: "百度一下"url: "https://www.baidu.com/#0.26427529194080224;0.877470845104408%3Bthis.setHeight(1387)"__proto__: Object

+1

l  have the same problem too  +1

I meet this error even when I do not close network.,
1、sometimes the page will show this problem and quickly show my nomal atticle page.
2、 sometimes the page will show this problem all the way,and do not have other changes.
case 1 ,  most appeared,Probably accounted for 90%
the pic below shows the error

and my english is poor, can you understand my describ?  ,i really really want to solve this problem,
Who can help me? thank you @naivehhr

Same here, does any one have a proper solution for this issue.

I  solve this issue  by implementing onShouldStartLoadWithRequest . the error occur when the webView try to open a wrong url.  we can intercept the request before it happen.

@LeonHwa do you have a sample on how you resolved this?

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with Canny! Check out its new home: https://react-native.canny.io/feature-requests/p/webview-error-encountered-an-error-loading-page-rn-0341

Yes, this is still an issue. I'm seeing this on the latest react-native.

Perhaps, wait for a solution to be posted before closing the issue.

+1

+1

+1

+1

+1

Please open a new issue and make sure to follow the template.


remove sourcemap in dist when publish

https://www.npmjs.com/package/@rspack/core?activeTab=code
we shipped sourcemap without shipping source, and it seems unnecessary ship source and sourcemap in the dist(which will cause npm download slow)
so it would be better we just remove sourcemap when publish(but we need sourcemap in development for better debug experience)

we may need two tsconfig.json, tsconfig.build.json for build and publish(without sourcemap) and  tsconfig.json for local development(with sourcemap).

@hardfist - I would be interested in contributing to this issue. I would likely use the existing "prepare" script in rspack/core, and run a clean/build using another tsconfig.build.json (or tsconfig.lib.json) with source maps disabled.
Assuming we wanted the build to be part of prepare, or just enhance the existing build script since there is already a dev script that performs a watch on the default tsconfig.json.

@hardfist - I would be interested in contributing to this issue. I would likely use the existing "prepare" script in rspack/core, and run a clean/build using another tsconfig.build.json (or tsconfig.lib.json) with source maps disabled.
Assuming we wanted the build to be part of prepare, or just enhance the existing build script since there is already a dev script that performs a watch on the default tsconfig.json.

yeah. that is what we want


Missing example file, input.example.json.

At this page https://github.com/PacificBiosciences/HiFi-somatic-WDL/blob/main/docs/step-by-step.md#modify-the-input-json-file-to-point-to-the-downloaded-files
there is a link to an example input json but the link is dead https://github.com/PacificBiosciences/HiFi-somatic-WDL/blob/main/docs/example_configs/input.example.json

Fixed. Thanks.

File is still not found on my end.


How to combine ParallaxScroll component with react-navigation's TabNavigator

I would like to have TabNaivagator inisde ParallaxScroll and have when user scrolling up have sticky header which is TabNaivagator header. Any help?

@zoranmarjanovic can u please show an example code how you achieved this. I am struggling with this. Please help


When Global DNS annotation for an ingress is updated  , it continues to be associated with the older fqdn.

Rancher server version - Build from master - Feb 2
Steps to reproduce the problem:

Create Multicluster App - mcapp1 for cluster1 and cluster2 with Global DNS enabled Ingress for fqdn - "test1.test.com"
Create globalDnsProvider with root domain - "test.com"
Create a globalDns entry  with fqdn - "test1.test.com" pointing to Multicluster App - mcapp1.
After this , we see external DNS entry created for  fqdn - "test1.test.com" as expected.
Edit MulticlusterApp and set the Global DNS enabled Ingress for fqdn to "test2.test.com"
External DNS entry continues for "test1.test.com" continues to point to MulticlusterApp

Expected Result:
External DNS entry for "test1.test.com" should get deleted at this point.

Tested with latest build from master - Feb 7
When Global DNS annotation for an ingress is updated to a new fqdn , DNS records relating to the older fqdn gets deleted from External DNS provider as expected.


Fixes to repr element names and allowing paging of firewall rules

Original contribution from @mingsheng36

Already reviewed. will merge

Already reviewed. will merge


Usage of servers in the Open Api specification with mount prefix

Currently, when all controllers of an API are loaded using the mount parameter of the global configuration, they are assigned the same prefix in the Open API specification as shown in the following image.

The question is. Is there currently any way to remove this common prefix from all endpoints and configure it, for example, in the servers section of the Open Api specification?

Currently, if the servers section is configured in the Open Api specification, the path that is configured is concatenated to the prefix defined in the mount configuration.
Thanks in advance 🙏

Also, tsed support various configuration. Mount option can many root paths, not onlu /api or /rest. In this case your needs isn’t possible.

Thank you very much, @Romakita . It was just to confirm. It's nothing blocking or extremely necessary, so I close the issue.
Thank you again 🙏


Margins in example notebooks

Unlike regular text blocks, code blocks have zero margins at the bottom. For a sequence of code blocks, this looks fine.
But when for notebooks where code and text are continuously alternating (for example PyVRP components), this results in the following:

This looks a bit too compressed to me. I'd want to see some margins between the code block and the subsection header. And also some margin between the code block and the next paragraph. For example:

Increasing the bottom margin of code blocks "fixes" this problem, but it makes a sequence of code blocks look ugly.
I think the solution is to add some top margin to paragraphs and sections headers (for h3 and smaller, h1-h2 already have some marigns at the top).

I really don't want to tinker with this in e.g. CSS. The RTD theme we currently use has more warts, like rendering lists that's really finnicky. We might be able to just switch themes, for example to sphinx_immaterial, which statsmodels and ruff also use.

I really don't want to tinker with this in e.g. CSS, mostly because it's really tedious to write and maintain, and annoyingly ties us to a particular theme. The RTD theme we currently use has more warts, like rendering lists that's really finnicky. We might be able to just switch themes instead? An alternative is sphinx_immaterial, which statsmodels and ruff also use.

Another theme is also fine by me (and something I have considered before). I'll update the issue to be about choosing a new theme.
Sphinx Immaterial looks good to me, and based on the statsmodels examples, the notebooks are rendered nicely.

I'll have a go at this!


Option to hide top panels

Following on from #18 which will really help us as we add new repositories to our big list.
The main niggle now is that even if we search with a wildcard in the URL, we still end up with a big list in the text box so the top area takes up a lot of space and we have to scroll down to see the actual chart.
It would be neat if there were a button and url option we could add that would hide/minimise the top panel, a bit like the maximise view in GitHub projects. We generally won't be changing any options within each sprint so the options at the top are redundant for us for 99% of the time.

Hi @mungojam. This is in fact already supported. Quoting from the Yoda manual (it may not be too visible...):

Note: A further URL argument, draw, will result in the graph being drawn immediately (without the user having to press “Draw Chart” button). Simply add “draw=true” to URL. Similarly, it is possible to hide the header (e.g. if including the graph into a dashboard as an iFrame) by putting “hideheader=true”

So, I'm closing this one for now.
However, I am looking at options to restrict the vertical space taken up by the repository selection box, when selecting many repos. Will explore a mechanim by which you will see something like (x repositories selected) when x exceeds say 10. Stay tuned.

Amazing, thanks. And apologies, I admit I hadn't even looled for a manual in this case which is unusual for me!
I realize now from this and my other issue that there are some things only supported in the URL so the manual is key for those. That's absolutely cool with us since we will always use bookmarks for these dashboards.

Additionally, I've done now a bit extra.
As part of the Yoda menu (the "Hamburger" menu top left), there are now two new entries. One to hide the top panel (except the hamburger) and one to get it back. You can control this as well with URL argument hidepanel=true.
The panels for the Burndown and Velocity reports have been slightly reorganized to ensure that the repository selection are in the top panel and - for Velocity report- that the Milestone definition was moved to the second panel.


Replace DETRAC dataset download link

This link provided in READEM will be redirected to the homepage of Albany Computer Vision and Machine Learning Lab. It took me fair enough time to finally download the dataset.
I'd like to replace the original link to this link to ease the download process.

Hi, that link is not working, could you please share where to download?

Try this
https://wayback.archive-it.org/org-652/20231112212640/https://detrac-db.rit.albany.edu/download


Switch setup.json to setup.cfg

We could look at the current output of aiida plugin cookiecutter for guidance.
@pzarabadip happy to take this on in case it would help.

This is PR from aiida-core is relevant https://github.com/aiidateam/aiida-core/pull/5312

I'm already doing it but not with setup.cfg. I have it in another branch which I will push once #37 is merged.


Fix radix_sort_by_key for desc order

This should fix https://github.com/boostorg/compute/issues/588.
Now in radix sort when greater<T>() is specified as comparison function (desc order) instead of performing asc-ordered radix sort and then reversing the result, descending radix sort is performed.
It's done by negating input value if its type is a signed type and by subtracting the input value from the maximum value of value's type in other cases (in radix() function). That way values seem "reversed" and we get desc order. It should also be faster than sorting in asc order and then reversing the result.
I also added some test for desc-ordered radix sort and radix sort by key.

looks good :+1:

Coverage increased (+0.1%) to 88.791% when pulling f29bbda7f89ee7cc7bb89309a7a4f7f5dcc7d592 on haahh:pr_sort_by_key_desc_fix into 437363b6b70784193085a0af1dd55f9d435e39de on boostorg:develop.


[feature-request] multiple Inheritance, Ractive.extend(A, B, C)?

You know if I had a common prototype which I want to extend to use for multiple components.
var Foo = Ractive.extend(protoA, protoB, protoC);

Right now I do this
var Foo = Ractive.extend($.extend(true, protoA, protoB, protoC));

Easy enough, but fyi Ractive does some magic on a) combining registries, and b) wrapping overrides with _super so the two examples you gave are not strictly equivalent.

ok - cool, thanks

What are the difference with doing this? https://github.com/ractivejs/docs.ractivejs.org/issues/14 https://github.com/ractivejs/ractive/issues/364#issuecomment-32001114
Ractive.extend(protoA).extend(protoB).extend(protoC)

@codler This PR for this #1704 is just syntax sugar to avoid calling .extend successive times.
Current API as I understand it, is to take options or another extended instance. I wouldn't pass in a .prototype of anything.
Maybe your question is related to #1687?

Okey so Ractive.extend(protoA).extend(protoB).extend(protoC) is same as Ractive.extend(protoA, protoB, protoC) now.
nope, my question was not related to #1687 :P

Okey so Ractive.extend(protoA).extend(protoB).extend(protoC) is same as Ractive.extend(protoA, protoB, protoC) now.

Yep, it was a gratuitous opportunity to use ES6 arguments splats. :)

...options nice


italian translation

ive added an italian translation for readME and contributing, added a link in the readME to go to the italian translate file

id like to add a publishing translate as well, in a separate folder alongside the English version as for the moment only available in English

id like to add a publishing translate as well, in a separate folder alongside the English version as for the moment only available in English

Could I join you in doing this?
Which platform are you using to translating it in italian?

yeah sure! for the most part i translate it myself, but i suggest to use "reverso" for a better translation, just make sure you review the text afterwards!

id like to add a publishing translate as well, in a separate folder alongside the English version as for the moment only available in English

Could I join you in doing this? Which platform are you using to translating it in italian?

yeah sure! for the most part i translate it myself, but i suggest to use "reverso" for a better translation, just make sure you review the text afterwards!

Hey @rickyl3aks, thanks for your work on the Italian translation. Grazie mille! The PR including the Italian translation has already been merged into Bruno and released in v1.2.0.
Closing this!


[StorageSync] Address self-review comments

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

/azp run net - storagesync - mgmt

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.StorageSync

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.StorageSync

some comments in the apiview

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.StorageSync

some comments in the apiview

resolved.

/azp run net - storagesync - mgmt


Tab to autocomplete doesn't fully highlight the text

Laptop keyboards are amazing. I will finish it rq

https://github.com/user-attachments/assets/1e68d65f-903a-4a29-920d-beb4a7a561d0

Do you get any errors in the F9 console when this occurs?


Webpack docs: what does api refer to?

On the Webpack configuration page, it says the following:

In api.chainWebpack and api.configureWebpack, you have access to some environment variables:

process.env.VUE_CLI_SSR_TARGET: Either 'client' or 'server'
process.env.VUE_CLI_MODE: Vue CLI mode

What does api refer to? Is it possible to provide a complete example that shows where the api variable comes from?

I think I figured it out and proposed a docs change in #142.


Global .ignore does not followed

I just put a .ignore file on my $HOME directory, it seems the Ag does not follow it, I have to rename it to .agignore to make it work.
Here comes with my ag --version:
ag version 0.33.0

Features:
  +jit +lzma +zlib

Going by Geoff's blog, I think the .ignore feature was only recently introduced so you likely need to install or build the latest version.

As far as I know 0.33.0 should be the first tag to include the .ignore file function
See here: https://github.com/ggreer/the_silver_searcher/compare/0.32.0...0.33.0

Building off of master right now, and running ag -D, it appears that it is looking for .ignore the directories that it is searching, but is only looking for .agignore in the home directory.

Updating here for anyone else coming to this issue, this is still the case.


bot seems to loooove speaking italian

https://github.com/microsoft/vscode/issues/96896
seems to be re-commenting onedit. It should only ever comment once.

Also https://github.com/microsoft/vscode/issues/96668

Closed in d9260c0619a23ae83da8665336ccc7b1db95efa2
See #97027
Verifiers:

make non english document (either in non-latin script for the automatic flow, or manually apply *english-please to an existing non-english latin script issue)
wait for bot to comment
edit issue (labels, title, body, etc.)
bot should not comment anymore


Remove Affinity Group dependency on owner ref

/kind bug
Cloudstack machine spec accepts affinity group in two ways.

Managed way where we can just say pro or anti and CAPC will create an affinity group accordingly for each node group. Ex. for KCP, Etcd, and each worker node group.
Provide an already created list of affinity group IDs

In the managed way, CAPC gets the owner ref of CAPI machine and uses the owner name to identify and generate a unique name per node group. While this would work in most situations, owner ref of CAPI machines is not a solid or strict API spec to identify the node role or node group. For instance, ETCD machines can be owned by etcdamd controller, but when the controller chooses to delete old machines, it simply removes its ownership, at which point, CAPI cluster owns those old etcd machines.
An intermediary API that is responsible to track node roles along with affinity or anti-affinity status should be established from which affinity group names should be generated. This API should derive the affinity details during KCP/Cluster creation time.

Fixed by https://github.com/kubernetes-sigs/cluster-api-provider-cloudstack/pull/286 cc @vignesh-goutham pl review and test, thanks.


No access to our own store item

Dear Developer Support of Microsoft Edge Extensions,
I am the managing director of the OceanHero GmbH which is the owner of oceanhero.today and also the oceanhero extension (https://microsoftedge.microsoft.com/addons/detail/oceanhero-save-our-ocean/ffilhkdaaipeaedlcappdbldhnpddjjn
Unfortunately, we don't have access anymore to the extension. I tried logging in with my normal Microsoft account<EMAIL_ADDRESS>but the extension is not shown in the partner portal (https://partner.microsoft.com/dashboard/v2/home).
Could you help us to identify the right user account and potentially assign the item to my existing microsoft account (marvin.burman@oceanhero.today)?
We would like to publish an update.
Best,
Marvin

Hi @marvinocean, could you please send us an email at<EMAIL_ADDRESS>along with your Extension Name and Store ID? This information will help us assist you further.


UseMysql not work

downloaded the latest version
addons only libk ps2 pac3
database working darkrp connects to it
[libk] addons/libk/lua/libk/server/sv_libk_database.lua:283: attempt to call method 'connect' (a nil value)
  1. ConnectToMySQL - addons/libk/lua/libk/server/sv_libk_database.lua:283
   2. getDatabaseConnection - addons/libk/lua/libk/server/sv_libk_database.lua:356
    3. unknown - addons/libk/lua/libk/server/sv_versioning.lua:16
     4. include - [C]:-1
      5. unknown - addons/libk/lua/autorun/_libk_loader.lua:36

[LibK] MANUAL: Loading file: libk/shared/sh_KPlayer.lua
LibK by Kamshak loaded
[BricksServer SQLLite] bricks_server_currencies table validated!
[PAC3] Loaded total 15 particle systems

[libk] addons/libk/lua/libk/shared/sh_addonloader.lua:151: attempt to call field 'updateAddon' (a nil value)
  1. InitializeAddon - addons/libk/lua/libk/shared/sh_addonloader.lua:151
   2. unknown - addons/ps2/lua/autorun/pointshop2_init.lua:23

[libk] addons/libk/lua/libk/shared/sh_addonloader.lua:151: attempt to call field 'updateAddon' (a nil value)
  1. InitializeAddon - addons/libk/lua/libk/shared/sh_addonloader.lua:151
   2. unknown - addons/ps2/lua/autorun/ps2_kinv_init.lua:1

Trying to create entities too early! (env_physexplosion)

[libk] addons/libk/lua/libk/server/sv_libk_database.lua:283: attempt to call method 'connect' (a nil value)
  1. ConnectToMySQL - addons/libk/lua/libk/server/sv_libk_database.lua:283
   2. getDatabaseConnection - addons/libk/lua/libk/server/sv_libk_database.lua:356
    3. DBInitialize - addons/libk/lua/libk/server/sv_libk_server.lua:38
     4. func - addons/libk/lua/libk/server/sv_libk_server.lua:42
      5. f - addons/libk/lua/libk/shared/1_sh_promises.lua:30
       6. Resolve - addons/libk/lua/libk/shared/1_sh_promises.lua:184
        7. func - addons/libk/lua/libk/server/sv_libk_server.lua:11
         8. unknown - lua/includes/modules/hook.lua:241

Just merged a MysqlOO pull request, please redownload the newest libk, delete fully and upload again. Make sure you install the newest mysqloo from here https://github.com/FredyH/MySQLOO#install-instructions


There are few changes in the following sections:Make an appointment s…

…ection,Contact section,Feedback Section
Description
I have enhanced the visual appeal of the website by arranging the sections, like giving perfect padding and margins which is required.
Related Issues
None

Closes #120

Type of PR

[x] Bug fix

Screenshots / videos (if applicable)
Before

After

Gave a border to the section and a gap between the time slot and date.

The message button was not taking the full size so made it take the full area, and improved the visual appeal.

Made the section come in the center and also made the submit button look good.
Checklist

[X] I have gone through the contributing guide
[X] I have updated my branch and synced it with project main branch before making this PR
[X] I have performed a self-review of my code
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.

Additional context:
[Include any additional information or context that might be helpful for reviewers.]

@Amitesh-Mishra-AM Can you also do for Mobile responsive?

there is conflicting files please resolve

there is conflicting files please resolve

The conflicting Issues have been resolved.

@Amitesh-Mishra-AM Can you also do for Mobile responsive?

Yeah sure, assign me the issue.

@Amitesh-Mishra-AM Assigned and resolve conflicts too

No activity! Any updates

Sorry for the delay in my response. I tried working on the responsiveness of the website, but I faced difficulties due to the unorganized codebase. I have solved my previous issue and resolved the conflict that arose. Please check once more; however, if any conflict arises again, I will be happy to solve it. For now, I will not be able to work on the responsiveness of the page. Again, sorry for the delay in my response and for any inconvenience caused.
Please close this PR if everything is ok.
Thank you.

@Amitesh-Mishra-AM Closing! There are many changes done after this PR was raised. We have recently removed the appointment section on the home page. Thanks! for your effort.


Select a VM name is different in story board and different in plug in

Environment: Windows/IntelliJ14
Select a VM name in plug in is 'Choose a Virtual Machine'.
PFB the screen shot.

Select a VM name in story board is 'Select a Virtual Machine'.
PFB the screen shot.

This issue is fixed and working as expected in the latest build.


WireGuard interface support

This is an idea for integrating WireGuard interfaces into containers.
The use case I have at the moment is App monitoring and hosting multiple services with different intranet hostnames on the same VM/Bare Metal machine without a reverse proxy. But I am sure that this could be a feature many other people can appreciate for private network setups.
Process
The process would probably look like follows:

Setup bridge network if desired
Create WireGuard interface inside the container namspace
Set up routing

if no bridge -> all traffic goes over WireGuard via default route
if bridge -> only specified  Ranges go over WireGuard interface. Default route still over bridge network

set up port-rebinding via ip/nftables if desired

Things to consider

WireGuard endpoints with a hostname need the Bridge network alive if DNS should be resolved (unless the interface is created on the host and then moved inside the container namespace)

Implementation
Have a .yaml file that takes the common WireGuard configuration parameters:
# ? -> optional
peers:
  - name:
    endpoint:
    public_key:
    persistent_keep_alive?:
    allowed_ips:
interface:
  private_key:
  MTU?:
  port:
  force_default_route?:

I can implement this feature if you (the maintainers) think this is desirable functionality.
At the moment I achieve this functionality with https://github.com/b-m-f/wg-pod, but having it directly integrated in Podman would surely be the most elegant way to integrate containers into WireGuard networks.
The biggest problem is probably the decision on whether or not the WireGuard interface should be the default route.
This could be done via a configuration param (simple) or by checking whether a default route to a bridge network exists(smart).

I have thought about this recently. I personally would like to add wireguard support in the not so far future.
My thoughts:

Create the wireguard interface and the host and them move it into the namespace, take a look at https://www.wireguard.com/netns/ If I understand it correctly with that we can only use the wiregurad interface in the container and do not have to worry about other interfaces in the netns to enable connectivity for wireguard.
This will make setup extremely simple.

I don't like to introduce a new yaml config format? Wireguard already has a specific config file format which is used by wg-quick so we should use the same.

Also I would like to not mess with the firewall here to enable port forwarding. Since you should be connected via VPN you should just directly connect to that continuer ip.

As far as default route is considered every network gets a default route at the moment, I think that should be the same for wireguard.  There is an internal option which is used to disable the default route.

It should work the same regardless of other drivers used, the wg driver itself should not check anything about other used networks, so --network <bridge> ---network <wg>  and only --network <wg> should result in the same task for the wireguard implementation.

Lastly, should we implement this directly via netlink API or shell out to the wg tools? I am not sure I would prefer to use it

In any case we should not do anything directly related to cryptography, i.e. key generation.

Create the wireguard interface and the host and them move it into the namespace, take a look at https://www.wireguard.com/netns/ If I understand it correctly with that we can only use the wiregurad interface in the container and do not have to worry about other interfaces in the netns to enable connectivity for wireguard.
This will make setup extremely simple

Yup, this also helps when a hostname is used for the Endpoint as the initial request to initiate the connection resolves it to an IP via the hosts DNS resolver settings.

I don't like to introduce a new yaml config format? Wireguard already has a specific config file format which is used by wg-quick so we should use the same.

Where do you think this .conf file should live? Just pass any path to podman and then have it passed through to this project where it could be read?

Also I would like to not mess with the firewall here to enable port forwarding. Since you should be connected via VPN you should just directly connect to that continuer ip.

How could we deal with sth like -p 8080:80 that is passed through a container that is connected to a VPN exclusively though?

As far as default route is considered every network gets a default route at the moment, I think that should be the same for wireguard. There is an internal option which is used to disable the default route.

It should work the same regardless of other drivers used, the wg driver itself should not check anything about other used networks, so --network  ---network  and only --network  should result in the same task for the wireguard implementation.

So, would you say that --network <bridge> is always the default route and if all traffic should flow through the WG network it would require setting AllowedIPs = <IP_ADDRESS>/0 in the WireGuard config?

Lastly, should we implement this directly via netlink API or shell out to the wg tools? I am not sure I would prefer to use it

I think it might be better to do this via the netlink API. As long as we document that the feature needs the WireGuard module in the Kernel.

In any case we should not do anything directly related to cryptography, i.e. key generation.
Totally agree with you :)

I have thought about this recently. I personally would like to add wireguard support in the not so far future.

Currently I have a lot of free time as I am stuck in german university burocracy and also still looking for a job.
If you have time we could jump in a call and talk about whether or not we can split the work on this somehow?

You can work on it, any help is welcome. I already have enough things to do anyways.
But we should first agree on the design.
I would also like the opinion of the other members @flouthoc @mheon @baude or anybody else who might be interested it this.

Where do you think this .conf file should live? Just pass any path to podman and then have it passed through to this project where it could be read?

Also I would like to not mess with the firewall here to enable port forwarding. Since you should be connected via VPN you should just directly connect to that continuer ip.

Yes I think something like --network <name>:config_path=/some/path for podman then pass it down the stack. This would obviously mean that we need changes in podman and containers/common (where most of the network code is).

How could we deal with sth like -p 8080:80 that is passed through a container that is connected to a VPN exclusively though?

Just ignore it like, this is already done by the macvlan driver. I don't see how we can support it when the peer endpoint is on other system. We cannot route anything directly to the wireguard container interface in this case since the host netns is not part of that subnet.

So, would you say that --network  is always the default route and if all traffic should flow through the WG network it would require setting AllowedIPs = <IP_ADDRESS>/0 in the WireGuard config?

Mhh, I think you are right. Maybe it is best to just setup the route for the AllowedIPs. In this case the user has full control.

either start a design doc (@Luap99 knows where that is) or start a discussion that mimics it ?

I have no real comments except RE: port forwarding - I feel like we should explicitly disable it with VPN'd networks to ensure all traffic goes through the VPN?

either start a design doc (@Luap99 knows where that is) or start a discussion that mimics it ?

Sounds like a good idea.
Is this a Redhat internal thing or can I get access too @Luap99 ?


Unable to connect to headscale

Bug description

Hi,
First of all want to say "Thank you" for such a cool product you build!
We have faced some issue with headscale server multiple times. The issue is that when tailscale client tries to connect to the headscale it gets could not register machine error returned in the browser (we use OIDC with Google provider). What happens on the server level at that time is:

the error ERR Failed to persist/update machine in the database error="database is locked (5) (SQLITE_BUSY)" handler=PollNetMap machine=<NODE_NAME> appears in the log
then the ERR Failed to persist/update machine in the database error="SQL logic error: cannot start a transaction within a transaction (1)" handler=PollNetMap machine=<NODE_NAME> error message just spamming the server log with different machine names in <NODE_NAME> field

I find it hard to say what exactly causing this but can definitely say that two times it happened after our two separate users updated their local macos tailscale clients to the latest version and were unable to connect to server after that. Also after this issue appears all the users who do a reconnect to the server get the same error. So from my understanding headscale tries to write some data to database when user connects but it's unable to do that because of the locked database.
According to the log message header the issue is linked to this function
Environment

OS: Ubuntu 20.04
Headscale version: 0.20.0
Tailscale version: definitely occurred on MacOS 1.38.1
Database: .sqlite file stored on the server

[ ] Headscale is behind a (reverse) proxy
[ ] Headscale runs in a container

To Reproduce

We were trying to reproduce this by downloading the older version of tailscale client (both windows and linux, don't have ability to test on mac), connecting to server and then updating the client to the latest version proposed by tailscale as it is the only way we are aware of that can cause this issue but we didn't have any success reproducing it so from my understanding this is the bug

@anton-livewyer latest stable version is 0.22.3, is this reproducable in that version?

We are seeing this on 0.22.3. Not sure if it's a coincidence, but a lot of our users upgraded from 1.36.x client to 1.38.x today

I'll also mention that this seems to occur after we restart our headscale deployment in kubernetes. I imagine that any brief overlap between pod uptimes may be the cause of db locking

I'll also mention that this seems to occur after we restart our headscale deployment in kubernetes. I imagine that any brief overlap between pod uptimes may be the cause of db locking

Yes, makes a lot of sense. I do not assume this would be fixed in v0.22, but I will make a ticket to make sure database is properly closed on kill in v0.23.
For your current use case switching to postgres might be a viable solution to the locking problem.

We're currently switching to using a Statefulset instead of a Deployment (should've done that in the first place) to address the overlap.
Postgres is a good idea, we'll do that later too

We're currently switching to using a Statefulset instead of a Deployment (should've done that in the first place) to address the overlap.
Postgres is a good idea, we'll do that later too. Thanks @TotoTheDragon

Alright, whrn you have tested a new environment please let us know if anything has changed


minor updates to contributing.md

link to open issues within dep
fix link to pull request help on GitHub

[ci skip]

LGTM


split db_utils.py into db_read_utils (for data retrieval functions) and db_write_utils.py (for data update functions)

@azurebrd Thanks!


Typecript typings

I would love to use this library with TypeScript, but there don't seem to be any typings :(

I started a typings file that I didn't end up using - hopefully it can help someone get started.
// nanomatch.typings.d.ts
declare module 'nanomatch' {

  interface MatchOptions {
    basename?: boolean;
    bash?: boolean;
    cache?: boolean;
    dot?: boolean;
    failglob?: boolean;
    ignore?: string | string[];
    matchBase?: boolean;
    nocase?: boolean;
    nodupes?: boolean;
    nonegate?: boolean;
    noglobstar?: boolean;
    nonull?: boolean;
    nullglob?: boolean;
    slash?: string | (() => string);
    star?: string | (() => string);
    /* https://github.com/jonschlinkert/snapdragon */
    snapdragon?: object; // a snapdragon instance:
    sourcemap?: boolean; // a snapdragon instance:
    unescape?: boolean;
    unixify?: boolean;
  }

  export const isMatch: (string: string, pattern: string, options?: MatchOptions) => boolean;
}


Margin between items

Hello
How can I add margin between items? Currently I add margins for every item in my adapter onCreateView() method but it is not work when I use ChipsLayoutManger.
Thanks

Hi,
I had the same problem but fortunately this issue is indirectly mentioned in Readme:

You may need an RecyclerView.ItemDecoration to work with ChipsLayoutManager for fast setup as showed in sample.
It contained in same package and named SpacingItemDecoration.

So you need to apply SpacingItemDecoration to your RecyclerView.
It could be written more clearly (why and how to use it), but at least it's there.
Its usage is quite straighforward, you can see a sample here: https://github.com/BelooS/ChipsLayoutManager/blob/9b271a816fcef538de3e13307bd8b8a7130295be/sample/src/main/java/com/beloo/chipslayoutmanager/sample/ui/BottomSheetDialogFragment.java#L61

Thanks...this code help for me.

@michalbrz  thank you so much


Unbound always uses DoT for Query Forwarding

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

DoT always enabled when upstream servers are set in Unbound DNS -> Query Forwarding.
Generated config:
# Forward zones over TLS
server:
  tls-cert-bundle: /etc/ssl/cert.pem

forward-zone:
  name: "."
  forward-tls-upstream: yes
  forward-addr: <IP_ADDRESS>@54
  forward-addr: <IP_ADDRESS>@54

Nothing enabled in Unbound DNS -> DNS over TLS therefore it should be:
forward-zone:
  name: "."
  forward-addr: <IP_ADDRESS>@54
  forward-addr: <IP_ADDRESS>@54

To Reproduce
Steps to reproduce the behavior:

Go to 'Unbound DNS -> Query Forwarding'
Add DNS servers without DoT support
Apply settings
DNS resolution doesn't work

Something is fishy, the “should be” includes “forward-tls-upstream: yes” which is likely wrong.

Sorry, you are right. Deleted line.

Can't reproduce either....
cat /usr/local/etc/unbound.opnsense.d/dot.conf
server:
  do-not-query-localhost: no

# Forward zones
forward-zone:
  name: "opnsense.local"
  forward-addr: <IP_ADDRESS>@654
forward-zone:
  name: "opnsense2.local"
  forward-addr: <IP_ADDRESS>@654

There are same servers in Services: Unbound DNS: DNS over TLS as in Services: Unbound DNS: Query Forwarding. When a server enabled in Services: Unbound DNS: Query Forwarding, it's enabled in Services: Unbound DNS: DNS over TLS too. I can't disable DoT without disabling Query Forwarding.

If you specify the same server in both Query Forwarding and DoT intentionally and enable them both, there are going to be two forward zones defined. How is Unbound supposed to know which one to pick? It simply picks the last one, which in our case is always DoT (the secure one). The Query Forwarding zone is going to be ignored. https://docs.opnsense.org/manual/unbound.html#dns-over-tls

I don't specify DoT servers.

Can't reproduce it. Using only query forwarding:
root@OPNsense:/var/unbound/etc # cat dot.conf

# Forward zones
forward-zone:
  name: "."
  forward-addr: <IP_ADDRESS>@53
  forward-addr: <IP_ADDRESS>@53

https://user-images.githubusercontent.com/37266727/164028743-c798adee-4409-4db6-af14-c379d26a7f09.mp4

It's not fixed yet.

Sure, that's why it's marked "not planned".

It's obvious: missing steps to reproduce, no other complaints in the forum/elsewhere.

@fichtner @AdSchellevis @definitio,
This issue does not appear to be fixed. However, I think I may have pinned the issue down, specifically the truthfully god awful matching of forwarding types here.
https://github.com/opnsense/core/blob/0da84ef19febf5cee53c7fca2b35a893816d0793/src/opnsense/mvc/app/controllers/OPNsense/Unbound/Api/SettingsController.php#L113-L125
On certain more privacy-focused browsers which tend to have little tweaks here and there, there is a tendency to hide the Referer header, which when combined with this matching "logic", results in the Query Forwarding tab not being detected as such, and defaulting to DoT.
For reference, this is the expected behavior on a completely vanilla Chromium,

and this is the unexpected behavior on a more privacy focused Firefox.

Perhaps the matching of this should take into account this little... quirk?

@lflare we're always open for suggestions, just open a PR to discuss the change you would like to see.

@lflare we're always open for suggestions, just open a PR to discuss the change you would like to see.

Perhaps instead of reusing the views, there should be a separate view for Query Forwarding? I see that the API documentation makes a stark differentiation of the dot and forward endpoints, but the UI reuses the dot view for the forwarder as well.
Unfortunately, it's been a long while since I have touched any kind of PHP development, so it is unlikely I will be able to contribute anything meaningful in the ways of a PR.


@IdView's error reporting mechanism is unclear, and error messages are sometimes available and sometimes not

For example, in the following entities, there should be error reports about IdView, but in fact there are none, and the program runs normally
interface Connector {
    @Column(name = "creator")
    val creatorId: Int

    val projectId: Int

    @ManyToOne
    @JoinColumn(name = "creator")
    val creator: User

    @ManyToOne
    val project: Project

    @ManyToMany(mappedBy = "connectors")
    val owners: List<User>
}

Try 0.7.69

Try 0.7.70


Log error in GET /payments/complete

Error in Apiary
Log error in GET /payments/complete
App\Http\Controllers\PaymentController - Error querying Square transaction
View on Bugsnag
Stacktrace
app/Http/Controllers/PaymentController.php:496 - App\Http\Controllers\PaymentController::getSquareTransaction
app/Http/Controllers/PaymentController.php:413 - App\Http\Controllers\PaymentController::handleSquareResponse
app/Http/Middleware/CASAuthenticate.php:63 - App\Http\Middleware\CASAuthenticate::handle

View full stacktrace
Created by RoboJackets Admin via Bugsnag

Log noise on OK transaction


Update version to 6.1.0

Release notes:

Elasticsearch 6.1.0 Release Notes
Logstash 6.1.0 Release Notes
Kibana 6.1.0 Release Notes

I have updated the ELK stack using the latest OSS edition

@antoineco Link has been updated, apologies for missing that

Perfect, thank you! :+1:


Cannot import name 'HiFiSingerV1Lightning'

On google colab and locally on my pc, I get this error while trying to use inference. In this situation I tried use HiFiSinger.
Quick note, 2-3 days ago I didn't had this problem.
Traceback (most recent call last):
File "/content/fish-diffusion/tools/hifisinger/inference.py", line 13, in
from fish_diffusion.archs.hifisinger import HiFiSingerV1Lightning, HiFiSingerV2Lightning
ImportError: cannot import name 'HiFiSingerV1Lightning' from 'fish_diffusion.archs.hifisinger' (unknown location)
also this while trying to infer with diffusion:
Traceback (most recent call last):
File "/content/fish-diffusion/tools/diffusion/inference.py", line 23, in
from fish_diffusion.utils.inference import load_checkpoint
File "/content/fish-diffusion/fish_diffusion/utils/inference.py", line 3, in
from fish_diffusion.archs.diffsinger import DiffSingerLightning
ImportError: cannot import name 'DiffSingerLightning' from 'fish_diffusion.archs.diffsinger' (unknown location)

Checkout the release/mmengine-fish-v2 branch and it should work:) We just converted all the configs to yaml and will no longer use mmengine.


[FEATURE] Add and Configure the website field

Description
Currently, if you visit the auth signup page (https://milanhub.org/auth/signup) you will notice that for the club's signup, an input for the website is missing even though we have it in the ClubDetails page.

Please add a field below Club Description labeled as Club website
Make sure that the data is validated through the useValidation hook when the user submits the data. The club website is optional.
Even though the backend for the same is not made yet, make sure it is passed to the backend.
Also apply it to the details page such that the club's website is shown IF the user has a website.

Screenshots

🥦 Browser
Brave
Checklist ✅

[X] I checked and didn't find similar issue
[X] I have read the Contributing Guidelines
[ ] I am willing to work on this issue (blank for no)

@tamalCodes I would like to work on this issue. Could you please assign it to me?

Hi @YuvarajSingh-0 , Thanks for showing interest in this issue as well, however, please complete #1070 first. Sometimes working on multiple issues can be time-consuming and difficult to track.

Hi @tamalCodes, I would like to work on this issue.


Review all asserts

An assert statement is used to catch bugs. We currently abuse them to validate inputs. Those abuses should be changed to raises of specific exceptions.
See comment by @fnothaft to the 1.0 PR.
https://github.com/BD2KGenomics/toil-scripts/pull/208#discussion_r57346105

def _assert(expression,message):
…

A nit, but I would prefer _require to _assert. This would be consistent with Scala's assertion library, and makes assertions (checking internal state) vs. requirements (checking user inputs) more distinctive.

Makes sense.

A require function is being added to the common lib via #268.  Once it is merged I can work on this.


run the tests on python 3.12

Think this one is waiting on lektor 3.4 before it is possible to declare python 3.12 compatibility

AttributeError: module 'pkgutil' has no attribute 'ImpImporter'. Did you mean: 'zipimporter'?

Lektor 3.4 will drop python 3.7

Despite the fact that lektor 3.3.12 is tested under python 3.12 the problem still exists. It doesn't look like the fix will be backported to 3.3.x so we're still waiting on 3.4


feat: added footer and input in replay mode

Added footer and input in replay mode.

Issue #104 - Need to add footer in Replay feature and do not remove message box in case of error or stop generating message

Refs: #104;

/deploy-review

/deploy-review


Upgrade to 0.1.36 for Mattermost support

Hi, can you upgrade to 0.1.36 for Mattermost support?
Thanks

This is complete. Thanks!


gitInitImage and builder_image as parameter to operator-ci pipeline with cosmetics in upstream-community playbooks

Signed-off-by: Martin Vala<EMAIL_ADDRESS>

@Allda Thanks for review. Can it be merged?


Getting Error log is undefined method iis_site resource

Cookbook version
6.2.0
Chef-client version
13.0.118
Platform Details
Windows Server 2012 R2
Scenario:
I am using the iis_site resource to create
Steps to Reproduce:

Add an iis_site that will create "Default Web Site" with bindings for http:80 and https:443
IIS feature is not enabled on server and therefore the appcmd.exe doesn't exist

Expected Result:
iis website is created
Actual Result:
NoMethodError
undefined method `log' for Custom resource iis_site from cookbook iis

nice find, #337 only covers one of the resources when i think this issue is in all of them.  Also it doesn't pass the Signed-off-by: test.  I want to make sure that I don't just write this real quick and not let you get credit.  If you could fix the signed off i'll fix the other resources.


Use only commands/envs in container info

the spec info is written by user
the container info is comes from inspect, which is merged the user config and image config during container creation

Then, we do not need to merge them again.
Signed-off-by: Xu Wang<EMAIL_ADDRESS>

github announce recovered, let's trigger the test again

LGTM


Add old backgrounds

Add back old backgrounds...
So there are still the new themes (animated) but also the old ones with static backgrounds and stuff...

https://github.com/max1mde/fancy-readme-stats/pull/12


Is random Bits transmission possible?

Hi,
I may sound lame from the question I asked but please allow me to give a context. So, I am not from communications background. Just a pure Computer Vision researcher. I have recently come across a problem where I need to transfer random bit string over a network and recover the same with zero loss. I experimented with a linear autoencoder architecture with one linear layer each and of course I could recover the string with almost zero loss. But the problems can't be so simple, right? Now when I tried to add noise to my data after encoding step; my decoder isn't able to recover the string due to obvious non-linearity induced in between the network.
Then I read more and came across communications field and could relate this with my problem statement because that's what I am trying to do -> just transmitting information bits and trying to recover it with ECC embedded.
I came across your work and trying to understand if this is it what I am looking for. But when I checked your dataset (POLAR); I found they're some kind of vector matrices with ones and zeros extending upto a length like [000011111110000], meaning no mixed bits like [01010000111010100110011].
So; please help me understand if such random bit strings are possible to recover and whether I am on the right path?
Thanks a lot for this work!

Hi,
The goal is to remain robust to (channel) noise by adding redundant information (bits).
You can have a look here: https://en.wikipedia.org/wiki/Generator_matrix
So, in your case, you should choose a code (plenty of codes) of the desired length (# of bits) and rate (~# of redundant bits) and train the DDECC (or ECCT) on it.
Regarding the mixing, you can use the standard_form flag in the code creation function to extract the (former) information bits. (You can also perform Gaussian elimination over GF2, but it is more advanced).

Thanks a lot! I'll look into your suggestions.


Password field doesn't mask inputted text

This can be a security risk, as peeping toms can see your full password in plain text.

will be fixed in the next login reform

v 0.3.8 has fixed this


[Feature Request] Connectors for Amazon Bedrock

Hi all,
I work at AWS, and would like to help our customers adopt Moonshot for testing models including those hosted on our Amazon Bedrock API - whether it's first-party models from Amazon, or our third-party partners like Anthropic, Meta, Mistral, etc.
From a quick play, I think it should be straightforward enough by adding a new connector class and some endpoint configuration JSONs.
If you're open to the idea, I can tidy my quick prototype up into a pull request?

@athewsey Hi Alex, Thank you for the initiative! Happy to have the API in moonshot, feel free to proceed with the pull request, our team will follow up accordingly to help with the merge.

^ Hey sorry this took a bit longer than expected - initial draft raised now!


Add some explicit test imports again

Addresses a comment from https://github.com/langium/langium/pull/1001#issuecomment-1487058653
Also fixes a small dev issue, where globals sometimes didn't register correctly in the TypeScript vscode extension.

@msujew shouldn't then
"types": [
     "vitest/globals"
]

be only added to the langium tsconfig.json? Is it really clever to have this mixed bag now?

@kaisalmen I've updated the PR according to our previous discussion about using assert.


Mistral API compatibility

Instead of using OpenAI api I want to use Mistral AI api using the 8x7B model.
The only blocker with this code is the  presence_penalty: 1 parameter in the index.js file, we just need to remove this line
to avoid OpenAI error 422 due to this Extra inputs not supported and permitted from Mistral API
I created a fork with this fix here
Now I'm using Mistral instead of OpenAI than is a lot cheaper

You can have it from the marketplace
https://github.com/marketplace/actions/ai-based-pr-reviewer-summarizer-with-mistral-8x7b-chat-capabilities
You just need to add this github action
name: Code Review

permissions:
  contents: read
  pull-requests: write

on:
  pull_request:
  pull_request_review_comment:
    types: [created]

concurrency:
  group: ${{ github.repository }}-${{ github.event.number || github.head_ref ||
    github.sha }}-${{ github.workflow }}-${{ github.event_name ==
    'pull_request_review_comment' && 'pr_comment' || 'pr' }}
  cancel-in-progress: ${{ github.event_name != 'pull_request_review_comment' }}

jobs:
  review:
    runs-on: ubuntu-latest
    steps:
      - name: AI-based PR Reviewer & Summarizer with Mistral 8x7B Chat Capabilities
        uses: Onigam/ai-mistral-pr-reviewer@mistralai-support-marketplace
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
          OPENAI_API_KEY: ${{ secrets.MISTRAL_API_KEY }}
        with:
          debug: false
          review_simple_changes: false
          review_comment_lgtm: false
          openai_base_url: "https://api.mistral.ai/v1"
          openai_light_model: "mistral-tiny"
          openai_heavy_model: "mistral-small"


Forgot password

What's the problem?

forgot my master password

How often you face this problem ?

Everytime

What is the expected output? What do you see instead?

What version of the product are you using? On what version of Windows?

windows 7, Lock a folder 3.10.3

Any previous version produced the same problem ?

What is the language of your operating system and keyboard ?

Polish

Please provide any additional information below and attach debug.txt file
generated in C:\Program Files\LocK-A-FoLdeR or C:\Program Files
(x86)\LocK-A-FoLdeR(64bit windows).

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 4 Feb 2014 at 9:47
Attachments:

debug.log

is this issue solved?


SSL Isue

What steps will reproduce the problem?
searching any tag at  any sankaku site ( idol.sankakucomplex, chan.sankakucomplex)
What is the expected behavior? What do you get instead?
cannot find any results , no previews, no pages
shows
[20:53:01.815] Loading page http://chan.sankakucomplex.com/post/index?login={edited}&password_hash={edited}&appkey={edited}&limit=20&tags=fairy&page=1
[20:53:07.862] Loading error: Error creating SSL context ()
[20:53:07.863] Loading using Html failed. Retry using Json.
[20:53:13.865] Loading error: Error creating SSL context ()
[20:53:13.866] Loading using Json failed. Retry using Xml.
[20:53:19.856] Loading error: Error creating SSL context ()
[20:53:19.856] No valid source of the site returned result.
in log file
What version of the program are you using? On what operating system?
5.1.1
Windows 10
Please provide any additional information below
It also shows on all sankaku headers following
Name: User-Agent
Value: SCChannelApp/2.0.1(Android;black)
the issue might have to do with the header
attachements:
settingsgrabber.txt
settingssankaku.txt
model.txt
Tried to make grabber work on sankakucomplex for someone who is running windows 10 as operation system.
Microsoft C Redistrutables are installed
Goldendict is installed

Does replacing the OpenSSL dlls (libeay32.dll, libssl32.dll, ssleay32.dll) by the latest version from here fix the issue?

no it did not, i tried both, win 64 and win 32

It seems installing MSVC redist 2013 might fix the issue.
Maybe you can give it a try?


Screenshot in README no longer available

The embedded screenshot in the README.md file is no longer available. It seems to be deleted from Flickr.

Oops, thanks for report. I'll fix it :)


Expose git branch description on UI

PR for #197752
Add branch description in branch selection

Add edit command to Source Control menu > Branch > Edit Branch Description

Operations are performed through the property in git local config. Suggestions for improvement are welcome.

@gsdstr please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree

Could this effort continue? We would like to use this feature. Thanks!

Could this effort continue? We would like to use this feature. Thanks!

This solution is not optimal. I think to be accepted into the main branch, support for the "description" attribute in "git for-each-ref" command is required.

Is there anyone else that could help out here? It'd be a very useful feature to have

Curious if anyone has been able to look at this. Just learned of branch descriptions and they're useful in my workflow and could love to see them available in the VS Code UI

W dniu 27 grudnia 2024 00:00:49 CET, bverkron @.***> napisał:

Curious if anyone has been able to look at this. Just learned of branch descriptions and they're useful in my workflow and could love to see them available in the VS Code UI
--
Reply to this email directly or view it on GitHub:
https://github.com/microsoft/vscode/pull/201357#issuecomment-2563150754
You are receiving this because you are subscribed to this thread.
Message ID: @.***>


num_relation mismatches in message_and_aggregate()

There was another issue on this that was closed but there wasn't really a resolution. The problem occurs when I use a custom dataset. The dataset loads correctly:
import torch

from torchdrug import datasets

dataset = datasets.flav("~/molecule-datasets/", kekulize=True,

                            atom_feature="symbol")

18:01:53   Downloading https://raw.githubusercontent.com/gdeol4/torchdrug/master/flav.csv to /root/molecule-datasets/flav.csv

Loading /root/molecule-datasets/flav.csv: 4807it [00:00, 70415.81it/s]
Constructing molecules from SMILES: 100%|██████████| 4806/4806 [00:10<00:00, 473.69it/s]

However, when attempting to train a model, I encounter the assertion error:
solver.train(num_epoch=1)
solver.save("graphaf_data_1epoch.pkl")

File /notebooks/torchdrug/torchdrug/layers/conv.py:416, in RelationalGraphConv.message_and_aggregate(self, graph, input)
    415 def message_and_aggregate(self, graph, input):
--> 416     assert graph.num_relation == self.num_relation
    418     node_in, node_out, relation = graph.edge_list.t()
    419     node_out = node_out * self.num_relation + relation

AssertionError:

There does seem to be a mismatch here:
dataset.num_bond_type

2

model.layers[0].num_relation

2

dataset[0]["graph"].num_relation

tensor(3)

My attempt at fixing it:
for data in dataset:
  data['graph'].num_relation = torch.tensor(2)

But the value remains unchanged

Hi! I think this problem is caused by unused relations. For kekulized molecules, there are 3 bond types allowed: single, double and triple, but your dataset only contains 2 of them. This is a relatively rare case. A quick fix is to specify num_relation=dataset[0]["graph"].num_relation when creating your model.
We may update the semantics of num_bond_type to be consistent with num_relation, but we need to verify the compatibility of this change.


F-Droid submission

How about submitting the project to F-Droid?
Additional information

Inclusion Policy
How to submit

Thank you for your suggestion. I'm actually working on getting it submitted right now. It should be on there soon.

Nice, thank you! :-)

@parmstrong842 maybe you missed the ping (or the ping missed you): the MR waits for your feedback. Can you please chime in? Thanks!

ping @parmstrong842

Looks like @parmstrong842 went AWOL. Which most likely means the app is also unmaintained now. So we probably cannot include it with F-Droid – and I'll abandon my merge request soon if I cannot get it done…

I'm closing here too.


This will never end

OPS Build status updates of commit 61078b6:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/enterprise-graph-docs-pr, working directory: W:\x1df-s, target branch: master, target branch commit: d08df34b00ffb4b6ca3c9741492656cbcddd7009, source branch: 3638ce60ebef810876cd6fbb8bd88ec9/master, source branch commit: .

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

enterprise-graph/api-overview.md
78
44
5
link
:white_check_mark:

enterprise-graph/conflation-concepts.md
76
39
34
link
:white_check_mark:

enterprise-graph/conflation-model-tutorial.md
82
56
19
link
:white_check_mark:

enterprise-graph/conflation-validation-tutorial.md
100
100
0
link
:white_check_mark:

enterprise-graph/deleting-your-ontology.md
92
73
3
link
:white_check_mark:

enterprise-graph/enterprise-graph-use-cases.md
75
51
19
link
:white_check_mark:

enterprise-graph/graph-queries-tutorial.md
80
59
9
link
:white_check_mark:

enterprise-graph/ingestion-tutorial.md
77
45
12
link
:white_check_mark:

enterprise-graph/input-data-concepts.md
80
41
7
link
:white_check_mark:

enterprise-graph/ontology-concepts.md
77
45
25
link
:white_check_mark:

enterprise-graph/quickstart.md
73
47
38
link
:white_check_mark:

enterprise-graph/schema-map-concepts.md
75
48
30
link
:white_check_mark:

enterprise-graph/schema-map-tutorial.md
79
36
7
link
:white_check_mark:

enterprise-graph/source-schema-concepts.md
78
40
23
link
:white_check_mark:

enterprise-graph/source-schema-tutorial.md
78
53
6
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

OPS Build status updates of commit 0612593:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/enterprise-graph-docs-pr, working directory: W:\i4ho-s, target branch: master, target branch commit: d08df34b00ffb4b6ca3c9741492656cbcddd7009, source branch: 3638ce60ebef810876cd6fbb8bd88ec9/master, source branch commit: .

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

enterprise-graph/api-overview.md
78
44
5
link
:white_check_mark:

enterprise-graph/conflation-concepts.md
76
39
34
link
:white_check_mark:

enterprise-graph/conflation-model-tutorial.md
82
56
19
link
:white_check_mark:

enterprise-graph/conflation-validation-tutorial.md
100
100
0
link
:white_check_mark:

enterprise-graph/deleting-your-ontology.md
92
73
3
link
:white_check_mark:

enterprise-graph/enterprise-graph-use-cases.md
75
51
19
link
:white_check_mark:

enterprise-graph/graph-queries-tutorial.md
80
59
9
link
:white_check_mark:

enterprise-graph/ingestion-tutorial.md
77
45
12
link
:white_check_mark:

enterprise-graph/input-data-concepts.md
80
41
7
link
:white_check_mark:

enterprise-graph/ontology-concepts.md
77
45
25
link
:white_check_mark:

enterprise-graph/quickstart.md
73
47
38
link
:white_check_mark:

enterprise-graph/schema-map-concepts.md
75
48
30
link
:white_check_mark:

enterprise-graph/schema-map-tutorial.md
79
36
7
link
:white_check_mark:

enterprise-graph/source-schema-concepts.md
78
40
23
link
:white_check_mark:

enterprise-graph/source-schema-tutorial.md
78
53
6
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues


Remove jQuery Dependency

Is it a possibility to remove jQuery in shell.js and pathhandler.js? It seems a bit overkill to me to add such a heavy dependency.

Pathhandler.js doesn't actually use it, so removing it there is no problem. Shell.js on the other hand is the optional UI layer and requires some cross-browser compatible DOM manipulation. Since it already makes DOM assumptions that are likely at odds with a lot of frameworks, I'd rather re-invent that wheel and let people opt-in to use shell.js with the jQuery or adapt their own shell based no their favorite DOM abstraction.
I'm not opposed to a pull request that rips it out as long as the DOM code does cover the usual suspects of browsers.


Clarify what console output is from which robot

Webots handily prefixed the output from each controller with the name of that controller. However since we're using the same controller for all robots it's hard to tell which robot is outputting what.
It would be handy if there was a way to tell them apart.

Note: since we raised this issue we have added log files which have the zone id in their name, so this is somewhat mitigated.


Some kind of support for directives

Right now, Ninja will see this as two objects:
#if something
codeunit 50001....
#else
codeunit 7013010..
#endif

Also, object ID assignment will occur from the first available range.
Some kind of support for this is needed.
Any ideas how this should behave, functionally?

Both ID's are consumed, are they not?

This issue is resolved in AL Object ID Ninja v2.3.1 and @vjeko.com/al-parser-ninja v1.0.8