sqrtf and sqrt in device code cannot be compiled

To reproduce replace line 46 (https://github.com/GPUOpen-ProfessionalCompute-Tools/HIP/blob/master/samples/0_Intro/square/square.hipref.cpp#L46)
with
        C_d[i] = sqrtf(A_d[i]);

hipcc  square.hipref.cpp -o square.hip.out fails with
Referencing function in another module!
  %34 = call float @sqrtf(float %33) #5
LLVM ERROR: Broken function found, compilation aborted!
Generating HSAIL BRIG kernel failed

Full log: hipcc-sqrt-log.txt
I don't know about other functions but the same error for sqrt(double).
But, for example, this can be compiled:
        C_d[i] = __fsqrt_rn(A_d[i]);

Checked on:
HIP version: 1.0.16503
HCC clang version 3.5.0  (based on HCC 0.10.16501-81f0a2f-02246a0 LLVM 3.5.0svn)

Hi,
You are using old version of HCC and HIP which have bugs related to math functions. We fixed it with https://github.com/GPUOpen-ProfessionalCompute-Tools/HIP/releases/tag/preview_1.0.17102

Thank you, I'll check the new version tomorrow.

Hi,
We released ROCm 1.5 (https://github.com/RadeonOpenCompute/ROCm). I recommend using it.

Sorry for delay. It works on ROCm 1.5.
I'll close the issue. Thanks!


Automatically link PR with issue based on issue mention in PR

Task Description
Epic Parent

resolved, just keword needs to be used to properly link it
https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue


non upgraded workspace have only access to small models

Will add globalAgents: to limits, as list of agentconfigurations sId
Null will mean no limit (just as -1 means no limit for numeric limits)

hmmmm it's super granular let's chat tomorrow about a less granular solution. Not sure I have it yet....

OK so let's:

Add a large_model flag on the model spec that @fontanierh built yesterday
Add a large_model boolean on plan (migrate existing plan to true) update upgrade logic
Check for that flag in getAgentConfiguration
Turn the throw in postNewUserMessage (and friends) in a skip if getAgentConfguration is not found (return { row: null and m: null } and filter)
Add a status for require_upgrade for global agents and return that so that we can act on it in UI (not show in InputBar (should alreaday be filtered by @PopDaph latest PR), and deactivate the toggle (set to off) in the list of global agents with a DropDown to contact us.
The goal is for them to become available automatically once they upgrade (which should be the case as per above)

This obviously lies on the critical path to switch let's push this out this morning 🔥

Related discussion: https://dust4ai.slack.com/archives/C050SM8NSPK/p1695368241304159

How would the dropdown to contact us on the list of global agents look? do we already have one somewhere? @Duncid

I think it needs to be a design task before it's a eng task!

We need it to rollout v2 so we'll stick with an alert but let's think about our paywall UX as a fast follow-up task :+1:


add Formats and AdaptiveFormats to Video

Description
add Formats and AdaptiveFormats to Video and
expose Formats and AdaptiveFormats of Video to user
Issues to fix
Please link issues this PR will fix:
#86

Please merge bros :D

@Julian-Chu I added another commit. What do you think?

LGTM.

Awesome. Please release then 👍

@greentornado  you can download the release v2 , later I will try to use goreleaser to release same thing again

I built the binaries with Go 1.15 and uploaded them.

Strange ... the release data disappared. I did another v2.0.0 release.

I ran go get -u github.com/kkdai/youtube but it stucked at 1.2.4 :D


fix: crash when analyzing node package with a local dependency

Fixes Issue #1947
Description of Change
When a package has a local dependency, the package-lock.json file contains an entry without a version number, with "link": "true". This lack of version attribute causes the NodePackageAnalyzer class to crash.
These entries should be skipped instead as they cannot be processed.
Have test cases been added to cover the new functionality?
yes

@jeremylong Let me know if there is anything else I can do on this? (like update to merge in latest master, etc).

Thanks! The change is good as is - I'm just debating on the release schedule. I'll likely put in one more release before 8.0.0.

@jeremylong  I would consider this for a patch -- it's pretty debilitating. We've had to completely disable use of DependencyCheck until it's resolved (largely in combination with the need for the fix for the field length + data issue that you released a fix for recently).

Seconded regarding the patch release – we were caught between a rock and a hard place because we needed the 7.4.4 fix for using DependencyCheck at all and then had to disable the Node package analyzer or we would have had no dependency check at all.


Feature: Create an animation on scroll at the 'about us' section progress bars.

this work has done well.


Put the component in partial

Hello,
I'm doing some tests with the plugin, and it seems that you cannot put a form in a partial.
What I mean is that insteaf of calling the component in your page, and override the default template using a partial, I would like to put the component directly in the partial and call it by using
{% partial 'myform' %}
Like that, the form could be configured in the partial directly (validation, messages etc...) and you just embed it when you want.
It works with all plugin I could test (blog etc...) but with this one, when I submit the form I get an error.

Ajax handler 'emptyForm::onFormSubmit was not found'

Would it be possible to fix it ?
Thanks
Alex

Hey @Zmove, it's this issue related with #12 right?
I will take a look at the problem.
The scenario is to put the component inside a partial and call this partial from a page, right?

Partials can't run ajax handlers defined on the component

docs here
read here
read here

So, here are a few alternatives:

Method 1:
1 .Insert the component emptyForm on the page
2. Configure component parameters
3. Insert {% component 'emptyForm' %} on the partial (just the code, don't drag the component)

Method 2:
If you need to put the same partial/form on differente pages, you can do the same but at layout level
1 .Insert the component emptyForm on your layout
2. Configure component parameters
3. Insert {% component 'emptyForm' %} on the partial (just the code, don't drag the component)

Hello,
Thank you for this complete answer. I don't know october well yet and I didn't know you cant run ajax handler in partials.
I finally used your method 1. I create my partial with all fields I need and instead of putting the component in my partial, I put it in my page. It's not a big drawback.
Alex


Fix '+tmux/cd' to send newline

Hey,
I recently started testing tmux module and found out that +tmux/cd (and other functions using it: +tmux/cd-to-here and +tmux/cd-to-project) never sends newline - looks like arg argument is somehow obsolete there? I'm not very proficient in elisp, so I may be wrong though :)

There was an issue with +tmux/cd I corrected in 09226f9, which should indirectly fix this issue. +tmux/cd should send a newline, but only if noreturn is nil (the default). Let me know if that gives you any trouble. Thanks for bringing it to my attention!


Abstraction layer to add metrics in Go hooks

Is your feature request related to a problem? Please describe.
The problem is that as for now, the only way to return a metric sample from a go hook is to append an internal structure to the slice of metric operations like
*input.Metrics = append(*input.Metrics, operation.MetricOperation{
  Name:   "my_metric",
  Action: "set",
  Value:   pointer.Float64Ptr(1),
})

It is not convenient in many ways.
Describe the solution you'd like to see
It seems reasonable to describe a thin level of abstraction around metrics operations.
Like this:
input.Metrics.ExpireGroup("my_group")

input.Metrics.WithName("my_metric_1").Set(1)
input.Metrics.WithName("my_metric_2").Add(1)
input.Metrics.WithName("my_metric_2").WithGroup("my_group").Observe(1)

Describe alternatives you've considered
Leave it as is.
Additional context

Metric collector is implemented, close this.


squashfsTools: 4.5.1 -> 4.6

Description of changes

Update squashfs-tools to 4.6. Changelog.
I had to rebase the 4k alignment patch for this, but the conflicts were easy to resolve, only due to context changes, nothing substantial.
I confirmed that passing -4k-align can still be used and produces a valid squashfs, I did not confirm whether it is actually 4k aligned.
As far as I can tell there are no breaking changes, so I don’t think an entry in the release notes is needed.

Things done

Built on platform(s)

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

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

This triggers >100 rebuilds, I will make a new pull request to target staging instead.


Remove redundant test

Codecov Report

Merging #143 (8941d31) into master (e9fea9a) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #143   +/-   ##
=========================================
  Coverage     17.06%   17.06%
  Complexity      364      364
=========================================
  Files           195      195
  Lines          8917     8917
  Branches        286      286
=========================================
  Hits           1522     1522
  Misses         7312     7312
  Partials         83       83

Flag
Coverage Δ

java
32.01% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


chore(tutorial): make diagram images responsive

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Chore (CSS styling update for tutorial app).
What is the current behavior? (You can also link to an open issue here)
See angular/angular-phonecat#376.

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change?
No.
Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Other information:
Fixes angular/angular-phonecat#376.

LGTM


Is project active?

Hi,
I saw that is using an old version on MongoDb C# Driver and also Hangfire is an old version so I wonder if the project is active or not.
Thx

@imperugo Yes, it's active. But as usual need a time to maintain it.
I will update everything a bit later. Sorry for inconvenience.


Add a way to merge 2 groups into one

Like /lp mergegroup   as in the first group stated gets merged into the second group however we still have to delete the old group

Eventually the web editor will let you move permissions from one group to another so this might not be needed in LP itself.

Out of scope for the plugin I think. Would be cool to be able to drag/drop in the editor though.


[Doris on ES] Accelerated computing for count(*)

Is your feature request related to a problem? Please describe.
业务上经常会有select count(*) from table where xxx的sql，这些sql目前均通过Scan数据然后在doris中计算，而ES原始的_count接口可以基于元数据快速返回。因此希望可以采用类似的办法来加速这些常用类的sql的性能
Describe the solution you'd like
（语法不变，性能提升）
Describe alternatives you've considered
（无）.
Additional context
sum(*) 不确定是否可以采用类似的优化。

This is what aggregation push dow A.K.A

https://github.com/apache/incubator-doris/issues/3306


Docs: Add new streaming data source tutorial

Add new streaming data source tutorial. Draws on source content by Hugo at https://dev.to/oshirohugo/grafana-streaming-backend-datasource-plugin-1ob7
Fixes: https://github.com/grafana/plugin-tools/issues/428

Awesome work @josmperez !
Reviewing your tutorial I've notice that I forgot to mention the changes in src/plugin.json. We need to add the "streaming": true to it. Then it should be something like:
{
  ...
  "id": "grafana-streamingbackendexample-datasource",
  "metrics": true,
  "backend": true,
  ...
  "streaming": true
}

Hugo, regarding this:
https://github.com/grafana/plugin-tools/pull/629#discussion_r1443048552
I want to add this as a tip, but please clarify where the code you provided should be added.

I can't find any part of the below code block in this tutorial. Could you please clarify where this line should be added?

{
  ...
  "id": "grafana-streamingbackendexample-datasource",
  "metrics": true,
  "backend": true,
  ...
  "streaming": true
}

Hugo, regarding this:
#629 (comment)
I want to add this as a tip, but please clarify where the code you provided should be added.

This code should be added to pkg/plugin/datasource.go. It should replace the CheckHealth method that already exists there and it was created by the scaffold.

I can't find any part of the below code block in this tutorial. Could you please clarify where this line should be added?

Your totally right, so far there are no mentions to src/plugin.json in the tutorial. It's a file that should also be present when the plugin is create by create-plugin tool. For example:
https://github.com/grafana/grafana-plugin-examples/blob/main/examples/panel-flot/src/plugin.json
And in this tutorial the full src/plugin.json will be something like:
https://github.com/oshirohugo/datasource-streaming-backend/blob/main/src/plugin.json
where we added the property "streaming": true.
We need to add a new part to the tutorial mentioning that we need of this modification. Maybe it can be a new step before the Step 4: Build the plugin.

Awesome work @josmperez !
Reviewing your tutorial I've notice that I forgot to mention the changes in src/plugin.json. We need to add the "streaming": true to it. Then it should be something like:
{
  ...
  "id": "grafana-streamingbackendexample-datasource",
  "metrics": true,
  "backend": true,
  ...
  "streaming": true
}

Where should this code be added?

Where should this code be added?

I think we can create a new step after Step 3: Set up the plugin backend and before Step 4: Build the plugin, where we say something like:
Grafana needs to recognize that the new plugin is a streaming plugin when it's first loaded, to accomplish this, simply add "streaming":true property to your src/plugin.json
{
  ...
  "id": "grafana-streamingbackendexample-datasource",
  "metrics": true,
  "backend": true,
  "streaming": true
  ...
}

It can be a new step called "Changing plugin.json`` or Set up the plugin as streaming` or something like that.

@oshirohugo - I have added the two new parts to this tutorial. Please re-review.


[Publisher] Add Import API Definition and Edit Source in resource tab

Description:

Add the Import API Definition and Edit Source UI features to Publisher API info page Resource tab
This is how it was in 2.2.x

These features have been implemented for APIM 3.0.0 m35.

API definition import page

API definition edit page


New community ready deployment example using WOPI necessary

Based on a discussion with @dragonchaser, with many thanks for the insights!
When writing the easy install docs, I came to the point that the current deployment example deployments/examples/ocis_wopi/docker-compose.yml is focused on dev needs. We need an example that fits to community needs (and could name it ocis_community).
Deriving from the current example, we need in particular the following changes:

Use the actual traefik version --> v3
Reduce the web office apps used to one
Remove companion if not mandatory required
Remove the supplemental monitoring configuration
Remove inbucket and make the ocis email notification settings configurable

The new example must be backported to 5.0 to be available for the admin docs.
Note that the admin docs are more or less finished to use the deployment example. Only a few minor adaptions need to be made. Text adaptions could be made really quick and we would have 2 out of 3 scenarious live.
@micbar @tbsbdr

+1

done on sstable-5.0

There is the oCIS "full" example which is flexible and with all relevant services. I think that should cut it.


CPU Profiling with Chrome DevTools for Node.js (ClearScript V8)

Hi there!
We're experimenting with using ClearScript V8 to "server side render" React components. Do you have any guidance for loading and executing these components such that the CPU profiling results could be used to detect hotspots within our React components?
Thus far we've been able to attach the Chrome Node.js tools to a running instance and all of the source mapping and what not seems to work. But when running the CPU profiler, only (program) appears in the results.
Thanks!
Tom

Hi Tom,

Do you have any guidance for loading and executing these components such that the CPU profiling results could be used to detect hotspots within our React components?

ClearScript provides CPU profiling support in the form of start/stop APIs that the host invokes to produce a JSON-formatted file that can be opened in Chrome/Edge DevTools to visualize the profile.

But when running the CPU profiler, only (program) appears in the results.

Please clarify. How are you invoking the CPU profiler? Where are you seeing the results? Finally, what are you expecting to see?
Thanks!

Hi there!

How are you invoking the CPU profiler?

Assuming the engine is configured to have the debug port listening on port 9229.

From within Chrome I open the "Inspect" panel by visiting: chrome://inspect
Once that panel is open I click on the link labeled: Open dedicated DevTools for Node
Once the application is running it will appear like so:

Where are you seeing the results?

Once the application is running and the debugger is attached I start the CPU profiler by clicking the "record" button :

Which when stopped (by clicking the "record" button once more) the results are as displayed above. It's important to note, that while the profiler is running I am exercising the application such that JavaScript instructions are being sent to ClearScript.

Finally, what are you expecting to see?

At the very least I'd expect to a call to ReactDOMServer.renderToString(... in there somewhere.

Hi @tombatron,
We've reproduced this issue and are investigating.
Thanks!

Hi @tombatron,
The problem here is that V8 only collects profile data on the thread that started the profile.
That limitation is not a problem for Chromium or Node.js. Those environments are based on the event loop model, which ensures that a V8 runtime always executes code and handles debugger commands on the same thread.
ClearScript, on the other hand, is designed to accommodate any application architecture, so it doesn't impose any particular threading model upon the host. It is our assumption that script code execution is an ancillary feature rather than the host's primary functionality. Requiring an event loop would make ClearScript integration much more involved and disruptive.
Unlike Chromium and Node.js, ClearScript doesn't reserve threads for V8 operations. Instead, it executes script code on the caller's thread and handles debugger commands on worker threads. This works perfectly well for most debugger commands, but when ClearScript receives a "start profiling" command, it has no way to predict what thread will end up running the code to be profiled.
Unfortunately, that means that debugger-initiated CPU profiling is unlikely to work with ClearScript.
Sorry!

Fair enough.
Does this mean that leveraging the {Begin|Collect|End}CpuProfile methods should yield results closer to what we're looking for, versus having the external debugger initiate?

Does this mean that leveraging the {Begin|Collect|End}CpuProfile methods should yield results closer to what we're looking for, versus having the external debugger initiate?

Yes. If you call BeginCpuProfile on the thread that will execute the code you wish to profile, EndCpuProfile should return useful data. You can then use ToJson to produce a file-based profile that can be loaded into DevTools for visualization.

Thank you so much for the effort here!


[terraform-ibm-event-notifications] Add estimated-usage.yml to improve the calculated cost estimation

We need to put in a yaml file like this : https://github.com/terraform-ibm-modules/terraform-ibm-scc-da/blob/main/estimated-usage.yml to help with COS bucket estimations

Picked up issue. Starting to add usage yaml to project now.


Fixed: recipes ordering when select random

Codecov Report

Merging #26 (309588f) into master (23ea37e) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##           master      #26   +/-   ##
=======================================
  Coverage   91.83%   91.83%
=======================================
  Files          13       13
  Lines         343      343
=======================================
  Hits          315      315
  Misses         28       28

Files Changed
Coverage Δ

recipes/repositories.py
100.00% <100.00%> (ø)


Explorer View Chart Resizes on Hard Refresh

Expected Behavior

Chart should fit page

Actual Behavior

Chart is about half the size

Steps to Reproduce the Problem

Navigate to an instance dashboard and then to the Explorer tab
Select an item
Hard refresh

Specifications

Version: 0.9.1
Platform: macOS Sierra 10.12.6
Subsystem: Confirmed bug in Chrome 61 & Safari 11

This feels related to the work that @7ruth and I were doing.

@kaitmore - I'm having trouble recreating this issue now. Do you see this still?

@spmcbride1201  Nope, it seems to be gone.


[React] Error: buffer is either not set or not loaded

Hi,
I'm trying to use the Sampler interface to sample the Salamander piano samples. My code is giving me the following error when I try to play the sampler instrument "Error: buffer is either not set or not loaded".
I'm using the sample code inside the sampler example in the documentation so the code should be working.
here's what I have:
import React from "react";
import Tone from "tone";
import SomeScreen from "./views/SomeScreen";
import "./App.scss";

export default function App() {
  const lead_synth = new Tone.Sampler({
    'A0': 'A0.[mp3|ogg]',
   ... rest of the path to samples
  }, {
      baseUrl: "/src/sounds/salamander/",
      onload: () => console.log("done") // never gets called
    }).toMaster();
  lead_synth.triggerAttack("A3"); // Causes the error mentioned
  return (
    <div className="App">
      < SomeScreen />
    </div>
  );
}

I've tried waiting a couple of seconds before calling the triggerAttack function and that didn't fix it. Also the onload function never gets called.
I've basically hit a dead end with my google searches so any help would be appreciated.

@ejarzo thanks for your comments here, it actually lead me to find this, article which seems to explain exactly why this is happening.
@tambien Yotam, can we link this to wiki/doc somewhere? http://sites.music.columbia.edu/cmc/courses/g6611/spring2018/week6/index.html


"User with this email not found" error when using generateLink

Bug report

[X] I confirm this is a bug with Supabase, not with my own application.
[X] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
In local development, everything works fine:
  const { data, error } = await supabaseService.auth.admin.generateLink({
    type: "signup",
    email,
    password,
  });

Works as expected.
In production, it throws "User with this email not found". I triple checked, email and password are legit non-existing values.
To Reproduce
I'm unable to reproduce this as local dev works as expected :(
Expected behavior
I really do not see how this error can be thrown by the generateLink() function.
System information

OS: local dev : macOS
Version of supabase-js: 2.39.7
Version of Node.js: 20

Ok, I reproduced this, and I confirm this is a bug recently introduced.
It basically breaks signups for apps where an email is sent using generateLink(), which is really bad.
To reproduce:

Create a supabase project
Clone this repository
Set env vars
Launch node index.js
It throws AuthApiError: User with this email not found

https://github.com/jide/supabase-issue-875

The issue has been fixed.

I'm also experiencing this issue.  Any solutions? Do I need to update my Supabase Postgres version?

Reopening to bring supabase maintainers attention.

Currently experiencing something similar, but using "invite" type when generating a link.
For local development, all is well, but in github actions the following is failing with [AuthApiError]: User with this email not found
const { data, error } = await supabase.auth.admin.generateLink({
  type: 'invite',
  email
})

System information:
@supabase/supabase-js 2.39.3
node-version: 18.19.1
Operating System
Ubuntu
22.04.4
LTS

Possible workaround for now:
import { supabase } from "./lib";

async function init() {
  const createUserResult = await supabase.auth.admin.createUser({
    email<EMAIL_ADDRESS>    password: "123456789.",
  });
  console.log("🚀 ~ test ~ createUserResult:", createUserResult);

  const generateLinkResult = await supabase.auth.admin.generateLink({
    email<EMAIL_ADDRESS>    type: "signup",
    password: "123456789.",
  });
  console.log("🚀 ~ test ~ generateLinkResult:", generateLinkResult);

  const verifyOtpResult = await supabase.auth.verifyOtp({
    email<EMAIL_ADDRESS>    type: "email",
    token: generateLinkResult.data.properties?.email_otp ?? "",
  });
  console.log("🚀 ~ test ~ verifyOtpResult:", verifyOtpResult);

  await supabase.auth.admin.deleteUser(verifyOtpResult.data.user?.id ?? "");
}

init();

Yep, same here. I'm on the following packages:
@supabase/ssr: 0.3.0,
supabase/supabase-js: 2.42.5,
supabase: 1.163.2

My machine:
Node version 18.19.0
MacOS 14.4.1 (23E224)
The error:
{
 __isAuthError: true,
status: 404,
code: 'user_not_found'
 }

The code triggering the error:
   const { data: linkData, error: linkError } =
      await supabase.auth.admin.generateLink({
        type: "signup",
        email,
        password,
      });

Facing same issue.
  const link = await supabase.auth.admin.generateLink({
    type: 'signup',
    email,
    password
  });

Is giving the following error
error: AuthApiError: User with this email not found
App
@supabase/supabase-js:  2.42.7
NodeJs: v18.18.0
Supabase
GoTrue version: 2.148.0
PostgREST version: 12.0.2
Postgres version: <IP_ADDRESS>

For my issue, the reason that it was working for local development but not in Github actions is because locally we were using v1.153.4 of supabase cli, which was installing v2.130.0 of gotrue. When I set my action to the following the issue resolved.
- name: Setup Supabase
        uses: supabase/setup-cli@v1
        with:
          version: 1.153.4

I have yet to try upgrading to the latest version of the CLI.

@vacas5 upgrading the CLI upgrades the auth service's docker image version used which should contain the fix


The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #8     +/-   ##
=======================================
- Coverage     100%     0%   -100%
=======================================
  Files           1      2      +1
  Lines          12   3959   +3947
=======================================
- Hits           12      0     -12
- Misses          0   3959   +3959

Impacted Files
Coverage Δ

src/jasmine-jquery.bundle.js
0% <ø> (ø)

src/index.js
0% <0%> (-100%)
:arrow_down:

Continue to review full report at Codecov.

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


Removing Unknown License from allowed licenses

Work towards #1395  - this makes sure Dependency License Checks validates no "Unknown" Licenses are valid.
Signed-off-by: Gustavo Carvalho<EMAIL_ADDRESS>

/approve


[WIP] Convert advanced-list to use cycle-collections

I'm not sure if we should ever merge this, but this is mostly for the sake of discussion and feedback atm.

@staltz have a look at the updated version.
I've renamed initialTickers.action$ to tickers.effect$. I thought about tickers.reducer$ but I think a reducer is (state): state but an effect is (state, item, event): state.
I cleared up the rest of the language around reducers. I've also moved the declaration of "child actions" or as I am now thinking of them, "effect handlers" inline to the Collection declaration. I think this improves the clarity of what they're actually doing, but I still wonder if it's too opinionated.

Actually thinking about it, tickers.effect$ is actually (state): state, since it's wrapped. Maybe tickers.reducer$ would be clear.

Thanks @Widdershin this is looking good and getting better. I'm still bike-shedding on some details, but that's just a sign that I like what I see so far, and I want to also fully understand the essence of this solution.

I've addressed most of your feedback @staltz. Collection is pretty much ready for initial release. Did you want to do that under the @cycle namespace? I'm keen if you are.

Great. Here's the repo: https://github.com/cyclejs/collection
You already have write access on it. It would be easy to move (by git pushing) your existing repo because there aren't issues or PRs there to be moved.
Let's start using @cycle/collection in examples and promoting it to the community, but it will probably only become "practically official" once it's documented on cycle.js.org.
👍 Great stuff

@staltz awesome, I've pushed up to that repo. How can I publish to @cycle/collection? I don't appear to have permission.

I have to do the first publish, and then I'm able to add you as a contributor and then you'll be able to publish.

Okay, well it should be ready to go when you have time to publish.
On 23/05/2016 8:18 PM, "André Staltz"<EMAIL_ADDRESS>wrote:

I have to do the first publish, and then I'm able to add you as a
contributor and then you'll be able to publish.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/cyclejs/examples/pull/25#issuecomment-220915659

https://www.npmjs.com/package/@cycle/collection ;)

Awesome, thanks! 😺

Closed in favour of https://github.com/cyclejs/examples/pull/27/files


Support for iWork 2014+

Updated the code and documentation for the new iWork format

@pascalbros Do you have a sample password-protected iWork 2015+ file which you can share with me (dhiru at openwall.com)? I would like to take a look at how the password protection is working in the absence of .iwph file.
Previously, I have written a small parser to extract password hashes from older iWork 2009 / '13 / '14 files. Thanks!

is there any reason this was never merged?


fix tile set editor crash when drawing polygon shapes

Fixes #21545, fixes #22905
i have reviewed the code draw_polygon_shapes , the crash happened because current_tile has changed when click on another tile, and then tileset->tile_get_shapes get wrong ShapeData .
So I think one solution is : when the workspace is on drawing a shape , In _on_workspace_input shoudn't change the current tile .

related to #21545 and #22905

Is it just related or does this fix those issues?

related to #21545 and #22905

Is it just related or does this fix those issues?

i have tested  ,  fixed the issues

Thanks!


implement rolling origin cross validation for training

write a rolling origin cross validation script
the purpose of this is to find a suitable lambda in ridge regression
steps

dividing the data into N folds
Use the last fold for validating, the rest is for training with different lambda

each time, the model would be trained with a different lambda and different data

requirements

the function should look similar to scores = cross_val_score(clf, X, y, cv=5, scoring='f1_macro')
takes the following as parameters

a training model
X and y
the number of folds
the scoring metric(could be RMSE, MSE or...)

return the scores from each training session

class SearchCV would have methods

_fit_and_score:

fit estimator and compute scores for a given dataset split
return a dict with train_scores, test_scores, fitted_parameter, fit_error(bool: whether the fit succeeded)

_cross_validate:

run _fit_and_score for all splits
return results

_run_search:

find the best parameter from the results obtained from _cross_validate

add parameters to test_seartch to test last fold corrsponding indexes
run train.py again (save nthist, coord_axis)
write evaluate.py (another branch)

load nthist, coord_axis to get X_test and y_test
load the corresponding fitted parameters and the best penalty term.
evaluate the model with MSE


Fix indentation of reducer file

Mixture of tabs and spaces is causing unintended misalignment between lines. This change will make the code more legible.

Thanks @hansenlin!


feature: add support for HikariCP

HikariCP is a solid, high-performance, JDBC connection pool. Spring Boot uses HikariCP as the default DataSource from 2.X.

brettwooldridge/HikariCP: 光 HikariCP・A solid, high-performance, JDBC connection pool at last.
MySQL Configuration · brettwooldridge/HikariCP Wiki

Codecov Report

Merging #2646 into develop will decrease coverage by 0.06%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             develop    #2646      +/-   ##
=============================================
- Coverage      51.25%   51.19%   -0.07%
  Complexity      2819     2819
=============================================
  Files            556      557       +1
  Lines          17803    17825      +22
  Branches        2101     2101
=============================================
  Hits            9125     9125
- Misses          7818     7840      +22
  Partials         860      860

Impacted Files
Coverage Δ
Complexity Δ

.../seata/server/store/HikariDataSourceGenerator.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

@diguage don't use rebase to merge commit log

@diguage don't use rebase to merge commit log
👌

If I update the code for some code reviews, how to merge the commit log?

@diguage don't use rebase to merge commit log
👌

If I update the code for some code reviews, how to merge the commit log?
git pull diguage <BranchName>
// update the code
git commit -am "Commit log"
git push diguage <BranchName>

Is it OK?

ok.
Squash merge will be used during the merge, and the incremental commit logs need to be viewed during the review.

Squash merge will be used during the merge, and the incremental commit logs need to be viewed during the review.

Thank you. I will follow the advice.

Squash merge will be used during the merge, and the incremental commit logs need to be viewed during the review.

@zjinlei
Just now, I updated the code. But there is one more the commit log.

I executed the following commands:
git commit -am "delete config for connectionTestQuery"

git pull diguage hikari-data-source

git push diguage hikari-data-source

Squash merge will be used during the merge, and the incremental commit logs need to be viewed during the review.

@zjinlei
Just now, I updated the code. But there is one more the commit log.

I executed the following commands:
git commit -am "delete config for connectionTestQuery"

git pull diguage hikari-data-source

git push diguage hikari-data-source

no problem

由于前期Seata社区治理规范问题部分代码作者未签署CLA，可能引发社区知识产权风险问题。请所有在Seata社区贡献过代码（包含：主项目、官网、samples和多语言项目等）的 contributor 帮忙在这个链接登录github账号签署相应的开发者CLA，https://cla-assistant.io/seata/seata  。2023.1.31 未签署CLA的代码将会被rewrite，拜托大家帮忙签一下。

signed.


Fix bug with the macro

Description of input that causes the bug
Using following definition causes the bug
AtomsCalculators.@generate_interface function AtomsCalculators.calculate(
       e::AtomsCalculators.Energy,
       sys,
       calc::MyType,
       pr=nothing,
       st=nothing;
       kwargs...
 )
    nothing
end

This definition however works
AtomsCalculators.@generate_interface function AtomsCalculators.calculate(
       ::AtomsCalculators.Energy,   # note removal of e from this line
       sys,
       calc::MyType,
       pr=nothing,
       st=nothing;
       kwargs...
 )
    nothing
end

The cause of the bug
The bug can be inspected by calling
expr = Meta.parse("function AtomsCalculators.calculate(
       e::AtomsCalculators.Energy,
       sys,
       calc::MyType,
       pr=nothing,
       st=nothing;
       kwargs...
 )
    nothing
end")

After which you get the bug by calling
expr.args[1].args[3].args[1].args[2].value  # Should return :Energy, but gives error

To get to the root of the error you can call
julia> expr.args[1].args[3].args
2-element Vector{Any}:
 :e
 :(AtomsCalculators.Energy)

Comparing that to the working case
working_expr = Meta.parse("function AtomsCalculators.calculate(
       ::AtomsCalculators.Energy,
       sys,
       calc::MyType,
       pr=nothing,
       st=nothing;
       kwargs...
 )
    nothing
end")

julia> working_expr.args[1].args[3].args
1-element Vector{Any}:
 :(AtomsCalculators.Energy)

Solution
Based on the above, the type of calculation is always on the last element of expr.args[1].args[3].args.
Thus using following call solves the bug and works for all the cases
expr.args[1].args[3].args[end].args[end].value  # Should return :Energy or calculation type in general

I changed the index for the last argument as well. It should read AtomsCalculators.Energy, where AtomsCalculators is the first index and Energy or calculation type the last index (2nd in practice). This is just to be safe for the future.
I also added (altered) a test to verify that all use cases are now tested.
Severity of the bug
This bug has been in AtomsCalculators from the beginning and not been encountered before, so I don't expect this having been causing issues.
The only use case where this could be an issue and where it was discovered is, when you have a wrapper calculator and you want to have specific calls for it.

This is a pretty straight forward bug, so I will merge this once the tests work and tag a new version.
So, we get this resolved fast without spending peoples time on it. Feel free to comment though, if you have something to say.


Add method to get conference services auth link

Adds a new method for getting the conferencing services authorization link for a user.

Thank you for your contribution @matthewelwell . We've made a couple of tweaks to your work and expect to release it very soon. I'll close this PR as your commits are now on https://github.com/cronofy/pycronofy/pull/69
Thanks again!


Why doesn't kali work on linux?

I downloaded the file.
GO111MODULE=on go install dw1.io/go-dork@latest
I got the controls.
from the terminal
When you run go-dork -h
go-dork: command not found
Gives a fault. Please, how do we install? Can you give detailed information about how to work?

same issue, cant run it even from git-cloned repository lol

Same here. This tool dosen't seem to work in Kali Linux.

u noobs can even set gopath properly

same issue, cant run it even from git-cloned repository lol

git clone https://github.com/dwisiswant0/go-dork
cd go-dork
go build
./go-dork


VideoPlayer: Changing VideoStreamPlayer.Stream causes a crash.

Godot version
v4.0.rc1.mono.official [8843d9ad3]
System information
Windows 10
Issue description
Changing VideoStreamPlayer.Stream causes fairly frequent crashes, whether in the editor or in-game.
I also tried to create an alternative scene that contained 8 video players with streams manually assigned, but attempting to set VideoStreamPlayer.Stream in the editor caused frequent crashes, and when it didn't, trying to load the scene afterward (From the editor or in-game) did.
I recreated this project in v3.5.1.stable.mono.official [6fed1ffa3] without any crashes. It seems to be a 4.0 only issue.
Steps to reproduce
In-Game Cycle Reproduction Steps:

Download the MRP project
Run "VideoTest.tcsn"
Keep pressing "Space" to cycle through the videos
Eventually the game will crash.

Editor Crash Reproduction Steps:

Create a VideoStreamPlayer node.
Assign a VideoStream.
Repeat step 2 until crash occurs.

Minimal reproduction project
Video Player Crash MRP.zip

UP

UP

@Hromon Which Godot version can you reproduce this with? It's not clear from your comment.

@Calinou Godot 4 rc.2, when trying to change the stream, an error flies in 99% of cases :\

I have this crush too. Godot 4 b.17. I write commercial project and this error is critical for me

I can confirm this on 4.0.rc 8c7b98d45 on Linux.
Backtrace:
================================================================
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v4.0.rc.custom_build (8c7b98d4526c6ba66d7f1636abb71ccbe54727c6)
Dumping the backtrace. Please include this when reporting the bug to the project developer.

-- END OF BACKTRACE --
================================================================

The crash occurs in libogg, so it's due to a third-party library (or how we use it specifically). cc @ellenhp

Pretty out of my element with video encoding stuff, but is there any chance that the videos are the problem? For example, if we're all following the same ffmpeg conversion from the docs - is there a chance that we're all stripping something out that the engine expects?
It seems to be failing on the block of code where it's expecting specific headers to be present if I'm understanding correctly. But, I'm confused as to why it occurs sporadically.

Godot 4 rc 2 dot net
Windows 10
My project is now stalled due to this error!

same problem, godot engine 4.0 rc3

If this occurs when changing the stream from one non-null to another I would suggest maybe executing the *_clear and/or *_reset methods for theora, vorbis and ogg in VideoStreamPlaybackTheora::set_file. Careful though because it looks like there might be another thread doing theora stuff so that will need to be locked during that operation. I won't be available to try my hand at fixing this for a while longer because I'm between jobs for another few days and want to enjoy the time off, but I'll try to keep this on my radar.

If this occurs when changing the stream from one non-null to another I would suggest maybe executing the *_clear and/or *_reset methods for theora, vorbis and ogg in VideoStreamPlaybackTheora::set_file. Careful though because it looks like there might be another thread doing theora stuff so that will need to be locked during that operation. I won't be available to try my hand at fixing this for a while longer because I'm between jobs for another few days and want to enjoy the time off, but I'll try to keep this on my radar.

I think 0103af1ddda6a2aa31227965141dd7d3a513e081 might be to blame for this. The logic is changed from reading the packet once per stream type (Theora or Vorbis) to reading them twice. In the MRP, both streams seem to be using Theora yet the Vorbis logic path is running.

https://github.com/davicr/godot/commit/802f080fd63c45e6ddfc1bfca87b1ace3c1d660b

I have an issue with VideoStreamPlayer that super easy to recreate. The game crashes on a Windows export without "Export with Debug"
I have created 2 simple scenes with VideoStreamPlayer and a button in each to switch between them. When I run the game in the editor, it works fine, when I export it with debug it works fine. As soon as I untick "Export with Debug" the game crashes randomly, sometimes after a few button clicks(changing scenes), sometimes on the first click... The console gives no errors whatsoever...
Tried to do it in Godot 4.0.3 and 4.1 beta3, the result is the same. I have also recreated this in Godot 3.5.2, the exported game without "Export with Debug" works fine, without crashes at all.
I have spent 4-5 hours yesterday searching internet and trying to fix it, but I couldn't. Tried different ffmpeg builds to convert the video, tried different video framerates. Is it a Godot 4 issue?

@Freecreativename Please open a new issue with a minimal reproduction project attached.

@Freecreativename Please open a new issue with a minimal reproduction project attached.

How do I do that? I'm new here and I can't find an option to open an issue in my profile.

@Freecreativename Please open a new issue with a minimal reproduction project attached.

I have managed to do it! The issue is created.


Roll back to watchOS 6.0 to keep armv7k support

While the swift compiler in Xcode links against tbd files in the sdk that contain an armv7k slice, the open source swift toolchain links against the stdlib dylb that is in the toolchain itself. This means that we cannot drop support for armv7k support in the stdlib dylib without losing support for building armv7k when back deploying to older watch targets. For now, roll back the recent deployment target bump from 9.0 to 6.0 so that we keep armv7k and i386 simulator.
rdar://135560598

@swift-ci please test

@swift-ci please build toolchain macOS

@swift-ci please test source compatibility


Orchard.Layouts - Extend snippet manifest format with category & description

While finishing off implementing the snippet manifest parsing in #6872 I noticed a few other improvements that could be made.

Move the default / fallback snippet ToolboxIcon value into the Snippet class and pull it from there.

Add a Category field to the yaml manifest to allow the snippets to blend in to whatever category the author chooses.

Add a Description field to the yaml manifest to customize the existing hard coded description String.Format("An element that renders the {0} shape.", shapeType)

As they will extend the manifest further I thought I would get approval before I start. Also, which branch should I target?

I like it, please continue and target the dev branch.

While I was working on it I noticed a bug in your code. Just giving you a heads up in case you use that library again in the future:
var fieldsConfig = snippetConfig.Fields != null ? snippetConfig.Fields.Children : new dynamic[0];

This snuck through because the manifest always had Fields in (that's all it was being used for before) but its never null, it return DynamicYaml regardless. Instead I updated it to this:
if(snippetManifest == null || snippetManifest.Fields.Count == 0) {
    return null;
}

var fieldsConfig = snippetManifest.Fields.Children;

Great catch, thanks


[WARNING ODB-0177] error: undefined via (via5_6_120_288_1_2_58_322) referenced

Describe the bug
unzip https://drive.google.com/file/d/1puNriOUxW5JdMKytTCQN0I7J3lyoGvEI/view?usp=sharing
The error is output pretty much immediately(seconds):
$ ./run-me-BoomTile-asap7-base.sh
OpenROAD v2.0-11423-g41f53d4db
This program is licensed under the BSD-3 license. See the LICENSE file for details.
Components of this program may be licensed under more restrictive licenses which must be honored.
[INFO ODB-0222] Reading LEF file: /home/oyvind/bazel_rules_hdl/megaboom/foo/platforms/asap7/lef/asap7_tech_1x_201209.lef
[INFO ODB-0223]     Created 24 technology layers
[INFO ODB-0224]     Created 9 technology vias
[INFO ODB-0226] Finished LEF file:  /home/oyvind/bazel_rules_hdl/megaboom/foo/platforms/asap7/lef/asap7_tech_1x_201209.lef
[INFO ODB-0222] Reading LEF file: /home/oyvind/bazel_rules_hdl/megaboom/foo/platforms/asap7/lef/asap7sc7p5t_28_R_1x_220121a.lef
[INFO ODB-0225]     Created 212 library cells
[INFO ODB-0226] Finished LEF file:  /home/oyvind/bazel_rules_hdl/megaboom/foo/platforms/asap7/lef/asap7sc7p5t_28_R_1x_220121a.lef
[INFO ODB-0222] Reading LEF file: bazel-bin/build/results/asap7/ebtb_128x40/base/ebtb_128x40.lef
[WARNING ODB-0177] error: undefined via (via5_6_120_288_1_2_58_322) referenced
[WARNING ODB-0177] error: undefined via (via5_6_120_288_1_2_58_322) referenced
[INFO ODB-0225]     Created 1 library cells
[INFO ODB-0226] Finished LEF file:  bazel-bin/build/results/asap7/ebtb_128x40/base/ebtb_128x40.lef
[INFO ODB-0222] Reading LEF file: bazel-bin/build/results/asap7/array_256x64/base/array_256x64.lef
[WARNING ODB-0177] error: undefined via (via5_6_120_288_1_2_58_322) referenced
[WARNING ODB-0177] error: undefined via (via5_6_120_288_1_2_58_322) referenced
[INFO ODB-0225]     Created 1 library cells
[deleted]

Expected Behavior
It should be either a warning or an error... Ideally the error should just be fixed...
Why does this happen and what should the user do about it? A more actionable error message would be great.
[WARNING ODB-0177] error: undefined via (via5_6_120_288_1_2_58_322) referenced

Environment
OpenROAD v2.0-11423-g41f53d4db

To Reproduce
See above
Relevant log output
No response
Screenshots
No response
Additional Context
No response

Are you sure the abstract you are using was generated after the fix to write the vias? This seems quite similar to what we had before.

Are you sure the abstract you are using was generated after the fix to write the vias? This seems quite similar to what we had before.

@eder-matheus @maliberty Ah!
I see what I'm doing wrong.
I'm testing out OpenROAD w/Bazel on MegaBoom. I forgot to bump the ORFS_VERSION variable that I added. Whether or not I want to rebuild everything from scratch upon an ORFS/OpenROAD version bump is a decision that I want to make manually, otherwise it would always rebuild, effectively.
I'm going to do a version bump. If the error/warning is still there, I will file a fresh issue.


Expand on 'library('

This is a very cool project -- thanks for this!
One thing that would be cool is detecting package names.
I think this would be fairly simple as you can simply scan .libPaths() + '/library' for a list of packages then expand on library(. This would have similar behavior to autocomplete-python when you start typing import package_name.
I am happy to help in any way I can.
Best,
Harold

It would also be great if you could simply expand on the package names.
If I get brave I might take a stab at this sometime next week.


Great tool, but the Youtube timestamp is no longer valid

The timestamp captured by the tool below is not functioning.
https://www.youtube.com/watch?v=T4Df5_cojAs&ab_channel=kubucation#t=488.7268409408722
However, it works for the subsequent timestamp.
https://www.youtube.com/watch?v=T4Df5_cojAs&t=488s
Could you fix this? Thank you so much.

<iframe src="https://www.youtube.com/embed/dqpTOGmghfM?start=20&end=22" />
This works for me.

Happens to me too.
For clarification, url timestamps of this form #t=488.7268409408722 do not work with youtube (maybe they used to but now they don't)
timestamps using url params though still work. ?t=488.7268409408722 / ?a=b&t=488.7268409408722
Would it be possible to update the extension to use url param timestamps for youtube?

Apologies for the long period of inactivity on this project. I wanted to inform you that the plugin has been fully rewritten for the v3 release. I'm currently preparing for the beta release, which addressed this issue and provided a fix. Once it's available, if you encounter similar issues, please don't hesitate to open a new issue. Thank you for your understanding.


Deleting a user should delete their users/name directory

Wouldn't want to delete a user, and then when a new user with the same name is created again, have their mod data still around.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/58237-deleting-a-user-should-delete-their-users-name-directory?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github).

Thoughts on this?

I am not sure. I would prefer having warning telling that the data is still there or warning when creating new user.

I would support a checkbox in the user deletion flow that allows for deleting data at time of user deletion.
If ZNC warns on user creation if the user's directory already exists, would it offer the option to delete the existing data? That would also be a good function.

It would be nice if something could be done to progress this. I've just started building a new server and, upon tarring up the existing config directory, discovered a lot of user directories belonging to people who had left long ago...
I don't mind the idea of moving a user's folder to the equivalent of a trash bin, so long as there is a command that can be sent to the relevant module (controlpanel, I guess) to flush the bin, as our exit process is automated and I really don't want to have to clean things up by clicking on a web button periodically if I can help it :)

In the absence of ZNC itself doing this, I've written a script that will either delete or move a user's directory, depending on the configuration. Hope people find it useful.
https://github.com/linaro-its/znc-user-cleanup

@pcolmer This script is very fragile: whenever we change output format of *controlpanel, it is likely to delete all your users.

@DarthGandalf You make a good point about the dependency on the output format of *controlpanel. I've updated the code so that it tries to make sure that its interpretation of the output of ListUsers is valid.
That said:

I can't find another way of getting ZNC to tell me the names of users with accounts. I could, in theory, parse the ZNC config file instead … but then that will break if you make changes to the config file like moving users to their own file. (If #1030 or #1253 get implemented, for example)

The script will only delete user directories if the trash folder path is not set. If someone is using my script and is concerned about the possibility of the ListUsers output changing format, they can set a trash folder path instead as a safety net.

I wrote the script to fill a gap. If I could have filled that gap by offering a PR to ZNC, I would have but I'm not a C programmer, I'm afraid. I needed a solution to this specific problem because I was horrified to find so many user directories on our corporate ZNC server and trying to delete them by hand by matching names against ZNC users was risky. I'm only sharing the script in case others can benefit from it.

I do appreciate the warning about the output of *controlpanel but I would like to think that the script is only a short-term solution and that someone will be able to add the required functionality to ZNC. Of course, if anyone else is concerned about the fragility of the script and wants to offer enhancements to make it more robust, it is an open source repo :)

I can't find another way of getting ZNC to tell me the names of users with accounts

I'm not a C programmer,

It could be a module written in python and access ZNC API directly, similar to how controlpanel itself gets the list. https://wiki.znc.in/Modpython

I wrote the script to fill a gap

Sure, I understand.

Thanks for the suggestion, @DarthGandalf. I hadn't appreciated that ZNC supported Python3 modules.
Having looked at the documentation, I think that it should be fairly straightforward to write a module that is triggered by the OnDeleteUser hook. It wouldn't clear up any existing disparity between user accounts and user directories but my existing Python script can help administrators with that.
A few questions about module development …

Is there a way for a module to get the path to the ZNC config directory (e.g. /znc-data when running under Docker)? If there is, that means that administrators would only need to provide one parameter to the module (the path to the trash folder, or an empty string if deletions are wanted instead).
If there isn't, no biggie - admins will just need to explicitly give that to the module.

Is the correct way to specify configuration values to a module by specifying them as part of the loadmod command? Is it desirable to support direct message to the module to override config settings without needed to unload and reload the module?

Any preference or naming convention for modules?

Thanks.

Is there a way for a module to get the path to the ZNC config directory

CZNC::GetZNCPath(), but I believe you're looking for CUser::GetUserPath() instead.

Is the correct way to specify configuration values to a module by specifying them as part of the loadmod command?

Either works. Different modules do it differently.

Any preference or naming convention for modules?

Take a look at existing modules.
If you want it to be shipped with ZNC itself rather than being a third-party module, you'll also need a test to ensure that directories are deleted when they should, but not deleted when ZNC is told to shut down.

@DarthGandalf I've made a start but I'm getting very frustrated by the fact that, during testing, the user's directory doesn't seem to actually get removed. I think it is because Linux doesn't actually delete files or directories if there is a lock on them but I'm having trouble proving it.
One of the main reasons I'm struggling to troubleshoot my code is that I'm having trouble getting it to produce any diagnostic output. If I use the znc Docker container, I can see the output from "print" if I enter the container and run the entrypoint script by hand so that stdout is visible.
I've tried using PutModule instead but nothing appears in the IRC client, even if I've started a private msg chat with the deluserdir module. I'm not sure what I'm misunderstanding about that function call.
The code has been added to the repo if you have a chance to look at what I've done. I'm struggling to find many ZNC modules written in Python and those that I have found do use PutModule …
Thanks.

PutModule uses some kind of context to understand which client (and user) to send the message to. In network modules (which are the default) it's always valid. But since you're writing a global module, that context does not always have a user or client. So you'll need to select the target client explicitly, by whatever means. Another option is to use CZNC::Broadcast

@DarthGandalf I think that there may be a bug in ZNC preventing me from getting my module to work properly. Let me show you what's happening …
I start up ZNC and load the deluserdir module:
15:32 <philip> loadmod deluserdir
15:32 -*status(znc@znc.in)- *** deluserdir is loaded. User directories will be deleted
15:32 <*status> Loaded module deluserdir: [/znc-data/modules/deluserdir.py]

The module implements a "dummy" OnModCommand which outputs a list of all directories underneath <configuration dir>/users. So, after starting ZNC:
15:34 <philip> foo
15:34 <*deluserdir> +------------------------+
15:34 <*deluserdir> | Users with directories |
15:34 <*deluserdir> +------------------------+
15:34 <*deluserdir> | linaro_znc             |
15:34 <*deluserdir> +------------------------+

Create a user via the web interface and repeat the command to check that the user directory exists for the newly added user.
15:35 <philip> foo
15:35 <*deluserdir> +------------------------+
15:35 <*deluserdir> | Users with directories |
15:35 <*deluserdir> +------------------------+
15:35 <*deluserdir> | pcolmer                |
15:35 <*deluserdir> | linaro_znc             |
15:35 <*deluserdir> +------------------------+

Now delete the user. The module outputs information to show what it has done then, as a debugging step, calls the same code that outputs the list of user directories.
15:36 -*status(znc@znc.in)- *** Deleting /znc-data/users/pcolmer
15:36 -*status(znc@znc.in)- *** +------------------------+
15:36 -*status(znc@znc.in)- *** | Users with directories |
15:36 -*status(znc@znc.in)- *** +------------------------+
15:36 -*status(znc@znc.in)- *** | linaro_znc             |
15:36 -*status(znc@znc.in)- *** +------------------------+

Now ask the module to repeat the output of user directories …
15:36 <philip> foo
15:36 <*deluserdir> +------------------------+
15:36 <*deluserdir> | Users with directories |
15:36 <*deluserdir> +------------------------+
15:36 <*deluserdir> | pcolmer                |
15:36 <*deluserdir> | linaro_znc             |
15:36 <*deluserdir> +------------------------+

and the directory has returned!
I was originally thinking that the problem was that the directory wasn't really getting deleted because something was holding a handle on it but now I'm more inclined to think that my code really is deleting the directory then something in ZNC is recreating the directory after the OnDeleteUser event has been triggered.
That theory is further reinforced if I look at the directory timestamps for the pcolmer directory. After deleting the user from ZNC, the timestamp on the directory and everything underneath it matches "now" whereas making config changes like adding a module don't affect the timestamp, again suggesting that the directory is being newly created.

OnDeleteUser is called before "anything" is done. Thus, e.g. user and network modules are only unloaded later. The destructor for CModule makes sure to save the registry of the module (SaveRegistry()). Thus, this creates the directory and the .registry file for the module. (But there might also be other things that could cause the directory to be queried and re-created.)
I would recommend to start a timer with a timeout of 0 seconds. When the timer fires, check that the user still does not exist and only then delete the directory.

OK - thanks, @psychon. I'll give that a go and see how I get on.

@psychon I've hit a snag … the code that handles the deletion of the user account and decides to delete the user's directory or move it to the nominated trash folder needs to be able to check whether the action is to delete or move:
        trashdir = self.nv["trashdir"]
        if trashdir == "":
            shutil.rmtree(userdir)
        else:
            shutil.move(userdir, trashdir)

The snag is that nv belongs to the Module, rather than the Timer. I thought I was going to be able to get around this by deriving deluserdir from both Module and Timer but it doesn't work. When RunJob fires, I get an exception because there is no attribute nv.
Any thoughts or suggestions?

Add a trashdir property on the timer and copy the value over before you cannot access nv any more?
Alternatively: CTimer has a GetModule() method that gives you the module which created the timer. Thus, self.GetModule().nv["trashdir"] should do what you want.

Also you don't need nv there. It's for persistent storage; if you always set the value from args in OnLoad(), just use an attribute on the object.

Thanks for the assistance and feedback, @psychon and @DarthGandalf.
I think that the module code is finished and does what I want it to do :).
Looking back over the comments, the only thing I haven't done is this:

When the timer fires, check that the user still does not exist and only then delete the directory.

If there is a concern that OnDeleteUser can fire but the user does still exist then I can add a safety check. I've set the timer to fire after 4 seconds as I wasn't sure how much else might be going on between OnDeleteUser firing and ZNC finishing its work.


Update counter template link

@wangcai0124 : Thanks for your contribution! The author, @rwike77, has been notified to review your proposed change.

@wangcai0124   Looks good, thanks.  #sign-off


Feature/dm 722 create additional experiment in project

What was changed
1.) Adapt routing to experimentpage to include experimentId with rerouting to active experiment if id is invalid.
2.) Include possibility to add additional experiments to a project via experimentList component
3.) Switch to newly created experiment after experiment creation
4.) Switch between experiments via clicking on an experimentcard within experimentList component

everything seems to work very well, but I am not finished with the review, yet.
a small point:
rollover pointer for experiment switching is missing


Enabling Basic Information Cluster in the ota-provider app

Problem
#14764
Commissioning the ota-provider-app fail, when using the CHIP-tool. Without the Basic Information Cluster enabled, the CHIP-tool fails commissioning after it tries to read the VendorID attribute.
Change overview
Used ZAP tool to enable the Basic Information Cluster in the ota-provider app.
Testing

Verified commissioning of the ota-provider app is successful with the change.

/rebase


Add Incident categorization model

Describe your changes
Add multi-label classification model to the scraper so that incidents categorized as Information can be more correctly utilized.
Checklist before requesting a review

[ ] The code runs successfully.

HERE IS SOME COMMAND LINE OUTPUT

The model is current hitting at:
(ucpd-incident-scraper-py3.11) michaelp@MacBook-Air-18 ucpd-incident-scraper % make download
python -m incident_scraper download
Accuracy Score:  0.6739718614718615
Precision Score:  0.8492655911389357
Recall Score:  0.753793545629408

It ain't 70%, but it's close enough for my purposes.
(ucpd-incident-scraper-py3.11) michaelp@MacBook-Air-18 ucpd-incident-scraper % make build_model
python -m incident_scraper build-model
Accuracy Score: 0.6953463203463204
Precision Score: 0.8904948554630083
Recall Score: 0.776875400726651

Any new additions to this need to wait until exam season is over. The thought of how this is fixed though, is the following:

Save the vectorizer like the model is being saved via pickle
Load the old, pre-fit vectorizer when not building
Vectorize incoming strings and do the thing

🤞🏻 🤞🏻 🤞🏻 🤞🏻 🤞🏻


Memory view permissions

From @psifertex on May 5, 2015 14:20
In the ELF/MACH-O/PE loaded memory view, integrate memory access permissions
Copied from original issue: Vector35/binaryninja#21

Dupe of #13


[gpo] Allow longer report IDs

The GPO OIG just added several unpublished reports that all started with Information Security: Penetration Testing of GPO's, so the report IDs all ended up being the same. Doubling the allowed length will fix the issue.

Nice get - it's too bad these aren't published. They look like interesting reports.


merge BadElfAnalysis table into PopulationAnalysis table

we might not be able to merge the full BadElfAnalysis into PopulationAnalysis because of custom parameters like elf_connect_cutoff but the badelf table should at least inherit from the population analysis table to centralize functionality

Yeah, I agree. I'll also likely be adding more custom parameters to BadElfAnalysis as time goes on because there's a lot more useful information that could be stored.


Float type on empty data (CSV)

When I am uploading a CSV file and all of the data is empty - The inferred data type is the one with the smallest weight that allows this situation (float).
In such cases the fallback data type should be string.

Will be solved as part of: https://github.com/ohadmata/shmessy/issues/79


Night Shift Incompatibility

When night shift is on screensaver either distorts into bars or doesn't load.
Disabling nightshift immediately fixes the issue.
Otherwise a beautiful screensaver, wish it came as standard on Macs 🙂

Not reproducible for me, works with NightShift enabled and macOS 10.13.

@JTOwden @benmarten I can reproduce on a 13" early 2015 MacBook Pro. As a workaround try enabling the "Show with clock" option.
This just seems to be more of the same Intel driver issue that Apple introduced with High Sierra. More discussion here #377

Closing as duplicate of #377


Header nav

In this pull request, I

Downloaded the free image for the app logo.
Created empty navigation (this will be filled in in separate tasks).

hj


plugin-vite: Electron spawned before build finished (electron-forge start)

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project uses.
[X] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Electron Forge version
6.1.1
Electron version
v21.4.3
Operating system
Windows 11
Last known working Electron Forge version
6.1.1
Expected behavior

Actual behavior
On bigger projects electron-forge start stopped working
Electron is spawned before build is fully done resulting in
Unable` to find electron app... Cannot find module '/.vite/build/main.js'

Steps to reproduce
? Start from template and extend the project
Additional information
No response

I am experiencing the same issue with a mostly blank project with sqlite installed. It tries to launch main before its even done building.

Until a new version is released, you can solve this problem by doing the following.
export function restart() {
  let config;
  return {
    name: 'electron-restart',
    configResolved(_config) {
      config = _config;
    },
    closeBundle() {
      if (config.mode === 'production') {
        return;
      }
      process.stdin.emit('data', 'rs');
    },
  };
}

vite.main.config.mjs and vite.preload.config.mjs

import { defineConfig } from 'vite';

// https://vitejs.dev/config
export default defineConfig({
  plugins: [restart()],
});

This still occurs on 6.1.1.

I'll try and reproduce tomorrow. I believe it's a race condition that happens when the Vite build takes a long time.

@jclem Yeah it should be open, but seen the "restart" code in some pr from @caoxiemeihao so was thinking it gets merged/"fixed" anyways. In the end it's not an proper fix starting/restarting electron 3 times and there are more issues. For example closing the electron process doesnt exit the npm process.
After closing electron you sit at the red arrow and it needs extra killing of npm. Works fine in webpack plugin.

Was easier for me to switch to custom setup using forge+parcel

I have the same issue. I am importing a module that requires some time for the build process to transform and the electron process starts and it doesn't find the main.js

The launch of Electron should wait until the Vite build has finished.
It's hard to give a repro because it only happens when there are many imports happening 😥

Can anyone provide a reproduction repo?

Can anyone provide a reproduction repo?

You can trigger it easy with vite template and monaco-editor
npm init electron-app@latest my-new-app -- --template=vite
cd my-new-app
npm i -D monaco-editor

and then add the monaco-editor workers to preload.js (https://github.com/microsoft/monaco-editor/blob/main/samples/browser-esm-vite-react/src/userWorker.ts)
import * as monaco from 'monaco-editor';
import editorWorker from 'monaco-editor/esm/vs/editor/editor.worker?worker';
import jsonWorker from 'monaco-editor/esm/vs/language/json/json.worker?worker';
import cssWorker from 'monaco-editor/esm/vs/language/css/css.worker?worker';
import htmlWorker from 'monaco-editor/esm/vs/language/html/html.worker?worker';
import tsWorker from 'monaco-editor/esm/vs/language/typescript/ts.worker?worker';

self.MonacoEnvironment = {
	getWorker(_, label) {
		if (label === 'json') {
			return new jsonWorker();
		}
		if (label === 'css' || label === 'scss' || label === 'less') {
			return new cssWorker();
		}
		if (label === 'html' || label === 'handlebars' || label === 'razor') {
			return new htmlWorker();
		}
		if (label === 'typescript' || label === 'javascript') {
			return new tsWorker();
		}
		return new editorWorker();
	}
};

Transforms ~900 modules in 20+ seconds but electron is starting way before

Tested https://github.com/electron/forge/pull/3220 and works fine here. Thx @caoxiemeihao
But the "exit" issue https://github.com/electron/forge/issues/3208#issuecomment-1514070208 still exists. Happens only with the vite plugin. For some reason it needs an extra Ctrl+C in the console window here after closing electron.

Hello, thanks for having a fix ready! When do you think it'll get released? :)

It's was merged into master 455e476eef5c16fcdda983705900afcd60d1d4f6 so you can use this until new release

@PinkiePieStyle
Electron App can exit correctly now 👉 https://github.com/electron/forge/pull/3236

How does one use this plugin from head, since a new version hasn't been released yet? I don't believe npm supports subpackages from a github repo (see https://github.com/npm/feedback/discussions/881). I tried multiple things, including copying the plugin locally. The local copy fails because it fails with this error: https://github.com/electron/forge/blob/ffcc88d415f7a51c84c710d6f6eeda9e17a358e8/packages/api/core/src/util/plugin-interface.ts#L45

@pkukkapalli The above plugin is a Vite plugin, not a Forge plugin.
If still not works in your project, you can provide a minimal reproduction repo for show it.

@caoxiemeihao sorry not your workaround, I meant how do we use the @electron-forge/plugin-vite plugin for Forge from head. The version on NPM does not have the version with https://github.com/electron/forge/commit/455e476eef5c16fcdda983705900afcd60d1d4f6 in it. I have tried copying the plugin down from the repo, but was not successful in getting forge to recognize it.

@erickzhao  Do you have any plans to release a new version?

@erickzhao Do you have any plans to release a new version?

A new version is planned for release next week.


Sanity UI: Badge does not respond to size property

Describe the bug
Size is a defined property on the Badge primitive but it appears unresponsive to that property. Setting a padding or wrapping in an Inline primitive doesn't help.
To Reproduce

Go to the Sanity UI Arcade.
Create a badge primitive. <Badge>Text</Badge>
Add a size property. <Badge size={3}>Text</Badge>

Expected behavior
I would expect the text size to change.
Which versions of Sanity are you using?
In the Sanity UI Arcade on 2021-04-09.

Hi @geoffreyjball. It seems the documentation was outdated for this property.
<Badge fontSize={3}>Text</Badge> will work.
Sorry about that!


Topology should allow delete a switch

Similar to Issue #154, it would be nice if we have ways to delete a switch from Kytos-ng (topology/database). After some migration/maintenance window, you may end up with a switch on the topology that is not active anymore and will never connect again.
We should then:

Delete the switch from controller.switches
Delete the switch and its interfaces from MongoDB topology collections
Delete flows that are associated with that switch?

Before all of this, we should make sure the switch is not being used for any service? Mef_eline, flows, etc? Provide means to force removal (even if it is being used by EVCs? Flows?)?

EVCs related to the soon-to-be-deleted switch should be deleted. An exception will be raised while creating a UNI object.

Provide means to force removal (even if it is being used by EVCs? Flows?)?

Seems doable. Recognizing EVC-created flows can be done by looking at "flow.owner" in each flow. mef_eline can delete EVCs that are related to the deleted switch. It can be kytos/topology.switch_deleted  the event that notifies other NApps. On the other hand, the EVC deletions could be done by the user as well.

Delete flows that are associated with that switch?

Taking into account that EVC flows are going to be deleted. coloring is the one currently not capable of deleting flows. And, we have the same case with deleting flows.
To recognize a disconnected switch, is it enough to look into status == EntityStatus.DISABLED?


CI : Introduce codecov

This PR adds a unit test and codecov in github test workflow

Codecov Report

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

@@           Coverage Diff           @@
##             main      #32   +/-   ##
=======================================
  Coverage        ?   40.74%
=======================================
  Files           ?        4
  Lines           ?       54
  Branches        ?        0
=======================================
  Hits            ?       22
  Misses          ?       32
  Partials        ?        0

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Getting linked items fails with deleted items

I'm using getLinkedItemWithRoles() and on a particular work item I get the error below:
    470 if self.linkedWorkItemsDerived is not None:
    471     for linked_item in self.linkedWorkItemsDerived.LinkedWorkItem:
--> 472         linked_items.append((linked_item.role.id, Workitem(self._polarion, self._project, uri=linked_item.workItemURI)))
    473 return linked_items

AttributeError: 'NoneType' object has no attribute 'id'

Looking at the work item in Polarion it seems that there are links to two items that have been deleted, so I assume the method is failing when it attempts to add the deleted items to the linked work items list. Could you add a check for work item validity?

I was not able to reproduce it, but I added a check that should solve this exception.

I have had the same issue, it happens because links in polarion do not guarantee the workitem exists, I would advise to check in your system for such dangling links, otherwise you can always hack "try: ... except: continue" around the list append

I think this happens when items are deleted using the API. it seems that the GUI of Polarion also cleans up all references, but this may not be the case for the API. Did @z3r0k3ys experience this issue with the latest package version?

I will be able to provide an answer by Monday, because unfortunately I'm using a forked version with the try except block to avoid this issue, but I can guarantee the problem is not the package, it's rather that those references are allowed to exist in the first place, which should be addressed by polarion

You could add a flag for safety to turn missing items into just a warning, but from my point of view this can be closed


Crash from doing... Well, I don't really know.

I clicked on the "run script" button and it crashed. It worked fine after I rebooted it.
Error

levelEditor/hexInspector.lua:40: bad argument #1 to 'sub' (number expected, got nil)

Traceback

[C]: in function 'sub'
levelEditor/hexInspector.lua:40: in function 'sectionRows'
levelEditor/hexInspector.lua:83: in function 'onReload'
ui/tools/details.lua:34: in function 'reload'
levelEditor/levelRoot.lua:65: in function 'reload'
levelEditor/levelRoot.lua:110: in function 'runScript'
levelEditor/scriptInterface.lua:48: in function 'onClick'
ui/widgets/button.lua:106: in function 'onInputDeactivated'
ui/base/container.lua:133: in function 'inputDeactivated'
...
ui/base/container.lua:126: in function 'inputDeactivated'
ui/base/container.lua:126: in function 'inputDeactivated'
main.lua:37: in function 'inputDeactivated'
libs/tyoeerUtils/input.lua:279: in function 'actionDeactivated'
libs/tyoeerUtils/input.lua:296: in function 'triggerDeactivation'
libs/tyoeerUtils/input.lua:211: in function 'mousereleased'
main.lua:54: in function <main.lua:53>
[C]: in function 'xpcall'

Steps to reproduce: save a level, then run a script on it.
Saving a level removes the information of which section are where in the file (because it is no longer valid). After running a script, the whole editor gets reloaded to mirror any changes made, at which point the Hex Inspector tries to use that information, and fails so miserably it crashes.
Should be fixed in the next release.


[tool_crash] ProcessException: Process exited abnormally:Command line invocation:    /Applications/Developer/Xcode.app/Contents/Developer/usr/bin/xcodebuild -listxcodebuild: error: Could not resolve package dependencies:  Packages are not supported when using legacy build locations, but the current project has them enabled. Command: /usr/bin/xcodebuild, OS error code: 74

Command
flutter build ios

Steps to Reproduce

...
...
...

Logs
ProcessException: Process exited abnormally:
Command line invocation:
/Applications/Developer/Xcode.app/Contents/Developer/usr/bin/xcodebuild -list
xcodebuild: error: Could not resolve package dependencies:
Packages are not supported when using legacy build locations, but the current project has them enabled. Command: /usr/bin/xcodebuild, OS error code: 74
#0      RunResult.throwException (package:flutter_tools/src/base/process.dart:172:5)
#1      _DefaultProcessUtils.run (package:flutter_tools/src/base/process.dart:323:19)
<asynchronous suspension>
#2      XcodeProjectInterpreter.getInfo (package:flutter_tools/src/ios/xcodeproj.dart:381:50)
#3      buildXcodeProject (package:flutter_tools/src/ios/mac.dart:122:78)
<asynchronous suspension>
#4      BuildIOSCommand.runCommand (package:flutter_tools/src/commands/build_ios.dart:93:43)
#5      _rootRunUnary (dart:async/zone.dart:1198:47)
#6      _CustomZone.runUnary (dart:async/zone.dart:1100:19)
#7      _FutureListener.handleValue (dart:async/future_impl.dart:143:18)
#8      Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:696:45)
#9      Future._propagateToListeners (dart:async/future_impl.dart:725:32)
#10     Future._completeWithValue (dart:async/future_impl.dart:529:5)
#11     _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:40:15)
#12     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:311:13)
#13     ApplicationPackageStore.getPackageForPlatform (package:flutter_tools/src/application_package.dart)
#14     _rootRunUnary (dart:async/zone.dart:1198:47)
#15     _CustomZone.runUnary (dart:async/zone.dart:1100:19)
#16     _FutureListener.handleValue (dart:async/future_impl.dart:143:18)
#17     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:696:45)
#18     Future._propagateToListeners (dart:async/future_impl.dart:725:32)
#19     Future._completeWithValue (dart:async/future_impl.dart:529:5)
#20     _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:40:15)
#21     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:311:13)
#22     BuildableIOSApp.fromProject (package:flutter_tools/src/application_package.dart)

[✓] Flutter (Channel stable, 1.20.1, on Mac OS X 10.15.6 19G73, locale th-TH)
    • Flutter version 1.20.1 at /Users/apple/Development/flutter
    • Framework revision 2ae34518b8 (7 days ago), 2020-08-05 19:53:19 -0700
    • Engine revision c8e3b94853
    • Dart version 2.9.0

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at /Users/apple/Library/Android/sdk
    • Platform android-30, build-tools 30.0.1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
    • Xcode at /Applications/Developer/Xcode.app/Contents/Developer
    • Xcode 11.6, Build version 11E708
    • CocoaPods version 1.10.0.beta.1

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

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

[✓] Connected device (1 available)
    • iPhone SE (2nd generation) (mobile) • 285F4671-6269-4AFC-87C3-7702BB79288B • ios • com.apple.CoreSimulator.SimRuntime.iOS-13-6 (simulator)

• No issues found!

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: f139b11009aeb8ed2a3a3aa8b0066e482709dde3
Plugins
flutter_facebook_login-79ff08a06e9aa38f4cce82bd5294a563f0884e79
flutter_line_sdk-0701f0575054ea53947d29617e33f7ff630dde13

Hi @harzancs,
There's similar open issue describing your case, 57083.
Please follow-up there for updates and questions.
Closing this as duplicate.


Add early internals document draft

I'd like to fill the holes and illustrate with inlined examples next (with tut).


Linked document section does not exist

articles/terraform/includes/terraform-plan-notes.md includes a link to https://developer.hashicorp.com/terraform/cli/commands/plan#security-warning but no security-warning section exists in linked page.

@redguyrichard Thank you for the feedback. The include file has been updated.


NBug issue template: Add default text

As suggested in https://github.com/gitextensions/gitextensions/issues/6607#issuecomment-494683221
and https://github.com/gitextensions/NBug/pull/18
The intention is to have something the bot can close automatically without having the submitter or other users "offended".

The dup bot is looking at the title.
We can build another bot that would close issues as incorrectly filled.
This text can be used as a marker.
On Thu, May 30, 2019, 6:26 PM Gerhard Olsson<EMAIL_ADDRESS>wrote:

@gerhardol commented on this pull request.
In NBug/Core/Util/ErrorReportMarkDownBodyBuilder.cs
https://github.com/gitextensions/NBug/pull/19#discussion_r288902292:

@@ -29,6 +29,8 @@ public string Build(Exception exception, string additionalInfo)
:warning: Review existing issues to see whether someone else has already reported your issue.
-->

+The sections below must be filled in and this text must be removed or the issue will be closed.
Good idea.
More important: is this text enough for the bot to close or at least add a
label?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/gitextensions/NBug/pull/19?email_source=notifications&email_token=ABBTEXX35CZIKGLE4XK6URTPX6FSJA5CNFSM4HQRNZFKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB2DENFQ#discussion_r288902292,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBTEXRPQQ6VB6RY65UELUTPX6FSJANCNFSM4HQRNZFA
.

I made a bot to handle that. Have a look at https://github.com/RussKie/Playpen/issues/26


Add Apache 2.0 License

This pull request adds an Apache 2.0 License to the repository. Fixes https://github.com/AdrianKlessa/QuantumRNG/issues/1.

Hello! Thank you for taking an interest in this repository. I've added an MIT license to the project. Since it's also a permissive license, similarly to Apache 2.0, I think that should be fine for you? I'll close this for now but feel free to create a new issue if you have any more concerns.


Backup to AWS S3 storage

AWS S3 is being considered as a cost effective backup solution for databases in cloud.  We are looking for a way to take the MSSQL backup directly to S3. There are several third party applications available which gives you the functionality to mount S3 storage as a windows Drive thus existing Mintenance Backup script and job would work in that case. MSSQL yet doesnt support taking backup directly to S3 like it does with Azure blob storage.
With current limitation between AWS & MSSQL is there a possibility that you integrate the functionality to copy the backup to S3 using default AWS CLI toolkit?

Take the backup to local/network path.
Once backup is completed copy them using AWS CLI S3 toolkit to S3 and clean them up upon successful copy?
https://docs.aws.amazon.com/cli/latest/reference/s3/

In SQL Server 2022 Microsoft finally created a way to backup directly to S3 bucket in AWS.
Below is the document describing the steps:
https://aws.amazon.com/blogs/modernizing-with-aws/backup-sql-server-to-amazon-s3/
The backup command syntax is similar, but slightly different from backup to URL, like so:
BACKUP DATABASE db1
TO URL = 's3://sql-backups-2022.s3.us-east-1.amazonaws.com/backups/db1.bak'
WITH FORMAT, COMPRESSION, MAXTRANSFERSIZE = 20971520;
Is there a way to include this functionality into the solution? That would be a life changing modification, allowing AWS/SQL customers to save a ton of money by not keeping the volumes for the backups and not paying for them.
Please, Ola?

For having ola hallengren backup solution work for s3 compatible storages like minio, a quick solution is to comment lines between 962 to 966 and 982 to 986. This will prevent URL format checking executed and then the SP would work for URLs beginning with "S3://" like amazon s3 storage buckets and/or any S3 compatible storage solutions like minio. below is the portion of backupdatabase SP that should be changed:
`
--// Check URLs                                                                          //--
--IF EXISTS(SELECT * FROM @URLs WHERE Mirror = 0 AND DirectoryPath NOT LIKE 'https://%/%')
--BEGIN
--  INSERT INTO @Errors ([Message], Severity, [State])
--  SELECT 'The value for the parameter @URL is not supported.', 16, 1
--END
IF EXISTS (SELECT * FROM @URLs GROUP BY DirectoryPath HAVING COUNT(*) <> 1)
BEGIN
INSERT INTO @Errors ([Message], Severity, [State])
SELECT 'The value for the parameter @URL is not supported.', 16, 2
END
IF (SELECT COUNT() FROM @URLs WHERE Mirror = 0) <> (SELECT COUNT() FROM @URLs WHERE Mirror = 1) AND (SELECT COUNT(*) FROM @URLs WHERE Mirror = 1) > 0
BEGIN
INSERT INTO @Errors ([Message], Severity, [State])
SELECT 'The value for the parameter @URL is not supported.', 16, 3
END

--IF EXISTS(SELECT * FROM @URLs WHERE Mirror = 1 AND DirectoryPath NOT LIKE 'https://%/%')
--BEGIN
--  INSERT INTO @Errors ([Message], Severity, [State])
--  SELECT 'The value for the parameter @MirrorURL is not supported.', 16, 1
--END
IF EXISTS (SELECT * FROM @URLs GROUP BY DirectoryPath HAVING COUNT(*) <> 1)
BEGIN
INSERT INTO @Errors ([Message], Severity, [State])
SELECT 'The value for the parameter @MirrorURL is not supported.', 16, 2
END
IF (SELECT COUNT() FROM @URLs WHERE Mirror = 0) <> (SELECT COUNT() FROM @URLs WHERE Mirror = 1) AND (SELECT COUNT(*) FROM @URLs WHERE Mirror = 1) > 0
BEGIN
INSERT INTO @Errors ([Message], Severity, [State])
SELECT 'The value for the parameter @MirrorURL is not supported.', 16, 3
END
`

I am finally working on this.
I have created a pull request: https://github.com/olahallengren/sql-server-maintenance-solution/pull/822
Branch: https://github.com/olahallengren/sql-server-maintenance-solution/tree/feature-backup-aws-s3
Please do some testing.

This has been released to main now.
https://ola.hallengren.com/versions.html


Use mvnw and Maven 3.6.3

So we can have unified Maven version.
Also fixed the issues when upgrading Maven.

Thanks @guylabs !


[material-ui][Select] Overflow blink when opened

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example: https://codesandbox.io/s/mui-overflow-bug-f32npc
Steps:
To reproduce it, just use a select component and click it
Current behavior 😯

When the select component is clicked, two properties should appear in the body: 'padding-right: 17px' and 'overflow: hidden'.
However, on the first click this does not occur, only the 'padding-right: 17px' property is added. On the second attempt, both properties are added.
As a consequence, a padding is added with the overflow appearing, this can break some layouts and the overflow blink
Expected behavior 🤔
Both properties must appear on the first select click, preventing layout breakdown and overflow blinking

Context 🔦
No response
Your environment 🌎
my OS and browser: Chrome 118 on Windows 10

I'm not able to reproduce an overflow blink on a Mac. Could you share a screen recording that shows the behavior you're seeing?

https://github.com/mui/material-ui/assets/58609013/ea0ef026-46f7-4151-925f-f8084fd447e0

Thanks for sharing that video—I see what you're describing now with the scrollbar flashing due to overflow: hidden apparently not being applied on the first click. I have no clue what could be causing that, though 😅 and I'm still not able to reproduce it. I don't know where that padding-right is coming from. When I try to reproduce this, I don't see that attribute being applied at all:
https://github.com/mui/material-ui/assets/71297412/7a222b14-9010-4fb2-8b1a-9fd5da27c7f9
Have you experimented with other browsers? I've tried Chrome and Safari on Mac. Or could it be a Chrome extension behaving badly?

For anyone having a similar issue:
The cause of the problem is not a bug in the MUI, but an incompatibility with the extension called 'Anti Adblock Detector' .
For some reason the extension detects some MUI components as anti ad blocker components and this causes the problems.
As this is an issue with the extension and not the MUI, this issue will be closed.


classification loss sticks around 2.30212

After 1 or 2 epochs of classification loss rightly decreasing, it increases to around 2.30212 and sticks with it and never changes. I'm stuck in this situation now= =
I use my own dataset which re-organized to CSV format as descripted in README.M.
Anybody knows what is going on?

i have the same problem,2.3021, classification = torch.clamp(classification, 1e-4, 1.0 - 1e-4)

i have the same problem 2.302

@zbpjlc low down the lr would solve it, at least works for me

hello，I have the same problem.I would like to know how low your lr is in(optimizer = optim.Adam(shufflenet.parameters(), lr=1e-5))

I had same problem while transfer learning on my custom dataset, so tried initialization done in model init part. At least both losses are decreasing now.
This is the part I added while init'ing my custom head
prior = 0.01

self.classificationModel.output.weight.data.fill_(0)
self.classificationModel.output.bias.data.fill_(-math.log((1.0-prior)/prior))

self.regressionModel.output.weight.data.fill_(0)
self.regressionModel.output.bias.data.fill_(0)

self.freeze_bn()


live webcam command does not work:  "Video stream has stopped."

Attempting to run this command:  darknet detector demo cfg/coco.data cfg/yolov4-tiny.cfg yolov4-tiny.weights -c 0
Instead of it showing the webcam, console messages are logged:
Video stream has stopped.
Video stream has stopped.
Video stream has stopped.
Video stream has stopped.
...etc...

A Darknet-enabled webcam application has since been added to the DarkHelp CLI.
See:  https://www.ccoderun.ca/darkhelp/api/Cam.html

This should be fixed in v2.0.222, hash #f072df1.
(But I still recommend the one in DarkHelp since it has more features than using Darknet directly.)


[ReactQueryDevtools] Handle in a better way the "close" button

Hello,
I've got an issue with the "close" button on the interface of the dev tools.

The "close" button doesn't allow you to see what's the name of the key behind.
Two suggestions to deal with it:

Use the react-query logo to close the panel
Add at least a bit of padding in the list of queries to be able to scroll enough to have some space to see the name of the key

Thank you 😊

I like using the logo. Do you want to contribute this?

I like using the logo. Do you want to contribute this?

Yes, 😊

Fixed this using the props: position and closeButtonProps
					<ReactQueryDevtools 						position={"top-left"} 						closeButtonProps={{ style: { top: 0, bottom: "unset" } }} 						initialIsOpen={false} 					/>

I think the general idea was to have the react-query icon that opens the devtools also have it close the devtools. one advantage of the close button is that it appears at the same spot as the open button, so you can click to open and click to close in the same spot. However, I think the react-query logo can always also close the devtools, basically giving two ways to close it: the icon and the close button. Then, you can hide the close button with:
<ReactQueryDevtools closeButtonProps={{ style: { display: "none" } }} />

and you could still close them with the logo. That solves the initial issue that the close button overlaps with many queries.

:tada: This issue has been resolved in version 3.39.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Why not replace the close button with the logo and remove the logo inside the dev tool panel? The space taken by the logo can be used for more useful things.

the close button is always placed where you opened the devtools, but it sometimes overlaps with queries. If we move the logo there, then the logo would overlap with the queries.
the implemented solution is backwards compatible (no need to educate users that the close button is now "gone"), but you can now hide the close button via:
<ReactQueryDevtools closeButtonProps={{ style: { display: "none" } }} />

and only use the logo to close them again to sidestep the issue of the closebutton overlapping with queries.

Sorry I forgot to help on this on this one.
Thank you!

:tada: This issue has been resolved in version 4.0.0-beta.14 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Missing change variable type code action in wait future expression

Description:
Add change variable type code action does not work for BCE2652. Check for other missing cases as well(if available).
Describe your problem(s)
https://user-images.githubusercontent.com/46120162/182302591-d14d65e3-a15b-40bf-a9a5-4f3050397cd5.mov
code
function test() {
    future<int> fut = start foo();
    int fVal = wait fut;
}

function foo() returns int {
    return 0;
}

Describe your solution(s)
Support change variable type for BCE2652. Check for other missing cases as well.

Case 2:  Missing change variable type code action for functions with inferred typedesc value (BCE3931)