v0.13 readStream don't fetch all rows

Hello,
I have an issue with the read stream function.
I have used the cbt util to count the number of rows contained by a table. The result is 888,902 rows.
Now when I am trying to get the same result with node (google-cloud/bigtable@0.13.1), I get a different result, which is 629,444 rows.
I am using the following code:
const Bigtable = require('@google-cloud/bigtable');

const bigtable = Bigtable({
  projectId: '**'
});

let count = 0;
const instance = bigtable.instance('**');

instance.table('table').createReadStream()
  .on('data', () => {
    count++;
  })
  .on('end', () => console.log('end, count:', count));

After struggling a bit I decided to try an older version (10.2) for a result of 889,240 which is the same as the cbt utils (with some rows added during the process) .
I tried with several version, getting the below results :
10.2: 889,240
11.1: 889,240
12.1: 889,240
13.0: 604,802
13.1: 629,444
We can see that from the version 0.13 the number of rows drop significantly.

Hi @Turniii thanks for reporting this bug.
I'm trying to reproduce this bug but I can't seem to get the same result you got. Below is the file I'm using. Let me know what I can change to get the same results you're experiencing, Thanks!
/* eslint-disable no-console, no-debugger */

const bigtable = require('@google-cloud/bigtable');

const INSTANCE_ID = 'mkolodny-nodejs-repros';
const TABLE_NAME = 'github-issue-148';
const COLUMN_FAMILY_NAME = 'cf1';
const TOTAL_ROWS = 1000000;

const forceRegenerate = false;

(async () => {
  try {
    const bigtableClient = bigtable();
    const instance = bigtableClient.instance(INSTANCE_ID);
    const table = instance.table(TABLE_NAME);

    // Ensure table exists.
    const [tableExists] = await table.exists();
    console.log(`table ${TABLE_NAME} exists: ${tableExists}`);

    if (forceRegenerate && tableExists) {
      console.log(`force deleting table ${TABLE_NAME}`);
      await table.delete();
    }

    if (!tableExists) {
      console.log(`Creating table ${TABLE_NAME}`);
      const options = {
        families: [COLUMN_FAMILY_NAME],
      };
      await table.create(options);

      // Insert data.
      console.log('inserting rows');
      const rowsToInsert = [];
      let i = 0;
      while (i < TOTAL_ROWS) {
        for (let j = 0; j < 10000; j++, i++) {
          rowsToInsert.push({
            key: `row-${i}`,
            data: {
              [COLUMN_FAMILY_NAME]: {
                column: `cell value for #${i}`,
              },
            },
          });
        }
        await table.insert(rowsToInsert);
        console.log(`inserted up to ${i}`);
        rowsToInsert.length = 0;
      }
    }

    let count = 0;
    table
      .createReadStream()
      .on('data', () => {
        count++;
      })
      .on('end', () => console.log(`end, count: ${count}`));

    console.log('done');
  } catch (error) {
    console.error('Something went wrong:', error);
  }
})();

I always get end, count: 1000000

@Turniii: we cannot reproduce this issue.  Is there anything more information that might help us debug this?

@kolodny: let's review the retry logic one more time.

I don't think that there's anything active that we can do without a repro.  We tried a whole bunch of ways to reproduce this, but we couldn't.  @Turniii, I'm going to have to close this until we have a repro.


Failing plugin stylelint-prettier

See: https://github.com/stylelint/stylelint-ecosystem-tester/actions/runs/11714613766/job/32629592702
Both latest and next are failing, so this is unlikely something we changed.
Possibly a change in prettier?
If someone has time and is willing to take a look?

https://github.com/prettier/stylelint-prettier/pull/367/files#diff-64ff6bad37ac810ea9c8b48aeba0a89f98c06dee1135c06ca667292449288c5aR117

@jeddy3 can you reopen?
If it's not our fault, it should remain open until an issue has been created on their side.

Node v22 got promoted to the LTS version a few weeks back.
https://github.com/stylelint/stylelint-ecosystem-tester/actions/runs/11675534073/job/32510262054 was the first stylelint-prettier ecosystem test run to start failing and also the first to use node 22.
In node 22, Prettier v3.2.5 and below raise a node deprecation warning. This was fixed in Prettier v3.3.0.
stylelint-prettier tested against Prettier v3.0.0 as the minimum prettier version supported.
The presence of the deprecation warning threw off our e2e tests as running stylelint --json no longer produced valid json in stderr - as the deprecation warning was also included.
In https://github.com/prettier/stylelint-prettier/pull/377 I've added node v22 to stylelint-prettier's CI config and updated the version of prettier that we use for tests to v3.3.0.
The ecosystem tests should pass now.

@BPScott Thanks for telling us! The tests have been recovered ✅

latest: https://github.com/stylelint/stylelint-ecosystem-tester/actions/runs/12208545897
next: https://github.com/stylelint/stylelint-ecosystem-tester/actions/runs/12208628586


Seems to be a bug in the source code, someone please help me.

What version of Go are you using (go version)?

$ go version
go version go1.13.3 windows/amd64

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
Windows
go env Output
$ go env
set GO111MODULE=
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\ma\AppData\Local\go-build
set GOENV=C:\Users\ma\AppData\Roaming\go\env
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=F:\wp\go
set GOPRIVATE=
set GOPROXY=https://proxy.golang.org,direct
set GOROOT=C:\Users\ma\soft\Go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=C:\Users\ma\soft\Go\pkg\tool\windows_amd64
set GCCGO=gccgo
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0 -fdebug-prefix-map=C:\Users\ma\AppData\Local\Temp\go-build601014506=/tmp/go-build -gno-record-gcc-switches

What did you do?
newDbName = "C:\\Users\\ma\\Desktop\\sqlite3\\malformed\\root\\test.db"
path = "C:\\Users\\ma\\Desktop\\sqlite3\\malformed\\root"
dmt := fmt.Sprintf("/c %s&cd %s&sqlite3.exe %s \".read tmp.sql\"", path[0:2], path, newDbName)
cmd := exec.Command("cmd.exe", dmt)
out1, err := cmd.CombinedOutput()
if err != nil{
	fmt.Println(err)
}

What did you expect to see?
make a new db file,
C:\Users\ma\Desktop\sqlite3\malformed\root\test.db
What did you see instead?
Error: unrecognized token: "".read"

For questions about Go, see https://golang.org/wiki/Questions.

I can't open that website

If you're in China behind the Chinese government's firewall, you'll have to search for a Chinese Go forum.
That wiki page says:
Asking Questions
Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.

For asking questions, see:

Stack Overflow with questions tagged "go"

The Go Forum, a web-based forum

Gophers Slack, use the invite app for access. The #general channel is a good starting point.

Go Community on Hashnode with questions and posts tagged with "go"

The golang-nuts mailing list

IRC channel #go-nuts on Freenode

If we closed your issue as a question with a link to this wiki, we apologize. Please ask the question on one of the above forums.


Expo or something like that.

This will be so useful for indie developers that dont have mac and only have windows and want to develop a ios app and test it on a physical device.

Hi @Denoic
Can you please elaborate this? It is not clear what are you proposing.
Thank you

oh sorry, so what i'm trying to say is that React Native has Expo and you can test your app on a real device. Some people don't have Mac and can't test it. I want to make a mobile app but I can't test it on a physical ios device.

And my english is not that good. After all I'm still a 15 year old kid.

Hi @Denoic
I don't think this is something the Flutter team would focus on. I would suggest you to look for existing alternatives to build without a Mac. Here is an article about that https://medium.com/flutter-community/developing-and-debugging-flutter-apps-for-ios-without-a-mac-8d362a8ec667.
Here is a great thread about that on Stack Overflow: https://stackoverflow.com/questions/47006906/developing-for-ios-device-in-windows-environment-with-flutter


PostgreSQL 9.6.12 Error execute plan

System information:

Operating system (distribution) and version: Windows 10 1803
DBeaver version: <IP_ADDRESS>903251040
Java version : 8 (jre-8u201-windows-x64)
Additional extensions

Connection specification:

Database name and version: postgresql 9.6.12

Driver name
org.postgresql:postgresql:RELEASE 42.2.5
net.postgis:postgis-jdbc:RELEASE 2.2.1

Do you use tunnels or proxies (SSH, SOCKS, etc)?
No

Describe the problem you're observing:
I want to see the query execution plan. But the error constantly flies.
The query seems to be clipped, so the execution plan does not work. Maybe somehow you can increase the number of characters to fulfill the request?
Steps to reproduce, if exist:
Connect to the database, see the session. Try to see the execution plan for the request.
Include any warning/errors/backtraces from the logs
<!-- Please, find the short guide how to find logs here: https://github.com/dbeaver/dbeaver/wiki/Log-files-

My log file:
dbeaver-debug.log

verified


Feature/geode 904

For GEODE-904, this:

Adds the additional bundle listings to the LICENSES.
Reformats the LICENSE files around the embedded licenses.

This looks awesome!!  Only change I would make is to add this to the CDDL license for the javax* dependencies:
   NOTICE PURSUANT TO SECTION 9 OF THE COMMON DEVELOPMENT AND DISTRIBUTION LICENSE (CDDL)

   The code released under the CDDL shall be governed by the laws of the State of California (excluding conflict-of-law provisions). Any litigation relating to this License shall be subject to the jurisdiction of the Federal Courts of the Northern District of California and the state courts of the State of California, with venue lying in Santa Clara County, California.

+1


add max-width option to DESCRIBE

Another proposal, this one will wrap the description text if the total line length is longer than max-width. Also fixes the padding for lines after the first line of the description being one character shorter than the first line's padding. Like my other pull request, I'll wait to update the tests until I know this is something you're actually interested in.

I added you as a collaborator so you can update add-text-padding to use your updated code. I believe we want (padding-next-lines (max 0 (1- padding))) to be (padding-next-lines (max 0 padding)) though or else each line after the first will be indented one less than the first line. Actually, looking at it again, I don't think we need a separate pad and pad-next-lines for anything, since I assume we'll always want all the lines aligned.

I added you as a collaborator so you can update add-text-padding to use your updated code. I believe we want (padding-next-lines (max 0 (1- padding))) to be (padding-next-lines (max 0 padding)) though or else each line after the first will be indented one less than the first line. Actually, looking at it again, I don't think we need a separate pad and pad-next-lines for anything, since I assume we'll always want all the lines aligned.


How to work fine with vuetify

I want to use tailwindcss + vuetify in nuxt, but the nuxt-tailwindcss will remove some vuetify class when build.
Can you help me?

I add prefix tw- for tailwindcss and change the nuxt-purgecss config, then it works for me. thank you

Hi.
purgeCSS: {
whitelistPatterns: [/^v-/, /^theme-/, /^application--/],
whitelistPatternsChildren: [/^v-/, /^theme-/, /^application--/],
whitelist: ['spacer', 'primary', 'secondary', 'accent', 'error', 'warning', 'info', 'success']
},
It's not working when build production

npm run build
npm run start
css will conflict
Could you help me?
@EryouHao

Hi @EryouHao
Could you help me?
Thanks

hi


Correct typo in options.skip deprecation error

The error message when the plugin is passed options.skip was
telling users to "use the main Rollup externals option instead".
Doing this makes rollup throw an Unexpected key 'externals' found
error, as the option required by rollup is external.

Dupe of #99


remove use of _get_yaml_content

This PR removes use of the _get_yaml_content argument passed to asdf.open. It is replaced by using asdf.generic_io.GenericFile.reader_until to read the file bytes until the yaml end-of-document marker.
The previous code did not read the asdf-standard nor the file version from the file using the call to asdf.open where _get_yaml_content was passed and instead opened the file a second time to read these values. The updated code does not require this second file opening and reads these versions with the same code block using reader_until.
I don't entirely trust the CI here as testing these changes requires some downstream library to use the asdf directive. I am opening this for review but will check and update this PR once I find and manually build documentation for a project that uses the updated directive.

The asdf documentation here:
https://asdf.readthedocs.io/en/latest/asdf/overview.html#hello-world
rendered with this PR looks identical:


i18n: add missing Arabic translation file

Translated all keys in the following language JSON files to Arabic: Ghost, Portal, Search, Signup-	Form, and Comments.
Ensured consistency and accuracy for improved user experience
Verified language integration and formatting for consistency

This commit provides the Arabic translation for all relevant UI components in Ghost CMS, filling a previously missing language option.
Got some code for us? Awesome 🎊!
Please include a description of your change & check your PR against this list, thanks!

[ ] There's a clear use-case for this code change, explained below
[ ] Commit message has a short title & references relevant issues
[ ] The build will pass (run yarn test:all and yarn lint)

We appreciate your contribution!

Hello, and thank you for this submission.  Although your post says you're adding a missing file, these appear to all be changes to existing translations.  @mostafty and @almrx and @ksaart , I would be grateful if you would review these translations and reach consensus on what the best wording is.

Let's see if @KHAWARIZMIX responds.  If K's unresponsive, you could certainly put in a new PR and this one could be closed without merging.  But perhaps K will respond (and we should give them a couple days as a courtesy) and you can collaborate. :)

Hello, thank you for your kind response!
To be honest, I had already pushed my translation to the repository a few days ago, as I noticed there wasn’t a complete Arabic translation available at the time. However, since it was my first time contributing, I forgot to create a pull request until today.
I deeply appreciate the efforts of all contributors, especially the Arabic translators. I didn’t attempt to merge my translation because I thought it was better than anyone else’s work, but simply because I wasn’t aware that others were already contributing.
I’m not here to compete or challenge others. My goal is to offer my help to the community that I admire and want to support.

@KHAWARIZMIX , thanks for the reply and for explaining what happened!  If you and the other Arabic speakers can work together to agree on a 'best' set of translations, that'd be great.  I don't have anything to add here. :)

Hello, I hope the conflicts have been fixed as expected!


EDU-205 Add button to create room

Closes https://educandu.atlassian.net/browse/EDU-205

Declining so that I call the existing controller endpoint for adding the room and the redirect.

When I create a new room I get:
TypeError
urls_default.getRoomDetailsUrl is not a function
This function doesn't seem to exist in the urls

We could also consider using the useReloadPersistedWindow on this page, otherwise a browser back will display an incomplete list of pages if you have a new room created and click back.

We could also consider using the useReloadPersistedWindow on this page, otherwise a browser back will display an incomplete list of pages if you have a new room created and click back.

This is the behavior on all our pages, as this is not a SPA 😬

When I create a new room I get: TypeError urls_default.getRoomDetailsUrl is not a function This function doesn't seem to exist in the urls

Interesting, readded it, thanks!

We could also consider using the useReloadPersistedWindow on this page, otherwise a browser back will display an incomplete list of pages if you have a new room created and click back.

This is the behavior on all our pages, as this is not a SPA 😬

I know, that's why I said we can consider it like we did on other pages where we thought it makes sense. I'm not insisting on this though, it was just confusing while testing.

We could also consider using the useReloadPersistedWindow on this page, otherwise a browser back will display an incomplete list of pages if you have a new room created and click back.

This is the behavior on all our pages, as this is not a SPA 😬

I know, that's why I said we can consider it like we did on other pages where we thought it makes sense. I'm not insisting on this though, it was just confusing while testing.

Got it, I see we use that just on import pages now. I'll made a ticket, cause maybe we should do it consistently on all pages listing content that needs re-fetching, and place this logic in a common place maybe.
https://educandu.atlassian.net/browse/EDU-216


feat(control_launch): add nearest search param

PR Type

New Feature

Related Links
issue
https://github.com/autowarefoundation/autoware.universe/issues/1581

Description
added nearest search param in control_launch

Review Procedure

Remarks

Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[x] Code follows coding guidelines
[x] Assign PR to reviewer

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code follows coding guidelines
[ ] (Optional) Unit tests have been written for new behavior
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets
[ ] Write release notes

CI Checks

Build and test for PR: Required to pass before the merge.
Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

@maxime-clem Could you give me LGTM


Suggestion: use Uiua386 to render glyphs

If you aren't already aware, Uiua386 is sort-of the inhouse font used for Uiua, although I don't actually know if it's used anywhere. It exists, though, so using it might be a good idea.

There are probably some benefits to using regular old Deja Vu Sans Mono over this, but in my opinion this looks a lot more consistent and more visually appealing -- although, it's likely it'll become obsoleted frequently due to how much the language is changing at this stage. Additionally, if the user's editor font hasn't been set to this already it might cause some dissonance between the editor font and the keypad font. Regardless, I think it's worth evaluating if this is something that would be a good addition.

As long as it is served as a webfont from a consistent location this should work... and i think CSS can define Deja as the fallback font anyway which is cool. :)

@thekifake I tried it today, but I changed it back. It improves some glyphs but i really don't like the circular glyphs like group, where, partition etc. The Uiua website still uses Deja. Is the Uiua386 font still under development?


Support installation of multiple packages at once

Description of the new feature / enhancement
Hi,
I'm sure someone already suggested it, but I was unable to find the issue.
I propose the idea of installing multiple packages with one command. Like chocolatay, apt-get, etc. support it.
Installing each packages seperatly is quite tedious...
Proposed technical implementation details
winget install Google.Chrome 7zip.7zip

Duplicate of https://github.com/microsoft/winget-cli/issues/219

Thanks for letting me know. I thought that it must exist, but was not able to find ist.


Unable to download ESPN+, told to report

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I'm running yt-dlp version 2023.10.13 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
United Stated
Provide a description that is worded well enough to be understood
I am trying to download a stream and I keep getting an error
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
yt-dlp.exe --ap-mso DTV --ap-username <username> --ap-password <password> "https://www.espn.com/watch/player/_/id/33a21881-e2c2-4624-b160-a7a096c34715" --verbose
[debug] Command-line config: ['--ap-mso', 'DTV', '--ap-username', 'PRIVATE', '--ap-password', 'PRIVATE', 'https://www.espn.com/watch/player/_/id/33a21881-e2c2-4624-b160-a7a096c34715', '--verbose']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b634ba742] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.22621-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: none
[debug] Optional libraries: Cryptodome-3.19.0, brotli-1.1.0, certifi-2023.07.22, mutagen-1.47.0, sqlite3-3.35.5, websockets-11.0.3
[debug] Proxy map: {}
[debug] Loaded 1890 extractors
[WatchESPN] Extracting URL: https://www.espn.com/watch/player/_/id/33a21881-e2c2-4624-b160-a7a096c34715
[WatchESPN] 33a21881-e2c2-4624-b160-a7a096c34715: Downloading JSON metadata
[WatchESPN] 33a21881-e2c2-4624-b160-a7a096c34715: Downloading Provider Redirect Page
[WatchESPN] 33a21881-e2c2-4624-b160-a7a096c34715: Downloading Provider Login Page
ERROR: [WatchESPN] 33a21881-e2c2-4624-b160-a7a096c34715: Unable to extract post url; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
  File "yt_dlp\extractor\common.py", line 715, in extract
  File "yt_dlp\extractor\espn.py", line 404, in _real_extract
  File "yt_dlp\extractor\adobepass.py", line 1715, in _extract_mvpd_auth
  File "yt_dlp\extractor\adobepass.py", line 1395, in post_form
  File "yt_dlp\extractor\common.py", line 1299, in _html_search_regex
  File "yt_dlp\extractor\common.py", line 1263, in _search_regex

Looks like a problem with adobepass / the MSO sign-in rather than the ESPN+ extractor itself. Closing as duplicate of #2188


chore: Go releaser workflow

Description
This PR fixes #470
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@mudler  comments addressed.

@nolanjacobson the changelog settings don't seem to be valid at all ? https://goreleaser.com/customization/changelog/?h=changelog#changelog

Blocked by @j2d3  - John will merge once its ready.


No validation of leaf certificate

Hey, I believe that your code is not validating the first certificate in the chain and therefore anyone can send a fake server-to-server notification and get validated. See more details in the comment I left in stackoverflow.
Happy to provide more info if needed!

Yes, you are right. To ensure the integrity of the certificate chain, it is necessary to verify the leaf certificate. I overlooked this point at the time. I could fix it later, or welcome to submit your PR.

@richzw okay yeah, leave it for me I will PR later. Always nice to contribute to opensource!


Add ddbeck to steering committee

Taking the place of Hermina on behalf of Mozilla. Thanks Hermina!

Super! @Elchi3 you're in good hands:)


Badges show on the origin screen in a randomised order

I'm unsure how exactly they show incorrectly but I'll take a look when I'm next on my pc.

Fixed by making the badges use an LinkedMultimap instead of a HashMultimap, as the latter does not properly support ordering.


duplicate symbol _kSwizzlerAssociatedObjectKey

[REQUIRED] Step 1: Describe your environment

Xcode version: 12.4
Firebase SDK version: 7.7.0
Installation method: CocoaPods
Firebase Component: Google Utilities

[REQUIRED] Step 2: Describe the problem
duplicate symbol '_kSwizzlerAssociatedObjectKey'
Some other library uses this name too. Since it is defined in class GULSwizzledObject, a GUL prefix could be added and make it become GULSwizzlerAssociatedObjectKey.

@luna-moonfang thanks for the report.
We'll take a look at renaming the variable. Just out of curiosity, can you please share which framework also declares that symbol? Thanks!


fix(fetching_hash_value): exception on expired entry

Fetching a hash value throws an exception and fails to delete once the value is expired because the prefix isn't being passed to delete_hash_entry.
Wrong number of arguments (given 1, expected 2)

Hello @wes4m , thank you for your PR, please, update your branch with origin master to make actions green

@wes4m Thank you for contributing!


DBZ-7040 Handle extra semicolon for create table

https://issues.redhat.com/browse/DBZ-7040

@ani-sha Would not this be a problem for other statements too, not only CREATE TABLE?

@jpechane I see the ; added for various other statements too like create_tablespace and create_context. Should we add for all others too?

I think we need to take a different approach to that problem. What if there will be 2 semicolons?
Is it the same problem for MySQL? If not, why?
As the robust solution probably the best approach would not be solving at the grammar level but at Debezium code level. I can envision that before we would pass the statement to parser we would

strip all trailing whitespaces
remove the semicolon if it exists
repeat until there is no semicolon at the end
pass the statement to the parser (and add 1 semicolon back if needed)

@jpechane That's a good idea, closing this PR.


chore: index.ts exports should be in alphabetical order

We want all the exported rules in lib/rules/index.ts to be listed in alphabetical order. The task is to figure out how to configure prettier or to use another plugin to show a lint warning if the exports are out of order.

sort-imports rule of eslint
https://eslint.org/docs/latest/rules/sort-imports

I will take this up


Replace ProjectTaskCurrent by ProjectTask in example

Try to make the differences between the similar ALTER TABLE commands easier to see as mentioned in #1270
Small formatting changes

@TheConstructor : Thanks for your contribution! The author, @CarlRabeler, has been notified to review your proposed change.


Multimap #1131

Current coverage is 96.04%

Merging #1240 into master will decrease coverage by -0.06% as of 0aeb835

@@            master   #1240   diff @@
======================================
  Files           78      83     +5
  Stmts         9170    9524   +354
  Branches      1717    1763    +46
  Methods          0       0
======================================
+ Hit           8813    9147   +334
- Partial         92      97     +5
- Missed         265     280    +15

Review entire Coverage Diff as of 0aeb835

Powered by Codecov. Updated on successful CI builds.

Super-awesome work!!!

Aaaand... merged :)
Many thanks, great feature!


MQTT Dim URL can't use variable substitutions?

My setup:
I am using 3.5.1 in a Docker environment on unRaid.
I have 4 echos, lots of Hues and other lights with custom firmware all tied together with MQTT.
I have found that ON and off work:
[{"clientId":"habridge","topic":"House/Lights/strip1/white1","message":"100"}]
[{"clientId":"habridge","topic":"House/Lights/strip1/white1","message":"0"}]
But the Dim does not work:
[{"clientId":"habridge","topic":"House/Lights/strip1/white1","message":"${intensity.percent}"}]
Sends ${intensity.percent} to the topic. If I remove the quotation marks, I get an error = SyntaxError: Unexpected Token...
I guess it may be getting confused with all of the curly brackets....

I knew I forgot to do something..... will be implementing that in the next release

Glad i checked here before getting really confused....
FYI in case it drives where/how you can implement i was hoping i could use something like:
[{"clientId":"habridge","topic":"HomeEasy","message":"{"emitter":50420536,"sendingStation":"Lounge2","receiver":0,"transmitValue":"Off","level":${intensity.math((X/100)*15)},"rngID":"rnglounge"}"}]
When all is working...
Thanks!


[BUG] Integration not working in HA 2023.9.0 Setup failed for custom integration ledfx: Unable to import component: cannot import name 'DeviceEntryType' from 'homeassistant.helpers.entity'

Describe the bug
The Integration will not work anymore due to the following error:
Setup failed for custom integration ledfx: Unable to import component: cannot import name 'DeviceEntryType' from 'homeassistant.helpers.entity' (/usr/src/homeassistant/homeassistant/helpers/entity.py)

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/setup.py", line 215, in _async_setup_component
    component = integration.get_component()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 818, in get_component
    ComponentProtocol, importlib.import_module(self.pkg_path)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1147, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/config/custom_components/ledfx/__init__.py", line 31, in <module>
    from .helper import build_auth, get_config_value
  File "/config/custom_components/ledfx/helper.py", line 17, in <module>
    from .updater import LedFxUpdater
  File "/config/custom_components/ledfx/updater.py", line 21, in <module>
    from homeassistant.helpers.entity import (
ImportError: cannot import name 'DeviceEntryType' from 'homeassistant.helpers.entity' (/usr/src/homeassistant/homeassistant/helpers/entity.py)

Please complete the following information:

HASS version: 2023.9.0
Component version 3.0.0
LedFx version [e.g. 0.10.7]

Same here!

This is also preventing the integration from being uninstalled.
Logger: aiohttp.server
Source: custom_components/ledfx/updater.py:21
Integration: LedFx ([documentation](https://github.com/dmamontov/hass-ledfx/wiki), [issues](https://github.com/dmamontov/hass-ledfx/issues))
First occurred: 3:00:51 PM (2 occurrences)
Last logged: 3:12:43 PM
Error handling request

Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/aiohttp/web_protocol.py", line 433, in _handle_request
    resp = await request_handler(request)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/aiohttp/web_app.py", line 504, in _handle
    resp = await handler(request)
           ^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/aiohttp/web_middlewares.py", line 117, in impl
    return await handler(request)
           ^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/http/security_filter.py", line 85, in security_filter_middleware
    return await handler(request)
           ^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/http/forwarded.py", line 227, in forwarded_middleware
    return await handler(request)
           ^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/http/request_context.py", line 28, in request_context_middleware
    return await handler(request)
           ^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/http/ban.py", line 80, in ban_middleware
    return await handler(request)
           ^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/http/auth.py", line 236, in auth_middleware
    return await handler(request)
           ^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/http/headers.py", line 31, in headers_middleware
    response = await handler(request)
               ^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/http/view.py", line 148, in handle
    result = await handler(request, **request.match_info)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/config/config_entries.py", line 87, in delete
    result = await hass.config_entries.async_remove(entry_id)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 1080, in async_remove
    await entry.async_remove(self.hass)
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 578, in async_remove
    component = integration.get_component()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 818, in get_component
    ComponentProtocol, importlib.import_module(self.pkg_path)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1147, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/config/custom_components/ledfx/__init__.py", line 31, in <module>
    from .helper import build_auth, get_config_value
  File "/config/custom_components/ledfx/helper.py", line 17, in <module>
    from .updater import LedFxUpdater
  File "/config/custom_components/ledfx/updater.py", line 21, in <module>
    from homeassistant.helpers.entity import (
ImportError: cannot import name 'DeviceEntryType' from 'homeassistant.helpers.entity' (/usr/src/homeassistant/homeassistant/helpers/entity.py)

Also impossible to delete the integration.

Same here. I noticed that ledfx web ui link to device is url/#/devices but link from ha ledfx integration is url/device. Was there an update to ledfx integration? Or did ha update brake it?

Same here. I resolved this issue by editing updater.py - removing the line importing DeviceEntryType and changing entry_type to = None. Seems to work but need to test further.

Thanks, that seems to work! Any problems after?

I ended up, instead of changing entry_type to none, I imported DeviceEntryType from device_registry helper (not entity). But unfortunately I'm having issues. My new issue is: AttributeError: 'LedFxLight' object has no attribute '_ActionType.DEVICE_on'.

Have found the source of this error.
https://github.com/home-assistant/core/pull/98149
I believe changing to the following code resolves. However I'm still receiving the same error as @seancmalloy when trying to undertake actions. Still investigating this - could be related to a separate deprecation or issue.
Replace:
from homeassistant.helpers.entity import (     DeviceEntryType,     DeviceInfo,     EntityCategory,     EntityDescription, )
with
from homeassistant.helpers.entity import EntityCategory, EntityDescription from homeassistant.helpers.device_registry import DeviceEntryType, DeviceInfo
in updater.py


Design maintenance prime 14 feb 2024

Update font size for code snippet
Add IconButton "secondary" variant

Reminder to check off things in the doc! Thanks!


[bp/1.30] CI migration stack

@phlax, we might have an easier time with DCO if you recreate this branch.

i had a similar problem on one of the branches i worked on - not sure why - all the commits seem signed off
either way you can resolve with:
git rebase HEAD~9 --signoff


Pasting an image into the PR comment results in plain  image.png text

Extension version: v0.74.0
VSCode Version:1.83.1
OS: Windows 11 Pro, Version 22H2

Steps to Reproduce:
You can use Shift+WindowsKey+S to take a quick screenshot of some specific area and immediately Ctrl+v it into the PR comment to embed an image. In Web GitHub UI this works. In VSCode will just produce an image.png  text.
Expected behavior - ideally - to be able to embed an image. Not ideally - just image.png text is misleading.

I am sad to say that GitHub's API doesn't support this and likely never will :(
Duplicate of https://github.com/microsoft/vscode-pull-request-github/issues/320


Update env to match azure-search-openai-demo

I think this rename allows me to deploy azure-search-openai-demo, then do azd env refresh with that env name, and use all those resources (not having to recreate them) and just this source code. I don't see that val used in source code so didn't change anything else.

Hm. So I renamed it here because we support non-Azure OpenAI models. We'd ideally rename in azure-search-openai-demo as well, but then that'd make it difficult for anyone who pulls from main.

Ok, I understand.


bootique/bootique#228 Add apache rat profile.

Looking at a list of excludes, I think the base module should not attempt to define them. They are fairly specific to the modules. So here we should place plugin version and lifecycle, deferring excludes to the specific sub-modules.

After an offline discussion:

the excludes list is actually merged between parent and child, so we can preserve common patterns. Still needs to be trimmed to exclude Cayenne files, html, tml, etc.
Let's rename the profile to "rat".


Timeline: Tab view selection

📒 Description
This PR introduces new layout and logic for the Tab View Selection.
🕶 Types of changes
New feature (non-breaking change which adds functionality)
🤯 List of changes

[x] Introduce MainDashboardViewController, which contains "Time Entry List", "Timer Bar" and "Timeline View"
[x] Stylish the TabView buttons
[x] Bundle TimeEntryList and Timeline as a TabView

👫 Relationships
Close #3015
Parent #3032
🔎 Review hints

The design should be meet
Able to switch between Timeline and Time Entry List
Switch dark/light mode will update the UI

Screenshots

Switching between List and Timeline tabs looks good.
I've found a small problem, I can't open the edit popup for the 2nd time, both for finished and for running entries. After the edit popup is hidden, I can open it only after I open another edit popup.

Thank @skel35. I investigated this issue and it turned out that it initially happens in https://github.com/toggl/toggldesktop/pull/3032 too.
I will check it out 👍

@skel35 I've fixed this issue at https://github.com/toggl/toggldesktop/commit/9ca209f4b493fee8ea76e3b68200591a2138bd8c from parent branch. I also rebased this branch successfully.
Please checkout on origin and test again. All works now 👍

Thank for the feedback @skel35, I would like to cumulate all minor UI improvements and create new ticket for it. The main goal of those PR is fulfilling the requirement of the ticket 👍
I will do it soon.


feat: ✨ rocket/release icon

Description
Adds a rocket generic file icon, meant to be used for config files of different release tools:

release-it/release-it

.release-it.{json,ts,js,cjs,yaml,yml}

crate-ci/cargo-release and cargo-bins/release-pr

release.toml

MarcoIeni/release-plz

release-plz.toml, .release-plz.toml

Closes: #2470
Contribution Guidelines

[x] By creating this pull request, I acknowledge that I have read the Contribution Guidelines for this project.
[x] I have read the Code Of Conduct and promise to abide by these rules

Just in case, don't merge it yet, I'll make a small correction, removing a few vertices to simplify the path. It won't change the shape, but it will make the file a bit smaller


Sunstone add option to select vCenter Template when importing a marketplace image

Description
Actually, when importing a marketplace image from Sunstone and instantiate it on vCenter will use a default template.

Use case
Use vCenter with apps from the marketplace with a different template.
Interface Changes
Add option to select vCenter templates when importing an image.

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

@tinova PR to be approved:

Tests


Android benchmark

Just let you know
A simple test:
https://github.com/KnIfER/LibPDFium#benchmarks

Hi.
Very nice @KnIfER :)
Do you have any simple project that use the built files to load a single PDF file and render their pages?
I want a simple project only to make unit tests.
Thanks anyway.

https://github.com/KnIfER/PDFium-Android-Demo/blob/5529a56e43fa3fd76c74838ecd93301d4a0a27a7/PDocViewer/src/main/java/com/knizha/PDocViewer/PDocMainActivity.java#L119
It's just AndroidPdfViewer using the new pdfium.so.
libpdfium.cr.so is compatible. but since AndroidPdfViewer is using an older version, the JNI source should be modified slightly (FPDFDest_GetPageIndex -> FPDFDest_GetPageIndex).

In Java, only the pdfium-lib.so is specifically loaded, linked dependencies will be loaded automcatically. So in my demo project, once you have built the jni layer, you can easily replace libpdfium.so with whaterever you want, as long as the substitute has the same API.

Hi @KnIfER,
Can you make any new benchmark with current version?
Thanks.

This commit adopted more custom Alloc methods, which seems to slower the test by 100ms. But when I cleaned up the mobile space, I found that the "regression" becames very small, so I chose to delete them.
Anyway the new test results:

650ms yours v1.0.0

645~660ms yours  v4408

656~663ms my 1(ndk)

590~620ms my 2(visual studio)

620~650ms Another guy's lib

Ok, nice. Now im using params to it link everything and generate final library.
I will close this but you can reopen if you want.


there is no ilc.exe

There is no ilc.exe on .nuget folder and its sub folders, i tried nuget restore and all things. cant find it on the web either. what to do?

It ships with this NuGet package.


modify_user

Please, stop creating pull-requests with project-specific changes. I won't merge them in any way.


No Color Shown Using Spot light

I've looked at Roy Stan's tutorial a bit, and seems like something was done for directional light to work, it does not seem to be the same for point and spot lights. I'm not too familiar with shader work, maybe someone more experienced with would know how to fix this? Or maybe it's a limitation in this case? Thanks!

I've been told the ForwardAdd Pass section, that's commented out probably takes care of that, sadly it seems like there was reason why it was commented out, result is different but not the right one.

Hi @ndumais
It's been a while since I've looked at this repository and I've wanted to update it for a while now, but haven't gotten around to doing so. Mostly because the standard render pipeline is really limiting in what it offers in terms of light in the forward rendering pass.
It also doesn't help that Geometry shaders in Unity are really undocumented and not well explored. Hence I was only able to find that in the vertex pass, the information of 4 important lights that affect the mesh are passed. Since I only get the position, intensity and color of these lights, I've limited the shader to use these lights as point lights at all time (since I miss an angle and spread that the spot-light offers).
Now I'm far from being good at writing shaders (far from it, since I'm more into backend), so I'll try to have a look at it in the following week to see if there's a way to make the ForwardAdd pass work, but for now I'd cross it out and say it's a limit of Unity's standard forward rendering.

Hi @ndumais
Just wanted to give a heads up that I'm still working on this issue and managed to book some progress. I've managed to read the right light information within the ForwardAdd pass. Currently this still only works for point lights, but from what I'm reading (and learning), point lights shouldn't be too far off.

Hey, thanks for the follow up!
Have been procrastinating on doing some follow here cause I've been sick the past 3 days.
Did read your first message, and also consulted with some people about what you were saying, someone said: "The "intensity" of each of those lights has already had all the angle falloff stuff (and cookies) calculated." Wasn't sure what to make of it though and didn't get a reply past that.
So would a white point light render the grass as green (without any directional lights) ?
That would be already a big step tbh!

Hahaha, yeah like I said, I'm not too familiar with shaders, just interested in them. I did find the macro they were talking about, a very handy UNITY_LIGHT_ATTENUATION macro that does all the light attenuation calculation for you (including cookies, spots, points and the likes). So I've used that in the forward pass and the lighting works correctly now.
Here's an example using no directional light and two spot lights, one of which has a cookie

The only "problem" that remains is that the light has a linear falloff, I'm still trying to make it resemble the default lighting more (which I believe uses an inverse squared falloff technique).
For now you can check if it's a result you're comfortable working with until I've got the inverse squared falloff working. I've setup a branch feature/forward-add if you want to test it out.

Definitely usable for prototyping now!
I think shadow casting for grass should be disabled in shader, what do you think?

Glad that it's usable now!
I've just pushed a change to the Forward Add pragma #pragma multi_compile_fwdadd to add shadows (which is the same with #pragma multi_compile_fwdadd_fullshadows attached to it) from point and spot lights (+ any additional directional lights).

If you want the grass to not cast any shadows, but only receive them, you can turn them off in the Mesh Renderer, which is Unity's preferred way as well.

Hey, just wanted to say a quick thanks, went with your solution and it worked out!


Exception in callbacks are caught by PyUAVCAN

Sometimes I hit a bug or an exception in a callback. However this never hits my debugger because UAVCAN  catches them and ignores them.
Example: https://github.com/UAVCAN/pyuavcan/blob/master/uavcan/node.py#L217-L220
I would suggest passing a flag to the node or the spin method, something like node.spin(catch_callback_exceptions=False) to let advanced users use exceptions like they want.
What do you think? Maybe I am missing something on this design choice?

Pyuavcan does that to ensure that exceptions that occur in a handler do not prevent the library from invoking other handlers. Adding a flag as you suggested seems sensible. Perhaps it's time for a pull request?

PR opened: #46

Closed in #46


how to listen if face is detected ?

i have implemented lib in my project and followed docs ,
i want to listen if face is detected , i know that there is a on faied result listenter , where i need to pass result listenere which gives me onSuccess and onFailed callback methods , but i am not getting any response , neither success nor failure . i even tried to impalement interface but results are same .
please let me know why does its happening , and thank you if you can help me

Dear Sir

open app build.gradle
//    implementation "com.github.husaynhakeem:android-face-detector:2.0" // Remote <---close
implementation(project(":facedetector")) // Local <---opne

modify  FaceDetector.kt at line 94 & 95
https://github.com/husaynhakeem/android-face-detector/blob/a8162ac724d1a439b9e2040c33cd2a1e69ae657a/facedetector/src/main/java/husaynhakeem/io/facedetector/FaceDetector.kt#L94

add "onFaceDetectionResultListener?.onSuccess(faceBounds)"

modify  demo  MainActivity.kt  line 44 & 45
https://github.com/husaynhakeem/android-face-detector/blob/a8162ac724d1a439b9e2040c33cd2a1e69ae657a/app/src/main/java/husaynhakeem/io/facedetectorapp/MainActivity.kt#L44

add
faceDetector.setonFaceDetectionFailureListener(object :
FaceDetector.OnFaceDetectionResultListener {
override fun onSuccess(faceBounds: List) {
Log.e("YOOOOO","Get face $faceBounds ")
}
override fun onFailure(exception: Exception) {
        }
    })


Refactor StartDevMode to return a single object instead of multiple args

What type of PR is this:
/area testing
/kind code-refactoring

What does this PR do / why we need it:
I find it confusing to work with multiple args returned by StartDevMode, using a single object makes it easier.
Which issue(s) this PR fixes:

Latest commit Fixes #6876
PR acceptance criteria:

[ ] Unit test

[x] Integration test

[ ] Documentation

How to test changes / Special notes to the reviewer:

/override OpenShift-Integration-tests/OpenShift-Integration-tests
Network issue.
  [odo]  âœ—  Syncing files into the container [4ms]
  [odo] Error occurred on Push - watch command was unable to push component: failed to sync to component with name mynode: failed to sync to component with name mynode: unable push files to pod: error while streaming command: error sending request: Post "https://c100-e.eu-de.containers.cloud.ibm.com:31049/api/v1/namespaces/cmd-remove-binding-test71qon/pods/mynode-app-57d77b45bb-xmjvz/exec?command=tar&command=xf&command=-&command=-C&command=%2Fprojects&command=--no-same-owner&container=runtime&stderr=true&stdin=true&stdout=true": dial tcp: lookup c100-e.eu-de.containers.cloud.ibm.com: operation was canceled
  [odo]
  [odo]
  [odo] â†ª Dev mode
  [odo]  Status:
  [odo]  Watching for changes in the current directory /tmp/3271381815
  [odo]
  [odo]  Keyboard Commands:
  [odo] [Ctrl+c] - Exit and delete resources from the cluster
  [odo]      [p] - Manually apply local changes to the application on the cluster
  [odo] I0616 11:43:50.565792   17461 implem.go:114] The path for preference file is /tmp/4157871543/preference.yaml
  [odo] I0616 11:43:50.566355   17461 segment.go:264] Checking telemetry enable status
  [odo] I0616 11:43:50.566388   17461 segment.go:282] Sending telemetry disabled by env variable
  [odo] Cleaning resources, please wait
  [odo] I0616 11:43:50.614310   17461 delete.go:176] Ignoring resource: servicebindings/my-nodejs-app-cluster-sample-ocp; it does not exist on the cluster
  [odo] I0616 11:43:50.657616   17461 all.go:46] starting to concurrently query 200 APIs
  [odo] I0616 11:43:50.659814   17461 all.go:62] fired up all goroutines to query APIs
  [odo] I0616 11:43:51.528455   17461 all.go:67] all goroutines have returned in 870.837356ms
  [odo] I0616 11:43:51.528485   17461 all.go:76] query result: objects=546
  [odo]  âœ—  Dev mode interrupted by user
  Deleting project: cmd-remove-binding-test71qon
  Running oc with args [oc delete project cmd-remove-binding-test71qon --wait=false] and odo env: []
  [oc] project.project.openshift.io "cmd-remove-binding-test71qon" deleted
  Setting current dir to: /go/odo_1/tests/integration
  Deleting dir: /tmp/3271381815
  Deleting dir: /tmp/4157871543
  << Timeline

  [FAILED] Expected
      <string>:
  to contain substring
      <string>: not found

/override windows-integration-test/Windows-test
  [FAILED] No future change is possible.  Bailing out early after 1.212s.
  Running oc.exe with args [oc get csv -o jsonpath={.items[?(@.status.phase=="Succeeded")].metadata.name}] and odo env: []
  Expected
      <int>: 1
  to match exit code:
      <int>: 0
------------
  I0616 05:03:50.613347    6492 util.go:198] path C:\Users\Administrator.ANSIBLE-T
  EST-VS\AppData\Local\Temp\2170439783\devfile.yaml doesn't exist, skipping it    [92m
  ? [97mDo you want to list services from:[m  [36m[Use arrows to move, type to filter][m      [36m
  > current namespace[m[61X[61C
    all accessible namespaces[53X[53C[?25h[?25l[HI[?25h
  ======================
  Unexpected error:
      <*xpty.errPassthroughTimeout | 0xc0008cc020>: {
          error: <*errors.errorString | 0xc0008cc010>{
              s: "passthrough i/o timeout",
          },
      }
      passthrough i/o timeout
  occurred
  In [It] at: C:/Users/Administrator.ANSIBLE-TEST-VS/4170/tests/helper/helper_interactive.go:111 @ 06/16/23 05:05:50.661
------------------------------
Summarizing 3 Failures:
  [FAIL] odo devfile deploy command tests deploying devfile with long-running exec when Automount volumes are present in the namespace [It] should mount the volumes
  C:/Users/Administrator.ANSIBLE-TEST-VS/4170/tests/helper/helper_cmd_wrapper.go:120
  [FAIL] odo add binding interactive command tests when running a deployment when binding to a service in a different namespace [It] should successfully add binding without devfile
  C:/Users/Administrator.ANSIBLE-TEST-VS/4170/tests/helper/helper_interactive.go:111
  [FAIL] odo describe/list binding command tests when creating a component with a binding as environment variables (service in namespace "") when Starting a Pg service [BeforeEach] when running dev session when changing the current namespace should list the binding with --namespace flag - command: [list binding]
  C:/Users/Administrator.ANSIBLE-TEST-VS/4170/tests/helper/helper_generic.go:58

/override Pull Request Website Preview / deploy-prev

/override deploy-preview


[Shifter] Add Threads for UI feedback

[x] Setup Thread to perform batch and not freeze main Maya session.
[ ] Update main UI with Batch results.

[ ] Check on Windows OS

The progress bar is green and appears when the export is occuring. If an error occurs it goes red, and an error is logged in the maya logs.


Linux build fails

I'm on Ubuntu 24.10 and have a fairly complete build environment installed on this machine as I do a lot of C++/Rust R&D.
TLDR; the build fails on missing spdlog, EASTL and reproc/reproc++.
error: failed to run custom build command for `luisa_compute_sys v0.1.1-alpha.1 (/home/ritz/code/crates/luisa-compute-rs/luisa_compute_sys)`

Caused by:
  process didn't exit successfully: `/home/ritz/code/crates/luisa-compute-rs/target/debug/build/luisa_compute_sys-8e3a5ff2dbde5797/build-script-build` (exit status: 101)
  --- stdout
  cargo:rerun-if-changed=./LuisaCompute/src/api
  cargo:rerun-if-changed=./LuisaCompute/src/ast
  cargo:rerun-if-changed=./LuisaCompute/src/backends
  cargo:rerun-if-changed=./LuisaCompute/src/core
  cargo:rerun-if-changed=./LuisaCompute/src/gui
  cargo:rerun-if-changed=./LuisaCompute/src/ir
  cargo:rerun-if-changed=./LuisaCompute/src/rust/luisa_compute_backend_impl
  cargo:rerun-if-changed=./LuisaCompute/src/rust/luisa_compute_ir
  cargo:rerun-if-changed=./LuisaCompute/src/rust/CMakeLists.txt
  cargo:rerun-if-changed=./LuisaCompute/src/py
  cargo:rerun-if-changed=./LuisaCompute/src/runtime
  cargo:rerun-if-changed=./LuisaCompute/src/vstl
  cargo:rerun-if-env-changed=CARGO_FEATURE_DX
  cargo:rerun-if-env-changed=CARGO_FEATURE_CPU
  cargo:rerun-if-env-changed=CARGO_FEATURE_REMOTE
  cargo:rerun-if-env-changed=CARGO_FEATURE_CUDA
  cargo:rerun-if-env-changed=CARGO_FEATURE_METAL
  cargo:rerun-if-env-changed=CARGO_FEATURE_PYTHON
  cargo:rerun-if-env-changed=CARGO_FEATURE_GUI
  cargo:rerun-if-env-changed=CARGO_FEATURE_OIDN
  cargo:rerun-if-env-changed=CARGO_FEATURE_WAYLAND
  cargo:rerun-if-env-changed=PROFILE
  cargo:rerun-if-env-changed=CC
  cargo:rerun-if-env-changed=CXX
  CMAKE_TOOLCHAIN_FILE_x86_64-unknown-linux-gnu = None
  CMAKE_TOOLCHAIN_FILE_x86_64_unknown_linux_gnu = None
  HOST_CMAKE_TOOLCHAIN_FILE = None
  CMAKE_TOOLCHAIN_FILE = None
  CMAKE_PREFIX_PATH_x86_64-unknown-linux-gnu = None
  CMAKE_PREFIX_PATH_x86_64_unknown_linux_gnu = None
  HOST_CMAKE_PREFIX_PATH = None
  CMAKE_PREFIX_PATH = None
  CMAKE_x86_64-unknown-linux-gnu = None
  CMAKE_x86_64_unknown_linux_gnu = None
  HOST_CMAKE = None
  CMAKE = None
  running: cd "/home/ritz/code/crates/luisa-compute-rs/target/debug/build/luisa_compute_sys-4f8faeea12917f90/out/build" && CMAKE_PREFIX_PATH="" LC_ALL="C" "cmake" "/home/ritz/code/crates/luisa-compute-rs/luisa_compute_sys/./LuisaCompute" "-G" "Ninja" "-DLUISA_COMPUTE_ENABLE_DX=ON" "-DLUISA_COMPUTE_ENABLE_CPU=ON" "-DLUISA_COMPUTE_ENABLE_REMOTE=ON" "-DLUISA_COMPUTE_ENABLE_CUDA=ON" "-DLUISA_COMPUTE_ENABLE_METAL=ON" "-DLUISA_COMPUTE_ENABLE_PYTHON=OFF" "-DLUISA_COMPUTE_ENABLE_GUI=OFF" "-DLUISA_COMPUTE_DOWNLOAD_OIDN=OFF" "-DLUISA_COMPUTE_ENABLE_WAYLAND=OFF" "-DLUISA_COMPUTE_CHECK_BACKEND_DEPENDENCIES=ON" "-DLUISA_COMPUTE_BUILD_TESTS=OFF" "-DLUISA_COMPUTE_ENABLE_DSL=OFF" "-DLUISA_COMPUTE_ENABLE_RUST=ON" "-DCMAKE_INSTALL_PREFIX=/home/ritz/code/crates/luisa-compute-rs/target/debug/build/luisa_compute_sys-4f8faeea12917f90/out" "-DCMAKE_C_FLAGS= -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_C_COMPILER=/usr/bin/cc" "-DCMAKE_CXX_FLAGS= -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_CXX_COMPILER=/usr/bin/c++" "-DCMAKE_ASM_FLAGS= -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_ASM_COMPILER=/usr/bin/cc" "-DCMAKE_BUILD_TYPE=Debug"
  -- Build for Linux-6.9.11-060911-generic (x86_64)
  -- Enable Rust support (toolchain found at /home/ritz/.cargo/bin/cargo)
  -- Build with sccache: /home/ritz/.cargo/bin/sccache
  -- IPO/LTO enabled for release builds
  -- Configuring incomplete, errors occurred!

  --- stderr
  CMake Warning at scripts/validate_options.cmake:70 (message):
    The DirectX backend is not available.  The DirectX backend will be
    disabled.
  Call Stack (most recent call first):
    scripts/validate_options.cmake:80 (report_feature_not_available)
    CMakeLists.txt:53 (include)

  CMake Warning at scripts/validate_options.cmake:70 (message):
    The Metal backend is not available.  The Metal backend will be disabled.
  Call Stack (most recent call first):
    scripts/validate_options.cmake:91 (report_feature_not_available)
    CMakeLists.txt:53 (include)

  CMake Error at src/ext/CMakeLists.txt:40 (add_subdirectory):
    The source directory

      /home/ritz/code/crates/luisa-compute-rs/luisa_compute_sys/LuisaCompute/src/ext/spdlog

    does not contain a CMakeLists.txt file.

  CMake Error at src/ext/CMakeLists.txt:41 (target_compile_definitions):
    Cannot specify compile definitions for target "spdlog_header_only" which is
    not built by this project.

  CMake Error at src/ext/CMakeLists.txt:46 (target_compile_definitions):
    Cannot specify compile definitions for target "spdlog" which is not built
    by this project.

  CMake Error at src/ext/CMakeLists.txt:51 (set_target_properties):
    set_target_properties Can not find target to add properties to: spdlog

  CMake Error at src/ext/CMakeLists.txt:9 (install):
    install TARGETS given target "spdlog_header_only" which does not exist.
  Call Stack (most recent call first):
    src/ext/CMakeLists.txt:53 (luisa_compute_install_extension)

  CMake Error at src/ext/CMakeLists.txt:135 (add_subdirectory):
    The source directory

      /home/ritz/code/crates/luisa-compute-rs/luisa_compute_sys/LuisaCompute/src/ext/EASTL

    does not contain a CMakeLists.txt file.

  CMake Error at src/ext/CMakeLists.txt:136 (set_target_properties):
    set_target_properties Can not find target to add properties to: EASTL

  CMake Error at src/ext/CMakeLists.txt:9 (install):
    install TARGETS given target "EASTL" which does not exist.
  Call Stack (most recent call first):
    src/ext/CMakeLists.txt:140 (luisa_compute_install_extension)

  CMake Error at src/ext/CMakeLists.txt:9 (install):
    install TARGETS given target "EABase" which does not exist.
  Call Stack (most recent call first):
    src/ext/CMakeLists.txt:141 (luisa_compute_install_extension)

  CMake Error at src/ext/CMakeLists.txt:142 (target_compile_features):
    Cannot specify compile features for target "EASTL" which is not built by
    this project.

  CMake Error at src/ext/CMakeLists.txt:144 (target_compile_definitions):
    Cannot specify compile definitions for target "EASTL" which is not built by
    this project.

  CMake Error at src/ext/CMakeLists.txt:168 (add_subdirectory):
    The source directory

      /home/ritz/code/crates/luisa-compute-rs/luisa_compute_sys/LuisaCompute/src/ext/reproc

    does not contain a CMakeLists.txt file.

  CMake Error at src/ext/CMakeLists.txt:170 (set_target_properties):
    set_target_properties Can not find target to add properties to: reproc

  CMake Error at src/ext/CMakeLists.txt:171 (set_target_properties):
    set_target_properties Can not find target to add properties to: reproc++

  CMake Error at src/ext/CMakeLists.txt:181 (target_compile_definitions):
    Cannot specify compile definitions for target "reproc" which is not built
    by this project.

  CMake Error at src/ext/CMakeLists.txt:182 (target_compile_definitions):
    Cannot specify compile definitions for target "reproc++" which is not built
    by this project.

  CMake Error at src/ext/CMakeLists.txt:183 (target_link_libraries):
    Cannot specify link libraries for target "reproc" which is not built by
    this project.

  thread 'main' panicked at /home/ritz/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/cmake-0.1.52/src/lib.rs:1115:5:

  command did not execute successfully, got: exit status: 1

  build script failed, must exit now
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...

NM the noise, my git clone apparently bailed halfway through so I my mirror was incomplete. I cloned again, build working fine now. :D


Update OUT_DIR based on cargo check output

This entire PR is basically annoying plumbing needed to pass through the correct OUT_DIR from a cargo check run to the right place in the crate graph.
It's in a semi working state: It correctly updates OUT_DIR in the crate graph, and get's to the point where macro's depending on env!("OUT_DIR") are re-evaluated, but I don't seem to get completion for the gl crate still. Not sure why, but decided to get the code public ASAP so we can discuss the right direction to take it.
Things that still need to be done/decided:

[ ] When/if to schedule a single cargo check run on startup to get the correct OUT_DIR values.
[ ] What's the story with features, targets and OUT_DIR?
[ ] All the other things that I've overlooked

Depends on rust-analyzer/ra_vfs#20 but is currently patched to use my fork so things can work.

but I don't seem to get completion for the gl crate still

I am investigating now, and it seem to be the FileLoaderDelegate::resolve_extern_path return a wrong id. And it is because in https://github.com/rust-analyzer/ra_vfs/pull/20 , we find the path by starts_with. However, usually the OUT_DIR  is lived in the main crate target diretory and thus this will return the main crate instead.

And it is because in rust-analyzer/ra_vfs#20 , we find the path by starts_with. However, usually the OUT_DIR lives in the main crate target diretory and thus it will return the main crate instead.

Will that handle a target directory that's not a subdirectory of the crate root? E.g. if you set CARGO_TARGET_DIR to an absolute path

@jyn514 It doesn't yet, but should once I get around to making it add a new vfs root for each out_dir.

Okay pushed a new take on the whole thing. This time it actually works!
So I've added a config option to run cargo check when loading a cargo workspace to get the right OUT_DIR paths. It did take some juggling to get the extern source stuff to a reasonable point.
It currently doesn't support setting OUT_DIR for sysroot crates, but I don't think any of them should be using build scripts with OUT_DIR?
I'll take a look at folding the custom path support into the project model config, so we can eliminate some duplication, and then add OUT_DIR support to the json workspaces.

Okay that wasn't too bad either.
@matklad Do you want me to hook up OUT_DIR support for CLI runs of rust-analyzer as well, while I'm at it?

Do you want me to hook up OUT_DIR support for CLI runs of rust-analyzer as well, while I'm at it?

Yeah, I guess that would be helpful! It should be behind a --get-output-dir flag (we don't want to cargo-check by default). And there should be a better verb then get. sniff?

--load-output-dir maybe? The name should probably imply up front that some work is being done.

And with that in place I think this is ready for some actual review.
~/testcase $ rust-analyzer analysis-stats --load-output-dirs .                            (master|…6) kiljacken@exheater
Database loaded, 20 roots, 109.269843ms
Crates in this dir: 1
Total modules found: 1
Total declarations: 2
Total functions: 2
Item Collection: 3.081167556s, 0b allocated 0b resident
Total expressions: 12
Expressions of unknown type: 0 (0%)
Expressions of partially unknown type: 0 (0%)
Type mismatches: 0
Inference: 2.604975548s, 0b allocated 0b resident
Total: 5.686150434s, 0b allocated 0b resident
~/testcase $ rust-analyzer analysis-stats .                                               (master|…6) kiljacken@exheater
Database loaded, 17 roots, 92.107763ms
Crates in this dir: 1
Total modules found: 1
Total declarations: 2
Total functions: 2
Item Collection: 1.61754515s, 0b allocated 0b resident
Total expressions: 12
Expressions of unknown type: 1 (8%)
Expressions of partially unknown type: 0 (0%)
Type mismatches: 0
Inference: 2.2312987s, 0b allocated 0b resident
Total: 3.84884987s, 0b allocated 0b resident

NPM audit check seems to be causing issues upstream, not sure if I can do anything about it.

Rebased to preempt bors hi-jinx due to CI changes

Concerns should be resolved in new commit. Also re-based once again to make it easier on poor bors

bors r+

Hm, so it seems like npm audit fails, and we can't really fix it, because eslint plugin is not updated yet....


⚠ Spryr has degraded performance

In f093c03, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 3907 ms

Resolved: Spryr performance has improved in 5851e8b after 7 minutes.


PRX Embed player shaping

After reviewing notes and conversations on this, the consensus is that the work around the PRX audio player can be summed up this way:

Rebuild the PRX embedded player so it is server-side rendered in React and thusly lighter weight. See this discussion: https://prx.slack.com/archives/C735D3DJQ/p1578414018002000
Provide a hook for a persistent player option (Based on discussions around usage within The World's React frontend).
The current set of audio controls should be preserved, especially the embed option.

If there are any other proposed changes you'd like to see, lets discuss.

Primary use case: For producers that use our platform to share their episode.
Future todo: Do some user interviews around this in the design phase.


Pitfall: nunjucks-date vs nunjucks-date-filter

I read the documentation on the timezone errors with formatting dates, but nonetheless had trouble getting stuff to format correctly. I've only just now realized that nunjucks-date and nunjucks-date-filter are two separate packages (with identical interfaces it seems) and that only the latter formats things in UTC by default as is needed here. I don't know if this belongs in the documentation because it's about an external package, but I was confused by it for a few days.

I vote for always assuming UTC by default.

Note that eleventy itself is always UTC and pretty clearly documented as such, this is just an issue of other technically unrelated packages.


Be able to run SQL CLI workflows on Airflow / Astro CLI locally

Please describe the feature you'd like to see
As of SQL CLI 0.2, all the workflows rely on SQLite. We mount those during astro flow commands but are not mounted during astro dev start. This means users cannot have an end-to-end journey using the SQL CLI / Astro CLI, where they'd visualise the workflow they built with SQL.
Describe the solution you'd like
It would be great to find a strategy so users can visualise/run example DAGs in Astro CLI - until we define the documentation journey.
Acceptance Criteria

[ ] All checks and tests in the CI should pass
[ ] Unit tests (90% code coverage or more, once available)
[ ] Integration tests (if the feature relates to a new database or external service)
[ ] Example DAG
[ ] Docstrings in reStructuredText for each of  methods, classes, functions and module-level attributes (including Example DAG on how it should be used)
[ ] Exception handling in case of errors
[ ] Logging (are we exposing useful information to the user? e.g. source and destination)
[ ] Improve the documentation (README, Sphinx, and any other relevant)
[ ] How to use Guide for the feature (example)

What astro-cli already has:

$AIRFLOW_HOME/include directory for mounting additional directories/files

What flow depends on:

data directory for local file (sqlite) dbs

One option could be to copy the flow projects into the include directory:
This could be done using something like:
astro dev start --include path/to/flow_project_1,another_path/to/flow_project_2

which would copy to astro's $AIRFLOW_HOME/include (natively). We create a new feature for copying on demand directories/files.
OR by using something like:
cp path/to/flow_project_1/data ./include/flow_project_1_data
cp path/to/flow_project_2/data ./include/flow_project_2_data

executed in the astro-cli project directory (non-natively). Note we only need data directory here, because we can be explicit as we control where the files/directories lands.
Then we create connections in airflow (as a starting point manually) and set the host path accordingly i.e. to where we have moved the files to.

Another option suggested by Tatiana:
We accept another parameter during projection initialisation that the user can specify as their example data directory.

TODO: update the docs once we can run the workflows within Astro CLI:
https://github.com/astronomer/docs/blob/main/astro/cli/sql-cli.md

Re-opening as docs are missing and need to verify once more that it works now.


mesheryctl: Improve error checking in perf subcommand

Fixes #784

@kushthedude a demo of this update in the community meeting today would be nice. 😄

@leecalcote As discussed on the call, I have added feature to display example usage and help section whenever user inputs mesheryctl perf without any arguement.
Please review & merge!

@anandrkskd it should reside in perf subcommand. Function should not be moved to vars.go since it is only used by perf.go and is a local function rather than a function with multiple usage across the Repository.

I thought of that reason and for that reason I don't want to move it to vars.go but simplicity of code is also a point and if I'm not mistaken that portion of code generates a test-name for any new test Right?
So it will be great if perf contains only the code related to its test.  So that It will be easy to understand by any new contributor.

No, I think you are mistaken here. This portion generated TestName
specifically for Perf command. Perf sub-command is CLI implementation of
Performance Page Meshery. For storage purpose we need testName for the mesh
to be tested but as the name flag is optional, we will utilise this
function to generate a random testName to use it to store the test records
and display them.
On Sat, 11 Apr, 2020, 17:41 Anand Kumar Singh<EMAIL_ADDRESS>wrote:

I thought of that reason and for that reason I don't want to move it to
vars.go but simplicity of code is also a point and if I'm not mistaken that
portion of code generates a test-name for any new test Right?
So it will be great if perf contains only the code related to its test. So
that It will be easy to understand by any new contributor.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/layer5io/meshery/pull/808#issuecomment-612406511, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKQMTLXD4WHVCBVJD4HOCQDRMBM7ZANCNFSM4MER2RTA
.

@leecalcote Made the required changes & improved the error handling. Please review & merge.

Yes indeed!
On Tue, 14 Apr, 2020, 15:10 Anand Kumar Singh<EMAIL_ADDRESS>wrote:

@anandrkskd approved this pull request.
LGTM, Thanks @kushthedude https://github.com/kushthedude,
Have you tried it on your local, by creating a local binary?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/layer5io/meshery/pull/808#pullrequestreview-392765230,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKQMTLXVWU23HL2UNJJUN2TRMQVSRANCNFSM4MER2RTA
.

@leecalcote Please review & merge!


Check bundle version in rake release task.

This PR brings the feature proposed in #285. I've decided to use a simple shell script with xcodebuild instead of bringing another dependency of Xcodeproj gem.

@krzysztofzablocki are we waiting for 1 approval, 2 approvals?


Normalize the framework name to "ExtJS"

The name "ExtJS" seems to be more widely used and it also appears more often in the blogs and books listed as resources. On their website, Sencha seems to stick to the "Ext JS" version. But, as developers we do not like spaces that much 😄
I just replaced all the "Extjs", "ExtJs", "Ext JS" with "ExtJS" (except for book titles and official description of repos) in an attempt to normalize the name.
I might be biased because I use this naming, or because JavaScript is often abbreviated as "JS".
Not a big deal, but I thought you might want take a look.

Thx for your PR !
I think you're right, ExtJS is the most appropriate, because it sticks with the official Sencha product name (without space).

I'm glad you agreed. It also might be a good idea to specify this in the Contribution Guidelines so that it stays consistent from now on!


fix: motoko canisters can import other canisters with service constructor

Description
Fixes SDK-1084
How Has This Been Tested?
The onchain part of the deps e2e test
Checklist:

[x] The title of this PR complies with Conventional Commits.
[x] I have edited the CHANGELOG accordingly.
[ ] I have made corresponding changes to the documentation.

Related PR on the Motoko side: https://github.com/dfinity/motoko/issues/3990


Client network socket disconnected before secure TLS connection was established" on package create

Summary
When trying to create a new package version, getting an error message  "Client network socket disconnected before secure TLS connection was established"
Steps To Reproduce:

Attempt to create a new package version  sfdx force:package:version:create -w 60 -x -c --package package_name --json

Expected result
Package version gets created
Actual result
{
"status": 1,
"name": "Fetch",
"message": "request to https://xxx.my.salesforce.com/services/data/v55.0/tooling/query?q=SELECT Id%2C Status%2C Package2Id%2C Package2VersionId%2C Package2Version.SubscriberPackageVersionId%2C Tag%2C Branch%2C CreatedDate%2C Package2Version.HasMetadataRemoved%2C CreatedById FROM Package2VersionCreateRequest WHERE Id %3D '08c5G000000PCXXXXX' ORDER BY CreatedDate failed, reason: Client network socket disconnected before secure TLS connection was established",
"exitCode": 1,
"context": "PackageVersionCreateCommand",
"stack": "Fetch: request to https://xxx.my.salesforce.com/services/data/v55.0/tooling/query?q=SELECT Id%2C Status%2C Package2Id%2C Package2VersionId%2C Package2Version.SubscriberPackageVersionId%2C Tag%2C Branch%2C CreatedDate%2C Package2Version.HasMetadataRemoved%2C CreatedById FROM Package2VersionCreateRequest WHERE Id %3D '08c5G000000PCXXXXX' ORDER BY CreatedDate failed, reason: Client network socket disconnected before secure TLS connection was established\n    at Function.wrap (/usr/local/lib/node_modules/sfdx-cli/node_modules/@salesforce/core/lib/sfError.js:61:25)\n    at PackageVersionCreateCommand.catch (/usr/local/lib/node_modules/sfdx-cli/node_modules/salesforce-alm/dist/ToolbeltCommand.js:255:44)\n    at async PackageVersionCreateCommand._run (/usr/local/lib/node_modules/sfdx-cli/node_modules/@salesforce/command/lib/sfdxCommand.js:93:13)\n    at async Config.runCommand (/usr/local/lib/node_modules/sfdx-cli/node_modules/@oclif/config/lib/config.js:173:24)\n    at async SfdxMain.run (/usr/local/lib/node_modules/sfdx-cli/node_modules/@oclif/command/lib/main.js:27:9)\n    at async SfdxMain._run (/usr/local/lib/node_modules/sfdx-cli/node_modules/@oclif/command/lib/command.js:43:20)\n    at async Object.run (/usr/local/lib/node_modules/sfdx-cli/dist/cli.js:162:47)",
"warnings": [],
"commandName": "PackageVersionCreateCommand"
}
System Information

Which shell/terminal are you using? (e.g. bash, zsh, powershell 5, powershell 7, cmd.exe, etc.)

If you are using sfdx

Run sfdx version --verbose --json

If you are using sf

Run sf version AND sf plugins --core

Paste the output here

$ sfdx version --json
{
"cliVersion": "sfdx-cli/7.160.0",
"architecture": "darwin-x64",
"nodeVersion": "node-v16.13.2"
}
C02Z91V3LVDQ:proact_base wra1365$ sfdx version --verbose --json
{
"cliVersion": "sfdx-cli/7.160.0",
"architecture": "darwin-x64",
"nodeVersion": "node-v16.13.2",
"pluginVersions": [
"@dx-cli-toolbox/sfdx-toolbox-package-utils 0.8.3",
"@oclif/plugin-autocomplete 1.3.0",
"@oclif/plugin-commands 1.3.0 (core)",
"@oclif/plugin-help 3.3.1 (core)",
"@oclif/plugin-not-found 1.2.6 (core)",
"@oclif/plugin-plugins 1.10.11 (core)",
"@oclif/plugin-update 1.5.0 (core)",
"@oclif/plugin-warn-if-update-available 1.7.3 (core)",
"@oclif/plugin-which 1.0.4 (core)",
"@salesforce/lwc-dev-server 2.11.0",
"@salesforce/sfdx-diff 0.0.6",
"@salesforce/sfdx-plugin-lwc-test 1.0.0 (core)",
"alias 2.1.0 (core)",
"apex 1.1.0 (core)",
"auth 2.2.2 (core)",
"bbdoc 4.0.5",
"community 2.0.0 (core)",
"config 1.4.14 (core)",
"custom-metadata 2.0.0 (core)",
"data 2.0.4 (core)",
"generator 2.0.2 (core)",
"info 2.0.1 (core)",
"limits 2.0.1 (core)",
"org 2.0.3 (core)",
"salesforce-alm 54.6.2 (core)",
"schema 2.1.1 (core)",
"sfdmu 4.15.0",
"sfdx-cli 7.160.0 (core)",
"sfdx-flowdoc-plugin 0.5.1",
"sfdx-waw-plugin 1.5.0",
"sfpowerkit 4.2.11",
"shane-sfdx-plugins 4.43.0",
"├─ @mshanemc/plugin-streaming 1.1.7",
"└─ @mshanemc/sfdx-sosl 1.1.0",
"signups 1.2.0 (core)",
"source 2.0.7 (core)",
"telemetry 2.0.0 (core)",
"templates 55.0.0 (core)",
"trust 2.0.1 (core)",
"user 2.1.0 (core)"
],
"osVersion": "Darwin 21.6.0"
}
Additional information
Feel free to attach a screenshot.

It appears that this may have been a temporary Salesforce glitch, as it is now working again.


Added implementation for Bitplane Representation of Quantum Images (BRQI)

Added implementation for Bitplane Representation of Quantum Images (BRQI) #131
The implementation of BRQI is present within the embeddings/image_embeddings/brqi.py file.
Added test cases for the BRQI Module in the tests/embeddings/image_embeddings/test_brqi.py file

Hey, @Dljdd
Could you make sure the unit tests and lint are passing?

Hi, @SaashaJoshi
I have modified the files, the unit tests and lint should be passing now

Hi @Dljdd, I am going to merge this repository. If there are any required changes, I will suggest making another repository. I do not want your hard work go waste by me being late in reviewing your code!


x/tools/gopls: extract variable returns an extra lhs expression and no template edit support is available

go version go1.20.1 darwin/arm64
golang.org/x/tools/gopls v0.14.2
<EMAIL_ADDRESS>h1:sIw6vjZiuQ9S7s0auUUkHlWgsCkKZFWDHmrge8LYsnc=
<EMAIL_ADDRESS>h1:9F2/+DoOYIOksmaJFPw1tGFy1eDnIJXg+UHjuD8lTak=
<EMAIL_ADDRESS>h1:O2Tfq5qg4qc4AmwVlvv0oLiVAGB7enBSJ2x2DqQFi38=
<EMAIL_ADDRESS>h1:we8PVUC3FE2uYfodKH/nBHMSetSfHDR6scGdBi+erh0=
<EMAIL_ADDRESS>h1:2O2DON6y3XMJiQRAS1UWU+54aec2uopH3x7MAiqGW6Y=
<EMAIL_ADDRESS>h1:dGoOF9QVLYng8IHTm7BAyWqCqSheQ5pYWGhzW00YJr0=
<EMAIL_ADDRESS>h1:zxkM55ReGkDlKSM+Fu41A+zmbZuaPVbGMzvvdUPznYQ=
<EMAIL_ADDRESS>h1:Vz7Qs629MkJkGyHxUlRHizWJRG2j8fbQKjELVSNhy7Q=
<EMAIL_ADDRESS>h1:brbkEFfGwNGAEkykUOcryE/JiHUMMJouzE0fWWmz/QU=
<EMAIL_ADDRESS>h1:ablQoSUd0tRdKxZewP80B+BaqeKJuVhuRxj/dkrun3k=
<EMAIL_ADDRESS>h1:Oku7E+OCrXHyst1dG1z10etCTxewCHXNFLRlyMPbh3w=
<EMAIL_ADDRESS>h1:KUas02EjQK5LTuIx1OylBQdKKZ9jeugs+HiqO5HormU=
<EMAIL_ADDRESS>h1:YGD4H+SuIOOqsyoLOpZDWcieM28W47/zRO7f+9V3nvo=
<EMAIL_ADDRESS>h1:JVf4NN1mIpHogBj7ABpgOyZc65/UUOkKQFkoURsz4MM=
<EMAIL_ADDRESS>h1:tzxjVAj+wSBmDcF6zBB7/myTy3gX9xvi8Tyr28AuQgc=
go: go1.20.1
vscode 1.86.2
Describe the bug
Given the following code
func hello() (string, int) {
	return "", 1
}

func call() {
	hello()
}

if try to extract variable for function hello, I will get the following with no proper template edits so I can rename both variable tuples

@suzmue You are correct. I think in other languages it is done by a custom action which consists of edits followed by a rename.

@suzmue I tried to improve the extract variable with the current features we have in LSP, Made the PR under https://github.com/golang/tools/pull/482

Also as part of the CodeAction we can send a command which will run after the edits are applied. But we will only be able to rename one variable when there are multiple return values. So I felt that I should just check here before continuing on that path since I didn't feel that the effort to implement it brings much value. of course it might in majority of scenarios where the return values are value, error where we can prompt to rename the value variable. But then I would guess we need to suggest a better name for error variable, like err with a suffix if there is a name conflict.


CAMEL-14068 Retry subscribe to Salesforce event when server returns 503 Server too busy

Currently we are using the Salesforce component version 2.23.4.
Every 3 hours the component needs to resubscribe due to expiring token (expected behavior). Sometimes however it fails on:
org.apache.camel.component.salesforce.api.SalesforceException: Error subscribing to /event/CustomerLegalEntity__e: 403:denied_by_security_policy:subscribe_deniedorg.apache.camel.component.salesforce.api.SalesforceException: Error subscribing to /event/CustomerLegalEntity__e: 403:denied_by_security_policy:subscribe_denied at org.apache.camel.component.salesforce.internal.streaming.SubscriptionHelper$7.onMessage(SubscriptionHelper.java:406)

It turns out (not in the logging) that the server returns additional information:
{ext={sfdc={failureReason=503::Server is too busy. Please try your request again later.}}, ...}

As described in the Salesforce documentation here: https://developer.salesforce.com/docs/atlas.en-us.api_streaming.meta/api_streaming/streaming_handling_errors.htm
the 503 Server too busy is a documented error that should/can be retried.
It would greatly add to the resilience of our system if the Salesforce component would automatically retry a subscribe if failed due to a temporary error. The same backoff mechanism as for the reconnect could apply.

Thanks, merged on master

Thanks!


s/Github/GitHub/

Quick doc patch.

Thanks for the contribution! This has been submitted as
https://github.com/google/bazel/commit/ad98ffdfde030a07c309a0f6c76a97ead6306bf5

:metal:


Development of Seedlot Review step 6 - Orchard

Describe the task
Create FE of Seedlot Review Step 6 (Orchard) and integrate endpoints as you go along.
Figma design here:  https://www.figma.com/file/ROgZJQezb1DDG4AvEgjtpw/Registration-Process----Reviewer?type=design&node-id=2744-284956&mode=design&t=XxxbormmPMPHmENb-4

Acceptance Criteria

[ ] Developed as per Figma
[ ] relevant architecture documentation updated
[ ] tested and merged
[ ] Develop floater save button place holder - design to be tested by TSC staff

figma link and image added


Add Bemuse project to examples.

Bemuse is an open source web-based rhythm game. I have been maintaining this game for 3 years (since webpack 1.4, React 0.13.2).
The user interface outside the game is built with React and Redux. It has been upgraded to the latest React version, 16.2.0, with the help of contributors, most notably @resir014.
I hope this project will be useful as an example of a project that’s been maintained over the years, and went through several big changes (Ractive → Vue → React).
Happy new year :D

If your PR references an existing issue, please add the issue number below:
(no issue referenced)

Thanks, but we'd like to keep this page focused on React. No Redux.


Better DataAll session Timeout handling

Short user time-out session. We are being logged out of DataAll (roughly 15 mins) and there is no warning or option to extend the session to the user before the system log out the user.

Hi Matt, thank you for the interest and for the issues you raised :)  I have edited the title of the issue a bit. I believe that in this issue you are asking for 2 things:
1. Increase users session duration.
When logging in, users get an access token and ID token that last for 1h, this are the default values for Cognito; but they can be increased to a maximum of 1day. You can check this configuration in the Cognito user pool deployed by data.all:

I think your issue comes from the platform being stale for >10 mins of time. In this case something that might be causing those "seems logged out" issues is the time to pause compute capacity in RDS (default for RDS is 5mins, in data.all 10mins). This is set to minimize costs and take advantage of serverless technologies, but the downside is that it takes time to warm up, you can extend the pause something bigger than 10 minutes so it will be warm-up.

I am going to do some experimentation on my end to confirm the issue. Let me know if this seems like your issue.
2. Better notification or workflow for logged out users
This is definitely an interesting feature that we would include in the open-source repository. You are more than welcome to start developing and we will assist you with any question and with the testing and merging conflicts. Otherwise this is a cool feature, but at the moment we have several items in our backlog so we cannot commit to implement it in the short-term.


my grave saying i can’t access it

i died and my game crashed on my way to my grave on my server, i got to it and it says i can’t access it only the owner and then my username can access it but it won’t allow me

also i can’t figure out how to make graves available to access by anyone

I can't think of anything that might cause the access issue, unless maybe if you're using a cracked client? Was the crash related to gravestones at all?
Making graves accessible to everyone should be as simple as setting gravestone_accessible_owner_only to false in the configs. If this is a multiplayer server, make sure to do this serverside (clients should sync any configs that are serverside)

crash was because of changing shaders as my laptop isn’t that great, not using a cracked client, thank you, i’ll look at trying to change that later :)

Tested it out and I've managed to recreate this! I'll get it fixed soon. In the meantime, creative mode players can break any grave and it will drop the contents, so if you have access to cheats you can get your stuff back without having to wait for me to update the mod.

Can confirm as well, I have one player who is affected by this issue on my server. The creative workaround worked like a charm. 🙏

since making that comment yesterday i also died in the nether with no crashed or anything and got back yo my stuff in less than a minute with it still telling me i wasn’t able to access my stuff

Did you leave the world and rejoin at all? This seems to happen whenever someone disconnects and reconnects, not just with crashes. (If so it should be fixed once I release the next update, but it's better to be sure)

the time of dying in the nether i had not left the world at all

I can't recreate that, but I'm fairly sure if it's still saying the grave is yours, it's a side effect of this issue. More specifically, I'm just really stupid and didn't understand how to properly compare UUIDs, so the saved UUID of the gravestone would sometimes not appear equal to the UUID of the player. If it's saved your name, it's most likely also saved your UUID, so it should just be this again.

Update released!

awesome! i’ll update my server with friends soon! :)


update(core): Replace cp command with cpy-cli package

To make build commands available in shell where cp is not available (i.e. Windows cmd)

Hey! Just gonna leave this for now, mostly because I don't want to spend any time trying to decide if we should add it as a dependency or just use npx like you have here. Thanks though!


Feat prms MMR to mf6

Codecov Report

Merging #140 (f4b2cfd) into main (43ffcbc) will decrease coverage by 0.59%.
The diff coverage is 71.15%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #140      +/-   ##
==========================================
- Coverage   82.33%   81.75%   -0.59%
==========================================
  Files          40       42       +2
  Lines        4982     5190     +208
==========================================
+ Hits         4102     4243     +141
- Misses        880      947      +67

Impacted Files
Coverage Δ

pynhm/utils/dis_hru.py
0.00% <0.00%> (ø)

pynhm/utils/prms_to_mf6.py
89.74% <89.74%> (ø)

pynhm/base/meta.py
81.30% <100.00%> (+1.51%)
:arrow_up:

... and 3 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Decluttering the map for turn restrictions

The data density of detected restrictions is starting to get high enough that the experience of interaction has come down. Its hard to identify missing data on OSM and verify existing data with imagery.

https://mapbox.github.io/osm-navigation-map/?days=990#17.3/37.77535/-122.41879
How can we improve the UX of the tool? The filter interface on Mapillary has gotten much better in the meantime that makes it easier to browse through the data, but still far from ideal.
What are low hanging fruits here?
cc @rasagy @kepta @batpad

@planemad we add a slider to increase of decrease the density of items shown in the screen.


Podcast with Ben & Dion

New podcast from I/O chats.
Target Live Date: Next deploy.

[ ] This has been reviewed and approved by @surma
[x] I have run gulp test locally and all tests pass.

I'm getting three warnings for misspelled 'ajax'. Two are 'Ajaxian', which is a brand name, and two are part of a URL, so the case shouldn't change.

[x] I have added the appropriate type-something label.
[x] I've staged the site and manually verified that my content displays correctly.

CC: @petele

Whoops!
There were 3 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/shows/http203/podcast/io-chats-ben-dion.md#L22 - Common typo found: 'Ajax' Should it be 'AJAX'? See: https://developers.google.com/style/word-list#ajax
src/content/en/shows/http203/podcast/io-chats-ben-dion.md#L22 - Common typo found: 'ajax' Should it be 'AJAX'? See: https://developers.google.com/style/word-list#ajax
src/content/en/shows/http203/podcast/io-chats-ben-dion.md#L22 - Common typo found: 'ajax' Should it be 'AJAX'? See: https://developers.google.com/style/word-list#ajax

:tada: This has been pushed live to https://developers.google.com/web/


refator: remove inline scripts from nocalhost addon

make cuefmt

refactor inline scripts according to https://docs.dagger.io/1226/coding-style#dont-inline-scripts

Inline scripts content is hard to maintain.
Meanwhile, copying script files into container is so boring, maybe we can wrap the logic into an independent CUE module, and call that module instead.

maybe we can wrap the logic into an independent CUE module, and call that module instead.

I don't get it. Can you give an example?


It would be great to have instructions for setting up on GKE

For most kubernetes based apps, it is easy to play around with them using Google Container Engine (GKE) because it is the easiest way to setup a native k8s cluster.  It would be nice to have simple instructions similar to https://github.com/projectodd/incubator-openwhisk-deploy-kube/blob/simplify-deployment-openshift/resources/README-GKE.md for how to setup kubeless on GKE.

Yes good point Jim, we need to write those docs.


Fix issues in BBE generation

Purpose
$title.
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #21885 into stage will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            stage   #21885   +/-   ##
=======================================
  Coverage   14.59%   14.59%
=======================================
  Files          51       51
  Lines        1411     1411
  Branches      219      219
=======================================
  Hits          206      206
  Misses       1189     1189
  Partials       16       16

Continue to review full report at Codecov.

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


GPII-3334: Add terraform fmt check to pipeline

As we don't seem to be very good at running terraform fmt before comitting code, I propose to add this step to our CI pipeline.
It will be annoying at first, and everyone will hate me for this :), but life is hard.
update: We even already have ticket for this GPII-3334

LGTM

thanks for reviews, @natarajaya I've added lint stage & renamed step as suggested, I agree it makes more sense this way

Small problem here: the new Job doesn't have a Tag. This works today because I allowed the AWS builder to build untagged Jobs until all the gcp/ and common/ stuff was deployed.
I will edit the config for the AWS Specific Builder and add a Tag to the lint Job when I fix after_script for GPII-3355.


Let build fail when using nodejs built in module

Description
I understand that vite does not provide nodejs polyfill by default.
Suggested solution
So, what if we make the build fail if we use nodejs modules? In cra, the build fails when using the fs module. But it doesn't fail with vite.
Some people apply polyfills, so how about giving an option like this: vite build no-polyfill
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

I think maybe a warning is enough instead of failing the build. Vite currently implements a mocked module for the builtin modules so that it emits a warning in runtime, maybe it make sense to surface it during build too.

Thanks for the reply. Thanks to vite, the development environment has improved.
I have introduced vite to an existing project that is quite large. And I'm not sure all the libraries I installed don't use nodejs built in modules. As explained in the official documentation, do not use nodejs built in modules in libraries. But like you said, I'd like to know if at least some library has this problem or not. @bluwy


Basic attributes don't work

The following code doesn't work.
class C:

     def __init__(self):
         self.e = None

c = C()
print(c.e)

codon run test.py
produces the following error message:
test_codon.py:4:10-14: error: 'C' object has no attribute 'e'
╰─ test_codon.py:6:5-8: error: during the realization of __init__(self: C)

Discussed in #150: currently Codon needs to know the attributes for the class, but we do have automatic detection of them implemented and will be incorporating it soon.


proc: add support for separate debug info

To save disk space, some distributions strip the debugging information
from the binaries, putting it in separate files, usually distributed in
separate packages.
To locate the file containing the debug information for a certain
binary, an ELF note named ".note.gnu.build-id" is added to the latter,
which contains a header and a build identification. This identification
can be used to compose a path with this form:
/usr/lib/debug/.build-id/BUILDID[:2]/BUILDID[2:].debug
With this patch, if Delve can't find the debug information in the main
binary, it'll try to locate and parse ".note.gnu.build-id", to compose
and attempt to open a path with the format described above.
Fixes #1206

As the Go compiler doesn't add a ".note.gnu.build-id" note by itself, writing a test for this for would require specialized linking, stripping, and composing of the path. I don't think that's feasible without important changes in the test framework, and that probably is not worth for such a small change.

@slp any chance you'll be able to take care of the rebase / feedback?

@derekparker  Sure, I'll do that ASAP. Probably tomorrow or Monday.