What is the correct way to update InheritedWidget from a widget that depends on it

What is the correct way to update that state of an inherited widget from a widget that is dependent on it?
use case:
The app is opened.
A widget checks local storage for session tokens.
If session tokens are valid it calls a method on the inherited widget to update its state.
Since the inherited widget has changed, the widget that called the update gets replaced
Since setState is async the widget that gets replaced calls the same method again on the inherited widget to update it.
This creates a recursive loop.
Inherited Widgets also cannot be used in the initState() life cycle which is the most logical place to call this method to load auth from local storage.
flutter: ══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
flutter: The following assertion was thrown building _InheritedAuthContainer:
flutter: inheritFromWidgetOfExactType(_InheritedAuthContainer) or inheritFromElement() was called before
flutter: RootState.initState() completed.
flutter: When an inherited widget changes, for example if the value of Theme.of() changes, its dependent
flutter: widgets are rebuilt. If the dependent widget's reference to the inherited widget is in a constructor
flutter: or an initState() method, then the rebuilt dependent widget will not reflect the changes in the
flutter: inherited widget.
flutter: Typically references to inherited widgets should occur in widget build() methods. Alternatively,
flutter: initialization based on inherited widgets can be placed in the didChangeDependencies method, which
flutter: is called after initState and whenever the dependencies change thereafter.
Widget that determines if user is authenticated
class Root extends StatefulWidget {
  @override
  State createState() => RootState();
}

class RootState extends State<Root> {
  static Map<String, Widget> routeTable = {Constants.HOME: Home()};
  bool loaded = false;
  bool authenticated = false;

  @override
  void initState() {
    super.initState();
    if (!loaded) {
      AuthContainerState data = AuthContainer.of(context);
      data.isAuthenticated().then((authenticated) {
        setState(() {
          authenticated = authenticated;
          loaded = true;
        });
      });
    }
  }

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
        initialRoute: '/',
        onGenerateRoute: (routeSettings) {
          WidgetBuilder screen;
          if (loaded) {
            if (authenticated) {
              screen = (context) => SafeArea(
                  child: Material(
                      type: MaterialType.transparency,
                      child: routeTable[routeSettings.name]));
            } else {
              screen = (conext) => SafeArea(
                  child: Material(
                      type: MaterialType.transparency, child: Register()));
            }
          } else {
            screen = (context) => new Container();
          }
          return new MaterialPageRoute(
            builder: screen,
            settings: routeSettings,
          );
        });
  }
}

Method on the inherited widget that updates the data in the inherited widget
Future<bool> isAuthenticated() async {
    if (user == null) {
      final storage = new FlutterSecureStorage();
      List results = await Future.wait([
        storage.read(key: 'idToken'),
        storage.read(key: 'accessToken'),
        storage.read(key: 'refreshToken'),
        storage.read(key: 'firstName'),
        storage.read(key: 'lastName'),
        storage.read(key: 'email')
      ]);
      if (results != null && results[0] != null && results[1] != null && results[2] != null) {
        //triggers a set state on this widget
        updateUserInfo(
          identityToken: results[0],
          accessToken: results[1],
          refreshToken: results[2],
          firstName: results[3],
          lastName: results[4],
          email: results[5]
        );
      }
    }
    return user != null && (JWT.isActive(user.identityToken) || JWT.isActive(user.refreshToken));
  }

Main
void main() => runApp(
  EnvironmentContainer(
    baseUrl: DEV_API_BASE_URL,
    child: AuthContainer(
      child: Root()
    )
  )
);

AuthContainerState data = AuthContainer.of(context); don't use it in initState(). I use didChangeDependencies something like data ??= AuthContainer.of(context); because didChangeDependencies can be called multiple times.
I actually want to find some good solution to init something once, based on context. I think flutter doesnt have something like this. https://pub.dartlang.org/packages/after_layout Is a nice package, it is clear to me that something like this needs to exists - before&after the first time the layout of a widget has been performed.

I use this for an alternative thanks to the Dart 2.7 extension.
`
import 'package:flutter/material.dart';
extension StateExtension on State {
Stream waitForStateLoading() async* {
while (!mounted) {
yield false;
}
yield true;

}
Future postInit(BuildContext context, VoidCallback action) async {
await for (var isLoaded in waitForStateLoading()) {}
action();
}
}`
and usage:
@override   void initState() {     super.initState();     postInit(context, () {       SomeInheritedState.of(context).doSomething()     });   }

You can also just use
WidgetsBinding.instance.addPostFrameCallback((timestamp) {
  // do stuff after frame is build
});

use this package to handle this issue
https://pub.dev/packages/after_init

This seems abandoned:
https://github.com/martinrybak/after_init/issues/4


Authentication Error when logging into specific character

When logging in with my alt corp CEO, I get the following error: Error: {"error":"authentication failure"} (line 254). The error only occurs on that character, and all other characters can use the API perfectly fine
Function being used:
=corporations_corporation_wallets(ALT_CORP_CEO_CHARACTER_NAME, True)

What happens if you re-auth that char? I.e. go thru the auth process again for them.

Error still pops up even after re-authentication. I've also tried de-authing and re-authing and the error persists. It could be that it's an issue with this particular character name that causes the error, since it has a dash in the name

Might also be worth trying the Reset option if you don't have too many chars authed.
If the error is still happening after that, I'd need some additional info to go on I think. If you're on the GESI discord can PM me and can work thru it so you don't have to expose stuff here.

We looked into this more and ultimately is an instance of https://github.com/esi/esi-issues/issues/1089. Workaround was to limit the required scopes via https://github.com/Blacksmoke16/GESI?tab=readme-ov-file#how-can-i-limit-what-scopes-are-used.


Unhandled Exception

When clicking on a folder inside an OST, the following exception is shown. The Folder seems to contain 639 objects and the file is localized in german.
Thanks in advance for any help or input!
************** Ausnahmetext **************
System.IndexOutOfRangeException: Index was outside the bounds of the array.
at XstReader.LTP.MapType[T](List1 blocks, HID hid, Int32 offset) in C:\Dev\iluvadev\dijji\XstReader\src\XstReader.Api\LTP.cs:line 864    at XstReader.LTP.ReadTableInternal[T](BTree1 subNodeTree, UInt64 dataBid, Action2 idGetter, Action1 postProcessAction) in C:\Dev\iluvadev\dijji\XstReader\src\XstReader.Api\LTP.cs:line 498
at XstReader.LTP.ReadTable[T](NID nid, Action2 idGetter, Action1 postProcessAction) in C:\Dev\iluvadev\dijji\XstReader\src\XstReader.Api\LTP.cs:line 181
at XstReader.XstFolder.GetMessages() in C:\Dev\iluvadev\dijji\XstReader\src\XstReader.Api\XstFolder.cs:line 165
at XstReader.XstFolder.get_Messages() in C:\Dev\iluvadev\dijji\XstReader\src\XstReader.Api\XstFolder.cs:line 105
at XstReader.App.MainForm.SetCurrentXstElement(XstElement value) in C:\Dev\iluvadev\dijji\XstReader\src\XstReader\MainForm.cs:line 56
at XstReader.App.MainForm.set_CurrentXstElement(XstElement value) in C:\Dev\iluvadev\dijji\XstReader\src\XstReader\MainForm.cs:line 46
at XstReader.App.MainForm.b__31_1(Object s, XstElementEventArgs e) in C:\Dev\iluvadev\dijji\XstReader\src\XstReader\MainForm.cs:line 101
at XstReader.App.Controls.XstFolderTreeControl.RaiseSelectedItemChanged() in C:\Dev\iluvadev\dijji\XstReader\src\XstReader\Controls\XstFolderTreeControl.cs:line 43
at XstReader.App.Controls.XstFolderTreeControl.b__1_0(Object s, TreeViewEventArgs e) in C:\Dev\iluvadev\dijji\XstReader\src\XstReader\Controls\XstFolderTreeControl.cs:line 30
at Krypton.Toolkit.KryptonTreeView.OnAfterSelect(TreeViewEventArgs e)
at Krypton.Toolkit.KryptonTreeView.OnTreeViewAfterSelect(Object sender, TreeViewEventArgs e)
at System.Windows.Forms.TreeView.OnAfterSelect(TreeViewEventArgs e)
at System.Windows.Forms.TreeView.TvnSelected(NMTREEVIEW* nmtv)
at System.Windows.Forms.TreeView.WmNotify(Message& m)
at System.Windows.Forms.TreeView.WndProc(Message& m)
at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, WM msg, IntPtr wparam, IntPtr lparam)

Hi, thanks for the issue and all the stacktrace, it helps a lot.
It seems that some inner header in the ost file is corrupt or it does not contain expected data.
I will analyze in more detail the trace and in a near future (i don't have a lot time, sorry🙄) I will protect these functions in order to evict this error and maybe, show some of the content
Again, thanks for reprort this ugly bug

Thanks for your quick reply! I am not in a hurry to read that file and really appreciate the time and effort developers like you put into tools. I can’t share the file, but more than willing to test potential fixes or try a version with more debug output if that helps to find the problem. Just reach out if I can help in any way if you come around to take a deeper look into this. Thanks again and take care!

Hi, I published a new release: 2022-10-10
There are more protection against unexpected exceptions, and the application tries to show all information and ignores that information than produced the exception.
In your case, I think this will not a solution. The exception  is thrown when the XstReader tries to read the table with descriptions of the messages in a folder. Without this information the application can't load any content.
I don't know, try it and tell me something.

Hello, i think i have similar problem with some folders being empty.
It worked up until 1.15. With 2.1.0 and 2.20 i just get some empty folders. A few up to 47 mails worked but the count just seems to be a coincidence, since i have a few with 10 mails not showing up an quite many not working with a lot of mails.
I have a a german ost file as well so it sounds related. To big for sharing though.
Is there any way to debug this without resorting to the sourcecode?

Hi, I have a sample ost having this issue.
<EMAIL_ADDRESS>- u2.zip
IMO this is an issue of Outlook.
This will occurs with high probability when we shutdown Outlook, before Outlook completes receiving data from IMAP server.
We can prepare repro for this:

Use open source IMAP SMTP server hMailServer - Free open source email server for Microsoft Windows
Accumulate dummy message by sending mail with SmtpClient Class (System.Net.Mail) | Microsoft Learn
Use Outlook to receive them. And shutdown Outlook before all of them are received and processed.

I have tried some ways for recovery, but none of them helped.
(1) SCANPST.EXE. But it couldn't help to fix the logical broken data: Repair Outlook Data Files (.pst and .ost) - Microsoft Support
<EMAIL_ADDRESS>- u2.log
The log file indices some issues. But the detail of error meaning is unknown to us.
  **Attempting to walk all folders

        Failed to add row to the FLT, RowID = 3
        Failed to add row to the FLT, RowID = 2
        Failed to add row to the FLT, RowID = 1
        Failed to add row to the FLT, RowID = 0

(2) Clicking Compact Now button on Outlook Data File didn't help too.
And... the code level workaround will be:

Skip broken table row record.
Place fallback for listing up items.

According to [MS-PST]: Outlook Personal Folders (.pst) File Format | Microsoft Learn documentation, OST/PST will be able to process structurally.
Messaging layer will be able to apply XstReader.Api directly.

XstFolder.GetMessages method seems to access EnidType.CONTENTS_TABLE.
This means that Outlook stores references of child items (including mails, contacts and so on) to the TableContext with fewer properties embedded.
The our issue relates to TableContext. (TableContext is a kind of table structure. Imagine like DataTable)
TableContext's actual data is stored via Heap-on-node.
Thus there might be data mismatch among TableContext and Heap-on-node.

For detail inspection, I have extracted property list from that TableContext as a JSON text. It is about node id 8590 which has enumerating issue.
u2.txt
Search ERROR! from text file.
There are 593 matches (reading errors) in 266 total records from that TableContext.
I'm surprised that error is not few.
The property names written in JSON were taken throught XstReader.ElementProperties.KnownCanonicalNames.GetById of XstReader.Api.
And thanks for maintain great works like XstReader!

I'm sorry to say: there was no problem in neither XstReader or my posted ost file.
The strange behavior came from a .NET 6.0 breaking change.
Breaking change: Partial and zero-byte reads in DeflateStream, GZipStream, and CryptoStream - .NET | Microsoft Learn
Workaround sample:
diff --git a/src/XstReader.Api/NDB.cs b/src/XstReader.Api/NDB.cs
index a53abd3..5f01c92 100644
--- a/src/XstReader.Api/NDB.cs
+++ b/src/XstReader.Api/NDB.cs
@@ -537,7 +537,17 @@ namespace XstReader
                         if (buffer == null)
                             buffer = new byte[rb.InflatedLength];

-                        decompressionStream.Read(buffer, offset, rb.InflatedLength);
+                        int remaining = rb.InflatedLength;
+                        while (remaining != 0)
+                        {
+                            var numRead = decompressionStream.Read(buffer, offset, remaining);
+                            if (numRead == 0)
+                            {
+                                throw new EndOfStreamException();
+                            }
+                            remaining -= numRead;
+                            offset += numRead;
+                        }
                     }
                     read = rb.InflatedLength;
                 }

Or simply wrap with BinaryReader.ReadBytes.

Finally I could obtain a broken ost file.
<EMAIL_ADDRESS>- u5.zip
This one won't occur breaks on XstFolder.GetMessages.
Instead, this broken data affects decoder on click of folder items.

************** 例外テキスト **************
System.IndexOutOfRangeException: Index was outside the bounds of the array.
   at XstReader.LTP.HidSize(List`1 blocks, HID hid) in V:\XstReader\src\XstReader.Api\LTP.cs:line 857
   at XstReader.LTP.GetBytesForHNID(List`1 blocks, BTree`1 subNodeTree, HNID hnid) in V:\XstReader\src\XstReader.Api\LTP.cs:line 836
   at XstReader.LTP.ReadTableColumnValue(BTree`1 subNodeTree, List`1 blocks, RowDataBlock db, Int64 rowOffset, TCOLDESC col) in V:\XstReader\src\XstReader.Api\LTP.cs:line 615
   at XstReader.LTP.<>c__DisplayClass25_2`1.<ReadTableData>b__1() in V:\XstReader\src\XstReader.Api\LTP.cs:line 580
   at XstReader.XstProperty.get_Value() in V:\XstReader\src\XstReader.Api\XstProperty.cs:line 32
   at XstReader.App.XstPropertySetExtensions.ToPropertyGridSelectedObject(XstPropertySet propertySet) in V:\XstReader\src\XstReader\Extensions\XstPropertySetExtensions.cs:line 28
   at XstReader.App.Controls.XstPropertiesControl.LoadProperties() in V:\XstReader\src\XstReader\Controls\XstPropertiesControl.cs:line 50
   at XstReader.App.Controls.XstPropertiesControl.SetDataSource(XstElement dataSource) in V:\XstReader\src\XstReader\Controls\XstPropertiesControl.cs:line 42
   at XstReader.App.MainForm.SetCurrentXstElement(XstElement value) in V:\XstReader\src\XstReader\MainForm.cs:line 65
   at XstReader.App.MainForm.set_CurrentXstElement(XstElement value) in V:\XstReader\src\XstReader\MainForm.cs:line 46
   at XstReader.App.MainForm.<Initialize>b__30_3(Object s, XstElementEventArgs e) in V:\XstReader\src\XstReader\MainForm.cs:line 101
   at XstReader.App.Controls.XstMessageListControl.RaiseSelectedItemChanged() in V:\XstReader\src\XstReader\Controls\XstMessageListControl.cs:line 63
   at XstReader.App.Controls.XstMessageListControl.<Initialize>b__1_1(Object s, ListViewItemSelectionChangedEventArgs e) in V:\XstReader\src\XstReader\Controls\XstMessageListControl.cs:line 48
   at System.Windows.Forms.ListView.OnItemSelectionChanged(ListViewItemSelectionChangedEventArgs e)
   at System.Windows.Forms.ListView.WmReflectNotify(Message& m)
   at System.Windows.Forms.ListView.WndProc(Message& m)
   at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
   at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, WM msg, IntPtr wparam, IntPtr lparam)

The broken ost is obtained through this situation, with Outlook 2013:

Synchronize and continuously receive mail messages from IMAP server.
Quit Outlook, when the ost file is going to grow from 16,424 KB to 32,712 KB

@kenjiuno thanks a lot for the information and your investigation!
First of all, sorry for my delayed redponse: I am having complicated weeks and I can't spend much time on the project.
But, feel free to make the PR that you want.
I will try to dedicate more time to this issue.
Thanks!

In the latest release, XstReader shows information if there are errors reading Folders, Messages or Attachments


MAINT: Move logic into appropriate modules

This PR resolves the first two points of #76. Specifically, the changes introduced are:

Plotting logic moved from inside RestorationArea to the plotting module.

Logic was converted from using self to access dates and stats to using a RestorationArea instance, and accessing attributes.

Validation functions moved inside of RestorationArea.

Many validation methods have been simplified by breaking up the stages of validation. First, property setters are used to check for individual attribute requirements, then additional functions check for required relationships between attributes (e.g dates vs. timeseries data, dates vs. other dates, etc.)

Moving compute_metrics into the metrics module turned out to be a bit of a larger task, so a new feature branch was created for that refactoring project.

@KavlinClein I moved the metric logic changes into this PR. Will need you to take another look!


Global lb bug

Global lb xp for bso is showing the max as 24 skills.  E.g. ruben has 104% max xp.

I don't see where this is happening. Can you post a screenshot

I don't see where this is happening. Can you post a screenshot

issue is the static value in the query for globalLb(). Could probably make this a dynamic value based off the count of skills to prevent this from happening in the future, especially since we got sailing coming eventually too.

Fixed in next update


[pipeline/e2e] end-to-end-tests.ps1 should fail with failure test

Fix end-to-end-tests.ps1, which should fail and stop with failure test case.
Change

Pipeline

Import module Pester for e2e tests
Update to action/checkout@4, actions/setup-node@v4, <EMAIL_ADDRESS>
https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/

Update macos-12 to macos-14

https://github.com/actions/runner-images/issues/10721

end-to-end-tests.ps1

Invoke-Pester for *.Tests.ps1 in end-to-end-tests.ps1
Check test $LASTEXITCODE

end-to-end-tests-prelude.ps1

Reset $LASTEXITCODE for Throw-IfNotFailed

Cases

autocomplete.ps1

Rename to autocomplete-command.ps1

env-tools.ps1

Add Throw-IfNotFailed for non-windows

e2e/ci-verify-versions.ps1

Remove-Problem-Matchers for malformed port

Test

Example of un-expected pass before fixed

https://github.com/microsoft/vcpkg-tool/actions/runs/11435724127/job/31811675237?pr=1518#annotation:14:22

Failed as expected

https://github.com/microsoft/vcpkg-tool/actions/runs/11436226023/job/31813205981?pr=1518#annotation:14:45

Passed as expected

https://github.com/microsoft/vcpkg-tool/actions/runs/11436488593/job/31814031787?pr=1518

Fix end-to-end-tests.ps1, which should fail and stop with failure test case.

As far as I can tell it already does this. Do you have a repro for it not doing so?

It looks like the repro might only happen with something like this "Pester" module ( as in https://github.com/microsoft/vcpkg-tool/pull/1512/ ) because all other existing e2e tests fail by throwing an exception, thus stopping further test runs.

Fix end-to-end-tests.ps1, which should fail and stop with failure test case.

As far as I can tell it already does this. Do you have a repro for it not doing so?
(EDIT: I see you have an example in a comment here but it's not clear how you got that to happen)

Sorry I forgot to attach the test commit https://github.com/microsoft/vcpkg-tool/commit/77554801dcc44a7618832d2ee28ec5c4751cd33f
Run this script with $ErrorActionPreference = "Stop", will get status $? to False and $LASTEXITCODE to 2.
BeforeAll {
    Import-Module $PSScriptRoot/not-exist/posh-vcpkg
}
Describe 'Module posh-vcpkg tests' {
    It 'Expect fail' {}
}

This issue has been fixed by:

Check $lastTextExitCode = $LASTEXITCODE
Reset $LASTEXITCODE for Throw-IfNotFailed

I'm now trying to fix the error message Unexpected character; expected property name in GitHub action page, for example https://github.com/microsoft/vcpkg-tool/actions/runs/11450810448

Not changed
Unable change runner machine runs-on from macos-12 to macos-14, even there aren't any build macos-12 string in the repository.
It hangs with PR check:
builds / build (macos-12, macos-ci) Expected — Waiting for status to be reported

Note
vmImage: macOS-12 is deprecated in azure-pipelines/signing.yml:208
https://devblogs.microsoft.com/devops/upcoming-deprecation-of-macos-12-hosted-pipeline-image/

Test
Expected fail with failed test case https://github.com/microsoft/vcpkg-tool/actions/runs/11925979891

Thanks!


Add addin to website to allow for automatic documentation

i.e. get your addin listed here:
http://cakebuild.net/addins
Please let me know if you have any questions about how to do this.

Done 👍

@SharpeRAD that is great news! Thank you for getting that done.


Pretty print

Automatically format source code based on predefined settings.
Line up variables, etc.

Added in v2.1.0: https://ibm.github.io/zopeneditor-about/Docs/zcodeformatting.html

Hi,
Is there an option to enable/disable source code formatting regardless of the presence of a zcodeformat.(json|yaml) file?
I didn't find any. I think it would be desirable to provide for this, both at the User level and at the Workspace level.

@FALLAI-Denis we could file this as an enhancement. I am not sure about the value for disabling. If someone does not explicitly select format-on-save then nothing will be formatted. For governance I would rely more on code reviews such as GitHub Pull Requests.

Hi @phaumer
I wonder about the impact that reformatting the code could have on our coding assistance tools.
We have developed a VS Code extension that does the work described in API for interacting with source code opened in Z Open Editor
If the zcodeformat file does not exist...
If the user does not trigger a reformat (Shift-Alt-F)...
If automatic reformatting on entry or saving is not requested...
So no risk.
Otherwise, an option in the workspace's settings.json file would be the last chance to block a reformat; it is still necessary that this reformatting which is a standard function of VS Code is also under control of Z Open Editor.

@FALLAI-Denis I see that for other languages VS Code allows disabling as well. We could add it for consistency.
Btw. if the zcodeoformat file does not exist it will format using the defaults from the schema.


Added draft for this sprint

Needs:

[ ] Checkover by @whyrusleeping, @diasdavid, @dignifiedquire
[ ] Add contributors; @eminence, can you PR the final script you used to this repo?
[ ] Merging with Dec 7th sprint; maybe send this all out as a 'Best of December' monthly with the next sprint in January, so that we can start the year with it?

Looks good :) also adding npm modules to IPFS is reaaaallly fast now! :D

@diasdavid see https://github.com/ipfs/weekly/pull/6 :)


CONTRIBUTING.md Invalid Links

Description
On line 235 and line 236 of the file CONTRIBUTING.md, the links to the JavaScript style guide as well as CSS style guide are invalid.
Expected behavior: Links "JavaScript" and "CSS" should point to their respective style guides.
Actual behavior: Links "JavaScript" and "CSS" encounter a 404 not found page.

Noticed a while back that those styleguides where removed, don't know what's replacing them:

https://github.com/styleguide/css
https://github.com/styleguide/javascript

I would assume that these could be replaced with those mentioned on https://github.com/styleguide, which would be http://primercss.io/ for CSS and https://github.com/styleguide/ruby for Ruby.

Ruby. what am i talking about. :D (me needs more coffee). Ok, no JS styleguide found yet.

The AirBnB styleguides seem to be held in high regard, what about linking those instead?

https://github.com/airbnb/javascript
https://github.com/airbnb/css


分页支持第一页，最后一页；slot解析 fixed

Please makes sure these boxes are checked before submitting your PR, thank you!

[x] Make sure you follow Element's Contributing Guide.
[x] Make sure you are merging your commits to dev branch.
[x] Rebase before creating a PR to keep commit history clear.
[x] Add some descriptions and refer relative issues for you PR.

可以补充下测试么 谢谢 😅


DEoptim inside a function

Object not found
asked the developer
https://github.com/ArdiaD/DEoptim/issues/11
https://stackoverflow.com/questions/12023403/using-parlapply-and-clusterexport-inside-a-function
promise already under evaluation: recursive default argument reference or earlier problems
https://stackoverflow.com/questions/4357101/promise-already-under-evaluation-recursive-default-argument-reference-or-earlie

x error target opt.res_AshleyDene_SD1

end pipeline
Error : 8 nodes produced errors; first error: no such table: PredictedObserved
In addition: There were 28 warnings (use warnings() to see them)
Error: callr subprocess failed: 8 nodes produced errors; first error: no such table: PredictedObserved

check if observation path included in the input_list

8 nodes produced errors; first error: argument "obspara" is missing,

this has been solved by 031352f
the targets will pass the object without its name attributes.
reassign the names will save the objects again.


An Error From LJspeech Dataset

In my experiment, I found that the LJsppeech-1.1 cannot be recognized by the soundfile.
If there's anyone meet the same situaiton? Please let me know.
soundfile.LibsndfileError: <exception str() failed>

I think it’s more of an issue of soundfile than this repo. I’m not a maintainer of soundfile. I would recommend you ask it in soundfile repo or other relevant repos.

In my experiment, I found that the LJsppeech-1.1 cannot be recognized by the soundfile. If there's anyone meet the same situaiton? Please let me know.
soundfile.LibsndfileError: <exception str() failed>

Did you eventually figure this out? It took me days and I still can't resolve this issue


add a article on working fast in Linux

Hacktoberfest notes:

due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
please read our contributing guidelines
be sure to check the output of Travis-CI for linter errors
if this is your first open source contribution, make sure it's not your last!

What does this PR do?
Add Resource(s) | Remove Resource(s) | Add info | Improve Repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[ ] Not a duplicate
[ ] Included author(s) if appropriate
[ ] Lists are in alphabetical order
[ ] Needed indications added (PDF, access notes, under construction)

Could you comment on why this belongs in courses rather than interactive tutorials?

Closing since article/blog!


Unclear on how to launch

Hello,
I am unsure on how to install/launch steam-tui. The instructions just say to "launch the binary". I tried downloading the one from the releases page, setting it as executable and running it from the shell, but all I get is a variation of "./steam-tui: not found".
My system is Kubuntu 22.04.3 LTS x86_64, by the way.

Closing this for #58 - I need to make better binaries and will clean up the documentation


Force resized images to be regenerated when the source image changes

This should fix https://github.com/wintercms/winter/issues/177, but it will cause all existing installations to resize all of their already resized images.
Another alternative could have been to include the hash of the source image; but it would have been at the expense of more work every time this function is called, which could potentially include generating hashes of multiple extremely large (10+ MB) images on every page load if the page includes such images that need to be resized since the filter needs to identify the configuration in order to identify the path to the resized image to check if it needs to return a resizer/ URL or a /resized one.
Further improvements could potentially be made in the isResized() / getPathToResized() methods to cache the results to avoid doing all the work involved with the getConfig() method while still allowing people to run a command to trigger a refresh.

@bennothommo did you have any ideas for a better experience for this:

it will cause all existing installations to resize all of their already resized images.

@LukeTowers I personally don't see a problem with that. The resized images should be considered temporary and treated as a "cache" so to speak.

@bennothommo fair enough, but if we treat it that way then we need to provide a way to clear the cache. Will wait to merge this until we can add a util command to purge the resized images cache.

With that as well, should we provide a migration that clears that cache as a part of the update to avoid unnecessarily cluttering up people's storage directory with now-invalid images or is the added server load of regenerating all existing images not worth it (especially if they're caching the generated HTML which could include references to final files that would no longer exist).

I would envisage that the server load would be staggered, as the resizing only happens on an "as-needed" basis, when people load up a particular page with the resized image(s), so I don't think that's too much of a concern.
We definitely will need some form of command though to clear the cached resized images, if cache:clear doesn't already do that.
I'm not sure what we can do about people caching the generated HTML (and ostensibly, the resized image's direct URL). I would hope that doing a cache:clear as part of this proposed migration to clear the resized images would also reset the page cache as well. The same problem would've occurred with the previous resize functionality as well.

@bennothommo fair enough, but if we treat it that way then we need to provide a way to clear the cache. Will wait to merge this until we can add a util command to purge the resized images cache.

I think it would not be about clearing the actual Cache, it would also be about removing all the resized images. But perhaps that is what you meant anyway.


[Bug]: keyboard.press('Tab') does not set active element

Bug expectation
I expect pressing the Tab key to behave as it does in the browser: focus the next focusable element in the document.
Instead, it does not set an active element.
Bug behavior

[ ] Flaky
[ ] PDF

Minimal, reproducible example
import puppeteer from 'puppeteer';

const browser = await puppeteer.launch();

const page = await browser.newPage();

await page.setContent('<input id="login">');

await page.waitForSelector('#login');

await page.keyboard.press('Tab');

const id = await page.evaluate(() => document.activeElement?.id);

if (id !== 'login') {
    throw new Error('login is not active')
}

await browser.close();

Error string
login is not active
Puppeteer configuration

Puppeteer version
19.4.1
Node version
18.12.0
Package manager
npm
Package manager version
8.19.2
Operating system
macOS

The code is missing an await and the server part and I am not able to repro if I try to reproduce:
import puppeteer from 'puppeteer';

const browser = await puppeteer.launch();

const page = await browser.newPage();

await page.setContent('<input id="login">');

await page.waitForSelector('#login');

await page.keyboard.press('Tab');

const activeElement = await page.evaluateHandle(() => document.activeElement);

console.log({ activeElement }); // { activeElement: ElementHandle {} }

Sorry, I added the extra browser & page setup manually because your bot yelled at me. There is actually an await in my setup.
Hmmm. I copy-paste your code, run it, and now I get an error about id of undefined (citing the line of document.activeElement.id).

that's strange. Have you tried using the latest version of Puppeteer?

It does not looks like it reproduces on the bots as well. Locally, I also get no error.

I updated to 19.5.2, but no difference.
But. I think I found the source of this baffling behaviour:

If the function passed to the page.evaluate returns a non-Serializable value, then page.evaluate resolves to undefined.

I don't see this in the puppeteer docs directly¹, but it does appear in devdocs.io's copy. I don't know if theirs is outdated, but it's the only thing I've found to explain this behaviour.
page.evaluate does not like returning document.activeElement itself, but properties from it are okay (as is a plain object of properties):
ok:
const id = await page.evaluate(() => document.activeElement.id);

also ok:
const activeElement = await page.evaluate(() => {
  const { activeElement } = document;

  return {
    id: activeElement.id,
    type: activeElement.type,
  };
});

silently fails:
const activeElement = await page.evaluate(() => document.activeElement);

Possibly related/contributing to page.evaluate() silently failing: in my document, there are multiple focusable elements. The first several (in terms of DOM order) are input fields; there are also some link elements (which occur after the inputs in DOM order). When the page is opened by a human in a browser, (as the first interaction with the page) pressing tab correctly focuses the first input field. Puppeteer, however, is focusing a seemingly random one of the links (like the 7th link of 8). As an experiment, I tried setting tabindexs on the inputs, but puppeteer does not behave correctly / does not respect the tabindex: the links are all focused first and then the inputs.
I'm trying to simulate a user navigating the page via keyboard, so I'm using page.evaluate(() => document.activeElement) to get the element that became focused after page.keyboard.press('Tab').

¹ side-note, the puppeteer docs for page.evaluate have some copy-pasta:

@JakobJingleheimer have you think this guide that we have https://pptr.dev/guides/evaluate-javascript#return-types ?
You can get the reference to the active element with evaluateHandle, that still works.

@JakobJingleheimer how do you check it's undefined? it is defined for me, it's a JSHandle instance. Full code:
import puppeteer from 'puppeteer';

const browser = await puppeteer.launch();

const page = await browser.newPage();

await page.setContent('<input id="login">');

await page.waitForSelector('#login');

await page.keyboard.press('Tab');

const output = await page.evaluateHandle(() => {
    const { activeElement } = document;

    return {
        activeElement,
    };
});

console.log(output)

await browser.close();

Output:
❯ node focus.mjs
JSHandle {}

This console.log(output) // { activeElement: undefined } cannot be true because evaluateHandle always returns a handle so it does not know what properties exist on the object it references, i.e., you cannot use it like output. activeElement in the script context.

Ahh, sorry I was overly concise. I was actually checking
const { activeElement } = await page.…

console.log({ activeElement })

I've finally been able to get the data I need with a combination of page.evaluateHandle() and elementHandle.evaluate():
const activeElement = await page.evaluateHandle(() => document.activeElement);

const {
  name,
  tagName,
  type,
} = await activeElement.evaluate(({ name, tagName, type }) => ({ name, tagName, type }));

However, page.keyboard.press('Tab') still focuses the wrong element (it's picking a random link, instead of the first focusable element).

@JakobJingleheimer have you seen this guide that we have https://pptr.dev/guides/evaluate-javascript#return-types ?

Sorry, I missed this previously. I had not seen it. The guide's title is rather generic, so I never would have looked there for this (or really anything). Perhaps it would be good to link to it in the relevant API docs, as well as change the title to something more specific, like "Evaluating in the browser" (it's all javascript, so no need to mention that by name)?

@JakobJingleheimer could you update the ticket with the actual code that demonstrates that the active element is wrong?

I tried to create a minimal reproduction with the same fields and links, but it doesn't repro with just HTML.
The actual document is a react app, and the page in question is re-renderer as data loads (for instance, input's datalists and the links are populated based on fetch calls).
At first, I thought it was a cart-before-the-horse issue, where the links finish before the form, so I tried a more localised selector (for page.waitForSelector()), but that didn't make a difference.

The actual layout of the DOM is like this:
INPUT (9x)
BUTTON (2x)
A (6x)
… (various others, not together)

Because the browser handles this correctly but Puppeteer does not, that makes me think Puppeteer is not handling the updates/re-renders correctly.

@JakobJingleheimer hm it looks like it might due to vdom. React might update the same element with a new content but Puppeteer always holds a reference to an element so it will fetch the latest data on an element. Also, React has its own scheduler and you probably want to wait before React finished rendering before you try to grab the elements.
You could try to build out a minimal app in React to demonstrate it but I think there is no bug here and just a race condition between React and Puppeteer.

How does one get Puppeteer to wait for react to finish rendering? I looked for that specifically, but I haven't found anything.
I thought of using componentDidMount to emit an event for Puppeteer to listen for, but I'm not sure it can do that.

@JakobJingleheimer at first, perhaps you could add a timeout await new Promise(resolve => setTimeout(resolve, 3000)) before and after sending the Tab call. If you get consistent result, it would confirm it's a race with React.

Just tried that timeout, and no change to the sequence of focusable elements. I also tried setting the timeout after the page.waitForSelector checking the page has loaded (also no change).

@JakobJingleheimer sorry, cannot help more without a repro.

Mm, I understand.
I'll try to get something together.
Thanks for your help thus-far! I'm at least in a state where I can work around the disorganised focusable elements 🙂

I'm working on the repro, and I think I've got all the bits except for one final piece: When using Tab in puppeteer to focus on the date field (which looks like dd/mm/yyyy 📅), the yyyy gets selected; when a human focuses on it the same way (Tab), the dd gets selected.
I believe the yyyy-selection (instead of dd-selection) in puppeteer is causing the interference for typing the value (when a human does it, the value is typed from left to right across the whole input field, automatically jumping to the next date piece). With puppeteer, it types right to left within the year (first the day in the two right-most digits, then the month, pushing the day left, and then overwrites the whole thing with the year).
Can you think of anything that could cause this behaviour? My locale is en-GB and AFAIK, I don't have any right-to-left things set. I had flex-direction: row-reverse set so the picker icon would be on the left, but I disabled that to seemingly no difference.

If you click on the element like date, Puppeteer will click in the middle by default which is likely to be the mm part. The next tab then will go the yyyy part. Perhaps, it is what's happening here.

I'm pretty sure that's not it: I captured a screen recording, and when viewing it frame-by-frame, it appears the yyyy is immediately selected as soon as the field is focused.

We found the cause of both issues (random/chaotic field focus and wrong sub-field focus within date/datetime fields): React's reconciliation.
TLDR: The workaround to set a tiny delay for Puppeteer whenever the DOM will be updated and Puppeteer will interact with that update.
Reconciliation is async process that happens at the very end of React's rendering phase, as a sort of clean-up step; it's very fast (well fast enough to finish before a human can do anything), but Puppeteer is faster—effectively jumping the gun (in Puppeteer's defence—there is no gun, just "ready, set, __").
Trying to come up with a simple reproduction was doomed from the start: It does not re-pro in a simple example because the issue requires a lot of DOM updates in order to cause reconciliation to take longer than Puppeteer.
I think an ideal solution would be to expose some way to inform Puppeteer things are "go". This is possible to know via react's componentDidMount and componentDidUpdate (react renders from leaves to root, so a "page" as a class component would call these after all its content has finished). Would it be possible (and not excessively difficult) to broadcast an event for which Puppeteer would listen and then "go"?
This would also avoid needing waitForSelector (which continuously probes for a match on the provided selector).

@JakobJingleheimer you can use page.exposeFunction to expose a callback to the app and then synchronize based on that.


vendor update; operatorkit

Following https://github.com/giantswarm/operatorkit/pull/325
apply operatorkit changes into chart-operator.

@giantswarm/sig-operator , @giantswarm/team-batman Anyone?

When you do vendor updates you can merge without approval. Especially when the changes you get in are not so big. This is here is not controversial. Just go ahead.


Update README.md

Before you start, please read our contribution guidelines.
Please discuss your idea with the development team before getting started, to avoid disappointment later. The way to do this is to open a new issue on GitHub. If you are planning to modify an existing functionality, please discuss it with the original author(s) by tagging them in the issue.

Tasks

[ ] Create an issue to discuss what you are going to do, if you haven't done so already (and add the link at the bottom)
[ ] Give this pull request a descriptive title that can be used as a one line summary in a changelog
[ ] Make sure your code is composed of functions of no more than 50 lines and uses meaningful names for variables
[ ] Circle/CI tests pass. Status can be seen below your pull request. If the tests are failing, click the link to find out why.
[ ] Preferably Codacy code quality checks pass, however a few remaining hard to solve Codacy issues are still acceptable. Status can be seen below your pull request. If there is an error, click the link to find out why. If you suspect Codacy may be wrong, please ask by commenting.
[ ] Please use yamllint to check that your YAML files do not contain mistakes
[ ] (Only if really necessary) Add any additional dependencies needed for the diagnostic script to setup.py, esmvaltool/install/R/r_requirements.txt or esmvaltool/install/Julia/Project.toml (depending on the language of your script) and also to package/meta.yaml for conda dependencies (includes Python and others, but not R/Julia)
[ ] If new dependencies are introduced, check that the license is compatible with Apache2.0

New recipe/diagnostic

[ ] Add documentation for the recipe to the doc/sphinx/source/recipes folder and add a new entry to index.rst
[ ] Add provenance information

Modified recipe/diagnostic

[ ] Update documentation for the recipe to the doc/sphinx/source/recipes folder
[ ] Update provenance information if needed
[ ] Assign the author(s) of the affected recipe(s) as reviewer(s)

New data reformatting script

[ ] Test the CMORized data using recipes/example/recipe_check_obs.yml, to make sure the CMOR checks pass without errors
[ ] Add the new dataset to the table in the documentation
[ ] Tag @mattiarighi in this pull request, so that the new dataset can be added to the OBS data pool at DKRZ and synchronized with CEDA-Jasmin

Modified data reformatting script

[ ] Test the CMORized data using recipes/example/recipe_check_obs.yml, to make sure the CMOR checks pass without errors
[ ] Tag @mattiarighi in this pull request, so that the updated dataset can be added to the OBS data pool at DKRZ and synchronized with CEDA-Jasmin

If you need help with any of the tasks above, please do not hesitate to ask by commenting in the issue or pull request.

Closes #issue_number

comment in pr

comment in pr 2

@esmvalbot all

@esmvalbot all2

@esmvalbot all23

@esmvalbot all4

@esmvalbot all5

@esmvalbot all6

@esmvalbot all7

@esmvalbot all8

@esmvalbot all81


Add files via upload

@corbytechnicalschool.org

@d0m591 I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host corbytechnicalschool.corbytechnicalschool.org not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


rtabmap: 0.21.0 -> 0.21.4

Description of changes
https://github.com/introlab/rtabmap/releases/tag/0.21.4
Things done

Built on platform(s)

[x] 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

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.
@ckiee

Thanks!


Muthuja/chef 5761 addrocky 9 selinuxenabled

Description
[Please describe what this change achieves]
Issues Resolved
[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussions that are relevant]
Check List

[x] New functionality includes tests
[ ] All buildkite tests pass
[ ] Full omnibus build and tests in buildkite pass
[ ] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/main/CONTRIBUTING.md#developer-certification-of-origin-dco
[ ] PR title is a worthy inclusion in the CHANGELOG

i will create one from main branch since this is not taken from main


Drag event: manipulate the current play time instead of manipulating the region through the drag event.

Wavesurfer.js version(s):
^6.6.3

Hello,
I would like to get advice on the following matter.
Rather than manipulating the region through the drag event, I want to manipulate the current play time.
Apart from controlling the currentTime through a click on the waveform, I would like to be able to set the currentTime by dragging the waveform.

This is implemented in v7 and won’t be added in v6. So please feel free to upgrade.


fix(Kotlin/JS): replace deprecated Kotlin/JS IR compiler API with a n…

The PR provides a new way to compile JS with IR.
The currently used API will be removed from the compiler soon, so the provided changes will prevent breaks on the compiler API side.

should I merge it? =)

Sure. We've just started to delete the described API, so, it will be great if we will merge it before compiler updates that will not contain deprecated API.

Okay, do we need to have test for such changes?

@JSMonk ping

@JSMonk friendly ping


igluctl: omit "no maxLength" warning if hostname format is present (c…

…loses #341)

@misterpig  has signed the Software Grant and Corporate Contributor License Agreement

Closing in favor of https://github.com/snowplow/iglu/pull/424. Thanks!


Warning variant of alert dialog

Description

Warning variant for alert dialog. Issue for adding into DNA: SDS-5068
Why do you need this feature or component?

Feature request from internal teams; already part of Spectrum Contributions
Additional context

There is no warning version of this, we have an alert error.  Closing because it's not needed anymore.

We're out of sync with where design is at on this, and we do need to build out the warning variant. Syncing to Jira. We'll catch this during our migration and refactoring work.


[HELP] coffee

hello i have this error
ubuntu@ubuntu:~/node_modules/pokemon-go-mitm$ coffee example.logTraffic.coffee TypeError: 8080.addRequestHandler is not a function     at Object.<anonymous> (/home/ubuntu/node_modules/pokemon-go-mitm/example.logTraffic.coffee:19:17)     at Object.<anonymous> (/home/ubuntu/node_modules/pokemon-go-mitm/example.logTraffic.coffee:32:4)     at Module._compile (module.js:409:26)     at Object.exports.run (/usr/lib/coffee-script/lib/coffee-script/coffee-script.js:83:25)     at compileScript (/usr/lib/coffee-script/lib/coffee-script/command.js:177:29)     at /usr/lib/coffee-script/lib/coffee-script/command.js:152:18     at FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:380:3)
help me please

Have you ever used javascript before? Any programming language at all? Issues are not a support forum. Please close.


!sku Strange Particulate Protector not working

Describe the bug
!sku Strange Particulate Protector gives null;null as a result
To Reproduce
Steps to reproduce the behavior:

Type "!sku Strange Particulate Protector" in bot's chat.

Screenshot
Version: v3.8.3

Thanks for the report. 👍

Fixed the module, but not the bot. It will be available once v4 is released.
https://github.com/TF2Autobot/node-tf2-schema-2/commit/b26db3435665f61e1901bc3e16afdecd9ef7074b


Microsoft.AspNetCore.Mvc.WebApiCompatShimTest currently does not build \ run tests on net461

Looks like we stopped compiling \ testing it on desktop. Adding net461 TFM back causes builds to fail.

Duplicate of https://github.com/aspnet/Mvc/issues/5822
cc @dougbu @pranavkm


Not possible to add only platform specific styles

Ionic version:  (check one with "x")
[ ] 1.x
[x] 2.x
I'm submitting a ...  (check one with "x")
[ ] bug report
[x] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
In case you build for specific platform i. e. for ios the build adds component styles for all platforms, there are plenty of unnecessary css rules.
Expected behavior:
It should be possible to only include platform specific styles.
Related code:
component.scss
Other information:
We use ionic-app-scripts
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Your system information:
Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.1.4
ios-deploy version: 1.9.0
ios-sim version: Not installed
OS: OS X El Capitan
Node Version: v7.6.0
Xcode version: Xcode 8.2.1 Build version 8C1002
ionic-angular: 2.0.0

This issue was moved to driftyco/ionic-app-scripts#788

You can close it. It is possible with appscripts


net: add ethernet link speed

Resolves #2636
Link speed  in read from /sys/net/<device>/speed, works only for Ethernet interfaces.
Includes additional option for pow_format to skip decimal place, so 1000 will be printed as 1Gb/s, instead of 1.0Gb/s, but 2500 always as 2.5Gb/s.

@Alexays :slightly_smiling_face:


How it will work with react-toolbox 2.0.0-beta.5

In this beta version they converted all .scss files to .css file and using css modules variables like
:root {
  --appbar-color: var(--color-primary-dark);
}

Can you please update your demo according to it ? or create a branch for this major change ?
Thanks

react-toolbox-example uses scss, while from react-toolbox 2.x it uses postcss. So this example won't work. Except this, meteor-css-modules can work with react-toolbox 2.x by using
postcss-cssnext plugins.
meteor npm install postcss-cssnext --save
meteor reset

then, in package.json:
  "cssModules": {
    "postcssPlugins": {
      "postcss-cssnext": {},
      "postcss-modules-values": {},
      "postcss-modules-local-by-default": {},
      "postcss-modules-extract-imports": {},
      "postcss-modules-scope": {}
    },
    "ignorePaths": [
      "node_modules/postcss-nesting",
      "node_modules/react-toolbox/components"
    ]
  }

Then you will get an error about dropdown.css.css mising }. This is cause by a bug of react-toolbox. It has been fix in dev branch, but it hasn't been release. You can fix this by using dev branch , or fix it by hand and wait for next beta release.

@clarkorz Thanks for you time and answer. with your answer i am able to run my project again. but the issue is "How we will override variables?"
first i was doing it like
    "globalVariables": [
      {
        "theme-building": true
      },
      "client/toolbox-theme.scss"
    ],

and inside toolbox-theme.scss file :
$color-primary: #008142 !default;
$color-primary-dark: #fff !default;

it was working fine and overriding these variables all over the app. but now its not working.
i have tried like this on same file :
:root {
  --color-primary: #008142;
}

any suggestions/help ?

Any word on this? I am able to use the beta version of React-toolbox 2 with Meteor, but I haven't figured out how to override the global variables like primary color.

Same problem here. Can anyone please help with this?

I don't use react-toolbox and unfortunately haven't had time to dig into this, but according to the react-toolbox docs, you can override the variables using a JSON object. I think that would look like this in package.json (using @clarkorz's example above):
"cssModules": {
    "postcssPlugins": {
      "postcss-cssnext": {
        "features": {
          "customProperties": {
            "variables": {
              "color-text": "#444548",
              "color-primary": "var(--palette-blue-500)",
              "button-height": "30px",
            },
          },
        },
      },
      "postcss-modules-values": {},
      "postcss-modules-local-by-default": {},
      "postcss-modules-extract-imports": {},
      "postcss-modules-scope": {}
    },
    "ignorePaths": [
      "node_modules/postcss-nesting",
      "node_modules/react-toolbox/components"
    ]
  }


correctly add features when using cache

resolves #282
continuation of #285

Is this PR ready for another review round?

Yes!


Problem with code CS0144

[I think you need to create a constructor. Try using lists and don't forget to type "using System.Collections.Generic;" at the beginning of your code. If you are using an array you need to declare the size of it in a constructor. For example:
class MyClass
{
AObject[] objects;
int objectsNum;
public Object(int projectsNum)
{
this.objectsNum = objectsNum;
objects = new AObject[objectsNum]; //here you declare the size of the array//
}
}
And for lists:
class MyClass
{
List objectList;
public Object()
{
objectList = new List();
}
}
]

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

ID: 8831b6c2-8b55-3e22-b2f4-d6eed52e9ea0
Version Independent ID: 5bf2b97e-b22a-4846-3423-34e3c68b5894
Content: Compiler Error CS0144
Content Source: docs/csharp/misc/cs0144.md
Product: dotnet-csharp
Technology: csharp-diagnostics
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

@andrija2304 It looks like you opened this issue with notes for your own learning.
Is there an issue for us to address here?


Issue with loading 4bit ipex-llm Mixtral saved model with generation length

Optimized the Mixtral model by using ipex_llm.optimize_model() to transform it to low-bit and then save it and then load it.
Set "max_length": 1024 yet getting a warning that max_length (=20) .
Code:
from ipex_llm.optimize import low_memory_init, load_low_bit
from transformers import AutoModelForCausalLM, AutoTokenizer
from ipex_llm.langchain.llms import TransformersLLM
from langchain import PromptTemplate, LLMChain
template ="""{question}"""
prompt = PromptTemplate(template=template, input_variables=["question"])
with low_memory_init():
model = AutoModelForCausalLM.from_pretrained(save_path_model, torch_dtype="auto", trust_remote_code=True)
model = load_low_bit(model, save_path_model)
tokenizer = AutoTokenizer.from_pretrained(save_path_model)
llm = TransformersLLM(model_id="",model=model,tokenizer=tokenizer, model_kwargs={"temperature": 0, "max_length": 1024, "trust_remote_code": True},)
llm_chain = LLMChain(prompt=prompt, llm=llm)
question = "what is AI?"
output = llm_chain.invoke(question)
print("====output=====")
print(output)
Output:
python3.11/site-packages/langchain_core/_api/deprecation.py:139: LangChainDeprecationWarning: The class LLMChain was deprecated in LangChain 0.1.17 and will be removed in 0.3.0. Use RunnableSequence, e.g., prompt | llm instead.
warn_deprecated(
The attention mask and the pad token id were not set. As a consequence, you may observe unexpected behavior. Please pass your input's attention_mask to obtain reliable results.
Setting pad_token_id to eos_token_id:2 for open-end generation.
/home/ceed-user/miniforge3/envs/lc-rag-ipex-2/lib/python3.11/site-packages/transformers/generation/utils.py:1355: UserWarning: Using the model-agnostic default max_length (=20) to control the generation length. We recommend setting max_new_tokens to control the maximum length of the generation.
warnings.warn(
====output=====
{'question': 'what is AI?', 'text': 'what is AI?\n\nAI is a branch of computer science that deals with the creation of'}

what is the version your langchain?

Seems you're running this example on CPU? maybe you can try follow this example to load low bit model. https://github.com/intel-analytics/ipex-llm/blob/main/python/llm/example/CPU/LangChain/low_bit.py

Hi,
We have reviewed your query and used our sample code to address the problem.
Solution:
We utilized our sample code to resolve the issue. Here is the relevant section of our sample code that addresses your issue:
import argparse
from ipex_llm.langchain.llms import TransformersLLM, TransformersPipelineLLM
from langchain import PromptTemplate, LLMChain
from langchain import HuggingFacePipeline

def main(args):
    question = args.question
    model_path = args.model_path
    low_bit_model_path = args.target_path
    template ="""{question}"""

    prompt = PromptTemplate(template=template, input_variables=["question"])

    llm = TransformersLLM.from_model_id(
        model_id=model_path,
        model_kwargs={"temperature": 0, "max_length": 1024, "trust_remote_code": True},
    )
    llm.model.save_low_bit(low_bit_model_path)
    del llm
    low_bit_llm = TransformersLLM.from_model_id_low_bit(
        model_id=low_bit_model_path,
        tokenizer_id=model_path,
        model_kwargs={"temperature": 0, "max_length": 1024, "trust_remote_code": True}
    )
    llm_chain = LLMChain(prompt=prompt, llm=low_bit_llm)

    output = llm_chain.run(question)
    print("====output=====")
    print(output)

if __name__ == '__main__':
    parser = argparse.ArgumentParser(description='TransformersLLM Langchain Chat Example')
    parser.add_argument('-m','--model-path', type=str, required=True,
                        help='the path to transformers model')
    parser.add_argument('-t','--target-path',type=str,required=True,
                        help='the path to save the low bit model')
    parser.add_argument('-q', '--question', type=str, default='What is AI?',
                        help='qustion you want to ask.')
    args = parser.parse_args()

    main(args)

And then use this command to run it.
python low_bit.py -m <path_to_model> -t <path_to_target> [-q <your question>]

Runtime Arguments Explained:

-m MODEL_PATH: Required, the path to the model
-t TARGET_PATH: Required, the path to save the low_bit model
-q QUESTION: the question

Output:
After implementing the solution, we obtained the following output:
Artificial Intelligence (AI) refers to the development of computer systems that can perform tasks that normally require human intelligence, such as visual perception, speech recognition, decision-making, and language translation. AI systems use algorithms and machine learning techniques to analyze data, identify patterns, and make decisions based on that data.

There are different types of AI, including:

1. Narrow AI: Also known as weak AI, this type of AI is designed to perform a specific task, such as voice recognition or image recognition.
2. General AI: Also known as strong AI, this type of AI is designed to perform any intellectual task that a human can do.
3. Superintelligent AI: This is a hypothetical type of AI that is much smarter than the best human minds in virtually all spheres, including scientific creativity, general wisdom, and social skills.

AI is used in various industries, including healthcare, finance, manufacturing, transportation, and education, to improve efficiency, accuracy, and productivity. It can also be used to develop new products and services, enhance customer experiences, and solve complex problems.

However, AI also raises ethical concerns, such as privacy, bias, and job displacement. It is important for organizations and individuals to use AI responsibly and ethically, and to consider the potential impact on society as a whole.
====output=====
What is AI?

Artificial Intelligence (AI) refers to the development of computer systems that can perform tasks that normally require human intelligence, such as visual perception, speech recognition, decision-making, and language translation. AI systems use algorithms and machine learning techniques to analyze data, identify patterns, and make decisions based on that data.

There are different types of AI, including:

1. Narrow AI: Also known as weak AI, this type of AI is designed to perform a specific task, such as voice recognition or image recognition.
2. General AI: Also known as strong AI, this type of AI is designed to perform any intellectual task that a human can do.
3. Superintelligent AI: This is a hypothetical type of AI that is much smarter than the best human minds in virtually all spheres, including scientific creativity, general wisdom, and social skills.

AI is used in various industries, including healthcare, finance, manufacturing, transportation, and education, to improve efficiency, accuracy, and productivity. It can also be used to develop new products and services, enhance customer experiences, and solve complex problems.

However, AI also raises ethical concerns, such as privacy, bias, and job displacement. It is important for organizations and individuals to use AI responsibly and ethically, and to consider the potential impact on society as a whole.


Testing MVP

Fixes weaveworks/weave#2664.
Fixes weaveworks/weave#2676.
Fixes weaveworks/weave#2682.
See also:

weaveworks/weave#2647.
weaveworks/weave#2665.
weaveworks/build-tools#52.
weaveworks/build-tools#53.

As it is assigned to the 1.9.1 milestone, could you change the base branch to 1.9?

@brb, as discussed:

rebased onto 1.9
PR's base branch updated to be 1.9
upgraded to latest version of build-tools's master branch.


Updated test dba compression

Fixes #
Changes proposed in this pull request:

Updated Test-DbaCompression per WSMelton review

How to test this code:

[Test-DbaCompression -SqlInstance ServerA -Database DBName]

Has been tested on minimum requirements:

[x]  Powershell 3
[x]  Windows 7
[ ]  SQL Server 2000

Has been tested on maximum requirements:

[ ]  SQL Server vNext
[ ]  Windows 10
[ ]  Azure Database

Tests for tester:

[x] Working/useful help content, including link to command on dbatools web site
[x] All examples work as advertised
[x] Does not contain template content
[x] Does not contain excessive/unnecessary amounts of comments
[x] Works remotely
[x] Works locally
[x] Works on lower versions or throws error specifying version not supported
[x] Works with named instances
[x] Works with clustered instances
[x] Handles offline/read only databases
[x] Supports multiple servers (at the command line or piped from Get-SqlRegisteredServerName)
[x] No un-handled errors which stop the command working with multiple servers

hey @js0505 - there is a conflict - did you sync dev recently? Can you resync and resubmit a proper PR please 🙇 ? I made most of the changes because I'm trying to get a release out the door by tuesday.
Thank you!


added lowering of FnOnce Call

This change is

crates/cairo-lang-lowering/src/test.rs line 34 at r1 (raw file):

        constant :"constant",
        coupon :"coupon",
        c11losure :"closure",

Suggestion:
closure :"closure",

crates/cairo-lang-lowering/src/lower/mod.rs line 110 at r2 (raw file):

        block_expr_id,
    )
    .unwrap();

revert?
Code quote:
  .unwrap();

crates/cairo-lang-lowering/src/lower/mod.rs line 1804 at r2 (raw file):

        encapsulated_ctx.db,
        semantic_db.concrete_trait_function_signature(concrete_trait_function)?,
    );

The signature here has TraitTypes; How are they going to be replaced?
Code quote:
    let signature = Signature::from_semantic(
        encapsulated_ctx.db,
        semantic_db.concrete_trait_function_signature(concrete_trait_function)?,
    );

crates/cairo-lang-lowering/src/lower/mod.rs line 1888 at r4 (raw file):

    encapsulated_ctx.lowerings.insert(
        GeneratedFunctionKey::TraitFunc(function, closure_ty.wrapper_location),
        lowered.clone(),

why clone?
Code quote:
.clone()

crates/cairo-lang-lowering/src/lower/mod.rs line 1917 at r4 (raw file):

        builder.semantics.closures.get(&capture_var_usage.var_id).unwrap(),
    )
    .unwrap();

why unwrap?
Code quote:
 .unwrap();

crates/cairo-lang-lowering/src/lower/mod.rs line 1809 at r10 (raw file):

            impl_items: GeneratedImplItems(iter::once((ret_ty, closure_ty.ret_ty)).collect()),
        }
        .intern(encapsulated_ctx.db),

Can you use semantic_db here?
Code quote:
encapsulated_ctx.db

crates/cairo-lang-lowering/src/lower/mod.rs line 1855 at r10 (raw file):

                .insert(param.clone(), captured_vars[i + closure_info.members.len()]);
        }
        let param_requests = generators::StructDestructure {

why rquests?
Suggestion:
param_vars

crates/cairo-lang-lowering/src/lower/mod.rs line 1855 at r10 (raw file):
Previously, TomerStarkware wrote…
Done.

I don't see it.


Allow setting of sticky headers.

Motivation:
Currently the only way to set sticky headers is to use Orbit Container, it should be possible to set them when setting up Stage too.
Modification:
Add setter for execution and allow adding sticky headers to Execution.
Result:
Sticky headers can be set programmatically.

I'd like to have more builders like this :)


Support for new activity types was added.

stringForHKWorkoutActivityType method was updated to support new(iOS 14.0) activity types.

Hey @alexeypopovich!
Thanks for the contribution. Currently, this project is being deprecated in favour of react-native-health
We have added a few new workout types on that project instead, which might be useful for your use case. Otherwise, feel free to let me know and we can create a task for those!


Restore test_apply_everywhere_floordivide

This is a for now experimental follow-up to #783 trying to restore the floordivide test, which may have failed due to unrelated open file exceptions, and to check if additional failures reported in https://github.com/radio-astro-tools/spectral-cube/pull/783#pullrequestreview-832819170 are only local to runs under macOS.

Codecov Report

Merging #788 (6c9a4dc) into master (7452dce) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #788   +/-   ##
=======================================
  Coverage   77.90%   77.90%
=======================================
  Files          24       24
  Lines        5826     5826
=======================================
  Hits         4539     4539
  Misses       1287     1287

Continue to review full report at Codecov.

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

One complication was probably that open_files seems to have everything run in a sort of sandbox, making it almost impossible to track the origins of the failures to the respective tests.
The twod_numpy tests, as far as I could see, were starting e.g. with test_twod_numpy[True-sum-slice-0-data_advs], which is skipped, but with the file already opened, and that probably interfered with the open-files check for the next test_twod_numpy[True-sum-cube-0-data_advs] in my local installation.


did not find any git tag for 0.2.1

Yes, I missed to create a tag for that. Did it and pushed it now.

thanks.


Update Composer dependencies (2019-11-11-00-05)

Visual regression test passed!
View the visual regression test report


Map

The buildings are not showing up in the map is it possible to update the map. When ever i open Bing maps all new buildings are shown

Please see this FAQ item on how to report issues with Bing and Mapbox imagery.
iD's background imagery sources come from the editor-layer-index project.  If you know of a better imagery source that is licensed for this use, please open a request there with the link and license details.
Bing might show different imagery at different zoom levels, so that might be why you see inconsistent results?  I'm not sure.  Can you provide a screenshot of what you see in Bing?

Ok what i found is that Bing is using an old map of when no buildings were in some areas i have to go to google map to find the buildings

Ok what i found is that Bing is using an old map of when no buildings were in some areas i have to go to google map to find the buildings

Thanks for looking into this..  We are really not allowed to use Google Maps as a source for adding anything to OpenStreetMap, so the best thing to do is to just wait until newer imagery is available of this location (or to go there personally and record the positions of the buildings with a GPS receiver or some other method).

It would be a good idea if custom imagery sources which have "google" in the URL were prevented from being used as a background in ID.  I've just come across one user , using ID, making changes to OSM based on  Google imagery

It would be a good idea if custom imagery sources which have "google" in the URL were prevented from being used as a background in ID. I've just come across one user , using ID, making changes to OSM based on Google imagery

We already do this - see code here, however I'm not sure these checks catch all google domains.  If we need to change this code, can you please open a new issue?


server: Don't close underlying UDP conn after handling

Should fix #53, and a bunch of other issues saying that UDP wasn't working properly, that were closed without the problem being totally understood

Thanks for the patch. Guess I didn't consider this (I don't really use the UDP sockets myself).
Does this cause a leak?

Umm, no, cause it gets closed correctly here: https://github.com/mholt/caddy-l4/blob/d1f4337132468dfe2d081baba3769dfd1f46fbf8/layer4/app.go#L97


Add support for limiting the number of active HTTPRequest objects

Too many requests can begin to fail on HTML5, or hurt performance on native, so we should add support for setting a maximum number of simultaneous requests

I guess this was related to d593685dcb12ac7fee79cf41119bfa84b99b6eb7 and http://community.openfl.org/t/openfl-4-8-1-preloader-error-load-assets-cause-stopping-loading-at-html5/8832
Not sure it's a good idea to throttle assets for all browser out there, especially since now with http/2 we can load up to 100 assets at the same time.

I think the issue that the original posters had happened when loading 400-500 assets at the same time. Increasing the private static var requestLimit = 4; from 4 to a much higher value should be relatively safe. Or making it public, so it can be overridden - could also be an OK solution.
4 seems to be a very low value. When hacking this to a value of 50+ then an assets heavy game significantly decreases load-times ( e.g. when opening a shop popup that loads its own assets ). Until now I was wondering why so many assets were slowly loading in sync, instead of async.

Changing https://github.com/openfl/lime/blob/develop/lime/_backend/html5/HTML5HTTPRequest.hx#L32 to :
-       private static var requestLimit = 4;
+       public static var requestLimit = 4;

And then defining lime._backend.html5.HTML5HTTPRequest.requestLimit = 100; in the app had the following impact on an asset heavy html5 game ( both loads with cache disabled ) :

That's ~40 seconds vs ~80 seconds.


[Bug]: AfterGroups function is not executed

Description
$subject
Steps to Reproduce
Execute bal test --groups g1 after-groups-with-disabled-test.bal on the following file:
https://github.com/ballerina-platform/ballerina-lang/blob/master/tests/testerina-integration-test/src/test/resources/single-file-tests/grouping/after-groups-with-disabled-test.bal
Affected Version(s)
2201.4.0-SNAPSHOT
OS, DB, other environment details and versions
No response
Related area
-> Test Framework
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

The issue is due to skipping the disabled test during the test ordering process, and then checking if all the tests are executed before running the before group method. But this condition is never met since the disabled test is not going to be executed.
Working on a solution as of now.


Divide large ast.cpp file

Currently all the AST classes' definitions are done in a large ast.cpp file that increases a lot the build time of NMODL.
We should find ways to break it down to accelerate the build time.

Do we really think that it will decrease build time?  Maybe this is not link to ast.cpp?


Monkey Patch _oauth2.getOAuthAccessToken to reformat Slack's OAuth2 v2 token response

A proposed change to add a hook that would allow this strategy to reformat the OAuth2 Token Response before sending it forward to the passport-oauth2 callback received some reasonable (though unfortunate) push back.
I took the advice in the push back, though probably the wrong part, and chose to not implement a solution that created a "maintenance burden".  This new pr monkey patches node-oauth's OAuth2.getOAuthAccessToken method to reformat the OAuth2 Token Response before sending it forward to the passport-oauth2 callback.
This change introduces the method getOAuthAccessTokenAndHandleResponse which wraps this._oauth2.getOAuthAccessToken and reformats the response before sending it forward to the callback.

Original Summary for PR #13
Please see https://github.com/jaredhanson/passport-oauth2/pull/174 AND https://github.com/nmaves/passport-slack-oauth2/issues/9
Slack's OAuth2 v2 implementation overloads the OAuthTokenResponse json payload to return multiple tokens.  The root of the json object is for bot tokens & user tokens exist in a property called authed_user.
Passport is designed to authenticate Users. When passport attempts to use the root level accessToken, it either does not exist (no bot scopes provided in authorization request) or is a bot token.
This change leverages a new hook in the passport-oauth2 library to reformat the OAuthTokenResponse payload to set the correct accessToken, refreshToken, and params for the following scenarios:

user
user and bot
bot

The profileUrl is no longer defaulted during configuration. When the profile is not being skipped & a custom profileUrl was not provided, the profileUrl is set during handleOAuthAccessTokenResponse depending on which token is the root of the params object.

if this is merged, the README should be updated to emphasize the verify callback that accepts the params object from the OAuthTokenResponse request.
the params object will be reformatted to:
user_scope - user only
{
  id: 'user-id',
  scope: 'user-scope-1,user-scope-2',
  token_type: 'user',
  access_token: 'user-access-token',
  refresh_token: undefined,
  authed_user: {
    id: 'user-id',
    scope: 'user-scope-1,user-scope-2',
    access_token: 'user-access-token',
    token_type: 'user'
  },
  authed_bot: {}
}

scope - bot only
{
  id: 'bot-user-id',
  scope: 'bot-scope-1,bot-scope-2',
  token_type: 'bot',
  access_token: 'bot-token',
  refresh_token: undefined,
  authed_user: { id: 'user-id' },
  authed_bot: {
    id: 'bot-user-id',
    scope: 'bot-scope-1,bot-scope-2',
    token_type: 'bot',
    access_token: 'bot-token',
    refresh_token: undefined
  }
}

scope & user_scope - bot & user
{
  id: 'user-id',
  scope: 'user-scope-1,user-scope-2',
  token_type: 'user',
  access_token: 'user-access-token',
  refresh_token: undefined,
  authed_user: {
    id: 'user-id',
    scope: 'user-scope-1,user-scope-2',
    access_token: 'user-access-token',
    token_type: 'user'
  },
  authed_bot: {
    id: 'bot-user-id',
    scope: 'bot-scope-1,bot-scope-2',
    token_type: 'bot',
    access_token: 'bot-token',
    refresh_token: undefined
  }
}

@nmaves what are the next steps to getting this merged?


ADFS Autnetication

How can i enable ADFS Authentciation to serenity?
Anybody has done this? Can some one assist if its explored by you already?
Thanks in Advance.

Hi @jabirmoolur ,
I never have done this with SF, but as it is an ASP.net MVC app, maybe this article can help you further?
https://developers.de/blogs/armin_kalajdzija/archive/2016/10/06/how-to-implement-web-sign-on-with-adfs-in-asp-net-mvc-using-owin.aspx
With kind regards,
John


Cannot start Client on RedHat 6

I confirm (by marking "x" in the [ ] below):

[x] This is not a feature idea. Feature ideas go here: http://www.mattermost.org/feature-ideas/
[x] This is not a troubleshooting question. Troubleshooting questions go here: http://www.mattermost.org/troubleshoot/
[x] This is a product defect issue that can be reproduced using the steps listed below.
[x] I have read CONTRIBUTING.md.

Summary
Mattermost Desktop 3.4.0 on RH6 does not start GLIBCXX_3.4.15 missing
Steps to reproduce

On standard RedHat 6.8 we extracted the Mattermost distribution for linux.
Start the app by ./Mattermost
Errors that specific versions of GLIBCXX_3.4.15 are missing.

Expected behavior
The app should start normally. No interface appears, stops directly.
Observed behavior
/usr/lib64/libstdc++.so.6: version 'GLIBCXX_3.4.14' not found (required by ./Mattermost)
/usr/lib64/libstdc++.so.6: version 'CXXABI_1.3.5' not found (required by ./Mattermost)
/usr/lib64/libstdc++.so.6: version 'GLIBCXX_3.4.14' not found (required by ./Mattermost)
/usr/lib64/libstdc++.so.6: version 'GLIBCXX_3.4.14' not found (required by ~abij/mattermost-desktop-linux/libnode.so)

On RedHat 7, these version are available, because its using a newer GCC version. This is bundled with a newer libstdc++ file which includes these versions. You cannot replace these on RH6.
Possible fixes
We have tried to install devtoolset-4 and enable it before starting Mattermost. This did not work.
Maybe it can be build for RedHat 6 by default with electron-packager.

It seems that your report means RedHat 6 can't use any Electron-based applications. electron-packager uses prebuilt Electron to make applications, so it depends on electron's spec.
I feel the app should work if there is correct libstdc++ in your system. But I'm not sure how devtoolset-4 works. Or, probably manually need to build Electron from source code.

Looks the same as: https://github.com/electron/electron/issues/1172
And relates to: https://github.com/atom/atom/issues/2177
I might try this 'solution':
https://chrome.richardlloyd.org.uk/build_library.html

Did the solutions work for you @abij?

Just Copy

libstdc++.so.6
ld-linux-x86-64.so.0
libc.so.6
libgtk-x11-2.0.so.0
under the path .....src/node_modules/electron/dist . You can get this from Centos 7 .
Then Download https://jlison.github.io/bin/centos6/soichih/mirror/gcc-4.7.2.tar.gz and extarct it to some folder and update LD_LIBRARY_PATH

glibpath=/opt/glib/  (path where you have extracted glib )
export LD_LIBRARY_PATH=$glibpath/glib-2.49.4-kgesagxmtbemim2denf65on4iixy3miy/lib:$LD_LIBRARY_PATH
export LD_LIBRARY_PATH=$glibpath/libffi-3.2.1-wk2luzhfdpbievnqqtu24pi774esyqye/lib64:$LD_LIBRARY_PATH
export LD_LIBRARY_PATH=$glibpath/pcre-8.39-itdbuzevbtzqeqrvna47wstwczud67wx/lib:$LD_LIBRARY_PATH
export LD_LIBRARY_PATH=$glibpath/gettext-<IP_ADDRESS>-aoweyaoufujdlobl7dphb2gdrhuhikil/lib:$LD_LIBRARY_PATH
I am able t o run electron on my centos 6.5 and nothing breaks !

Thanks @blackSea4477, glad to hear!
Closing for now -- if anyone finds further issues on RedHat 6, let us know!

 relocation error: libc.so.6: symbol _dl_starting_up, version GLIBC_PRIVATE not defined in file ld-linux-x86-64.so.2 with link time reference
I'm facing above issue. Any solutions?


locations.json creating duplicate location "name" due to county/city dup (Virginia #1)

locations.json is creating multiple records with the exact same location "name" due to the following...
If you have the following:
Country | State | County | City
US         |  VA     | Bristol City  |   null
US         |   VA     | null              |  Bristol city
You can't concatenate and create "name" =
US, VA, Bristol City
US, VA, Bristol city
On my BI report I concatenate like...
US-VA-Bristol City-
US-VA--Bristol city
The hyphen delimiter always keeps it straight (even null gets a delimiter) if we have a county or a city.
Here is an example of the actual JSON.

And here is a complete list of locations with duplicates.

I've just added a big PR which fixes this for all counties, can you check it on current master? (Or just check it tomorrow when new data will be published.)

 The release is going out right now.
@fordmaxson, let us know if the issue is fixed tomorrow. Thanks!

Great job.  I concur that we have no duplicate "name" values anymore.  Thanks!


Adding R as a supported language

Is it possible to add R to the supported languages as I use it a lot for statistical analysis and I think this would be a good addition to the package.

R is part of the package, introduced back in #114.


Import errors hiding parse errors

If you have a .arcs file that does not parse properly is imported by another .arcs file, the errors generated make it appear the second .arcs file has the error.
This error has been seen in the creation of the Javascript demo for tutorials. (See https://github.com/SHeimlich/arcs/tree/tutorial-demo
In the tutorial demo, the Board particle is defined in Board.arcs which is imported into Demo.arcs. There was a parse error (incorrect tabs) with one of the Cell particles in the in the Board.arcs file. Because the Board.arcs file did not parse correctly, it was not imported as is reflected in the fourth error message (see below). But, since the actual error is totally hidden from developer, it's very difficult to figure out what is going on.
Error messages that appeared in the log:

I didn't commit when the issue was appearing, but I've been able to reproduce it pretty simply on my machine. If you go into Board.arcs and put a tab on line 50 before "particle DisplayAvatar3 ..." you should be able to reproduce the error messages.
I've used the term "hidden" since the error messages all relate to Board, but the actual error is coming from the DisplayAvatar3 particle definition. When debugging this, I spent a significant amount of time looking at the definition of the Board particle within Board.arcs, and never thought to scroll down and look at the other particles in the file.
I agree this seems similar to #2570.

To clarify, do you mean hidden as in not shown (I haven't been able to repro this), or hidden as in swamped in other errors (I've been able to repro this)?
I think we can improve the latter problem by failing fast (i.e. in the import, without finishing the importing file) but need @shans and @piotrswigon to sign off on changing our approach to failure (as they're the ones who I believe know why it is currently this way).

The way the error appears to read to me is something along the lines of:

didn't expect inout at line 56 of Board.arcs (this would be because of the indenting at line 50)
.. so therefore couldn't import Board.arcs
no Board defined but it's referenced in Demo.arcs
..so therefore couldn't import Demo.arcs

Point being that now I know exactly what's gone wrong - Tutorial.arcs couldn't import Demo.arcs, because Demo.arcs expected Board from Board.arcs; but Board.arcs had a parse error on line 50.
Failing fast would make the first error more prominent but hide how you got there; I agree what we have is wordy though. Is there any way for us to summarize errors after the first, so we still get the chain of evidence but without drowning people?

Yep. I agree about the interpretation.
Failing fast would show errors even if the containing construct didn't get used and (correspondingly) would not explain the dependency chain. We could still show the import chain which lead to the error (impl here: https://github.com/PolymerLabs/arcs/pull/3608).
This does remove some information that is currently available to the developer, but it also means that the particles they have written are available when they reach for them and should reduce cognitive load when doing actual debugging (as I don't have to worry about all the particles depending on my broken particle until it works in isolation).

Can you explain your second paragraph in more detail? AFAIK "fast fail" just means die as soon as an error is detected; how does this mean that particles are available more frequently or that one doesn't need to worry about particles depending on a broken one?

(Also how does #3608 show the import chain? It doesn't appear to)

Sorry for the comment spamming..
From memory, there's a good reason that importing a broken manifest didn't cause the current manifest to die too - it makes a user's context extraordinarily brittle as any changed (and now failing) import nukes everything.
We could import differently into the context, and at some point we shouldn't be relying on the current version of things but rather the version at which they were instantiated into arcs; these are both bigger changes that we need to work through a design on though.

My assumption is that these parse errors would accumulate in projects (not unlike C++ warnings) slowly making devEx worse and then developers would be burdened with having to search through the (now potentially large) error logs for relevant errors. Fast failing avoids this being possible as the code base is broken until you respond to the broken code.
I'm also concerned about the potential for misinterpreting manifests if we fail part way through. Fast failing avoids this too, as if the parser fails the developer has to step in (rather than having an approach to work with some kind of best guess).
#3608 shows fast fail, the imports should be accumulated by the pre-existing code
   manifest.errors.push(new ManifestError(item.location, `Error importing '${target}'`));
But re-throwing, rather than pushing the exception ensures we do not continue after an error.
I agree with the need for a design on versioning of imports but hopefully this is out of scope.

The notion of the context doesn't match your assumption; a context isn't a project that developers explicitly add to, it's the accumulation of user state. If it is brittle then the user experience is really bad as a changed file renders all the user's arcs inaccessible. I think this was actually happening; that's why we changed to this approach in the first place.

BTW I'd be interested to see a manifest that was meaningfully misinterpreted; failing and recovering just means that we skip the contents of a manifest that was imported.

Ah, I had only been concerned with imports during development.
I think the versioning system (and capturing manifests at instantiation time) should fix the brittleness (keeping the serialized and current versions in sync is a challenging problem, equivalent to doing a database migration with every particle change), I don't know that we actually want to try to do that anyway (instead allowing users to choose to use old versions).
I don't think the error reporting system should be made to paper over this complexity, its harder to use (this is the third issue opened about import based debugging troubles that I can think of).
About the BTW, I'm pretty sure that with today's manifest language the worst result of the mis-parse would be a subset of suggestions being generated which would reduce options / confuse the user, but it's a long way from a catastrophe.

It definitely fixed the problem.. the problem being that modifying a single manifest had the potential to render all of your arcs useless.
I don't think improving developer understanding at the expense of going back to this behavior is an acceptable trade-off.

My .2c would be that we should be able to improve the DevEx if we improve the error reporting.
Some ideas:

The first warning mentions the Demo.arcs file first, and then the Board.arcs file and the same line number for both (56) - this is confusing and maybe even wrong. It may be better if we only mentioned the actual file in which the error appears - the following warnings show the causality trail.
If we have given more than one line of context (e.g. +2  surrounding lines, that could make it easier to debug issues), but that seems to be unusual... thoughts? I may be biased just because this error is padding related...

Talked to shane this morning to iron out my understanding of the context in its current form.
@piotrswigon I'd prefer to avoid printing the context (in favour of developers using their own editor highlighting, etc. This also better supports usage of the language server where error message length is visually costly).
Having the first report be the core error and the rest be the causality trail sounds very reasonable, though if we can condense this it would be very helpful.
It seems like fast fail is not going to be viable for the foreseeable future and summaries would be a huge improvement.

Would you have bandwidth to investigate why we're having both the path to Demo.arcs and Board.arcs in the first error? I presume the thinking is that it's showing the trail of import, but IMO it would be more readable if we had the path to the file with error first, something like:
Parse error: Expected foo but found bar:
   bar bar barh
   ^

in /a/b/c/d/File.arcs:38
  imported at Demos.arcs:12
  imported at Canonical.arcs:5

Or something along these lines. WDYT?
@SHeimlich maybe could comment.

Small update to jog my memory in future: I believe this issue was mostly due to the 'retry' mechanism that allowed schemas to be added to the manifest until there was a topological ordering found that insert them such that they didn't throw any exceptions.
This was removed  last quarter in favour of a two pass system: one finding all the schemas and the other populating them.
I expect the exceptions to be far more understandable now.
As such, I'm closing this for now and we can re-open if we find that it's still occuring for other reasons.


So Issue of editor

why all bots is unknow For example (unknow enemy in first map :dabot: ) and if i  inport map to old sandox dabot is black and i cant make him die

This issue is getting stale at this point... Closing issue.

This issue is getting stale at this point... Closing issue.


Automated daily update to rustc (to nightly-2024-08-14)

Automated update to Github CI workflow rust.yml by create-pull-request GitHub action

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.18%

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
19702

Relevant Lines:
23686

💛  - Coveralls


Destructing is not supported in the diagram.

Description:
The mentioned destructure binding patterns are not supported in the diagram.

Tuple Destructure Binding Pattern
Record Destructure Binding Pattern
Error Destructure Binding Pattern

Steps to reproduce:
refer examples :
tuple_destructure_binding_pattern.bal
record_destructure_binding_pattern.bal
error_destructure_binding_pattern.bal
Affected Versions:
0.992.0-m2-SNAPSHOT

When in expanded mode, some statements are not shown. Eg: below statement is not shown
ballerina  (a1, (a2, a3)) = ("Ballerina", (453, false));

https://github.com/ballerina-platform/ballerina-lang/blob/master/examples/tuple-destructure-binding-pattern/tuple_destructure_binding_pattern.bal#L8

Issue is no longer valid for Swan Lake release. Hence, closing it.

Issue is no longer valid for Swan Lake release. Hence, closing it.


Fix Debian's family service name

fixes error introduced in #26 (reported in this comment)

@javierbertoli sorry for all the commotion with this formula, but it was in desperate need for some refactoring...


uutils-coreutils: disable general unix build for now

Motivation for this change
Currently the installation on Darwin fails due to a bug in the
installation script: https://github.com/NixOS/nixpkgs/pull/34840#issuecomment-369227222
As I lack knowledge and resources to properly test this package against
Darwin I'll disallow Darwin builds for now.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Thanks!


Rename micro-http to dbs-uhttp

Rename micro-http to dbs-uhttp to avoid conflicting with upstream
project.
Signed-off-by: Liu Jiang<EMAIL_ADDRESS>Reason for This PR
[Author TODO: add issue # or explain reasoning.]
Description of Changes
[Author TODO: add description of changes.]
License Acceptance
By submitting this pull request, I confirm that my contribution is made under
the terms of the Apache 2.0 license.
PR Checklist
[Author TODO: Meet these criteria.]
[Reviewer TODO: Verify that these criteria are met. Request changes if not]

[ ] All commits in this PR are signed (git commit -s).
[ ] The reason for this PR is clearly provided (issue no. or explanation).
[ ] The description of changes is clear and encompassing.
[ ] Any required documentation changes (code and docs) are included in this PR.
[ ] Any newly added unsafe code is properly documented.
[ ] Any user-facing changes are mentioned in CHANGELOG.md.

Codecov Report

Merging #9 (bc21307) into main (52e7ffe) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #9   +/-   ##
=======================================
  Coverage   95.98%   95.98%
=======================================
  Files           9        9
  Lines        2542     2542
=======================================
  Hits         2440     2440
  Misses        102      102

Impacted Files
Coverage Δ

src/common/headers.rs
97.56% <ø> (ø)

src/common/mod.rs
87.94% <ø> (ø)

src/lib.rs
100.00% <ø> (ø)

src/request.rs
97.24% <ø> (ø)

src/router.rs
90.00% <ø> (ø)

src/server.rs
93.25% <ø> (ø)

Continue to review full report at Codecov.

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


Create swap.java

What does the program do ?
swap two number
In what programming language it is written?
Java

Thank you for participating  in hacktoberfest ❣ if you liked working on this repo you can share it with friends and star this project for helping others


Conflict with Mac and Office 365 - Typically MS Excel

Took me a while to work this out - but Quicksilver and Excel no longer live in harmony on my Mac.
Copy / Paste of cells using keyboard shortcuts... and general slowness result.
When I shut down Quicksilver and rebooted - the Excel problem went away.
Did a fair bit of testing on this... so I guess it's a thing. I told the Microsoft Support Network people about this too in case you need to know!
I have MS Office 2011 for Mac also on my computer - and it runs fine - always did! - This issue is with 365 on a MacBook Pro late 2015.

It certainly does! - I just tested it! - Well done - and thanks for helping me.

Well, that’s not a fix, but it gives us an idea where the problem is. Something about the Excel data must be confusing the clipboard code.
A less extreme solution might be to list Excel under “Ignore Applications” in the Clipboard plug-in’s preferences.

Gotcha - thanks for the advice. I shall make the necessary alterations.

@spiketrain  - what version of Excel are you using, and is this still an issue?
I am unable to reproduce here using Excel version 16.46