The Build pipeline screens are completely differemt.

The Pipeline section screenshots don't match with the new default YAML based build. Please show the default YAML screen shots. Also how does one change the Agent pool to the one running on the AzureStack host.

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

ID: a0e54ed0-e139-1569-adc8-1833de468eb9
Version Independent ID: 0b03d258-e1ee-e59b-997f-e9096c6f53e2
Content: Deploy apps to Azure and Azure Stack
Content Source: azure-stack/user/azure-stack-solution-pipeline.md
Service: azure-stack
GitHub Login: @BryanLa
Microsoft Alias: bryanla

@kprpg Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Thanks for your feedback, @kprpg. I'm looking into this. I'll add another comment as soon as we resolve..

@kprpg - Can you access and review these proposed changes: https://review.docs.microsoft.com/en-us/azure-stack/user/azure-stack-solution-pipeline?view=azs-1802&branch=pr-en-us-1446#create-a-build-pipeline-and-run-a-build, and see whether it fits with your experience and expectations?
You can also select "Show assistant" to the right of the YAML, which opens a form that helps you add tasks and steps. Do you think it would be more useful to show that process, or just give the code as now?
Note that if you have the Preview "multi-stage pipeline" experience turned on, things will look and act different. We won't document that until it becomes the default.
Currently, it's also possible to write Release pipelines in YAML, but that isn't the default experience, so I've left that section mostly as is with a few small updates.
Thanks very much for providing your opinion on these changes if you can. @BryanLa FYI.

@kprpg - The build pipeline changes are in PR https://github.com/MicrosoftDocs/azure-stack-docs-pr/pull/1446. Please take a look and let us know if anything more is needed. Thanks. @BryanLa FYI.

Thanks @v-thepet.
@kprpg - when you get a chance, please let us know if this resolves the doc issue you cited.
@TravisCragg-MSFT - I believe this one can be closed now, unless the "how does one change the Agent pool to the one running on the AzureStack host" question is still outstanding.

#please-close


how to type object in for-in

I have an object type:
interface ShortUrlParam {
    openid: string;
    avatar: string;
    nickname: string;
}

const param: ShortUrlParam = {
    openid: 'abc123',
    avatar: '',
    nickname: 'wenzi'
}

let query = '';
for(let key in param) {
    query += `&${key}=${encodeURIComponent(param[key])}`; // error tip
}

in param[key], error tip is:

Element implicitly has an 'any' type because expression of type 'string' can't be used to index type 'ShortUrlParam'.
No index signature with a parameter of type 'string' was found on type 'ShortUrlParam'.ts(7053)

I have two programmes, but not perfect.
1. redefine interface ShortUrlParam
interface ShortUrlParam {
    openid: string;
    avatar: string;
    nickname: string;
    [key: string]: string;
}

2. param as any
for(let key in param) {
    query += `&${key}=${encodeURIComponent((param as any)[key])}`; // error tip
}

my question is there has any better solution?

It looks like this is a question rather than a bug report. This issue tracker is for tracking bugs and active work on TypeScript itself, rather than a general forum for programmers using TypeScript to get help or ask questions.
You can ask questions on sites like Stack Overflow. We are not able to provide one-on-one support on the issue tracker. Please read the issue template carefully - it has important information on what kinds of reports can be acted on here, as well as links to useful TypeScript resources. Thanks!

thank you, I go ask in StackOverflow


Mechanism to compile pages from memory or an alternate source path

This should wait until #276 is done.


Timers.refresh() method does not work using jest.advanceTimersByTime() method

🐛 Bug Report
Jest fake timers does not support .refresh() method from Node.js v10.2-latest.
This leads to not be able to test code for Node.js projects.

To Reproduce
Steps to reproduce the behavior:
Try to write test on code, which uses .refresh method.
Or take a look on repl.it
Expected behavior
We would be able to write tests which have .refresh inside using fake timers.
Based on my investigation, jest fake timers are using @sinon/fake-timers. That library introduce .refresh() support in version v7, but fake timers are still on v6.
Link to repl or repo (highly encouraged)
repl.it
envinfo
  System:
    OS: macOS 10.15.7
    CPU: (8) x64 Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz
  Binaries:
    Node: 12.18.4 - ~/.nvm/versions/node/v12.18.4/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 6.14.6 - ~/.nvm/versions/node/v12.18.4/bin/npm
  npmPackages:
    jest: ^26.6.3 => 26.6.3

As you note, it's a dependency issue for modern timers 👍 Legacy timers would need an implementation if anybody is up for it


A suggestion

If it would be possible, a feature to use custom skins would be cool so you can for example have Portal (the game) portals

I need to get the base down before I focus on visuals, prolly will do that though


Add dependencybot to update go.mod

The TrustBloc VCS uses Hyperledger Aries Framework Go as a library for VC related operations. The AFG is under active development, and usually, changes affect VCS build. Currently, the team updates the dependency version manually. We should explore the option to add dependencybot to update the version.
Reference from TrustBloc Wallet repo https://github.com/trustbloc/wallet/blob/main/.github/dependabot.yml#L60

@vkubiv you can use the official Dependabot documentation as a reference as well

Dependency bot creating pull requests separately for each go.mod file. But this will not work for vcs project, just updating root go.mod without updating at same time cmd and bdd will fail PR checks.


readme: Move the Ruby buildpack to the compatible list

Since it's now compatible as of heroku/heroku-buildpack-ruby#1013.

Cool, good idea.
The timestamps stop working after the ruby buildpack runs,  not sure why, but I mean to investigate one of these days.  It's usually our last buildpack anyway, but would be nice if for the few buildpacks we stick after if the timestamps still showed.

Ah I didn't know about that.
I took a look at why, using:
git clone https://github.com/heroku/ruby-getting-started testcase-timestamps-ruby && cd $_
h create
heroku buildpacks:add edmorley/timestamps
heroku buildpacks:add heroku/ruby
heroku buildpacks:add heroku-community/inline
echo 'echo inline' > bin/detect
echo -e 'env | sort\ncat /tmp/codon/tmp/buildpacks/*/export' > bin/compile
chmod +x bin/detect bin/compile
git add -A && git commit -m '.' && git push heroku master

Which outputs:
remote: -----> Timestamp app detected
remote: -----> Ruby app detected
remote: 2020-08-20 19:22:58 -----> Installing bundler 1.17.3
...
remote: 2020-08-20 19:23:08 -----> Buildpack compile completed in 10s
remote:
remote: -----> inline app detected
...
remote: PATH=/tmp/build_41686243/bin:/tmp/build_41686243/vendor/bundle/bin:/tmp/build_41686243/vendor/bundle/ruby/2.6.0/bin:/tmp/build_41686243/vendor/ruby-2.6.6/bin:/tmp/codon/tmp/buildpacks/50d5eddf222a9b7326028041d4e6509f915ccf2c/vendor/ruby/heroku-18/bin/:/usr/local/bin:/usr/bin:/bin:/tmp/build_41686243/bin:/tmp/codon/tmp/buildpacks/a568b51b1100125b8f91020e7ae69d14a7b05600/timestamper:/usr/local/bin:/usr/bin:/bin
remote: PWD=/tmp/codon/tmp/buildpacks/f2fbc9b1d342d3ce7c18a3bfecd6d9d722197713
...
remote: TIMESTAMPER_DIR=/tmp/codon/tmp/buildpacks/a568b51b1100125b8f91020e7ae69d14a7b05600/timestamper
...
remote: export PATH="/tmp/build_41686243/bin:/tmp/build_41686243/vendor/bundle/bin:/tmp/build_41686243/vendor/bundle/ruby/2.6.0/bin:/tmp/build_41686243/vendor/ruby-2.6.6/bin:/tmp/codon/tmp/buildpacks/50d5eddf222a9b7326028041d4e6509f915ccf2c/vendor/ruby/heroku-18/bin/:/usr/local/bin:/usr/bin:/bin:/tmp/build_41686243/bin:$PATH"
remote: export GEM_PATH="/tmp/build_41686243/vendor/bundle/ruby/2.6.0:$GEM_PATH"
remote: export LANG="${LANG:-en_US.UTF-8}"
remote: export PATH="/tmp/codon/tmp/buildpacks/a568b51b1100125b8f91020e7ae69d14a7b05600/timestamper:${PATH}"
remote: export TIMESTAMPER_DIR='/tmp/codon/tmp/buildpacks/a568b51b1100125b8f91020e7ae69d14a7b05600/timestamper'
remote: -----> Discovering process types

So the issue here is that the Ruby buildpack's export file is broken, in that instead of doing something like:
export PATH="/tmp/build_41686243/bin:/tmp/build_41686243/vendor/bundle/bin:/tmp/build_41686243/vendor/bundle/ruby/2.6.0/bin:/tmp/build_41686243/vendor/ruby-2.6.6/bin:/tmp/codon/tmp/buildpacks/50d5eddf222a9b7326028041d4e6509f915ccf2c/vendor/ruby/heroku-18/bin/:$PATH"

It's hardcoding the original PATH (/usr/local/bin:/usr/bin:/bin) back in the export file:
export PATH="/tmp/build_41686243/bin:/tmp/build_41686243/vendor/bundle/bin:/tmp/build_41686243/vendor/bundle/ruby/2.6.0/bin:/tmp/build_41686243/vendor/ruby-2.6.6/bin:/tmp/codon/tmp/buildpacks/50d5eddf222a9b7326028041d4e6509f915ccf2c/vendor/ruby/heroku-18/bin/:/usr/local/bin:/usr/bin:/bin:/tmp/build_41686243/bin:$PATH"

Which means that the real bash shadows the wrapper script in the timestamper buildpack.
This appears to originate from:
https://github.com/heroku/heroku-buildpack-ruby/blob/858adcbe3a53997192be9f12aa475c109fc490e9/lib/language_pack/ruby.rb#L406-L408


Update sbt-scoverage to 2.0.9

About this PR
📦 Updates org.scoverage:sbt-scoverage from 1.9.3 to 2.0.9 ⚠
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scoverage", artifactId = "sbt-scoverage" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scoverage", artifactId = "sbt-scoverage" }
}]

labels: sbt-plugin-update, early-semver-major, semver-spec-major, commit-count:1

Codecov Report
Patch coverage has no change and project coverage change: +0.32% :tada:

Comparison is base (2eb2ffe) 91.06% compared to head (ba2a4b3) 91.38%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #124      +/-   ##
==========================================
+ Coverage   91.06%   91.38%   +0.32%
==========================================
  Files          10       10
  Lines         347      360      +13
  Branches       14       35      +21
==========================================
+ Hits          316      329      +13
  Misses         31       31

see 7 files with indirect coverage changes

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


Add e2e test for testing user data w/ exception

Test to demonstrate #195

This test assumes that testServer.events[0].event.data.user is the correct place to look for user data, but that is mostly a guess since this seems to be the first e2e test that looks for user data.

Thx!

Thanks for the investigation, superseded by https://github.com/getsentry/sentry-electron/pull/242


registering devices for no db option

There is no need to register devices when using a file as the db. The current implementation in tango_launcher tried to fetch the device list from the real tango DB and went on to start the device server using a default test name test/nodb/tangodeviceserver which does not match devices registered in the file.
current implementation
https://github.com/ska-sa/tango-simlib/blob/b72e486a1eff97bdb9dae6f469600c355f30e9d1/tango_simlib/tango_sim_generator.py#L460-L477
This PR addresses the run time error encountered (image below) when the kat-tango-sim-launcher script executes.

@lvdheever Please review again, after latest changes.

@ajoubertza thanks for pointing out the need for the PR description update.


Change dayNames not working [?]

First of all nice project you made! I'm struggling to just change interface day names, I'm passing an array of names I want show in calendar but it is not changing. What am i loosing ?

but it keep en-us names:

Hi again, thank you.
It should be fixed in latest release.

You can check it out here

Hi again, thank you.
First make sure the config object is same in all renders.

The configuration object should remain unchanged during React re-renders to maintain consistent behavior, and it's recommended to declare it outside the component to avoid unnecessary re-creations.

to do that just move config outside the component.
It should be fixed in latest release.

I'll give a try. your link is broken.


Renamed u8timeseries to darts

@endrjuskr @hrzn is it going to work with Sphinx?

So we our library on pypi is called u8darts, but package is called darts. Will it create conflict?

Normally having the package in pypi called u8darts shouldn't cause conflicts, but we actually might be able to get the name darts so let's wait a week:


Invisibly analytics adapter updated docs

Hey @bretg,
Could you please merge this PR? The earlier version is failing on download action.
Let me know if any additional inputs are required.

This wasn't quite the correct solution -- I fixed in https://github.com/prebid/prebid.github.io/pull/1647


Add info around dropdb and createdb

Is your feature request related to a problem? Please describe.
I was recently trying to use the migrationsDir config option to handle migrations but when running ESLint, it would just hang indefinitely with no message. Looking into the code I found that as part of the migrations, the dropdb and createdb binaries are used. After installing these everything worked fine.
Describe the solution you'd like
It's probably worth mentioning in the docs that these binaries need exist in order for migrations to work, or throw an error if they don't exist.

Usually when installing postgresql, dropdb and createdb should be available. I haven't stumbled across a case where they weren't in PATH.
I agree that it should probably be mentioned in the docs. PRs are welcome :)

Thanks for the quick response. I was thinking that an alternative solution would be to switch these calls to use CREATE/DROP DATABASE SQL commands instead. This would mean the postgresql binaries wouldn't need to be installed at all in order to use this tool. Do you think this is a viable option? I'd be happy to raise a PR for this.
The situation I'm currently in is I'd like to use this in my company's CI pipeline, but there are heavy restrictions on what can be installed.

That would make sense, go for it :)


Allow more control from CameraActivity when hitting flashpoint

In my project I have an Activity extending CameraActivity. When the user hits the flashpoint (WillTakePicture event) I need to save the current device orientation (Azimuth, Pitch, Roll) and return it to the calling activity (using getResultIntent() to add extras).
I don't know if there's a better way to do this, or if it's not suitable at all. But you tell me ;)

I don't have a problem with what you're doing here. However, as I note in the project contribution guidelines, I cannot really accept significant pull requests directly. I will look to implement a similar hook that you can use.
And, if what you are seeking is covered by this bit of documentation, I may add an option to collect them for you, as that doesn't seem especially difficult.
Thanks!

Yes, the SensorManager data is exactly what I'm collecting. Would be great if that's already built-in.

I had thought that there might be EXIF headers for the sensor readings that I could fill in, but I cannot find any. So, I'm implementing a hook for you, reminiscent of your implementation. The differences:

The event name is PictureCaptureStartEvent instead of WillTakePicture.

I am not handling the event anywhere, so you can decide the thread policy that makes sense for you. Your code uses MAIN; I would think that you really want the event to be delivered on the posting thread, so that actual picture capture is blocked until you can schedule your work.

I went with buildResultIntent() instead of getResultIntent().

Addressed in v0.7.6 as noted above. Thanks for the suggestion!


⚠ BCR Bot has degraded performance

In cc46171, BCR Bot (https://bcr.alternativasinteligentes.com/BCRApi/web/chatbot) experienced degraded performance:

HTTP code: 200
Response time: 1206 ms

Resolved: BCR Bot performance has improved in 8ded660.


[BUG] Can not redirect when the URL has trailing slash

Environment

Operating System: Darwin
Node Version:     v18.20.2
Nuxt Version:     3.6.5
Nitro Version:    2.9.4
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      css, runtimeConfig, extends
Runtime Modules:  -
Build Modules:    -

Reproduction
I use ESV site for testing.
Go to below URLs to test.
http://localhost:3000/about-esv/contact-us/
http://localhost:3000/about-esv/faqs/esvconnect-licensing/#collapse-37-title
http://localhost:3000/licensing-coes/licence-training-assessments/cpd/
Below URLs without trailing slash is working:
http://localhost:3000/about-esv/contact-us/
http://localhost:3000/about-esv/faqs/esvconnect-licensing#collapse-37-title
http://localhost:3000/licensing-coes/licence-training-assessments/cpd/
Describe the bug
The endpoint /api/tide/page at @dpc-sdp/nuxt-ripple/server/api/tide/page.ts, is currently taking the route as is.
However Drupal endpoint will return 404 if the path is a redirect
https://content.esv.vic.gov.au/api/v1/route?site=13&path=/about-esv/contact-us/ => return 404
https://content.esv.vic.gov.au/api/v1/route?site=13&path=/about-esv/contact-us => return details
I have raised a quick PR to resolve it since it is affecting prod issue,
https://github.com/dpc-sdp/esv-vic-gov-au/pull/93
The solution is to remove the trailing slash
    const path = query.path !== '/' ? query.path.replace(/\/$/, '') : query.path
    const pageResponse = await tidePageApi.getPageByPath(
      path,
      query.site,
      {},
      headers,
      sectionId
    )

Let me know if this is a correct solution, and I am happy to raise a PR if needed.
Additional context
No response
Logs
No response
Final checks

[X] Read the contribution guide.
[X] Check existing discussions and issues.

@awset Thanks for this, we had a similar solution in Ripple 1 but were expecting it to be resolved at the back end this time (it hasn't been resolved). Would be happy to accept a PR for this, the solution you've suggested looks fine.

hi @waitingallday I have no permission to push my changes, so I attached below the patch file.
remove-trailing-slash.patch


feat: add the correct info to immobiliare.it and add more tags

[feat/immobiliare-labs afa0894] feat: add the correct info to immobiliare.it and add more tags.
Thanks

Hey, @JellyBellyDev can you please reduce the number of tags to something below 20. Having a large number of tags create some format problems on the README.
I added this rule to the CONTRIBUTING.md too.
Thank you for the understanding

@alessandromr done! ;)
Thank you so much for this wonderful initiative! ;)


Add wwe games to PKGJ

WWE games have a large target market and has been frequently requested. It would be great if these were added to the store.

This repository has nothing to do with PKGJ. And pull requests are not the place where to request these kind of things.


[FR] Replace markuplint with djlint

Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find a similar feature request

Problem statement
We currently rely on markuplint - a node-based linter for html - for linting django templates. This means we are requiring a whole lot of js packages just for linting.Django
Suggested solution
There is a tool built for django templates specifically called djlint. We could replace the node dependencies with it and merge the HTML step with the pre-commit step.Django
Describe alternatives you've considered
The current way works.
Examples of other systems
No response
Do you want to develop this?

[X] I want to develop this.

Thoughts @SchrodingersGat?

Does it also support JavaScript linting? We use jslint for that currently I believe

I think that should be done in a second step; the diff for the templates alone is big enough


Header

Create Header with personal image, page title and nav bar

Header created and merged in pull
https://github.com/PeaFunk3000/portfolio/pull/3


feat: experimental add module from DevTools

Merge this for now to iterate faster, it will be shipped under the experimental flag.


Error => {"extra": -1330794744, "what": -1003}

Error while playing sound from specific URLs. I get the following log for onError function : **{"extra": -1330794744, "what": -1003} **.

Is your issue with...

[ ] iOS
[x] Android
[ ] Windows

Are you using...

[x] React Native CLI (e.g. react-native run-android)
[ ] Expo
[ ] Other: (please specify)

Which versions are you using?

React Native Sound: 0.11.1
React Native: 0.65
iOS: x
Android: 7.1.2
Windows: x

Does the problem occur on...

[ ] Simulator
[x] Device

If your problem is happening on a device, which device?

Device:

I am also getting this error on Android 12

this error is still persisting on current android versions

LOG  Failed to load the sound {"extra": -2147483648, "what": 1}
LOG  Failed to load the sound {"extra": -2147483648, "what": 1}
LOG  Failed to load the sound {"extra": -2147483648, "what": 1}
LOG  Failed to load the sound {"extra": -2147483648, "what": 1}
LOG  Failed to load the sound {"extra": -2147483648, "what": 1}
LOG  Failed to load the sound {"extra": -2147483648, "what": 1}
LOG  Failed to load the sound {"extra": -2147483648, "what": 1}

import React, { useState } from 'react';
import { View, Text, Button, StyleSheet } from 'react-native';
import Sound from 'react-native-sound';

const AudioPlayerCard = () => {
  const [sound, setSound] = useState(null);
  const [isPlaying, setIsPlaying] = useState(false);

  const audioUrl = 'https://2cf8-2405-201-a005-e965-1526-ca68-f154-8f82.ngrok-free.app/assets/quran.wav';

  const loadSound = () => {
    const newSound = new Sound(audioUrl, Sound.MAIN_BUNDLE, (error) => {
      if (error) {
        console.log('Failed to load the sound', error);
        return;
      }
      setSound(newSound);
      playSound();
    });
  };

  const playSound = () => {
    if (sound) {
      sound.play((success) => {
        if (success) {
          console.log('Successfully finished playing');
        } else {
          console.log('Playback failed due to audio decoding errors');
        }
        setIsPlaying(false);
      });
      setIsPlaying(true);
    } else {
      loadSound();
    }
  };

  const pauseSound = () => {
    if (sound && isPlaying) {
      sound.pause();
      setIsPlaying(false);
    }
  };

  const stopSound = () => {
    if (sound) {
      sound.stop(() => {
        setIsPlaying(false);
      });
    }
  };

  return (
    <View style={styles.card}>
      <Text style={styles.title}>Audio Player</Text>
      <View style={styles.buttonContainer}>
        <Button title="Play" onPress={playSound} />
        <Button title="Pause" onPress={pauseSound} disabled={!isPlaying} />
        <Button title="Stop" onPress={stopSound} />
      </View>
    </View>
  );
};

const styles = StyleSheet.create({
  card: {
    padding: 20,
    margin: 10,
    backgroundColor: '#fff',
    borderRadius: 10,
    elevation: 3,
    shadowColor: '#000',
    shadowOffset: { width: 0, height: 2 },
    shadowOpacity: 0.1,
    shadowRadius: 4,
  },
  title: {
    fontSize: 18,
    marginBottom: 10,
    fontWeight: 'bold',
  },
  buttonContainer: {
    flexDirection: 'row',
    justifyContent: 'space-between',
  },
});

export default AudioPlayerCard;


Comments with @~~ cause compiler errors when accessing member of Generic

TypeScript Version:  2.4.0
Code
  export interface PureAction <ACTION_TYPE> extends Redux.Action {
    readonly type: ACTION_TYPE
  }

  export interface PayloadAction <ACTION_TYPE, PAYLOAD> extends Redux.Action {
    readonly type: ACTION_TYPE
    readonly payload: PAYLOAD
  }

  export type Action = PureAction<any> | PayloadAction<any, any>

  export function createActionCreator <ACTION extends PayloadAction<ACTION['type'], ACTION['payload']>>(type: ACTION['type']): (payload: ACTION['payload']) => ACTION
  export function createActionCreator <ACTION extends PureAction<ACTION['type']>>(type: ACTION['type']): () => ACTION
  export function createActionCreator <ACTION extends PayloadAction<ACTION['type'], ACTION['payload']>>(type: ACTION['type']): (payload: ACTION['payload']) => ACTION {
    return (payload: any) => {
      if (payload == null) {
        return ({
          type,
        } as ACTION)
      }
      return ({
        type,
        payload,
      } as ACTION)
    }
  }

  type Reducer<STATE, ACTION> = (state: STATE, action: ACTION) => STATE
  /**
   *
   * test
   *
   * @export
   * @class MappedReducer
   * @template STATE
   * @template ACTION
   * @template Action
   * @template ACTION_TYPE
   * @template ACTION
   */
  export class MappedReducer<STATE, ACTION extends Action = Action, ACTION_TYPE extends ACTION['type'] = ACTION['type']> {
  }

Expected behavior:
No errors
Actual behavior:
[ts] Type '"type"' cannot be used to index type 'ACTION'. error occurred twice on Line 41:
export class MappedReducer<STATE, ACTION extends Action = Action, ACTION_TYPE extends ACTION['type']=ACTION['type']>
It works well if I delete all comment lines with @~~ like @export and @template.

It should be duplicate of https://github.com/Microsoft/TypeScript/issues/16358, fixed by https://github.com/Microsoft/TypeScript/pull/16413 (v2.4.1).
For now you can try typescript@next, it works fine without errors.

@ikatyang Thanks!


Handle quotation of env var value

It is somewhat common to have quoted values on env files:
ANSWER="42"

and this should work fine
(env :ANSWER) # => "42"

instead this yields:
(env :ANSWER) # => "\"42\""

As a side-effect of not handling quotations, quoted env vars that hold equal signs totally break the code:
SOMETHING="this=that"

Then
CompilerException java.lang.IllegalArgumentException: Vector arg to map conj must be a pair

Fixed with release of version 1.1.0


Beta into alpha

Description
Merge beta changes into the alpha branch
Breaking Changes
None
JIRA Link
N/A
Checklist

[ ] Updated the Readme.md (if required) ?

:tada: This PR is included in version 19.0.0-alpha.4 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 19.0.0-beta.5 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 19.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


I just want to run it behind some authentication.

I just want to run it behind some authentication. How can I do that?

I just want to run it behind some authentication. How can I do that?

You can add an auth token. The readme.md explains how to obtain the auth token, and then you can fill in the token in service/.env

Hi @zhpd , No I mean I want to register and force login into the system


chore: Update welcome.yml

Describe the changes you've made
Replacing EHub Welcome action by SCP.
Using checkout@v1 instead of v3.
Type of change
What sort of change have you made:

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update.
[ ] This change requires a documentation update

How Has This Been Tested?

Code of Conduct

[ x] I agree to follow this project's Code of Conduct

The action still crashes till the PR is merged :

Im not on my computer, but there is some tools to test action localy. Will test and remove the WIP mention once it's ok on my side. ✌

Hi back.
The welcome action is working locally:

The prettier one still bugged, but we can merge this PR.

Hi back,
The welcome github workflow is working locally:

The Prettier job is still bugged but it's maybe caused by a wrong prettier local file. We can merge this PR.


[SPARK-43711][SPARK-44372][CONNECT][PS][TESTS] Clear message for Spark ML dependent tests.

What changes were proposed in this pull request?
Similar to https://github.com/apache/spark/pull/42955, this PR proposes to correct the message for Spark ML only tests from Spark Connect.
Why are the changes needed?
Among Spark ML dependent tests, there are some edge tests that can only be tested using Spark ML features. We need to be clearer about why these cannot be tested.
Does this PR introduce any user-facing change?
No, it's test-only.
How was this patch tested?
Updated the existing tests.
Was this patch authored or co-authored using generative AI tooling?
No.

Merged to master for Apache Spark 4.0.0.


cursor.serialize is undefined v2

Example
var cursor = tree.select();
console.log(typeof cursor.serialize); // undefined

Hello @kevbook, I cannot reproduce your issue. The same code as yours produce "function" on the latest dev version.

I am sorry, Its not an issue. I have npm v3.2x. It flattens the dependencies, and somehow messes up the dist build.

Ok. Should I check with npm v3 just to be sure something is not broken?

npm v3 is in beta. Its not an issue with your package. NPM is just beta.


Why is bottle not listed in github explore list for web frameworks.

I was browsing through github explore list at https://github.com/showcases/web-application-frameworks and could not find bottle there. Projects having less stars and forks than bottle are listed there but not bottle. Why so?
I filed it here because it is pretty important and limits exposure of bottle to a more wider audience.

https://github.com/blog/1802-showcasing-interesting-projects-in-explore
"We take the most interesting trending repositories and curate lists to explore by topic"
"On a showcase page, you'll find the full list of repositories that we're showcasing, including why we think they're special."
Maybe Bottle is not "trendy" enough yet for the Github folks !?


Icons fonts

Are you planning to add font icons support as FontAwesome for example?

No, but if you want an icon font with Simple Icons glyphs you can generate one with IcoMoon. There are good reasons why I won’t be offering an icon font:
Performance issues with icon fonts

Your user needs to download every single glyph before one can be rendered. For Simple Icons that means 300 brand icons (~100kB), most of which you’re probably not using.
Your user is caching the entire glyph set in a single file, so changing one glyph invalidates the whole cache.
HTTP/2 (now at 80% support) favours multiple, small files (like SVGs).

Accessibility issues with icon fonts

If an icon font can’t or won’t load, there’s no graceful fallback (unless you use custom ligatures).
You have no control over users with user stylesheets, which may override your icon fonts (e.g. dyslexic people using weighted fonts).
Screen readers will not read icon font glyphs (unless you add aria-label attributes to your icon elements).

I'll just point out this project (as seen in #478) for anyone interested in this


Unsubscribe from notifications

Currently there is no obvious way for subscribers to any of the controller/devices-endpoints (like "notify-controller-attribute-value-changes") to unsubscribe again and will continue receiving messages until all controllers are removed via the "remove-controller"-endpoint. In BasicServices there exists the "end-subscription"-endpoint, should this be used to unsubscribe subscribers from specific subscriptions of IndividualServices in NP?

We now have seen in config file, that "end-subscription" in BasicServices is the correct function.
If this understanding is correct, please close the question.

Yes, I confirm that /v1/end-subscription shall be used for ending both basic and individual subscriptions.
If there would be no further need for clarification, please close the issue.


Make metadata work in windows again

you original code is all here:
https://github.com/musselwhizzle/Focus-Points/tree/feature/rderimay_windows
But all we need to know if add exiftool.exe and set the text to be multiline.

@musselwhizzle : You write "(Note, dont do this in the metadata viewer as converting to a table will throw out duplicate key names)"
This is already what is done in createParts used in the MetadataDialog.
So to me, we should use readMetadataAsTable there also and clean up the 3 unused methods.
Much quicker to build the Metadata View and known how much line there is etc.
I would do it part of this bug, as this is anyway the base of the problem here.
ok ?

as it stands (or stood before today commits) Metadata viewer showed duplicates if there were duplicates in the metadata (see screenshot attached).

It was nicer for the viewer since the values can change. I think it's nice. If you really hate those methods, move them from the utils to the showmetadata class.
Either way, I'd like for this feature to stay.

ah sorry misread the createParts(metaData)
I though it was anyway creating a table but it is creating a numbered table

https://github.com/musselwhizzle/Focus-Points/pull/64


License?

I just noticed it -- there is no license given. So what is it? :-) It would be great to see OSI compliant one.
Please note that once you add open source license GitHub Copilot will start harvesting the code violating the license. This is why I started migration of my projects (for now I am evaluating CodeBerg and so far so good).

What do you think about MIT licence?

I wish I have firm answer to that. I still don't have clear picture.
But my take is this -- if you want to open possibilities to everyone, then I would say MIT and alike is good choice. If you want however limit the possibilities of dishonest people (along with honest ones) then GPL and alike is better license.
Considering history of Identity Server (and my belief that as long as human beings write the code closed source does not scale in sense of development progress) I picked the more restrictive from GPL family -- AGPL. SSPL is very interesting but it is not OSI compliant.


Support TLS and authentication setup in the Helm chart

Implemented in https://github.com/pbufio/helm-charts/pull/1


Patch for issue #81

Issue 81:
https://github.com/Microsoft/msbuild/issues/81

Thank you for the fix.  The code change looks good, but can you please add a unit test to catch the original problem and amend your commit message to be more descriptive, following the guidelines at https://github.com/dotnet/coreclr/blob/master/Documentation/project-docs/contributing.md#commit-messages?

This was resolved with #240.


Request broader platform support for libpsl

I am trying to build libpsl on a POSIX platform other than Linux and it has issues when not using gcc. I am limited to c99. The initial build problems are as follows:
  CC       libpsl_la-psl.lo
        struct in6_addr addr6;
                        ^
"/home/randall/libpsl-0.21.5/src/psl.c", line 1611: error(163): incomplete type is not allowed

        return inet_pton(AF_INET, hostname, &addr) || inet_pton(AF_INET6, hostname, &addr6);
               ^
"/home/randall/libpsl-0.21.5/src/psl.c", line 1613: error(114): identifier "inet_pton" is undefined

2 errors detected in the compilation of "psl.c".

I cannot find inet_pton outside of very limited Linux distros. If is not in a common library. The incomplete type is probably just a missing include of in6.h. I can help with that, but intergrating with inet_pton is problematic.

I fixed the compile issue - it as actually simple - see commit 475e238 - but need DLL support, which is not according to what configure wants. I need to specify -Wshared.

This issue has been fixed in git master.


Missing space chars in the Getting Started command line examples

Some space characters aren't being rendered in the command line examples in the docs.

Connected with @11ty/eleventy-plugin-syntaxhighlight#43

Thank you. Disabling the AXE browser extension resolved the issue.


Show the concert limit (0/5) at the very beginning

as part of demo feedback https://github.com/taver-ca/Taver_Hackathon/issues/72
people felt surprised when the hard limit was hit
add a count on top of concert list as shown in screenshot.

So user won't be surprised when they hit the 5 artist limit when it comes

Hello @sam9116! I would like to work on this issue.

@GeorgiosDrivas sure! I will also send you the environment variable you will need to test this locally

do you have an email I can send the environment variable to?

Hello Sam, everything works perfect with the environment variable you sent me. I would like to make sure the UI is approved before making any pull requests, just to avoid making changes later. Attached you will find an image of the feature.

@GeorgiosDrivas that UI is good, I approve 👍

Closed with https://github.com/taver-ca/Taver_Hackathon/commit/87553ab7c5d98ebd3012498feb533395dc3997de


Error handling

We need to add error handling and recovery.
This is relevant for situations where something goes wrong with the connection to the video stream, or with the connection to the redis server. (If anyone can think of additional possible scenarios, please share in the comments)
The easy and naive way to do this would be to add an infinite loop which tries to re-do whichever task has failed. But it would be preferable to add the ability to shut down the component/whole pipeline after a certain period of time (if this is case, then we need to finish  #27 first)

With the PipelineManager you are able to stop and start any component whenever you need, about the errors that you mentioned, I think that every routine individually need to decide what to do on this cases.
Maybe we need to let the user know what is happening in each routine using its logger and add this type of mechanism/method to the PipelineManager.

I agree with both points - I think the routine should report the error and also try to handle it. If it reaches a certain configurable timeout (meaning that it can't handle the error) then it should delegate the handling to the PipelineManager.
I'd be happy to handle this issue once the PipelineManager is done :)


2023 Hyundai Sonata FW - Fingerprint

Can you provide the dongle id that these came from?

10702b68bb2db4f7On Apr 3, 2023, at 4:19 PM, Shane Smiskol @.***> wrote:
Can you provide the dongle id that these came from? It will be in Device settings.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Improve Gradle incremental build

Currently most of the tasks in the Gradle build script use inputs and outputs to control the up-to-date state of a task. When there is a change in the inputs a task does all its job unconditionally, even if there is a very small change - for example a modified JavaScript file. This behavior affects the incremental build times, especially with the Static Binding Generator (:asbg:xxx tasks).
A simple user scenario like Modify a JS file -> Build should take no more than 2 seconds at most.
To achieve this we may take advantage of the IncrementalTaskInputs feature of Gradle.

[x] For 2.4 Extend the AST parser to accept as an optional parameter a collection of JS files to process.
[x] For 2.4 Make the runAstParser task incremental and pass only the changed files to the parser.
[x] Extend the metadata generator with support for traversing DEX files. This way we will skip the collectAllJars task (~1s) which solely purpose is to feed the metadata generator.
[ ] Upgrade to Gradle 3.1 and Android Gradle Plugin 2.2.0. These new versions significantly improve the configuration phase times and overall build performance.

With this PR, there is a performance improvements ~4.5 seconds, when ast parser is ran incrementally.

This issue will be finished when there is a stable version of the android plugin for gradle, that supports incremental builds.


Fix/expanding by default

Rewrite updateDecendantState in order to propagate checked and expanded state on  the way to the top of the tree in FhiTreeViewSelectionComponent
Remove internal state descendantStateConfirmed as it is no longer needed

Use this file in example to test a large file in Tree View example component: alle_atckoder_som_items.json

@proand , @yleaxeman , @dafn , I don't know your routines for completing the PR, will you lot  handle the PR from here? Fhi statistikk is very eager to see the fix in a release as soon as possible :-)

Normally the assignee would complete the PR, but I can do it 😉


API Key scopes

Currently, all API keys are implicitly trusted. This is a reasonable assumption when all the keys are stored server-side (such as in Phoenix or Aurora). It's potentially dangerous now that we'll have keys that are stored in client-side code (like the mobile app) and traveling over public networks.
For example, any user that finds the application ID & key by either inspecting the application bundle or sniffing HTTP traffic would be able to spoof any request that the mobile app is able to do, which includes things like creating new API keys.
We should consider adding scopes to differentiate trusted clients and "untrusted" clients.
/cc @jonuy @angaither

+1 to scoping keys. But don't bundles get obfuscated in some sort of way? Though I guess anything could get decompiled and inspected. Same deal with https, I thought we'd be safe from general sniffing of packets since the data should be encrypted when being sent. But I guess there's probably something someone could do to act as a secure proxy and get the info they need.

Just for documentation, @jonuy did some digging and it turns out Android application bundles are pretty easy :sweat_smile: to decompile. Luckily it shouldn't be too hard to make a quick fix, and if we get around to using a standard OAuth implementation we should get this out of the box.

And #239 is deployed to Thor now. Hopefully no issues should crop up and we can deploy this to production sometime this week. :boat:
Some other things we should lock down to admin scoped keys:

[ ] Deleting users
[ ] Seeing full profile details of other users

(Anything else that comes to mind for anyone?)

This has been running on Thor's Northstar for a while now, with no reported errors. I think we should be safe to deploy to production, and then update our aurora and phoenix API keys to have admin scoped privileges.
(I'd like to open an endpoint for modifying key scopes as well, but obviously would want to limit that to only be available to admin keys so it's a bit of a chicken-and-egg problem. :hatching_chick:)

:+1:

:+1: As long as it's only the keys endpoint getting locked down for now (since the mobile apps use all the other endpoints), this sounds good to deploy.

:+1: As long as it's only the keys endpoint getting locked down for now (since the mobile apps use all the other endpoints), this sounds good to deploy.

Oooof, good to double-check. I accidentally locked down /signup-groups as well. Made a fix in #248.

The remaining task here is to allow admin-scoped keys (or the currently authenticated user, for their own profile) to see full profile information, by conditionally adding those fields to the response in the transformer.

This is done! The last piece (allowing authorized users without admin-scoped keys) to view their own profile data is ready to ship in #265.


[FR] Anonymous installs for MS Store

Description of the new feature / enhancement
Currently, installing anything, even free apps, from MS Store has a hard requirement of an account:
Found Windows Calculator [9WZDNCRFHVN5] Version Unknown
...
Verifying/Requesting package acquisition...
Verifying/Requesting package acquisition failed: no store account found

There is no reason these apps that aren't tired to any account-based subscription should be still gated behind account, i.e. apt-get or yum dont require an account just to fetch python.
Proposed technical implementation details
The actual content of free apps does not require any account tokens to be accessed. For example, if you copy the link for Windows Calculator app:
https://www.microsoft.com/en-us/p/windows-calculator/9wzdncrfhvn5
and use link-generator for MS store
https://store.rg-adguard.net
you will get temporal links to download not just the app package but all dependencies as well. They are not gated by any means. Therefore, same mechanism can be be used for any anonymous installs of free apps with winget.

Duplicate of #1585

Duplicate of #1585

shamefully closing stuff that isnt fixed


Simplesafe Integration stops working when upgrading to Home Assistant Core 0.106.0

The problem

Environment

Home Assistant release with the issue: 0.106.0
Last working Home Assistant release (if known): 0.105.5
Operating environment (Hass.io/Docker/Windows/etc.): Home Assistant core in docker on Ubuntu 18.04.
Integration causing this issue: Simplisafe
Link to integration documentation on our website:  https://www.home-assistant.io/integrations/simplisafe/

Problem-relevant configuration.yaml

Traceback/Error logs

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 215, in async_setup
    hass, self
  File "/usr/src/homeassistant/homeassistant/components/simplisafe/__init__.py", line 222, in async_setup_entry
    await simplisafe.async_init()
  File "/usr/src/homeassistant/homeassistant/components/simplisafe/__init__.py", line 482, in async_init
    self.systems = await self._api.get_systems()
  File "/usr/local/lib/python3.7/site-packages/simplipy/api.py", line 235, in get_systems
    self._request, self._get_subscription_data, system_data["location"]
  File "/usr/local/lib/python3.7/site-packages/simplipy/system/__init__.py", line 150, in __init__
    ] = self._generate_system_notification_objects()
  File "/usr/local/lib/python3.7/site-packages/simplipy/system/__init__.py", line 251, in _generate_system_notification_objects
    for raw_message in self._location_info["system"]["messages"]
KeyError: 'messages'

Additional information

Have tried removing and re-adding the integration.
Have tried stopping home assistant, deleting home-assistant_v2.db and restarting home assistant.

Closing as a duplicate of https://github.com/home-assistant/home-assistant/issues/32248. This is resolved in 0.106.1.


UID2-2738 retrieve access token and use to access okta

I have manually tested this against my admin instance running in AWS uid2-test account. To verify the refresh implementation, I reduced the access token lifetime in Okta and logged the request in the interceptor. I can confirm that it will use the same access token until it is within 10 minutes of expiration, at which point it will get a new one. I have not written any tests as of now. I think the change is simple enough to verify manually.

Approved - as long as the new values are configured for integ and prod environments.


statvfs

I'm looking for something similar to statvfs (that can e.g. be accesses through https://www.npmjs.com/package/diskusage). However, I would like to be able to run it from a worker (i.e. no native addons).

@ronag I acrualy do not understand your question/request ... can you be more precise on what you would expect?

Is it something like networkStats(), fsStats(), disksIO() in my package? What is missing?

Something like fsSize(). However, fsSize() does not include docker mounted volumes mounted. I basically want something like:
const disk = require('diskusage')

const { total, available } = await disk.check(dirPath)

@ronag Ok, understand partially ;-) Can you give me en example ... so what exactly you mean with docker mounted volumes ... so inside docker or what? Would be great if you can describe your use case a little bit. Then I can really understand the problem.
Is it more like statvfs where you just want to retrieve free disk space (so more like what I have in si.fsSize()? Can you provide also examples (maybe with output of df which volumes are not shown?

Is it more like statvfs

It is exactly like statvfs.

maybe with output of df which volumes are not shown?

The "mounted" volumes are not shown in df. I think docker doesn't actually mount it as a "volume" but just "symlinks" the directory to the docker volume.

@ronag I still don't get what you are really looking for. Can you please describe your use case? I still don't know if you are trying to conreoll something INSIDE of a docker container or from OUTSIDE. Please describe the full story. Then I could think of how to implement something ...

Inside a docker container I want to find out how much free and total space the drive of a specific path has. The path has been linked to docker volume (-v /volumes/media:/volumes/media).

@ronag OK, I guess I understand ... so this is not trivial. There are attempts to get fs.statfs() into node but this is still an unresolved issue. Need to think if I can find a solution.


[WIP] Attempt to simplify PHP installs

To call this script, for example: brew install-php-7-3-33-m1


Implement parsing of Postgres regex match operators

Hey all,
This PR adds support for parsing Postgres' POSIX regex matching operators. These are similar in nature to LIKE et al but with full support for regex patterns rather than being restricted to wildcards, so you can use select * from my_table where name ~ '^pattern$'.
I've restricted the parsing to only run when using the Postgres dialect, but I've not given lexing the same treatment as that doesn't seem to be the done thing (e.g. PGBitwiseXor is lexed unconditionally). If this is actually desired, please let me know!
I'm not entirely happy with the token names, but given the overlap (e.g. ~ is both a unary and bitwise op now) I didn't want to attach names that were overly specific either.

Pull Request Test Coverage Report for Build 749129485

65 of 73 (89.04%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 90.041%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/parser.rs
8
11
72.73%

src/tokenizer.rs
43
48
89.58%

Totals

Change from base Build 676410191:
-0.01%

Covered Lines:
5524

Relevant Lines:
6135

💛  - Coveralls

I think this is looking good @returnString . Can you fix or ignore the linting errors?

Possibly made redundant by https://github.com/ballista-compute/sqlparser-rs/pull/328

Closing this PR as https://github.com/sqlparser-rs/sqlparser-rs/pull/328 was merged


Version 3.2 no longer works on JupyterLab with Dask

After upgrading from 3.1.2 to 3.2. some of our code stopped working with peculiar exception:
...
/usr/local/lib/python3.7/site-packages/robot/__init__.py in <module>
/usr/local/lib/python3.7/site-packages/robot/rebot.py in <module>
/usr/local/lib/python3.7/site-packages/robot/conf/__init__.py in <module>
/usr/local/lib/python3.7/site-packages/robot/conf/settings.py in <module>
/usr/local/lib/python3.7/site-packages/robot/output/__init__.py in <module>
/usr/local/lib/python3.7/site-packages/robot/output/output.py in <module>
/usr/local/lib/python3.7/site-packages/robot/output/pyloggingconf.py in <module>
/usr/local/lib/python3.7/site-packages/robot/utils/__init__.py in <module>
/usr/local/lib/python3.7/site-packages/robot/utils/argumentparser.py in <module>
/usr/local/lib/python3.7/site-packages/robot/utils/encoding.py in <module>
/usr/local/lib/python3.7/site-packages/robot/utils/encodingsniffer.py in get_console_encoding()
/usr/local/lib/python3.7/site-packages/robot/utils/encodingsniffer.py in _get_encoding()
/usr/local/lib/python3.7/site-packages/robot/utils/encodingsniffer.py in _get_stream_output_encoding()
/usr/local/lib/python3.7/site-packages/robot/utils/compat.py in isatty()
ValueError: I/O operation on closed file

It looks like Robot now calls isatty() at sys.__stdin__ object. That leads to aforementioned exception. MWE from JupyterLab in case someone wants to double-check:
def process():
    import sys; sys.__stdin__.isatty()

import dask.distributed
with dask.distributed.Client("[REDACTED]") as executor:
    next(dask.distributed.as_completed([executor.submit(process)])).result()

There are lots of moving parts in our setup, so in case you have additional questions (like Dask version, JupyterLab version), please ask.

for the record I believe this is the culprit:
https://github.com/robotframework/robotframework/commit/ae0d0a3f1d72dbafaabf59b2d3c37e63343e1f13

I don't know the JupyterLab setup that well myself but perhaps @datakurre can comment this. Based on the experimenting below I can easily believe the isatty() call is causing this:
>>> f = open('README.rst')
>>> f.isatty()
False
>>> f.close()
>>> f.isatty()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: I/O operation on closed file

We can easily enhance our isatty() utility to catch that ValueError and return False in that case. Could you @szborows possibly try does that fix the issue in your environment?
Kind of good news is that we need RF 3.2.1 soon anyway (planned for next Monday) so we'll get a fix out soon.

Adding guard for closed files to the isatty() util felt like so good idea that I decided to do it in the above commit. You can know install the latest version with the fix using this command:
pip install https://github.com/robotframework/robotframework/archive/master.zip

Thanks for heads up. Curious that I did not run into this while using the latest JupyterLab with 3.2.

The code where the error occurs is here:
https://github.com/robotframework/robotframework/blob/master/src/robot/utils/encodingsniffer.py#L87
Apparently one of sys.__stdout__, sys.__stderr__ and sys.__stdin__ is closed and thus steam.isatty() inside the isatty() util raises a ValueError. Closed stdout or stderr sounds pretty strange, so I'm guessing the closed stream is stdin.

@pekkaklarck
yup, that's stdin, look at my original MWE (or rather MNWE):
def process():
    import sys; sys.__stdin__.isatty()

anyway.. thanks for such a quick reaction. I will try to confirm ASAP that the fix works.

I've tested the fix on local dask cluster and it obviously works!

from my POV the issue is resolved, hence I'm closing it. Thanks!

Great that the fix works! RF 3.2.1 with this fix as well fix for #3561 will be released next Monday.

The error occurs when Robot tries to detect console encoding from standard streams and it won't happen in these cases:

Windows with Python 3.6+. In this situation we don't look at standard streams at all because their encoding is always UTF-8 and we want the real encoding the system uses.
If stdout or stderr have encoding set. Then we just use that and don't look at stdin.

My understanding is that this only occurs if stdout/stderr are redirected and stdin is closed. That's pretty rare, but the whole execution crashing is so severe that this issue deserves high priority.

reproduction steps are below, just for the record. Maybe this is rare, idk. The setup is JupyterLab + Dask cluster.
python3 -mvenv venv
. ./venv/bin/activate
pip3 install "dask[distributed]"

from dask.distributed import Client
def process():
    import sys; sys.__stdin__.isatty()
Client().submit(process).result()


[perflint] Add PERF403

Summary
Reopening of https://github.com/astral-sh/ruff/pull/5313
One open question as described in https://github.com/astral-sh/ruff/pull/5313#issuecomment-1645113101
FYI previous reviewers: @MichaReiser @dhruvmanila @charliermarsh
Old Description
Summary
Adds PERF403 mirroring W8403 from https://github.com/tonybaloney/perflint
Implementation is similar to original/upstream implementation, with the exception of not yet checking whether the ExprSubscript value is a dictionary type value. I tried adding this but ended up doing a lot of
let scope = checker.semantic().scope();
if let Some(binding_id) = scope.get(subscript_value_id) {
    let binding = &checker.semantic().bindings[binding_id];
    if binding.kind.is_assignment() || binding.kind.is_named_expr_assignment() {
        if let Some(parent_id) = binding.source {
        etc....

again for what felt like very little added value.
Open to any feedback/input to do this more efficiently. If there is no more efficient way to do this, also let me know and I'll see if I can make some kind of helper or util for this, as I've bumped into this a few times now with various rules.
I've set the violation to only flag the first append call in a long if-else statement. Happy to change this to some other location or make it multiple violations if that makes more sense.
Test Plan
Fixtures were added based on perflint tests
Issue Links
Refers: https://github.com/astral-sh/ruff/issues/4789

We can probably implement this more confidently now by leveraging some of the basic type inference stuff added in the refurb module.

Sounds good! We'll need to pull the stuff out of refurb/helpers.rs into (probably) ruff_python_semantic::analyzers::typing.

Is that just a simple copy paste to ruff_python_semantic and reimport in the refurb module? If so I'll open a separate PR for that and then rebase this one


feat:  add a start-up check for minimum node and npm versions

Description:

Currently, the user is not warned if their node/npm version is outdated and doesn't meet the minimum requirements.
Solution:
Add a start-up check for the node/npm version.
Related issue(s):
Fixes #863
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Codecov Report
Attention: Patch coverage is 43.75000% with 9 lines in your changes missing coverage. Please review.

Project coverage is 61.98%. Comparing base (2f9cc8c) to head (be58977).
Report is 2 commits behind head on main.

Files with missing lines
Patch %
Lines

src/state/InitState.ts
43.75%
9 Missing :warning:

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #866      +/-   ##
==========================================
- Coverage   62.89%   61.98%   -0.92%
==========================================
  Files          30       30
  Lines        1237     1252      +15
  Branches      149      152       +3
==========================================
- Hits          778      776       -2
- Misses        438      458      +20
+ Partials       21       18       -3

Files with missing lines
Coverage Δ

src/state/InitState.ts
80.50% <43.75%> (-13.67%)
:arrow_down:

----
🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests
JS Bundle Analysis - Avoid shipping oversized bundles

Nice! That one was long overdue if you ask me 🚀


Changes in part 1: some text and start with - Glacier advance-

Hi,
sorry that I want to merge directly a master branch into the master, but I think it's fine as long as I am the only one to work on the nb for the moment.
I changed a little bit of the text of part 1 and started with the glacier advance.

Thanks!

sorry that I want to merge directly a master branch into the master,

No worries, for the main repo it doesn't make a difference at all


Feature to use YAML in .butane

Would you mind if I created a feature to use serde_yaml to use YAML instead of JSON for the files under .butane ?

Or TOML.  Anything but JSON, really. ;-)

Hmm, talk me through the rationale. Why?
I certainly won't claim that JSON is perfect, but it's human-readable and ubiquitous. TOML is better for configuration files, but that's not really what these are. I've never been a big fan of YAML due to the indentation-dependence and general complexity, but I'll grant that it's more human-writable than JSON. But these files aren't supposed to be written by humans, they're just a persisted representation of state intended to be operated on only by the butane library/cli.
Moreover, regardless of the merits of any particular format I do think there's value in having a single format. Otherwise either everybody working on a project using the non-default format has to compile their Butane CLI with special flags when installing it, or the CLI needs to support all the formats by default and try to intelligently figure out which one any given project is using. Which is certainly doable, but it seems undesirable.
None of this is to say I'm set against it, I just want to understand the why before it goes forward.

The reason is JSON is terrible for being stored in a repository, especially if the file will change.  The current directory especially changes with each change to our schema, and I want the diff to be readable so that checking it becomes an important part of our change review process.

Another example of the problem is handling merge conflicts.  Line based formats handle merge conflicts better.  This is most obvious in .butane/migrations/current/types.json because it doesnt use line breaks at all, but even if it does, JSON is still problematic because it cant handle "trailing commas".

Overall, the source control diff rationale makes sense and seems like a good motivation for a different format. I'm curious to hear more about how checking the diff of current would be an "an important part of [your] change review process" though. Current should be mechanically generated from the current Rust sources, so I'm not sure what a review would focus on? If you get any merge conflicts I'd generally recommend just re-compling to regenerate the contents of current.

The code review of current should verify it contains exactly and only the changes which were made to the Rust structs.  Even if it will be regenerated correctly every time, the code review process will ensure everyone becomes familiar with the changes that are expected under current, and then becomes concerned if they dont see what is expected, and either learn more about butane if it was behaving correctly, or investigate if the unexpected change turns out to be a bug.


MF-1061 - Fix cassandra-reader count for json format

Signed-off-by: Manuel Imperiale<EMAIL_ADDRESS>Related to #1061

Codecov Report

Merging #1327 (00ccdae) into master (8e5a9cf) will increase coverage by 0.01%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master    #1327      +/-   ##
==========================================
+ Coverage   59.88%   59.89%   +0.01%
==========================================
  Files         113      113
  Lines        8834     8835       +1
==========================================
+ Hits         5290     5292       +2
+ Misses       3082     3081       -1
  Partials      462      462

Impacted Files
Coverage Δ

readers/cassandra/messages.go
58.16% <50.00%> (-0.60%)
:arrow_down:

things/service.go
54.54% <0.00%> (+1.13%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #1327 (00ccdae) into master (8e5a9cf) will increase coverage by 0.01%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master    #1327      +/-   ##
==========================================
+ Coverage   59.88%   59.89%   +0.01%
==========================================
  Files         113      113
  Lines        8834     8835       +1
==========================================
+ Hits         5290     5292       +2
+ Misses       3082     3081       -1
  Partials      462      462

Impacted Files
Coverage Δ

readers/cassandra/messages.go
58.16% <50.00%> (-0.60%)
:arrow_down:

things/service.go
54.54% <0.00%> (+1.13%)
:arrow_up:

Continue to review full report at Codecov.

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


feat: add --pid-file and --overwrite-pid-file

Add documentation related to PID file.
These options will be available starting in OSS 2.7.11.

PR implementing feature in OSS: https://github.com/influxdata/influxdb/pull/25474

@sanderson The plan was to release this in 2.7.11. I noticed that influxd_flags.yml only lists major and minor for when flags were added. Is it terrible to add this new flag on a patch release, even if it is primarily for a single use-case?

@gwossum

I noticed that influxd_flags.yml only lists major and minor for when flags were added.

This data file was originally added when we were documenting each minor version in the versions specified in the influxd_flags.yml were used to filter flags based on the current minor version. We no longer document each minor version so this property is essentially obsolete. You can just put 2.7. It won't make a difference.

Is it terrible to add this new flag on a patch release, even if it is primarily for a single use-case?

Generally this would be considered a "new feature" and would be added on a minor release, but I think it's fine. I don't think this needs to wait until 2.8.

@sanderson influxd-flags.yml has been updated.


any code about the visualization?

any code about the visualization? such as plotting in the video?

Hi bro，Do you find code about the visualization


add intellij projects to gitignore

As the title says

We're already ignoring other IDE files as well so I guess it can't hurt !


ci: some more CircleCI debug.

Signed-off-by: Harvey Tuch<EMAIL_ADDRESS>
Description of what is de

is this for testing locally in this PR or are we planning to merge? if the latter, i would add a prefix to the echo saying what is being output.

@danielhochman It looks like CircleCI already does this, e.g. https://circleci.com/gh/envoyproxy/envoy/41409. It basically is doing the equivalent of set -x on the run commands.

(.. and yep, would like to merge, so we can cast a wider net)


Add tests for RequestLineStep + small changes

Codecov Report

Merging #472 into master will increase coverage by 0.23%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #472      +/-   ##
==========================================
+ Coverage   67.61%   67.85%   +0.23%
==========================================
  Files          49       49
  Lines        6831     6826       -5
==========================================
+ Hits         4619     4632      +13
+ Misses       2212     2194      -18

Impacted Files
Coverage Δ

src/common/http.cc
73.92% <ø> (+0.55%)
:arrow_up:

include/pistache/http.h
91.52% <100%> (-0.1%)
:arrow_down:

src/common/reactor.cc
80.08% <0%> (-0.41%)
:arrow_down:

src/common/os.cc
54.72% <0%> (+0.67%)
:arrow_up:

include/pistache/async.h
90.39% <0%> (+1.17%)
:arrow_up:

src/client/client.cc
82.99% <0%> (+1.21%)
:arrow_up:

src/common/net.cc
85.71% <0%> (+1.64%)
:arrow_up:

include/pistache/net.h
90% <0%> (+5%)
:arrow_up:

Continue to review full report at Codecov.

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


add line display sample to readme

@muhsinking,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@muhsinking,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Thanks. I'll insert it in alphabetical order and fix the columns.


dep init should not require network access

Dep init cannot be used when there's no network available, because
it uses the network to try to resolve package paths to VCS information.
If there's no network, we see a panic like this:
% dep init
panic: shouldn't be possible unable to deduce repository and source type for "bazil.org/fuse": unable to read metadata: unable to fetch raw metadata: failed HTTP request to URL "http://bazil.org/fuse?go-get=1": Get http://bazil.org/fuse?go-get=1: dial tcp: lookup bazil.org on [::1]:53: read udp [::1]:47862->[::1]:53: read: connection refused

goroutine 1 [running]:
github.com/golang/dep/internal/gps.(*solver).selectRoot(0xc420376300, 0xc42007b8c0, 0xc42007a480)
	/home/rog/src/go/src/github.com/golang/dep/internal/gps/solver.go:549 +0xb2a
github.com/golang/dep/internal/gps.(*solver).Solve(0xc420376300, 0x1e, 0xaec400, 0xc42007a2a0, 0xc420160a55)
	/home/rog/src/go/src/github.com/golang/dep/internal/gps/solver.go:394 +0x86
main.(*initCommand).Run(0xc42014f8fc, 0xc420084fa0, 0xc42000c080, 0x0, 0x0, 0x0, 0x0)
	/home/rog/src/go/src/github.com/golang/dep/cmd/dep/init.go:171 +0x6d6
main.(*Config).Run(0xc420062f00, 0xc420062f00)
	/home/rog/src/go/src/github.com/golang/dep/cmd/dep/main.go:158 +0x768
main.main()
	/home/rog/src/go/src/github.com/golang/dep/cmd/dep/main.go:45 +0x1fc

Even if the panic is fixed, I'd suggest that "dep init" should not need to access the network at all. The repository information can be inferred from the files that are in the repo.

Did you try dep init -gopath?
By default, we rely on the network to fetch VCS metadata about the dependencies. I believe "gopath" mode in init attempts to fetch everything from the info available on disk (in GOPATH), and uses the network as a fallback.
@darkowlzz should be able to confirm my understanding. 😁

This panic seems to be coming from deduce() which is, as @ibrasho mentioned, used by deduceRootPath() to deduce various metadata about a given dependency.

deduceRootPath takes an import path and attempts to deduce various
metadata about it - what type of source should handle it, and where its
"root" is (for vcs repositories, the repository root).

I'm not sure if we can avoid reaching the network to figure out all the necessary information, mostly because we are consider dependencies as projects, not just packages.
init -gopath only fetches the code from GOPATH, if the project already exists in GOPATH. This part comes much later.
@sdboyer would be able to help us more with this, if there is a way to make it work offline or why we don't do it.

This is where we are explicitly panicking in this case https://github.com/golang/dep/blob/master/internal/gps/solver.go#L549 .
As commented, we can handler this with a better error message.

FWIW, reproducing this is pretty easy, as I just discovered:
export HTTPS_PROXY=foobar
export HTTP_PROXY=foobar
dep init

(and if you're a programmer working at an enterprise with an internal proxy, you'll bump into this often!)


Add config for Prater fork

Motivation
See https://github.com/eth2-clients/eth2-networks/pull/58
Description
Update Prater config

What about INACTIVITY_SCORE_BIAS and INACTIVITY_SCORE_RECOVERY_RATE? Are they unrelated to Altair fork?

What about INACTIVITY_SCORE_BIAS and INACTIVITY_SCORE_RECOVERY_RATE? Are they unrelated to Altair fork?

It looks like those are already covered by the mainnet config and don't change from the default.


[Executors] Clean up how we unwrap the local executor of distributed custom actor

The previous implementation relied on a hacky _ protocol requirement.
This is a proper implementation of the "we want to unwrap and offer a nice error" and removes the temporary workaround.
Thank you @slavapestov for the help here!

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test


fix(roboticWarehouse): goal conditional

Fix the goal conditional for reward in robotic warehouse.
Closes #190

Looks good to me. Would it be possible to start a few steps of a2c training (with default params) and seed either 1, 2 or 3 to make sure the behavior is fixed?

We ran some experiments to confirm, everything works as expected now. In essence, the problem remained the same but with inverted goal positions which caused a mismatch between the rendering. In the runs we did, the agents trained and the rendering looked good. I will also add an updated gif to this PR.


rocksdb: Fix cache key collision and turn off force-consistency-checks (#8788)

What problem does this PR solve?
Problem Summary: The force-consistency-checks was set by default to investigate #8243 and prevent it from corrupt data further. However, the config add significant latency to writes (up to hundreds of ms) by holding global mutex. Despite effort in #8519 to reduce its performance impact, we still see it hurt performance in cases. Now that we learned the root cause of #8243, let's disable it.
Cherry-pick #8788 and https://github.com/tikv/tikv/pull/9013
What is changed and how it works?
What's Changed: disable force-consistency-checks
Also update rocksdb to include:

key order check https://github.com/tikv/rust-rocksdb/pull/566
fix cache key collision https://github.com/tikv/rust-rocksdb/pull/567

Related changes
previous effort to optimize the config's performance: #8243
Check List
Tests: CI
Side effects: For user vulnerable to #8243 (with newer kernel version, >= 4.15, depend on distribution), without force-consistency-checks it may cause data corruption.
Release note

Add check to avoid corruption due to RocksDB block cache error #8243. Also disable force-consistency-checks by default to improve performance.

/merge

/test

/run-integration-ddl-test

Maybe we should also update the rust-rocksdb version?
Some changes in  #8788 and #9013 are not picked, and the rust-rocksdb version is still https://github.com/tikv/rust-rocksdb/commit/6d8620d13c83b6388ac4d6fe629d43f94612372e.

@cosven nice catch! the update is not included by mistake


Update Tailwind version

Just checking in here to see if there is anything else you need from me on this PR.

Thanks!


[ROCKETMQ-320] Message loss when shutdown with dispatch behind

What is the purpose of the change
Avoid message loss when shutdown with dispatch behind
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a JIRA issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ROCKETMQ-XXX] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage increased (+0.04%) to 39.171% when pulling c572d03366b9c0120ca8f274940b82240eed97f2 on dongeforever:ROCKETMQ-320 into<PHONE_NUMBER>55fe1533859761908acce1e1eaf706 on apache:develop.

+1, Cool~

@shroman It's only for unit test, any better solution?

@zhouxinyu I would throw an exception from DefaultMessageStore#shutdown if something goes wrong. This will also simplify unit tests -- no special flags are needed.


Extended Networking gets disabled on startup

Steps to reproduce
Enable Extended Networking, start wsa, after a few seconds it gets disabled (unknown vpn config).
✔ Expected Behavior
Extended Networking stays enabled
❌ Actual Behavior
Disables itself after a few seconds even though i dont have any VPN on my PC. I also dont use proxys or anything else. My Internet config is set to everything automatic (dhcp) and i didn't install anything vpn or network related in wsa.
Other Software
No response
Please specify the version of Windows Subsystem for Android
2209.40000.28.0

@keith-horton @nieubank, is this something we're aware of?

Hi there. Could you collect traces for us?
Open Windows Subsystem for Android Settings --> then click Turn off
Run the following from an administrator cmd prompt:
netsh trace start InternetClient provider={a322581f-bff3-5bf6-6bfc-6a067db5f8d8} level=0x5
Open WSA and repro the issue
Run the following from an administrator cmd prompt:
netsh trace stop
(this will generate a NetTrace.etl file)
If you could also run:
netsh nlm query all nlmquery.log
Then send back nlmquery.log and the generated NetTrace.etl.
Thanks!

NetTrace.zip
Here is the NetTrace etl and cab file. I could not find the nlmquery.log and the command produced only empty lines in the console.

Thanks! The issue here is there's a virtual ethernet interface called "LogMeIn Hamachi Virtual Ethernet Adapter". Because it's a virtual ethernet adapter, we can't prove it's a VPN interface or not (sadly, this is a challenge on Windows, as this was designed so VPN vendors could use any solution for their VPN - including virtual Ethernet; and many do use virtual Ethernet).
I'm assuming this is not a VPN interface - but we don't know if it should be mirrored into WSA or not. So we take the conservative approach and default back to NAT mode.
I'm looking to see if there's a workaround.

Ok, thank you. If i manualy deactivate the interface it works again.

@keith-horton Is there a way to manually force various networking settings that you found?
Ideally I'd like advanced networking - ie, use the IP address of the host (or bridged mode), even when a VPN is active.
(I want to use various TV remote or home iot software, which often only work if you are on the same subnet)
And of course, my host OS certainly has various virtual ethernet adapters (VPNS, vmware, whatever)

Same issue here. I don't have any VPN interfaces, just the vEthernet Bridge from Hyper-V. But WSA refuses Extended Networking due to unknown VPN configuration. What can I do to fix this?


installation: pinning of elasticsearch version

Signed-off-by: Jiri Kuncar<EMAIL_ADDRESS>

@inveniosoftware/maintainers please have a look so we can release this quickly before pinning it in all dependent packages.

No problem for B2Share. We pinned it the same way.

Same here, we did the same so no problems for us. 👍

No problem for B2Share. We pinned it the same way.

@nharraud it's not for overlays, that should have compiled dependencies anyway, but for all packages that depend on Invenio-Search.

LGTM

Thanks for LGTM @egabancho. I am going to merge and prepare quick release so we don't have to pin elasticsearch in other packages.


The implications of encryption and backups

Please add a section to discuss how encrypting a drive impacts backups. For example, what happens if pre-encryption backups exist...can a non-encrypted backup be restored to a now encrypted drive? Any impact to an existing backup set when switching to encryption?

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

ID: 7a2b6bec-05af-1e20-a22e-d2a0c71a7dca
Version Independent ID: 5c644e57-ae1b-a510-ebdb-52391fb6407f
Content: BitLocker frequently asked questions (FAQ) (Windows 10)
Content Source: windows/security/information-protection/bitlocker/bitlocker-deployment-and-administration-faq.md
Product: w10
GitHub Login: @brianlic-msft
Microsoft Alias: justinha

@officedocsbot assign @mypil

@domyorker could you please provide more information about the way you do the backup, especially if you're using Windows Backup or something else.
You can restore a non-encrypted backup to an encrypted drive - what will happen is that you will need to re-encrypt the drive after restore.

@domyorker - We would like to follow up. Can you please respond to the query of VLG17? Thanks.

@domyorker - We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

@officedocsbot close


Feature request: Launch browser into a unique origin

Problem
npm start launches http://localhost:3000. If I've ever worked on a different app, its localStorage is now polluting my new app as they share an origin.
Solution
If HOST === 'localhost' then call opn with target set to http://{folder-name}.localhost:{port}
This way every app has a unique origin and this its own localStorage. I know that I can do this manually, but having CRA do it for me would be nice.
Thoughts?
This is related to #2578, but writing a script to get this to work seems like a lot of duplicated effort for each project
I also can't just set HOST as it won't bind to {folder-name}.localhost

This seems like a pretty good idea. It would probably solve some other caching related issues with cookies, service workers, etc. Want to make a PR?


Google action does not access my fulfillment URL

I have created a Smart Home Actions project in google actions console, and setup the OAuth2 and fulfillment to process the Smart Home intents and return responses. After I add my test action to Google Assistant, the Google Assistant App is able to redirect to my OAuth2 page and successfully get the tokens from my OAuth2 server after the authentication. However, after the authentication, the Google Assistant App always shows “Something went wrong. Try again.”
I have tried many times to use the command line (gactions update) on my server to update the fulfillment URL and each time it shows my project was successfully updated. I also checked the fulfillment URL on the Google Actions Console and found it is pointed to my server correctly. However, I found the Google server never access my fulfillment URL after authentication, as my fulfillment logic will record all the access activities for debug.
I have tried to test smart-home-nodejs on another server (server A) before and successfully got everything working well. After that, I have implemented the OAuth2 and fulfillment by my own code on that server (server A) and also everything working well with my own code including SYNC, QUERY and EXECUTE requests. Now, I just use the same code (my code) on a new server and all the configurations are the same, but I found the Google server does not access my fulfillment URL. Is there any step / procedure that I am missing?

Can you check Stackdriver to see what errors may be happening during the account linking step? It could be an issue to your SYNC response.

I have created a Smart Home Actions project in google actions console, and setup the OAuth2 and fulfillment to process the Smart Home intents and return responses. After I add my test action to Google Assistant, the Google Assistant App is able to redirect to my OAuth2 page and successfully get the tokens from my OAuth2 server after the authentication. However, after the authentication, the Google Assistant App always shows “Something went wrong. Try again.”
I have tried many times to use the command line (gactions update) on my server to update the fulfillment URL and each time it shows my project was successfully updated. I also checked the fulfillment URL on the Google Actions Console and found it is pointed to my server correctly. However, I found the Google server never access my fulfillment URL after authentication, as my fulfillment logic will record all the access activities for debug.
I have tried to test smart-home-nodejs on another server (server A) before and successfully got everything working well. After that, I have implemented the OAuth2 and fulfillment by my own code on that server (server A) and also everything working well with my own code including SYNC, QUERY and EXECUTE requests. Now, I just use the same code (my code) on a new server and all the configurations are the same, but I found the Google server does not access my fulfillment URL. Is there any step / procedure that I am missing?

How did you solve it? I am having the same problem. but instead, Google Cloud functions access the fulfillment route several times before giving "Something went wrong. Try again." my fulfillment function returns the json response and 200 status code. what could be wrong ?


Added Visual Studio, Azure Data Studio and LINQpad info

Added information how to get Vim like bindings in Visual Studio 2015 and above, Azure Data Studio and LINQpad

Sweet, great add to the list! Thank you :)


System error

Error like this:

Login success
Enrolling start
[Fail] 092M6003H
System error
'UCASEvaluate' object has no attribute 'courseId'

What may it mean?  ==

Ouch, there is a spelling mistake.
Fix it right now.

Fixed


Lazy initialization + cleanup

Made ReservationsService in OffersController lazy loaded to fulfill the lazy loading requirement. While this probably doesn't give any performance benefit for such a simple object, ReservationsService is quite a good candidate for lazy loading as theoretically offers are listed more often than they are reserved.
Made showExpired a proper boolean parameter. Saw it used this way in some Microsoft article. This resulted in cleaner controller code. The value defaults to false when no parameter is given.
Added includeExpired parameter to FindAll method in OffersService and extracted conditional statement here. You may argue that both the controller and service parameters should be named the same but I think that they have different semantic meaning. The controller "shows" the data as it sends it to the user while the service method only decides what to include in the list.
Installed EFCore.NamingConventions package which automagically transforms all table and column names to snake_case when UseSnakeCaseNamingConvention extension method is called DbContext options builder. The database queries do not need quotes anymore. What an awesome package!
Deleted all weather forecast stuff. I believe it's been here for long enough and we already have enough examples that these weather forecasts no longer provide any value.

Let's go! These changes are so nice, glad you found them

@edvinasstaupas I am sorry, I forgot to set the target branch and it was set to main by default so there were a ton of irrelevant changes. Did you take that into account? Now I fixed it 😃


Add restricted people to interviews

Allow to hide an interview from some people (ex. your own interview)

exclude field with list of user
access checks
user picker for edition


Compile directory/single file

Since there are still cases where sources do not get automatically recompiled when a dependency changes (e.g. access modifier or db dependency) you currently have no other option but to rebuild the whole project to ensure all r-code is up to date.
This can be very time consuming.
It would be nice if there was a way to compile a single file or directory (recursive).

A workaround for the file question is to save the file after making a change (I add a space which is removed by vscode). This is usually enough to trigger a compile.

Just pressing CTRL-S without adding a space is usually enough.

For a single file, I agree although this is still a workaround.
For a complete directory this isn't feasible though...

I have an extra use case for this.
In OEDT, when editing includes, you could set an option in the outline to compile only a single file instead of all sources using the include file. This is very useful when editing central tool includes that are using in a lot of sources.
vscode-abl has not such option.
This feature could provide an alternate way to achieve the same

Change build mode to don't compile
Manually compile a couple of sources using this feature.
Change build mode back to compile all once the changes to the include are ready

Current problem is that changing build mode restarts the language server, which can take some time. It should be possible to change this value without having to reload everything (and it should be possible to reload individual projects, or change options, without restarting the LS too).

I have an extra use case for this.

@cverbiest See https://github.com/vscode-abl/vscode-abl/discussions/239#discussioncomment-11164006