Ninja: Cross-compiling in subfolder fails

Bug description
When cross-compiling a static library in a subfolder, the generated ninja doesn't work and fail with this error:
CreateProcess failed: The system cannot find the file specified.

Tools

SCons v4.5.2
Cross-compilation with a GCC-like compiler
Windows Environment
Ninja 1.11.1

Setup/To redproduce
SConstruct:
import os
SetOption('experimental', 'ninja')

# Define cross-compilation variables
env_options = {
    "PLATFORM": "posix",
    "HOST_OS": "posix",
    "CC"    : "ccppc",
    "LD"    : "ldppc",
    "AR"    : "arppc",
    "CCCOM" : '$CC $CFLAGS $_CPPINCFLAGS -c -o $TARGET $SOURCES',
    "LINKCOM" : "$LD $LINKFLAGS -o $TARGET $SOURCES $_LIBDIRFLAGS $_LIBFLAGS",
    "CFLAGS": "-MP -MMD -MF ${TARGET}.d -fno-cprop-registers -fno-guess-branch-probability",
    "PROGSUFFIX" : "",
}

# Create an environmnet
env = Environment(**env_options)
env.Tool('ninja')
env.Append(
            ENV = {'PATH' : os.environ['PATH']},
            CPPPATH = [
                '#/dependencies/a',
                '#/dependencies/b',
                '#/dependencies/c',
                '#/dependencies/d',

            ],
            LINKFLAGS = [],
            #CCCOMSTR = "Compiling $TARGET",
            LINKCOMSTR = "Linking $TARGET"
        )

# Export the environement for build scripts
Export('env')

# Start the build
src_obj = SConscript('SConscript')

SConsript:
Import('env')

# Build the manual code lib
SConscript('subfolder-a\subfolder-b\SConscript')

subfolder-a\subfolder-b\SConscript:
Import('env')
LIB_MANUAL = env.Library(target = "LIB_MANUAL",
            source = [
                       'src/filea.c',
                       'src/fileb.c',
                       'src/filec.c',
                       'src/filed.c'
           ])

in your list of sources in the subdir sconscript, the items are not properly comma-separated, which means string catenation is in effect.  Is that just a transcription error, or does it change the behavior to fix that?

Yes it's a transcription error my bad, I'm going to correct that

@mwichmann ccppc is sadly a private compiler, but if that helps, it's GCC 3.3.2 based compiled for windows (Yes an antique)

Maybe post a snip from the build.ninja file that shows the instruction gone wrong (and a bit more of the error message)? Commenting out the cross-compile bits it "looks okay to me" here (I'm not the ninja expert at all, though). e.g., produces stuff like: ```
build subA/subB/src/filea.o: CC subA/subB/src/filea.c | /usr/lib64/ccache/gcc subA/subB/src/filea.c
CC = gcc
env =
rspc = "-MP" "-MMD" "-MF" "subA/subB/src/filea.o.d" "-fno-cprop-registers" $
"-fno-guess-branch-probability" "-Idependencies/a" "-Idependencies/b" "-Idependencies/c" $
"-Idependencies/d" "-c" "-o" "subA/subB/src/filea.o" "subA/subB/src/filea.c"

Here is my output:

Here is my output:
I will see what part of the Ninja file I can share

Please paste the text. Posting a screenshot is far less useful and harder to read/use.

Can't tell much from that, personally, but the line that starts with BUILD looks like the end is garbled?

BTW, when I asked can you post more, I was hoping the sample pasted into the issue here was a working (broken) one, and you could paste info from that.

Here is one of the generated ninja rule:
build build\vaps_gen\QCG\main\fileA.o: CC vaps_gen\QCG\main\fileA.c | $
    vaps_gen\QCG\main\fileA.c
  CC = ccppc
  env = export PATHEXT='.COM;.EXE;.BAT;.CMD';
  rspc = "-MMD" "-MF" "build/vaps_gen/QCG/main/fileA.o.d" "-fno-cprop-registers" $
      "-fno-guess-branch-probability" "-fno-delayed-branch" "-fno-if-conversion2" $
      "-fno-if-conversion" "-fno-crossjumping" "-fno-loop-optimize" "-fno-thread-jumps" $
      "-fno-merge-constants" "-fno-defer-pop" "-fno-zero-initialized-in-bss" "-fsigned-char" $
      "-mcpu=603" "-mstrict-align" "-G" "0" "-g" "-O1" "-fvolatile" "-fno-builtin" "-Wall" $
      "-mvthreads" "-DCPU=PPC603" "-DTOOL=gnu" "-D__TARGET__" $
      "-Idependencies/CIG_PART_HMI/DEPA/" $
      "-Idependencies/MRCL_CIG_LIB/DEPB/" $
      "-Idependencies/MRCL_CIG_LIB/DEPC/" $
      "-Idependencies/MRCL_CIG_LIB/DEPD/" $
      "-Idependencies/MRCL_CIG_LIB/DEPE" "-c" "-o" $
      "build/vaps_gen/QCG/main/filaA.o" "vaps_gen/QCG/main/fileA.c"

And the text outut:
scons: Reading SConscript files ...
Initializing ninja tool... this feature is experimental. SCons internals and all environments will be affected.
SCons running in ninja mode. build.ninja will be generated.
scons: done reading SConscript files.
scons: Building targets ...
 ALL MY DEPS... ] "-c" "-o" "build/vaps_gen/QCG/main/fileA.o" "vaps_gen/QCG/main/fileA.c"
CreateProcess failed: The system cannot find the file specified.
ninja: build stopped: subcommand failed.scons: *** [build.ninja] CalledProcessError : Command 'ninja' returned non-zero exit status 1.
Traceback (most recent call last):
  File "C:\Tools\DEV\scons\scripts\..\SCons\Action.py", line 1310, in execute
    result = self.execfunction(target=target, source=rsources, env=env)
  File "C:\Tools\DEV\scons\scripts\..\SCons\Tool\ninja\__init__.py", line 123, in ninja_builder
    for output in execute_ninja():
  File "C:\Tools\DEV\scons\scripts\..\SCons\Tool\ninja\__init__.py", line 120, in execute_ninja
    raise subprocess.CalledProcessError(return_code, 'ninja')
subprocess.CalledProcessError: Command 'ninja' returned non-zero exit status 1.
scons: building terminated because of errors.

BTW, when I asked can you post more, I was hoping the sample pasted into the issue here was a working (broken) repro, and you could paste info from that.

I'm sorry I can't dedicate much more time for a sample repo right now, but if I can I will

@mwichmann Is it possible that the text after BUILD are terminal control characters?
The exact sequence shows up in (at least the first link):
https://stackoverflow.com/questions/66395499/sbt-has-lost-its-ability-to-format-a-screen-with-ctrl-characters-why
https://stackoverflow.com/questions/16755142/how-to-make-win32-console-recognize-ansi-vt100-escape-sequences-in-c#comment92954461_16799175

How did you install ninja on your system?
Where is the binary located on your system?

From the error, looks like either ninja couldn't be found, or ccppc couldn't be found.
Can you try specifing ccppc in your example SConstruct  with it's full path rather than just as ccppc ?

ninja produces this error when the binary it's trying to execute is not in the path:

CreateProcess failed: The system cannot find the file specified.

It is trying to create a subprocess to perform some task and the binary it tried to use in that process creation can not be found.
SCons does not generally rely on shell environment from the shell its launched from (unless you manually forced it do so) and so SCons is constructing a path internally for the commands it needs to run. Ninja however requires you to setup the shell, because it is launching the process with the given binary from the current shell environment.

looking at your SConstruct a little closer I see its setting the PLATFORM to 'posix', this is causing ninja to export to set things in ninja commands:
export PATHEXT='.COM;.EXE;.BAT;.CMD';ccppc ...
The export command is not in the default windows shell environment (cmd.exe), so it could that the The system cannot find the file specified. message is actually from export.

How did you install ninja on your system? Where is the binary located on your system?

I tried two ways, one with the executable inside the root folder of the project and one with the pip install ninja. Both gave the same output

Can you try specifing ccppc in your example SConstruct with it's full path rather than just as ccppc ?
I will try a little later

looking at your SConstruct a little closer I see its setting the PLATFORM to 'posix', [,,,]

Yes sadly it's a bit of a tricky environment where I'm on windows but I need posix-style commands/arguments. I tried export and it indeed doesn't work.


Update variant list to use label

#1327
@pnadolny13 thoughts on this PR?

@pnadolny13 ping here when you get a chance.

@tayloramurphy I'm good with renaming to remove the "Community" part. Its a little tricky though because something like tap-github is truly maintained by the community although its in meltanolabs. Maybe in the future we can differentiate them we wanted.
I open to the change but honestly can't decide how I feel about using the maintainer label as the variant name. On one hand its a nicer format but on the other it could be confusing to see the Simon Data variant listed on the hub but then you need to install it as the radico variant.

didn't mean to close 🤦
@pnadolny13 we could change the language from Available Variants to Maintainer Variants or Contributor Variants. We say in https://docs.meltano.com/concepts/plugins#variants "Alternative Implementations" which could work too.
They'd have to click in to get the exact command to run still.

@pnadolny13 updated to "Available Implementations"! If you'll approve we can merge.


WFCORE-6908 Remove usage of deprecated AbstractAddStepHandler constructors in wildfly-controller/host-controller/server

https://issues.redhat.com/browse/WFCORE-6908

Core -> Full Integration Build 13981 outcome was UNKNOWN using a merge of 205471c56c570b8b3f9fd92e99bf7e6a09ef0e2e
Summary: Canceled (Tests passed: 6033, ignored: 72; exit code 143 (Step: Test full with JDK 11 (Maven)) (new)) Build time: 06:02:20

Thanks @pferraro !


Crash - no available platforms on Xcode Server

Happens on certain instances of Xcode Server
Application Specific Information:
fatal error: Couldn't find intended platform in list of platforms: []!: file /Users/honzadvorsky/Documents/Buildasaur/Pods/XcodeServerSDK/XcodeServerSDK/API Routes/XcodeServer+Bot.swift, line 172

Originally reported by @valeriomazzeo in https://github.com/czechboy0/Buildasaur/issues/143.

Solution:

[ ] instead of crashing, just generate an error describing this issue and fail the bot-creation process
[ ] make sure Buildasaur shows this error in the UI

You rather want it to return some kind of completion or simply throw?

I think if we want to fail bot creation process we should stick with throw 😉 I'll try to do something with this issue today...

It's an asynchronous process so I don't think throw will work. Just
creating an error and completing should be enough.
On Thu, Oct 1, 2015 at 9:53 AM Mateusz Zając<EMAIL_ADDRESS>wrote:

I think if we want to fail bot creation process we should stick with throw
😉 I'll try to do something with this issue today...
—
Reply to this email directly or view it on GitHub
https://github.com/czechboy0/XcodeServerSDK/issues/114#issuecomment-144659844
.

private func replacePlaceholderPlatformInBot(bot: Bot, platforms: [DevicePlatform]) is the method in which the fatalErrors are called... Looks to me like it's totally synchronous. The bot creation isn't synchronous... Anyway, you suggest returning of the optional ErrorType from this method? Seems legit 😉

Oh right, I was talking about the big picture of the whole process. Makes
perfect sense here to throw, sorry. What I meant is that that error should
then be fed into the failing completion block.
On Thu, Oct 1, 2015 at 10:02 AM Mateusz Zając<EMAIL_ADDRESS>wrote:

private func replacePlaceholderPlatformInBot(bot: Bot, platforms:
[DevicePlatform]) is the method in which the fatalErrors are called...
Looks to me like it's totally synchronous. The bot creation isn't
synchronous... Anyway, you suggest returning of the optional ErrorType
from this method? Seems legit 😉
—
Reply to this email directly or view it on GitHub
https://github.com/czechboy0/XcodeServerSDK/issues/114#issuecomment-144662510
.

Communication is always an issue in my case 😜 Great that we're now on the same tracks!


feat(FocusZone): Export Focus Zone utilities

FocusZone utilities have been proven to be useful for Stardust users. For e.g, finding next focusable elements or previous tabbable element, etc.. So let's export it!

could you, please, suggest what was exact reason (use case) for making these changes? Also, I am concerned about all the functionality of this module being implicitly part of public API - do we really need to export everything?

updated, exporting now 2 functions


ImportImagePlugin should create ImageStream if it doesn't exist yet

See https://github.com/projectatomic/osbs-client/pull/181#issuecomment-127175198 for reasoning/explanation.

It will need to be given an extra argument for the dockerImageRepository.
Also, osbs-client will need to provide API to create an ImageStream.

So just to clarify, this is the registry passed to osbs-client when invoking build + image name, correct? (For example registry.foo.com/fedora:22)

Also, I think it should accept two arguments - name of imagestream and dockerimagerepository.

Oh, right, because we're not putting an imagestream label in the BuildConfig metadata any more.


old MyDashWallet building guide

Hello, it seems the new wallet is depended on old wallet for making transactions right? so where is the old MyDashWallet building guide?

@BenjaminNitschke any help on building old wallet? https://github.com/DeltaEngine/MyDashWallet/tree/54b73658120eb601929011e644792bd6a1291d84

sorry, we are not developing the old api atm, if you have any question for the new website, let us know.
the new site has no backend, but since not all features are 100% available in javascript right now, it depends on some very simple calls to build raw transactions and send signed transactions into the dash network, you can reproduce them in few lines of code in any language (e.g. node.js, c++, python, c#, java, doesn't matter).
also it seems like our proposal to continue MyDashWallet development is not getting much support, so we are not inclined to put much more time into the new website to replace all the old api calls into new plain javascript and explorer calls.

Also just noticed that the published code contains old unused asp.net core code, which is not needed, the new site runs purely on React.js

@BenjaminNitschke yes, got it thanks.


⚠ Bofrost.at has degraded performance

In 60bbe35, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2134 ms

Resolved: Bofrost.at performance has improved in 8ec04c0 after 6 minutes.


At the end of a round, shuffle the modifiers deck if a *2 or Null modifier was drawn

Per the rule book :

This pr adds this functionality https://github.com/orichalque/yet-an-other-gloomhaven-companion/pull/129

This pr adds this functionality https://github.com/orichalque/yet-an-other-gloomhaven-companion/pull/129


Move mode argument to states or forward function?

Currently, train vs. test mode is defined at module construction. This means that if a user wants to run in both modes (e.g. evaluate accuracy on validation examples every 1000 training steps) that the user needs to construct the module twice (or write code to implicitly do so).
Example version A:
import xnn
train_forward, params, states  = xnn.Sequential(
  xnn.Embed(1024, 32),
  xnn.Conv(32, 64, (3,)),
  xnn.ReLU(),
  xnn.Dropout(0.3),
  xnn.Conv(64, 32, (3,)),
  xnn.ReLU(),
  xnn.Dropout(0.3),
  xnn.Mean(axis=-1),
)
test_forward, _, _  = xnn.Sequential(
  xnn.Embed(1024, 32),
  xnn.Conv(32, 64, (3,)),
  xnn.ReLU(),
  xnn.Dropout(0.3, mode='test'),
  xnn.Conv(64, 32, (3,)),
  xnn.ReLU(),
  xnn.Dropout(0.3, mode='test'),
  xnn.Mean(axis=-1),
)
for (i, example) in enumerate(train_set):
  params = update(params, train_forward, states)
  if i % 1000 == 0:
    all_metrics = [evaluate(params, test_forward, states, batch) for batch in eval_set]
    avg_metrics = average(all_metrics)
    log(avg_metrics, step = i)

A more intelligent user might write:
import xnn
def MyModule(mode='train'):
  return xnn.Sequential(
    xnn.Embed(1024, 32),
    xnn.Conv(32, 64, (3,)),
    xnn.ReLU(),
    xnn.Dropout(0.3, mode = mode),
    xnn.Conv(64, 32, (3,)),
    xnn.ReLU(),
    xnn.Dropout(0.3, mode = mode),
    xnn.Mean(axis=-1),
   )
train_forward, params, state = MyModule(mode='train')
test_forward, _, _ = MyModule(mode='test')

for (i, example) in enumerate(train_set):
  params = update(params, train_forward, states)
  if i % 1000 == 0:
    all_metrics = [evaluate(params, test_forward, states, batch) for batch in eval_set]
    avg_metrics = average(all_metrics)
    log(avg_metrics, step = i)

But this still requires: 1. Creating the module twice, 2. Tracking train_forward and test_forward and handling both, 3. Forces the user to have to handle this themself.
Flax & Haiku handle this by passing in a 'deterministic' or 'is_training' keyword argument on the forward call.
Pytorch, equinox, treex handle by calling module.train() and module.eval() (one can imagine an immutable version where: eval_module = module.eval() ... train_module = eval_module.train() or eval_module = set_mode(module, 'eval') ... train_module = set_mode(eval_module, 'train'))
I think it would make more sense to move the train vs test mode argument to the forward function or into the states, or a functional way to set the train mode (e.g. eval_module = set_mode(module, 'eval')) particularly as there are a few different modules that behave differently at train and test time (dropout, batch norm, layerdrop, etc.)
Thoughts?

I think it's an awesome idea! I'm leaning towards having a 'mode' entry in the state dictionary for that. It will have to work well with:

Container module constructors such as xnn.Sequential, xnn.Parallel etc.
Vectorization using xnn.vectorize.

For 1, I'm thinking that we should clear the 'mode' state for all modules when constructing the container, and only change the top-level 'mode' state entry when calling set_mode. In container forward, we insert the 'mode' entry to all the constructed states for submodules.
For 2, we need to let xnn.vectorize ignore the 'mode' entry so that it does not vectorize string states.
Want to prototype this and send a pull request?

A 3rd requirement:

It has to work with jax.jit.

This means that we cannot use Python strings as values for mode. Perhaps an integer ENUM constant should do.


Skip the flaky test: testGetsWorkingGroupLens

Opens https://linear.app/sourcegraph/issue/CODY-2665/fix-flaky-test-testgetsworkinggrouplens.
Opens https://linear.app/sourcegraph/issue/CODY-2667/fix-flaky-test-testundo.
Test plan

Green CI

I think they all are flaky, at least I have seen undo test failing as well.
Is there anything specific about this one that you want to skip only it?

I think they all are flaky, at least I have seen undo test failing as well. Is there anything specific about this one that you want to skip only it?

I want to skip all flaky tests. Do you have an example of a run with undo test failure?

Here's a test run in which four tests fail. The PR doesn't touch the plugin code at all.
https://github.com/sourcegraph/jetbrains/actions/runs/9681170872/job/26711117391?pr=1819

Here's a test run in which four tests fail. The PR doesn't touch the plugin code at all. https://github.com/sourcegraph/jetbrains/actions/runs/9681170872/job/26711117391?pr=1819

@odisseus, it was before https://github.com/sourcegraph/jetbrains/pull/1829. Now we have seen only the mentioned two failing.


Archive this repo

On my search for django and fullcalendar apps, I found this software, but it is horribly outdated, and seems completely unmaintained. Fullcalendar is not at 1.x but 5.x AT, and the repo was updated last in 2015, this is 6 years ago.
Would you consider to archive this repository, so that if someone finds it accidentally, he/she is not inclined to use it (and waste time)...
Thanks!

Hi, @nerdoc ! Thanks for taking the time to open this issue. You're right, I totally forgot about this repo. I'm going to archive it now.
Thank you


Wrong hash for the original map file

I was getting the error 'Could not find valid underpass map file.!' as probably many others. It is because manageMapFiles.js originalMapHashes array doesnt include the correct hash. I added this hash into the array and now it works as expected: '1202a06278bb9a4b39e48e8b1cd7bdce'

This is correct. For some reason, Psyonix makes tiny changes in every little update to the underpass map file and it needs a new hash. I am updating this behavior to have a more safe way of storing the map file so the user can't overwrite it and then I can abandon the hash comparisons all together.


Enable to fix the position of figure/table

With the package float and the code  \begin{figure}[H] , you can fix the figure/table exactly after the text above.

Thank you for your pull request!
It’s correct to use option [H] to force LaTeX to place the figure exactly here. However, the reason why we use floating bodies is that it can place figures and tables in appropriate positions automatically. If you use option [H] and then insert some text above the figure, it may result in large blank at the bottom of the page. So I think in most cases, it’s better to use options like [ht] (give more than one options and let LaTeX to choose where to place the figure) instead of [H].


Container dos not start when change volume in kitmatic

Expected behavior
Container star
Actual behavior
Container does not start
Information about the Issue
After changing the volumes path the following message appears (HTTP code 404) no such image - no such image: linuxserver/radarr:latest: No such image: linuxserver/radarr:latestand the container does not start
Steps to reproduce the behavior

... Change Volumes
...

Change volume, or change expose port -> container stop, with start/immediate-exit situation.


Move registerModelView behind proposed API flag

We should use the proposedApiFunction(extension,...) flag handling in sqlExtHost.api.impl. Any feature using proposed APIs should only be able to get them if the enabledProposedFeatures flag is enabled, they're doing ext host debugging etc. It shouldn't be on by default to avoid extensions shipping with breakable changes.
This matches how VSCode handles proposed APIs.

Not needed / closing for now


fix: circular dependencies

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[ ] I have tested these changes either locally on my machine, or GitPod.

Closes #45307
Closes #45277

So I'm taking a stab at this.
Currently in ESLint config, I've set the maxDepth for the no-cycle rule to 2. The command lint:js took 50s to run in my system and here's the current output:

Click to expand
><EMAIL_ADDRESS>lint:js
> eslint --max-warnings 0 .

The circular loop here is that both index.js(same directory) and code-lock-epic.js are importing each other. So I moved the Redux action for unlockCode to code-lock-epic.js to get rid of the import which is causing the error. Now index.js is only importing the respective epic(codeLockEpic) and no circular loops are present. Basically the loops are happening because the epics are imported from the individual epic files to their respective index.js and those individual files import index.js again for the Redux action to be used in the same epic.
I have a solution in mind for the remaining files, which is we move all the Redux actions to their respective epic files. Yes by doing this the Actions will be split out and stored in a lot of files instead of being stored in one single file but we get rid of the circular dependencies.

Currently I've set the maxDepth value to 1 so the test passes without running out of memory.

@Sboonny,
Can you confirm you intended to mark this for review because the checks aren't passing yet?

I was assigned immediately after this came to my attention, when it came to my attention the test were still running so I didn't know about them failing sorry about the label.
About the review, I don't know how to test it and you seemed the best to review it and give guidance .

cc - @ojeytonwilliams


[TODO] Linux: remove the libgtk-3-dev dependency

We want to load GTK functions dynamically instead of making it a hard dependency to simplify the build process on Linux platforms.

Note to self: the Telegram team has code to load GTK3 dynamically.


Fix crush single message inside long message

Add test for single message inside long message

Fix crush single message inside long message

Note: adapted from fix originally submitted by @a1ien to OpenCyphal v0.3
Reference:
https://github.com/OpenCyphal/libcanard/commit/b7e2be5fc3b65fbf34dfced1a4e572697bb90989

@tridge ?

@adolfogc interesting, thanks!
we'll need to do some testing of this


SliverAppBar snap broken animation

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Add SliverAppBar pinned: true, floating: true, snap:true
Add some any EditableText
Scroll down so appbar isn't visible
Tap into EditableText to get focus and keyboard appeared
Get broken animation

Expected results
Normal animation
Actual results
Animation is freezing or glitching
Code sample
Code sample
import 'package:flutter/material.dart';
import 'package:flutter/rendering.dart';

void main() => runApp(const NestedScrollViewExampleApp());

class NestedScrollViewExampleApp extends StatelessWidget {
  const NestedScrollViewExampleApp({super.key});

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      home: NestedScrollViewExample(),
    );
  }
}

class NestedScrollViewExample extends StatefulWidget {
  const NestedScrollViewExample({super.key});

  @override
  State<NestedScrollViewExample> createState() =>
      _NestedScrollViewExampleState();
}

class _NestedScrollViewExampleState extends State<NestedScrollViewExample> {
  @override
  Widget build(BuildContext context) {
    final List<String> tabs = <String>['Tab 1', 'Tab 2'];
    return DefaultTabController(
      length: tabs.length,
      child: Scaffold(
        body: NestedScrollView(
          headerSliverBuilder: (BuildContext context, bool innerBoxIsScrolled) {
            return <Widget>[
              SliverOverlapAbsorber(
                handle:
                    NestedScrollView.sliverOverlapAbsorberHandleFor(context),
                sliver: SliverAppBar(
                  surfaceTintColor: Colors.transparent,
                  leading: IconButton(
                    onPressed: () {},
                    icon: const Icon(Icons.close),
                  ),
                  title: const Text('Books'),
                  pinned: true,
                  snap: true,
                  floating: true,
                  expandedHeight: 180.0,
                  forceElevated: innerBoxIsScrolled,
                  bottom: PreferredSize(
                    preferredSize: const Size.fromHeight(128),
                    child: Column(
                      mainAxisSize: MainAxisSize.min,
                      children: [
                        const Padding(
                          padding: EdgeInsets.symmetric(horizontal: 16.0),
                          child: TextField(
                            decoration: InputDecoration(label: Text('Input')),
                          ),
                        ),
                        TabBar(
                          // These are the widgets to put in each tab in the tab bar.
                          tabs: tabs
                              .map((String name) => Tab(text: name))
                              .toList(),
                        )
                      ],
                    ),
                  ),
                ),
              ),
            ];
          },
          body: TabBarView(
            children: tabs.map((String name) {
              return SafeArea(
                top: false,
                bottom: false,
                child: Builder(
                  builder: (BuildContext context) {
                    return CustomScrollView(
                      keyboardDismissBehavior:
                          ScrollViewKeyboardDismissBehavior.onDrag,
                      key: PageStorageKey<String>(name),
                      slivers: <Widget>[
                        SliverOverlapInjector(
                          handle:
                              NestedScrollView.sliverOverlapAbsorberHandleFor(
                                  context),
                        ),
                        SliverPadding(
                          padding: const EdgeInsets.all(8.0),
                          sliver: SliverFixedExtentList(
                            itemExtent: 48.0,
                            delegate: SliverChildBuilderDelegate(
                              (BuildContext context, int index) {
                                return ListTile(
                                  title: Text('Item $index'),
                                );
                              },
                              childCount: 30,
                            ),
                          ),
                        ),
                      ],
                    );
                  },
                ),
              );
            }).toList(),
          ),
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
After research I've found out that the reason for such behaviour is a few things:

If EditableText is focused it will call _scheduleShowCaretOnScreen(withAnimation: true); that will cause animation start of a RenderSliverFloatingPersistentHeader (by calling showOnScreen with provided Duration of 200 milliseconds).

After first step EditableText will handle software keyboard appearance and will call _scheduleShowCaretOnScreen(withAnimation: false); causing insta call of showOnScreen again but now with Duration.zero which leads to broken animation

I've tried to add dummy additional check and it fixes problem:

Without fix:
untitled.webm
With fix:
fixed.webm

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[!] Flutter (Channel master, 3.17.0-1.0.pre.33, on macOS 13.4 22F66 darwin-arm64, locale en-RU)
    ! Upstream repository https://github.com/feduke-nukem/flutter is not a standard remote.
      Set environment variable "FLUTTER_GIT_URL" to https://github.com/feduke-nukem/flutter to dismiss this error.
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] VS Code (version 1.84.0)
[✓] Connected device (4 available)
[✓] Network resources

Hi @feduke-nukem. Thanks for filing this. I can't seem to reproduce this on my end (The keyboard coming up) running the code sample above (tested on real device and emulator). Could you point me to more information on how this could be reproduced with the code sample provided.
untitled.webm
flutter doctor -v

Tap at the header. There is an EditableText

Hi @feduke-nukem. Thanks for filing this. I can't seem to reproduce this on my end (The keyboard coming up) running the code sample above (tested on real device and emulator). Could you point me to more information on how this could be reproduced with the code sample provided.
untitled.webm
flutter doctor -v

I have updated sample code to be more obvious where TextField is placed

Thanks for your response and the information.  Flutter (Channel master, 3.17.0-1.0.pre.60, on macOS 14.0 23A344 darwin-arm64, locale en-NG)
    • Flutter version 3.17.0-1.0.pre.60 on channel master at /Users/damilolaalimi/fvm/versions/master
    ! Warning: `flutter` on your path resolves to /Users/damilolaalimi/sdks/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/damilolaalimi/fvm/versions/master. Consider adding /Users/damilolaalimi/fvm/versions/master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /opt/homebrew/Cellar/dart/3.1.5/libexec/bin/dart, which is not inside your current Flutter SDK checkout at /Users/damilolaalimi/fvm/versions/master. Consider adding /Users/damilolaalimi/fvm/versions/master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 6007b91a37 (8 minutes ago), 2023-11-07 02:43:32 -0500
    • Engine revision 7f4d56a7f4
    • Dart version 3.3.0 (build 3.3.0-102.0.dev)
    • DevTools version 2.30.0-dev.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/damilolaalimi/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A507
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[!] Android Studio (version unknown)
    • Android Studio at /Users/damilolaalimi/Downloads/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

[✓] VS Code (version 1.84.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] VS Code (version 1.83.1)
    • VS Code at /Users/damilolaalimi/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (5 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554                        • android-arm64  • Android 14 (API 34) (emulator)
    • Damilola’s iPhone (mobile)  • 00008110-001964480AE1801E            • ios            • iOS 17.0.2 21A351
    • iPhone 15 Pro Max (mobile)  • 56A225C9-B8F6-4201-A0AE-8C3CFA4CE152 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-17-0 (simulator)
    • macOS (desktop)             • macos                                • darwin-arm64   • macOS 14.0 23A344 darwin-arm64
    • Chrome (web)                • chrome                               • web-javascript • Google Chrome 119.0.6045.105

[!] Network resources
    ✗ A network error occurred while checking "https://github.com/": Operation timed out

! Doctor found issues in 3 categories.

[!] Flutter (Channel stable, 3.13.9, on macOS 14.0 23A344 darwin-arm64, locale
    en-NG)
    • Flutter version 3.13.9 on channel stable at
      /Users/damilolaalimi/sdks/flutter
    ! Warning: `dart` on your path resolves to
      /opt/homebrew/Cellar/dart/3.1.5/libexec/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/damilolaalimi/sdks/flutter.
      Consider adding /Users/damilolaalimi/sdks/flutter/bin to the front of your
      path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d211f42860 (12 days ago), 2023-10-25 13:42:25 -0700
    • Engine revision 0545f8705d
    • Dart version 3.1.5
    • DevTools version 2.25.0
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/damilolaalimi/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)
    ! Some Android licenses not accepted. To resolve this, run: flutter doctor
      --android-licenses

[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A507
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)

[!] Android Studio (version unknown)
    • Android Studio at /Users/damilolaalimi/Downloads/Android Studio
      Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build
      17.0.7+0-17.0.7b1000.6-10550314)

[✓] VS Code (version 1.84.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] VS Code (version 1.83.1)
    • VS Code at /Users/damilolaalimi/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (5 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554                        •
      android-arm64  • Android 14 (API 34) (emulator)
    • Damilola’s iPhone (mobile)  • 00008110-001964480AE1801E            • ios
      • iOS 17.0.2 21A351
    • iPhone 15 Pro Max (mobile)  • 56A225C9-B8F6-4201-A0AE-8C3CFA4CE152 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-17-0 (simulator)
    • macOS (desktop)             • macos                                •
      darwin-arm64   • macOS 14.0 23A344 darwin-arm64
    • Chrome (web)                • chrome                               •
      web-javascript • Google Chrome 119.0.6045.105

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 3 categories.

Thanks for your response and the information. I still can't seem to reproduce the glitch on my end ( or maybe it isn't always reproducible)
stable	master
Simulator.Screen.Recording.-.iPhone.15.Pro.Max.-.2023-11-07.at.09.31.09.mp4
Simulator.Screen.Recording.-.iPhone.15.Pro.Max.-.2023-11-07.at.09.24.24.mp4
stable, master flutter doctor -v

It is actually glitching on the attached videos

Also you could compare this aspect of animation (smoothness I'd like to say) on provided videos to the issue before and after applied fix

I have already created pull request that fixes this issue: https://github.com/flutter/flutter/pull/137913.
I'll be happy to be reviewed
@dam-ease

Removed engine label, this is a framework issue.


Problems just getting the zipped download to work

The repository as downloaded into my apache htdoc folder not working, what I am doing wrong?  When I navigate to the codemirror directory, the index.html page comes up just like the website but it can not find codemirror.js because
    and codemirror.js is in the /src/ directory, not the /lib directory.
If I change the line to src/codemirror.js then it finds it but then I get the JS error:
SyntaxError: import declarations may only appear at top level of a module

Which zip, exactly, are you downloading?

The one right here on github
https://github.com/codemirror/CodeMirror
Green button on the upper right "Clone or download",  click on it, then at the bottom of the little dropdown "Download Zip"  link.

Ah, that's the repository, not a built package. Get it from the website or NPM instead.


Upstream rebase

Don't merge. PR to review changes only.
Important: sgx-runtime is now in the worker repo. Not separate any more (easier to maintain).
TODO:

[x] run enclave tests with ./integritee-service test --unit
[x] cargo test
[x] cargo clippy
[ ] adapt .github/workflows/build_and_test.yml to use Ajuna node and tests
[x] system tests
[ ] clean up PR history. For this, a force push is needed. Not sure if that's a good idea, while this branch is used by many

Closing in favor of https://github.com/ajuna-network/worker/pull/59


Requests to Meetup API are missing headers

I just noticed that the requests to Eventbrite are using headers for authorization. Do the requests to Meetup's API not require any headers for the sake of authorization?
I bring this up because when I go to the Meetup API endpoint that's used in App.py I get this output:
{"errors":[{"code":"auth_fail","message":"Invalid csrf credentials"}]}

@ramonaspence check out this old issue about Meetup v2 vs v3.
I believe we changed the URLs to use the v3 API, but since we're only doing GET requests which are publicly available, I don't think we needed to setup OAuth to make those base requests.
Do you think something you were doing was hitting a v2 API OR trying to pass the &key=  GET parameter which is now deprecated?  We were using key={a value from config.ini} but we simply removed it and started using v3 public GET requests.

Oh I see. So, it was actually an invalid "group name" that I was giving the url.
meetup.api.com/Hack-Greenville/events works for example but,
meetup.api.com/1234567890/events does not work.
So no issue with the headers then.

Nice. Do we need to keep this issue open, or is it good now?

We're good now 👍🏻


Latest date available in line graph is not available in calendar pulldown menu

Aug 29 is currently (as of Aug 30 1:40pm) available for both FB signals and for 'combined', but the calendar pulldown menu is only available till Aug 28 even when any of these signals are selected.

https://deploy-preview-403--cmu-delphi-covidcast.netlify.app/?sensor=fb-survey-smoothed_hh_cmnty_cli&level=county&date=20200828&signalType=value&encoding=color&mode=overview&region=OR

Rating Scale (1 is minor, 7 is severe):  4  (cannot display map of the latest date available)

Context

Browser:

Additional context

@RoniRos from a visualization perspective, if there's not a signal available for the current date, it currently will take you to the latest date.  For example, in currently it looks like "Doctors Visits" have data only to Oct 3., but we have more recent signals from most others.
Would it make more sense to default the map from "Doctors Visits" to Cases or Deaths?

See #279 for context on the default map choice. We wanted to highlight our unique data sources, and Doctor's Visits is a key one, but yes, there are some disadvantages to the choice. We could be convinced to switch.

What he (capnrefsmmat) said. ;-)

Closing this as obsolete since we'll be changing the default view in the redesign and the "latest-date available" issue shouldn't be an issue anymore.


Update follower reads docs to use SET TRANSACTION

... to make the feature easier to access from drivers and ORMs.
Fixes #5613.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/5eab2d8326d241d2f2fba737eb8b6ad0dbdc7e2f/
Edited pages:

v19.2/begin-transaction.md
v19.2/follower-reads.md
v19.2/topology-follower-reads.md

TFTR Nathan!


Set maxConcurrency for move and copy tasks to 1

Ticket: []
Feature flag: n/a

Purpose

Summary of Changes

Screenshot(s)

Side Effects

QA Notes

These look good.


CD_Pair accuracy (PRECISION and EPSILON) and NaN with STPBV

I have some issues with the precision of sch.
When I make a distance query between two hulls, if I repeat the query multiple times (at least twice time) and set
the same configuration to the two hulls, I can obtain a slightly different positions of the closes points.
This is really disturbing for nonlinear optimization, because that can break the continuity of the gradient.
I have been looking at the _PRECISION_ and _EPSILON_ values in CD_Pair.cpp (https://github.com/jrl-umi3218/sch-core/blob/master/src/CD/CD_Pair.cpp#L18-L19).
I feel like _PRECISION_ is not well set.
In the constructor of CD_Pair, precision_ copies the value of _PRECISION_ but in CD_Pair::setRelativePrecision
precision_ is set to the square of the precision given as a parameter.
Also, default _EPSILON_ value is far from the real machine epsilon on 64-bits systems. Why not use std::numeric_limits<double>::epsilon() instead of hard-coding it ?
By calling setRelativePrecision(1e-6) and setEpsilon(std::numeric_limits<double>::epsilon()) (https://github.com/jorisv/PG/blob/master/src/CollisionConstr.cpp#L96-L97) I obtain "better" results (precision-wise) for the closest-point query and my optimization process.
The only issue I have with this solution is when I start using STPBVs. In that case, CD_Pair can return NaN…
It appears that CD_Pair::{det_,lambda0_,lambda1_,lambda2_} values are set to -inf (https://github.com/jrl-umi3218/sch-core/blob/master/src/CD/CD_Pair.cpp#L232-L235).
I still can't have the guarantee to have a continuous gradient by using convex hulls, but the result is still better than when using STPBVs with standard epsilon and precision.

This bug concerns an old version after which, NaN issues have been fixed by f32a846c37b94a790385e14399b32fb91992c22e


Many songs not downloading

I tried downloading a playlist using the link, supplied a subfolder name, but it throws says, java.netMalformedURLException: no protocol:. And java.lang.IndexOutOfBoundsException: Index: 1, Size: 1.
From the playlist of about 3360 songs, about 48 were downloaded correctly.
From around 1500 songs it seemed to retry the skipped songs earlier and jumps between (2-3-5-... upwards) and (1500 and up) and continued this till the playlist was completed (not fully downloaded, but tried to download all songs).

can you share me your Spotify playlist link, I'll see where it goes wrong. Because It worked with a Playlist link of 900 songs for me.

https://open.spotify.com/playlist/4oTx9r8HldEowENK1r971p?si=GWiMtEDtSU6zp1teuXNLxg, is the literal spotify share link, but if I am correct officially, https://open.spotify.com/playlist/4oTx9r8HldEowENK1r971p is enough. The app does not handle these differently as I tried both to be sure.

Found major bugs and are fixed. Released v1.1. It should download all the songs on list now.
There is an issue which still exists. That is if  YouTube search results show same result for two different music because the names are almost similar, it downloads the same music file again and replaces because it thinks it's the same music file. It has to do with search results of YouTube.
Good luck downloading 3000 songs.


Make script to generate gif from active layers

For a quicker preview. Use layer naming to control options of gif. Use:
https://www.lcdf.org/gifsicle/

Code in-progress here:
https://github.com/mhulse/illy-gif


[UI] Adopt fundamental styles - view Configurations

Target

[ ] Adopt the fundamental styles in Configurations view

Use https://github.com/dirigiblelabs/ide-next as reference

Issue cloned to https://github.com/eclipse/dirigible/issues/1521


come up with a way to add addidtional metadata to alternative names

comments, references and dates
names can change due to marriage, pseudonyms can change, there are various types of alternative names

resolved in 36b6003d97455378fca5cd82b02f21a97b21480b


Draft to make configuration file for freezeyt

Draft for #34
at this moment is problem with make cname when we want to freeze some app to certain folder.
Error:
line 87, in freeze
path.joinpath('CNAME').write_text(cname)
FileNotFoundError: [Errno 2] No such file or directory: 'freezeyt_blog/static-pages/CNAME'
I think there is no folder static-pages

Create it with path.mkdir(parents=True, exist_ok=False).
(I didn't look at the change yet.)

Create it with path.mkdir(parents=True, exist_ok=False).
(I didn't look at the change yet.)

I ' ve tried to move certain part of code below creation of new target folder. It seems to be working but I dont know if my idea of configuration is right. So be really observant when you check my code.


python3Packages.tox: drop toml, use tomli for Python < 3.11

In version 3.26.0, the tox package switched from using the toml library to using either tomli or the Python 3.11 built-in tomllib one; see e.g. https://github.com/tox-dev/tox/commit/6b76e18fcaa7c9610b642555bcb94aab1d37f2b3
For an example of Tox failing when trying to parse a pyproject.toml file without this change, see the tox-delay.nix file at https://gitlab.com/ppentchev/utf8-locale/-/commit/fb25e83decb3e349fcced0c368cd25e70c263f2b (passing --arg pkgs 'import <nixpkgs> {}' to nix-shell)
This is my first attempt to contribute to the Nix ecosystem, so, in all probability, I have not done everything right. Thanks in advance for your time and attention, and keep up the great work!
Description of changes
Drop toml from tox's build inputs, add tomli instead for Python < 3.11.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:
[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: apache-airflow, devpi-server, privacyIDEA, and skyfield failed for unrelated reasons.
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

This has been merged in https://github.com/NixOS/nixpkgs/pull/196360 🎉 , so I'll close this PR. Thanks for helping!


Stateless function components cannot be given refs

"react-native": "0.56.0",
"@okgrow/react-native-copilot": "^2.4.1",
simulator iPhone 6 with iOS 12.1
I get the warning "Stateless function components cannot be given refs. Attempts to access this ref will fail" when trying to wrap my custom functional component with walkthroughable. I haven't found any mentioning in the docs that react-native-copilot requires only stateful components to work with. Is there a way to highlight a stateless (functional) component?

Good catch. We may need to document this, if react-nactive-copilot does only require only stateful components.

Copilot needs to access the measure function for the outermost "native" component i.e. , , , etc. which requires refs.
Does that mean that the use of functional component here in the README is incorrect?

Were you able to highlight your custom components?

@mohebifar, the example in docs is correct. If I wrap my functional component inside a RN View like in the example you provided it works as expected. I just thought I could make walkthroughable my functional component directly passing it to the walkthroughable function.
@AishwaryaSurana, just wrap you component like in the link @mohebifar provided.


[Question] Can the Publisher override the Api policy when we publish the APIs across higher environments

Release version
v6.1.1
Question Details
I am trying to deploy my API in to other environments test/test02/test03 as an example. They all have minor differences in their Api policies like caching etc.
Can the Publish tool handle the API Policy overrides?
Expected behavior
NA
Actual behavior
NA
Reproduction Steps
I have a caching policy / call rates policy that are having different config each environment. Would like to know if the publish tool can handle the override at Api Policy?

Yes through the use of namedvalues.
Please check closed issues before opening new ones as this topic has been addressed before.


Doesnt work in v7

Simple as that. Error: Caused by: java.lang.ClassCastException: class arc.math.Rand cannot be cast to class mindustry.entities.Effect (arc.math.Rand and mindustry.entities.Effect are in unnamed module of loader 'app')

duplicated #12


Add a release step to unleash Scala Steward after announcing.

See https://contributors.scala-lang.org/t/coordinate-compiler-and-tooling-releases/5312/45?u=sjrd for context.

Hum, perhaps. I only saw exact versions in that file before, so I followed that lead, and the author did not point out any other way.
And for 2, actually we wouldn't want that anyway, because when we reach Scala.js 2.0.0, supposedly we would still be releasing more 1.x releases until it becomes EOL. And as such, using a greater-than comparison would incorrectly prevent all future releases of Scala.js 2.x to be rolled out; or conversely, would not prevent any 1.x version from being rolled too early anymore.


Renamed instances of 'dockerfile' as 'Dockerfile'

I noticed that several Dockerfiles in this repo were named 'dockerfile' and as such, the syntax highlighting was not working in the github file viewer and other editors. So I've renamed all of the instances of 'dockerfile' as 'Dockerfile'.

Thank you!


Unable to compile on windows with MinGW from scoop

Godot version: https://github.com/godotengine/godot/commit/bcbd480c32f7c9a6fdaad297ef1fbb8fa5fa12ee
OS/device including version: Windows 10
Issue description:
I cloned the project  in C:/godot
I installed scoop and ran scoop install gcc python scons yasm make in my powershell
then I cd c:/godot  in the folder and run scons platform=windows verbose=yes use_mingw=yes arch=x64
I receive the following error:
PS C:\godot> scons platform=windows verbose=yes use_mingw=yes arch=x64
scons: Reading SConscript files ...
Configuring for Windows: target=debug, bits=default
Using MinGW
Checking for C header file mntent.h... no
scons: done reading SConscript files.
scons: Building targets ...
[Initial build] g++ -o platform\windows\godot_windows.windows.tools.x64.o -c -std=gnu++17 -g3 -mwindows -Wall -Wshadow-local -Wno-misleading-indentation -Werror=return-type -DDEBUG_MEMORY_ALLOC -DDISABLE_FORCED_INLINE -DDEBUG_ENABLED -DWINDOWS_SUBSYSTEM_CONSOLE -DWINDOWS_ENABLED -DWASAPI_ENABLED -DWINMIDI_ENABLED -DWINVER=0x0601 -D_WIN32_WINNT=0x0601 -DVULKAN_ENABLED -DMINGW_ENABLED -DMINGW_HAS_SECURE_API=1 -DTOOLS_ENABLED -DMINIZIP_ENABLED -DZSTD_STATIC_LINKING_ONLY -Ithirdparty\freetype\include -Ithirdparty\libpng -Ithirdparty\vulkan -Ithirdparty\vulkan\include -Ithirdparty\vulkan\loader -Ithirdparty\zstd -Ithirdparty\zlib -Iplatform\windows -I. platform\windows\godot_windows.cpp
=====
b''
=====
scons: *** [platform\windows\godot_windows.windows.tools.x64.o] Error 1
scons: building terminated because of errors.

I tried looking up information but I couldn't find anything. The error seems to be related to a char in godot_windows.cpp

Any reason this is still open?

Any reason this is still open?

This was likely resolved by https://github.com/godotengine/godot/pull/66965 and will be further improved by https://github.com/godotengine/godot/pull/94747, closing.


python311Packages.publicsuffixlist: <IP_ADDRESS>31109 -> <IP_ADDRESS>31120

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 268771 run on x86_64-linux 1

14 packages built:

parsedmarc (python311Packages.parsedmarc)
parsedmarc.dist (python311Packages.parsedmarc.dist)
python310Packages.checkdmarc
python310Packages.checkdmarc.dist
python310Packages.parsedmarc
python310Packages.parsedmarc.dist
python310Packages.publicsuffixlist
python310Packages.publicsuffixlist.dist
python311Packages.checkdmarc
python311Packages.checkdmarc.dist
python311Packages.publicsuffixlist
python311Packages.publicsuffixlist.dist
trustymail
trustymail.dist


Added Raspeberrypi project

Added Qemu RaspberryPi simulation Pi-hole

hey please do another PR since there was some issues with the previous PR that caused workflow to trigger issues

Hi @Geetanjali2! Thanks for this detailed documentation on the QEMU Raspberry Pi simulation. I've noticed a few issues causing the build to fail that we can easily fix:
The main issue is with the image paths in your markdown file. The build is failing because it can't find the images due to:

The folder name case mismatch ("playbook" vs "Playbook")
Using backslashes instead of forward slashes in the paths

For example, change:
![QRPS](playbook\img\QRPS2.png)
to:
![QRPS](Playbook/img/QRPS2.png)
You'll need to update all image references in your markdown file to use this correct format. Your images are correctly uploaded in the "Playbook" folder (with capital P), so we just need to make sure the markdown references match this exactly.
I also notice some package updates (package.json and yarn.lock) in this PR. While these updates are valid, it might be better to handle dependency updates in a separate PR to keep the changes focused and easier to review.
Let me know if you need any help making these adjustments!

Hi Amir,
Issues Resolved Can you please review it once again?
Thank you

@Geetanjali2 Hi , Amazing work thanks !


Applies box-sizing = border-box to all elements

Adjusts page container width so it's 1000 again
Adds in inherit for flex

Motivation
The root cause of https://github.com/theramis/aucvid/pull/113 was most probably the removal of tailwindcss - tailwind was apply a lot of resets and defaults, one of was box-sizing: border-box;
Border-box means padding is counted as part of an elements width, which I think IMO is the easiest mental model to work with - although @jishaal @finnclark keen to hear your thoughts.
Changing this, means we do need to go back to the original assumption about max-width - that it's 1000px + space on the sides - On that, I think it's pretty important we have a width in there that matches the designs, otherwise we'll probably run into a scenario where the page will break in unexpected ways, but the designs will look fine. 1000px is nice and easy to remember too!

? The root cause of #113 was most probably the removal of tailwindcss - tailwind was apply a lot of resets and defaults, one of was box-sizing: border-box;
Ah really, I thought this was already applied when I checked the Devtools in one of the previews, but must have been tw giving it as you said 😅
Nice thanks for the fix


examples[ci]: Build examples dir in CI

updated prisma dep to be pinned, was causing issues when it had the carrot(^)

Stale, closing for the foreseeable future


write compaction: chunk 63 not found: invalid encoding "none""

Thanos, Prometheus and Golang version used
Thanos  Version: 0.5.0
Prometheus Version: 2.7.2
What happened
What you expected to happen
How to reproduce it (as minimally and precisely as possible):
Full logs to relevant components
Logs

level=debug ts=2019-07-22T11:29:07.217897384Z caller=compact.go:824 compactionGroup="0@{cluster_replica=\"shenzhen_test\",dc_replica=\"lab\",project_replica=\"test\",prometheus=\"monitoring/k8s\",prometheus_replica=\"prometheus-k8s-1\"}" msg="downloaded and verified blocks" blocks="[/var/thanos/store/monitoring-thanos/compact/0@{cluster_replica=\"shenzhen_test\",dc_replica=\"lab\",project_replica=\"test\",prometheus=\"monitoring/k8s\",prometheus_replica=\"prometheus-k8s-1\"}/01DG5S1JWSR4HMJWDDVR9ZV7AT /var/thanos/store/monitoring-thanos/compact/0@{cluster_replica=\"shenzhen_test\",dc_replica=\"lab\",project_replica=\"test\",prometheus=\"monitoring/k8s\",prometheus_replica=\"prometheus-k8s-1\"}/01DG6Q0F4K5BVCR334NJB92CMX /var/thanos/store/monitoring-thanos/compact/0@{cluster_replica=\"shenzhen_test\",dc_replica=\"lab\",project_replica=\"test\",prometheus=\"monitoring/k8s\",prometheus_replica=\"prometheus-k8s-1\"}/01DG90J9GADDXQVMNJ9EX2VHJA /var/thanos/store/monitoring-thanos/compact/0@{cluster_replica=\"shenzhen_test\",dc_replica=\"lab\",project_replica=\"test\",prometheus=\"monitoring/k8s\",prometheus_replica=\"prometheus-k8s-1\"}/01DG9E988WREQM2XKTK3SVGYFQ]" duration=1m1.245448142s
level=error ts=2019-07-22T11:29:08.386446072Z caller=main.go:182 msg="running command failed" err="error executing compaction: compaction failed: compaction failed for group 0@{cluster_replica=\"shenzhen_test\",dc_replica=\"lab\",project_replica=\"test\",prometheus=\"monitoring/k8s\",prometheus_replica=\"prometheus-k8s-1\"}: compact blocks [/var/thanos/store/monitoring-thanos/compact/0@{cluster_replica=\"shenzhen_test\",dc_replica=\"lab\",project_replica=\"test\",prometheus=\"monitoring/k8s\",prometheus_replica=\"prometheus-k8s-1\"}/01DG5S1JWSR4HMJWDDVR9ZV7AT /var/thanos/store/monitoring-thanos/compact/0@{cluster_replica=\"shenzhen_test\",dc_replica=\"lab\",project_replica=\"test\",prometheus=\"monitoring/k8s\",prometheus_replica=\"prometheus-k8s-1\"}/01DG6Q0F4K5BVCR334NJB92CMX /var/thanos/store/monitoring-thanos/compact/0@{cluster_replica=\"shenzhen_test\",dc_replica=\"lab\",project_replica=\"test\",prometheus=\"monitoring/k8s\",prometheus_replica=\"prometheus-k8s-1\"}/01DG90J9GADDXQVMNJ9EX2VHJA /var/thanos/store/monitoring-thanos/compact/0@{cluster_replica=\"shenzhen_test\",dc_replica=\"lab\",project_replica=\"test\",prometheus=\"monitoring/k8s\",prometheus_replica=\"prometheus-k8s-1\"}/01DG9E988WREQM2XKTK3SVGYFQ]: write compaction: chunk 63 not found: invalid encoding \"none\""

Anything else we need to know
Environment:
Openshift Version: v3.11.43
Kubernetes Version: v1.11.0+d4cacc0
Object Storage: S3

Seems like a case of https://github.com/thanos-io/thanos/issues/1335. Were there any partial uploads? Could you elaborate more on how to reproduce this or what happened?

We encounter the same problem as described above.
Oct 02 08:26:03 thanoscompact01 thanos[105774]: level=warn ts=2019-10-02T08:26:03.028686797Z caller=prober.go:154 msg="changing probe status" status=unhealthy reason="error executing compaction: compaction failed: compaction failed for group 0@{environment=\"production\",monitor=\"infrastructure\",replica=\"prom01\"}: compact blocks [/var/cache/thanos/compact/0@{environment=\"production\",monitor=\"infrastructure\",replica=\"prom01\"}/01DNHEV4WJ042J94AS3HJHGJXH /var/cache/thanos/compact/0@{environment=\"production\",monitor=\"infrastructure\",replica=\"prom01\"}/01DNHNPW4JPNWSP76WH18G6DMR /var/cache/thanos/compact/0@{environment=\"production\",monitor=\"infrastructure\",replica=\"prom01\"}/01DNHWJKCK1Y0DG1N3MGBD5W4R]: write compaction: chunk 266 not found: invalid encoding \"none\""
Oct 02 08:26:03 thanoscompact01 thanos[105774]: level=error ts=2019-10-02T08:26:03.028868443Z caller=main.go:213 msg="running command failed" err="error executing compaction: compaction failed: compaction failed for group 0@{environment=\"production\",monitor=\"infrastructure\",replica=\"prom01\"}: compact blocks [/var/cache/thanos/compact/0@{environment=\"production\",monitor=\"infrastructure\",replica=\"prom01\"}/01DNHEV4WJ042J94AS3HJHGJXH /var/cache/thanos/compact/0@{environment=\"production\",monitor=\"infrastructure\",replica=\"prom01\"}/01DNHNPW4JPNWSP76WH18G6DMR /var/cache/thanos/compact/0@{environment=\"production\",monitor=\"infrastructure\",replica=\"prom01\"}/01DNHWJKCK1Y0DG1N3MGBD5W4R]: write compaction: chunk 266 not found: invalid encoding \"none\""

However, if we run bucket verify, no error is found. Local cache has been deleted.
thanos is version 0.7.0.

looks like some block corruption(unclean shutdown, crash, using remote storage etc.). Could you send me that block privately(thanos-dev slack or prometheus-dev irc) and I will be able to confirm.
Otherwise to unblock can just delete this block.

I was able to replace the broken block, because we keep the original blocks on prometheus itself for quite a while. But thanos could handle this gracefully instead of just stopping. Sorry, I deleted the corrupt block - would have been interesting why verify did not find any problems ...

Verify does not look if none of the chunks are malformed, especially in such way. Can't think of any operation that would cause this. Even partial upload... It looks really odd.
I am really interested in what's the root cause of this. I guess you deleted it from object storage? What's object storage you guys use?
Wonder If the replaced block will not cause the same problem because it might look like the original block was wrongly produced by Prometheus.

But thanos could handle this gracefully instead of just stopping.

Some colleges that use Prometheus at large scale mention that hard fail is better than just log an error an continue. I think the main reason for that is that it would lead to false alerts and misleading data when querying.

@bwplotka we're using ceph with radosgw as object storage, and the blocks have been processed now. I think this could be related to some problems within our storage infrastructure we had around that time, but i did not find any errors in the sidecar logs regarding failed uploads ...

Ah that makes sense.
Anyway, what would be action items for Thanos in this case? Maybe

Increment metric for malformed block?
continue compacting whatever we can avoiding this "group"
Add check against malformed blocks in verify? This might be quite difficult.

Incrementing metric for malformed blocks and continue compacting for healthy groups seems like a good idea - shouldn't thanos_compact_group_compactions_failures_total be the metric already? But that was always at 0 - because the service just died and restarted.

Hello, we are facing the same issue.
We have this block:
| 01DX0WGA3DBRS0E964JGYHYG63 | 12-12-2019 01:00:00 | 26-12-2019 01:00:00 | 336h0m0s       | -296h0m0s     | 7,389,671  | 268,583,707,631 | 2,237,389,868 | 4          | false       | <MY_LABELS_KEY_VALUE> | 0s         | compactor |

And the compactor is in crashloop with:
level=info ts=2020-01-07T11:17:09.156025559Z caller=compact.go:290 msg="start first pass of downsampling"
level=info ts=2020-01-07T12:22:00.795975326Z caller=downsample.go:269 msg="downloaded block" id=01DX0WGA3DBRS0E964JGYHYG63 duration=1h4m32.513475834s
level=info ts=2020-01-07T14:51:13.634701294Z caller=streamed_block_writer.go:219 msg="finalized downsampled block" mint=1576108800000 maxt=1577318400000 ulid=01DXZZDE3MW3QC78VJYSFMYQ38 resolution=300000
level=warn ts=2020-01-07T14:51:35.919396231Z caller=prober.go:117 msg="changing probe status" status=not-ready reason="error executing compaction: first pass of downsampling failed: downsampling to 5 min: downsample block 01DX0WGA3DBRS0E964JGYHYG63 to window 300000: get chunk<PHONE_NUMBER>246, series 447305847: invalid encoding \"none\""
level=info ts=2020-01-07T14:51:35.920068131Z caller=http.go:78 service=http/server component=compact msg="internal server shutdown" err="error executing compaction: first pass of downsampling failed: downsampling to 5 min: downsample block 01DX0WGA3DBRS0E964JGYHYG63 to window 300000: get chunk<PHONE_NUMBER>246, series 447305847: invalid encoding \"none\""
level=info ts=2020-01-07T14:51:35.920141985Z caller=prober.go:137 msg="changing probe status" status=not-healthy reason="error executing compaction: first pass of downsampling failed: downsampling to 5 min: downsample block 01DX0WGA3DBRS0E964JGYHYG63 to window 300000: get chunk<PHONE_NUMBER>246, series 447305847: invalid encoding \"none\""
level=error ts=2020-01-07T14:51:35.920212385Z caller=main.go:194 msg="running command failed" err="error executing compaction: first pass of downsampling failed: downsampling to 5 min: downsample block 01DX0WGA3DBRS0E964JGYHYG63 to window 300000: get chunk<PHONE_NUMBER>246, series 447305847: invalid encoding \"none\""

What "surprises" me is the caller=streamed_block_writer.go:219 msg="finalized downsampled block" log line before the invalid encoding "none" error.
Even if the block is "corrupted", the finalized downsample log would be called?
Is there any script I could run to validate the block is corrupted and that there is not so many things to do anymore...? This is quite a big range of time ^^'
More info: Thanos 0.9.0, Objectstore: S3 Scality on-premise


Getting Module UserLocation Not Found Please help

After Runnung the project telling module UserLocation Not Found

I don't recognize this error at all. Can you post more details? Maybe include a stack trace from the logs?


Add transfer status (failed / successful) - token_transfers on solana.

It’s impossible to analyze the data effectively because successful and failed transfer transactions are mixed together.
solana/token_transfers

@0xweb3dev can you suggest the queries for filtering and we'll split @jeffling ideas?


Add PDF + e-mail deck export

This will propably include backend, but maybe there is some tool with online API

Or just prepare HTML page ready-to-print.

http://www.harrypottertcg.com/wp-content/uploads/2016/05/HPTCG-Decklist.pdf


Difficulty running an aspnet-prerendering script through webpack

I would like to bundle up all of my javascript using webpack. However, when passed through webpack, even your example script
var prerendering = require('aspnet-prerendering');

module.exports = prerendering.createServerRenderer(function(params) {
    return new Promise(function (resolve, reject) {
        var result = '
Hello world!
'
            + '
Current time in Node is: ' + new Date() + '
'
            + '
Request path is: ' + params.location.path + '
'
            + '
Absolute URL is: ' + params.absoluteUrl + '
';

        resolve({ html: result });
    });
});

will not work. That's with a very basic webpack config, such as:
...
        // bundling code for server-side rendering using Microsoft SPA Services
    {
        entry: {
            'aspnet-prerendering-bundle': './server/aspnet-prerendering-bundle.js'
        },
        output: {
            // [name] corresponds to the keys in the above entry object
            filename: '[name].js',
            path: path.resolve(__dirname, '../../../server-js/'),
            library: 'ServerRender',
            libraryTarget: 'umd'
        },
        module: {
        },
        plugins: [],
        bail: true
    }
...

The issue appears to be here: https://github.com/aspnet/JavaScriptServices/blob/dev/src/Microsoft.AspNetCore.SpaServices/npm/domain-task/src/main.ts#L43. When I simply import the module without using webpack, I get past this line just fine. However, in my webpacked bundle, this line hangs.
That binding context seems to hang indefinitely, I think because the webpacked version doesn't know what domain.active is.
The webpacked script gives the following error in the SpaServices C# code:
The Node invocation timed out after 60000ms.
You can change the timeout duration by setting the InvocationTimeoutMilliseconds property on NodeServicesOptions.

The first debugging step is to ensure that your Node.js function always invokes the supplied callback (or throws an exception synchronously), even if it encounters an error. Otherwise, the .NET code has no way to know that it is finished or has failed.
Error,   at Microsoft.AspNetCore.NodeServices.HostingModels.OutOfProcessNodeInstance.<InvokeExportAsync>d__13`1.MoveNext()

webpack: 2.3.3
asp-net-prerendering: 2.0.5

I believe I found a solution.
Registering those node dependencies such as path and domain as externals in webpack appears to have fixed the issue.

Confirmed that that solved the problem.
For future reference, added these to webpack:
        externals: {
            domain: 'domain',
            path: 'path'
        }

Then was able to use the following entrypoint to webpack:
/* Render react components server side */
import 'babel-polyfill';
import React from 'react';
import { renderToString } from 'react-dom/server';
import { createServerRenderer } from 'aspnet-prerendering';
import Component from '../components/component';

export default createServerRenderer(function(params) {
    return new Promise(function (resolve, reject) {
        var result = renderToString(<Component />);

        resolve({ html: result });
    });
});

And pointing spa services to the bundle works perfectly.

Thank you man


feat: updates workflow to use gh app token

updates the secureliCI workflow to use the new GH App installation token instead of Sascha's PAT.
adds space between workflow jobs/steps to make the file easier to read

Seems to be stuck on the homebrew test

Seems to be stuck on the homebrew test

It finished up. It's showing an error at the end of that step, but the job still passed. Does that look right?


Repl time information

more time than you ever wanted. show the time breakdown when you evaluate a query. probably too much details but I'd rather that than not enough.
! @jystic @tranma
λ :set +time
λ feature ints500 ~> latest 2 ~> fields

icicle: Compile Query
icicle: Compile Source
icicle: Compile Source (took 0.01s)
icicle: Compile Core
icicle: Compile Core (took 0.01s)
icicle: Compile Avalanche
icicle: Compile Avalanche (took 0.00s)
icicle: Compile Flattened Avalanche
icicle: Compile Flattened Avalanche (took 2.98s)
icicle: Compile Flattened Avalanche Check
icicle: Compile Flattened Avalanche Check (took 0.03s)
icicle: Compile Query (took 3.03s)
icicle: Compile Sea
icicle: Compile Sea Avalanche->Sea
icicle: Compile Sea Avalanche->Sea (took 0.00s)
icicle: Compile Sea Sea->object
icicle: Compile Sea Sea->object (took 39.45s)
icicle: Compile Sea (took 39.45s)
icicle: Evaluate Zebra
icicle: Evaluate Zebra Open
icicle: Evaluate Zebra Open (took 0.00s)
icicle: Evaluate Zebra Schema
icicle: Evaluate Zebra Schema (took 0.00s)
icicle: Evaluate Zebra Transmute
icicle: Evaluate Zebra Transmute (took 0.00s)
icicle: Evaluate Zebra Decode
icicle: Evaluate Zebra Decode (took 0.00s)
icicle: Evaluate Zebra Snapshot
icicle: Evaluate Zebra Snapshot (took 0.01s)
icicle: Evaluate Zebra Output
icicle: Evaluate Zebra Output (took 0.00s)
C evaluation
------------

icicle: Evaluate Zebra (took 0.01s)

what's "flattened avalanche" vs "flattened avalanche check" again? optimisation vs check?

👍

just typechecking the simplified/transformed flat.


Update chain-of-custody.svg

Replaced the on-prem Active directory icon
Before selecting the "Create pull request" button:

Enter a meaningful title above^, using a prefix if necessary and keywords "New" or "Update" indicating the nature of changes.

Describe the summary, scope, and intent of this PR:
the SVG contain the update icon of on-premises active directory

AFTER YOUR PR HAS BEEN CREATED, expand this section for tips and additional instructions.
These are common guidelines for contributions across the repos managed by the Cloud Architecture Content Team (CACT). Some repositories may have additional specific requirements that are not listed here.
Guidance for all contributors

Topic
Guidance

Draft PR
If your PR will be a work-in-progress for more than a day or two, select the Convert to draft link in the upper right of the page (under Reviewers) to change it to a draft. For future reference, you can also do this using the Create pull request button drop-down during PR creation.

ms.date metadata
Don't update an article's "ms.date" metadata property unless you've done a full freshness review of the content. A full freshness review includes changes required to correct or improve the full technical accuracy of the article.Don't update "ms.date" if you're doing targeted changes to improve non-technical aspects of the article, such as editorial quality, art improvements, article template alignment, etc.If you've changed any "ms.date" properties for work that wasn't part of full review for freshness, please reset them to their previous value.

Placement and linking
If you're creating a new article or articles, include updates to the related TOC.yml file to propose where the article(s) should be placed. Also consider other places within the document set where it would be beneficial to cross-reference and link to your new article(s).

PR build
After you open your PR, and for each successive commit that you push to your branch, the publishing platform will run validation on the files in your pull request. A summary of the build results for each file will be inserted inline into your pull request, which includes any build suggestions/warnings/errors. PRs cannot be merged until all build errors and most warnings are resolved.

Publishing
Following a successful merge, most repos publish to the live site at least once per (business) day, usually around 10am Pacific.

Additional resources
Learn.Microsoft.Com contributor guide

Additional guidance for private repos and internal contributors

Topic
Guidance

PR size
If your PR is more than ~5 lines of changes, or you'd like for the changes to go through editorial or larger review, open a contribution request at https://aka.ms/Contribution and include a link to the PR in response #8. Once it's processed, you'll be notified of the next steps.

PR title prefix
Select the Edit button to the right of the PR title if you need to revise it. The following prefixes are reserved for specific contribution types:[Quality Check] - maintenance work related to content quality (edit passes, art improvements, template alignment)[LinkFix] - recurring/adhoc PRs to correct link URLs[Pipeline] - new/updated contributor success pipeline content[WIP] - a work-in-progress draft requiring several days/weeks

PR preview
Following successful build of your PR, publishable files will also include Preview URL links to staged previews of your new/updated articles. Be sure to review these for verification of your intended contributions, or to send to other internal contributors for review.

PR sign-off (public repo)
If an article you own is updated in a public repo PR, you are responsible for sign-off. You will be automatically notified via email. The PR will not be merged until you've had a chance to review and sign-off.

PR sign-off (private repo)
After you've completed your proposed changes, addressed build warnings, and completed all review work, you can begin the sign-off process for review and merge:If your PR is in draft mode, remove "[WIP]" from the title and select Ready for review button at the bottom of the PR.Enter "#sign-off" in a new comment. This comment indicates that you're confident the work meets or exceeds Microsoft's standards for publication, and will trigger the review process.Your PR may be selected for initial review by the CACT. Following CACT review, you may receive questions or requests for additional changes. You should have initial feedback from CACT review within a few business days. If you have an urgent request or need to contact the team, please mention @MicrosoftDocs/cloud-architecture-content-team-pr-reviewers in your PR and someone will get back to you. After CACT review is complete, a CACT #sign-off will be added.Final review/merge is done by the PR review team. The PR team may also respond with feedback, categorized as "Blocking" (requires action from you), or "Non-blocking" (to be addressed in a future PR).

Additional resources
Learn.Microsoft.Com internal contributor guideAuthoring templates: architecture-center-pr, well-architected-prTo contact the CACT use e-mail, or @mention our GitHub team in your PR comments using: @MicrosoftDocs/cloud-architecture-content-team-pr-reviewers

@simonesavi
Can you review the proposed changes? Note that the new image needs to have an opaque background (so that the dark text is visible in dark view) before we can merge.
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@Court72 for me the schema is ok, also because it has been provided by @fabmas that his the co-author of article.
But before go ahead, could you please read the last comment of @ckittel ad let us know if there are further actions on our side?

Hi @simonesavi,
Yes, the new image should be replaced with one that has an opaque background (so that the dark text is visible in dark view).

Hi @Court72,
what it's not clear to me it's: the picture uploaded with this merge request has the "opaque background" or we must provide a new picture?

The picture in this PR needs to have a solid, white background.  That's currently not the case in this PR.  Please replace docs/example-scenario/forensics/media/chain-of-custody.svg in this PR with one that has a solid background, like it was prior to you making the update.

Hi @Court72, what it's not clear to me it's: the picture uploaded with this merge request has the "opaque background" or we must provide a new picture?

Oh I see - you can tell the background for the new image is transparent because of the checkered background:

It should be solid, like in the diagram it replaced.

Hi @ckittel
in attache the pictures with no transparent background. Can you please check and upload them?

This is your PR, can you please push these to your fork's branch?  This PR will then update automatically.

@ckittel @fabmas I believe we should also replace this file: https://archcenter.blob.core.windows.net/cdn/chain-of-custody.vsdx with the new Visio.
@ckittel This seems to be a copy in CDN, in which way we can replace it?

For our convenience, I copied the new VSDX file here:
https://github.com/fabmas/coc/blob/main/.diagrams/chain-of-custody.vsdx
...but I don't know where it should be replaced.

CACT #sign-off
#remove-label:"cact-review-in-progress"
#remove-label:"do-not-merge"
#label:"ready-to-merge"


App broken after update today

Versions

vee-validate:"^2.1.3"
vue: "2.5.17"

Describe the bug
After running npm install today, app is broken and I got this error:
Failed to compile.

./node_modules/vee-validate/dist/vee-validate.esm.js
Module build failed: Error: ENOENT: no such file or directory, open 'E:\*\*\*\node_modules\vee-validate\dist\vee-validate.esm.js'

How can I resolve this?
Any help would be appreciated!
To reproduce
Just install the versions of vue and vee-validate mentioned above
Expected behavior
Mentioned above

seems to work now.


IDialogService: Forward event DialogInstanceAddedAsync to OnDialogInstanceAdded for compatibility

Description
Fixes: https://github.com/fgilde/MudBlazor.Extensions/issues/98
It's ugly, but there is nothing much I can do, and it's event, if it was a delegate or just typical Func it would be better.
How Has This Been Tested?
Manually by making sure the DialogInstanceAddedAsync is not mandatory to be implemented when implementing the IDialogService.
Type of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (fix or improvement to the website or code docs)

Checklist

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[ ] I've added relevant tests.

I understand forwarding the new event to the old one. But doesn't removing public from the old events break existing code where people rely on the interface to subscribe these two events when using IDialogService as the type of the injected service?
[Inject]
IDialogService DialogService { get; set; }

I understand forwarding the new event to the old one. But doesn't removing public from the old events break existing code where people rely on the interface to subscribe these two events when using IDialogService as the type of the injected service?

Interface members in C# are public by default. it's a contract.
Explicit visibility should only be defined when you're making a default implementation or logic that shouldn't be visible (again, this is used for the default implementation).
Before C# 8.0, it wasn't possible to set the public keyword, and this is the exception you'd get:

Adding the public keyword to members without default implementation doesn't do any harm, I just find it incorrect language usage since it was intended solely for dealing with default implementations, and here we have only one such member.

Understood. Thanks for explaining ;)


Into for references

I think that having the below implementation would be helpful in the standard library:
impl<'a, T: Clone> From<&'a T> for T {
    fn from(val: &'a T) -> T {
        val.clone()
    }
}

Use case: functions which take Into<T> arguments.
For example, let's say that I have the below some
struct MyStruct(String);
impl MyStruct {
    fn new<S: Into<String>>(s: S) -> MyStruct {
        MyStruct(s.into())
    }
}

A user might mistakenly do the below, when they actually meant to clone inner:
let inner = String::from("hello world");
let s = MyStruct::new(inner);

And because &'a str implements Into<String>, one might be tempted to do:
let inner = String::from("hello world");
let s = MyStruct::new(&inner);

But this doesn't work, because &String doesn't implement Into<String>. You could do &*inner, but this seems needless and the idea of making references implement Into makes sense.
Would be willing to write up an RFC for this, but I figured I'd make an issue here first to discuss. I feel like accidental cloning wouldn't be too much of an issue because there are easy ways to lint that out, and clippy already does this in many other cases where there may be accidental cloning. So, having this for the sake of clarity might make sense.
Also, this could potentially be a breaking change, but we could probably get around that with default specialization and/or a crater run.

Sounds good, but is an rfc necessary? What about just open a PR and request a crater run?

You might want this for Copy but I kinda thought Clone was meant to be "explicit" since it could get expensive.  An Arc is fast to Clone but cannot be Copy.  Implicit Clones of Arcs might cost debugging time, right?  Am I missing something?
Are there garbage collected pointer types G<T> for which Clone means one thing but impl From<&'a G<T>> for G<T> should do something else?

From the documentation of Into:

A conversion that consumes self, which may or may not be expensive.

There are already, for example From<&str> implementations for String.

@leodasvacas I wasn't sure what the requirements were for changes that require Crater runs; I assumed that all of them needed to go through some sort of RFC process. I could make a pull request for it though!

@clarcharr An RFC would need a PR to crater against anyhow. Formally this change would require an RFC, but now that we have FCPs for PRs and with the precedent of From<&str>, I would hope this change can get by with just a PR if crater is favorable.

Closing for now because a pull request was made. If a formal RFC is needed I'll write one up in a PR.


Cortana doesn't work if proxy-login-automator is the default proxy

If you setup proxy-login-automator to listen to localhost:8080 and configure the system to use this address as default proxy for all Windows 10, Cortana stops working, together with Windows Apps (like Weather). Everything else works fine, except Windows Apps and Cortana.

thanks for your feedback.
It seems the term "Windows Apps, like Weather" are Universal Windows Apps(UWP).
I'v heard about UWP that

the system proxy gets used by default and apps have the option of not going through the proxy."

(https://wpdev.uservoice.com/forums/110705-universal-windows-platform/suggestions/11323269--uwp-need-http-proxy-function)
So you can try to set UWP not use proxy.
Or, Maybe UWP are using other settings? You can try following ways to set proxy:
http://www.digitalcitizen.life/how-set-proxy-server-windows-10
Good luck

Thank you for the answer, but this doesn't work. The Proxy configuration through Windows Apps is the same as the proxy configuration through Internet Connection windowed application. If you change one, it change the another one. The netsh doesn't help too. It is quite a same I can't use proxy-login-automator for everything because of this issue. It is running in background in my system and used just for some applications.
Thank you for your time.

Everyone works with Windows Store Apps should look at Isolating Windows Store Apps on Your Network.
It seems Windows Store Apps have its own network policy such as how to connect to internet.
There are group policy "Computer Configuration\Policies\Administrative Templates\Network\Network Isolation\Internet proxy servers for apps" to serve these apps.

Exactly, someone have already resolved this problem by above group policy settings.
See the last method of http://www.thewindowsclub.com/setup-proxy-metro-application-windows-8


Imu steering angle

Psched fixed, can output gyro and accel readings.

Using a new IMU sensor compared to last year, closing this PR to implement elsewhere


Sonic the Masked Werehog: Watch (?-i:^90 98$)

Sonic the Masked Werehog requests the watch of the watch_keyword (?-i:^90 98$). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

(?-i:^90 98$) has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Same person as the one in #4062. They've used this name in the past, and are using it now again: https://meta.stackexchange.com/questions/349882

Approved by Makyen in Charcoal HQ


Link to Admin page needs update

Issue in ./help/admin/admin/company/single-signon-admin.md
New page location is https://helpx.adobe.com/enterprise/using/set-up-identity.html or you can use the following GoURL: https://www.adobe.com/go/analytics_sso_en

Thanks Antonio! Fixed.


Consul UI unavailable

https://:8500 returns an error

Use :8700 for the /ui/. The UI is blocked on 8500.

Thanks Chris. I see.
Could we change consul-ui to listen on some other port so that Nginx can proxy on the standard 8500?

Hmmm 8700 is not working for me either with Vagrant or OpenStack.

Are you using https?
I'm testing a fix that will put /ui/ on 8500.

Yea I was testing with https.

Fixed.


Added new modules for mssql- namely enum_impersonate, enum_logins,, enum_links, exec_on_link, link_enable_xp, link_xpcmd

Thanks for the PR! Finally the MSSQL protocol gets some love :)

@deathflamingo do you got a twitter handle i could mention in a post?

Hey, sorry no I don't have a Twitter handle


Test Changelog Entries

No need to review, this is just for testing but I opened it for viewability


Directed graph plugin update

Undetected graph should only have preorder
directed graph should have traversal, edge classification, preorder, inorder, postorder, and reversePostorder.

The plugin should have graph methods with groovydoc. Methods added to the graph through metaClass should call the plugin methods.

Intwllij does not recognize dynamic methods added at runtime. This looks like a good fix for that problem.

https://confluence.jetbrains.com/plugins/servlet/mobile#content/view/15796912

Not going to make these changes


The project counts number of people entering the train station

The repo contains
Proper code with formatting
index.html
index.css
index.js

Pl take a look @gantavyamalviya


Package request: protonhax

Project description
Protonhax is a utility designed to manage Proton configurations for Steam games on Linux. It allows users to customize and control game-specific settings, helping to optimize the gaming experience when using Proton to run Windows games on Linux.
Metadata

homepage URL: https://github.com/jcnils/protonhax
source URL: https://github.com/jcnils/protonhax.git
AUR package URL: https://aur.archlinux.org/packages/protonhax
license: BSD 3-Clause
platforms: Linux

Duplicate of #334957


pallet-evm: log created address

cc @danforbes
Currently it's not trivial to fetch the created address after contract creation. This PR adds Event::Created(H160) so that the address is apparent.

Works like a charm...thanks so much!


Allow bond re-initialization from zero debt

This update to the bond depository allows policy to re-initialize a bond
that currently holds zero debt. This is useful in the case where the premium
has not allowed for profitable bonding in 5 days, allowing the depository to
start accepting bonds when profitable again.

looks good


Clarify which fields are optional in the Google Analytics extractor

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1273
Originally created by @dmor on 2019-11-06 17:33:52

According to the Google Analytics extractor docs, the fields for End Date and Reports are optional. However, this does not match the UI.
Recommend noting which fields are optional in a standardized way so that users don't get bogged down by filling in unnecessary fields.

View 10 previous comments from the original issue on GitLab


ADDED FILES FOR A NEW INSTITUTE

The file “eep-igruop.txt” has been added for the eep-igruop institute, corresponding to the School of Professional Studies.
Sample email to be accepted<EMAIL_ADDRESS>Thank you very much.

This is the link: https://eep-igroup.es/

@Jesusantiago Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course.

Thank you!

Hello @philipto, of course:

The school official website URL:
https://eep-igroup.es/

The school street address, including city and country:
C. Torrelaguna, 123, Cdad. Lineal, 28043 Madrid. - https://g.co/kgs/rf4546z (Link)

This is the institute where I study, but they have several branches.

An URL of a page on the official website where the school offers an IT-related long-term (>1 year) course:
https://www.eep-igroup.es/tecnico-superior-doble-titulacion-dam-daw.php

The professional studies that I am doing are those sent, I have seen that they do not have the duration expressed in years, but in hours. Every 1000 hours corresponds to 1 year.
Thank you for your time.

@Jesusantiago  Thank you for the clarifications. Pull request merged. Please start requesting the licenses in about an hour, to let the changes to propagate through our system.

Thank you very much for your time, have a nice day 😃.


When downloading a file using presigned URLs, set the actual filename

Hi there.
Currently I'm calling stat on a file:
const { data } = await client.get(
    `/repositories/${repositoryName}/refs/${ref}/objects/stat`,
    {
      params: {
        presign: true,
        path
      }
    }
  )

and downloading the file from S3 using the physical_address returned field. The filename, however, seems to be an internal LakeFS identifier. Is there a way to set the downloaded file name to the "real" name?
From my understanding if the S3 object had this header:
Content-Disposition: attachment; filename="foo.bar"

it would work. Thanks!

This seems to be possible using a response-content-disposition query parameter that could be added when constructing the pre-signed URL returned to the client