Javascript generated URLs

I tried to crawl a site and get following error in log:
site: 2015-06-10 21:38:12 DEBUG - ACCEPTED document reference. Reference=http://www.site.com/Projects/c2c/channel/images/'+L140413[1+Math.round(Math.random()*(1-1))]+' Filter=com.norconex.collector.core.filter.impl.ExtensionReferenceFilter@3ec300f1[onMatch=EXCLUDE,extensions=jpg,gif,png,ico,swf,css,js,pdf,doc,xls,ppt,txt,odt,zip,rar,gz,swf,xlsx,docx,pptx,mp3,wav,mid,caseSensitive=false]
site: 2015-06-10 21:38:12 ERROR - site: Could not process document: http://www.site.com/dccom/0-5-7171-1-1810165-1-0-0-0-0-0-9296-0-0-0-0-0-0-0-0.html (Invalid URL syntax: http://www.site.com/Projects/c2c/channel/images/'+L140413[1+Math.round(Math.random()*(1-1))]+')
com.norconex.commons.lang.url.URLException: Invalid URL syntax: http://www.site.com/Projects/c2c/channel/images/'+L140413[1+Math.round(Math.random()*(1-1))]+'
	at com.norconex.commons.lang.url.URLNormalizer.<init>(URLNormalizer.java:181)
	at com.norconex.collector.http.url.impl.GenericURLNormalizer.normalizeURL(GenericURLNormalizer.java:159)
	at com.norconex.collector.http.pipeline.queue.HttpQueuePipeline$URLNormalizerStage.executeStage(HttpQueuePipeline.java:133)
	at com.norconex.collector.http.pipeline.queue.AbstractQueueStage.execute(AbstractQueueStage.java:31)
	at com.norconex.collector.http.pipeline.queue.AbstractQueueStage.execute(AbstractQueueStage.java:24)
	at com.norconex.commons.lang.pipeline.Pipeline.execute(Pipeline.java:90)
	at com.norconex.collector.http.pipeline.importer.LinkExtractorStage.executeStage(LinkExtractorStage.java:94)
	at com.norconex.collector.http.pipeline.importer.AbstractImporterStage.execute(AbstractImporterStage.java:31)
	at com.norconex.collector.http.pipeline.importer.AbstractImporterStage.execute(AbstractImporterStage.java:24)
	at com.norconex.commons.lang.pipeline.Pipeline.execute(Pipeline.java:90)
	at com.norconex.collector.http.crawler.HttpCrawler.executeImporterPipeline(HttpCrawler.java:213)
	at com.norconex.collector.core.crawler.AbstractCrawler.processNextQueuedCrawlData(AbstractCrawler.java:473)
	at com.norconex.collector.core.crawler.AbstractCrawler.processNextReference(AbstractCrawler.java:373)
	at com.norconex.collector.core.crawler.AbstractCrawler$ProcessURLsRunnable.run(AbstractCrawler.java:631)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
	at java.lang.Thread.run(Thread.java:745)
Caused by: java.net.URISyntaxException: Illegal character in path at index 60: http://www.site.com/Projects/c2c/channel/images/'+L140413[1+Math.round(Math.random()*(1-1))]+'
	at java.net.URI$Parser.fail(URI.java:2848)
	at java.net.URI$Parser.checkChars(URI.java:3021)
	at java.net.URI$Parser.parseHierarchical(URI.java:3105)
	at java.net.URI$Parser.parse(URI.java:3053)
	at java.net.URI.<init>(URI.java:588)
	at com.norconex.commons.lang.url.URLNormalizer.<init>(URLNormalizer.java:179)
	... 16 more

After reading the page's HTML I found the following fragment there:
<SCRIPT language=javascript><!---
 document.write('<img src="/Projects/c2c/channel/images/'+L140413[1+Math.round(Math.random()*(1-1))]+'"  border="0">');
// --->
...

Seems it were causing the exception, and looks it is the reason there is no pages imported while crawl.
Is there a way to avoid such links, or don't break importing if something happening like that?

I'll flag this as a bug, since a single broken URL should not prevent the extraction of good ones in the same document (and that document should not be rejected).
Whether we should be considering URLs generated via script is another question.  Most often than not, since Javascript is not interpreted, I think we'll get errors.  So I think we shall also make sure script-generated URLs are not considered in the first place.  If one day we add javascript support, it should be with something browser-like that interprets javascript.

Thank you.
It is logical that the collector doesn't process JS-generated urls because it hasn't JS interpreter (at least yet), and correctly continue with correct ones.

The fix is in the latest snapshot release.

Thanks a lot! It is working perfectly.

Looking at the regex that matches the script tag, it will only match <script> and not <SCRIPT language=javascript>

Norconex HTTP Collector 2.2.0 official release is out. It includes this fix. You can download it here.


Add development branch with functioning random number generator

Co-authored-by: Matt Herbert<EMAIL_ADDRESS>

We may need to look at both index.html and index.js to review the code and clean it up. I think there was an extra import for getKanye.


Problem with peered_domains in PSA config

There is a problem with the peered domains for PSA in this line
https://github.com/GoogleCloudPlatform/cloud-foundation-fabric/blob/a95e681f059af9e112636befd03efb124439115f/modules/net-vpc/psa.tf#L28
dns_suffix attribute in google_service_networking_peered_dns_domain  requires the dns suffix to end with a ".". If we pass the trailing "." when setting the peered_domains in the psa_configs variable then the name for the DNS peering results in a string ending in "-" which is not a valid name making terraform fail with the following error.
  # module.shared_vpc.google_service_networking_peered_dns_domain.name["servicenetworking-googleapis-com-backend-myorg-com-"] will be created
  + resource "google_service_networking_peered_dns_domain" "name" {
      + dns_suffix = "backend.myorg.com."
      + id         = (known after apply)
      + name       = "servicenetworking-googleapis-com-backend-myorg-com-"
      + network    = "vpc"
      + parent     = (known after apply)
      + project    = "g-prj-cd-sb-af-h-03"
      + service    = "servicenetworking.googleapis.com"
    }

Plan: 1 to add, 0 to change, 0 to destroy.

Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

  Enter a value: yes

module.shared_vpc.google_service_networking_peered_dns_domain.name["servicenetworking-googleapis-com-backend-myorg-com-"]: Creating...
module.shared_vpc.google_service_networking_peered_dns_domain.name["servicenetworking-googleapis-com-backend-myorg-com-"]: Still creating... [10s elapsed]
╷
│ Error: Error waiting for Create Service Networking Peered DNS Domain: Error code 3, message:
│
│   with module.shared_vpc.google_service_networking_peered_dns_domain.name["servicenetworking-googleapis-com-backend-myorg-com-"],
│   on .terraform/modules/shared_vpc/modules/net-vpc/psa.tf line 83, in resource "google_service_networking_peered_dns_domain" "name":
│   83: resource "google_service_networking_peered_dns_domain" "name" {

If we don't pass the "." the name of google_service_networking_peered_dns_domain is OK but the dns_suffix isn't, so it does not work either. We would need to either:

Agree on passing the "." in the to peered_domains attribute and add a trimsuffix to remove the training "-" for the peering name, or,
Agree on nont passing the "." in peered_domains and add the trailing "." to the DNS suffix.

Both options are valid. Let me know which one you prefer and I'll change it

@apichick this was fixed with #2246, right?


Ordered and Unordered List Buttons for Alloy Editor Toolbar

Not sure if this is a duplicate of #49 but it would be nice to have the option to create a list from the toolbar.
The current toolbar is limited to the following:

A more robust one such as this would be great:

Hey @jsfowles - that should be very easy by changing the configuration: https://alloyeditor.com/docs/features/ul.html
We are very happy about a PR if you would like to share the ❤ :)
Thanks!


Default value for custom class

Hello form io community I am looking for a solution which helps me to set the default value for the custom class.
I wanted to hard code the value for this custom class for each type of controls, so that I can hide it from the user, while they are  building the form.
Thanks.

Hello @abhishek0196
Thank you for your interest in Form.io!
In the future, please submit your questions to Gitter( located here: https://gitter.im/formio/formio) for our community support.  We do offer Support Packages that are for training, support or development services. The hours can be used in one-hour increments at your convenience over a 90 day period.  We will assign one of our senior developers to work directly with you in online sessions. If you are interested in learning more about our Support Packages, then please contact<EMAIL_ADDRESS>Thank you and have a great day!

We're currently addressing a backlog of GitHub issues, and as part of this effort, some inactive issues may be marked as closed. This isn't a dismissal, but a step toward more efficient tracking.
If you feel the issue is still relevant, please re-open and we'll ensure it gets the attention it deserves. Your understanding is appreciated as we work to enhance our open-source responsiveness.


Updating I18n language config on first launch

I'm trying to set the app language by user choice on first app launch.
I have this stack overflow question, but no luck there.
const Stack = createStackNavigator();
const HAS_LAUNCHED = "hasLaunched";
const ENGLISH = "en";
const HEBREW = "he";

//Save the language as AsyncStorage for other times the user will open the app
async function setAppLaunched(en) {
  AsyncStorage.clear()
  AsyncStorage.setItem(HAS_LAUNCHED, "true");
  AsyncStorage.setItem(en ? ENGLISH : HEBREW, "true");
  if(await AsyncStorage.getItem(HEBREW)){
    i18n.locale = "he";
    I18nManager.forceRTL(true);
  }
  else{
    i18n.locale = "en";
    I18nManager.forceRTL(false);
  }
}

//If first launch show this screen
function CheckIfFirstLaunch({ onSelect }) {

  const selectLaunched = (value) => {
    setAppLaunched(value);
    onSelect();
  };

  return (
    <View>
        <Text>Choose Language</Text>
        <Button onPress={() => selectLaunched(false)} title="Hebrew"/>
        <Button onPress={() => selectLaunched(true)} title="English"/>
    </View>
  );
}

export default function App() {
  const [selected, setSelected] = useState(false);

  const verifyHasLaunched = async () => {
    try {
      const hasLaunched = await AsyncStorage.getItem(HAS_LAUNCHED);
      setSelected(hasLaunched != null);
    } catch (err) {
      setSelected(false);
    }
  };

  useEffect(() => verifyHasLaunched, []);

  if (!selected){
    return <CheckIfFirstLaunch onSelect={() => setSelected(true)} />;
  }
  else{
    const verifyLang = async () => {
      const lang = await AsyncStorage.getItem('he');
      if(lang != null){
        i18n.locale = "he";
        I18nManager.forceRTL(true);
      }
      else{
        i18n.locale = "en";
        I18nManager.forceRTL(false);
      }
   };
   () => verifyLang;
  }

  return (
    <NavigationContainer>
      <Stack.Navigator screenOptions={{headerShown: false}} initialRouteName="Login">
        <Stack.Screen name="Login" component={Login} />
        <Stack.Screen name="Register" component={Register} />
        <Stack.Screen name="Dashboard" component={Dashboard} />
      </Stack.Navigator>
    </NavigationContainer>
  );
}

So far I have managed to display the first launch screen with 2 language options (Hebrew, English).
After I choose language the app re-renders and the language remains as default.
Only after I proceed to another screen/refresh the screen I can see the language change.
But the component has already re-rendered, so what can cause that?

Same issue is appear to me as well, In my case I18n.locale get successfully updated but when I logged out & send user to Sign In screen at first render it is the new language but on second render it changes to default, don't know what's causing this behaviour

Same issue for me too, do you have any solution>??

This will help https://www.youtube.com/watch?v=1JhdbLl-Kdg


[CELEBORN-803] Increase default timeout for commit files

What changes were proposed in this pull request?
As title.
Why are the changes needed?
In 0.2.1-incubating, commit files default timeout is NETWORK_TIMEOUT, which is 240s.
It's more reasonable because commit files costs relatively long time. In my testing with tough disks,
2 times of 30s timeout is not enough.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Passes GA and manual test.

cc @pan3793 @AngersZhuuuu


prebuild-install can't install node-canvas prebuild

also having some problem on target node14-macos-x64
pkg . --debug --target node14-macos-x64 --compress gzip -o coco-macos-x64

prebuild-install WARN install No prebuilt binaries found (target=v14.18.3 runtime=node arch=x64 libc= platform=darwin)
> [debug] prebuild-install failed[/Users/<***>/node_modules/canvas/build/Release/canvas.node]:
  Command failed: /Users/<***>/.nvm/versions/node/v14.18.3/lib/node_modules/pkg/node_modules/.bin/prebuild-install --target v14.18.3 --platform darwin --arch x64
prebuild-install WARN install No prebuilt binaries found (target=v14.18.3 runtime=node arch=x64 libc= platform=darwin)

details
#prebuild-install can't install node-canvas prebuild

https://github.com/vercel/pkg/issues/1545#issuecomment-1063750231


Requesting public API for decoding unicode characters

Description:
In the OpenAPI tool, we have a requirement for supporting to generate  OpenAPI spec when the ballerina service has any Unicode with string (example 01)
Example 01: ballerina service (tool input)
service /'limit on new http:Listener(9090) {
    resource function put พิมพ์ชื่อ(string ชื่\u{E2D}) {
    }

Example 02: expected OpenAPI spec(tool output)
openapi: 3.0.1
info:
  title: Limit
  version: 0.0.0
servers:
  - url: "{server}:{port}/limit"
    variables:
      server:
        default: http://localhost
      port:
        default: "9090"
paths:
    /พิมพ์ชื่อ:
    get:
      operationId: operation_get_/พิมพ์ชื่อ
      parameters:
        - name: ชื่อ
          in: query
          required: true
          schema:
            type: string
      responses:
        "202":
          description: Accepted

For getting the above expected OpenAPI spec (example 02) , we tried to implement the same behaviour function with the unescapeBallerina which come from under the identifiers. During the implementation seems that we are going to duplicate the code in openapi tool as well and it will cause requesting several APIs as well.
Due to that, we used java reflection API as a workaround to generate this task.
We would like to get to know whether it is possible to give a public API, like isKeyword() in SyntaxInfo to access decode Unicode.

Related Issue: #37046

Closing since the support has been provided with https://github.com/ballerina-platform/ballerina-lang/pull/38318


feat(lazygit): add lazygit

we already have lazygit keymaps
https://github.com/ayamir/nvimdots/blob/6c1933580ed297cdcfc2094abb815cbb1473e996/lua/keymap/tool.lua#L58-L63

Thanks @CharlesChiuGit my bad, i didn't notice it.


Update isaric-ccpuk.json

edited ethnicity attributes. check for correctness.

thanks! needed to change the mapping to string / null so that they are included in the list -- otherwise the list would be like [true, false, true ....]


Bot detection: Extra experience out of nowhere

If you play legit, you'll notice that sometimes, the total XP doesn't add up to the actual XP you got from catching pokemon, evolving, hatching eggs or spinning pokestops. AFAIK, the smallest unit of XP you can get is 10 (or, any XP you can obtain is always divisible by 10), but on my current legit account I have 245655 XP. I can't find out where that 5 extra XP comes from but it might be from a request that we're missing somewhere. This might enable Niantic to detect the bot.
I'm also almost 100% sure I have seen other values different than 5XP while playing using the app.

Gym fights don't give "divisible-by-10" XP

Training at a gym can net you anything from 1-50 XP depending on your level difference.

@tiagoAnon Those answers suffice?

I have a question about these "Extra experience out of nowhere". When you play with normal app on phone I do not remember to get any exp for vanished pokes - let's analyze this case:
2016-08-27 15:13:59,589 [UpdateLiveStats] [INFO] [log_stats] XXXPlayer | Uptime : 0:03:13 | Earned 200 Stardust | +900 XP | 16,764 XP/h | Visited 4 stops
2016-08-27 15:13:59,595 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY - 0.12km
2016-08-27 15:14:00,439 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY - 0.11km
2016-08-27 15:14:01,375 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY - 0.11km
2016-08-27 15:14:02,448 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY - 0.10km
2016-08-27 15:14:03,673 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY - 0.10km
2016-08-27 15:14:05,228 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY - 0.10km
2016-08-27 15:14:06,205 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY - 0.09km
2016-08-27 15:14:07,034 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY - 0.09km
2016-08-27 15:14:08,053 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY - 0.08km
2016-08-27 15:14:09,400 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY - 0.08km
2016-08-27 15:14:12,304 [UpdateLiveStats] [INFO] [log_stats] XXXPlayer | Uptime : 0:03:26 | Earned 200 Stardust | +900 XP | 15,761 XP/h | Visited 4 stops
2016-08-27 15:14:12,307 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY - 0.07km
2016-08-27 15:14:13,554 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY - 0.07km
2016-08-27 15:14:14,342 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY - 0.06km
2016-08-27 15:14:15,666 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY - 0.05km
2016-08-27 15:14:16,845 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY - 0.05km
2016-08-27 15:14:17,608 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY - 0.04km
2016-08-27 15:14:18,949 [  SpinFort] [INFO] [spun_pokestop] Spun pokestop YYY. Experience awarded: 100. Items awarded: {u'Potion': 1, u'Pokeball': 5}
2016-08-27 15:14:23,497 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY2 - 0.08km
2016-08-27 15:14:26,666 [UpdateLiveStats] [INFO] [log_stats] XXXPlayer | Uptime : 0:03:40 | Earned 200 Stardust | +1,000 XP | 16,371 XP/h | Visited 5 stops
2016-08-27 15:14:26,875 [PokemonCatchWorker] [INFO] [pokemon_appeared] A wild Pidgey appeared! [CP 62] [NCP 0.09] [Potential 0.64] [A/D/S 4/13/12]
2016-08-27 15:14:33,396 [PokemonCatchWorker] [INFO] [threw_pokeball] OK throw! Used Pokeball, with chance 78.21 (114 left)
2016-08-27 15:14:33,601 [PokemonCatchWorker] [INFO] [pokemon_caught] Captured Pidgey! [CP 62] [NCP 0.09] [Potential 0.64] [4/13/12] (69/800) [+100 exp]
2016-08-27 15:14:33,616 [PokemonCatchWorker] [INFO] [gained_candy] You now have 54 Pidgey candy!
2016-08-27 15:14:40,763 [UpdateLiveStats] [INFO] [log_stats] XXXPlayer | Uptime : 0:03:54 | Earned 300 Stardust | +1,100 XP | 16,913 XP/h | Visited 5 stops
2016-08-27 15:14:40,995 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY2 - 0.07km
2016-08-27 15:14:43,184 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY2 - 0.07km
2016-08-27 15:14:44,087 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY2 - 0.06km
2016-08-27 15:14:45,312 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY2 - 0.06km
2016-08-27 15:14:46,252 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY2 - 0.05km
2016-08-27 15:14:47,221 [  SpinFort] [INFO] [spun_pokestop] Spun pokestop YYY2. Experience awarded: 50. Items awarded: {u'Pokeball': 2, u'Revive': 1}
2016-08-27 15:14:50,549 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY3 - 0.10km
2016-08-27 15:14:53,426 [UpdateLiveStats] [INFO] [log_stats] XXXPlayer | Uptime : 0:04:07 | Earned 300 Stardust | +1,150 XP | 16,775 XP/h | Visited 6 stops
2016-08-27 15:14:53,426 [RecycleItems] [INFO] [force_recycle] Forcing item recycle based on schedule
2016-08-27 15:14:53,427 [RecycleItems] [INFO] [next_force_recycle] Next forced item recycle at 15:16:05
2016-08-27 15:14:57,321 [ItemRecycler] [INFO] [item_discarded] Discarded 2x Potion.
2016-08-27 15:15:02,146 [ItemRecycler] [INFO] [item_discarded] Discarded 1x Revive.
2016-08-27 15:15:02,151 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY3 - 0.09km
2016-08-27 15:15:03,625 [PokemonCatchWorker] [INFO] [pokemon_appeared] A wild Fearow appeared! [CP 319] [NCP 0.18] [Potential 0.62] [A/D/S 13/12/3]
2016-08-27 15:15:06,625 [PokemonCatchWorker] [INFO] [vip_pokemon] This is a VIP pokemon. Catch!!!
2016-08-27 15:15:12,520 [PokemonCatchWorker] [INFO] [threw_pokeball] OK throw! Used Pokeball, with chance 22.91 (115 left)
2016-08-27 15:15:12,723 [PokemonCatchWorker] [INFO] [pokemon_capture_failed] Fearow capture failed.. trying again!
2016-08-27 15:15:25,362 [PokemonCatchWorker] [INFO] [threw_pokeball] OK throw! Used Pokeball, with chance 22.91 (114 left)
2016-08-27 15:15:25,570 [PokemonCatchWorker] [INFO] [pokemon_capture_failed] Fearow capture failed.. trying again!
2016-08-27 15:15:31,073 [PokemonCatchWorker] [INFO] [threw_pokeball] OK throw! Used Pokeball, with chance 22.91 (113 left)
2016-08-27 15:15:31,269 [PokemonCatchWorker] [INFO] [pokemon_capture_failed] Fearow capture failed.. trying again!
2016-08-27 15:15:37,114 [PokemonCatchWorker] [INFO] [threw_pokeball] OK throw! Used Pokeball, with chance 22.91 (112 left)
2016-08-27 15:15:37,308 [PokemonCatchWorker] [INFO] [pokemon_capture_failed] Fearow capture failed.. trying again!
2016-08-27 15:15:43,882 [PokemonCatchWorker] [INFO] [threw_pokeball] OK throw! Used Pokeball, with chance 22.91 (111 left)
2016-08-27 15:15:44,346 [PokemonCatchWorker] [INFO] [pokemon_capture_failed] Fearow capture failed.. trying again!
2016-08-27 15:15:51,280 [PokemonCatchWorker] [INFO] [threw_pokeball] OK throw! Used Pokeball, with chance 22.91 (110 left)
2016-08-27 15:15:51,500 [PokemonCatchWorker] [INFO] [pokemon_vanished] Fearow vanished!
2016-08-27 15:15:58,518 [UpdateLiveStats] [INFO] [log_stats] XXXPlayer | Uptime : 0:05:12 | Earned 300 Stardust | +1,175 XP | 13,556 XP/h | Visited 6 stops
2016-08-27 15:15:58,731 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop YYY3 - 0.09km
Do we get 25 exp for having poke vanished?


Hoobs

Pardon my total idiocracy— how can I or could I get this plugin to work in Hoobs?

Ahh found it! Excellent. Hoobs says newest version is 1.2.1; in Github it says 1.3 is newest. How often is Hoobs version aligned with Github?

I’m not sure why that is.  I am not using HOOBS, but have it homebridge installed on a linux box and had no problem getting the latest.  I’ll see if I can figure anything out.

On Sep 15, 2022, at 9:04 PM, blaine07 @.***> wrote:
Ahh found it! Excellent. Hoobs says newest version is 1.2.1; in Github it says 1.3 is newest. How often is Hoobs version aligned with Github?
—
Reply to this email directly, view it on GitHub https://github.com/iklein99/homebridge-smartthings/issues/27#issuecomment-1248800730, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABG55EQAUCH6RHGX3FMATLLV6PBQPANCNFSM6AAAAAAQN4IBNU.
You are receiving this because you are subscribed to this thread.

I’m not sure why that is. I am not using HOOBS, but have it homebridge installed on a linux box and had no problem getting the latest. I’ll see if I can figure anything out.
…

Maybe it pulls itself to Hoobs?
This is inside Hoobs:

Looks to me like someone forked my build. You aren’t pulling the right one.
Look for homebridge-smartthings-ik.  Like that.  It will show up as SmartThings Plugin.

On Sep 15, 2022, at 9:04 PM, blaine07 @.***> wrote:
Ahh found it! Excellent. Hoobs says newest version is 1.2.1; in Github it says 1.3 is newest. How often is Hoobs version aligned with Github?
—
Reply to this email directly, view it on GitHub https://github.com/iklein99/homebridge-smartthings/issues/27#issuecomment-1248800730, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABG55EQAUCH6RHGX3FMATLLV6PBQPANCNFSM6AAAAAAQN4IBNU.
You are receiving this because you are subscribed to this thread.

Looks to me like someone forked my build. You aren’t pulling the right one. Look for homebridge-smartthings-ik. Like that. It will show up as SmartThings Plugin.
…

Yes that’s my issue. Joe sure how or if I can define what I want to pull in Hoobs :-(

You should be able to go into the homebridge user interface and select “Plugins” from the top menu.  From there, you should be able to search for homebridge-smartthings-ik, then instal it.  Read the readme file to get the access token from smart things.

On Sep 15, 2022, at 8:50 PM, blaine07 @.***> wrote:
Pardon my total idiocracy— how can I or could I get this plugin to work in Hoobs?
—
Reply to this email directly, view it on GitHub https://github.com/iklein99/homebridge-smartthings/issues/27, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABG55ETCN4CE5GXVT7UTKOTV6O757ANCNFSM6AAAAAAQN4IBNU.
You are receiving this because you are subscribed to this thread.


add new version with bower main path to bower

The current bower.json file on bower is outdated :(

Outdated how? What do you mean?

Current version on bower 0.1.0 is missing the main path to the main .js file.
If you download the zip from here the path is missing :)
https://www.versioneye.com/javascript/ianmurrays:angular-colorthief/0.1.0

Done ;)

legend :D have an awesome wonderful cat filled adventure day!

😸


Add a screenshot on the event doc

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

🙌
/lgtm

Cool!
/approve


[Bug]: Shopify cli 3 output wrong encoded characters

Please confirm that you have:

[X] Searched existing issues to see if your issue is a duplicate. (If you’ve found a duplicate issue, feel free to add additional information in a comment on it.)
[X] Reproduced the issue in the latest CLI version.

In which of these areas are you experiencing a problem?
App
Expected behavior
shopify cli 3 can output correctly encoded string.
Actual behavior
shopify cli 3 output wrong encoded characters

Stack trace
None trace,
Because I don't know where to find it, I have adjusted the default terminal encoding of linux, but in fact, the final output of console characters is done by Shopify CLI, so I don't know whether it is the CLI that performs the final output of the text special code

Reproduction steps
1 In the app file system, enter the web directory
2 Output a line of text from other countries, Thai, Chinese, and other Asian languages from the terminal
3 Confused character encodings appear
Operating System
Ubuntu 20.04
Shopify CLI version (check your project's package.json if you're not sure)
3.39.0 (The latest version)
Shell
bash
Node version (run node -v if you're not sure)
v18.14.1
What language and version are you using in your application?
Nodejs

1 In the app file system, enter the web directory
2 Output a line of text from other countries, Thai, Chinese, and other Asian languages from the terminal
3 Confused character encodings appear

When you say output a line of text, you mean like a console.log from the frontend process?

1 In the app file system, enter the web directory
2 Output a line of text from other countries, Thai, Chinese, and other Asian languages from the terminal
3 Confused character encodings appear

When you say output a line of text, you mean like a console.log from the frontend process?
In fact, it is not in the front-end process, but in the root directory of the web directory, which is the back-end program of the Shopify APP to output characters. I have tested the character encoding of the current terminal in other node programs (using nodmon to run), the fact is The character encoding of the current Ubuntu system is UTF8, so confusing characters should not be output, so I am basically sure that the encoding confusion is caused by the Context logger of Shopify Cli.

Hi @forwells, I was able to reproduce your bug. I'll assign it to the team to prioritize.
For the moment, I'm sorry but you'd need to avoid using non-latin characters :(

cc/ @matteodepalo this seems related with Ink outputs, what do you think?

Steps to reproduce:

Add a console log with japanese characters at ./fixtures/app/web/backend/index.js:L42 (just at the start of createServer)
pnpm shopify app dev --path fixtures/app

Hi @forwells, I was able to reproduce your bug. I'll assign it to the team to prioritize. I'm sorry but for now you'd need to avoid using non-latin characters :(

Thanks


"Could not find vizInterface when import attempted." Error with default settings on Linux.

Describe the bug
The build appears successful, but when I run python3 examples/scenarioBasicOrbit.py, I get the following warning:

Could not find vizInterface when import attempted.  Be sure to build BSK with vizInterface support.

And with that no Vizard simulation is streamed or file created.
To reproduce

I removed previous basilisk and $HOME/.conan folders.
Re-clone basilisk: git clone git@github.com:AVSLab/basilisk.git.
Checkout the 2.3.0 release tag: git checkout v2.3.0.
Followed the Linux package management steps listed in the install on Linux instructions.
Created a virtual environment, entered it: python3 -m venv .venv and source .venv/bin/activate.
Installed wheel, conan<2.0.0, and cmake: pip3 install wheel 'conan<2.0.0' cmake.
Installed setuptools, because pkg_resources was not found: pip3 install setuptools.
Built basilisk: python3  conanfile.py.
Turn on simulation saving and streaming by uncommenting the corresponding lines in examples/scenarioBasicOrbit.py.
Run a scenario: python3 examples/scenarioBasicOrbit.py

Expected behavior
With vizInterface as the default, it should have been installed automatically.
Desktop (please complete the following information):

OS: Fedora
Version: 39
Python version: 3.12.3
Linux kernel: 6.9.4-100.fc39.x86_64

Additional context
The documentation said building with vizInterface support is the default and I see that in the beginning of the build output:
python -m conans.conan install . --build=missing -s build_type=Release -if dist3/conan -o opNav=False -o vizInterface=True -o buildProject=True

And vizInterface build appears to have been successful in the build output:
/home/patrick/basilisk/src/simulation/vizard/vizInterface/vizInterface.cpp: In member function ‘void VizInterface::ReadBSKMessages()’:
/home/patrick/basilisk/src/simulation/vizard/vizInterface/vizInterface.cpp:376:53: warning: comparison of integer expressions of different signedness: ‘Eigen::EigenBase<Eigen::Matrix<double, -1, 1> >::Index’ {aka ‘long int’} and ‘std::vector<MultiSphere*>::size_type’ {aka ‘long unsigned int’} [-Wsign-compare]
  376 |                     if (msmChargeMsgBuffer.q.size() == scIt->msmInfo.msmList.size()) {
      |                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/patrick/basilisk/src/simulation/vizard/vizInterface/vizInterface.cpp: In member function ‘void VizInterface::WriteProtobuffer(uint64_t)’:
/home/patrick/basilisk/src/simulation/vizard/vizInterface/vizInterface.cpp:932:28: warning: comparison of integer expressions of different signedness: ‘int’ and ‘std::vector<int>::size_type’ {aka ‘long unsigned int’} [-Wsign-compare]
  932 |             for (int i=0; i<scIt->oscOrbitLineColor.size(); i++){
      |                           ~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/patrick/basilisk/src/simulation/vizard/vizInterface/vizInterface.cpp:937:28: warning: comparison of integer expressions of different signedness: ‘int’ and ‘std::vector<int>::size_type’ {aka ‘long unsigned int’} [-Wsign-compare]
  937 |             for (int i=0; i<scIt->trueTrajectoryLineColor.size(); i++){
      |                           ~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
.
.
.
[100%] Linking CXX shared module Basilisk/simulation/_vizInterface.so
[100%] Built target vizInterface

You might be running into #525 since you're removing $HOME/.conan every time.

As a workaround, I found that building with python3 conanfile.py --vizInterface=True fixes the problem. This is undesirable behavior, since this flag should be running with the defaults for python3 conanfile.py and the script even confirms that this flag is set to true.

You might be running into #525 #523 since you're removing $HOME/.conan every time.

It's possible, but I only started deleting $HOME/.conan everytime after I started running into the issue.

That is curious @patrickwalton .  I agree that python3 conanfile.py --vizInterface True should be the same as python conanfile.py.  I don't have access to a Linux system at the moment.  On macOS the vizInterface default is working as expected.  I'm working on other build related changes right now, but will need to revisit this later on.  Glad you found a work-around for now.


Android: Method not found

Sending to WebViewBridge works well, but the opposite way fails.
Error on chrome inspector:

Relevant JS Code:
prop code:
onBridgeMessage={this.onBridgeMessage.bind(this)}

JS code:
    onBridgeMessage = (object) => {
        console.log(object);
    };

Hi @avishayil, can you clarify the flow - are you sending a message from the web view to RN?  If so, can you update the issue with the web view js code?
Also, what version of Android are you using?  Thanks

Hey, it was my fault, I passed old arguments I used in old fork, and that's what caused the error.


[FEATURE]: Update Model Table Endpoints to Support New DynamoDB Layout

Feature Name
Update Model Table Endpoints to Support New DynamoDB Layout
Your Name
Mugdhesh
Description
Feature Name
Update Model Table Endpoints to Support New DynamoDB Layout
Your Name
Mugdhesh
Description
Update the endpoints to the Model Table in the backend to reach the new table layout shown below:

This includes making the create, update, delete, and get endpoints for the Model table with the correct parameters as well as creating unit tests for each endpoint, to be done in subtasks:

[ ] create endpoint
[ ] update endpoint by id
[ ] get endpoint by id

Should there also be update & get endpoints for accessing info using uid?


commonlib follow-up

Pursuant to feedback from other core team members.
Signed-off-by: kingthorin<EMAIL_ADDRESS>

Yup


Draft: feat(package): add vue integration

Hello!
I tried to add Vue integration for tma.
There are already:

integration for tma
eslint vue config
tsconfig for vue
docs page for vue

It isn't testted at all!!
That's why I labeled PR as Draft

Hey Denis! Thank you for your contribution!

It isn't testted at all!!

Do you mean missing Vitest tests or you didn't try to use the code in the PR?

I didn't try to use it in TG for now. I plan to check this next week.
For now the PR is just the starting point

Glad to hear it. Waiting for updates then!

Closing due to inactivity. Feel free to return with this idea


Added missing virtual destructors

When building under Clang 6.0.0 (trunk 314768) the following warning (promoted to error) is caught when attempting to use derived classes from Directory and SecureRandomFactory:
error: destructor called on non-final 'DefaultSecureRandFactory' that has virtual functions but non-virtual destructor [-Werror,-Wdelete-non-virtual-dtor]

Those changes have already been added. Sorry for the super late reply.


Organize browse layer code

We have enough files to organize a bit now.
I'd still like to reduce verbosity / visual noise by removing the "LayerControl" suffix from the filenames, making LayerControls.svelte easier to skim, but that's optional

I  was thinking this needed doing so this is very nice! I will say that I think removing layer control  from filename is fine if we renamed the directory to layer controls?  Reason I ask is because I think it's nice to have <XLayerControl> as the svelte component because it describes what it does.

We can do import XyzLayerControl from "./layers/Xyz.svelte"; the default export can be named however. I'm not too opinionated about it though. https://imfeld.dev/writing/svelte_domless_components is some inspiration for a Svelte component not representing anything in the DOM at all. But in our case, the one component mixes stateful map changes and a bit of DOM


feat(fetchFormData): add fetchFormData API for produce fetch for formData response

#22 for fetchFormData

Codecov Report

Merging #31 (361d4e5) into main (b2105c5) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #31   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           40        41    +1
  Branches         8         9    +1
=========================================
+ Hits            40        41    +1

Impacted Files
Coverage Δ

tests/test-cases.js
100.00% <ø> (ø)

src/fetch.macro.js
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@all-contributors add @mupinnn as code and example

true

Thank you @mupinnn LGTM


First render performance

First render performance is poor due to the massive amount of emojis we have here, I have a few ideas to speed up the perceived load...

[x] Do a first pass render with just the dialog and headers
[x] Render the emojis in category chunks
[ ] Move to faster loops (remove underscore?)

I'm seeing 45 seconds after first load until an emoji click registers in the console, on a late-model MBA with latest Chrome. Diving into speeding that up now.


Switch to "gatsby-plugin-image"

The "fixed" and "fluid" resolvers are now deprecated.

 Switch to "gatsby-plugin-image"


feat(stages): send a notification to ExExes only when Execution stage commits

Fixes https://github.com/paradigmxyz/reth/issues/8696 by separating the ExExNotification sending from the execute and unwind methods. It ensures that the stage provider commits before sending a notification, making the data available for querying on the receiver end of an ExEx.

potentially of relevance: https://github.com/paradigmxyz/reth/issues/8696#issuecomment-2158439305


Denoify

So, this started out as a PR to add first-class Deno support to the library, but it became quite a bit bigger than that...
The good news: Test coverage has increased significantly, as we now test the TypeScript definitions we generate, and we actually test things work in the various runtimes we aim to support.
The bad news: This PR is a huge mf, and I haven't even covered two of the examples yet.
Summary:

Turn packages/autometrics into a first class Deno package. Once this PR is merged, we can set up https://deno.land/x so it will automatically publish Deno packages from tags on this repository.
Introduced justfiles so we have a consistent command set to use across packages and examples, regardless of runtime.
Extended test coverage. For instance, we now check that module paths actually work in Bun.
Implemented support for modern decorators, and renamed the existing decorator to AutometricsLegacy since it's only compatible with the legacy TypeScript decorators.

Still trying to figure out why Yarn thinks the lock file needs to be updated in CI. Running it with CI=1 env locally works fine...


Getting error in Dashboard on a fresh installation

After having add 2 roles to roles I switch to user in order to add roles to a user I get this error:
ErrorException in 70a34edc433640635e97bdad90b795b7c04bb18a.php line 56:
Invalid argument supplied for foreach() (View: C:\laravel\z\resources\views\scaffold-interface\users\edit.blade.php)

@spitfire64
Make sure that you add HasRole dependency to User.php
Check the following link.

terrible sorry! 😌😌😌😌
MVH/Mårten Hansson 0703 - 06 71 81
2016-10-25 0:20 GMT+02:00 Amrani Houssain<EMAIL_ADDRESS>
@spitfire64 https://github.com/spitfire64
Make sure that you add HasRole dependency to User.php
Check the following link
http://amranidev.github.io/blog/site/scaffold-interface/#iii-control-panel-adminlte
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/amranidev/scaffold-interface/issues/76#issuecomment-255882883,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACwBSEZ_IqSASjGDAwwZtozqEbhhQQdhks5q3S8fgaJpZM4KfL5K
.


Create styles for the saving behavior on cg-dashboard

Can you give your approval on the changes @jcscottiii and @fureigh ?

LGTM


Load default html.css stylesheet. Fix #994.

This probably needs some cleanup, so I'd appreciate some comments on where to move the relevant code - I've left some comments inline. Issue #994

Wow, this is really great! I didn't figure on us getting this fixed so fast :D. Comments inline...

Thanks, merged as 89c1c77f9c553b8aebfbdaea6239f57fd3c22a9e!!


Is there a way to run reload?

@krasimir Hi,
Thanks for the nice lib.
Is it possible to run reload for router to re call routing logic. Example:
router.navigate('/users'); # -> API.getUsers() called, ui rendered with users
API.deleteUser(123); # User clicks a button, some API method called
router.reload(); # -> to back to the step 1 above and then update ui

The only way I found that allows me to do a reload is the following:
const currentUrl = router._lastRouteResolved.url
router._lastRouteResolved = null;
router.navigate(currentUrl);

But this solution has issues with leave logic

Hey,
I'll suggest to shift this concerns out of the router. I mean at the end you have functions/handlers that Navigo fires right. So why not define the handlers as named functions and call them when after you update the data:
function showUsers() { ... }
router.on('/users', showUsers);

// later on
router.navigate('/users');
API.deleteUsers(123);
showUsers();

Following this approach you'll decouple the rendering of the routing.

@krasimir Yes, it also works but require me to update the code. Thank you.

Wrapping @killmenot approach into a more convenient method. Using a 'leave()' with this worked for me.
Navigo.prototype.forceNavigate = function(path, absolute) {
  if (absolute === undefined) {
    absolute = false;
  }
  this._lastRouteResolved = null;
  this.navigate(path, absolute);
}

Sorry i really don't understand why not implement a router.reload() function,,, ? Congratulations this is amazing project.


Not clickable to switch spaces

Hello
I am using Yabai and Sonoma 14.0. I have installed the last version of YabaiIndicator it shows up on my mac bar but when i click on the numbers it does't switch to the spasce.
My SIP is disabled
~ ❯ csrutil status
System Integrity Protection status: disabled.
I could not find any other thing to try. Can someone help?
Thank you!

Have you installed yabai? And can you switch spaces manually with yabai commands? E.g. yabai -m space --focus <space number>.

Hey. Yes Yabai is installed and i am using keybinds to switch e move apps to spaces. But on your app if i click on the space numbers it won't switch.


./rustpython -m dis: No code object available for dis

Feature
dis: Disassembler for Python bytecode.
I want to should compiled bytecode for a .py script, and I got this error:
$ ./rustpython -m dis xxx.py
./rustpython: No code object available for dis

In cpython, most of the std module will be pre-compiled during build.
Python Documentation
https://docs.python.org/3.8/library/dis.html

See also #367

This is because dis is implemented as a native module, not using cpython's dis.py that's specific to their bytecode format.

I think we can move our native dis module to _dis and create our own dis.py with _test in CPython dis.py

I expect cargo run -- -m dis x.py works like python -m dis x.py after resolving this issue

@youknowone thanks for #3469 , for me it is working like
(rustpython-session) [^_^]─[~/work-my-projects/pyconapac-2021-sprint/rustpython-session/RustPython] tai271828@syakaro: 35 file 300Kb
$ cargo run -- -m dis ./demo.py
   Compiling rustpython-pylib v0.1.0 (/home/tai271828/work-my-projects/pyconapac-2021-sprint/rustpython-session/RustPython/vm/pylib-crate)
   Compiling rustpython-stdlib v0.1.2 (/home/tai271828/work-my-projects/pyconapac-2021-sprint/rustpython-session/RustPython/stdlib)
   Compiling rustpython-vm v0.1.2 (/home/tai271828/work-my-projects/pyconapac-2021-sprint/rustpython-session/RustPython/vm)
   Compiling rustpython v0.1.2 (/home/tai271828/work-my-projects/pyconapac-2021-sprint/rustpython-session/RustPython)
    Finished dev [unoptimized + debuginfo] target(s) in 1m 05s
     Running `target/debug/rustpython -m dis ./demo.py`
(rustpython-session) [^_^]─[~/work-my-projects/pyconapac-2021-sprint/rustpython-session/RustPython] tai271828@syakaro: 35 file 300Kb
$


fix reading ska tsv

After syncing with @katrinakalantar I realize I misinterpreted this code and introduced a logical error. I also realized all this code could be replaced with panda's built-in methods. I think this makes things much simpler and more performant. I tested locally on a file @katrinakalantar provided and it worked. She mentioned some tsv parsing weirdness with these files so we can keep an eye out when testing the workflow as a whole and address any issues as they arise.

@katrinakalantar thanks actually I will add a comment in the code as well. If there is weirdness we can use pandas' regex separator so the code won't change too much.


[Mobile] Landing/Home Screen - description disappears when screen is rotating

env. https://staging.fightpandemics.work/
Steps:

Open https://staging.fightpandemics.work/ on your mobile
Rotate screen
When screen is rotated description : Pandemics will continue to happen. We help communities prepare and respond - is not visible

Expected results: description is visible
More info:
Android 9 , Chrome

#345 related


fix: pass along continuation token correctly

When using From -> ListObjectsV2, the marker field is mistakenly translated into start_after instead of continuation_token.
S3 is weird in that it has two ways of specifying pagination; start_after lets you start after a given key, whereas marker (continuation_token) is a fully opaque thing that the server then uses to handle pagination.
Please double check this PR: I am reasonably sure it is correct, but S3 is a complex beast and I may be missing something obvious.

Thank you!
I'm not very sure about this but the AWS doc says that start-after is the same thing with marker.
(AWS docs are not always correct though)
https://docs.aws.amazon.com/AmazonS3/latest/API/API_ListObjects.html

marker
Marker is where you want Amazon S3 to start listing from. Amazon S3 starts listing after this specified key. Marker can be any key in the bucket.

https://docs.aws.amazon.com/AmazonS3/latest/API/API_ListObjectsV2.html

start-after
StartAfter is where you want Amazon S3 to start listing from. Amazon S3 starts listing after this specified key. StartAfter can be any key in the bucket.

continuation-token
ContinuationToken indicates to Amazon S3 that the list is being continued on this bucket with a token. ContinuationToken is obfuscated and is not a real key. You can use this ContinuationToken for pagination of the list results.

Our test suites didn't cover this case.
How can we test this behavior? Or Is there any reference implementation?

Hi Nugine,
I think you're right here, actually. This PR is wrong.
It looks like ListObjects marker is meant to be a real key name, and ListObjects(v1) doesn't support a continuation-token style parameter.
I think the original code might actually be correct, and I've gotten confused myself.


Preparing last details for start the flow campaign

Summary
Preparing last details to start the flow campaign, including migration of
trivial files (only @flow tag needed), make all objects definitions stricter, add the checks list for check-style,
test and flow, and adding the flow to test execution.
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit tests passed
[x] Ran make flow to ensure type checking passed

Test Information
This PR was tested on: [Arch Linux]

@lieut-data is out so I switched in @sudheerDev


Vk auth not working on iOS

Capacitor v3.
Following the instructions, I create standalone application and add vkID to my app
    <key>CFBundleURLSchemes</key>     <array>         <string>capacitor</string>         <string>vk723****</string>     </array>
When i try to log in i get an error "OAuth authorization cannot be used from native VK applications"

@fannyfan414 Maybe you using "Site app" instead of "Standalone-application".
I will update the Readme

Apparently my carelessness.
The readme contains a sample code VKAuth.initWithId ({id: '7569443'}), but I used id as a number VKAuth.initWithId ({id: 723}).
Therefore, apparently id was not set and by default it was set as 123456


FileSystemInfo can not createSnapshot with symbol link

Bug report

What is the current behavior?
I install deps with npminstall(it can install faster), but the file tree in node_modules is special bacause npminstall install pkgs with symbol link to download and write less files.
The file tree looks like this
app
├── index.js
└── node_modules
    ├── _a@1.0@a(real dir)
    |   ├── index.js
    |   └── node_modules
    |       └── b(symbol link to app/node_modules/_b@1.0@b)
    ├── _b@1.0@b(real dir)
    |   ├── index.js
    |   └── node_modules
    |       └── a(symbol link to app/node_modules/_a@1.0@a)
    ├── a(symbol link to app/node_modules/_a@1.0@a)
    └── b(symbol link to app/node_modules/_b@1.0@b)

Because of a,b is reference by each other, contextTimestampQueue and contextHashQueue in FileSystemInfo will never reach the end.
If the current behavior is a bug, please provide the steps to reproduce.

The minimal reproduction repo is here.
The reproduce step is
git clone git@github.com:killagu/npminstall_webpack.git
cd npminstall_webpack
npm i -g cnpm
# must use cnpm install
cnpm i
npm run build

Build snapshow will fail.
What is the expected behavior?

Use fs.lstat to get file info, and use target path to process.
Skip files if has been processed

Other relevant information:
webpack version:  5.11.0
Node.js version: 12.16.3
Operating System: macOS Big Sur 11.0.1 (20B29)
If it is ok, I'm happy to make a pr to fix it.

Yep bug

@alexander-akait Thanks for confirm, I can have a try to fix it.

Other example with symlinks https://github.com/webpack/webpack/issues/12503#issue-793467072

Other example with symlinks https://github.com/webpack/webpack/issues/12503#issue-793467072

Another example https://github.com/webpack/webpack/issues/12810

bump

Do we have an ETA on when this might be fixed?

https://github.com/webpack/webpack/pull/14019

Hi @sokra, I found that some specific libs will still cause the snapshot to fail in v5.52.0, such as video.js. Install video.js in the above reproduction repo, and it can be reproduced after running.
git clone git@github.com:killagu/npminstall_webpack.git
cd npminstall_webpack
npm i -g cnpm
# must use cnpm install
cnpm i
# NEW: install the video.js library
cnpm install video.js -S
npm run build

<--- JS stacktrace --->

FATAL ERROR: invalid array length Allocation failed - JavaScript heap out of memory

@qiqiboy update wepback to the latest stable version

@qiqiboy update wepback to the latest stable version

Yes, I have tried, but still fail.
cnpm install video.js webpack@latest -S
npm run build

The problam is still

Can't reproduce the problem

@alexander-akait my env info:
$ cnpm -v
<EMAIL_ADDRESS>(/usr/local/lib/node_modules/cnpm/lib/parse_argv.js)
<EMAIL_ADDRESS>(/usr/local/lib/node_modules/cnpm/node_modules/npm/lib/npm.js)
<EMAIL_ADDRESS>(/usr/local/bin/node)
<EMAIL_ADDRESS>(/usr/local/lib/node_modules/cnpm/node_modules/npminstall/lib/index.js)
prefix=/usr/local
darwin x64 20.3.0
registry=https://registry.nlark.com

Increase your memory usage, file is so bug, not related to this problem


Unable to rerun specific test without changes from gradlew

I need to make any change (e.g. new line, empty space, whatsoever) before running any specific test, otherwise execution doesn't start. Here's what I'm running:
./gradlew spoonTestingTest -PspoonClassName=com.myapp.test.SomeTest
Outputs:
First execution (tests running as expected):
...

:MyApp:preTestingTestBuild
:MyApp:prepareTestingTestDependencies
:MyApp:compileTestingTestAidl UP-TO-DATE
:MyApp:processTestingTestManifest UP-TO-DATE
:MyApp:compileTestingTestRenderscript UP-TO-DATE
:MyApp:generateTestingTestBuildConfig UP-TO-DATE
:MyApp:mergeTestingTestAssets UP-TO-DATE
:MyApp:generateTestingTestResValues UP-TO-DATE
:MyApp:generateTestingTestResources UP-TO-DATE
:MyApp:mergeTestingTestResources UP-TO-DATE
:MyApp:processTestingTestResources UP-TO-DATE
:MyApp:generateTestingTestSources UP-TO-DATE
:MyApp:compileTestingTestJava
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
:MyApp:preDexTestingTest UP-TO-DATE
:MyApp:dexTestingTest
:MyApp:processTestingTestJavaRes UP-TO-DATE
:MyApp:packageTestingTest
:MyApp:assembleTestingTest
:MyApp:spoonTestingTest
2014-02-18 17:00:46 [SR.runTests] Executing instrumentation suite on 1 device(s).

BUILD SUCCESSFUL

Total time: 32.391 secs

On next execution (without making any changes):
:MyApp:preTestingTestBuild
:MyApp:prepareTestingTestDependencies
:MyApp:compileTestingTestAidl UP-TO-DATE
:MyApp:processTestingTestManifest UP-TO-DATE
:MyApp:compileTestingTestRenderscript UP-TO-DATE
:MyApp:generateTestingTestBuildConfig UP-TO-DATE
:MyApp:mergeTestingTestAssets UP-TO-DATE
:MyApp:generateTestingTestResValues UP-TO-DATE
:MyApp:generateTestingTestResources UP-TO-DATE
:MyApp:mergeTestingTestResources UP-TO-DATE
:MyApp:processTestingTestResources UP-TO-DATE
:MyApp:generateTestingTestSources UP-TO-DATE
:MyApp:compileTestingTestJava UP-TO-DATE
:MyApp:preDexTestingTest UP-TO-DATE
:MyApp:dexTestingTest UP-TO-DATE
:MyApp:processTestingTestJavaRes UP-TO-DATE
:MyApp:packageTestingTest UP-TO-DATE
:MyApp:assembleTestingTest UP-TO-DATE
:MyApp:spoonTestingTest UP-TO-DATE

BUILD SUCCESSFUL

Total time: 16.998 secs

Thanks!

I just encountered the same behavior and was surprised because the Android plugin for Gradle always runs connectedCheck tasks. This makes sense because the entire state of external devices can't be known, and since there is no Android simulator, the device's state is practically guaranteed to be different every time the tests are executed. Perhaps the Spoon plugin should also treat connected tests as never UP-TO-DATE to match the Android plugin's behavior. On subsequent test runs these tasks are always executed:
[...] UP-TO-DATE
:app:connectedAndroidTest
:app:connectedCheck

It's a good point. I created #66.

Thanks @roman-mazur!

@roman-mazur  . I am using the latest plugin, but still i am facing the same issue. :(


Wait container cannot kill distroless sidecars

Summary
Using istio-9.1-distroless with automatic sidecar injection. The workflow executes the main container, then tries to kill the sidecar container using exec.
https://github.com/argoproj/argo-workflows/blob/e6fa41a1b91be2e56884ca16427aaaae4558fa00/workflow/executor/k8sapi/client.go#L99
│ 2021-03-02T08:16:34.500261732Z wait time="2021-03-02T08:16:34.500Z" level=info msg="https://<IP_ADDRESS>:443/api/v1/namespaces/cicd/pods/REDACTED-tszxw-603544277/exec?command=%2Fbin% │
│ 2Fsh&command=-c&command=kill+-15+1&container=istio-proxy&stderr=true&stdout=false&tty=false"

Since the istio-proxy container does not have a sh shell, there is a timeout error.
 time="2021-03-02T08:17:04.671Z" level=error msg="executor error: Timeout occurred\ngithub.com/argoproj/argo/v2/errors.Wrap\n\t/go/src/github.com/argoproj/argo/errors/errors.go:88\ngithub.com  /argoproj/argo/v2/errors.InternalWrapError\n\t/go/src/github.com/argoproj/argo/errors/errors.go:71\ngithub.com/argoproj/argo/v2/workflow/common.GetExecutorOutput\n\t/go/src/github.com/argopr  oj/argo/workflow/common/util.go:240\ngithub.com/argoproj/argo/v2/workflow/executor/k8sapi.(*k8sAPIClient).KillContainer\n\t/go/src/github.com/argoproj/argo/workflow/executor/k8sapi/client.go  :87\ngithub.com/argoproj/argo/v2/workflow/executor/common.TerminatePodWithContainerID\n\t/go/src/github.com/argoproj/argo/workflow/executor/common/common.go:92\ngithub.com/argoproj/argo/v2/w  orkflow/executor/common.KillGracefully\n\t/go/src/github.com/argoproj/argo/workflow/executor/common/common.go:98\ngithub.com/argoproj/argo/v2/workflow/executor/k8sapi.(*k8sAPIClient).killGra  cefully\n\t/go/src/github.com/argoproj/argo/workflow/executor/k8sapi/client.go:92\ngithub.com/argoproj/argo/v2/workflow/executor/k8sapi.(*K8sAPIExecutor).Kill\n\t/go/src/github.com/argoproj/  argo/workflow/executor/k8sapi/k8sapi.go:71\ngithub.com/argoproj/argo/v2/workflow/executor.(*WorkflowExecutor).KillSidecars\n\t/go/src/github.com/argoproj/argo/workflow/executor/executor.go:1  268\ngithub.com/argoproj/argo/v2/cmd/argoexec/commands.waitContainer.func1\n\t/go/src/github.com/argoproj/argo/cmd/argoexec/commands/wait.go:33\ngithub.com/argoproj/argo/v2/cmd/argoexec/comm  ands.waitContainer\n\t/go/src/github.com/argoproj/argo/cmd/argoexec/commands/wait.go:87\ngithub.com/argoproj/argo/v2/cmd/argoexec/commands.NewWaitCommand.func1\n\t/go/src/github.com/argoproj  /argo/cmd/argoexec/commands/wait.go:16\ngithub.com/spf13/cobra.(*Command).execute\n\t/go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:846\ngithub.com/spf13/cobra.(*Command).ExecuteC\n\t/  go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:950\ngithub.com/spf13/cobra.(*Command).Execute\n\t/go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:887\nmain.main\n\t/go/src/github.co  m/argoproj/argo/cmd/argoexec/main.go:13\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:203\nruntime.goexit\n\t/usr/local/go/src/runtime/asm_amd64.s:1357"                                   time="2021-03-02T08:17:04.671Z" level=info msg="Alloc=6418 TotalAlloc=18948 Sys=70592 NumGC=6 Goroutines=9"                                                                                stream closed

Workaround
Add a curl sidecar which will trigger the /quitquitquit istio sidecar endpoint.
      sidecars:
        - name: istio-cleanup
          image: curlimages/curl:latest
          command: ["sh", "-c"]
          args:
            - istio(){ curl -X POST --silent --fail http://localhost:15020/quitquitquit; echo Sidecar stopped; exit $?; }; trap istio SIGINT; trap istio SIGTERM; while true; do sleep 1; done

As a bonus, this is the script I use to wait for istio proxy. Sometimes either curl/wget are not available, so using netcat:
sidecarReady()
{
  set +e
  if [ -x "$(which nc)" ] ; then
    echo -n "GET /healthz/ready HTTP/1.1
Host: <IP_ADDRESS>
Connection: close

" |
    nc -i 1 <IP_ADDRESS> 15021 |
    head -1 |
    grep "200 OK"
  elif [ -x "$(which curl)" ]; then
    curl --silent --fail  http://<IP_ADDRESS>:15021/healthz/ready
  elif [ -x "$(which wget)" ] ; then
   wget -O /dev/null --quiet http://<IP_ADDRESS>:15021/healthz/ready
  else
    echo "Could not find curl or wget, please install one." >&2
    exit 1
  fi
}

echo Waiting for sidecar
until sidecarReady
do
  echo "Waiting for Sidecar..."
  sleep 3
done
echo Sidecar ready

This is the kill sidecar using netcat (busybox wget cannot POST):

sidecarCleanup()
{
  echo Stopping sidecar
  set +e
  if [ -x "$(which nc)" ] ; then
    echo -n "POST /quitquitquit HTTP/1.1
Host: <IP_ADDRESS>
Connection: close
Content-length: 0

" |
    nc -i 1 <IP_ADDRESS> 15020 |
    head -1 |
    grep "200 OK"
  elif [ -x "$(which curl)" ]; then
    curl -X POST --silent --fail http://localhost:15020/quitquitquit
    echo Sidecar stopped
  elif [ -x "$(which wget)" ] ; then
    wget -O /dev/null --quiet --method=POST http://localhost:15020/quitquitquit
    echo Sidecar stopped
  else
    echo "Could not find curl or wget, please install one." >&2
  fi
  exit $?
}

trap sidecarCleanup SIGTERM
trap sidecarCleanup SIGKILL

Workaround drawbacks
Seems that argo wait container does not recognize that the sidecar was killed by /quitquitquit, and waits for the timeout anyways, which adds additional delays.
Diagnostics
What Kubernetes provider are you using?
1.20.2
What version of Argo Workflows are you running?
2.12.9

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

Nice to see distroless in the wild. You cannot kill sidecars with either the k8sapi or kubelet executors.  It is not possible to fix.  Instead, use PNS, Docker or Emissary executors.

Hi @alexec
The /quitquitquit endpoint works well. Though, it introduces additional delay for each task.
Perhaps a possible improvement would be that
func (c *k8sAPIClient) KillContainer(pod *corev1.Pod, container *corev1.ContainerStatus, sig syscall.Signal) error {

Could poll the sidecar container status to see if it was already killed, instead of waiting for the command to return an error. But that's probably not a priority

Workround1
Pattern1. mount hostpath busybox
   sidecars:
    - image: pause
      name: pause
      volumeMounts:
      - mountPath: /bin
        name: busybox
  volumes:
    - hostPath:
        path: /opt/busybox
        type: DirectoryOrCreate
      name: busybox

Pattern2. initContainer copy busybox
    volumes:
    - name: busybox
      emptyDir: {}
    initContainers:
    - image: busybox
      name: busybox
      command:
        - sh
        - -c
        - |
          cp /bin/busybox /opt/busybox/
          cd /opt/busybox
          busybox --list | xargs -I{} ln -s busybox {}
      volumeMounts:
      - mountPath: /opt/busybox
        name: busybox

Workaround1 drawbacks
distroless container security
Workaround2
EphemeralContainers
kill $(pgrep <sidecar command>)

Workaround1 drawbacks
slow

I think we need something better here.
Can you confirm if ISTIO contains a /bin/kill file?


expose isScriptableTag() and getAttributeType()

CLA is valid!

@yukinying / @neraliu, please help review/merge.

is it good leaving out some old forms of tags like vbscript and ecmascript ?

@yukinying , you meant  and < ecmascript>?

Kindly refer to
https://github.com/yahoo/secure-handlebars/blob/293cc43b851dc1a238e57973e6ea3ae372702d4a/src/context-parser-handlebars.js#L552-L670
and
https://github.com/yahoo/secure-handlebars/blob/293cc43b851dc1a238e57973e6ea3ae372702d4a/src/context-parser-handlebars.js#L449-L463
for reasons to have isScriptableTag() and getAttributeNameType()

Can you elaborate further?

the context parser is based on the HTML state as core principle and isScriptableTag is being used in secure-handlebars for filters decision making (especially for the state of tagname), so i would recommend move the function of isScriptableTag to the secure-handlebars instead.


feature/fix types

changes

update demo page
update types
update version

Codecov Report

Merging #14 into master will increase coverage by 0.03%.
The diff coverage is 28.57%.

@@            Coverage Diff             @@
##           master      #14      +/-   ##
==========================================
+ Coverage   18.01%   18.04%   +0.03%
==========================================
  Files          19       19
  Lines         816      820       +4
  Branches      115      117       +2
==========================================
+ Hits          147      148       +1
- Misses        669      672       +3

Impacted Files
Coverage Δ

src/ui/ui.manager.ts
0% <ø> (ø)
:arrow_up:

src/ui/ui.pipe.ts
0% <0%> (ø)
:arrow_up:

src/global.ts
66.66% <100%> (ø)
:arrow_up:

src/ui/ui.helper.ts
50% <33.33%> (-3.85%)
:arrow_down:

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 803dd6a...be60086. Read the comment docs.

Pull Request Test Coverage Report for Build 158

2 of 6 (33.33%) changed or added relevant lines in 3 files are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.005%) to 13.402%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/ui/ui.helper.ts
1
2
50.0%

src/ui/ui.pipe.ts
0
3
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/ui/ui.pipe.ts
5
0.0%

Totals

Change from base Build 155:
0.005%

Covered Lines:
149

Relevant Lines:
816

💛  - Coveralls


add a default duration for when end not specified

Addresses https://github.com/code4recovery/tsml-ui/issues/245

thanks! i will check this out with sites currently using TSML UI who are not supplying end times, to see if they are comfortable with this update
changes look great overall, some small bits of feedback:

it would be great if you could format with Prettier, looks like at least one of the files would be affected
i don't think we need the google sheet change - the purpose of that function is to prepare the end_time for processing in the main load-meeting-data function, so in the HH:MM text format basically
in format-ics i think we can just remove the end_time logic now, since meetings with a start will also now have an end

can got prettier installed and formatted, undid the google change but wasn't sure what to do on "in format-ics i think we can just remove the end_time logic now"
Also thoughts on an issue to add prettier as dev dependency and npm command to run it?

left a comment on format-ics
sounds great on creating an issue to add prettier as dev dependency and npm command to run it -- i wasn't aware that was possible

Oops sorry been swamped last two days got the issue in though. Any other fixes needed after your commit?

Hey, no, should be all good now. I just have to give AA SF / Marin a few days to implement end times if they want to. Looking to merge on Sunday / Monday


calc_semantic_segmentation_confusion dynamic matrix

In eval_semantic_segmentation in the function of computing the confusion matrix, since the matrix is defined dynamically based on the value of labels/ground_truth , is it necessary to define a 1D confusion matrix in line 37? Maybe I'm missing something but I think any reference to confusion could be removed and make use only of expanded confusion.

Probably it's me missing something... The question was mainly because the confusion matrix is initialized as an empty matrix and n_class is hardcoded as zero so I was not seeing the point of that initial confusion matrix.

My bad...It is completely needed. Thank you for the info!


examples: fix pipeline layout in compute runner

(Remove the outdated comment which refered to the prior empty compute shader and doesn't apply anymore as the current example takes one buffer descriptor.)
The motivation behind min_binding_size is not totally clear to me but seems to involve the buffer size bound to the descriptor. Two options appear valid here:

None: defers the wgpu check to runtime and it will internally validate that the bound buffer matches the shader interface.
Some(n * sizeof::<u32>()): It needs to be able to hold at least one element ([u32] buffer) -> Some(NonZeroU64::new(4).unwrap()) at least

Both approaches work fine locally but the None case seems less of a hassle

Still seems to be some CI issue, a couple tests fail.

The test errors seem unrelated: https://github.com/EmbarkStudios/rust-gpu/issues/865

Oh right. I'll go ahead and merge this then, let's get those solved separately.


Extensions

Provides extension support for

Content Control
Data Binding

First pull request in.
Second one for wiring custom XML files to follow next week.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Sebastian Rogers
Sent: Wednesday, January 29, 2020 1:55:05 AM
To: alonrbar/easy-template-x<EMAIL_ADDRESS>alonrbar/easy-template-x<EMAIL_ADDRESS>Cc: Author<EMAIL_ADDRESS>Subject: RE: [alonrbar/easy-template-x] Extensions (#23)
Understood – I will be able to do a new pull request based on these recommendations next week.
From: alonrbar<EMAIL_ADDRESS>Sent: 29 January 2020 09:47
To: alonrbar/easy-template-x<EMAIL_ADDRESS>Cc: Sebastian Rogers<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [alonrbar/easy-template-x] Extensions (#23)
Also, I know I suggested to PR the extension but to avoid unnecessary diffs it may be better to PR the develop branch.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/alonrbar/easy-template-x/pull/23?email_source=notifications&email_token=ABNUOXFHZYG4UHVWVSFV4ZLRAFGANA5CNFSM4KGZT62KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKGSPAI#issuecomment-579676033, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABNUOXBZ2BZLCSBQVP3WL2LRAFGANANCNFSM4KGZT62A.

Great! Will go over it today.

Closing this PR - replaced by #24.


Azure App Service: 502.5 – ANCM Out-Of-Process Startup Failure error

I have a .Net Core app that I added support for App Configuration using a managed Identity.  It works perfectly when I am debugging and testing on my local machine, but when I deploy to an Azure App Service, I get a "502.5 - ANCM Out-Of-Process Startup Failure error".
If I use the connection string to connect to the App Configuration service, I do not have any problems when deployed to an Azure App Service.
Here are the exceptions from the eventlog when using the Managed Identity:
Framework Version: v4.0.30319
Description: The process was terminated due to an unhandled exception.
Exception Info: System.ArgumentException
at Microsoft.Azure.AppConfiguration.ManagedIdentityConnector.ManagedIdentityConnector+<GetConnectionString>d__4.MoveNext()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(System.Threading.Tasks.Task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(System.Threading.Tasks.Task)
at Microsoft.Azure.AppConfiguration.ManagedIdentityConnector.AzconfigClientFactory+<CreateClient>d__0.MoveNext()
Exception Info: System.AggregateException
at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean)
at System.Threading.Tasks.Task1[[System.__Canon, mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089]].GetResultCore(Boolean)    at System.Threading.Tasks.Task1[[System.__Canon, mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089]].get_Result()
at Microsoft.Extensions.Configuration.AzureAppConfiguration.AzureAppConfigurationOptions.ConnectWithManagedIdentity(System.String)
at growboxPortal.Program+<>c__DisplayClass1_0.<CreateWebHostBuilder>b__2(Microsoft.Extensions.Configuration.AzureAppConfiguration.AzureAppConfigurationOptions)
at Microsoft.Extensions.Configuration.AzureAppConfiguration.AzureAppConfigurationSource+<>c__DisplayClass2_0.<.ctor>b__0()
at Microsoft.Extensions.Configuration.AzureAppConfiguration.AzureAppConfigurationSource.Build(Microsoft.Extensions.Configuration.IConfigurationBuilder)
at Microsoft.Extensions.Configuration.ConfigurationBuilder.Build()
at Microsoft.AspNetCore.Hosting.WebHostBuilder.BuildCommonServices(System.AggregateException ByRef)
at Microsoft.AspNetCore.Hosting.WebHostBuilder.Build()
at growboxPortal.Program.Main(System.String[])

@maphil007 just to confirm, did you grant your app service access to the App Configuration?
https://docs.microsoft.com/en-us/azure/azure-app-configuration/howto-integrate-azure-managed-service-identity#grant-access-to-app-configuration

@maphil007
Are you using system assigned identity for the App Service or a User assigned identity?

@maphil007
Are you using system assigned identity for the App Service or a User assigned identity?

It is a system assigned identity that does have contributor role access to the app configuration.

We haven't seen this error happen on our end. I'm going to try to reproduce what you're doing to see if we can figure out what's going on. I'll need some info to do it.
What package of Microsoft.Extensions.Configuration.AzureAppConfiguration are you using?
What version of ASP.NET Core runtime are you using?
Are you using Kestrel or HTTP Server?

@maphil007 Was unable to reproduce the issue just now using

Microsoft.ExtensionsConfiguration.AzureAppConfiguration: 1.0.0-preview-008920001-990
ASP.Net Core 2.1
Default ASP.NET Core web host.
Azure App Service

So it must be some specific combination causing the issue. As soon as I get the setup you're using I'll try to come close to it and reproduce it again.

@jimmyca15 I have the following setup:

Microsoft.ExtensionsConfiguration.AzureAppConfiguration: 1.0.0-preview-008920001-990
ASP.Net Core 2.2 on .NET Framework 4.6.1
Default ASP.NET Core web host
Standard SPA Angular template
Azure App Service

@maphil007 thanks a ton, that helps a lot. I'm going to use your setup and see if we can force the issue to see what's happening.

I created two netfx 4.6.1 projects and deployed them to azure to test this out. You can find the source code here for comparison: https://github.com/jimmyca15/ASPNetCoreUsingAzureAppConfigurationOn461
No issues were encountered.
The exception at the top of the stack you provided was an ArgumentException. We throw that if

appconfig uri is null
appconfig is not an absolute uri
The identity does not have access to a config store with that uri.

If the uri is not hardcoded and is coming from configuration, is it possible that the key to pull it from configuration (environment variable) is wrong? Or isn't there at all?

@maphil007 did you get a chance to review @jimmyca15's question?

I managed to get some time to try this out again and I am still getting the same problem.  I have checked the config in the scm portal and it is pointing to the correct uri.  If I use the uri to the app config store, when deployed to an Azure App Service, I keep getting the 502.5 error when starting.  If I change back to the connection string, it works fine.  I do not have this problem with either config setup when testing in VS on my local dev machine.
I checked through the code that @jimmyca15 posted and I am doing exactly the same thing.
@zhenlan do you have some time to take this offline and have a look at what is happening?

@maphil007 thanks for giving this another try. I am wondering if it's possible you deploy @jimmyca15's app to your app service. It will be helpful to narrow down the issue to the app issue or the environment issue.

@zhenlan do you have some time to take this offline and have a look at what is happening?

Absolutely. You can also reach us through either slack or email.

It would be extremely helpful when troubleshooting this error to address #133

With the v3 release of config provider (release notes), the managed identity support has been revised.
Closing this issue.


feat: Public a function add_cursor(row, col) to add cursor in customized way

Public a function add_cursor(row, col) to add cursor in customized way, such as flash.nvim

I've added a new function in the feat_add_cursor branch. The signature is add_cursor(lnum, col, curswant). Is that OK?

Maybe a seperate init() is required, because keymap set is too early.

You could take a look at this plugin vscode-multi-cursor.nvim

Maybe a seperate init() is required, because keymap set is too early.

Do you get an error? Is there a problem with settings key maps when add_cursor is called?

Maybe a seperate init() is required, because keymap set is too early.

Do you get an error? Is there a problem with settings key maps when add_cursor is called?

No, I mean, keymap set too earl that I can't type letters to choose label in flash, I need to init keymaps manually

I'm not sure how flash.nvim works, and I haven't looked much at how this plugin integrates with other plugins. Is there anyway you can create the cursors after choosing labels?

Yes, flash.nvim allow you to search for pattern, and display labels for user to press and jump, (like hop and easymotion).
And it provide a hook function to do something when user input a label, for me, I want add a cursor at label position that have choosed

Otherwise is there a way you can save the new cursor positions somewhere, then add them all at once?
Sounds like a good idea


Dockerfile Issue during environment setup

Hello mentors! @kamoltat @amathuria
I was successful in setting up the environment before the new merge. But after the merge when I tried again it was showing error as follow-

But when I removed  "COPY .env* /teuthology_api/ " from Teuthology API's dockerfile. I didn't encounter any error.  I am not sure how this is affecting environment, but I think removing it might helps in making installation easy.
Do let me know if I am right or not. Thank you!

@Med16-11 should be good now, let me know if there are still problens


Terminal: inline chat widget is not consistently hidden

For the voice integration to chat, I need to rely on an event when inline chat is hidden. I found that there is a hide method:
https://github.com/microsoft/vscode/blob/743d036d66778d17eb1240dcb110d48e3087d324/src/vs/workbench/contrib/terminalContrib/chat/browser/terminalChatWidget.ts#L161
However, I see that it is not being called in these cases:

open a terminal in panel and start inline chat, start voice chat
open another panel so that terminal is hidden
🐛 voice chat does not stop

A similar issue when terminals are in the editor and you open another editor.
Would it be possible for terminal to ensure hide is called in these cases? I have https://github.com/microsoft/vscode/pull/212653 for then stopping the voice chat session, but it needs to be called.

Thanks, but it does not work for me, when this._focusTracker.onDidBlur is called, this._instance.isVisible is still true. I tried this:

open a terminal, open inline chat
start recording voice (notice on macOS how a new yellow icon appears in the top right indicating audio recording)
switch to another panel
recording does not stop

If I have a chat open in a terminal (panel) and click on another panel, the chat is still not getting hidden, so voice actions continue to work even though not visible.

That also happens in the chat in the editor, so I was following suit. When I hide the editor by clicking a different one, if there's been a response, the voice actions continue to work even though they're not visible.
https://github.com/microsoft/vscode/assets/29464607/0f26dbd8-e0ea-4ba5-91c4-898aca52ab30

Yeah fair, as long as the voice stuff can still be turned off from the opened widget.


[AutoPR] policyinsights/resource-manager

Created to accumulate context: policyinsights/resource-manager

Closing PRs that have been open for more than 1 month.