Remove Yii autoloader

Whether yii2 is installed via composer or via zip archive, the composer autoloader is always included.
I propose to:

Remove classes.php and instead add classmap configuration to composer.json.
Remove Yii::autoload();
Remove Yii::$classMap;

This will not break BC for people not manually adding stuff to the classmap.
Of course for all others it will break BC so might be better to put it in 2.1.

composer will not autoload anything using aliases. e.g. anything in namespace app\ will stop working.

@cebe Composers' classmap key will scan files just like you do, example from my autoload_classmap.php:
    'app\\components\\ActiveDataProvider' => $baseDir . '/protected/components/ActiveDataProvider.php',
    'app\\components\\ActiveField' => $baseDir . '/protected/components/ActiveField.php',
    'app\\components\\ActiveForm' => $baseDir . '/protected/components/ActiveForm.php',
    'app\\components\\ActiveQuery' => $baseDir . '/protected/components/ActiveQuery.php',
    'app\\components\\AfterSaveException' => $baseDir . '/protected/components/AfterSaveException.php',
    'app\\components\\AggregateValidator' => $baseDir . '/protected/components/AggregateValidator.php',

this means that every time you add a class, you have to rebuild composer classmap?

Yes, same is true for classes.php right? I don't see the difference between running composer dumpautoload (which is automatically done after composer install / composer update) and the script (i'm assuming) you use when preparing zip archives.

I am talking about development workflow when working with yii. If I create classes in phpstorm I do not want to run composer dumpautoload in order for them to work.

Of course you don't, but you don't want to add it to classes.php either do you??

Ah wait, my bad, I'm not communicating clearly.
I want to remove the classmap part of the autoloader, since that's 100% superfluous. (I hadn't notice Yii also has an autoloader for namespaces which support aliases).
The autoloader using aliases can be replaced by configuration in composer.json as well but that is indeed not a valid option for people not using composer.
So then my suggestion becomes a bit different:

Remove classes.php and instead add classmap configuration to composer.json.
Remove the classmap loader from Yii::autoload();
Remove Yii::$classMap;

I did investigation to this question.
Indeed classes.php together with Yii::$classMap could be removed painlessly. Composer completely substitutes this functionality.
But Yii::autoload should be left in it's place because of classes based on Yii aliases: app\, frontend\ and so on.

@hiqsol Thanks, i'm just thinking, couldn't we also (eventually) replace that part by passing setAlias on to   the composer classloader? (Just a thought)

I am talking about development workflow when working with yii. If I create classes in phpstorm I do not want to run composer dumpautoload in order for them to work.

No, you don't need to. Because classmap functionality just makes class resolving faster.
When you create class and it is available through PSR-4 autoload it will work immediately without composer dumpautoload.

Does anyone remember Yii2 can be installed without Composer?
https://github.com/yiisoft/yii2/blob/master/docs/guide/start-installation.md#installing-from-an-archive-file-
https://github.com/yiisoft/yii2/blob/master/docs/guide/start-installation.md#other-installation-options-

@klimov-paul Yes I do and I explicitly mentioned that in the issue; the zip archive also includes the vendor directory and thus the composer autoloader.

We could theoretically do many thing. What problem are you trying to solve here? I do not get the point?

Remove functionality from Yii that is already provided by a library that's available 100% of the time?
Essentially making the codebase smaller without removing any functionality.

there is no need to include composer autoload file if I want to skip it so this will remove functionality.
Also classmap can be used to load classes from libraries that do not support composer.
The provided classmap file is 23kb which is 0.4% of the framework size. You would not notice any improvement. Yii comes with a fully functional PSR-4 autoloader. I see no sense in removing any functionality from it. dropping classMap property is a BC break and not shipping the classes file does not make sense when we have the classMap property.

@SamMousa, concerning remove Yii::$classMap
it's not really good idea because of the way autoloading works.
First autloader (Yii) looks for the class then next autoloader (Composer) looks for the class. And classmap mechanism is here to speed up the look up process for well known classes.
Normal PSR autloading requires actual looking for files including calling file_exits() while classmap is a straight mapping from class to path.
So to get the most boost classmap looking has to be done in the very first place.
What I tried (and it works fine) is to use composer generated classmap instead of classes.php:
diff --git a/Yii.php b/Yii.php
index 83cd858..afff18a 100644
--- a/Yii.php
+++ b/Yii.php
@@ -26 +26 @@ spl_autoload_register(['Yii', 'autoload'], true, true);
-Yii::$classMap = require(__DIR__ . '/classes.php');
+Yii::$classMap = require(parent(__DIR__, 2) . '/composer/autoload_classmap.php');

It gives you small thing - you don't need to keep (generate) classes.php file.
Also it speeds up resolving of other (non Yii) classes in composer/autoload_classmap.php.

@hiqsol I know, that is exactly what my goal was ;-)
I didn't suggest removing classmap altogether, just removing it from Yii and using the one provided by composer.

We could add this information to the guide here: http://www.yiiframework.com/doc-2.0/guide-tutorial-performance-tuning.html#optimizing-autoloader
But should also add a note, that loading huge classmap files can also have negative impact on processing time.

It's already mentioned (dumpautoload -o) is essentially the same.


Do not allow changing a workflow's first state (closes #3778)

@normanjaeckel As discussed, please review again.

Rebased.
@FinnStutzenstein Please review 2nd commit. Then I'll rebase and merge.

@emanuelschuetze Ok.


Update how-to-track-experiments-mlflow.md with information about filtering datasets

The MLflow documentation mentions the capability of filtering runs by datasets. This is not supported when MLflow is used with Azure Machine Learning as a tracking backend.
This PR adds "Filtering by datasets" to the Support matrix section in the how-to-track-experiments-mlflow.md doc.

@msakande
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Thank you for contributing to this Azure documentation. Unfortunately, changes to Machine Learning content can't be accepted this week. The Skilling Content group is migrating this content to a new repository: azure-ai-docs.
In the meantime, we are closing this PR and tagging it so that when the content is available in the new repo, we can follow up with new location information.
#label:"migrate-followup"
#please-close

@Izzzu We’re happy to share that contributions to machine-learning, ai-studio, ai-services, genomics, open-datasets, search are once again possible at azure-ai-docs. We appreciated your patience during this migration.
Find your article in the new repository and make the changes there (or copy/paste the raw markdown from your changed file) and open a new PR.
#remove-label:"migrate-followup"


Bridge: Erroneous 4NT bid

Again bot response inappropriate.


GMs can edit games at the con

Mostly an issue as Kris's multiplied GMs (last year's "David" issue).
But be nice to have a flag to turn that on / off.

Repro:
ACUS has "Allow Game Signup" config flag set to "Admin" but Edwin Voskamp (user id 25), regular user, can edit his games.

Mostly because it still shows "Become a GM" button which gets you to your games.

Huh, it's a feature. Become a GM is always live. If you have games submitted to the current con, it shows you those and you can keep editing those.
I just edited a game on ACNW, and then back to what it was.


Migrate OhMySmtp to Mailpace

Thank you both for the review. @taobojlen @kurtome
If not more changes are required I'll merge it in a couple hours and release a new version.


Interval broken with small step and > 24 hour time?

Steps to reproduce
pry(23)> s = Timex.now() |> Timex.shift(days: -2)
#DateTime<2018-10-14 21:49:33.322328Z>
pry(24)> e = Timex.now() |> Timex.shift(days: 2)
#DateTime<2018-10-18 21:49:38.646606Z>
pry(25)> Timex.Interval.new(from: s, to: e, step: [minutes: 1])
%Timex.Interval{
  from: ~N[2018-10-14 21:49:33.322328],
  left_open: true,
  right_open: false,
  step: [minutes: 1],
  until: ~N[2018-10-15 21:49:33.322328]
}

Description of issue
It seems that making a new interval with a step in minutes ignores the day; for example, see above were asking for an interval in minutes between 10/14/2018 and 10/18/2018 results in an interval between 10/14 and 10/15.  I suspect this is because of the small step-size, but it's unclear to me how to make a potentially multi-day interval.
My use case is being able to detect if two multi-day events overlap at all; is there a different API I should be using for that case with Timex?
Thanks!

Oi - my problem is using to: instead of until: as the param!  Happens every time - as soon as I post an issue the answer becomes obvious!


Questions about the SPD installation process

I find that the versions of MMDET3D 0.17.1 and AB3DMOT Dependencies are completely different when I try to install them. How do I solve this problem? Thank you very much.

Hi there! To address the compatibility issue between MMDET3D 0.17.1 and AB3DMOT, you may want to try using the following versions of dependencies, which are known to be compatible:

mmdet3d==0.17.1
scikit-learn==1.0.2
filterpy==1.4.5
numba==0.48.0
matplotlib==3.5.2
pillow==9.4.0
opencv-python==<IP_ADDRESS>
pyyaml==6.0
llvmlite==0.31.0
wheel==0.38.4

Please make sure to install these versions and check if the issue persists. If you encounter any specific errors or issues during installation, feel free to provide more details, and the community can assist you further.
Hope this helps! Let me know if you have any more questions.

Thanks for your reply. I will try relevant suggestions and continue to ask questions if there are any questions. Thank you very much.


encapp: tests: handle process exception in system tests

// ignore it, will rebase on top of the latest test changes


Feature: Mac OS Support

Mac OS support (Intel)
Apple Silicon support will be handled in #85

Implemented in #50


Dismissed Insights - UI defects

Please change the height of the bar to 44px

Please change the height of the button

Please fix the tooltip when text is too long

Broken layout

Align the arrow to the left red line and color it icon/disabled, and keep hover only for text

Make the footer height 44px, the same as the dismissed bar, so the content don't jump when user clicks on dismiss

Related to:
#611
#681


JobService not executed when specified to run in non default process

Version: com.firebase:firebase-jobdispatcher:0.5.0
Android Version: 4.4.4
If the JobService is specified to execute on a different process.
Like:
<service
      android:exported="false"
      android:name="com.mydomain.myapp.MyJobService"
      android:process=":myprocess">
      <intent-filter>
          <action android:name="com.firebase.jobdispatcher.ACTION_EXECUTE"/>"
      </intent-filter>
  </service>

the JobService is not executed.
The following warning message is reported:
W/FJD.ExternalReceiver: Unknown service connected
Job Scheduling:

       final FirebaseJobDispatcher dispatcher =
            new FirebaseJobDispatcher(new GooglePlayDriver(this));
        final Builder builder = dispatcher.newJobBuilder();
        builder.setService(MyJobService.class);
        builder.setTag(NOW_JOB_TAG);
        builder.setLifetime(Lifetime.UNTIL_NEXT_BOOT);
        builder.setTrigger(Trigger.NOW);
        builder.setReplaceCurrent(true);
        builder.addConstraint(Constraint.ON_ANY_NETWORK);
        final Job job = builder.build();
        final int result = dispatcher.schedule(job);

Have the same issue. If not same process, onStartJob won't be called at all.
Consider at least an error message in logcat?

@MartinNordh  there's a simple solution.
Override the manifest merger, by declaring the service "GooglePlayReceiver" with
the process attribute:
<service
  android:name="com.firebase.jobdispatcher.GooglePlayReceiver"
  android:exported="true"
  android:process=":myprocess"
  android:permission="com.google.android.gms.permission.BIND_NETWORK_TASK_SERVICE">
  <intent-filter>
    <action android:name="com.google.android.gms.gcm.ACTION_TASK_READY" />
  </intent-filter>
</service>

<service
      android:exported="false"
      android:name="com.mydomain.myapp.MyJobService"
      android:process=":myprocess">
      <intent-filter>
          <action android:name="com.firebase.jobdispatcher.ACTION_EXECUTE"/>"
      </intent-filter>
  </service>

In April 2019 we announced that Firebase Job Dispatcher would be deprecated today, April 7th 2020.  For this reason we are going to close all open issues and archive the repository.  We will also stop supporting FJD for new app installations (such as those targeting sdk versions greater than Android Q).  Existing apps that rely on FJD should continue to function as usual.
While we will no longer work on FJD, we encourage you to migrate to Work Manager which is a superior tool and fully supported by the Android team.
Thank you for taking the time to try the library and provide this feedback.  We sincerely appreciate your effort and we apologize for not addressing it in time.


Added dropdowns

Addresses #8
Summary of changes: Added the dropdown basic as well as dropdown states and dropdown styles

@andresgalante I've addressed the comments you made and beefed up the documentation

ok @andresgalante I think it's finally ready!

👍 yay! thanks @cshinn

LGTM


Run a sanity check on node startup and shutdown

We have had failed tests and benchmarks that resulted from wrong node configurations. This required several teams to investigate the source of the problem. We should perform a sanity checklist on startup and shutdown that would issue warnings when the configuration looks suspicious. Examples of said check include:

k is the same for all HFC eras.
Certain protocol versions (e.g. the HF triggers) are in ascending order.
Warning if certain (derived) parameters seem "unreasonable", e.g. the ledger snapshot interval.

I'm working on the assumption that this ticket can for all practical purposes subsumes #75 entirely, but please let me know if you don't think this is the case (@dnadales)
does this have any particular reason for being both a startup and a shutdown check? javier and I thought that running checks on startup would be sufficient, but did you have anything specific in mind that necessitates being checked at shutdown?

does this have any particular reason for being both a startup and a shutdown check?

None that I can remember. So running these tests at startup should be fine. Feel free to edit the title and description!

I edited the Issue description to include k < maxBound. EG their node would (slowly?) run out of memory due to the LedgerDB being effectively unbounded.

The first change to cardano-node has the new SanityCheckIssue exceptions simply being traced in the standard way, but in future it should terminate the node if a fatal SanityCheckIssue is detected. Whether or not an issue is fatal will be determined by a new function inside Ouroboros.Consensus.Block.SupportsSanityCheck.


Add highlight hexes on move

Solve #78

TODO:

Find, when and where walls become blue
Decide, how to highlight and dim different types of hexes

Solve todos

Close because I create issue for this problem and rename branch
New pr is https://github.com/Miol-Mor/battle-game/pull/141


one-dimensional data instead of pictures

I want to change the input to one-dimensional data instead of pictures. What should I do?

hi,Have you solved it?


Add a banner on JSF to "try the new UI" in the beta testing environment

Overview of the Feature Request
To encourage use of the new UI among beta users, we will add a banner in JSF that redirects to the new UI.

To link from the JSF side we can simply edit the description of the root collection.
Or we could add a custom header.
Or we could add a banner message.

What kind of user is the feature intended for?
Beta testing environment users
What inspired the request?

frontend weekly meeting

What existing behavior do you want changed?
None
Any brand new behavior do you want to add to Dataverse?
New banner on JSF to "try the new UI" in the beta testing environment
Any open or closed issues related to this feature request?
None

I went for a banner. Here's how it looks:

The link works. If we want, we could write more, maybe even link to https://dataverse.org/blog/restructuring-dataverse-ui-single-page-application or other docs. Maybe even this dataverse-frontend repo.
Here's the banner as JSON if we'd like to edit it:
$ cat try-react.json
{
  "dismissibleByUser": "false",
  "messageTexts": [
    {
      "lang": "en",
      "message": "Try the new Dataverse UI at beta.dataverse.org/spa!"
    }
  ]
}

curl -H "Content-type:application/json" -X POST "http://localhost:8080/api/admin/bannerMessage" --upload-file try-react.json


Refactor some files

I refactored some lines during to check this repository.
Please merge it if no problem 😉

Many thanks!!


Make action comment on Pull Requests once and edit

In https://github.com/EnricoMi/publish-unit-test-result-action/issues/28#issuecomment-710080411 we have discussed that the action should allow to configure it commenting on PRs only once and later edit that comment with latest test results. Then, the test results stay at the top of the PR (always at the same position) and the PR does not fill up with more and more test result comments.

I'd go for this one


Folding: handling blinders

We want to add blinders to our polynomials to handle commitments to zero.
We suppose the blinders are single scalars, and instead of committing to P(X), we commit to P(X) + r, which results in Comm(P) + r G where G is the first element of our SRS.
We must coin a new random blinder for each folding execution, and use it for instance and witness.
The blinders must be coined before computing and absorbing the commitments to the error terms

The default blinder should be at the folding scheme level. Updating it.


reflect: can't call methods on StructOf with embedded interface field

What version of Go are you using (go version)?

$ go version
go version devel +4c78d54fdd Thu Apr 30 22:06:07 2020 +0000 darwin/amd64

Same behavior with:

$ go version
go version go1.14.2 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/mitchellh/Library/Caches/go-build"
GOENV="/Users/mitchellh/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GONOPROXY="github.com/mitchellh,github.com/hashicorp"
GONOSUMDB="github.com/mitchellh,github.com/hashicorp"
GOOS="darwin"
GOPATH="/Users/mitchellh/code/go"
GOPRIVATE="github.com/mitchellh,github.com/hashicorp"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.14.2_1/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.14.2_1/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/xj/vc6mm2px1x5745hbvfpxkx5h0000gn/T/go-build768892661=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
Use reflect.StructOf with one field with an embedded interface and then attempt to call a method on that interface (with a non-nil value set). https://play.golang.org/p/GwTyV_AFWv6
Note that using an embedded struct (not an interface) works fine: https://play.golang.org/p/jVFT3pcw7ia
What did you expect to see?
The function to be called and the result shown: "42".
Or, if this isn't supported, a panic probably.
What did you see instead?
A segfault.

I'm not sure if this is supposed to be supported or not. I did a bit of digging and this whole method table setup for embedded interface types stood out to me as suspect, but I'm not familiar enough with the Go internals to really figure this out: https://github.com/golang/go/blob/master/src/reflect/type.go#L2466-L2471
This may also be related to https://github.com/golang/go/issues/16522 but I hope not!

This is not expected to work at present, but it should panic or fail rather than getting a segmentation fault.

@mitchellh https://github.com/TheCount/go/commit/b97bc377d699b1c624ce5fe12163c890ac6398f7 contains a fix at least for your particular issue.
What's going on:

The previous code used the interface method type to create the implementation with MakeFunc. However, the interface method type has no receiver, so it has to be altered to include the receiver. For that reason, the generation code had to be farther down within StructOf because the receiver type has not yet been constructed in the old location.
I think the previous code checked the kindDirectIface flag on the wrong kind of object to determine how the tfn and ifn method members are to be set: it should be checked on the created type. I'm not actually sure if kindDirectIface can ever be set on a struct with an embedded interface field since an interface is already two words long, but to be safe, I left it in for the time being (see https://github.com/TheCount/go/blob/b97bc377d699b1c624ce5fe12163c890ac6398f7/src/reflect/type.go#L2750).
As you noticed, the previous code uses the addresses of the reflect.Values tfn and ifn directly, which is wrong. We need to call the closure created by MakeFunc (located at tfn.ptr and ifn.ptr). However, method expects a direct code pointer. This is indeed related to #16522: we cannot give method the MakeFunc generated actual code pointer because then it can't find its closure context. So we store the closure elsewhere and use some special dispatch code instead as the method code pointer. The dispatch code then retrieves the closure and calls it.

This patch has some deficiencies:

The current solution works with single embedded interfaces (your case), but probably not with more complex embeddings (I haven't checked). Maybe this can be left for later.
The current solution works only on AMD64. This is because the special dispatch code uses some assembly hackery. It shouldn't be too hard to extend it to the remaining architectures, though I could probably use some help here as I don't have access to some of the hardware Go supports (or maybe I could try QEMU).
The current solution can generate only a finite number of working methods (812 to be exact, number is AMD64 specific) within the whole process. This is because dispatch entry points need to be mutually different so some information is available as to which MakeFunc closure to use. The number 812 is such that the entire dispatch code section fits into one memory page (4096 bytes); I had a vague idea using virtual memory mapping to remove this limit without using more physical RAM, but I'd need some help from a compiler or runtime expert to pull this off. Alternatively, we could just say, 812 methods (or whatever number) is enough for almost everyone and accept that there is a limit. Once the limit is exceeded, a method which always panics is created instead.

@ianlancetaylor Can you give me some advice how to proceed from here? Should I create a pull request despite the deficiencies? Or connect to some people first (who?)? TIA!

@TheCount This seems complicated enough that it's probably worth writing a little design doc for how to implement this.  I think the limitations you describe are a bit too extreme, as they will cause programs to work in limited cases but fail surprisingly in general.  Perhaps the best person to review a design doc would be @randall77 .  Thanks.

Sure, I can take a look when it is ready.

Sorry for dropping the ball on this. FWIW, in TheCount/go@155eedb4ba1c43c600e9b72124c3d5d8ebf2b5c4 I adapted my initial patch to the current version of Go. However, the simple little dispatch hack in ·dispatchLabel doesn't work any more because of the internal Go ABI change. Possibly it would now require special treatment similar to ·makeFuncStub or ·methodValueCall.
I'll see whether I can find the time to write that design doc. No promises, though. Sorry.

It looks like this no longer segfaults as of 1.22.0, the playground link returns a coherent panic message instead.

It looks like this no longer segfaults as of 1.22.0, the playground link returns a coherent panic message instead.

Yes, the faulty code was removed shortly after I polished up my patches :sob:
I mean, good that there's clean behavior now. Before I touch that code again, I should find out whether the MMU/page aliasing magic I have in mind even works in this context.


Control has already been added to another page: {}".format(ctrl)

Dear developer!!
I create a progress to wait for data to load, for that I used dialog and have await page.update_async , when opening a browser tab there is no problem but opening another browser tab gives an error as attached picture.
Please help me so I can run multiple tabs.
Sincerely thank you.

Please provide sample code.


Codespace symmetrical guacamole 6x5gx7ppp5c5jxx

This PR is not particularly helpful.  Please check the issues list for ideas which improve the project.


direct link "error loading data"

as we can't search room, I hope can we can join using direct link, in my case when I get direct link have "error loading data". I hope can fix in the next update.

Same error on Oneplus 8.

same error on redminote 5

Same error xiaomi mi 9 lite

I also has the same error.
I think that this error came from the fact that <event_hashid> from www.joinclubhouse.com/event/<event_hashid> is not the same as <channel_name> from www.joinclubhouse.com/room/<channel_name> which you can use to connect to the channel directly.  I suggest that you can fix this error by retrieving <channel_name> from <event_hashid> first.
I am not sure how to do it in java. But in python (based on stypr/clubhouse-py), I used event_data = Clubhouse.get_event(event_hashid=<event_hashid>) and then get <channel_name> from channel_name = event_data['event']['channel'] before connecting to the channel.
Hope this error can be fixed soon!

Ah so events and channels are different things after all.

@grishka, Yeap to start https://www.joinclubhouse.com/event/ID working you should grub channel_name for that.

Can I have an example link to test?

@grishka Sure, Here you go: https://www.joinclubhouse.com/event/VMR2qYMD
BTW - Thank you, Nice Job. Спасибо =)

@grishka One more - https://www.joinclubhouse.com/event/xjY9LOAN This one will start in half-hour.

Interesting. I'm sending get_event with event_hashid but I get a 500 error.


Update AlphaSlider  accessibility

Description
related https://github.com/yamada-ui/yamada-ui/issues/1505
related https://github.com/yamada-ui/yamada-ui/issues/1727
This issue aims to address accessibility deficiencies in our slider component as per W3C guidelines.
The component currently does not meet the accessibility standards outlined in the W3C documentation.
The goal is to ensure the component is accessible to all users, including those using assistive technologies.
Ensure that all unchecked items in this checklist should be addressed.

[ ] If the value of aria-valuenow is not user-friendly, e.g., the day of the week is represented by a number, the aria-valuetext property is set to a string that makes the slider value understandable, e.g., "Monday".

Problem Statement/Justification
improve accessibility
Proposed Solution or API
No response
Alternatives
No response
Are you willing to participate in implementing this feature and create a pull request include the implementation?
None
Additional Information
No response

This issue is related to #3019
If above PR is merged, this issue closed without additional address


Technical indicators support

Hello guys,
Excellent plugin :)
any ideas how to add support for https://www.highcharts.com/plugin-registry/single/16/technical-indicators !? in the code or with char.add_JSscript ?!
Thanks in advance

On a similar note, is there a way to load Highcharts plugins in python-highcharts?  For example, i would like to load the patterns plugin.

from highcharts.highstock.highstock_types import PLOT_OPTION_ALLOWED_ARGS

# a patch to add sma options. I don't know why the author make it UNFLEXIBLE.Totally anti-python.
PLOT_OPTION_ALLOWED_ARGS.update({"sma": {
    "linkedTo": basestring,
    "params": dict,
}})

from highcharts import Highstock

chart = Highstock()

# import the indicators.js
chart.add_JSsource('https://code.highcharts.com/stock/indicators/indicators.js')

# source data
chart.add_data_set(df1[['id','open','high','low','close']].values.tolist(),'candlestick','bchbtc',id='bchbtc')

# add sma indicators with source data
chart.add_data_set([], 'sma', 'sma', linkedTo='bchbtc', params={'period': 14})

chart

Hope it can help.


How to install Dependencies?

Ran into some trouble trying to set up utk-web:

Can you provide instructions on how to install the right MFTCLI version?
Can you provide instructions on how to install a version of uefi_firmware that has the --json argument flag?

I will update this ticket with any additional dependency issues I encounter.  Thank you!

Let's go slow on this. :-)
I have to admit that the mftcli I am using is a modification I haven't pushed yet - mostly due to lack of understanding of Go modules. There is https://github.com/Mimoja/MFT-AnalyserV2/tree/master/cmd/mftcli which I modified a bit on my local machine. I will publish more on this over time. If you understand Go modules well, I'd appreciate any help!
One core issue I faced was working with changes in other modules that mftcli uses. I would need to swap out dependencies, but actually want to develop in multiple places over time, and eventually upstream things again. That's a bit weird with Go mirroring DNS and web/git URLs.
I actually exchanged a bit with @mimoja but stalled for the time being. Now I had some ideas again and wanted to get back to the project. :-)

Oh, I did push some of my changes. It's actually mentioned in the utk-web README:
https://github.com/orangecms/MFT-AnalyserV2/tree/json

A note on uefi-firmware-parser: For Arch, there is a package in the AUR. Otherwise, I would ask you to follow your OS distro's way of installing software, or the main way Python offers for your system, or creating a package yourself, or working with the upstream repository directly. I cannot offer much support for the complexity it implies, unfortunately.
Now regarding mftcli: I have started to work on a branch for the wasm back-end, where I am adding actual Go code to this repository, which will replace mftcli.
There is already code pulling in utk itself. :-)

Update: We now have a fully self-contained app; the prototype as been archived to the utk-web branch for reference.
Releases can be downloaded prebuilt for Linux from https://github.com/fiedka/fiedka/releases.
The current code already has capabilities (though limited) to parse and visualize AMD/PSP. Further work needs to be done in https://github.com/linuxboot/fiano which is the tooling for the backend that is running in WASM. Closing this now.
Please let me know if you would like to contribute somewhere, either in the respective issue here or add a new one. Thank you!


Merge some of your projects

Hello Julien,
No issue, no pull request, just a thought. My name is Jean-Francois, I live in France, and I am a (low level) astrophotographer.
I use NINA and Ascom devices. My setup is EQ6-R / Askar FRA600 /ASI2600color.
I was seeking for an automated dust cap for my telescope and I found your videos ! (Dark Sky Geek)
You did such a great job for 2 projects in which I am interested : dust cap and focuser.
I am not a developer and I discovered your channel today !
I bought hardware stepper-motors drivers called TIC from Pololu and I did a small GUI to drive manually stepper-motors.
This was easy, but to integate TIC with NINA (native or Ascom) is much to dffificult for me.
A stepper-motor and a stepper motor is much less expensive that any Zwo, Pegasus or else equipment.
If for any reason, you plan to use Pololu TIC hardware to drive stepper-motors for your new projects, please let me know.
Sincerely
Jean-Francois
<EMAIL_ADDRESS>

Thank you for your interest. I do not plan to use "Pololu TIC" hardware (I have never heard of that brand before). I generally stick to dirt cheap, standard stepper motors or servos that you can find anywhere online. That being said, adapting what I've done to your specific hardware shouldn't be too difficult. Best of luck!

Thank you for your quick answer. By the way I choosed TIC + stepper-motors because I can buy both on Amazon for around  50 euros (plus a 3D printed hardware for virtually nothing) when a focuser from ZWO or Pegasus costs more than  250 euros
Clear skies !


Remove Emotion dependency

PatternFly's StyleSheet functionality, which depends on Emotion, is being dropped from react-styles in PatternFly v4. This is due to a security issue related to using Emotion. The Koku UI should strongly consider dropping Emotion as well, in favor of using CSS style sheets.

This code includes

Isn't it related to https://github.com/project-koku/koku-ui/pull/1418 ?

@boaz, 1418 removed PatternFly's StyleSheet object. However, we still depend on Emotion to add CSS style in a few places.

Created Jira issue: https://issues.redhat.com/browse/COST-49


[BUG] Pandoc cannot find image file unless full relative path is used

Hello! I was trying to convert one of my files to PDF, I received the following error

The thing is, that picture isn't in the root folder, it's in a nested folder called __Assets.
The full path is C:\Users\Username\Documents\[[Obsidian]]\Folks and Tales\Ash In The Snow\PCs\__Assets
If the full relative path (Ash In The Snow/PCs/__Assets/Axiom.png) is specified, then works. The only way to do this in obsidian (that I know of?) is by having multiple files by the same name, which forces obsidian to have a full relative link to that exact file.
Is there any way to work around this issue? I can provide any files or my exact file structure if needed.

Came here to report the same issue. Hope it can be solved!

I had this same issue.
My problem was that my images were marked in Obsidian as WikiLinks, so pandoc couldn't reason out the file path.  I converted all my Vault Wikilinks to CommonMark format (there's a plugin for this), and any export at any part of my vault seems to succeed so far (I've tried approx 50).  Hope that helps

subscribing to this one. when in Obsidain Options, Files and Links, I have defined a subfolder name for images, "attachments" for example, all pictures are in /attachments folder. Obsidian sees them and displays render them correctly. Pandoc plugin cannot see them and complains those binaries do not exist.

I had a similar problem on Macos. Hope muy solution translates to Windows... I tried to specify the image folder by using the parameter in the plugin settings:
--resource-path='[abs_path_name_containing_spaces]'
but it failed with the (no such file or directory) error. Even when I escaped the spaces. Using the same parameter on commandline with pandoc it did work. The problems seems to be that parameters cannot contain spaces when you specify them in the settings for this plugin. The path to the Obsidianvault when placed in iCloud allways contains at least one space. So removing the spaces in the path is impossible. I discovered a pandoc parameter so worked around it by specifying instead:
--defaults=[path_to_yaml_file_no_spaces]
I put an   .pandoc-obs.yaml file (you can use any filename withoud spaces) in my homedir (no spaces in the path there). And the yaml-file contains the parameter:
resource-path: ["/Users/ncdegroot/Library/Mobile Documents/iCloud~md~obsidian/Documents/Notes/00-Meta/03-Attachments"]
That worked. At least after I changed the ![[image_file_name_no_path]] links to the more MD standard !()[image_filename_no_path] to get the image to appear in the generated .docx file.  To create .PDF I still need to install some more latex stuff, but thats another story.
Hope its helps!

Your solution works on Windows, thank you! Here's what we did:

Converted [[Obsidian image links]] to ![Markdown image links](/path/to/image.png)
Created C:/Users/Joshu/pandoc-defaults.yaml, containing this line:
resource-path: ["C:/Users/Joshu/OneDrive - DIU/Josh Stuff/Obsidian/Attachment Folder"]
Added this line to "Extra Pandoc Arguments" in the Pandoc plugin settings:
--defaults=C:/Users/Joshu/pandoc-defaults.yaml

After following these steps, Pandoc was able to export Markdown files into .docx. Only image links that specified a size at the end (image.png|300) wouldn't work; we'll just clean those out manually.
Thanks for your help with this workaround!

While Obsidian is currently not able to copy images from a note, I am currently using this Pandoc plugin to export the images from a note so I am able to publish it on a different platform, in my case Confluence.
A recent extension was added to Pandoc to be able look files relative to the note. To implement this on this plugin I modified line 11286 in main.js file file: inputFile, format: 'markdown', to file: inputFile, format: 'markdown+rebase_relative_paths',. Next, all new links format are set to "Absolute path in vault". This is a very rough workaround. If someone familiar concatenating the links in this plugin to avoid messing with the new links format setting please add it to this plugin. Thanks.

Adding --from markdown+rebase_relative_paths to the "Extra Pandoc arguments", and setting "Export files from HTML or markdown?" to markdown as (implicitly) suggested by @ZeroCommits, solved it for me!

I have completely left behind ObsidianPandoc specifically because of this issue, and because it is way beyond my level of knowledge, and I simply don't have the time to learn it for this issue alone.
As an alternative workaround, I am now using ObsidianHTML to compile a standard markdown file from a note. This I then convert to R-Markdown (just changing file type to .rmd), use my own (windows-only) script to convert the image src's created by ObsidianHTML to valid knitr-compatible image-embeds for RMarkdown and optionally auto-knit it to the desired output formats.*
It's an arguably weird workaround, but I can now ditch ObsidianPandoc completely with minimal overhead (exporting to standard md can take a few seconds, running the script can take a few seconds) and have the side-advantage of being able to leverage R in my manuscripts - which I regularly have to do anyways.

*Not completely done with the project yet sadly, a few niceties are left on the list.

The enhancing export plugin fixes this.

I have a latex hack (to be added in yaml) for compiling from various folders, I often using includes for figures and move files, you need to adjust it to your paths accordingly, you can add as many path variation as you want
header-includes: |
  \graphicspath{{figures}{chapters/figures}{../figures}{chapters}}

I had a similar problem on Macos. Hope my solution translates to Windows... I tried to specify the image folder by using the parameter in the plugin settings:
--resource-path='[abs_path_name_containing_spaces]'
but it failed with the (no such file or directory) error. Even when I escaped the spaces. Using the same parameter on commandline with pandoc it did work. The problems seems to be that parameters cannot contain spaces when you specify them in the settings for this plugin. The path to the Obsidianvault when placed in iCloud allways contains at least one space. So removing the spaces in the path is impossible. I discovered a pandoc parameter so worked around it by specifying instead:
--defaults=[path_to_yaml_file_no_spaces]
I put an  .pandoc-obs.yaml file (you can use any filename withoud spaces) in my homedir (no spaces in the path there). And the yaml-file contains the parameter in YAML syntax:
resource-path: ["/Users/ncdegroot/Library/Mobile Documents/iCloud~md~obsidian/Documents/Notes/00-Meta/03-Attachments"]  That worked. At least after I changed the ![[image_file_name_no_path]] links to the more MD standard !()[image_filename_no_path] to get the image to appear in the generated .docx file. To create .PDF I still need to install some more latex stuff, but thats another story.
Hope its helps!

Also MacOS, but these method does not work for me.

Works perfect with the yaml setup, thx. With the plugin "Link Converter" can help here. oneclick on your file, to set all links to realtive path... perfect.

Works without this yaml too. only convert links to relative pathes. (![[../../../Extra/Attachments/Pasted image<PHONE_NUMBER>2709.png]]) With the plugin "Link Converter" can help here. oneclick on your file, to set all links to realtive path... perfect.

This worked for me, however, it does not pick up the size argument in the link.


Support for Async/Await in Actions

XamlBehaviors are really powerfull and more lately noticed that some designers are going faster with productivity with the more small components made into it. But we are running into some small behaviors/actions issue when it comes to actions that require async/await.
A common scenario that we have is that the we have a lot of small actions that help the designers just plug in the feature where needed and out later instead of in code, this really sped up a lot of implementations and requirements but we are having issues when the designers need to do few actions in a row. For example assume there are 3 actions required to be executed but the second one is a IO operation with Async execute only that you either need to await or .Wait/Result and the third action relies on it. Currently you now either perform a blocking UI action while you want fluent UI or do in code behind (and dozen of places).
I would like to propose an addition of an IAwaitableAsync with the method Task Execute(object sender, object parameter);
SideNote: Maybe even introduce a base class like with Behavior, one named BehaviorAction and AwaitableBehaviorAction
There are few critical changes that would be required though, like the Interaction.ExecuteActions would have to do a blocking support or replaced by a new method named Interaction.ExecuteActionsAsync.
All Behaviors of the package would have to use that method to support Async/Await.
CallMethodAction should also ignore the check if a method returns nothing to match the perfect signature and in case it returns a Task it should await it.

Bumping this as I came across this issue, or am I missing something? Using EventTriggerBehavior -> CallMethodAction does not support async methods as far as I could tell.


chore: Renormalize line endings

Ran git add --renormalize .

Didn't really notice, just something I run because it causes issues/large diffs when doing find/replace stuff when there are encoding issues


Added missing ProducesResponseType, aligned response types

Fixes #10613
Also added the types in the ResponseType for .NetCore > 2.1

@scottaddie I overlooked the paragraph that the 200 return code is implied by the ActionResult. Personally I would still add it for clarity (to other devs) showing that it returns both 200 and 404, not only 404.
However for the sake of testing, I removed ALL ProducesResponseType attributes in the PetsController for .NET Core 2.1 (not 2.2), cleaned and rebuilt the solution. All response types and codes are correctly shown in Swagger. Makes me wonder if they're all added automatically because of the code paths? If so (which would be nice), which are the cases where they still have to be added manually?

@bartlannoeye One of the benefits of using ActionResult<T> is reducing "attribute soup" on actions. There's absolutely nothing wrong with being explicit. Continue to do that in your code if that's your preference. We choose not to use the attribute in the docs so that you're getting the full benefit from ActionResult<T>.
The 200 status code is an assumption the API is making. In other words, the API isn't intelligently scanning the code paths in the action to determine that.

Thanks for the response. How am I getting the other response codes even though I removed all attributes in the .NET Core 2.1 project? If I remember correctly, configuration in the Startup to handle everything was only added in .NET Core 2.2? Am I overlooking something?

@bartlannoeye Do you have a sample project on GitHub that reproduces what you're seeing?

@scottaddie 	is this ready to merge?

Can't reproduce anymore after I restarted VS, assumes 200's now as you explained.

@scottaddie is this ready to merge?

@bartlannoeye Thank you for all your work on this PR! Your changes will go live within the next 24 hours.


Fix MSSQL JSON and queries list API

Adds work around for lack of proper JSON support in SQL Server.
While the JSON datatype can be added to SQL Server as a string, Sequelize does not parse that string to an object. As a solution, the field checked and parsed in application code after the fact.
This isn't great, but it works for now.
This also fixes a SQL incompatibility in the queries list API route to address the fact that SQL Server does not support LIMIT/OFFSET.

@eladeyal-intel I'm seeing duplicate actions being run on my end for both push and pull_request triggers.
Looks like others hit this https://github.community/t/duplicate-checks-on-push-and-pull-request-simultaneous-event/18012/2
Do you see any issue changing this to pull_request only or potentially adding the if-check on repo name?

I'm not very exipirnces with github actions. In #736 I tried to do what you suggested - let's see if it helps.

@eladeyal-intel yeah I was thinking about that and the switch to PR only.

I think mssql might be green now 🤞

👏


MudTextField doesn't re-renders the text inside the field box

Things to check

[X] I have searched the existing issues for this bug
[X] To rule out a caching problem I made sure the bug also happens in an incognito tab

Bug type
Component
Component name
MudTextField
What happened?
Hi,
When you are using Value and ValueChanged parameters on MudTextField. If you do any transformation to the variable in ValueChanged, it doesn't reflects it back in the box.
It is better if i show you.
In the image below you would see that it works perfectly fine on Mud playground. When you enter the text "hello1", it uppercases it and showcases the change both in the variable as well as the frontend text field box.

However, it is not the case when you're actually using MudBlazor on your project. To showcase this, i made a new Blazor project on NET8 with the latest version of MudBlazor.
The code:

How it looks in the UI:

As you can see that, it does change the variable. However, it is not reflecting the transformation inside the MudTextField box. The same case happens with MudInput as well.
Kindly, please fix this issue. If you have any questions related to from my side, please do ask.
Thank you.
Expected behavior
In the image below you would see that it works perfectly fine on Mud playground. When you enter the text "hello1", it uppercases it and showcases the change both in the variable as well as the frontend text field box.

Reproduction link
https://try.mudblazor.com/snippet/muQovvwLIQllkrWk
Reproduction steps

Have NET 8 project
Install mudblazor on it and its service in program.cs
Copy paste the code onto a new file that i have shown
See it in the /testing page

Relevant log output
No response
Version (bug)
7.15.0
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On which operating systems are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Thank you for reporting this bug. The description was helpful and I can reproduce.
I will try to fix it this weekend, but no idea for a potential release date. I will ask to the team.

MudTextField (and MudInput) has the property MudTextField.TextUpdateSuppression :

Prevents the text from being updated via a bound value. Defaults to true. Applies only to Blazor Server (BSS) applications. When false, the input's text can be updated programmatically while the input has focus.

You can disable it to get the expected behavior.

Thank you, that works.

The property MudTextField.TextUpdateSuppression doc is explicite about it, but the information is missing in the main MudTextField documentation.
@henon, do you think we should add an annotation in the main documentation?

Yes, we should.


Images not loaded or loading too slow

Happens on on HTC one mini 2 - Android 4.4.2. Works well on Nexus 7 (Android 6.0.1)
See it in SDK Examples - ListView -> Item Layouts -> Linear/Grid
https://github.com/telerik/nativescript-ui-samples
"nativescript": {
        "id": "org.nativescript.sdk",
        "tns-android": {
            "version": "2.1.1"
        },
        "tns-ios": {
            "version": "2.1.0"
        }
    },
    "dependencies": {
        "nativescript-fresco": "^1.0.3",
        "nativescript-telerik-ui-pro": "1.2.0",
        "tns-core-modules": "2.1.0"
    },
    "devDependencies": {
        "babel-traverse": "6.9.0",
        "babel-types": "6.9.0",
        "babylon": "6.8.0",
        "filewalker": "0.1.2",
        "lazy": "1.0.11",
        "nativescript-dev-typescript": "^0.3.2",
        "typescript": "^1.8.10"
    }

A quick update on the issue.
The issues reproduced when setting the imageUri to {N} resoruce files (imageUri="res://myImage").

Looks like this is an old and closed issue but I'm currently having this issue only when loading images from res://image.  I'm using {N} with Angular

@9thwall are you using the latest 1.0.12 version of the plugin. If so can you also try to reproduce it with the previous 1.0.11.
In order to install a specific version simply do:
tns plugin add<EMAIL_ADDRESS>If you continue to experience this issue can you give me some more information regarding your application. A runnable project or code snippets would be great help.

Hi Vladimir,
I had 1.0.11 installed. I upgraded to 1.0.12 and the issue still exists.
If this is suppose to work I'm assuming this is most likely something simple I am doing wrong.
My code is like this..
<FrescoDrawee #avatar [imageUri]="user.picurl" row="0" col="0" backgroundUri="~/images/placeholder-avatar.jpg" roundAsCircle="true" width="38" height="38"></FrescoDrawee>
I console.log user.picurl and sure enough it's showing the correct image link.
Strange thing is that even if I hard code the link in like this..
<FrescoDrawee #avatar imageUri="res://me" row="0" col="0" backgroundUri="~/images/placeholder-avatar.jpg" roundAsCircle="true" width="38" height="38"></FrescoDrawee>
It still doesn't show the image.
I should mention that that I really do have the image named "me.png" and I hope there isn't any restrictions on using short image names.

Sorry about this.. hard coding the value directly like imageUri="res://me" does show the image.
Only when the value is set dynamically like [imageUri]="user.picurl" or imageUri="{{user.picurl}}" it doesn't show.

Geez... Nevermind.
It's working.... It was my bad code. Sorry to bug you about this...


Support default creation of symbolizers using 'auto' syntax

Say I want to create a point without supplying a png/svg file. I know that Mapnik supports creating a PointSymbolizer without any arguments and will internally just create a simple black square. We should have a simple way to request this in carto that can be used for other symbolizers that have a no-arg default contructor, including line, polygon, and markers symbolizer (and perhaps more in the future). Maybe something like:
    #city {
        point: auto;
    }

or
    #city {
        point-visibility: auto;
    }

Fixed by #477.


Tests break when adding relative paths

Describe the bug
When adding tsconfig paths, like suggested in https://vitest.dev/guide/common-errors#cannot-find-module-relative-path, tests break
To Reproduce
1 - Run npm add -D vite-tsconfig-paths
2 - Add a vitest.config.ts file, with the following content:
import { defineConfig } from "vitest/config";
import tsconfigPaths from "vite-tsconfig-paths";

export default defineConfig({
  test: {
    clearMocks: true,
    coverage: {
      enabled: false,
      // all: true,
    }
  },
  plugins: [tsconfigPaths()],
});

3 - Reload VSCode
Expected behavior
Tests should still work.
Environment
{
  "System": {
    "OS": "macOS 14.2.1"
  },
  "Binaries": {
    "Node": {
      "version": "20.9.0",
      "path": "~/.nvm/versions/node/v20.9.0/bin/node"
    },
    "Yarn": {
      "version": "1.22.19",
      "path": "~/.nvm/versions/node/v20.9.0/bin/yarn"
    },
    "npm": {
      "version": "10.1.0",
      "path": "~/.nvm/versions/node/v20.9.0/bin/npm"
    }
  },
  "IDEs": {
    "VSCode": {
      "version": "1.85.1",
      "path": "/usr/local/bin/code"
    }
  },
  "npmPackages": {
    "vite": {
      "installed": "2.9.16",
      "wanted": "^2.9.9"
    },
    "vitest": {
      "installed": "1.1.0",
      "wanted": "^1.1.0"
    }
  }
}

Can you please retest with the latest version of the extension?

I can confirm this seems to work fine on latest versions


More flexibility for URL parameters

Use query parameter in route path if no URL parameter is explicitly available.

We had this in Slim 2, maybe an oversight that we missed it out in Slim 3. I will check the code again and look at some issues to see if there was a reason its not in Slim 3.

Any news for this issue besides if the initialization should be different?

After reading through the code more now I understand what you are trying to do, what is the use case for this as we intended for it to work the way it currently does as its more clear that $data will replace placeholders in path while $queryParams will be added at the end.
I think the current setup is as flexible as it gets, your change is limiting that flexibility by checking in both arrays while not necessary. If the developer wants the placeholder replaced then place the variable in the $data array otherwise if they want it added to the end do so in the $queryParams array.

I agree with @silentworks. At the moment there are two parameters, $data and $queryParams, which are each responsible for one thing only which makes sense.

For us, the separation of $data and $queryParams is rather artificial because it limits what our users (developers using the Aimeos e-commerce library) can do. This is most apparent in the generated URLs if you want to create an online shop based on Aimeos.
We have a few fixed parameters like site, locale and currency which fits very well into the $data array to create URLs like http://localhost/shop1/en/. But then, there are dynamic parameters too like the f_catid, f_name and f_sort for the list pages or d_prodid, d_name and others for the detail pages. These parameters do not only depend on the page but also on the used parameters in the concrete implementation.
To get nice looking URLs, the default routes provided in the aimeos-slim package to to fit for most cases but can be adapted by the developer, e.g.
/list[/{f_catid:[0-9]+}[/{f_name}]]
/detail/{d_prodid:[0-9]+}[/{d_name}]

As we don't have any control over the needed parameters for the $data array, we simply push all query parameter into $data. But if they are not used in the URL, they must be added as query string as well. There's no known way to find out if a parameter will be used as URL or query parameter so our URLs currently look like
/list/1/Home?f_catid=1&f_name=Home&f_sort=price
/detail/1/Test?d_prodid=1&d_name=Test&l_pos=0

which are very ugly and no one would like to use that in his online shop. We can't leave out the query parameters completely because then no product sorting would be possible and some parts wouldn't work at all.
Our use case is probably not the most common one where a single developer has full control over everything but we think it's a valid use case. In the current form, nobody will build an e-commerce application based on Slim and use the Aimeos components due to the URLs.
We can only ask you to rethink your decision. There's no compelling advantage of separating URL and query parameters for URL generation, but only major drawbacks like shown above. Also, no other framework does such a separation.

Thanks for the use-case and detailed explanation.
If we were to go down this route, then we'd deprecate the use of $queryParams so that the approved use of relativePathFor would be relativePathFor($name, $params=[]). That would imply that pathFor's documentation would also need changing.
I've reopened for discussion on this. Certainly, I don' t want $data and $queryParams to be interchangeable.

In that case, the implementation can be simplified :-)

Thoughts, @silentworks ?

@silentworks What are your thoughts? This change is rather important for us :-)

@silentworks @akrabat Any decisions how to proceed with this issue?

@aimeos This is a breaking change that will have to wait until 4.0, it would lead to an unexpected behaviour for the developer, potentially exposing data they didn't wish to expose in the querystring.
An alternative would be to provide a setting option to turn this on and this would be set to off by default. This I could merge in right away.

@silentworks Wouldn't an additional optional parameter for the relativePathFor() and pathFor() methods be a better alternative? The router class shouldn't depend on any external configuration settings IMHO and using a setCombinedParameter() seems to be only the second best option.

@aimeos what would be the default behaviour with this optional parameter?

@silentworks The current one to avoid BC breaks. Using the new behavior has to be enabled explicitely.

@aimeos I am happy with it working like that. Update PR and we will get it merged in if no-one has any objections. @codeguy @akrabat

@silentworks @codeguy @akrabat Maybe we've found a very elegant way to extend the functionality without the need to change the interface. It uses only a subtle, undocumented difference when handling route attributes: Instead of dropping surplus route attributes silently, they are added as query parameters.
You now have these behaviors (route, data, queryParams -> result):

/{name} + ['name' => 'josh'] + [] -> /josh (same as before)
/{name} + ['name' => 'josh'] + ['a' => 'b'] -> /josh?a=b (same as before)
/{name} + ['name' => 'josh', 'c' => 'd'] + ['c' => 'e'] -> /josh?c=e (same as before)
/{name} + ['name' => 'josh', 'c' => 'd'] + [] -> /josh?c=d (new, before: /josh only)
/{name}/{c} + ['name' => 'josh'] + ['c' => 'd'] -> exception (same as before)

What do you think? If it's still too much of a change, we can still add the additional parameter to activate the new behavior.

@silentworks Are you fine with the approach?

While i'm not entirely sure if my opinion on this will change anything, but i'm better off expressing it than not. Fore warning, this is in reference to the edge case of the second iteration. I do not know (or care to check, since if it still exists, the comment will be relevant) if the third iteration will have this edge case as well.
I'm personally against such a change, because at the core level you are essentially denying the usage of certain Query Parameters, purely because they have the same name as something in the Route Parts, as shown in the edge case of /user/{name} having problems with /user/john?name=test in the second iteration.
In relation to the third iteration, one of the tested examples was not the edge case of a Route Param and Query Param having the same name like /{name} + ["name" => "john"] + ["name" => "test"] = ??. Only missing a route part and trying to provide it in query params was shown.
Even if the worst case scenario was avoided and @geggleto's (imo) horrible suggestion of "throwing a Router Exception because the Request data had a key that is the same a routes part key" isn't implemented, the edge case of which gets what still exists with the second iteration still exists, however if the fore mentioned test solves this problem with the third iteration, and more extensive tests are conduct when provided keys that exist in the route parts as query params are done, than i see no problem, however since the entire problem about this issue is that providing the data for both the Route Part and Query Params produces duplicates in the url, as expected, i doubt that it will work as it currently does (Which would make it a BC change, since it would break existing applications that have the same key in both route parst and query params).
While this is kind of irrelevant, it does explain my opinion on this matter, and why i'm against it. The way the routes are setup for this example usage, in my opinion, goes against the entire routing philosophy of slim, which has always been SEO friendly urls, even if urls that were not SEO friendly were supported it was never documentation, shown, or even mentioned anywhere. So while this is a perfectly good example of using optional parts, trying to polyfill them with Query Params to make them optional goes against what i believe, and what appears to be the routing philosophy of slim.
I'm sure i've said some things here that some people will not like, and i'm arguing the entire point of this Pull Request that the core maintainers have pretty much accepted, but none the less thank you for reading my comment, and opinion on this matter.
Also @aimeos sorry, but i, personally, don't think Slim will benefit from adding this, and "just because everyone else is doing it you should too" is not a valid reason, and i do believe it's just peer pressure. Maybe you should consider another approach, like extending the Router, and providing your modified relativePathFor function to uses, which would just entail using $container->extend(callable) to inject it into Slim.

There's no known way to find out if a parameter will be used as URL or query parameter so our URLs currently look like.

Instead of altering current behaviour wouldn't it be better to add a way to get the placeholder names for a given route. This would solve the root of the problem since developer would have information which parameters belong to $data and which belong to $queryParams.

Changes Unknown when pulling 56ae4474b36ac0fa727b676a49b5fb539a717b6c on aimeos:3.x into ** on slimphp:3.x**.

Changes Unknown when pulling 3bf5be28f19c6d678ddf8cfbe7cd379f09862cdc on aimeos:3.x into ** on slimphp:3.x**.

Coverage increased (+0.004%) to 96.607% when pulling 872a9d0c5f5e6737c89c1ea36fbf39fcfde7ea16 on aimeos:3.x into 805150e058b48471426d2deadc247e00d0f38c76 on slimphp:3.x.

A quick survey suggests that Symfony, Yii2 and Laravel will automatically convert extra route parameters to query parameters while ZF2 and CakePHP require you to separate them out.

So this is a matter of implicit vs explicit. I am always for the explicit way of doing things, but I see no harm in this being one of those occasions we do it implicitly.

Coverage increased (+0.003%) to 97.315% when pulling c71d8805cddc5262d857fc0f5a27f86e34cbf2f8 on aimeos:3.x into f68e58cee9f6213647f88cfc62fb05240b01e3a7 on slimphp:3.x.

I think I prefer explicit too.
However:

Instead of altering current behaviour wouldn't it be better to add a way to get the placeholder names for a given route. This would solve the root of the problem since developer would have information which parameters belong to $data and which belong to $queryParams.

Should be implemented.


Unclear documentation regarding memory ownership on failed set0 calls.

The OpenSSL manpage for RSA_set0_key states:

Calling this function transfers the memory management of the values to the RSA object, and therefore the values that have been passed in should not be freed by the caller after this function has been called.

However, in practice, memory management is transferred only success of the function. In this particular case, this is relevant only if only one of n or e is passed, and the corresponding value is NULL in the RSA structure initially; in this case the function returns without modifying the RSA structure or freeing the provided value.
Please clarify in the documentation the intended ownership of memory for objects passed to a set0 routine that returns failure (and, if appropriate, correct the implementation as well).

On failure, nothing has been modified.

I agree with Kurt. If a function fails then it doesn't perform any of its advertised operations. I don't think there is a need to point this out for the large number of functions in our man pages. Closing this.


Fail2ban with HTTP status and logger (#2291 #2384)

I'm not sure if this patch is acceptable. I never wrote golang before but I was needing this features.

introduce a new context function RenderWithErrStatus that allows to
specify an HTTP status when rendering errors on form datas

status 403 set on repository permission denied
status 401 set on authentication failure
status 401 set when no associated email found on forgotten password
procedure

add warning log message, containing client IP address, for
authentication failure
add warning log message in routers/user/auth.go AutoSignIn function
add http_status int in returned values for routers/user/auth.go
AutoSignIn function
change default log level to Info

:+1: for this.
@yweber Do you also have a Fail2Ban RegExp for this?
If so you might also want to add it to the doc somewhere.

@rugk Thx ! And on my installation I don't use fail2ban directly on gogs logfiles but on the nginx reverse-proxy access log ( I filter on HTTP status ). I have written and quickly tested this RegExp for gogs logfiles : ^[^\[]*\[W\] <HOST> Authentication failure for user .*$
Writing this RegExp I figured out that on password retrieval failure (when password retrieval is asked for a non registered email address) no log message is produced and that a different HTTP status is returned (401 instead of 200). This behavior can allow an attacker to find if an email is registered or not.
What do you think ? Is it a better idea to invert this behavior, returning HTTP 200 and producing a log message containing client IP when password retrieval fails ?

Is it a better idea to invert this behavior, returning HTTP 200 and producing a log message containing client IP when password retrieval fails ?

I think in both cases either 401 or 403 should be returned and you can additionally write the log file.

Please sign the CLA.

Any update on this?

+1

Maybe @yweber could try to introduce this PR in the fork @gitea.

I find it of poor taste, @rugk, to come to a project, and try to send people to another. You could have opened an issue on Gitea about this, and mention the OP in it.

It does not really make a difference. Also it's @yweber' choice…

The merge commit https://github.com/gogits/gogs/pull/3357/commits/47fc72f3d1422426866d430d806ac44d50081cdd is just about solving merge conflicts and making the compilation (and apparently the software) work :
I didn't take the time to check if some other 4xx, 5xx HTTP status or other log lines should be emitted with code changes. I also did not check if a function was implemented achieving the same goal than
RenderWithErrStatus.

Why hasn't this made it into the source?  This is a useful feature.


img src=""

I have some stuff that needs to fetch images dynamically, is there any way to wait until it all is rendered before printing?

This would help?
https://github.com/pofider/phantom-html-to-pdf#programmatic-pdf-printing

I also replied here: https://github.com/IonicaBizau/nodeice/issues/11#issuecomment-163232839
Maybe it helps someone in the future. :smile:

If there where images in your task rows of your example and the task rows
was dynamic where would you place the onload script?
:)
Sorry for all the questions! ;)
onsdag 9 december 2015 skrev Ionică Bizău<EMAIL_ADDRESS>
I also replied here: IonicaBizau/nodeice#11 (comment)
https://github.com/IonicaBizau/nodeice/issues/11#issuecomment-163232839
Maybe it helps someone in the future. [image: :smile:]
—
Reply to this email directly or view it on GitHub
https://github.com/pofider/phantom-html-to-pdf/issues/17#issuecomment-163245283
.

seeing the referenced issue i think this is solved.. re-open if not :)


seaweedfs: 3.44 -> 3.45

Description of changes

https://github.com/seaweedfs/seaweedfs/releases/tag/3.45
Things done

Built on platform(s)

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

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

Result of nixpkgs-review pr 224447 run on aarch64-darwin 1

1 package built:

seaweedfs


Update weave-network-policy.md for zh

Relaled to #24100
/language zh

/lgtm thanks

Thank you.
/approve
/lgtm


refactor: initialize chain without user-provided genesis transactions file

What this PR does / why we need it?
Initialize chain without user-provided genesis transactions file.
The data in the transaction file will be re-written, so this file is useless, it could be removed.
:information_source: This change doesn't change the preset hashes in unit tests.
CI Settings

CI Usage
Tip: Check the CI you want to run below, and then comment /run-ci.
CI Switch

[x] Web3 Compatible Tests
[x] OCT 1-5 And 12-15
[x] OCT 6-10
[x] OCT 11
[x] OCT 16-19
[x] v3 Core Tests

CI Description

CI Name
Description

Web3 Compatible Test
Test the Web3 compatibility of Axon

v3 Core Test
Run the compatibility tests provided by Uniswap V3

OCT 1-5 | 6-10 | 11 | 12-15 | 16-19
Run the compatibility tests provided by OpenZeppelin

/run-ci


Added action to increment app and helm chart versions

Closing PR, and moving the action to greenforce-frontend


Message routing: Bug in has_sys_id ?

While writing unit tests for the endpoints, I stumbled over this line in Endpoint::has_sys_id():
https://github.com/mavlink-router/mavlink-router/blob/85d2a77fb93c2d54b10fcc09369eafa3464b057d/src/endpoint.cpp#L418
In my understanding, the condition should only be true, if the sysid is in the list of _sys_comp_ids.
But this line is always true since the bitwise OR will set all bits of sysid and then compare with itself!?

See https://github.com/mavlink-router/mavlink-router/commit/0931d6c28bf166e0342e09a0b16400e073801c30 for a fix (when I'm right with my assumption about the intended behavior). I'll add this to PR #305 after finalizing the discussion on the spec.

Looks like a bug indeed. should be (id >> 8) == (sysid & 0xff)

fixed by #328


static int64 GetDirectConvCost - Integer overflow

/tensorflow/core/kernels/deep_conv2d.cc: ln 74,
static int64 GetDirectConvCost(int filter_rows, int filter_cols, int in_depth,
int out_depth, int out_rows, int out_cols) {
return filter_rows * filter_cols * in_depth * out_depth * out_rows * out_cols;
}
Can lead to integer overflow and weird results
I think, it should be smth like that
return (int64)filter_rows * (int64)filter_cols * (int64)in_depth * (int64)out_depth * (int64)out_rows * (int64)out_cols;

Ok, I will create a PR tomorrow morning. Thanks for reply!
Ср, 28 авг. 2019 г. в 18:24, Yong Tang<EMAIL_ADDRESS>
@iur-kvasniuk https://github.com/iur-kvasniuk That makes sense. Are you
willing to create a PR for that?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/32045?email_source=notifications&email_token=ANAZPT44S3JEJKU4WRLX3RTQG2KC5A5CNFSM4IRG4QXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5LPHSA#issuecomment-525792200,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANAZPT6BCNSI4ZFINLAPKNTQG2KC5ANCNFSM4IRG4QXA
.

Please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version. Also, did you compile from source or install a binary?
Make sure you also include the exact command if possible to produce the output included in your test case. If you are unclear what to include see the issue template displayed in the Github new issue template.
We ask for this in the issue submission template, because it is really difficult to help without that information. Thanks!

OS Platform and Distribution: Linux Ubuntu 16.04 LTS
TensorFlow built from source (tf_nightly-1.14.0-cp35-cp35m-linux_x86_64)
For Conv2D parameters
in_depth = 4, input_cols = 1920, filter_cols = 3, input_rows = 1080, filter_rows = 3, out_depth = 32
output of GetDirectConvCost function: -1906180096, but should be<PHONE_NUMBER>
For Conv2D parameters
in_depth = 32, input_cols = 1920, filter_cols = 3, input_rows = 1080, filter_rows = 3, out_depth = 32
output of CanUseDeepConv2D function - false (deep_direct_ratio: 7.14965), but should be true (deep_direct_ratio: 0.722222)

The function clearly overflows, but the fix suggested is not enough.

created new PR with additional checks

The caller should check, this is defined behavior. It's very probable that the caller already has all checks in place, if not, that's where they should go

There are more issues in that file. I'll handle those changes this week

Hi There,
We are checking to see if you still need help on this, as you are using an older version of tensorflow which is officially considered end of life . We recommend that you upgrade to the latest 2.x version and let us know if the issue still persists in newer versions. Please open a new issue for any help you need against 2.x, and we will get you the right help.
This issue will be closed automatically 7 days from now. If you still need help with this issue, please provide us with more information.


Xcode 8 and swift 2.3 - Baseline is unavailable

New Issue Checklist

[X] I have looked at the Documentation
[X] I have read the F.A.Q.

Issue Info

Info
Value

Platform
ios

Platform Version
8.0

SnapKit Version
0.19.1

Integration Method
cocoapods

Issue Description
After migrating to Xcode 8, I get the following error:
"'Baseline' is unavailable in Swift: Use 'lastBaseline' instead"
I cleaned my building folder and I also set the pod target to use legacy swift.

@mime29 you must use 0.22.0 for Swift 2.3 support and you've indicated you are on 0.19.1. Upgrade and open a new issue if you have persisting problems.


improve cleanup

dont depend on controller to remove nodeMaintenance during cleanup as it may not operate properly. improper cleanup might affect other tests.
to achieve that we delete object then remove finalizer.
in addition wait for all objects to get removed from api during cleanup.

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

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

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
1060

Relevant Lines:
1513

💛  - Coveralls


versions incompatibility problems

I'm trying to run the file in colab and I always find versions incompatibility problems.
 Has anyone managed to run successfully lately?
I also cannot finish the execution of the remaining codes on my personal computer... Any news about the correct versions to be able to run this code?
It would be extremely useful to me...

even solving the versions so as not to cause this initial problem:
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
xarray 0.18.2 requires pandas>=1.0, but you have pandas 0.25.3 which is incompatible.
torchtext 0.11.0 requires torch==1.10.0, but you have torch 1.5.0 which is incompatible.
torchaudio 0.10.0+cu111 requires torch==1.10.0, but you have torch 1.5.0 which is incompatible.
google-colab 1.0.0 requires pandas>=1.1.0; python_version >= "3.0", but you have pandas 0.25.3 which is incompatible.
fbprophet 0.7.1 requires pandas>=1.0.4, but you have pandas 0.25.3 which is incompatible.
datascience 0.10.6 requires folium==0.2.1, but you have folium 0.8.3 which is incompatible.

version difficulties arise again later, never managing to overcome this situation


Memory leak test

We need to create a test that can simulate SSR and make sure we have no memory leaks when rendering styles for each request.

cc @oliviertassinari

@kof take a look at this https://www.npmjs.com/package/leakage

This issue was moved to cssinjs/jss#809


Weird infer by const modifier in spread arguments

Bug Report
The const modifier does not behave as expected. Typescript would warn me to infer the const generic parameters when I use the spread argument in a function when the literal type object consists of 2 different primitive literals.
And I am not sure whether this type of behaviour is intentional.
🔎 Search Terms
const modifier, typescript5, spread syntax, spread argument
🕗 Version & Regression Information
This behaviour begins starting at typescript 5.0 (where the const modifier is introduced)
I have been seeing this behaviour between typescript 5.0 to the nightly build of 5.1
⏯ Playground Link
Playground link with relevant code
💻 Code
type A = 1 | '2';

function concat<const U>(...items: (A | U)[]) {
  return [...items];
}

const originalArray: A[] = [1, '2'];
const perfectConcatenatedArray = concat(...originalArray, 3, 4 ,5);
const okayConcatenatedArray = concat(...originalArray, '3', '4', '5');
const concatnatedArray = concat(...originalArray, 3, 4, 5, '6', '7');

🙁 Actual behaviour
Typescript is inferring that the const U should be 3 instead of 3 | 4 | 5 | '6' | '7'
It prompts an error stating that Number 4 is not assignable to the parameter of type A | 3 for concatnatedArray.
While both perfectConcatenatedArray and okayConcatenatedArray are perfectly fine
🙂 Expected behaviour
I would expect concatnatedArray to have type 1 | '2' | 3 | 4 | 5 | '6' | '7'.
It is possible to do so if I wrote it in a class manner and chain the concat function. But I cannot do it within the same concat function.
Playground link to the class version of the issue

Inference will infer that it's OK to form a union of types from the same primitive literal family, but not different ones. See also
https://github.com/microsoft/TypeScript/issues/32596#issuecomment-516054244


Azure OpenAI call results in 400 error because best_of parameter is set in axios request

When using an Azure gpt-35-turbo deployment, logprobs, best_of, and echo parameters are not supported. https://learn.microsoft.com/en-us/azure/cognitive-services/openai/reference
It looks like
   const model = new OpenAI({
      temperature: 0,
      azureOpenAIApiKey: this.configService.get('OPEN_AI.AZURE.API_KEY'),
      azureOpenAIApiInstanceName: this.configService.get(
        'OPEN_AI.AZURE.INSTANCE_NAME',
      ),
      azureOpenAIApiDeploymentName: this.configService.get(
        'OPEN_AI.AZURE.DEPLOYMENT_NAME',
      ),
      azureOpenAIApiVersion: this.configService.get('OPEN_AI.AZURE.VERSION'),
    });

   await model.call('Say this is a test')

sets the best_of parameter by default in the axios request body to the Azure REST API, causing it to fail with error 400. "logprobs, best_of and echo parameters are not available on gpt-35-turbo model. Please remove the parameter and try again. For more details, see https://go.microsoft.com/fwlink/?linkid=2227346."

Temporary workaround:
    delete model.bestOf;

After initializing new OpenAI

#1371 Look here.


Question with set var doesn't parse

<{?a,c}-->b>?

You cannot use a comma in a set. Commas are the sequence operators.
use <{?a c} --> b>?


Sharepoint.com

 First of all, thank you for the work you have done.
Is there any hope that the project will be updated, allowing downloads from sharepoint.com? (some professors do not allow download of videos...)

This subject has been addressed in issue #275

Note that in the last few weeks I've noticed a change where you can't even download the videos from sharepoint. Looks like only the organiser can.
Q1 of 2021, only the meeting recorder and organizer will be able to download the meeting recording. All other attendees will only be able to view it.
https://support.microsoft.com/en-us/office/play-and-share-a-meeting-recording-in-teams-7d7e5dc5-9ae4-4b94-8589-27496037e8fa#bkmk_downloadmeetingrecording

Bumping the last comment by @palmerj

Can confirm that destreamer no longer works for meeting recordings as they are shared in Sharepoint now.

As someone mentioned in #275 someone has developed a new tool, but for now we've been using a workaround to open the video file directly:

Go to the "files" section of your teams channel;
Open the "Recordings" folder;
Right click on the video file and select copy link;
You should get a direct link to the .mp4 file, you can view it in your browser or download it.

The link structure should be something like https://[ORGANIZATION].sharepoint.com/sites/[FOLDER NAME, USUALLY THE NAME OF THE TEAM GROUP OR SUBJECT]/Shared%20Documents/General/Recordings/[FILENAME OF THE RECORDING WITH %20 INSTEAD OF SPACES].mp4
If you only have a link to the sharepoint player you can try to recover the direct link to the mp4 file by using some website like https://www.url-encode-decode.com/ to remove the utf-8 to ascii encoding and then editing the url to look like the example above.
Scusate per la grammatica ma l'ho scritto un po di fretta 😅

Looks like this shouldn't take too much effort to implement into destreamer. Will take a look. Thanks for your writeup, appreciate the effort.

You are welcome 😊 ("writeup", è giusto la descrizione di un url 😅)
Just so you know,  "General" was the name of the channel, and the folder "Recordings" might not have the same name for everyone.
Honestly I don't know how universal this method is, maybe it only works in my organization because of some oversight when setting up the privileges.
I haven't looked at his code, but I assume @kylon 's software might be more robust and harder to patch for Microsoft seeing the presence of FFmpeg to obtain the video stream (I guess).

As mentioned in #275 someone has developed a new tool, but in the meantime we've been using a workaround to open the video files directly:

Go to the "Files" section of your teams channel;
Open the "Recordings" folder;
Right click on the video file and select copy link;
You should get a direct link to the .mp4 file, you can view it in your browser or download it, no restrictions.

The link structure should be something like this: https://[ORGANIZATION].sharepoint.com/sites/[FOLDER NAME, USUALLY THE NAME OF THE TEAM GROUP OR SUBJECT]/Shared%20Documents/General/Recordings/[FILENAME OF THE RECORDING WITH %20 INSTEAD OF SPACES (or other ascii to utf-8 codes)].mp4
If you only have a link to the sharepoint player you can try to recover the direct link to the mp4 file by using some website like https://www.url-encode-decode.com/ to remove the utf-8 to ascii encoding and then editing the url to look like the example above. (no need to convert it back when opening the new url btw).
Scusate per la grammatica ma l'ho scritto un po di fretta 😅

After a bunch of fenagling I finally got a url that ends in .mp4 (https://companyName-my.sharepoint.com/:v:/r/personal/employeeName_companyName_com/Documents/Recordings/Company%20Meeting-20210712-Meeting%20Recording.mp4) but I get a "Sorry, you don't have access." :

Yeah I saw links with the ":v:" sometimes in the link generated inside the chat, they don't contain the full path.
Again it probably depends on how the organization has set up its privileges.
When I get my url from Sharepoint or the Files section on Teams I get this: (after going trough https://www.url-encode-decode.com/)
https://[ORG].sharepoint.com/sites/[TEAM NAME]/Shared Documents/Forms/AllItems.aspx?id=/sites/[TEAM NAME]/Shared Documents/General/Recordings/[FILENAME].mp4&parent=/sites/[TEAM NAME]/Shared Documents/General/Recordings
That can be edited into this:
https://[ORG].sharepoint.com/sites/[TEAM NAME]/Shared Documents/General/Recordings/[FILENAME].mp4
[ORG] is already present int the url, [TEAM NAME] should be easy to recover, "General/Recordings/" might change as I said up there and [FILENAME] if not apparent int the url is obtainable by opening the details section of the Sharepoint player;
It's mostly custom but it look something like:
[Custom part by the organizer] [dd.mm.aa date]-[aaaammdd, basically ISO 8601-like date]_[hhmmss time]-Meeting Recording.mp4
...also by looking at that "other details" section there's a button that says "copy direct link" (it's in italian so idk the exact name 😅)

It doesn't work with sharepoint

The issue related to invalid urls seems to be located in this regex check

I don't know if I got the regex right, but this is what I wrote in order to match correctly the url
(?<domain>https:\/\/.+\.sharepoint\.com)(?<basesite>\/:.?:+\/*\/.*)(?:(?<filename>\/.*\.mp4)|\/.*id=(?<paramFilename>.*mp4))

Unfortunately, this resulted in another error i'm not able to fix,as i'm unable to debug typescript

I sincerely hope these info will help you :)

@lukaarma can you check one thing please. With your new branch, when you get the API URL formed, dump the JSON output for the streams,

@snobu I could not find something like that but I'll e-mail you a couple of json I got so that you could take a look. When I have more time I'll dig more into the api to see if I missed something.

it seems to be a timeout problem because Chrome close it when i'm writing password or second authentication factor.

@DaniDiaz71 Could you please make sure that you are on the latest version of the Sharepoint branch? There was an error during the login process that should have been fixed, if you are on the latest branch and you are still having trouble loggin in please open a separate issue.

But in our company, now the url of the online video for meeting recording changed to something like below: https://xxxxxx-my.sharepoint.com/%3Av%3A/g/personal/xxxxxx_xxxxxx_xxxxxx_com/EQ5y7SkmWbRDtzS0uE3Rd2sBjxHSkc_GkbzTx9PreCrXig

@snqqyj if you are comfortable enough to share the complete url with me via the e-mail you find on my profile it would be great and really help me. If you don't want could you at least tell me if it ends with '.mp4'

The wird thing is i can downlaod without problmes videos that were recorded until last week, but with the recordings of this week I'm receiving this error:

@gionac Was there any change in the url of the video between last week and this one? Could you please check.

I have the same problem. I can't download videos from this week.

@gionac Was there any change in the url of the video between last week and this one? Could you please check.

@lukaarma I copy there the two links. The first (newer) doesn't work, instead the second works like charm..
[...]/L1%20-%20relativit%C3%A0%20e%20Locard%20-%20L2%20-%2020211026_152611-Meeting%20Recording.mp4?csf=1&web=1&e=XxMFGP
[...]/L0-lab-20211005_160042-Meeting%20Recording.mp4?csf=1&web=1&e=7E0Pk7
I omitted the first because it's identical

I omitted the first because it's identical

@gionac could you please open a new issue and provide me the full verbose output of your command (add -- verbose to your command)
I don't see anything special in your link so it may be something else

I omitted the first because it's identical

@gionac could you please open a new issue and provide me the full verbose output of your command (add -- verbose to your command) I don't see anything special in your link so it may be something else

I did a new issue here, with the verbose output, here: https://github.com/snobu/destreamer/issues/416
Thank you @lukaarma !

I also had an issue with personal videos.
I needed to add "personal" to the list:
const shareUrlRegex = new RegExp(/(?https://.+.sharepoint.com).?(?/(?:teams|sites|personal)/.?)(?:(?/.*.mp4)|/.*id=(?.*mp4))/);

I also had an issue with personal videos.
I needed to add "personal" to the list:
const shareUrlRegex = new RegExp(/(?https://.+.sharepoint.com).?(?/(?:teams|sites|personal)/.?)(?:(?/.*.mp4)|/.*id=(?.*mp4))/);

Same here. After adding "|personal" to the regex, download started to work (even for videos where I have no download permission). Thanks!

Thanks for @muzso advise, and I have created a PR base on this, as well to include a small fix on the file name with characters that requires encodeURIComponent-ed.


[ISSUE] AccountClient for AzureDatabricks available?

Description
Is the AccountClient only available for AWS at the moment? Receiving nothing but Endpoint not found for /2.0/accounts/<account_id>/ when trying to use any of the AccountClient methods.
Reproduction
# Use an azure databricks account and pass in appropriate variables
from databricks.sdk import AccountClient
from dotenv import load_dotenv
from os import getenv as env

load_dotenv("dd.env")

account_id = env("ACCOUNT_ID")
azure_workspace_resource_id = env("AZURE_WORKSPACE_RESOURCE_ID")
azure_tenant_id = env("AZURE_TENANT_ID")
azure_client_id = env("ACCOUNT_CLIENT_ID")
azure_client_secret = env("ACCOUNT_CLIENT_SECRET")

a = AccountClient(
    account_id=account_id,
    azure_workspace_resource_id=azure_workspace_resource_id,
    azure_tenant_id=azure_tenant_id,
    azure_client_id=azure_client_id,
    azure_client_secret=azure_client_secret,
    debug_headers=True,
)

billing = a.billable_usage.download(start_month="2023-06", end_month="2022-07")

Expected behavior
To be able to access any of the account methods like groups, users, billing, workspaces
Debug Logs
Traceback (most recent call last):
  File "c:\sdk\scripts\dd_get_billable_usage.py", line 25, in <module>
    billing = a.billable_usage.download(start_month="2023-06", end_month="2022-07")
  File "C:\Users\user\.virtualenvs\sdk-bqxZoST4\lib\site-packages\databricks\sdk\service\billing.py", line 492, in
download
    self._api.do('GET', f'/api/2.0/accounts/{self._api.account_id}/usage/download', query=query)
  File "C:\Users\user\.virtualenvs\trino-bqxZoST4\lib\site-packages\databricks\sdk\core.py", line 932, in do
    raise self._make_nicer_error(status_code=response.status_code, **payload) from None
databricks.sdk.core.DatabricksError: Endpoint not found for /2.0/accounts/<account_id>/usage/download

Other Information
Windows 10

@jordandakota

DATABRICKS_ACCOUNT_ID, not ACCOUNT_ID
you don't have to azure_tenant_id = env("AZURE_TENANT_ID"), because AccountClient() loads from environment variables by default.
billable usage apis are not available on azure. only IAM and Unity Catalog APIs are available on account level for azure - see https://docs.databricks.com/api/azure/account/introduction - please try those and see if the issue persists

While we can authenticate to Azure Databricks Account, we can't list/get/update workspaces.  Any ideas on when the Azure Account API will reach parity with the AWS version?

I also ran the following code on Azure Databrick, but an error occurred.
%pip install databricks-sdk --upgrade -q

dbutils.library.restartPython()

from databricks.sdk import WorkspaceClient
from databricks.sdk import AccountClient

w = WorkspaceClient()
account_client = AccountClient()

account_client.metastore_assignments.list(
    metastore_id = w.metastores.list()[0].metastore_id,
)

DatabricksError: Endpoint not found for /2.0/accounts/None/metastores/b590278a-5927-XXXX-XXXX-XXXXXX/workspaces

The oct 3, 2023 code doesn't work for me either. Looks like a default AccountClient() or AccountClient(account_id=account_id) fails to retrieve anything through the account client.  When I add the account ID, it does change the message from account id none, to "Endpoint not found for /2.0/accounts/{account id redacted}/scim/v2/Groups"


drivers: flash: replace DT_FLASH_DEV_NAME with DT macro

Replace DT_FLASH_DEV_NAME with DT_CHOSEN_ZEPHYR_FLASH_CTRL_LABEL.  We now
set zephyr,flash-ctlr in the chosen node of the device tree to the flash
controller device.
(NOTE: For a SoCs with on die flash, this points to the controller and
not the 'soc-nv-flash' node.  Typically the controller is the parent of
the 'soc-nv-flash' node).
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

I'm not removing DT_FLASH_DEV_NAME from dts_fixup.h in this PR as to try and avoid both merge conflicts and to allow the dts_fixup.h change to be in smaller groups of PRs.

@mbolivar-nordic Changed it to CONTROLLER since its in sync with things like 'gpio-controller' as well as I kept mixing up ctlr and ctrl.

^^ @erwango see https://github.com/zephyrproject-rtos/zephyr/pull/24620#issuecomment-618045717

^^ @erwango see #24620 (comment)

Txs @nvlsianpu

The commit message still mentions zephyr,flash-ctlr. Perhaps it would be good to update it.

fixed the commit message.


Issue#75 weibull

Since the other reviewers are currently very busy, checks are passing and there are no conflicts, I merge without waiting for reviews.


fix: update docker image

update docker image from 0.7.1 -> 0.8.0
remove on-hold job from CI

Closing. Shellcheck is now in the circleci cimg/base image.


Improve DESCRIPTION file

The Description field is intended to be a (one paragraph) description of
what the package does and why it may be useful. Please add more details
about the package functionality and implemented methods in your
Description text.

Please always write package names, software names and API (application
programming interface) names in single quotes in title and description.
e.g: --> 'Palantir Foundry'
Please note that package names are case sensitive.

Please provide a link to the used webservices (Palantir Foundry) to the
description field of your DESCRIPTION file in the form
http:... or <https:...>
with angle brackets for auto-linking and no space after 'http:' and
'https:'.

We do not need "| file LICENSE" and the file as these are part of R.
This is only needed in case of attribution requirements or other
possible restrictions. Hence please omit it.

Please add small executable examples in your Rd-files to illustrate the
use of the exported function but also enable automatic testing.

Released 0.8.0


feat(api): improving crt.sh query service

Description
updating crtsh, viewset crtsh
Fixes
Crtsh is now fixed

fixes #0

Changes include

[ ] Bugfix (non-breaking change that solves an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)

Breaking changes
Please complete this section if any breaking changes have been made, otherwise delete it.
Checklist

[ ] I have assigned this PR to myself
[ ] I have added @vrn-sh/members to the reviewers, or specific members of the team
[ ] I have added the needed labels
[ ] I have tested this code
[ ] I have added / updated tests (unit / functionals / end-to-end / ...)
[ ] I have updated the README and other relevant documents (guides...)

Additional comments
Please post additional comments in this section if you have them, otherwise delete it.

can you fix the conflict with next please ? @Loris-Reggiani

closed in order to address #262 instead


Permanent Sidebar for Local Graph and Backlinks

Huge thanks for this amazing plugin! I've always wanted to publish my notes, but most alternatives are complicated or missing some key features. With this plugin, I set up the website in 5 mins and can directly publish notes in Obsidian on the fly!
I have some small feature requests. Can there be an option to make the sidebar for the local graph and backlinks be permanent rather than togglable? Just like help.obsidian.md, this will provide more info and help navigation.
Also, the local graph doesn't show the neighbor links or the direction. I think these elements will make the local graph more useful.

Thanks for the implementation! It looks perfect!
On my end, the right sidebar and the note content overlap a little bit. Thus I added this snippet to my custom-style.scss:
div.content { max-width: 800px; }

You're right, thanks for telling me. The specific note that I tested with yesterday didn't have that problem.
I've released a patched version with your change.

Closing this, as the permanent sidebar has been implemented. Created a new issue regarding the neighbouring links: #148


Add signingUrlQueryParams

Usage
<DropzoneS3Uploader
  signingUrl="signed_url"
  signingUrlQueryParams={{ upload_type: 'avatar' }}

I tried using https://babeljs.io/repl/#?evaluate=true&presets=es2015%2Creact%2Cstage-1%2Cstage-2&code= but it came out vastly different than your lib file. Though the component still worked during my tests. Just didn't know if you would want it processed through there or not.

Thanks! It should probably just pass through any props to S3Uploader, there's still a few props like this one left out. I'll fix that up at some point.
This project still uses babel 5, so it's likely that any build would be different with that builder. npm run build should make it spit out the same thing as me.