toggle mode gives error in console on resize on pages without table

Thanks for awsome plugin
I am facing an issue. The tablesaw plugin is activated on an ajax page that contains table with toggle enabled. Everything works fine on that page until switching to another page by second ajax request where there is no tablesaw table.
init code -
$("#grid table").attr("data-tablesaw-mode","columntoggle");
 $("#grid table").attr("data-tablesaw-minimap","");
 $("#grid table").attr("data-tablesaw-sortable","");
 $("#grid table").attr("data-tablesaw-mode-exclude","columntoggle");
 $("#grid table").attr("data-tablesaw-mode-switch","");
 $(document).trigger( "enhance.tablesaw" );

CSS class for table -  tablesaw tablesaw-columntoggle
Also data-tablesaw-priority is provided for each column between value 1 to 6
resizing console in chrome gives following error in second tab without table: seems like javascript binding is causing some issue
tablesaw.js:470 Uncaught TypeError: Cannot read property 'eq' of undefined
(anonymous function)    @   tablesaw.js:470
jQuery.extend.each  @   jquery.js:611
jQuery.fn.jQuery.each   @   jquery.js:241
ColumnToggle.refreshToggle  @   tablesaw.js:467
(anonymous function)    @   tablesaw.js:460
jQuery.event.dispatch   @   jquery.js:3058
elemData.handle.eventHandle @   jquery.js:2676

This error happens only with Toggle option and on pages where there is no table on resize.
How can we fix this ?

Only when I am running as below on second ajax request without table , errors go away related to refreshToggle -
$(window ).unbind( "resize.table-987" );
Is there any wrong configuration in my setup ?

Try out 3.0 and let me know if this doesn’t work for you!

I replaced 2.0.1 tablesaw.js with latest v3.0.0-beta.0 tablesaw.jquery.js and same with css
I am using jquery 1.8.3
On resizing console on pages without table , I get now below error ->
tablesaw.js:521 Uncaught TypeError: Cannot read property 'cells' of undefined

I am current intercepting all Ajax requests and doing as below -
XMLHttpRequest.prototype.send = function(data) {
        	$(window).OffRegexNameSpaces("resize",/^table-\d{3}/);
                send.call(this, data);
 };
})(XMLHttpRequest.prototype.send);

For regex function, using below link -
[http://stackoverflow.com/questions/38806912/unbind-to-multiple-namespaced-events-in-jquery]

I think I’m going to need a test case to fix this properly. I don’t quite understand what you’re doing. Can you put a reduced test case that shows the problem on jsbin (or similar)? If you do this soon, it’ll probably make it into 3.0. Thank you!

Please reopen with a test case!


Show how to use broadcasting effectively

We want to show students how to use broadcasting to write concise code. However currently in notebook 01 the Matmult with broadcasting section takes 28ms, whereas the Swift loop and Pytorch broadcasting version both take ~0.25ms.
If we can't show how to write concise performant code, then we aren't giving people a way to actually write their own layer logic - which means that they can't really take advantage of s4tf at all; they'd only be able to glue together pre-written layers, which isn't terribly useful...
If fixing this requires XLA then this and implementing performant RunningBatchNorm are likely to go together, so tagging @jekbradbury here.
@saeta provided this information:

Unless I'm confusing two issues, we've done an investigation into this. The issue is that we've written our APIs to facilitate GPE, but this results in lots of small eager ops that have to hit the old TF core, resulting in relatively inefficient execution. This is of course a temporary issue while we migrate to MLIR + the new runtime. In the meantime, we're exploring using XLA to optimize at a function level: https://bugs.swift.org/browse/TF-407

One thing I don't understand here is why PyTorch (which is also "eager") can do this 100x faster than s4tf, despite not using any XLA/JIT type functionality. Is that an underlying TF issue, or something we can fix at our level?

Closing to move this to forum.


Added checks for reverse.tail.reverse and reverse.head

Added inspections:

reverse.tail.reverse can be replaced with init
reverse.head can be replaced with last

:+1:

Thanks! That was fast.

No, thank you for contributing.
On 22 February 2016 at 15:19, Vassil Dichev<EMAIL_ADDRESS>wrote:

Thanks! That was fast.
—
Reply to this email directly or view it on GitHub
https://github.com/sksamuel/scapegoat/pull/135#issuecomment-187226570.


Modify CI checks to use latest CI actions

Signed-off-by: divyansh42<EMAIL_ADDRESS>

Did something wrong while rebasing


Performance improvement: don't copy the entire Activity in JObject

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Version
4.11.0
Describe the bug
TelemetryInitializerMiddleware stores the entire activity in the HttpContext. This increases memory pressure and decreases performance. We don't need to store the entire serialized activity there, we just need to store a few properties like conversationId or activityId. We should create a new class like BotTelemetryContext and copy all required properties from the activity.

@Robulane do you have a clear picture of what the performance impact here?
I don't believe we can / should change this.  I think we would risk breaking people who have overridden parts of the telemetry pipeline and are existing the full object to be there.
@Robulane I think the right approach would potentially be for you to override the appropriate parts of the pipeline and just store what you need.

I didn't perform any benchmark but basically for each activity sent there is one Serialization-Deserialization roundtrip for no valid reason.

Yes, I created my own middleware to workaround this.

I didn't perform any benchmark but basically for each activity sent there is one Serialization-Deserialization roundtrip for no valid reason.

Yes, I created my own middleware to workaround this.

Closing this on as Roman already has a workaround.


Future time redux

This is a continuation of @markmcspadden's excellent pull from a year and a half ago: #42 (which is also related to #38).
Basically, time-elements should support future dates too. I'm living in the future so the present is my past, time-elements is a present, merge if tests pass (they do).
The majority of this is pulled from @markmcspadden's work; I just updated the syntax from the deprecated <time is="local-time"> form and updated the README.
This pull adds an additional time-until element (which we can axe if it's not worthwhile), and it updates relative-time to see the future, too.
/cc @josh @dgraham @github/techno

PR to PR suggestion https://github.com/holman/time-elements/pull/1

PR to PR suggestion holman#1

It's a PR to my PR which is based on another PR. That's a new Personal Record for me.

Thanks @holman for resurrecting this one. Enjoy the future!

Auto-merged branch holman-future-time into master with Monster reference


[$1000] iOS - Unread LHN - Unread username unbolded after minimize or open command center

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Run NewDot App
Open LHN and receive new msg while focused on the tab
Minimize App or open Command center few times

Expected Result:
Unread indication functionality should works without any issues at LHN tab
Actual Result:
If the user focused on the LHN tab and new msg is received: App minimising or opening command center will lead to unbold username at the LHN tab
Workaround:
Unknown
Platform:

Where is this issue occurring?

iOS

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers): any
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/196318472-54550aed-4e91-4185-8006-dc9c4503182c.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

hmm strange! tossing to the pool, I don't think this need to be a daily but def isn't great UX.

I think this needs the External label, we've engineering reviewed and triaged out of order

I'm getting an error when I try to post to Upwork. I've raised it in #bugzero https://expensify.slack.com/archives/C01SKUP7QR0/p1666183255766669

Just needed to wait a little bit and now it's posted

The job is here https://www.upwork.com/jobs/~01b43085bb2505c069

I think this issue was reported first in here by me : https://github.com/Expensify/App/issues/11514

I don't see any similarities. It's a different issue.

Price has been doubled to $500

Should we double again?

Price has been doubled to $1000 - will cross post to Slack.

Proposal
Why this issue is produced ?
This issue is produced because unsubscribeVisibilityListener is not unsubscribed correctly when navigating to home.
As you know, VisibilityListener was defined in the src/pages/home/report/ReportActionsView.js.
There is also an unsubscriber in componentWillUnmount but not called when page is navigated to home by back button because opening the drawer will not destroy the component (navigating to home will be done by DrawerActions.openDrawer()).
componentDidMount() {
        this.unsubscribeVisibilityListener = Visibility.onVisibilityChange(() => {
            if (!this.getIsReportFullyVisible()) {
                return;
            }

            // If the app user becomes active and they have no unread actions we clear the new marker to sync their device
            // e.g. they could have read these messages on another device and only just become active here
            this.openReportIfNecessary();
            this.setState({newMarkerSequenceNumber: 0});
        });

        this.keyboardEvent = Keyboard.addListener('keyboardDidShow', () => {
            if (!ReportActionComposeFocusManager.isFocused()) {
                return;
            }
            ReportScrollManager.scrollToBottom();
        });

Inside the callback of this listener, openReportIfNecessary was defined to update the report occasionally.
As mentioned in the comment, this method is needed to update the unread message status. (Internally, it will call API with the OpenReport command and update the onyx data).
Because AppState is changed when the app is minimized and open, and the listener is still alive,  openReportIfNecessary is called in the callback.
And it updates the report data as if user is still on ReportActionsView component (chat page).
Solution:
From the above problem, important point is to unsubscribe the VisibilityListener when page is leaving from ReportActionsView component.
So, I am going to write unsubscribing method like below, and call it by ref. (I know there are several way to unsubscribe. But I used ref here. Hopefully, let me know if this is not a good way for coding style in this project)
src/pages/home/report/ReportActionsView.js
componentWillUnmount() {
+        this.unsubscribeEvents()
   }

+    /**
+     * unsubscribe all of the events when component is unmounted or navigated to home
+     */
+    unsubscribeEvents = () => {
+        if (this.keyboardEvent) {
+            this.keyboardEvent.remove();
+        }
+
+       if (this.unsubscribeVisibilityListener) {
+          this.unsubscribeVisibilityListener();
+        }
+
+       if (this.unsubscribeFromNewActionEvent) {
+            this.unsubscribeFromNewActionEvent();
+        }
+
+       Report.unsubscribeFromReportChannel(this.props.report.reportID);
+    }

src/pages/home/ReportScreen.js
        super(props);

+        this.reportActionsViewRef = React.createRef();
        this.onSubmitComment = this.onSubmitComment.bind(this);

...
      <OfflineWithFeedback
              pendingAction={addWorkspaceRoomOrChatPendingAction}
              errors={addWorkspaceRoomOrChatErrors}
              shouldShowErrorMessages={false}
          >
              <HeaderView
                  reportID={reportID}
-                  onNavigationMenuButtonClicked={()=> Navigation.navigate(ROUTES.HOME) }
+                  onNavigationMenuButtonClicked={() => {
+                      if (this.reportActionsViewRef.current !== null) {
+                          this.reportActionsViewRef.current.unsubscribeEvents()
+                      }
+                     Navigation.navigate(ROUTES.HOME)
+                  }}
                  personalDetails={this.props.personalDetails}
                  report={this.props.report}
                  policies={this.props.policies}
              />
          </OfflineWithFeedback>

...
      {this.isReportReadyForDisplay() && (
              <ReportActionsView
+                  ref={this.reportActionsViewRef}
                  reportActions={this.props.reportActions}
                  report={this.props.report}
                  session={this.props.session}
                  isComposerFullSize={this.props.isComposerFullSize}
                  isDrawerOpen={this.props.isDrawerOpen}
                  parentViewHeight={this.state.skeletonViewContainerHeight}
              />
          )}

This will resolve the issue.
PS: I've also noticed that Android has same problem, and tested in both ios/android platforms.
Those are working fine.

The proposal looks good at first glance but I won't be able to look much into it right now (4 AM for me) will look into it ASAP in the morning.

~The proposal looks good at first glance~ Taking my words back, it doesn't look good after looking into the issue.
@railway17, I am little lost here. I am not sure how your proposal solves this issue, can you confirm the issue? Also can you share a video after the fix?

I found that if you mark the last message as unread and refresh the page then it gets read.
https://user-images.githubusercontent.com/83179295/200791424-bc08dffa-2c80-4722-b0a3-288e75809e27.mov
Steps:-

Navigate to any chat
Mark the last message as unread
Observe that the username is bold
Navigate to a different chat
Refresh the page
Observe that the username is not bold anymore

@iwiznia, should we include it in this issue or should I post this on slack.

I found that if you mark the last message as unread and refresh the page then the chat gets read.

Navigate to any chat
Mark the last message as unread
Observe that the username is bold
Navigate to a different chat
Refresh the page
Observe that the username is not bold anymore

Actually, this issue is reported with the reproducible platform - iOS.
So, I tested on mobile platforms (in both iOS and Android) and checked what is working.
But I missed the testing case by reproducing with Mark as Unread.
Just tested by sending a new message from another user.
I will update you with videos for both cases.

@thesahindia
https://drive.google.com/file/d/1mcirZLECKnUCn-tWBT-d9fsmLk0fp_PS/view?usp=sharing
Here is the video for fixing issue when the user received the new message.
This video is produced from my proposal code.
On the other hand, I've tested with Mark as Unread.
But looks like it is caused by another reason. I will catch it up and update you.

It's really strange, I was able to repro this in the morning (on android and ios both) but I can't repro it anymore.
@kbecciv, can you test this again?

It's really strange, I was able to repro this in the morning (on android and ios both) but I can't repro it anymore. @kbecciv, can you test this again?

It's able to be reproduced if you go through Report page > back to LHN > send message from another device  > minimize > open.
If user entered into Report page,  unsubscribeVisibilityListener listener will be created in the componentDidMount.
But it will not be unsubscribed when go back to LHN page as I mentioned.
Because of it, issue would be produced

Still no luck.
https://user-images.githubusercontent.com/83179295/201019649-54e31de1-c7d7-42b0-9553-001134cadfd5.mp4

Also I am facing another issue. Chat isn't getting bold when receiving message while I am at LHN

@thesahindia
Are you referring, you got another issue if you use my above change? 🤔

@thesahindia Are you referring, you got another issue if you use my above proposal code? 🤔

No, I am seeing this issue on production as well.

@thesahindia
One question.
Do you think Onyx data can be updated by logging into the console?
I thought all of the onyx data is changed by logging: [Onyx] set() called for key: ${key}.
But I've noticed that onyx data is changed without logging.
Who can give me a full answer to my question regarding Onyx?

But is not working for Mark as Unread.

I was only able to repro this issue by using Mark as Unread. I think the solution should work for Mark as Unread case too.

Who can give me a full answer to my question regarding Onyx?

You can ask Onyx related questions at #expensify-open-source

was only able to repro this issue by using Mark as Unread. I think the solution should work for Mark as Unread case too.

Yes, I agree with it and still debugging for it 😓

You can ask Onyx related questions at #expensify-open-source

Thank you

@thesahindia
Which channel do I need to use to ask the API command problem?
Need to ask in the #expensify-open-source channel through slack?

Need to ask in the #expensify-open-source channel through slack?

Yep. If you haven't already joined the channel, you can request by sending an email at<EMAIL_ADDRESS>

Looks like we had a lot of great discussion yesterday! @railway17 and @thesahindia - think we can get a working proposal today? There's $1000 in it for ya! 😉

Looks like we had a lot of great discussion yesterday! @railway17 and @thesahindia - think we can get a working proposal today? There's $1000 in it for ya! 😉

😉
It's almost fully investigated.
I think I can submit updated proposal today

Any update @railway17 @thesahindia?

Issue not reproducible during KI retests. (First week)

Not overdue.

Oh fun another one of these 😅
If I had to guess I'd say the chat is being marked as read in a call to OpenReport that is marking the report to "read"
Debugging the values of getIsReportFullyVisible() would be a good place to start
https://github.com/Expensify/App/blob/00d7c619cc409b96e6dddca5cc0ad406f0785331/src/pages/home/report/ReportActionsView.js#L253-L256

also I think, generally speaking, if we identify a new bug while investigating another one it's a good practice to report a new issue unless you are certain they have the same root cause (which can be difficult to see if we don't yet have a root cause).

@marcaaron if this is no longer reproducible (see here) do we close it out?


Automatically remove Sileo Installer

Added some code to automatically remove Sileo Installer. Note that I didn't test this, but it should work as intended (someone should still test it though). Let me know if anything goes wrong if/when you test it.

Thanks Ultra :) Merged.


Fix: xreadgroup command

The current version does not accept message IDs in its input and subsequently does not write them out as part of the command - but that is a requirement for the xreadgroup command.
It's still possible with the current API, but feels like more of a hack then a feature. You could pass in []string{ "streamA", "streamB", ">", ">" } - but the field is "streams" so it seems odd to pass it IDs as well. Whereas a map for the streams field makes sense because it maps from the stream to the message ID.

It indeed feels like a hack, but it was done deliberately because maps are unordered. I think we should not merge this.


javascript.json syntax error

Missing comma after every object

Duplicate object key:

snippet-URLJoin
snippet-sortedLastIndex

Invalid escape character in string:

snippet-yesNo
snippet-untildify
snippet-URLJoin
snippet-median
snippet-escapeRegExp
snippet-escapeHTML

@lupu60 Maybe, I fixed from PR: #4. Check again.
Thank you.


[hiera] - make hiera usable with databases configuration

Hi !
Just to be able to configure db with hiera.
I didn't know where to put the create_resources() call, so i putted it in init.pp.
Let me know if you think that's a bad idea.
Cheers,

@gpenaud this looks good, if it's still on your radar, would you be able to rebase, add the new param to the README and add a unit test?

closing due to no progress since a longer time.


Adding an empty Directory.builds.props, Directory.build.targets, etc. crashes VS 2017 RC

I was in this repo (it doesn't really matter): https://github.com/MiniProfiler/dotnet/tree/261d3d1bdaf4047d59f0a1eba4b3b779f26a6e84
Repro steps:

Open MiniProfiler.sln in Visual Studio 2017 RC
Open the <repo>\samples\Samples.Mvc5 folder in Visual Studio Code (any editor)
Create an Directory.build.props file that is empty (VS Code will save it immediately)

Visual Studio 2017 will crash instantly, as soon as MSBuild sees the file. The exception underneath is:

Microsoft.Build.Exceptions.InvalidProjectFileException: 'The project file could not be loaded. Root element is missing.'
Inner Exception:
XmlException: Root element is missing.

Tagging @lifengl @jviau @adrianvmsft. MSBuild is throwing the exception because it indeed an invalid project. We should probably handle this in CPS.

@lifengl What do you reckon we should do in this situation? Unload the project?

We should report an error when we load it, certainly should not crash.
Sent from my phone
On Jan 16, 2017, at 2:45 PM, David Kean <notifications<EMAIL_ADDRESS>wrote:
@lifenglhttps://github.com/lifengl What do you reckon we should do in this situation? Unload the project?

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/roslyn-project-system/issues/1199#issuecomment-272979132, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALGWwgILndJZnGWOO03-GjWryX2Sv82kks5rS_MWgaJpZM4LlC7A.

The project is already loaded, so what code path is causing this to be loaded/pick up in this case?

Any evaluation or invalid project file errors we display as a load failure to user. If this is crashing VS then this is either bubbling out of msbuild as the wrong exception type, or it is occurring before we expect it and need to handle that.

They are imported in common props and targets.
https://github.com/Microsoft/msbuild/pull/751

This has been fixed and resolved in 15.6. It should be available in Preview 2.


AssertionError: a novel class has no pixels - Label id mismatch

Hi,
I execute the bash script run.py for task 7mc on coco dataset.
When training on step 1 with ns=2, I get an assertion error on line 176 in methods/imprinting.py: AssertionError: Error, a novel class has no pixels.
Here are some of my inspections:

In imprinting.py script, the new_classes is [74] which corresponds to the total number of classes in the previous step;
However, the unique labels in the sample are [18, 19, 255] for is=2, [19, 255] for is=3. In other words, we only select image with label 19 on coco-7mc-step1;
In task.py, the actual new class id is 21;

So I doubt that the issue could be the way we filter images by labels in dataset/dataset.py.
I would appreciate if you provide some help. Thank you in advance.

Hi! Can you please check the instructions that I've posted today?
I think they will solve the issue! Sorry if I uploaded them late!

Hi, yes the label mismatch is solved. Thanks.


Use new parameters for Read more wikidata descriptions.

https://phabricator.wikimedia.org/T193076
( The Read more transform is only used by iOS. )

@montehurd not affecting android, as expected! also patch lgtm!

@niedzielski I added Add some tests for page lib footer bits to my todo. It's a fair sized list (~25 items) but it's on it now :) Would it be ok to merge this in the mean time?

@montehurd it's fine with me I'm just encouraging you to test somewhere!


Can we show gitlab CD/CD or Pipeline in hygieia like jenkins and bamboo?

Hello ,
Most of our projects repo are located in gitlab and use gitlab ci/cd for our deployment.
So, I tried hygieia and it can only show the push/commit/issues/pull . Not include CI/CD process of gitlab pipeline. All I want to know is , is there any plan to add more collector to hygieia for gitlab.
BTW, Thanks for this awesome project.

Currently we don't have a collector for Gitlab CICD, However if you understand the webhook for Gitlab CICD you may enhance the hygieia api to translate the payload from gitlab and convert to the canonical model support for deployment widget. We have a reference implementation for Github and Sonar webhook and a custom implementation for builds, hence we need a way to parse the payload from gitlab and invoke the leverage the existing service methods to achieve this functionality. Hope this helps.


BTLE Scan under Android not working using BleCentral

IF YOU DON'T ANSWER THIS TEMPLATE - THE BOT WILL AUTOMATICALLY CLOSE YOUR ISSUE!
Please check all of the platforms you are having the issue on (if platform is not listed, it is not supported)

[ ] iOS
[X] Android
[ ] UWP
[ ] watchOS
[ ] tvOS
[ ] Tizen
[ ] .NET Standard

Feature Library

[ ] Core
[X] BluetoothLE
[ ] Beacons
[ ] Locations
[ ] HTTP Transfers
[ ] Notifications
[ ] Sensors

Version of Library
<IP_ADDRESS>3-beta
Version of OS(s) listed above with issue
Android 10
Steps to Reproduce
1.Install and open Shiny app built from shinysamples.
2.Click Central, if Bluetooth is off then click Toggle Adapter and it turns on.
3.Click Scan, and no advertising peripherals appear here.
Expected Behavior
In the iOS build of both shinyexamples and my standalone app that only uses BleCentral, I am able to observe BLE peripherals in the scan, connect, then perform GATT operations such as read, write, and notify.  Shiny.BluetoothLE under iOS has been working almost perfectly to collect > 8 hours of sensor data.
On Android, the ACR Bluetooth example built from the samples folder of the previous bluetoothle project is scanning correctly, so I know it's at least possible for BTLE to work under Android.  Since Shiny is the future, I'd rather use Shiny.BluetoothLE than the old lib.
Actual Behavior
I have tried several of the -beta versions of the Shiny.BluetoothLE from nuget, and I have not been able to scan successfully under Android.  I have tried the scan under both shinyexamples and my standalone app.  I have a few devices that have older Android revisions than 10, thinking about testing on those too.
A general question, for the use case of BleCentral only, is there a revision of Shiny.BluetoothLE that was known to work or is this still being developed out?  I am interfacing many sensors this year so happy to contribute test time.

I may have partially solved my problem, though don't understand the reason since I'm new to these BTLE frameworks.  In my standalone app that has only services.UseBleCentral(), I am now able to scan after adding these lines to MainActivity.cs:
From:
LoadApplication(new App());
To:
LoadApplication(new App());
this.RequestPermissions(new[]
{
Manifest.Permission.AccessCoarseLocation,
Manifest.Permission.BluetoothPrivileged
}, 0);
I copied the extra lines out of Plugin.BluetoothLE.
In addition to scan, GATT notifications are starting to work as well.

Good afternoon,
After I added the permissions request in my MainActivity, Shiny.BluetoothLE appears to be working well under Android in my standalone app (not using shinysamples).  The importance of the permissions should be noted in documentation when using Shiny.BluetoothLE as standalone.
Setting to closed since both iOS and Android appear to be just fine.


buildTree() not updating lft & rgt

On frontend I use this plugin https://github.com/dbushell/Nestable
My method code

        $data = \Request::all();

        $categories = json_decode($data['sort_category'],true);

        Category::rebuild(true);

        Category::buildTree($categories);

        Category::rebuild(true);

        return redirect()->route('admin.categories.sortorder');

So copied from dd($categories) I have structure like this;
array:4 [▼
  0 => array:2 [▼
    "id" => 1
    "children" => array:5 [▶]
  ]
  1 => array:2 [▼
    "id" => 2
    "children" => array:5 [▶]
  ]
  2 => array:2 [▼
    "id" => 3
    "children" => array:1 [▼
      0 => array:2 [▼
        "id" => 15
        "children" => array:1 [▼
          0 => array:1 [▼
            "id" => 16
          ]
        ]
      ]
    ]
  ]
  3 => array:1 [▼
    "id" => 17
  ]
]

Then I tried to move category id: 17 to the top of category id:3
When I dd($categories) the array like this;
array:4 [▼
  0 => array:2 [▼
    "id" => 1
    "children" => array:5 [▶]
  ]
  1 => array:2 [▼
    "id" => 2
    "children" => array:5 [▶]
  ]
 2 => array:1 [▼
    "id" => 17
  ]
  3 => array:2 [▼
    "id" => 3
    "children" => array:1 [▼
      0 => array:2 [▼
        "id" => 15
        "children" => array:1 [▼
          0 => array:1 [▼
            "id" => 16
          ]
        ]
      ]
    ]
  ]

]

I update the Categories with Category:buildTree($myarray), when I see the database the lft & rgt didnt change, even I use Category::rebuild(true)
Is this bug ?
Can anyone here help me to fix this?
Thanks

Same here. Have you found a solution ?

I've forked this repo and am looking to continue it's development. If you'd like me to look in to this, please create a failing unit test and submit a pull request here: https://github.com/gazsp/baum


Add .nvmrc

I had to nvm-install Node.js v20.17.0 before I could properly build the plugin. Please consider adding an .nvmrc file to specify the expected node version.

Fixed in this commit. 👌


Rename any instances of the func (ax *Nexodus) to func (nx *Nexodus)

Following the rename of the project a while back, we did not rename the Nexodus receiver from ax to nx everywhere.
This is a great first issue that only involves renaming the receiver everywhere in the internal/nexodus/ path. Ping @nerdalert if you have any questions on building/testing, ty!

I agree with the change. I suggest submitting PRs to do this in smaller chunks. One PR may be painful to rebase or for others to rebase on top of. It would also make review easier.
$ git grep ax internal/ | wc -l
588

Hey i'd like to this.
I'll send PRs file by file so that its done in smaller chunks.

I believe this is done


Chore/ddw 48 Update small dependencies

This PR introduces small dependencies update.

Looks great, i will do the upgrade to webpack 2 now …


Add the following fields to the visual wells/areas:

Module: Design a Report in Power BI Desktop, Part 1
Lab/Demo: 00
Task: 1
Step: 33
Description of issue
Add the following fields to the visual wells/areas:
Repro steps: 33
1.Add the following fields to the visual wells/areas:
Axis: Product | Category it should be x-Axis
Value: Sales | Quantity ( it should y Axis)

Duplicate issue, already included in November 2022 updates.


RCE via Read Collections Function

Hi Cockpit Team,
I found that i Can Execute get RCE via php code injection at read rules.
This attack can be executed with authentication  and without authentication
Steps [auth]

Go to dashboard then Click on Create Collection

enter any dummy data

at the permission part enable the read rule and modify it with the php payload.

Go to the Collections

Click On the Created Collection

pass a value to the missing parameter

Note That the file Created At storage/collections/rules

[without auth]
You Can Access it without authentication at /storage/collections/rules/RCE.read.php?cmd=id

Yes, you can write php code, and this should only be done by admins. You should only give admins the right to create|edit collections.I'll try to find a solution for the direct un-authenticated access

BTW how can u let the admin execute a PHP code !! if I add this CMS to my website then I gave an admin access to someone he can easily access my server, files, etc.... !!

Why would you give someone admin rights whom you don't trust? he can also delete your account then and take over the cms etc. Just don't give anyone admin rights 🤷♂

I'm not talking as my side it's an example.
Sometimes The Website owner doesn't change the default Credentials for the CMS, so if the attacker tries it he will log in as admin then execute the PHP code. 🤷♂🤷♂

Can u just explain to me why u allow the admin to execute PHP code?

This meant for configuring complex content access rules (see comment: https://github.com/agentejo/cockpit/issues/675#issuecomment-367120881)


pod lib lint: module 'RealmSwift' was created for incompatible target arm64-apple-ios9.0

!!! MANDATORY TO FILL OUT !!!

Goals

I'm using Realm and Realm swift as vendored framework in my library. Executing Cocoapods pod lib lint failt with error:
ERROR | [iOS] xcodebuild:  error: module 'RealmSwift' was created for incompatible target arm64-apple-ios9.0: Frameworks/RealmSwift.framework/Modules/RealmSwift.swiftmodule/arm64.swiftmodule
Expected Results

pod lib lin executes successfully
Actual Results

Error like above
Steps for others to Reproduce

Simple project with RealmSwift import and distributed as vendored framework
Code Sample

Version of Realm and Tooling

Realm framework version: 5.4.2
Realm Object Server version: ?
Xcode version: Xcode 12 GM
iOS/OSX version: ?
Dependency manager + version: Cocoapods 1.9.3

Please try updating cocoapods to 1.10.rc1 and pointing it at the lm/pod-xcframework branch. Note that this requires setting the branch for both Realm and RealmSwift even if you only use RealmSwift directly:
pod 'Realm', git: 'https://github.com/realm/realm-cocoa.git', branch: 'lm/pod-xcframework', submodules: true
pod 'RealmSwift', git: 'https://github.com/realm/realm-cocoa.git', branch: 'lm/pod-xcframework', submodules: true

Hello, there is no way that I can use that with pod lib lint as in .podspec you can only define name and version of the dependency. I will try to use that directly in my app and let you know.

I have the same problem except I use the Realm.Framework and RealmSwift.Framework downloaded directly.
Both version 5.4.3 & 5.4.2's iOS Xcode 12 build have the same error on Xcode 12 GM

@adamszeremeta

there is no way that I can use that with pod lib lint as in .podspec you can only define name and version of the dependency

You could try excluding the arm64 simulator in podspec file with:
s.pod_target_xcconfig = { 'EXCLUDED_ARCHS[sdk=iphonesimulator*]' => 'arm64' }

Which should write this into the .xconfig.

Archives, building, pod installing, etc. worked. But specifically pod lib lint still failed. Now lib lint applied my test framework that excludes arm64 simulators to a target called arm63-apple-ios-simulator:
 /var/folders/4j/b9ld7rcn5mb_wn84z9hlh1m80000gn/T/CocoaPods-Lint-20200928-30767-wl561z-framework4/App/main.swift:1:8: error: could not find module 'framework0' for target 'arm64-apple-ios-simulator'; found: i386, x86_64-apple-ios-simulator, x86_64, i386-apple-ios-simulator
    import framework0

I couldn't find a way for pod lib lint to skip just arm64 simulators, but maybe pod lib lint --skip-import-validations --verbose is an alternative. It worked for me, but I recommend using verbose because it wasn't immediately clear if I was skipping things that shouldn't be skipped.

Firstly,I try add  my podspec file with:
s.pod_target_xcconfig = { 'EXCLUDED_ARCHS[sdk=iphonesimulator*]' => 'arm64' }
and execute pod lib lint error:
Ld /Users/xxxx/Library/Developer/Xcode/DerivedData/App-brhsnyjzdntvcufqhukpqdhzdqzh/Build/Intermediates.noindex/Pods.build/Release-iphonesimulator/Realm.build/Objects-normal/arm64/Binary/Realm normal arm64
Secondly,I update realm version:
dependency 'Realm', '5.4.3' and execute pod lib lint with same error

I have same problem with carthage.

@ericjordanmossman  i add   s.pod_target_xcconfig = { 'VALID_ARCHS' => 'x86_64 armv7 arm64' }
in my podsepc  execute pod lib lint sucess

@ericjordanmossman i add  s.pod_target_xcconfig = { 'VALID_ARCHS' => 'x86_64 armv7 arm64' }
in my podsepc execute pod lib lint sucess

I do the same but no luck.

I have used Realm version 4.3.2 in my pods and it is working now in Xcode 12.1

Using version 5.5.0 seems to work however 10.1.0 throws the error Framework not found realm-sync

@bstillitano Please install Cocoapods 1.10 to resolve the Framework not found realm-sync error.

Just Confirming that this did infact fix the issue. Simply running sudo gem install cocoapods and updating fixed it.

Closing as this is no longer an issue in Realm Cocoa v10.1.1

@ericjordanmossman 我 s.pod_target_xcconfig = { 'VALID_ARCHS' => 'x86_64 armv7 arm64' }
在我的 podsepc 中添加 执行pod lib lint成功

我在使用s.pod_target_xcconfig = { 'EXCLUDED_ARCHS[sdk=iphonesimulator*]' => 'arm64' }同样报错,改成 s.pod_target_xcconfig = { 'VALID_ARCHS' => 'x86_64 armv7 arm64' }可以成功校验通过


Correcting the Pranav Goel Image size to maintain same size across the file.

Description
Correcting the Pranav Goel Image size to maintain same size across the file.
Attach Screenshot

Note 2 code reviewer approval needed. Approach in twitter group & discord channel.

Please don't create these PR to fix images right now. There is already an PR #553 to fix all the images. These PRs are simply create unnecessary merge conflicts for that PR. Once that PR is merged, you can check your individual images and make any changes, if needed. Thanks.

Please don't create these PR to fix images right now. There is already an PR #553 to fix all the images. These PRs are simply creating unnecessary merge conflicts for that PR. Once that PR is merged, you can check your individual images and make any changes, if needed. Thanks.

I have this PR before that one 😅
but will be closing this now


Don't VACUUM SQLite databases on startup with online delete enabled.

Online delete does not reduce the size of ledger.db and transaction.db SQLite
databases. Having rippled VACUUM these on process startup was intended to
compact these databases so that space would be reclaimed. However, this
causes a delay in startup time. The larger the databases, the longer the delay.
What should happen instead is that the user should be provided with a shell
script that does a VACUUM on these databases that the user specifically
calls when they want to shrink these files. That could be in a startup script,
or as simple as giving them a couple of command-line arguments to call.
@BobWay Please comment on this if you have questions, I think this issue will be
noticeable to partners as they restart rippled for upgrades and such.

I think it is much better for rippled to startup fast on reboot.
Provided that the SQLite databases don't grow like a wildfire, having a separate vacuum maintenance script should work well. We can integrate that call into our standard init script on the delivery VMs.

Here is what should happen if a user wants to manually compact these data files:

shut down rippled
As a user that can modify rippled data files:
sqlite3 </rippled/database/directory>/ledger.db 'VACUUM;'
sqlite3 </rippled/database/directory>/transaction.db 'VACUUM;'

Where </rippled/database/directory> is the rippled.cfg section [database_path]
Also, the sqlite3 command needs to be installed. On ubuntu, for example, "sudo apt-get install sqlite3"
3) start rippled
This activity only should be done if the user wishes to spend the time compacting those data files. The amount of space being reclaimed is unknown, but the activity ensures that these files will be the smallest possible for the data included in them. More information about SQLite VACUUM is here: https://www.sqlite.org/lang_vacuum.html

After reading the NuDB documentation, a similar task (though probably moving a lot more data) seems to be possible there, to remove "spilled" buckets and re-add them along with existing buckets to a new database, throwing the old one away. This could also be part of a database maintenance script.

@MarkusTeufelberger The spilled buckets take up about half of one percent, there's not really much to be gained by getting rid of them and also consider that you have to have at least enough space for double the size of the database to rewrite the data file. Also note, when you use the rocksdb to nudb import tool there will be very few spilled buckets if at all, because it creates all the buckets needed right up front instead of splitting them as it goes (this accounts for part of the speedup).

Yeah, it's not mainly about my specific use case, more of a general database maintenance thing. If people are already taking their server offline for longer periods (I just tried vacuuming a ledger.db on an SSD - this takes surprisingly long and created a huge WAL file that is then being written back to the db file afterwards - all in all it saved about 4% file size though), they might be happy if they can recover a few unused GB on the way for free.
Maybe it might be possible to run this test, then ask the user if (s)he wants to bother recovering about X GB of NuDB space with the caveat that it would take quite long and needs 2 times the current database size? Out of scope for this pull request though, this should probably go on JIRA.

The only real case where you'd want to vacuum the databases is if you have more history than you want when you first start using online delete or if you decide to reduce your rotation interval.

@MarkusTeufelberger
https://ripplelabs.atlassian.net/browse/RIPD-780

:+1:
Obviously correct. You can always vacuum the databases with the sqlite command line tool. If anybody really wants, we can add a unit test, config option, or command line switch to do it.

@MarkusTeufelberger Thanks for doing the test that found 4% savings in space from VACUUM. Confirms that this step should be removed from rippled.

You'll find a lot of illusory "savings" from VACUUM if you do it right after a rotate delete. But it will just mean that the file has to be extended through the next cycle as the data size grows again. It's better to just leave it at its maximum size rather than shrinking and growing it repeatedly. The exceptions would be when you first enable online delete (assuming you have lots of history when you do) or when you reduce your online delete interval (again, assuming you have lots of history when you do).

By the way, I mostly did it just because of the mental image of a tiny wall-e style robot vaccuming a database... but I recommend NOT running this on a transaction.db file ever (unless REALLY needed) - from current results so far it looks like this is going to take on the order of several days to complete (yes, on SSD and modern hardware).
Running a few minutes for 4% gains... well, if you just need to fit that single file on that disk, it might be worth it. If you have Google fiber speeds, it might be faster to just redownload the entire transaction history than to clean it up though.

So shall we close this then?

So what are we doing with this??

Tom, please +1 this if you have no problems with it.

Well, I don't know if I don't have any problems with it.  :-)
Markus' last comment sort of convinced me that it wasn't worth doing.  Am I missing something?
But actually - there's no reason not to remove this.  Costs nothing, tiny but non-zero gain.  :+1:


Dynamic Versioning and Deployments

Describe
Been thinking a lot about managing deployments, and think there is a way to publish schema using ENS to help track both releases (e.g. package manifests as content-address URIs) and deployments (leveraging ENS's ability to describe the ABI and addresses for the deployment on multiple chains)
Specification
In your ape-config.yaml, add support for an object {"ens": "<ENS Record Name>"} inside of version package metadata key (similar to how setuptools-scm configuration works w/ dynamic)
If that key is provided (and the ape-ens plugin is installed) check what the value of the text record "latest" (should this become an EIP to namespace? e.g. eipXXXX:version instead?) to find a string value (which should adhere to Semantic Versioning) for the latest version of that package. That record should correspond with a subname (releative to the given ens record where "latest" was found) in RDNS-like ordering (e.g. "vX.Y.Z" would be Z.Y.vX.`)
The entry at that given location would then contain a text record "manifest" (or eipXXXX:manifest?) containing a public location (or locations?) where the fully compiled manifest for that version is published, as well as a text record "types" (or eipXXXX:types?) which is an array of strings for all the types available in the compiled manifest. For each string value in "types", there should be an additional subname for it (e.g. <Contract Type Name>.Z.Y.vX.<ENS Record Name>) at that record which contains commonly useful records like ABI (there is already direct support for this within ENS today), and then the address of each deployment on whatever chain it has been deployed too (ENS also already has support for this today)

Similarly, later on we can allow specifying a dependency type ens: "<ENS Record Name>" in your project's dependencies that can be used to look up already published manifests in a similar way, with an optional version key to specify a specific version (with "latest" being the default)
Dependencies
Many dependencies here, but most importantly:

[ ] ability to create a publish-able manifest (this may require v4 revision EIP, or a new iteration on the concept leveraging a filesystem-like structure instead of JSON)
[ ] updates to ape-ens to support subname and text record lookups
[ ] leveraging chain-specific address lookups via ape-ens converter
[ ] adding support for this via ape-ens converter (something like "<Contract Type Name>@<ENS Record Name>[:latest]"?)
[ ] writing this all up as an EIP and/or ENSIP and going through that process

related: https://github.com/ApeWorX/ape/issues/365


luci-base: use correct regex for time validation

Before this change, values "further in time" than 23:59:59 was allowed, such as 24:00:00, 23:60:00 and 23:00:60
First reported in https://github.com/openwrt/openwrt/issues/14756 and https://forum.openwrt.org/t/openwrt-23-05-2-service-release/177709/303

After fix:

[X] This PR is not from my main or master branch :poop:, but a separate branch :white_check_mark:
[X] Each commit has a valid :black_nib: Signed-off-by<EMAIL_ADDRESS>row (via git commit --signoff)
[X] Each commit and PR title has a valid :memo: <package name>: title first line subject for packages
[ ] Incremented :up: any PKG_VERSION in the Makefile (package doesn't have one, not applicable)
[X] Tested on: (x86/generic, OpenWrt 23.05.2, Chrome and Firefox) :white_check_mark:
[X] ( Preferred ) Mention: @ the original code author for feedback @jow-
[X] ( Preferred ) Screenshot or mp4 of changes:
[ ] ( Optional ) Closes: e.g. openwrt/luci#issue-number
[ ] ( Optional ) Depends on: e.g. openwrt/packages#pr-number in sister repo
[X] Description: (describe the changes proposed in this PR)

Your fix is wrong.
It disables entering e.g. 18:00:00, as the second digit in first number can only be 0-3:
^[0-2][0-3]
That part of the regex needs further tweaking.
Likely have (0-1)(0-9) or 2(0-3) or something similar for that number.

Your fix is wrong. It disables entering e.g. 18:00:00, as the second digit in first number can only be 0-3: ^[0-2][0-3]
That part of the regex needs further tweaking.
Likely have (0-1)(0-9) or 2(0-3) or something similar for that number.

Ah of course, thanks, totally missed that case. Will update it.

I added a capture group for the hour parameter, so now it allows either 00-19 or 20-23 but no values outside of that range
Also; regex is hard.

I just discovered that the capture groups doesn't work in Lua since it's actually pattern matching, not regular expressions, being used in the match function. The only place the timehhmmss validator is used today is in luci-app-firewall and doesn't rely on luci-compat, so I've dropped my change for datatypes.lua since new apps won't be developed in Lua anyway. I don't think it's worth spending time on getting it to work for the Lua code.

Before this change, values "further in time" than 23:59:59 was allowed, such as 24:00:00, 23:60:00 and 23:00:60

Not only can regex be difficult but time is as well. So I’m going to mention a few frequently encountered problems as we humans attempt to match civilian time with solar time:

Leap seconds make …:60 happen. I’ve seen them. They’re real.
Leap minutes are theoretically possible making …:60:.. happen.
During DST changes, clocks in some time zones can contain 25 hours. So the above mentioned “further in time” values are actually… correct. Poke around in some tzinfo files and their tools and you’ll quickly understand what I mean.

Yes they’re seldom encountered but they’re necessary.
Would you like some linked references or is the above explanation sufficient?
Time keeping is not a boring subject. 🤓

regex nitpick - will JS handle a non-capturing group? (?:...) Just code-golf at this point.

Before this change, values "further in time" than 23:59:59 was allowed, such as 24:00:00, 23:60:00 and 23:00:60

Not only can regex be difficult but time is as well. So I’m going to mention a few frequently encountered problems as we humans attempt to match civilian time with solar time:
* Leap seconds make …:60 happen. I’ve seen them. They’re real.

* Leap minutes are theoretically possible making …:60:.. happen.

* During DST changes, clocks in some time zones can contain 25 hours. So the above mentioned “further in time” values are actually… correct. Poke around in some tzinfo files and their tools and you’ll quickly understand what I mean.

Yes they’re seldom encountered but they’re necessary.
Would you like some linked references or is the above explanation sufficient?
Time keeping is not a boring subject. 🤓

I'm actually aware of all of this, however since the timehhmmss validator doesn't contain any more information other than time (we would need date and time zones as input to the validator to accurately determine if for example the leap seconds are valid) this isn't possible to do right now unless the it's rewritten.

we would need date and time zones as input to the validator to accurately determine if for example the leap seconds are valid

Disregard this complexity. We just need to 'generally' account for those possibilities - 'good enough'. Trying to cover every eventuality can make you go insane.
* tzh_typecnt ttinfo entries, each defined as follows:

              struct ttinfo {
                   int32_t        tt_utoff;
                   unsigned char  tt_isdst;
                   unsigned char  tt_desigidx;
              };

         ... Also, in
         realistic applications tt_utoff is in the range [-89999, 93599]
         (i.e., more than -25 hours and less than 26 hours); this allows
         easy support by implementations that already support the POSIX-
         required range [-24:59:59, 25:59:59].

and
       When a positive leap second occurs, readers should append an
       extra second to the local minute containing the second just
       before the leap second.  If this occurs when the UTC offset is
       not a multiple of 60 seconds, the leap second occurs earlier than
       the last second of the local minute and the minute's remaining
       local seconds are numbered through 60 instead of the usual 59;
       the UTC offset is unaffected.

So if an event is scheduled to occur on a (just before) midnight leapsecond, but we cannot enter it, it will never be encountered to be triggered.

Let's not overly complicate things for really rare corner-case occurrences.
Good enough validation is the goal.

@systemcrash youre right, seconds is 00-60
ie 0..23:0..59:0..60
root@suse:~# nft -c -f -
table inet test {
  chain test {
    meta hour "23:59:60"
}
}^D
root@suse:~# nft -c -f -
table inet test {
  chain test {
    meta hour "23:60:59"
}
}^D
/dev/stdin:3:15-24: Error: wrong time format
    meta hour "23:60:59"
              ^^^^^^^^^^
root@suse:~# nft -c -f -
table inet test {
  chain test {
    meta hour "23:58:60"
}
}^D
root@suse:~#

24:00:00 is invalid too

@systemcrash youre right, seconds is 00-60 ie 0..23:0..59:0..60
root@suse:~# nft -c -f -
table inet test {
  chain test {
    meta hour "23:59:60"
}
}^D
root@suse:~# nft -c -f -
table inet test {
  chain test {
    meta hour "23:60:59"
}
}^D
/dev/stdin:3:15-24: Error: wrong time format
    meta hour "23:60:59"
              ^^^^^^^^^^
root@suse:~# nft -c -f -
table inet test {
  chain test {
    meta hour "23:58:60"
}
}^D
root@suse:~#

24:00:00 is invalid too

Buggy OSes :D
OK - we'll go with just leap seconds for now.

Now frontend validation 1:1 reflects backend validation, including leap second, no chance planting invalid automatically disabled rule.

Nicely done. Thanks!


Reenable configcheck work to ensure color is respected in non tty sessions

What does this PR do?
This PR re-introduces the work done https://github.com/DataDog/datadog-agent/pull/26993 which got reverted in https://github.com/DataDog/datadog-agent/pull/27382
The PR caused CI in main to to break because one e2e to end was testing an endpoint that got removed in
https://github.com/DataDog/datadog-agent/pull/26993
This PR a part from adding the previous code it:

removes the e2e test endpoint that no longer exists https://github.com/DataDog/datadog-agent/pull/27389/commits/4fcfc0e2e48f858de8ca737af2dec2dabb59395f
Adds unit test for the configcheck output https://github.com/DataDog/datadog-agent/pull/27389/commits/c892016e3637dccdb3a455d0982ea794fc8b2482 and https://github.com/DataDog/datadog-agent/pull/27389/commits/373156e80df799ff36ed297cf0ea3791f0306db7

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Ensure the main agent flare archive includes the config-check.log file with the right information. Ensure the output does not have ANSI color strings

Ensure the cluster agent flare archive includes the config-check.log file with the right information. Ensure the output does not have ANSI color strings

Ensure CLI command cluster-agent configcheck works as expected. The output must include colors

Ensure CLI command cluster-agent configcheck -n works as expected. The output must not include colors

Ensure CLI command cluster-agent configcheck | less works as expected. The output must  not include colors

Ensure CLI command agent configcheck works as expected. The output must include colors

Ensure CLI command agent configcheck | less works as expected. The output must not include colors

Ensure CLI command agent configcheck -n works as expected. The output must not include colors

Ensure CLI command agent snmp walk works as expected.

/merge


'cf help' should show 'unset-env' command

Command
cf help
What occurred
It prints a few things, but not 'unset-env'
What you expected to occur
It should totally print this command, because I forgot how to do it.
CLI Version
cf version 6.28.0+9e024bdbd.2017-06-27

Hi @tjarratt
We intentionally don't display all commands. You probably remember what help looked like a year ago, several pages of text scrolling over the screen and without grep it was impossible to find anything fast.
We addressed that by reducing cf help to only show the most commonly used app developer commands. The rest is available under cf help -a.
I image you would agree that in general set-env is used more than unset-env, just like push is used more than delete and run-task more than terminate-task. That is why we left unset-env out and are able to keep cf help to a concise list of no more than a single page.
Regards,
Dies Koper
CF CLI PM

Ah, I see. It's still a little hard to find, I think I had to run through the following commands in order to find it.
cf help
cf help set-env
cf help --all
cf --help

Would it be possible to maybe add a note to the help page for set-env that unset-env exists ? That would greatly increase the ability for me to remember how to find that command's help.

We can look into that.
But if the first command you ran was cf help, which displays These are commonly used commands. Use 'cf help -a' to see all, with descriptions. at the end, which I assume you missed, would you have picked up anything at the end of cf help set-env?

We have added unset-env to set-env's SEE ALSO in cf CLI 6.29.0.


Fix incorrect target name for Math 99

The diff target file specified after index 20b61df..26433d6 100644 (b/src/test/org/apache/commons/math/util/MathUtilsTest.java) doesn't match the file's header diff target (b/org/apache/commons/math/util/MathUtilsTest.java, causing an error when parsing the patch automatically (e.g., unidiff.
This PR replaces with diff target file with the header's.

Do you need more information to take a decision on this PR?

Hi @Thomsch, as far I can see, the incorrect line in the patch is the first and not the fourth (the one for which you're proposing a fix).  In other words, what's wrong is
diff --git a/src/test/org/apache/commons/math/util/MathUtilsTest.java b/org/apache/commons/math/util/MathUtilsTest.java

which should be
diff --git a/src/test/org/apache/commons/math/util/MathUtilsTest.java b/src/test/org/apache/commons/math/util/MathUtilsTest.java

as the modified file in the original bug-fixing patch is src/test/org/apache/commons/math/util/MathUtilsTest.java and not org/apache/commons/math/util/MathUtilsTest.java
Right?

Hi @jose, you are totally right. Thank you for catching that. I pushed the real fix.
(I'm not sure how I ended up pushing the wrong fix when I fixed it correctly previously locally! 😅)


Mark02 Summary Convergence Trends

DO NOT MERGE
This PR contains notebooks to generate report specific graphics for the investigation of trends of convergence diagostics.

Saved this notebook in another repo.


#1117: Introducing the duplicate-finder maven plugin to catch duplicate resources and classes

@johanhaleby demonstrating how to fail the build on a duplicate class/resource found on the compile or runtime classpath.
CC // @ponziani @azzazzel

I would expect this to fail, as the Maven duplicate-finder-plugin is now catching duplicate classes and resources in the resulting artifacts:
[WARNING] Found duplicate (but equal) classes in [io.rest-assured:json-path:3.3.1-SNAPSHOT, io.rest-assured:rest-assured-common:3.3.1-SNAPSHOT, io.rest-assured:xml-path:3.3.1-SNAPSHOT]:
[WARNING]   io.restassured.exception.PathException
[WARNING]   io.restassured.internal.assertion.AssertParameter
[WARNING]   io.restassured.internal.assertion.Assertion
[WARNING]   io.restassured.internal.assertion.AssertionSupport
[WARNING]   io.restassured.internal.assertion.EndToEndQuoteFragmentEscaper
[WARNING]   io.restassured.internal.assertion.GetAtPathFragmentEscaper
[WARNING]   io.restassured.internal.assertion.HyphenQuoteFragmentEscaper
[WARNING]   io.restassured.internal.assertion.PathFragmentEscaper
[WARNING]   io.restassured.internal.classpath.ClassPathResolver
[WARNING]   io.restassured.internal.mapper.ObjectDeserializationContextImpl
[WARNING]   io.restassured.internal.path.ObjectConverter
[WARNING]   io.restassured.mapper.DataToDeserialize
[WARNING]   io.restassured.mapper.ObjectDeserializationContext
[WARNING]   io.restassured.mapper.TypeRef
[WARNING]   io.restassured.mapper.factory.ObjectMapperFactory
[WARNING]   io.restassured.mapper.resolver.ObjectMapperResolver
[WARNING] Found duplicate classes/resources in compile classpath.
[WARNING] Found duplicate (but equal) classes in [io.rest-assured:json-path:3.3.1-SNAPSHOT, io.rest-assured:rest-assured-common:3.3.1-SNAPSHOT, io.rest-assured:xml-path:3.3.1-SNAPSHOT]:
[WARNING]   io.restassured.exception.PathException
[WARNING]   io.restassured.internal.assertion.AssertParameter
[WARNING]   io.restassured.internal.assertion.Assertion
[WARNING]   io.restassured.internal.assertion.AssertionSupport
[WARNING]   io.restassured.internal.assertion.EndToEndQuoteFragmentEscaper
[WARNING]   io.restassured.internal.assertion.GetAtPathFragmentEscaper
[WARNING]   io.restassured.internal.assertion.HyphenQuoteFragmentEscaper
[WARNING]   io.restassured.internal.assertion.PathFragmentEscaper
[WARNING]   io.restassured.internal.classpath.ClassPathResolver
[WARNING]   io.restassured.internal.mapper.ObjectDeserializationContextImpl
[WARNING]   io.restassured.internal.path.ObjectConverter
[WARNING]   io.restassured.mapper.DataToDeserialize
[WARNING]   io.restassured.mapper.ObjectDeserializationContext
[WARNING]   io.restassured.mapper.TypeRef
[WARNING]   io.restassured.mapper.factory.ObjectMapperFactory
[WARNING]   io.restassured.mapper.resolver.ObjectMapperResolver
[WARNING] Found duplicate classes/resources in runtime classpath.
[WARNING] Found duplicate (but equal) classes in [io.rest-assured:json-path:3.3.1-SNAPSHOT, io.rest-assured:rest-assured-common:3.3.1-SNAPSHOT, io.rest-assured:xml-path:3.3.1-SNAPSHOT]:
[WARNING]   io.restassured.exception.PathException
[WARNING]   io.restassured.internal.assertion.AssertParameter
[WARNING]   io.restassured.internal.assertion.Assertion
[WARNING]   io.restassured.internal.assertion.AssertionSupport
[WARNING]   io.restassured.internal.assertion.EndToEndQuoteFragmentEscaper
[WARNING]   io.restassured.internal.assertion.GetAtPathFragmentEscaper
[WARNING]   io.restassured.internal.assertion.HyphenQuoteFragmentEscaper
[WARNING]   io.restassured.internal.assertion.PathFragmentEscaper
[WARNING]   io.restassured.internal.classpath.ClassPathResolver
[WARNING]   io.restassured.internal.mapper.ObjectDeserializationContextImpl
[WARNING]   io.restassured.internal.path.ObjectConverter
[WARNING]   io.restassured.mapper.DataToDeserialize
[WARNING]   io.restassured.mapper.ObjectDeserializationContext
[WARNING]   io.restassured.mapper.TypeRef
[WARNING]   io.restassured.mapper.factory.ObjectMapperFactory
[WARNING]   io.restassured.mapper.resolver.ObjectMapperResolver
[WARNING] Found duplicate classes/resources in test classpath.

@johanhaleby rebased with the latest upstream/master, including the changes in https://github.com/rest-assured/rest-assured/pull/1137 from @azzazzel.

@johanhaleby I think this is ready to go! I cleaned this up and squashed @azzazzel latest changes to get this unblocked. There's a few dependency cleanups I made as well given the failures from the maven duplicate finder plugin.

Thanks a lot for all your help and efforts. Really appreciated.


Match colorize output with Morgan

Morgan only colors the status code in the log message according to code, documentation and quick testing. This PR makes the log output when using the colorize option to match it. IMHO this also makes the output a lot more readable because the current setting of grey on black has really low contrast with many color themes.

Awesome, thanks for contributing! I'm happy with this change since it improves contrast and readability.
The only thing I'm worried about is that this is technically a breaking change. I'd love to merge it soon, but I have some more breaking changes in mind for express-winston v3. I think I'm going to hold off for now, but I may end up changing my mind later just to get this in. 🙂

Would you also consider matching expressFormat with morgan in v3? Currently it matches the morgan dev format but it's missing content length from the end.

@Hilzu Sorry, responding very late.
I'm torn on adding the content length to be on by default with expressFormat. I started writing a longer response but ended up deleting it (that's how torn I am).
I'm good with adding content length to expressFormat, but I'm inclined to hold off on that until a v3 release as well. I know it's kind of pedantic, but I don't really want to mess with the log formats of people's files without a major version bump.

This is a very old PR. What are we doing with this?


Honor file extensions

Check whether Fly honors file extensions if the last filter does not return { extension: ".." }.
For example:
  yield this.source("coffee/src/**/*.coffee")
  .coffee()
  .uglify()
  .target("coffee/dist/")

If uglify does not set the file extension, Fly should use whatever extension set by coffee and so on.

The reduce filter where each of the transformations is applied to the incoming source will use the lastly exported extension. So, in the case above, the result will be .js even if uglify did not specify { ext: ".js" }, because that's fly-coffee exports.


Modal: Expose Modal heading reference

There have been various use cases presented where it would be beneficial for adopters to be able to manually refocus the heading inside the modal. We own the heading and the heading ref completely, so it's currently not impossible with a clean solution. This has come up a few times for multiple teams from Perkins reviews.
Current Workaround:
  useEffect(() => {
    setTimeout(() => {
      const headings = document.querySelectorAll('h1');
      const targetHeading = Array.from(headings).filter(
        heading => heading.innerText === transformDialogTitle(activeDialog)
      )[0];
      setHeadingNode(targetHeading);
    }, 100);
  }, [activeDialog]);

Acceptance Criteria

The Modal component accepts a headingRef prop so we can give adopters better control.
There is an example of this new prop working in both storybook and docs site, where a Modal has a custom heading and it can be programmatically focused
The Drawer component is built on the Modal, so this should also be documented and function for Drawer

Should we explore alternatives to using a specific ref as a prop? Perhaps there's another way to pass the focus on a custom header component.
The biggest concern is how we presently use a tabindex="-1" in our Heading component within the Modal itself and somehow replicating that to a prop(s) somehow.

We can use something like this example for the docs site example of a modal with a custom heading

Testing Notes:

The Modal Header example on codesandbox has an error with multiple missing imports:

It may be helpful to also mention the necessary tabIndex prop in the docs: ```When a reference to the header is needed, use the headerRef prop. When using a custom header, you must also use tabIndex={-1} property.`

The example for This modal has a header has text that says This modal has no header.

@chris-cedrone-cengage Sorry I missed it too, CheckIcon needs to be imported from react-magma-icons, not dom 😢. The rest looks great

Looks good!


Features/migrate dot net 8

The project cli has 3 target framework (.net 6/7/8), but for other projects only 1 target is possible (sometime 6 or 7). It is impossible to build for the 3 targets with the sln.
Moreover, the github action to setup dotnet use only latest version available
To keep consistency and use a LTS version use only 8 version

Thanks for the PR! I really appreciate help here. My main expertise is in C99 or C# libraries. This is the first time I've released a C# console application/tool.
My main priority is that users can install and use the cli tool regardless of whether they have net6/7/8 installed. A lot of StateSmith users don't actually use C# and won't want to upgrade from net6 to net8. If we make the suggested change, will net6 users still be able to use the cli tool?
I was basing my .csproj off the dotnet-script .csproj because that one works with net6/7/8: https://github.com/dotnet-script/dotnet-script/blob/master/src/Dotnet.Script/Dotnet.Script.csproj#L8


Is it possible to dump the "Thumbnails" info of the assets?

i'm try to get asset class name, which is inside Thumbnails->index.

@donghangssn Does the latest version of uasset-rs address your issue? You can test it with cargo run --features commandline-tool dump-thumbnail-info <path-to-uasset>


issue:3709767 PDR Plugin - update high BER analysis

updated isolation analysis according to the new algorithm provided by Eitan. Z
https://redmine.mellanox.com/issues/3709767

bot:retest


Passing arguments while calling exported functions

To call an exported function we need to be able to pass arguments to it. I have a two straw man approaches to implement that.
The first one: pass arguments dynamically. That is, every exported function should have a thunk generated. This thunk reads all arguments from the dynamic list of argument values (a-la wasmi's RuntimeValue) and then call into the original function with the whatever ABI it has. Upon calling the exported function, we check that the argument slice matches the signature and then call we call the thunk with the fixed signature (vmctx, args_ptr). This thunk could also convert and write the return value to the specifc pre-allocated location in the callee. Dumb but easy.
The second one. Generate exported functions with the specific ABI, say system_v. Then introduce an argument wrapper trait.
unsafe trait ArgsWrapper {
  fn signature() → WasmSignature;

  fn call(fn_ptr: usize, vmctx: usize, args: Self);
}

This trait's main purpose is to provide a way to get the signature dynamically and unpack the values for the actual call. We can generate impls for this trait by a macro. Here is an example impl for function with arity 2.
unsafe impl<A: HasWasmType, B: HasWasmType> ArgsWrapper for (A, B) {
  fn signature() → WasmSignature {
    WasmSignature::new(&[A::wasm_type(), B::wasm_type()])
  }
  unsafe fn call(fn_ptr: usize, vmctx: usize, args: (A, B)) {
    let (a1, a2) = args;
    let f = fn_ptr as extern "C" fn(usize, A, B);
    unsafe {
      f(vmctx, a1, a2)
    }
  }
}

Then actual call can be implemented as following:
fn call<A: ArgsWrapper >(func_name: &str, args: A) {
  let wasm_func = self.funcs.get(func_name).unwrap();
  assert_eq!(wasm_func.signature, A::signature());
  unsafe { A::call(wasm_func.ptr, args) }
}

I think this approach can be scaled to handle returning values as well.
@sunfishcode what do you think? Am I missing something, do you have better proposals?

Sorry for the late response here! I've been busy and somehow lost track of this issue.
My general preference would be for the second approach. Cranelift has system_v and windows_fastcall specifically to allow it to implement platform-native calling conventions, and I think it makes sense to take advantage of that here. I like the idea of using a helper function like this to wrap up the unsafe code and the vmctx argument.

I've now implemented an approach where in we use the JIT to generate trampoline functions. We store the argument values onto the stack, the trampoline function loads them and makes the call, and then stores the return values onto the stack. Take a look at trampoline_park.rs for details, or just use InstancePlus and call invoke as desired :-).


System.InvalidOperationException, EUC-JP is not a supported.

Hi, I am facing this error in the DotNET Core version.
{System.InvalidOperationException: The character set provided in ContentType is invalid. Cannot read content as string using an invalid character set. ---> System.ArgumentException: 'EUC-JP' is not a supported encoding name. For information on defining a custom encoding, see the documentation for the Encoding.RegisterProvider method.
Parameter name: name
at System.Globalization.EncodingTable.GetCodePageFromName(String name)
at System.Text.Encoding.GetEncoding(String name)
at System.Net.Http.HttpContent.ReadBufferedContentAsString()
--- End of inner exception stack trace ---
at System.Net.Http.HttpContent.ReadBufferedContentAsString()
at System.Net.Http.HttpContent.<>c.b__36_0(HttpContent s)
at System.Net.Http.HttpContent.d__62`2.MoveNext()}
Here is more information.
.NET Command Line Tools (1.0.0-preview2-1-003177)
Product Information:
Version:            1.0.0-preview2-1-003177
Commit SHA-1 hash:  a2df9c2576
Runtime Environment:
OS Name:     Mac OS X
OS Version:  10.11
OS Platform: Darwin
RID:         osx.10.11-x64
How can I avoid this problem ? Thanks.

Please show me the code to reproduce this. I find it rather unlikely that the problem originates from AngleSharp (if so then I did not use the TextEncoding helper to get the encoding - as this one will fall back to UTF-8 when the encoding is not available).
I see a lot of HttpContent calls in there, apparently ReadBufferedContentAsString is used (AngleSharp does not read the string though, but rather the stream directly).


create ISSUE_TEMPLATE

less people with useless issues

don't hardcode the version, tell them to check package.json in master

why are there html comments in this

they won't show up in markdown

those checkboxes should be visible so we can verify they checked them

additionally there is no point in having things markdown formatted if they will just be commented out

your checkboxes are not valid markdown

Sorry, but we've had some discussions on Discord and decided to make our own one. Your help is still very much appreciated, of course; I took some inspiration from that when making this one – thanks for that!


Split the maximum page cache size up among each parser.

Split the configured page cache size up among the different parsers, one per interface and per direction.
This successfully reduces the memory; we don't have a way to measure what it does to "goodput" or if we lose some packets because we could not buffer enough to see the retransmission.

Whoops, I forgot I was going to update the unit tests today before making this a PR.  :(


Error thrown when trying to promote guest user to admin

When trying to promote guest user without an email to site admin, it throws an error.
To prevent this error, we can define a custom validation check on the schema.

It works, but Zod is throwing a type error. @rmunn will follow up on this with more details.

Type error fixed with commit<PHONE_NUMBER>.


Volunteering section modified

Made the following changes:

Added new images in the experience section folder itself
Added new animations
used cardv for volunteering section as card class was already used in mentorship section
Added comments in CSS file

Issue #80

Volunteering is something, we usually do out of passion. So, having a dedicated view certificate link to prove it seems odd
@anushbhatia will suggest you changes for the animation

Okay, I'll remove that view certificate option.
Also, I commented out the Js part cause the images weren't loading up (or showing ) in pc. So, the same issue persisted even when I tried uploading images via css too.
Moreover, I wasn't able to change animation when the text was uploading from JS part. So, hence tried putting all display part in html.

@smaranjitghose @anushbhatia plz suggest

Don't upload images with the help of the CSS use HTML for that. See use main.js where you store the data for different images and the text. In the HTML just load the data form that file so that if we add a new volunteering experience it automatically gets added.

@raksha009 Resolve the merge conflict


Display of user info should (optionally) include visible apps

🗣 Context
When I was trying to get information about an individual user I could not see what apps were "visible" to them.
💬 Narrative
As a user
I want include visible apps data when fetching user info
So that I can see what apps a user can see
📝 Notes
asc users info does not include a --include-visible-apps option like asc users list does.
✅ Acceptance Criteria
GIVEN I am requesting a user's info
WHEN I include the --include-visible-apps option
THEN the output should include a list of apps visible to the user

Something worth noting is that it seems that if a user has the provisioningAllowed property on their user data they seem to be able to see all apps by default. ie, allAppsVisible will also be true.


Enable writing events to external database

This simple PR introduces ability to route TryLogEvent calls to external database specified in Parameters.EventLogConnectionString. On service startup there is one extra database call to cache external connection string. If connection string value is NULL or not present (normal case) calls are made to the same database as today.

/azp run


Can't change theme in Firefox

in Firefox, try to change the theme. when you change the selection in the dropdown, the modal closes so you can't hit save.
https://share.getcloudapp.com/z8ubOB0Z

This is really looking like a bug in Firefox. The form uses a standard HTML dialog and doesn't have any custom behavior for displaying or closing the modal. Considering it works as expected in Safari, it seems like an issue with Mozilla's implementation.
I tried changing the form by removing method="dialog" and adding formmethod="dialog" to the Cancel button, but it still doesn't work in Firefox (but is fine in Safari).
One fix might be to redo the way form/dialog handling works (with more custom behavior instead of using the dialog element). @progrium what do you think?


Add cfssl issuer

What this PR does / why we need it:
It adds support for a cfssl issuer to allow issuing certs from a cfssl based CA.
Release note:

Add support for CFSSL issuers

/ok-to-test

/ping @munnerz
Please review when you get a chance to. I'd like to get this going and hopefully accepted and merged by the next release 🙂


🍓 feat: migrate from yarn to pnpm

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@5ed4c39). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             main        #3   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?         1
  Lines           ?        49
  Branches        ?         0
========================================
  Hits            ?        49
  Misses          ?         0
  Partials        ?         0

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 5ed4c39...5acc62b. Read the comment docs.


IntentRecognizer not working with au.luis.ai (australiaeast)

Describe the bug
IntentRecognizer will not work with a LUIS app created in australiaeast (https://au.luis.ai). This is true even with a non-Starter_Key resource. Trying to run the C# console sample will return:
CANCELED: ErrorCode=ConnectionFailure and CANCELED: Did you update the subscription info?
I created a dummy LUIS app and resource in westus (https://www.luis.ai) and replaced the keys accordingly to test the hypothesis and the C# console sample worked fine.
To Reproduce
Steps to reproduce the behavior:

Create, train and publish a LUIS app in australiaeast (https://au.luis.ai)
Create and assign a australiaeast resource to the LUIS app
Use the above keys and region in the C# console sample in this repos
Build and run console app will return the error

Expected behavior
Initiates RecognizeOnceAsync() because australiaeast should be a supported region according to docs
Version of the Cognitive Services Speech SDK
Microsoft.CognitiveServices.Speech.csharp <IP_ADDRESS>
Platform, Operating System, and Programming Language

OS: Windows 10 Build 18363
Hardware - x64
Programming language: C#

We've done some service side investigating and determined that there were some transient auth issues a few days ago, but as of now everything is working correctly.
Can you ensure that you're using a Prediction Resource key and not an Authoring Key and try again?
It would also be helpful if you had a session ID and an approximate time of connection for us to look through the service telemetry.

It would've been before or on the same day as when I opened the issue, unfortunately no time stamp logged. Definitely was a Prediction key, not Authoring, but I tried both at that time when the first did not work.
I migrated to westus anyway so I don't have the original Luis resource to test now. Glad to hear it's working now.


ValueError: numpy.ndarray size changed, may indicate binary incompatibility. Expected 96 from C header, got 88 from PyObject

  File "<stdin>", line 1, in <module>
  File "/home/user/Automix/automix/model/classes/track.py", line 281, in __init__
    self.features = featuresGetter(path)  # features dictionnary from the MIR estimators
  File "/home/user/Automix/automix/model/classes/track.py", line 35, in __init__
    self._estimators = self.getExecutionGraph()
  File "/home/user/Automix/automix/model/classes/track.py", line 52, in getExecutionGraph
    from automix.featureExtraction.beats.madmomBeatDetection import MadmomBeatDetection
  File "/home/user/Automix/automix/featureExtraction/beats/__init__.py", line 4, in <module>
    from .madmomBeatDetection import MadmomBeatDetection
  File "/home/user/Automix/automix/featureExtraction/beats/madmomBeatDetection.py", line 5, in <module>
    import madmom
  File "/home/user/.local/lib/python3.8/site-packages/madmom/__init__.py", line 24, in <module>
    from . import audio, evaluation, features, io, ml, models, processors, utils
  File "/home/user/.local/lib/python3.8/site-packages/madmom/audio/__init__.py", line 27, in <module>
    from . import comb_filters, filters, signal, spectrogram, stft
  File "madmom/audio/comb_filters.pyx", line 1, in init madmom.audio.comb_filters

please help! Spent too much time trying to make this work. I have all the dependencies installed. and this is working for me: madmomDrumsEnv/bin/python madmom-0.16.dev0/bin/DrumTranscriptor
not sure what is still missing. numpy version outside of madmomDrumsEnv is 1.20.3 and in madmomDrumsEnv is 1.16.6.
I appreciate any tip to solve this. My environment is python3.8.10

Hej, I will try to help you!
The program needs two different versions of Madmom to work: one for drum transcription, and one for beat detection. This is confusing, but sadly the algorithms for beat and drums estimation are in two different forks of madmom. Because I don't know how you can't install both versions in the same environment with pip, I did the ugly workaround of creating with the installation process the virtual environment "madmomDrumsEnv". This one seems to work for you. Now, you still require the version of madmom for the beat detection which seems to be the one raising the issue:
  File "/home/user/Automix/automix/featureExtraction/beats/madmomBeatDetection.py", line 5, in <module>
    import madmom

This one is not contained in the virtual environment "madmomDrumsEnv". Instead, its location depends on how you installed Automix. Could you check if madmom is correctly installed with:
pip3 install . --editable
The version should be 0.16.1
Good luck!

Hey @MZehren ,
Really appreciate the help!
Yes I have that version
Requirement already satisfied: madmom in /home/user/.local/lib/python3.8/site-packages (from AutoMix==0.1) (0.16.1)

Ok, this is strange. Since the program fails when you load Madmom, I expect that you wouldn't be able to load Mamdom (the one outside of the madmomDrumEnv) from your own code either. Can you replicate the issue somewhere else, or is it only when importing my file?
The software works with both .mp3 and .wav IIRC.

Hey Michael,
I set up another venv and installed madmon again. Its working fine. Thank you ! :)

Great to hear!


Graph rollback

LLMs are fiddly, so we often need to say "hey, try again" to a completion. This could be a feature of graph runner - just retry the previous step N tries or until some validation passes.

Duping this to #342


Nose-cov win-64 bit incompatibility?

Hi guys -
Please excuse the n00bism here, but I can't quite get geopy to install via conda build. Root anaconda installation is 3.4.3, but I have a few environments running 2.7. Machine is Windows 7, 64-bit. I run conda skeleton pypi geopy then conda build geopy and I keep getting the same error. This is the last bit of output:
Installed c:\anaconda3\envs\_build\lib\site-packages\geopy-1.11.0-py3.4.egg
Processing dependencies for geopy==1.11.0
Finished processing dependencies for geopy==1.11.0

C:\Anaconda3\conda-bld\work\geopy-1.11.0>if errorlevel 1 exit 1
found egg: C:\Anaconda3\envs\_build\Lib\site-packages\geopy-1.11.0-py3.4.egg
number of files: 4
Fixing permissions
Fixing permissions
BUILD END: geopy-1.11.0-py34_0
TEST START: geopy-1.11.0-py34_0
Fetching package metadata: ......
Error: No packages found in current win-64 channels matching: nose-cov

Interestingly, installing nose-cov separately via the same process does, in fact, work.
Apologies if this is a simple misconfiguration.

We are about to drop nose in favor of pytest (#266), so I'm closing this.


Scrolling using a custom renderer

Hello,
I have a general question about how to implement scrolling using a Pixi render. I'm scrolling my Pixi game world like this inside my update function:
if (centreDist > 200) {
      // create a vector to translate the view, allowing the user to control view speed
      var direction = MATTER.Vector.normalise(deltaCentre),
        speed = Math.min(10, Math.pow(centreDist - 100, 2) * 0.0002);

      let translate = MATTER.Vector.mult(direction, speed);

      //Move the level container (a Pixi sprite) in the opposite direction to the mouse's movement
      this.levelContainer.x -= translate.x;

This works. But my question now is: how can I shift the MatterJS bodies so that they stay aligned to their positions on the Pixi container (levelContainer)?
I have been studying this example: https://github.com/liabru/matter-js/blob/master/examples/views.js. However, this implements scrolling using the built-in MatterJS Renderer, which I am not using.
Any general advice would be greatly appreciated!
Thank you!

... In answer to my own question, it turned out to be very easy. All I needed to do was to translate all the physics bodies by the value of translate.x, like this:
let bodies = MATTER.Composite.allBodies(
        this.gameModel.physicsEngine.world
      );
      //console.log(bodies);
      bodies.forEach(body => {
        let xOffset = body.position.x - translate.x;
        //MATTER.Body.setPosition(body, { x: xOffset, y: body.position.y });
        MATTER.Body.translate(body, { x: -translate.x, y: 0 });
      });

If there's a better way, please let me know 😄


thumbUrl empty for videos

When uploading videos, the column thumbURL remains empty.
I've installed ffmpeg and no error is thrown during upload.

Huh... Can you send the output from the server log (storage/logs/laravel.log) generated while uploading a video file? Thanks!

After having a look at the server log, it looks like being the same issue as #223
Here is the server log: https://pastebin.com/jXEwWjy3
ffprobe is available at /usr/bin/ffprobe

Agreed; it's the same issue as #223. I'm going to close this one to avoid duplicates. Could you indicate though what Linux distro and http server you're using?


packit status tracebacks on JSON exception

I am trying to get packit status on colin upstream repository, but it failed on JSON traceback.
packit --debug status
13:27:28.138 packit_base.py    DEBUG  logging set to DEBUG
13:27:28.138 local_project.py  DEBUG  working_dir is set and git_repo is not: let's discover...
13:27:28.139 local_project.py  DEBUG  it's a git repo!
13:27:28.185 local_project.py  DEBUG  remote url of the repo is git@github.com:user-cont/colin.git
13:27:28.186 local_project.py  DEBUG  Parsed namespace and repo name from url: user-cont/colin
13:27:28.188 config.py         DEBUG  Local package config found: /home/phracek/work/colin/.packit.yaml
13:27:28.196 config.py         DEBUG  Package config:
{
    "specfile_path": "colin.spec",
    "synced_files": [
        "colin.spec"
    ],
    "upstream_project_name": "colin",
    "downstream_package_name": "colin",
    "dist_git_url": "https://src.fedoraproject.org/rpms/colin.git"
}
WARNING  dist_git_url is no longer being processed, it is generated from dist_git_base_url and downstream_package_name
13:27:28.196 utils.py          DEBUG  can't obtain upstream_ref: 'upstream_ref'
13:27:28.196 ogr_services.py   DEBUG  Authenticating with Github using a token.
13:27:28.730 distgit.py        DEBUG  Using dist-git repo https://src.fedoraproject.org/rpms/colin.git
13:27:28.730 local_project.py  DEBUG  Url https://src.fedoraproject.org/rpms/colin.git parsed. Resulting scheme: https
Cloning repo: https://src.fedoraproject.org/rpms/colin.git -> /tmp/tmpyq0yz360
ERROR    Error while decoding JSON: Expecting value: line 1 column 1 (char 0)
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/libpagure/libpagure.py", line 88, in _call_api
    output = req.json()
  File "/usr/lib/python3.7/site-packages/requests/models.py", line 897, in json
    return complexjson.loads(self.text, **kwargs)
  File "/usr/lib64/python3.7/json/__init__.py", line 348, in loads
    return _default_decoder.decode(s)
  File "/usr/lib64/python3.7/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib64/python3.7/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/packit/cli/utils.py", line 57, in covered_func
    func(config=config, *args, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/packit/cli/status.py", line 60, in status
    api.status()
  File "/usr/local/lib/python3.7/site-packages/packit/api.py", line 407, in status
    ds_prs = status.get_downstream_prs()
  File "/usr/local/lib/python3.7/site-packages/packit/status.py", line 61, in get_downstream_prs
    pr_list = self.dg.local_project.git_project.get_pr_list()
  File "/usr/local/lib/python3.7/site-packages/ogr/services/pagure.py", line 102, in get_pr_list
    raw_prs = self._pagure.list_requests(status=status_str)
  File "/usr/local/lib/python3.7/site-packages/libpagure/libpagure.py", line 215, in list_requests
    return_value = self._call_api(request_url, params=payload)
  File "/usr/local/lib/python3.7/site-packages/libpagure/libpagure.py", line 92, in _call_api
    raise Exception("Error while decoding JSON: {0}".format(err))
Exception: Error while decoding JSON: Expecting value: line 1 column 1 (char 0)
13:27:30.403 local_project.py  DEBUG  Cleaning: /tmp/tmpyq0yz360

It looks like libpagureAPI does not work.

I guess, even if libpagure fails with traceback, packit status should print a traceback, but continues with getting all other statuses

Reopening, I still see it even with #306 merged.

See https://github.com/packit-service/ogr/issues/67

Also https://pagure.io/libpagure/pull-request/43

tried this locally and this has worked for me, shall we close?

Ok, the traceback is not being thrown out so we can close.
The real cause is tracked in packit-service/ogr#67


TypeError('value is out of bounds')

This error is crashing my app, and I'm having a hard time tracking down the parameter and value that's causing it:
buffer.js:784
    throw TypeError('value is out of bounds');
          ^
TypeError: value is out of bounds
    at TypeError (<anonymous>)
    at checkInt (buffer.js:784:11)
    at Buffer.writeUInt32LE (buffer.js:841:5)
    at WritableTrackingBuffer.writeUInt32LE (./node_modules/mssql/node_modules/tedious/lib/tracking-buffer/writable-tracking-buffer.js:92:17)
    at Object.TYPE.writeParameterData (./node_modules/mssql/node_modules/tedious/lib/data-type.js:264:25)
    at new RpcRequestPayload (./node_modules/mssql/node_modules/tedious/lib/rpcrequest-payload.js:66:22)
    at Connection.execSql (./node_modules/mssql/node_modules/tedious/lib/connection.js:712:56)
    at ./node_modules/mssql/lib/tedious.js:563:33
    at Transaction.next (./node_modules/mssql/lib/main.js:738:28)
    at Request._release (./node_modules/mssql/lib/main.js:895:33)
    at Request.userCallback (./node_modules/mssql/lib/tedious.js:462:23)

It's happening in a batch of about 2k inserts inside a transaction. Any pointers to figuring this out would be greatly appreciated.

Floating point is not corrected by Tedious, as they now do Math.round()
https://github.com/tediousjs/tedious/blob/50e76efdcd87f687106c722d54c6ea4096f0a127/src/data-type.js#L421

I'm running into this issue even with the Math.round in the tedious code.
Here's the SQL that I manually inserted successfully:
INSERT INTO foo (id, foo_id, timestamp, value, value2)
VALUES ('ff3bd5e2-dbd8-424b-bb4f-455df1fe9f9c', 'b3a0a2c6-3b58-4659-8b4f-9466cede43be',
        '2016-11-01T00:00:00+00:00', null, 66666.55);

Here is the code:
const bluebird = require('bluebird');
const sql = require('mssql');

const db = require('./database.js');

sql.Promise = bluebird;

const COLUMNS = {
  id: 'id',
  foo_id: 'foo_id',
  timestamp: 'timestamp',
  value: 'value',
  value2: 'value2'
};

exports.create = data => {
  return db.getConnection()
    .then(conn => {
      const ps = new sql.PreparedStatement(conn);
      ps.input(COLUMNS.id, sql.UniqueIdentifier);
      ps.input(COLUMNS.foo_id, sql.UniqueIdentifier);
      ps.input(COLUMNS.timestamp, sql.DateTimeOffset(0));
      ps.input(COLUMNS.value, sql.Numeric(30, 15));
      ps.input(COLUMNS.value2, sql.Numeric(30, 15));

      // Have to convert string to JavaScript date for insert to work
      data.timestamp = new Date(data.timestamp);
      const header = Object.keys(COLUMNS).map(key => COLUMNS[key]).join();

      const query = `INSERT INTO foo (${header}) VALUES (@${COLUMNS.id},` +
        ` @${COLUMNS.foo_id}, @${COLUMNS.timestamp}, @${COLUMNS.value},` +
        ` @${COLUMNS.value2})`;
      return ps.prepare(query)
        .then(() => ps.execute(data)
          // Releases the connection back to the pool
          .then(() => ps.unprepare()))
        .then(() => bluebird.resolve());
    });
};

The tedious code handles null values properly but it doesn't like the floating point number that I used. Any help would be appreciated!

Wtf u talk m8s kys lel lmao 420

I still receive this error with decimals...

make sure you don't mix up precision and scale.
decimal(precision, scale)
When I switched the two I got the out of bounds error

I get this error on Windows, but not Linux or Mac

This is (was) and issue with the tedious driver which is now fixed. Please use the latest version of node-mssql if you wish to resolve this issue. locking this thread as resolved


RoBERTa: A Robustly Optimized BERT Pretraining Approach

집현전 중급반 스터디

2022년 5월 29일 일요일 9시
이다영님 권혜빈님 김혜수님 박은빈님 발표
논문 링크: https://arxiv.org/abs/1907.11692

Abstract
Language model pretraining has led to significant performance gains but careful comparison between different approaches is challenging. Training is computationally expensive, often done on private datasets of different sizes, and, as we will show, hyperparameter choices have significant impact on the final results. We present a replication study of BERT pretraining (Devlin et al., 2019) that carefully measures the impact of many key hyperparameters and training data size. We find that BERT was significantly undertrained, and can match or exceed the performance of every model published after it. Our best model achieves state-of-the-art results on GLUE, RACE and SQuAD. These results highlight the importance of previously overlooked design choices, and raise questions about the source of recently reported improvements. We release our models and code.

집현전3기 11조 RoBERTa 발표자료 입니다.
https://drive.google.com/file/d/1vokSJX2UBOZz4aqqz7G4KastPl0m_02g/view?usp=sharing

발표링크: https://www.youtube.com/watch?v=3U2yQoCkrW0&list=PL2tRglRS_GqjCcBVM26gYNdoOnX12lLIh&index=10


Single linux lib

Not really possible, since openjdk generally does not link to librt, and we depend on it. Once we drop support for glibc older than 2.17, we could revisit this.


Mental Health Page

Mental Health & Wellness Center
https://bsclibrary.net/mental-health

This will not be necessary when the new page maker is created.


Try to fix labeler for UI

At the Moment PRs will not get the kind/ui label when they change something the templates directory and also change  templates/swagger/v1_json.tmpl. Wha we want is, that PRs get the kind/ui when they change any file in the tmeplates direcory, but ignore templates/swagger/v1_json.tmpl. According to the documentation we should use any instead of all to archive this, but I haven't tested it.

Maybe we should just move the swagger file somewhere else?

In general, I agree that moving the swagger file would be the best approach, even if it means having one template at a place other than templates/.
templates should only define UI.
The only question is then: What is the correct place for this file?

Maybe assets/swagger.json.tmpl? assets already has a bunch of other generated files, it would fit there.

We could even un-template the file, there are only two simple vars set in it which I'd say can instead be set from go into the json:
  "version": "{{AppVer | JSEscape | Safe}}"
  "basePath": "{{AppSubUrl | JSEscape | Safe}}/api/v1",

I think this PR is now outdated?
Please reopen if that isn't the case.


Filtered categories should appear in gray in filter dialog

I've discussed with Bikram that we don't need PRs for changing colors, labels etc. Just push it.

Yes. Also use the same gray for the category name below.

ok

@bikramkawan, the link to the corresponding PR or commit is missing.

It is included in this PR
https://github.com/Caleydo/mothertable/pull/167


I can use it ,but i can't make it ,why?

Hi,
i want to compile the project ,but some errors happen .
$ make
Compiling source for linux
cmd/lora-gateway-bridge/main.go:10:2: import "github.com/Sirupsen/logrus": cannot find package
cmd/lora-gateway-bridge/main.go:11:2: import "github.com/brocaar/lora-gateway-bridge/backend/mqttpubsub": cannot find package
cmd/lora-gateway-bridge/main.go:12:2: import "github.com/brocaar/lora-gateway-bridge/gateway": cannot find package
cmd/lora-gateway-bridge/main.go:13:2: import "github.com/brocaar/lorawan": cannot find package
cmd/lora-gateway-bridge/main.go:14:2: import "github.com/codegangsta/cli": cannot find package
make: *** [build] Error 1

/////create folder as following
mkdir -p gw-bridge/src/github.com/brocaar/
cd gw-bridge
export GOPATH=$(pwd)
cd src/github.com/brocaar/
/////clone the project here
/////name this folder as lora-gateway-bridge and then compile
make build

@lancezhangsf make sure you're using the latest Go version or that the GO15VENDOREXPERIMENT env variable is set.