Adding Moya as a dependency in Swift Package Manager causes the Previews in Xcode11 to crash.

Moya version: 13.0.1
I added the dependency in Swift Package Manager, and i see that the Package.swift for Moya includes in it's targets all the dependencies incluiding ReactiveMoya and RXMoya.
The project builds and runs without any problem. But for some reason, loading the previews in Xcode does not work.
I also tried to change my project using Cocoapods instead of SPM, and using pods the previews actually worked.
The actual error message is the next one:
linker command failed with exit code 1 (use -v to see invocation)

----------------------------------------

Failed to build the scheme "theMovieUI": Link /Users/user/Library/Developer/Xcode/DerivedData/theMovieUI-fvtkhqlopskkqueykjpedojefpoa/Build/Intermediates.noindex/Previews/theMovieUI/Products/Debug-iphonesimulator/libCwlCatchException.dylib:
Undefined symbols for architecture x86_64:
  "_catchExceptionOfKind", referenced from:
      CwlCatchException.(catchReturnTypeConverter in _841BEC3F94220F0DFD4EA4E9DDB3DC93)<A where A: __C.NSException>(_: A.Type, block: () -> ()) -> A? in CwlCatchException.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Link /Users/user/Library/Developer/Xcode/DerivedData/theMovieUI-fvtkhqlopskkqueykjpedojefpoa/Build/Intermediates.noindex/Previews/theMovieUI/Products/Debug-iphonesimulator/libCwlPreconditionTesting.dylib:
clang: error: no such file or directory: '/Users/user/Library/Developer/Xcode/DerivedData/theMovieUI-fvtkhqlopskkqueykjpedojefpoa/Build/Intermediates.noindex/Previews/theMovieUI/Products/Debug-iphonesimulator/libCwlCatchException.dylib'
Command Ld failed with a nonzero exit code

I think the error comes from another dependency. But with the built-in SPM in XCode 11 i can't configure my dependencies to be excluded when compiling, because it does not generate a Package.swift file.
Maybe there's something I'm doing wrong? Or some configuration that I'm missing?

Hey @uguizu, this is definitely not your fault. This is the same problem we have noticed on SwiftyUserDefaults as well.
Basically because SPM fetches testing dependencies, even though they are not present in the main target, Xcode Preview sees them and tries to build it.  I will make PRs to fix both libraries ASAP. Sorry for the problem.

@sunshinejr Sure, thanks the for the quick response and i gonna be waiting for the next update.

@uguizu FYI I just released 14.0.0-beta.3 that should fix the issue.

Thanks, testing the new version. now @sunshinejr

I'm having this problem. Using Xcode 12 beta, Big Sur beta; Moya branch set to master in SPM. Perhaps I don't understand how to get the proper version?

Hey @barrypress, so ideally you wouldn't use a branch but a version of Moya of your choice. This is because it's not advised from the code perspective, as things might break, but also because we have release scripts that hide certain dependencies that you don't need (and in this case it also broke Xcode Previews).
Here's a list of versions with changelog so you can take a look and see what fits you best. In case you have more questions, let me know!

Thanks. I appreciate the reply.
I'm still confused, though. From the date stamps here https://github.com/Moya/Moya/issues/1919, I’d expected 14.0.0 to have fixed the problem. Nevertheless, my unit test builds (indeed, not previews) are failing in just this way. Is this perhaps a new problem, or do I just not understand what I need to be doing?
Thanks again
Barry

On Jul 10, 2020, at 3:30 AM, Łukasz Mróz<EMAIL_ADDRESS>wrote:
Hey @barrypress https://github.com/barrypress, so ideally you wouldn't use a branch but a version of Moya of your choice. This is because it's not advised from the code perspective, as things might break, but also because we have release scripts that hide certain dependencies that you don't need (and in this case it also broke Xcode Previews).
Here's https://github.com/Moya/Moya/releases a list of versions with changelog so you can take a look and see what fits you best. In case you have more questions, let me know!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/Moya/Moya/issues/1919#issuecomment-656605970, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABILZJ4ZOEP4GG6QTYNXDDR23UV3ANCNFSM4IXKDHEQ.

@barrypress Hmm, I didn't notice any issues with unit tests that were caused by 14.0.0 - can you create a separate issue and give a little bit more details?

I didn’t want to take up your time without good cause, so I dug into it further. That shows me a few things
This is a multiplatform app with the new build system, and the frameworks had to be added to the macOS target separately and using a different part of Xcode vs. how it’s done with iOS. I fixed that, but it didn’t help.
I can build and run both iOS and macOS versions of the app having the unit test problem, and what functionality I’ve built so far works with Moya
I also verified that a new, empty app with just enough code to cause Moya to be linked would build for test cases just fine.
I’m not ignoring your request to open an issue, but don’t expect much from it if you can't repro the problem. To that end, I tried to build a cut-down app that demonstrates the problem, but after a day’s work I’m not there. I’d be happy to send the existing project to you, and if this is more of a packages issue than one of the code itself, then maybe stripping down to minimum code isn’t as important as it would otherwise be.
So, how do you want me to put the issue together? Error captures only, or include the project?
Barry

On Jul 14, 2020, at 3:16 AM, Łukasz Mróz<EMAIL_ADDRESS>wrote:
@barrypress https://github.com/barrypress Hmm, I didn't notice any issues with unit tests that were caused by 14.0.0 - can you create a separate issue and give a little bit more details?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/Moya/Moya/issues/1919#issuecomment-658097954, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABILZI7UWGCGZ6EPD5WT3TR3QV75ANCNFSM4IXKDHEQ.


Lot's of missing features?

So I just installed this module and I'm a bit perplexed as a lot of the features don't seem to be showing up/working for me.

None of the extra buttons that show the module author, website, link to open settings, etc show on the module management page.
I'm not seeing any conflicts but I'm not sure if that's because there are none or if it's just not showing.
I'm only able to rename or lock the first module in the list
The spreadsheet button is missing from the module management screen even after enabling it in the settings
With libthemer installed and the alternative checkbox option activated, the green check doesn't show and all the modules show blank making it impossible to know which are active and which are inactive without using the filter at the top

I used find the culprit and deactivated everything except Module Management+ (And find the culprit) and all of these issues still persist so it doesn't seem to be a conflict with any other mod.
I'm on Foundry core 10.291 and Module Management+ 2.1.6 and I've tested it on both PF2E 4.12.6 and DND5e 2.2.1
I'm not sure if this is user error or if it's the mod, I've fiddled with the settings and I can't seem to figure out why a lot of the features aren't working so any help would be much appreciated!

I haven't seen this complaint yet; I can take a look at it when I have a moment.

Can you confirm if v2.2.1 fixes this issue for you?

Assuming this is resolved as it appears to be resolved for most other people

Hey sorry, for the late reply. I just tested with 2.2.1 and it's still not working.

OK so new update. I tried doing a find the culprit again and this time it sort of worked. So with only module management+ and find the culprit it shows the buttons, I can rename/lock modules, and the checkbox works but only on the first 35 modules in my list any after that exhibit the same issue I first reported.
In addition to that There seem to be some conflicts with other modules when I activate them they shrink the number of modules that have the extra features provided by module management+, I'll list the conflicts below:
Advanced Macros - With this one enabled no modules have the extra features
PF2E Animations  - With this one enabled only the first module in the list has the extra features
Bing Bong  - With this one enabled only the first 2 modules have the extra features
Break Time - With this one enabled only the first 3 modules have the extra features
Combat Ready - With this one enabled only the first 4 modules have the extra features
Compendium Folders  - With this one enabled only the first 5 modules have the extra features
DF Chat Enhancements - With this one enabled only the first 8 modules have the extra features
DF Manual Rolls - With this one enabled only the first 9 modules have the extra features
DFreds Droppables - With this one enabled only the first 10 modules have the extra features
DF User Logger  - With this one enabled only the first 11 modules have the extra features
Dice So nice - With this one enabled only the first 12 modules have the extra features
Dice Tray - With this one enabled only the first 19 modules have the extra features
Foriens Quest Log - With this one enabled only the first 21 modules have the extra features
Health Estimate - With this one enabled only the first 23 modules have the extra features
JB2A  - With this one enabled only the first 24 modules have the extra features
Chat Commander (Library for DF Chat Enhancements) - With this one enabled only the first 25 modules have the extra features
Color Settings Lib (Library for several mods) - With this one enabled only the first 26 modules have the extra features
Lib themer - With this one enabled only the first 29 modules have the extra features
Libwrapper (Library for multiple mods)  - With this one enabled only the first 30 modules have the extra features
Token Action Hud PF2e - With this one enabled only the first 32 modules have the extra features
Lock View  - With this one enabled only the first 33 modules have the extra features
Material Deck - With this one enabled only the first 34 modules have the extra features
Regardless of modules enabled or disabled I'm still not seeing any conflicts (Though I'm unsure of where those normally show up if any)
And also redardless of what modules are enabled or disabled the google sheet doesn't show when the option is marked in settings.
I'm not sure what is causing these weird incompatibilities is there like a limit in the mod that it only works on a certain number of modules? That's the only reason I could see for why it only works on the first 35.
As for all the mod incompatibilities I understand that's a lot and especially if no one else is having issues so if I'm just SOL then oh well but I still thought it important to make you aware of.
This is after updating the module to 2.2.1 BTW

Are you using version <IP_ADDRESS> or 2.2.1?

<IP_ADDRESS>. I just updated it yesterday. Before the update nothing was showing even with all modules disabled other than module management+ and find the culprit

Wait so does v<IP_ADDRESS> work or not?

Sorry if I wasn't clear <IP_ADDRESS> does not work. It exhibts the behavior I detailed in the comment above

Can you please Download Source Code (zip) from https://github.com/mouse0270/module-credits/releases/tag/<IP_ADDRESS> and extract these files and replace the content inside of module-credits folder in your foundry UserData?
After doing this, you may need to restart your foundry instance and then start up the world. Once in the world please hit F12 and report any errors in your consol.
I really don't know whats going wrong and need more details.

Ok, I downloaded the files and replaced all the files, and restarted the server.
I opened up the console and then opened the manage modules menu, the error persists  and it gives this error:
"module.mjs:776 Uncaught (in promise) TypeError: undefined. Cannot read properties of undefined (reading 'Configure')
[Detected 1 package: module-credits]
at Object.renderModuleManagement [as fn] (module.mjs:776:160)"

Looks like I messed up and did game.i18n.translations.SETTINGS.Configure instead of game.i18n.localize('SETTINGS.Configure') I suspect your foundry is not set to English as its default language?
This should be easy enough to fix, I will patch it tomorrow and see if I did this anywhere else and fix that too.

Ohhhhh that makes sense. I'm using a mod to edit the CSS of the login screen which changes the language to Latin to do so, everything shows in English but the language is set to Latin to change the CSS.

Well to be fair, you're language setting isn't the problem... This is my screw up, but if the reason you were getting this issue is because of this mistake of mine. I'll get it patched tomorrow.

Ok sounds great! Glad we got it figured out!

Should be fixed in version <IP_ADDRESS> https://github.com/mouse0270/module-credits/releases/tag/<IP_ADDRESS>
If not let me know.


[LLAMA] Export Pathway Script

@dsikka maybe i missed something, but what makes llama so special that it gets is own export pathway?


Add instruction and conversational data support

Issue #ENG-10912
Add support for instruction and conversational training data support for FT jobs.
The support goes in two ways:

File checks to make sure the data is formatted correctly
Add train_on_inputs flag to train on completion/assistance phrases

Reviewed new changes and lgtm👍


Update citi-program-tutorial.md

Added link to CITI Program SSO subscription information and additional optional attribute.

@jeevansd

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#please-close

Hi @v-hgampala, was there a reason this was closed and not merged?

@jfineout-citi, Apolize, I was created the pull request for your changes in our private repository, where the content team will verify the Acrolinx and do the content validation for better document. The PR will merge within 1-2 working days.


Nans in additional columns

When a dataframe has additional data created, e.g. by interpolate, additional columns are often given np.nan as the value, rather than, say, an empty string or a 0.0.. This is presumably a choice rather than a bug. However when there are nans in these columns, they will not appear in timeseries, and will also disappear if the dataframe is recalculated by pyam.IamDataFrame(database_with_nans.data). This seems like a bug.

Thanks for raising this issue @Rlamboll but I'm not sure which part here you see as a bug and which is bad (or unexpected) design choices - or if this is related to some not-yet-supported aspect of continuous-time data.
My understanding:

interpolate() (and the underlying fill_series()) return nan if the requested time is outside of the range of a specific timeseries (doing something else would be not inter- but extrapolation)
nan are dropped when initializing a new IamDataFrame because pandas behaves strangely with nan in columns (examples provided @znicholls, see the docs). But calling [timeseries(https://pyam-iamc.readthedocs.io/en/latest/api/iamdataframe.html#pyam.IamDataFrame.timeseries)] fills nan in any time-column where row data is missing (ie. if at least one data point exists for that time index).

Can you specify or provide a short test case that illustrates what you would expect to see?

Perhaps I wasn't clear, all these nans occur in additional columns. Interpolate returns a value correctly in the value column, but not in other columns (not a bug). However I'd expect that initialising a dataframe only drops cases where important columns are nan, not where any column is nan. If it is going to do that, can we infill those columns with something other than nan. Here is an example (appologies for lack of indentation in this format):
import pandas as pd
import pyam
tdb = pd.DataFrame(
[["_ma", "_sa", "World", "CO2", "Gt CO2", 0, 2, 3],
["_ma", "_sa", "World", "CH4", "Gt CH4", 0, 200, 300],
],
columns=["model", "scenario", "region", "variable", "unit", "sub_annual"] + [2050, 2070],
)
df = pyam.IamDataFrame(tdb)
df.interpolate(2060)
df.data   # Contains 2060 data, nan in additional column looks innocuous
df.timeseries()  # No 2060 appears here
pyam.IamDataFrame(df.data).data  # Interpolated data has gone

Thanks for the clarification, indeed a bug because the extra_cols feature is not yet tested thoroughly. PR with a bugfix coming soon.

In terms of how this should work though, should we expect that extra_cols is left out of timeseries? This means that df->timeseries->df loses info. Alternatively, all rows of a model/scenario/variable/region grouping have to have the same additional cols info (and will need to be initialised in this way by interpolate) otherwise they will result in separate rows on the timeseries, with nans shared between rows. Effectively we then have an unspecified (and infinitely extensible) primary key in timeseries format.

good question, let me bounce it back to you:
if there is dataframe like
TEST_DF = pd.DataFrame([
    ['model_a', 'scen_a', 'World', 'Primary Energy', 'EJ/yr', 'foo', 2005, 1],
    ['model_a', 'scen_a', 'World', 'Primary Energy', 'EJ/yr', 'bar', 2010, 3],
],
    columns=pyam.IAMC_IDX + ['extra_col', 'year', 'value'],
)

i.e, all index values the same except for extra_col, should there be any interpolation or not?
The bugfix in #352 assumes no (there will not be any interpolation because the first and second line have a different "index" so are different timeseries).
If you think it should yes, how should extra_col in the new row be filled?

Yes, I would like to see interpolation, i.e. extra columns are ignored for this index. E.g. if I have a meta column with 'infilled' at one timepoint, I'd like to infill around it. I'm mostly treating these cols as extra info, not core function. OTOH, I have no strong preference for what to infill. The annoying answer would be check if the two points around it have identical entries, copy them if so and a non-breaking empty value (0 or "" etc) if not. But I also want this function to work quickly and currently don't have a strong use-case, so depends on how much effort it takes to do that. Always using a non-breaking empty value is fine too.

Ok, I think we agree. Please review #352, where I just added another test based on the example above to ensure that timeseries with non-matching extra-column values are not interpolated.

OK, will there be an update later to make it fail that test?

OK, will there be an update later to make it fail that test?

No, that test shouldn't fail - that's the expected behaviour.

This confuses me because I just said that I did want interpolation in that case.

Ok, then we were talking past each other. Can you please specify which output you expect from interpolate(2007) on the following timeseries data:

model
scenario
region
variable
unit
extra_col
2005
2010

model_a
scen_a
World
Primary Energy
EJ/yr
foo
1
np.nan

model_a
scen_a
World
Primary Energy
EJ/yr
bar
np.nan
3

model_a
scen_a
World
Primary Energy
EJ/yr
dummy
np.nan
5

In my understanding, because there are no data points with the same index (including extra_col) before and after the time to be interpolated, there should not be any data added to this dataframe.

I didn't realise we were planning to allow that sort of data. I thought that (model x scenario x region x variable x time) was a unique key, hence this is an invalid df. Similar to the multi-unit question in #338, it's massively annoying to work out if a model-scenario is "complete" if I need to ask whether an unknown number of extra columns are also populated in every combination. Can we combine data from extra_col = foo with data from extra_col = bar? Without knowing what those columns are, it seems impossible to know. If the extra column is just a label to say 'infilled', clearly it's silly to predicate anything on the basis of that, and I assumed we'd want to combine infilled and non-infilled data into the same row of a timseries. If it somehow changes the meaning (like it's a unit working under a different conversion metric), we need to consider that. I believe we're trying to set up a zoom chat this week on nomenclature for additional columns.

At least we got to the bottom of the misunderstanding...
Yes, this use case was very much the driver for the extra_cols - think of having a variable Temperature|Global Mean and then having an additional column climate_model to specify which climate model generated that timeseries. Rather than what we did previously, having something like Temperature|Global Mean|<Climate Model>.
What you seem to have in mind is more akin to #287 - tracking which operations modified specific subsections of the data. This should, in my opinion, not be part of the data dataframe.
I suggest the following way forward:

(with your permission) merge PR #352 because it still fixes a bug identified by this conversation and enables the original use of extra_cols
have a conversation what is strictly necessary vs. what might be useful for AR6 (with @byersiiasa, @znicholls and others)
start a new issue here or in a related repository on how to handle comments, annotations and workflow tracking

The merge fixes the most acute of the problems, so go ahead with that. This issue doesn't currently exist in emissions data since these columns are unused. I think the analogy with how things work there would be Temperature (FaIR model)|Global Mean, since different simulations are effectively different measurement metrics.

closed via #352


Make sure Start is only called once the old scene is unloaded

Fixes #193

:tada: This PR is included in version 8.2.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


tensorflow placeholder for vertex and transition

 self.in_vertex1 = tf.placeholder(tf.float32, [None, self.in_nv_dims], "vertex1")
            self.in_vertex2 = tf.placeholder(tf.float32, [None, self.in_nv_dims], "vertex2")
            if self.diffusion:
                self.power_transition1 = tf.placeholder(tf.float32, [None, self.maxpower, None], name="power_transition_matrices")
                self.power_transition2 = tf.placeholder(tf.float32, [None, self.maxpower, None], name="power_transition_matrices")
                input1 = self.in_vertex1, self.power_transition1
                input2 = self.in_vertex2, self.power_transition2

Can I know why None is used here instead of an actual value?

None indicates that the shape in that dimension is not known ahead of time. Looks like placeholders me ght be deprecated now, but see here for a similar idea: https://www.tensorflow.org/api_docs/python/tf/TensorShape


4.8.1 Gameplay Cue Definition

I'm using a mixed replication ASC in my listen server game in UE 5.0.3 and On Removed does not fire twice for Gameplay Cue Actors (added via duration GEs). On Added does fire twice just like the documentation suggests. Is this an oversight in the documentation?

From what i know:
Did you apply the GE (which is stack-able) twice?
So this is intended, because it's the same tag, same cue.
Your second OnActive call is from a RPC explicitly.

The GE is only applied once and only by the Listen Server. According to the documentation, this should trigger both On Added and On Removed twice on the Listen Server for the GC associated with the GE (due to minimal effect replication netmulticast), however, from my testing, only On Added is called twice, On Removed is only called once (thankfully).


使用glide4.6.1点击返回键时异常

Caused by: com.jkb.fragment.rigger.c.c: Can not find Puppet annotation.please add Puppet annotation for the class com.bumptech.glide.manager.SupportRequestManagerFragment
        at com.jkb.fragment.rigger.rigger.Rigger.getRigger(Unknown Source:57)
        at com.jkb.fragment.rigger.rigger._Rigger.dispatchBackPressed(Unknown Source:45)
        at com.jkb.fragment.rigger.rigger.Rigger.onBackPressed(Unknown Source:9)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.jkb.fragment.rigger.b.a.h(Unknown Source:26)
 ```

@ufbeans 感谢反馈，周末会对你提出的这种场景进行尝试，如果有问题会尽快解决。

@JustKiddingBaby 提交了。你有时间看一下

@JustKiddingBaby since no one is going to use Glide fragments by hand, you could just ignore any instance of com.bumptech.glide.manager.SupportRequestManagerFragment and not throw a RiggerException


If emacs reports a version number with three digits, packages are not included in the load path

Due to a change in how the constant 'doom-packages-dir' is initialized, emacs versions with more than three numbers will not be able to load packages downloaded by cask.
The current version of emacs in Arch, for instance, is 25.1.1. In it's current state, cask packages fail to load.

I recall addressing this issue recently.
Here's the problem: in a recent update, Cask changed where it downloads plugins to:
It installs "bootstrap" packages to .cask/25.1.1/bootstrap, and installs the actual packages to .cask/25.1/elpa. I don't know why they made this change. It used to be all in the same folder (.cask/25.1.1/{elpa,bootstrap}/*). doom-packages-dir correctly accounts for this, but it won't move over your plugins.
So here are two solutions:

Move the old packages to the new place:

mkdir ~/.emacs.d/.cask/25.1
mv ~/.emacs.d/.cask/25.1.1/elpa ~/.emacs.d/.cask/25.1/elpa

Or reinstall your plugins with make install

Sorry about the sudden change! I don't even know if this is a bug on their end or intended behavior!

Ah, I found the relevant commit in Cask

I'll close this for now. I hope the above solution helped! Let me know if it didn't.


Representing enumeration values

When a property is of type enumeration, we had not a clear expectation on how the value should be represented. In different places we were expecting:

an EnumerationLiteral
a String
a DynamicEnumerationValue

With this PR we define the interface EnumerationValue (and the class EnumerationValueImpl, previously called DynamicEnumerationValue). The standard approach should be to use this to represent enumeration value. This will make it clear that we are not representing just a string or the definition a new enumeration literal but representing an instance of an enumeration corresponding to a certain literal.
Users can still decide to represent enumeration values in other ways, if they wish so, but in that case they would need to provide primitive serializers and deserializers. If they instead use the EnumerationValue serialization and deserialization will work out of the box.

Do we still have tests that use actual java enums?

Do we still have tests that use actual java enums?

I am not sure if we ever had such tests. However this comment gives me the idea that we may have support for using enum values to represent values of properties of Enumeration type. In that case, if we had an enum instance we would not not from that to which Enumeration and EnumerationLiteral it would be connected. However, knowing the property to which it is assigned we could get the Enumeration from there and then look at the EnumerationLiteral with the same name as the enum instance. Do you think it would make sense to add support for that as part of this PR?

Do you think it would make sense to add support for that as part of this PR?

Yes, as this PR claims to bring the flexibility to do both. Then we should also test it.

Yes, as this PR claims to bring the flexibility to do both. Then we should also test it.

This is not what I meant initially, but I think it is a good idea so I added support for using Enum instances to represent instances of EnumerationLiteral and added some tests for serialization and deserialization of such values


Interaction with Java's raw generic types is not possible

See for instance Javafx's method https://docs.oracle.com/javase/8/javafx/api/javafx/scene/control/TreeTableView.html#setColumnResizePolicy-javafx.util.Callback- it takes a Callback<TreeTableView.ResizeFeatures,Boolean> but do notice that ResizeFeatures defines one generic parameter, the method defines a callback that takes it raw though. In scalac, you would pass a javafx.util.Callback[javafx.scene.control.TreeTableView.ResizeFeatures[_], Boolean] and it would typecheck, but with dottyc it doesn't work

Regrettably reported this on the wrong github, please delete.


remove union mapping as :binary node when performing DepthFirst enumeration

When using an in statement on a union a lot of garbage ends up in the generated SQL.  There might be a chance that this line was not suppose to be checked in during this commit.  A reason I lean that way was another Union related change that was quickly removed after that commit here
cc @tenderlove maybe you have some more insights about this?

Backported in 9bfe504


test manifest params

related to #53

DV get user info API endpoint now available on prod

For testing, check, https://requests-mock.readthedocs.io/en/latest/mocker.html


It would be useful to be able to remap the keybindings

Refactoring of EventHandlers.coffee is on its way in a pull request.

Preparations done in pull request #33


Parse UserAgent without HttpContext

Is your feature request related to a problem? Please describe.
I'm trying to parse a UserAgent that is NOT coming from the request and I can't find an easy and efficient way of doing that.
Describe the solution you'd like
public class MyClass(IUserAgentParser parser) {
  var result =  parser.Parse("Mozilla/...");
  Console.WriteLine(result.Browser.Name);
}

Describe alternatives you've considered
I'm currently doing it like this:
    public class SimpleUserAgentService : IUserAgentService
    {
        public HttpContext Context => throw new NotImplementedException();

        public UserAgent UserAgent { get; private set; }

        public SimpleUserAgentService(string userAgent)
        {
            this.UserAgent = new UserAgent(userAgent);
        }
    }

        private Browser ParseBrowser(string userAgent)
        {
            var userAgentService = new SimpleUserAgentService(userAgent);
            var platformService = new PlatformService(userAgentService);
            var engineService = new EngineService(userAgentService, platformService);
            var browserService = new BrowserService(userAgentService, platformService, engineService);
            return new Browser(browserService.Name.ToString(), browserService.Version.ToString(), platformService.Name.ToString());
        }

it works OK, but it's highly inefficient.

What is your use case? Console application or ASP.NET web application?

Complete this with PR #469


Added RegisterCustomTypeFunc method and usage example.

The CustomTypeFuncs work great.
I figured it would be nice if folks could register custom types w/o having to touch the reflect package,
so I put together a small wrapper function to (hopefully) simplify usage.

Love it! will merge but against v6-development but I have to add tests to keep my 100% test coverage before merging to v6
Thanks! @johnniedoe , will let you know when it's done.

pull request being handled in #140 , needed to merge into v6-development and add coverage tests


[Release-1.31] - Update kine for multiple fixes

Backport fix for Update kine to pull in fix for failed compaction causing the next iteration to be skipped

#11269

Validated using latest commit on release-1.31 branch following https://github.com/k3s-io/k3s/issues/11269#issuecomment-2471861675
ubuntu@ip-172-31-11-239:~$ k3s -v
k3s version v1.31.2+k3s-4a3ea1c3 (4a3ea1c3)
go version go1.22.8
ubuntu@ip-172-31-11-239:~$ journalctl -u k3s |grep locked
Nov 13 19:41:33 ip-172-31-11-239 k3s[1668]: time="2024-11-13T19:41:33Z" level=info msg="Bootstrap key locked for initial create"
ubuntu@ip-172-31-11-239:~$


Looks like the 7.2 build broke something, it's not playing nice with the core and the belt handling.

Belt_expandable still found after 1 seconds.
Tried completely unedited files from release, same exactl error.  I have noticed the old version started doing something weird too, its suddenly buying pots to fill up the inventory when there's already potions filling the belt.   Not sure if it just doesn't like something that was changed about the game itself recently or..I dunno.
Exception in thread Thread-7:
Traceback (most recent call last):
File "threading.py", line 973, in _bootstrap_inner
File "threading.py", line 910, in run
File "bot.py", line 153, in start
File "bot.py", line 173, in trigger_or_stop
File "transitions\core.py", line 899, in _get_trigger
File "transitions\core.py", line 401, in trigger
File "transitions\core.py", line 1201, in _process
File "transitions\core.py", line 426, in _trigger
File "transitions\core.py", line 435, in _process
File "transitions\core.py", line 272, in execute
File "transitions\core.py", line 1123, in callbacks
File "transitions\core.py", line 1144, in callback
File "bot.py", line 274, in on_maintenance
File "inventory\belt.py", line 105, in update_pot_needs
File "inventory\belt.py", line 72, in cut_potion_img
File "utils\misc.py", line 94, in cut_roi
TypeError: 'NoneType' object is not subscriptable
Traceback (most recent call last):
File "main.py", line 96, in
File "main.py", line 88, in main
File "keyboard_init.py", line 886, in wait
KeyboardInterrupt
Press Enter to exit ...
Traceback (most recent call last):
File "main.py", line 100, in
EOFError
[17216] Failed to execute script 'main' due to unhandled exception!
Unsure if compile went poorly or if there's something interfering now..

Fix your belt hotkey

Yep, thanks.  Apparently this, and similar errors, are all showing up because SOMEONE decided to clear all the hotkeys for d2r... probably in one of the recent maints.  Couldn't figure out why a known good setup AND a new clean build both werent working suddenly.


Include md output: Planner bucket & Planner plan. Closes #4275

Closes #4275

Merged manually. Awesome work, thank you! 👏


Implement additional mean functions

Want to add this feature to enable integration with climate models.

This feature was implemented over several PRs (#74, #77, and #81), closing


Create a Firefox Photon theme

Because A-Frame is made by Mozilla, it would only make sense that there would also be a Firefox Photon theme.
This would result in this project changing into aframe-ui.

I did not even know this existed haha but A-Frame is more like "supported by" then "made by" Mozilla.
Hum.. I am not a huge fan of this option, the web is open and should stay that way.
But I guess this project could be adapted for other types of UI design. :)


Gitlab package name changed with 7.10.0, breaking idempotency and causing constant re-installation of Gitlab

The packages are now no longer named "gitlab" but "gitlab-ce" or "gitlab-ee". See https://gitlab.com/gitlab-org/omnibus-gitlab/blob/master/CHANGELOG.md for confirmation.
This breaks idempotency and Puppet (re)installs Gitlab on every run, because it doesn't find the expected package installed:
# puppet agent -t
Info: Retrieving pluginfacts
Info: Retrieving plugin
Info: Loading facts
Info: Caching catalog for gitlab.example.com
Info: Applying configuration version '1430360716'
Notice: /Stage[main]/Gitlab::Install/Package[gitlab]/ensure: ensure changed 'purged' to 'latest'
Info: /Package[gitlab]: Scheduling refresh of Exec[stop gitlab]
Info: /Package[gitlab]: Scheduling refresh of Exec[/usr/bin/gitlab-ctl reconfigure]
Notice: /Stage[main]/Gitlab::Config/Exec[stop gitlab]: Triggered 'refresh' from 1 events
Info: /Stage[main]/Gitlab::Config/Exec[stop gitlab]: Scheduling refresh of Exec[/usr/bin/gitlab-ctl reconfigure]
Info: /Stage[main]/Gitlab::Config/Exec[stop gitlab]: Scheduling refresh of Exec[/usr/bin/gitlab-ctl reconfigure]
Notice: /Stage[main]/Gitlab::Config/Exec[/usr/bin/gitlab-ctl reconfigure]: Triggered 'refresh' from 3 events
Notice: /Stage[main]/Gitlab::Config/Exec[start gitlab]/returns: executed successfully

Puppet calls /usr/bin/dpkg-query -W --showformat '${Status} ${Package} ${Version}\n' gitlab to find the package and its status and version, and this command fails because the package is now named "gitlab-ce" and not "gitlab".
Unfortunately I have no immediately great idea how to work around this problem, other than relying on parsing the value of gitlab::gitlab_branch and/or gitlab::gitlab_download_link. As in something like this:
if versioncmp($gitlab::gitlab_branch, '7.10.0') == 0
  or versioncmp($gitlab::gitlab_branch, '7.10.0') == 1 {

  $package_name = "gitlab-ce"
} else {
  $package_name = "gitlab"
}

...

package { 'gitlab':
  ensure   => latest,
  name     => $package_name,
  source   => "${download_location}/${omnibus_filename}",
  provider => $package_manager,
  require  => Exec['download gitlab'],
}

I find changing the package name to be a really bad move on GitLab B.V.'s part, especially for a supposedly backwards-compatible feature release.

perhaps a better fix would be to use the apt/yum repositories?

Not, it would not. The package name issue would be the same (or possibly even worse), regardless of where the .deb or .rpm file was downloaded from.

well, you haven't seen my fix yet :P

This is already open as #140.  I have a fix we we're working on because both the rpm names and package names change to include gitlab-ce.

@antaflos https://github.com/spuder/puppet-gitlab/compare/master...ahuffman:issue_140.  Check out my proposed changes there.  It fixes the gitlab and gitlab-ce issue as well as the default package name changes when using the gitlab_branch param.

Closing as duplicate of #140


#22299 Text hidden in sql editor by a strange diagonal white rectangle

Turned on Read metadata for semantic analysis by default.
Removed underline style.

verified


fix: balances.locks(AccountId32) require 1 argument

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[x] Bugfix
[ ] Feature
[ ] Refactoring

What's new?

[x] PR closes #3919

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[ ] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[ ] I've tested PR on mobile and everything seems works
[ ] I found edge cases
[ ] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot

[ ] My fix has changed something on UI; a screenshot is best to understand changes for others.

pay 30 usd

😍 Perfect, I’ve sent the payout
💵 $30 @ 42.28 USD/KSM ~ 0.71 $KSM
🧗 DY4SQF2iD456tH89aQtz5wv1EV3BbSW8wKKuMcwbmXaj1pM
🔗 0x5785213a9a9aa9e045b35be3279573805a13bbb96cf5382dba53d435caea84c0
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues

What do you think about converting this to an async function?

hi, sorry for the late response. hmm, it should be ok I think. on that file all the functions  output using Promise<>


Identifiable Array and TestStore

While doing the TCA tour videos I got stuck if I change the signature of the todo's to IdentifiedArrayOf (the suggested one).
Cannot assign to property: subscript is get-only
struct AppState: Equatable {
	var todos: IdentifiedArrayOf<Todo>
}

store.send(.todo(id: UUID(), action: .checkboxTapped)) {
        $0.todos[0].isComplete = true
}

Hi @ivangodfather, I'm going to convert this to a discussion and answer there, as we keep issues for bug tracking in the library. In the future, if you'd like to ask questions about episode content, our episode code sample repo might also be a better spot, especially for non-TCA episodes :smile:
https://github.com/pointfreeco/episode-code-samples


Implement compiler runtime

The runtime is needed to keep Lambda program semantics in the compiled form. It solves at least the following problems:

filling the gap between Lambda API and standard JavaScript API, e.g. Array and String prototypes;
replacing marshaling and unmarshaling procedures in the compiled form;
keeping the semantics of integer arithmetics by invoking and possibly polyfilling Math.imul;
providing native types not provided by JavaScript, e.g. complex values.

Obsolete. The current compiler doesn't work and it's going to be removed.


[New Port Request] sfeMovie

can you add that link "https://github.com/Yalir/sfeMovie" in vcpkg

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now.


Remove status section from README

No longer applies

Thanks for your contribution! Please replace this text with a description of what this PR is changing or adding and why, list any relevant issues, and review the contribution guidelines below.

[ ] I’ve reviewed the contributor guide and applied the relevant portions to this PR.

Contribution guidelines:

See our contributor guide for general expectations for PRs.
Larger or significant changes should be discussed in an issue before creating a PR.
Contributions to our repos should follow the Dart style guide and use dart format.
Most changes should add an entry to the changelog and may need to rev the pubspec package version.
Changes to packages require corresponding tests.

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

I'm okay with this, but some APIs can still evolve in the future e.g. adding support for iterables. That's captured in the semantic version though, so this warning isn't needed anymore.


shrink model

integer quantize
QAT?
prune?

INT8 quantization (while preserving inputs as floats) cuts  a second off of the inference time (the interpreter.invoke()).. so 4s not 5s..
I am closing this issue because i think i either need to go for a coral #3 or a smaller model #5


Inconsistencies in top nav and card layout

Description
Getting inconsistent top nav (sign in / sign out) and form card layout (one column / two column) occasionally.
Steps to reproduce
Not quite sure what triggered the differences but did something close to this:

Created a new account
Created a new form without a name
Noticed the form card in "Forms" was long (one-column layout)
Named the form file in "Edit"
Went back to "Forms" and noticed form card was smaller (two-column layout)
Navigating in French noticed the unauthenticated top nav with "Sign in" as an option appeared even though I was signed in
Went back to form and it reverted to signed in top nav as expected

Details

Browser: Edge
Operating system: Windows
Environment: App router PR preview env
Language: French
Viewport: Tablet

Expected behaviour
Predictable and consistent behaviour and layouts
Screenshots or videos
Unable to capture / replicate top nav change on tablet again
Card layout discrepancies

Potentially related to #3168

I was unable to reproduce this behaviour on Windows+Edge (or MacOS+Chrome). Is this error still happening for you?

Yes, getting the one-column layout with the creation of a new account and then the creation of a new form saved without a name.

Thanks for retesting. The language switching idea, nice, but hmm I still can't reproduce this locally in my windows virtual machine. Also it looks like the PR environment is no longer reachable - maybe it destroys itself after N days. So I couldn't test on my tablet.
We'll be doing some serious refactoring of the application code in the coming weeks. I suspect this bug will resolve itself but I'll leave it open for re-testing just encase.


Enhancement: Add action-yamllint

Add action-yamllint to .github/workflows/

@vk26 Thank you for your contribution! 🚀


Cdb better path directory

Open up smarter working directory when editing a path cell :

If the cell is empty, open the working directory to the last opened directory.
If the cell is filled with a path within the project, open the directory of this file.
If the cell is filled with a path outside of the project, open the root directory of the project.

Note : Please refactor by adding optional arguments at the end of arguments list, while Haxe allows optional arguments anywhere it's more practical to add them after non optional ones.


[Proposal] Remove readonly modifier from all SearchOptions properties (and most likely other types)

The problem
Currently, most of the types (if not all?) are marked with readonly all over the place. In some types it totally makes sense, though in some types it's just super inconvenient to have the property marked as readonly. One example is the SearchOptions type, which is used in things like SearchIndex's search method for the requestOptions argument:
export declare type SearchIndex = SearchIndex_2 & {
    readonly search: <TObject>(query: string, requestOptions?: RequestOptions & SearchOptions) => Readonly<Promise<SearchResponse<TObject>>>;
    readonly searchForFacetValues: (facetName: string, facetQuery: string, requestOptions?: RequestOptions & SearchOptions) => Readonly<Promise<SearchForFacetValuesResponse>>;
    readonly findAnswers: <TObject>(query: string, queryLanguages: readonly string[], requestOptions?: RequestOptions & FindAnswersOptions) => Readonly<Promise<FindAnswersResponse<TObject>>>;
};

Now imagine having an application, that conditionally needs to add more properties to the requestOptions argument, depending on some internal application logic, which could look roughly like this:
import {RequestOptions} from '@algolia/transporter';
import {SearchOptions} from "@algolia/client-search";

let requestOptions: RequestOptions & SearchOptions = {
    facets,
    filters,
    page,
    hitsPerPage,
};

if (someConditions)
{
    requestOptions.aroundLatLng = "some, value;

    if (someOtherConditions)
    {
        requestOptions.aroundRadius = "someValue";
    }
}

index.current.search(query, requestOptions);

This is currently strictly forbidden as SearchOptions explicitly says that the property is readonly, thus it cannot be assigned after its initial creation.
Now I have a couple of options, which all are equally a hack and come with potential side effects, that I'm not very fond of:
Option 1: Remove the readonly modifier within my app code
We could simply remove the readonly modifier from the type like so:
import {RequestOptions} from '@algolia/transporter';
import {SearchOptions} from "@algolia/client-search";

type Writeable<T> = { -readonly [P in keyof T]: T[P] };

let requestOptions: RequestOptions & Writeable<SearchOptions> = {
    facets,
    filters,
    page,
    hitsPerPage,
};

if (someConditions)
{
    requestOptions.aroundLatLng = "some, value;

    if (someOtherConditions)
    {
        requestOptions.aroundRadius = "someValue";
    }
}

index.current.search(query, requestOptions);

The potential downside is that we'd make it appear that some property, which really should be readonly, not readonly and can be assigned. It's also not very forward stable.
Option 2: Don't use the Algolia types
import {RequestOptions} from '@algolia/transporter';
import {SearchOptions} from "@algolia/client-search";

let requestOptions: Record<string, string|string[]|boolean|number|undefined> = {
    facets,
    filters,
    page,
    hitsPerPage,
};

if (someConditions)
{
    requestOptions.aroundLatLng = "some, value;

    if (someOtherConditions)
    {
        requestOptions.aroundRadius = "someValue";
    }
}

index.current.search(query, requestOptions);

It sucks for multiple reasons: I'm not getting any auto-completion, real type and property checking and it's not forward stable.
Option 3: Remove the readonly from the type itself and use Readonly<SearchOptions> where necessary within Algolia
import {RequestOptions} from '@algolia/transporter';
import {SearchOptions} from "@algolia/client-search";

let requestOptions: RequestOptions & SearchOptions = {
    facets,
    filters,
    page,
    hitsPerPage,
};

if (someConditions)
{
    requestOptions.aroundLatLng = "some, value;

    if (someOtherConditions)
    {
        requestOptions.aroundRadius = "someValue";
    }
}

index.current.search(query, requestOptions);

The updated type definitions for SearchIndex would look like this:
export declare type SearchIndex = SearchIndex_2 & {
    readonly search: <TObject>(query: string, requestOptions?: Readonly<RequestOptions & SearchOptions>) => Readonly<Promise<SearchResponse<TObject>>>;
    readonly searchForFacetValues: (facetName: string, facetQuery: string, requestOptions?: Readonly<RequestOptions & SearchOptions>) => Readonly<Promise<SearchForFacetValuesResponse>>;
    readonly findAnswers: <TObject>(query: string, queryLanguages: readonly string[], requestOptions?: Readonly<RequestOptions & FindAnswersOptions>) => Readonly<Promise<FindAnswersResponse<TObject>>>;
};

Conclusion/Proposal
By far the best option is number 3 because I as a library consumer will get all of TypeScript's benefits and the only downside would be the amount of work necessary to refactor internal (and potentially external) APIs to use Readonly<SearchOptions> where really necessary.
This moves the readonly-check away from the assignment into the function and does exactly what you want: Not accidentally modify your input arguments.
A slightly dumbed down example can be seen here:

Current version with readonly properties: https://www.typescriptlang.org/play?#code/C4TwDgpgBAyhCGAnAxgCwPJmASwPYDsBnKAXigG8AoKGqRBAEwIBsQp4B+ALikOEWz4A5gG5qtevCb5WUAEbde-QaMoBfMZQBmAV3zIcBKFty45SKAAoG8YPB5wkaTIaIBKKDwBuubA0pUtFA2dgB08KRQAETwUWJBIfChcpFRcnHqlJTMEMDGpg4IKBhYeESRgbT20Sa4UQA0mZTYWlb8OhBuAeI0tcmp5ogZalm1g5a1biJAA
Proposed version with Readonly<T> arguments: https://www.typescriptlang.org/play?#code/C4TwDgpgBAyhCGAnAxgCwPJmASwPYDsBnKAXigG8AoKGqeAfgC4pDhFt8BzAbmtoCMmLNhx6UAvr0oAzAK75kOAlGm5c-JFAAUAE3jB4zAEoIdBADYgAPHCRpMSogD4AlFGYA3XNh2UqtKD0DADp4UigAIngI3gCg+GD+cIj+GIlKSnMIYBU1ZlsUDCw8InD-WkNI1VwIgBp0ymxpbTZZCBc-PhpqxOSNRDTxDOr+rWqXbiA

What do you guys think?
/cc @Haroenv

hey, thanks for the very thorough proposal.
I can only agree, it has been a bit complicated to handle those readonly params even readonly. They make sense and are perfectly logical, but too restrictive.
We are planning a next major for this client, and those types will most likely be gone. (No ETA)
In the meantime we could remove them but that would requires a proper major too to avoid issues with other users.

Is there anything I can help with speeding up this process? Or is this part of a major overhaul as part of a new major release?

I actually think removing readonly might not be a major version, as it's less restrictive: https://www.typescriptlang.org/play?#code/C4TwDgpgBAhgjFAvFA3gWAFBW1AThGAEwHsA7AGxCgCMAuKAZ2FwEtSBzTAX001ElgAmJKkw4a9Jqw7deGAGYBXUgGNgLMlHlwAFDFzt68AJSoeC5Wo2ktgvQaODTKc5m06UdKAHJvXY252nvS+-nLuwT5+sAxQKmRMAQpBXqExcQnASZjxpEywBnBGCMiRoeG6+uxwSfJ2VTVyuflVgo4iZX4V9uxOgT1OQA
Not sure if I'm missing anything though, maybe you can't cast or assume something is readonly anymore?

I'm not sure either, though I'd believe that it'd only really affect Algolia's internals if some new method code hasn't been updated to Readonly<T> and it's really trying to mutate the input params. Existing internal and external calling code shouldn't break with this change as we're not suddenly trying to mutate everything everywhere.
Only newly written code should be affected if it doesn't rely on Readonly<T>, which might be the only good reason against putting this change into a minor version as documentation and internal training would take place first since all engineers need to be made aware of this potentially dangerous change first, as they can no longer rely on getting automatic readonly on each input param, as they'd have to explicitly opt-in first.
At the end of the day I'm fine with either option: Major or minor version. That's totally up to you :)

I'd like to know if there is any progress on this issue?
I have to agree with @keichinger this is really cumbersome in many scenarios.

Hey @unwobbling, readonly have been removed in v5 (https://github.com/algolia/algoliasearch-client-javascript/tree/next), you can track development on https://github.com/algolia/api-clients-automation


Validate answer when forcing update

Is your feature request related to a problem? Please describe.
When my template includes questions with specific choices, there is no validation of the choice when I run copier update.
For instance in my .copier.yaml file I have
preferred_linter:
    help: What tooling would you like to use to enforce code style?
    type: str
    default: pylint
    choices:
        pylint: pylint
        black: black
        none: none

If I run the following update after creating the new project: copier --force --data preferred_linter="turtles" update, there will be no validation that the updated answer corresponds to one of the original choices. Subsequently, "turtles" is recorded in the .copier-answer.yml file, and the assorted jinji logic that was expecting one of ("pylint", "black", or "none") get's really confused :)
Describe the solution you'd like
Validate new answers entered via the CLI against the available .copier.yml file choices, types, validator, etc...
Describe alternatives you've considered
Other alternatives are to not use the --data CLI option. But this of course requires the user to go through all the questions again.
Additional context
Let me know if additional context is required.

Support for validating answers (including choices validation) provided via CLI/API was added by #962. Could you please try the latest Copier version from the master branch to confirm that your problem has been solved?

Yeah, this looks like it does what we're hoping for. Do you happen to have any idea when the next public release of Copier will be available?

It shouldn't belong anymore. See https://github.com/copier-org/copier/discussions/1058#discussioncomment-5452026.


[MOD-673] Provide instructions to a usr if a client auth fails with missing token

Handled client-side to tie the local lack of creds to the server-side AuthError
Hardcoding domain is a minor annoyance.

Nice!


APIM System assigned managed identity vs User assigned managed identity

The article states mentions a number of scenario's that are supported with user assigned managed identity such as:
Supported scenarios using user-assigned managed identity
Obtain a custom TLS/SSL certificate for the API Management instance from Azure Key Vault
You can use a user-assigned identity to establish trust between an API Management instance and Azure Key Vault. This trust can then be used to retrieve custom TLS/SSL certificates stored in Azure Key Vault. You can then assign these certificates to custom domains in the API Management instance.
If Key Vault firewall is enabled on your key vault, you can't use a user-assigned identity for access from API Management. You can use the system-assigned identity instead. In Key Vault firewall, the Allow Trusted Microsoft Services to bypass this firewall option must also be enabled.
With the regard to the last highlighted section is my question: If I complete disable public access on my key vault can I use a user assigned managed identity to obtains TLS/SSL certificates for APIM from a Key Vault?

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

ID: 1e6fa6cc-8307-4391-7cd9-2b6d5ccd9b7b
Version Independent ID: c78c84c8-99e6-9746-f3f2-f1f5098582a3
Content: Use managed identities in Azure API Management
Content Source: articles/api-management/api-management-howto-use-managed-service-identity.md
Service: api-management
GitHub Login: @dlepow
Microsoft Alias: danlep

@neok-g
Thanks for your feedback! We will investigate and update as appropriate.

Any update on this one?

Any update on this one?

At this time I'm closing this issue because we don't plan to make changes to our documentation about this feature. If you want to discuss this further with community experts, I suggest using Microsoft Q&A. Thanks for taking the time to raise this issue!
#please-close


[vRPex] Radio

If you quickly switch radio stations, you may receive an error that stops playback.

Fixed after cef update


InventoryClickEvent

[18:32:57 ERROR]: Could not pass event InventoryClickEvent to CrazyEnchantments v<IP_ADDRESS>

Please pastebin the whole error.

Please make a new issue if you still have a problem


Notification Sound on System

📑 I have found these related issues/pull requests
I have not found any related requests. I'm sure this wouldn't be hard to add.
🏷 Feature Request Type
New notification-provider
🔖 Feature description
Could you add a notification sound for when a service goes down? The monitoring software we use right now is end of life. The software we have right now alerts us visually and sounds an alert to let us know that something is down. Your software checks all of our boxes, except for the sound notification part. We don't want care to get an alert on our phones or teams etc. We have a TV that we have our current monitoring software on that sounds an alert when one of our services goes down.
✔ Solution
We would like to be alerted with a sound when a service goes down. Also add a sound alert for when you do add SNMP.
❓ Alternatives
No response
📝 Additional Context
No response

Duplicate of https://github.com/louislam/uptime-kuma/issues/3091

All of the notification types that existing are throw text message or some kind of messenger. It would be nice to have an option just for notification sounds straight from the program, and not having to reply on a 3rd party service. The duplicate request that you marked this under actually as nothing to do with that at all.

https://github.com/louislam/uptime-kuma/issues/3091 wants

detect a malfunction [..] notification is accompanied by a sound prompted on the web

You want

would like to be alerted with a sound when a service goes down

=> both this issue and #3091 want to be notified via a sound once a service is DOWN.
Am I missing somethign here?

I was requesting to add sound without having to go through a 3rd party service. I would have this up on a TV and would like the program to have its own sound for when something goes down. In easy terms, add sound wihtout going through a 3rd party app.

I don't know how to explain it any easier than that.

But that is exactly the feature that #3091 is requesting? Don't get what I am missing if anything.


[WIP] feature: store mode adds redis implementation

Ⅰ. Describe what this PR did
store mod adds redis implementation
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #2465 into develop will decrease coverage by 1.27%.
The diff coverage is 0.26%.


[ci][flexible-ipam] Use k8s 1.27.2 and containerd for e2e

/test-flexible-ipam-e2e

/test-flexible-ipam-e2e

/test-flexible-ipam-e2e

/test-flexible-ipam-e2e

/test-all

@gran-vmv @XinShuYang could you help to check if passed jenkins jobs are good enough to prove the change is ready to merge? we may move forward and back-port this as early as we can. Thanks.

@gran-vmv @XinShuYang could you help to check if passed jenkins jobs are good enough to prove the change is ready to merge? we may move forward and back-port this as early as we can. Thanks.

Yes, in this PR we only changed FlexibleIPAM related CI code and it passed jenkins-flexible-ipam-e2e and other required pipelines.


added TypeScript tutorial

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/getting-started-2017/media/vs-taco-tutorial-typescript/figure-01.png
:white_check_mark:Succeeded

Details

articles/getting-started-2017/media/vs-taco-tutorial-typescript/figure-02.png
:white_check_mark:Succeeded

Details

articles/getting-started-2017/media/vs-taco-tutorial-typescript/figure-03.png
:white_check_mark:Succeeded

Details

articles/getting-started-2017/media/vs-taco-tutorial-typescript/figure-04.png
:white_check_mark:Succeeded

Details

articles/getting-started-2017/media/vs-taco-tutorial-typescript/figure-05.png
:white_check_mark:Succeeded

Details

articles/getting-started-2017/vs-taco-tutorial-typescript.md
:white_check_mark:Succeeded

Details

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Validation Status: Errors
Articles

Article
Status
Details

vs-taco-tutorial-typescript.md
Failed
Details

vs-taco-tutorial-typescript.md

[Error] The following tags were not recognized: ms.technology, ms.prod.
[Warning] Remove the locale (culture segment) from links to the destination website pages:

 [Gulp tutorial](http://taco.visualstudio.com/en-us/docs/tutorial-gulp-readme/)

JobID: b818f928-14d5-466d-822b-0396980e076c

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/getting-started-2017/media/vs-taco-tutorial-typescript/figure-01.png
:white_check_mark:Succeeded

articles/getting-started-2017/media/vs-taco-tutorial-typescript/figure-02.png
:white_check_mark:Succeeded

articles/getting-started-2017/media/vs-taco-tutorial-typescript/figure-03.png
:white_check_mark:Succeeded

articles/getting-started-2017/media/vs-taco-tutorial-typescript/figure-04.png
:white_check_mark:Succeeded

articles/getting-started-2017/media/vs-taco-tutorial-typescript/figure-05.png
:white_check_mark:Succeeded

articles/getting-started-2017/vs-taco-tutorial-typescript.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Validation Status: Errors
Articles

Article
Status
Details

vs-taco-tutorial-typescript.md
Failed
Details

vs-taco-tutorial-typescript.md

[Error] The following tags were not recognized: ms.technology, ms.prod.

JobID: 0ccec033-bee2-4e26-88c5-21fb686cf9e2


joinRaw inside the belongsToMany

I create a belongsToMany relationship in the lucid models, like the diagram shown below:

However i put the car name on the car_name table which includes
car_name_id integer -- primary key
car_id integer -- foreign key to car table
name string -- value
language string -- language

when I join them in the controller, it seems have something wrong with it
Under the CarController.js
var language = params.language
var ret = await User.query().with('car', function(builder){
builder.select([
  'a.name',
  'car.model'
])
builder.joinRaw('LEFT JOIN car_name as a on a.car_id=car.id and a.language=?', [language])
}).first()

However, an error show below
Error: select car.a.name, car.car.model from car LEFT JOIN car_name as a on a.car_id=car.id and a.language='en' inner join car_user on car.id = car_user.car_id where user.user_id in (1, 2, 3)
it seems that car is auto added as the prefix in the select statement
As i tried to remove the builder.select() method and it's work,
but i still need to support multi language in here
how can i do that?

Hey @hinng0302! 👋
Could you create a repository with some code so I can reproduce this issue on my own?

sure! @RomainLanz
https://github.com/hinng0302/simple_adonis_joinRaw
I just created a sample for this issue.
// params.language will support different language for example: english, german, french
http://localhost:3333/get_all_users/car/language/:language

any luck?


Differentiate colours for escaped sequences in strings/regex

Hello,
It would be nice to see the parser identify escaped/special sequences and symbols in a string/regex with a different colour compared to regular strings, as this would help with string formatting and regex building, etc.
Thank you for your time and consideration.

Here are some example suggestions:

I hope you find these useful.


Where I can get password

It saves the password but site closes so where I can get that password

Same thing happened to me but thanks to Blackhathacker111 who helped me get the account hacked,message him on telegram his telegram username is https://t.me/Blackhathacker111


Dataverse should display name of Dataverse account user next to the repository

Reported by @jinluyuan on 0.43 trello board
Link: https://trello.com/c/lzluMd0Y/110-dataverse-should-display-name-of-dataverse-account-user-next-to-the-repository
Testing card: https://trello.com/c/WvbiX4Fu/78-user-settings-connect-dataverse-account
Issue
In User Settings, Dataverse should display "Authorized on Repository_Name by Dataverse_Account_User".

Mendeley, which has a similar structure to the Dataverse settings due to multiple accounts, does display the Mendeley account holder.

Comments
@sloria: Possibly defer this? It's not critical, and it might be desirable to show which Dataverse host was used. Ideally, we'd show both the user and the host, e.g. "Authorized by Jinlu Yuan on dataverse-demo...", but it's not absolutely necessary to get this in before releasing.

Noted -- this is/was anticipated functionality, I believe, due to Dataverse approach -- But good to aim for consistency.


[SUPPORT] Update cal data

📝 Description
Run import-cal-tokens under libs/ledgerjs/package.json to update CAL data.
Not sure it's necessary since, based on https://github.com/LedgerHQ/ledger-live/pull/4525 it's supposed to be done automatically at each release.
But since the script was broken (cf. https://github.com/LedgerHQ/ledger-live/pull/5195) and I need to update the CAL as part of https://github.com/LedgerHQ/ledger-live/pull/5173 anyways, let's update it in this independent PR in order not to mix everything up.
🔗  Related to:

https://github.com/LedgerHQ/ledger-live/pull/5195
https://github.com/LedgerHQ/ledger-live/pull/5173

❓ Context

Impacted projects: ``
Linked resource(s): []

✅ Checklist

[ ] Test coverage
[ ] Atomic delivery
[ ] No breaking changes

📸 Demo

🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

I think this test in not ran in this PR has no desktop file is impacted but I think you need to update cryptoassets.md by running: pnpm desktop test:jest src/generate-cryptoassets-md.test.ts

Good point, tested locally and it seems to already be up to date

I think this test in not ran in this PR has no desktop file is impacted but I think you need to update cryptoassets.md by running: pnpm desktop test:jest src/generate-cryptoassets-md.test.ts

Good point, tested locally and it seems to already be up to date

build before

I think this test in not ran in this PR has no desktop file is impacted but I think you need to update cryptoassets.md by running: pnpm desktop test:jest src/generate-cryptoassets-md.test.ts

Good point, tested locally and it seems to already be up to date

build before

Indeed! 🤦♂
Side note: looks like there is (still) to many "manual" or "have to know" steps in this process, therefore increasing chances of mistakes or oversight...

First easy step could be to document this process (even if "it's simple"). The weird thing being that you change stuff under libs/ledgerjs/ but it has an incidence on an MD file stored under LLD (apps/ledger-live-desktop/cryptoassets.md) that is itself generated by running a test 🤯 🤔
Second step would be to review this whole weird logic and make it a one step (I do one command and it builds the necessary deps + updates the CAL + the cryptoassets.md file)

cc. @haammar-ledger FYI


Add support for mjs and cjs extensions with CSS imports

Closes: #5562

[ ] Docs
[x] Tests

In the latest release of React Spectrum we added support for Node ESM, and now distribute a .mjs build. This appears to have broken the CSS loader with Remix: https://github.com/adobe/react-spectrum/issues/4118. Looks like the .mjs and .cjs extensions were missing in the esbuild plugin filter, so I've gone ahead and added them here.
Testing Strategy: Updated integration tests, and tested by patching an example app with React Spectrum.

Thanks for the PR! 🙏


Breakpoints are not working on web for external packages

Steps to Reproduce

I wrote an application which runs successfully on Android and iOS (in fact they are on production since some time ago).
The app is organized by packages (following clean architecture pattern), so I enabled the debugExternalPackageLibraries flag in my settings.json file to be able to debug all my code.
When I started configuring the environment to deploy web version I realized that the breakpoints which are outside the main package are not working (meanwhile they are working on Android and iOS platforms).
I mean, I am able to add a breakpoint at any file inside the main lib/ path, but if I try to add it on a package outside of this package, the Visual Studio marks it as a Unverified Breakpoint.
my_project
|
|__ lib
|      |__any_file.dart (can add breakpoint)  ✅
|
|_account_domain (any other package)
| any_file.dart (can't add breakpoint)  ❌
I tried different combinations of the settings.json file using these flags:
    "debug.allowBreakpointsEverywhere": true,
    "dart.debugSdkLibraries": true,
    "dart.debugExternalPackageLibraries": true,

I also tried to debug the project using Dev Tools in web browser, but in this case the tool doesn't allow to add breakpoints in external packages (the small white dot which shows that the line code admits a breakpoint does not appear at any line of any file outside the lib/ folder).
Also tried to downgrade Chrome version following this issue but still not working.
This is my launch.json:
{
    "version": "0.2.0",
    "configurations": [
        {
            "name": "Project Dev",
            "request": "launch",
            "program": "lib/main_dev.dart",
            "type": "dart",
            "flutterMode": "debug",
            "args": [
                "--flavor",
                "dev",
                "-t",
                "lib/main_dev.dart"
            ]
        },
        {
            "name": "Project Prod",
            "request": "launch",
            "program": "lib/main_prod.dart",
            "type": "dart",
            "args": [
                "--flavor",
                "prod",
                "-t",
                "lib/main_prod.dart"
            ]
        },
        {
            "name": "Project (profile mode)",
            "request": "launch",
            "type": "dart",
            "flutterMode": "profile"
        }
    ]
}

Code sample
Flutter analyze:
0 issues found. (ran in 10.6s)

Flutter doctor:
[✓] Flutter (Channel stable, 3.0.3, on macOS 12.1 21C52 darwin-arm (Rosetta), locale es-ES)
    • Flutter version 3.0.3 at /Users/jugarcs/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 676cefaaff (hace 6 días), 2022-06-22 11:34:49 -0700
    • Engine revision ffe7b86a1e
    • Dart version 2.17.5
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/jugarcs/Library/Android/sdk
    • Platform android-32, build-tools 33.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

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

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

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.1 21C52 darwin-arm (Rosetta)
    • Chrome (web)    • chrome • web-javascript • Google Chrome 103.0.5060.53

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Hi @jkydevelopment. Thanks for filing this issue. Could you please provide a minimal, reproducible example so that we can verify and investigate it?

Are you able to reproduce it using local packages? Maybe you could create a repo reproducing it that way

Yes, I'm able to reproduce it locally. I tried to copy the project and remove a lot of code just to reproduce the same situation but I'm not able to "break" the debugging as it's happening in my project. Could we make some live session or something like that to show it to you?

Maybe you could put in into a repository and share it here so that others on the team could also look at it

No I can't, my company doesn't allow to me to share it. Don't worry, I hope I can find the problem. If I do it, I will let you know about it.

I am not asking you to share proprietary code, but rather create a new project with sample code in a new repo. Let me know of you can do that instead :)

The problem is that I am not able to reproduce this issue in a new project, even I have tried to copy the same structure and removing a lot of unnecessary files. It's a bit annoying...

Unfortunately, if the issue is not reproducible, we cannot take action on it. If you are able to produce a sample that may reproduce this issue, please post it here. Until then, I am closing this issue. Thank you for filing it :)

Hi @exaby73 I found again the problem and I created a repository with the bug and the instructions to reproduce it.
REPO
I you have any question about it please ask for them.
Thanks in advance!

Hey @jkydevelopment. I am able to reproduce this on stable but due to a package you included not supporting the master channel, I am unable to test this on master as well. Can you modify the sample to work on master as well?
Logs
Launching lib/main_dev.dart on Chrome in debug mode...
lib/main_dev.dart:1
: Error: The non-abstract class 'FwfhTextStyle' is missing implementations for these members:
../…/lib/fwfh_text_style.dart:11
 - TextStyle.fontVariations
Try to either
 - provide an implementation,
 - inherit an implementation from a superclass or mixin,
 - mark the class as abstract, or
 - provide a 'noSuchMethod' implementation.

Hi @exaby73, thanks for your answer.
Currently I am on stable channel. How did you get that log? I have tried it capturing a whole session and I am not able to find anything related to that FwfhTextStyle class you mentioned.
About the channel, do you refer any lib I'm using is not supporting the master channel? How could I find it?

Thank you @jkydevelopment. I can reproduce this issue on stable and master. Though by the looks of it, it looks like a Dart issue rather than a Flutter issue. There is a similar issue on the dart-lang/sdk repo (https://github.com/dart-lang/sdk/issues/43197).
@DanTup is this issue the result of https://github.com/dart-lang/sdk/issues/43197?

I can reproduce this too. I don't think it's related to https://github.com/dart-lang/sdk/issues/43197 as that seems testing (and not-web) specific.
I captured a log while reproducing, and found that the web proxy is rejecting the breakpoint like this:
==> [VM
] {
	"jsonrpc": "2.0",
	"id": "1694",
	"method": "addBreakpointWithScriptUri",
	"params": {
		"isolateId": "1",
		"scriptUri": "file:///Users/danny/Dev/TestStuff/flutter_web_bug/account_presentation/lib/controllers/sign_in_controller.dart",
		"line": 21
	}
}
<== [VM
] {
	"jsonrpc": "2.0",
	"error": {
		"code": 102,
		"message": "The VM is unable to add a breakpoint at the specified line or function",
		"data": {
			"request": {
				"jsonrpc": "2.0",
				"id": "1694",
				"method": "addBreakpointWithScriptUri",
				"params": {
					"isolateId": "1",
					"scriptUri": "file:///Users/danny/Dev/TestStuff/flutter_web_bug/account_presentation/lib/controllers/sign_in_controller.dart",
					"line": 21
				}
			}
		}
	},
	"id": "1694"
}

I think this could be the same issue as https://github.com/dart-lang/webdev/issues/874. Interestingly, I note that you say this doesn't happen if you remove the import of package:router/router.dart and that import seems to introduce a cycle (controllers/sign_in_controller.dart -> package:router/router.dart -> package:account_presentation/account_presentation_library.dart -> controllers/sign_in_controller.dart) although I'm not sure if that's what's triggering it.
@annagrin @elliette do you think this is the same as https://github.com/dart-lang/webdev/issues/874, or should this be moved to a new dwds issue?

I think this is a separate issue. I've opened a new bug in DWDS for this (https://github.com/dart-lang/webdev/issues/1692). Closing this in favor of the new DWDS issue. Thank you!


Drawback with the position of the option

I have the following problem, I want my combo has the options at the bottom and not at the top as image puestra

Select2 will attempt to detect if it can open the dropdown facing down. If it believes it does not have enough space below, but there is enough space above it, it will swap the direction to face up.

This appears to be more of a usage question than a bug report or common feature request.  As we continue to keep the Select2 issue tracker geared towards these two areas, bug reports and common feature requests, we are redirecting usage questions to other communities such as the mailing list, IRC, or Stack Overflow.


allow a third element in position!

A position is an array of numbers.  There MUST be two or more
elements.  The first two elements are longitude and latitude, or
easting and northing, precisely in that order and using decimal
numbers.  Altitude or elevation MAY be included as an optional third
element.

Position could be up to 3 elements

I misread it's already in 🤦


Tables with hyphen character cause mysql client to drop connection

The mysql client likes to read table names and column names on startup to enable autocompletion:
> mysql -h"some_url"
Reading table information for completion of table and column names
You can turn off this feature to get a quicker startup with -A

If one of the tables it reads contains a hyphen, for example test-table then the following query fails to be parsed.
SELECT * from test-table LIMIT 0;

This causes the mysql client to think the connection was dropped.
Repro:
> dolt sql -q 'CREATE table `test-table` (pk int PRIMARY KEY);'
> dolt commit -am "add test table"
> dolt sql-server

# separately
> mysql -h"<IP_ADDRESS>" -u"root"
mysql> use test;
mysql> SELECT * from dolt_branches;
ERROR 2008 (HY000): MySQL client ran out of memory
No connection. Trying to reconnect...
Connection id:    5
Current database: old_format

ERROR 2008 (HY000): MySQL client ran out of memory
No connection. Trying to reconnect...
Connection id:    6
Current database: old_format

ERROR 2008 (HY000): MySQL client ran out of memory
mysql>

dolt sql-server logs:
2022-07-18T15:10:47-07:00 WARN [conn 6] error running query {connectTime=2022-07-18T15:10:47-07:00, connectionDb=old_format, error=syntax error at position 20 near 'test', query=SELECT * FROM test-table LIMIT 0;}
2022-07-18T15:10:47-07:00 INFO [conn 6] ConnectionClosed {}

The workaround is to supply the -A flag when using mysql to connect to a GMS backed server like Dolt.


[SUGGESTION] Remove highlight on unselection

When selecting an object from the tree and having the highlight enabled, it will create a transparent red-ish frame within its borders. That is intended behavior. However, when you click on the blank area of the tree, it "unselects" whatever you had selected. What should happen is that the highlight should also go away when unselecting an object, but it instead stays to the last-selected object. I love the highlight as it makes navigating the tree visually appealing, however, the red highlight obstructs the view and there is no way to disable it without disabling the entire feature.

hey @chky-nmnsoftware this is a great suggestion!
I will look into implementing it soon!


[WIP] Add Docker Desktop check

This is a work in progress, but could help for future use cases with Docker Desktop on macOS and WSL2.

Yes, I like where this is going!
I'd like to add a section specifically for runtime-specific preparations for the cluster.
In there, we can then distinguish per OS and per installation variant (like here: DfD and docker-machine) :+1:

Hey @iwilltry42 - Apologies I got side tracked on this PR. Did any logic from...

I'd like to add a section specifically for runtime-specific preparations for the cluster.
In there, we can then distinguish per OS and per installation variant (like here: DfD and docker-machine) 👍
...come in release v4.0.0?

Hey @iwilltry42 - Apologies I got side tracked on this PR. Did any logic from...

I'd like to add a section specifically for runtime-specific preparations for the cluster.
In there, we can then distinguish per OS and per installation variant (like here: DfD and docker-machine) 👍
...come in release v4.0.0?

Hey @nickgerace , no, unfortunately, this didn't get in the release.
However, I'm just now thinking, that we might benefit from an init() function, which populates some (to be created) fields of the chosen runtime (struct), like OS, arch, etc.
What do you think?

Hey @nickgerace , no, unfortunately, this didn't get in the release.
However, I'm just now thinking, that we might benefit from an init() function, which populates some (to be created) fields of > the chosen runtime (struct), like OS, arch, etc.
What do you think?

@iwilltry42 this sounds like a great idea. I'll close this PR open for now and we can talk about it.


fixed: Dictionary should have pointer to pointer

Dictionaries built from cgo pointers should keep the reference to the original pointer.
For instance, avformat.Context has a MetaData() that returns a dictionary.
This fix makes it return always a non nil Dictionary that, when modified, will alter the metadata of the Context object.

Current coverage is 19.85%

Merging #10 into master will increase coverage by +0.40% as of d3dcbb5

@@            master     #10   diff @@
======================================
  Files            4       4
  Stmts         2349    2357     +8
  Branches        59      61     +2
  Methods          0       0
======================================
+ Hit            457     468    +11
- Partial         24      26     +2
+ Missed        1868    1863     -5

Review entire Coverage Diff as of d3dcbb5

Powered by Codecov. Updated on successful CI builds.


fix(lualine): weird path truncation when not under cwd/root

Description
It does not make sense to truncate unless root actually in path.
Also, should check if opts.relative == "root"
otherwise the substitution can be surprising
(since user expects substitution only wrt. cwd)
Related Issue(s)
N/A
Checklist

[x] I've read the CONTRIBUTING guidelines.

I agree about the check for root existing in path, but not with the condition about opts.relative = "root". Without it, it always shows the first part in the root directory component, whereas with opts.relative = "root" it doesn't show it.
For example if root is ~/projects/plugins/LazyVim and cwd = ~/projects/plugins/LazyVim/lua/lazyvim/plugins if I open a file (for example util/lualine.lua) then in lualine it will be printed lua/.../util/lualine.lua without the check for opts.relative = "root". With the check opts.relative = "root" that you're suggesting in lualine it will be printed /.../util/lualine.lua.
This is just my own personal preference as a simple user but I prefer how it's printed without the check for opts.relative = "root" because it also shows the first part of the path relative to the root dir.

bump.  still an issue I ran into as well and ended up filing a duplicate PR to fix.

I don't see what this fixes.
If I open a file that's not in my cwd, then I want to see the file like it is currently implemented.
root will always be part of the path, so that's why there was no check for that

@folke did you see the description and screenshots in #4499, which was a dupe of this.  Does that help explain what this fixes?

Just checked and I can't reproduce that.
With the exact cmd op used, I get a properly formatted path...

Added an extra check just to be sure


pypodman: 'attach' doesn't

I think pypodman/lib/actions/attach_action.py got created as a skeleton but never actually implemented. I'm sorry, rewriting it is currently more than I can dive into right now.

@edsantiago It looks like I got run and attach confused.  I'll take a look at attach tomorrow.  The non-CLI attach works so we should be close.


feat: add children to figure schema, setup basic demo

Jira
http://vjira2:8080/browse/BDS-1781
Summary
To bring Twig schema more in line with WC schema, I'm demoing a potential new pattern: passing inner renderable items (children) as an array of objects, with an optional slot.
Relates to #1548.
Details
(Explain the changes in enough detail fo reviewers to understand.  Raise any questions for reviewers to consider.)
How to test
(List clear steps to test that the changes are working)

@colbytcook @remydenton closing this PR. I don't see us going in this direction, using a generic "children" prop to pass an array of items, which are then sorted into slots inside the Twig template.
I think it's simpler and more consistent to add flat, component-specific props with type "array" when we need to do this sort of thing, and handle it on a case-by-case basis, rather than introducing a whole new "children" pattern.
Keeping branch open if you want to take a look.

Thanks @danielamorse, I think that's the right decision.
As a general rule, I'd like to keep components as decoupled as possible.  So whenever possible (and I think 95% of the time it is), instead of this kind of thing:
{% if image %}
    {% include<EMAIL_ADDRESS>with image only %}
{% elseif icon %}
    {% include<EMAIL_ADDRESS>with icon only %}
{% elseif video %}
    {% include<EMAIL_ADDRESS>with video only %}
{% elseif table %}
    {% include<EMAIL_ADDRESS>with table only %}
{% endif %}

just do this, leaving the inclusion of the child pattern to the consumer:
{{ content }}

Is there a reason we couldn't do that in this case?  Not that we need to spend time cleaning it up now, just curious if this is a use case where my proposed pattern wouldn't work.

@remydenton agreed we shouldn't do this:
{% if image %}
    {% include<EMAIL_ADDRESS>with image only %}
{% elseif icon %}
    {% include<EMAIL_ADDRESS>with icon only %}
...

{{ content }} suffices. It leaves the door open to "unapproved" content, but that's something we can address with documentation rather than strictly preventing it via schema.
The "children" pattern gives us a little more control over how each item is output. You might put one item in a slot or wrap another in a <div>. Still, these are things we can do already by other means.

Cool.  I think a children pattern is fine if you're trying to add wrappers.  I would still keep it agnostic though, i.e.:
{% for children as child %}
  <div>
    {{ child }}
  </div>
{% endfor %}

The ability to pass unapproved content is one of the few downsides to that approach.  IMO, other approaches that force certain content have far more downsides, and I have yet to see a case where someone actually tries to pass something unapproved.  At the end of the day, it's the same as passing an invalid value for any of the other props (there's nothing stopping you, but it likely won't work).


Improve documentation

Hi,
just discovered this webpack plugin and it may be exactly what I need.
However I have a hard time finding out the intended workflow (I use Tampermonkey in Google Chrome on Windows 10), so I have two questions regarding "Hot Development":
1.)
You wrote that webpack-dev-server will build the userscript in watch mode. How do you run webpack-dev-server? Simply call it on the command line? Which arguments? Usually I insert a script key to package.json to use npm run dev or something like that to run webpack-dev-server --mode=development --hide-modules --devtool inline-source-map --hot. I learned that both --devtool inline-source-map and --mode=development blows up my generated user.js to >1 MB and that lets the browser freeze. Without those arguments it seems to work, but that means production mode and my js is minimized.
2.)
I use this in my webpack.config.js:
plugins: [
    new WebpackUserscript({
      metajs: true,
      headers: {
        "name": "Test",
        "version": dev ? "[version]-build.[buildNo]" : "[version]",
        "downloadURL":  "https://mydomain/Test/Test.user.js",
        "updateURL": "https://mydomain/Test/Test.meta.js",
        "match": "*",
        "grant": [
          "GM.getValue",
          "GM.setValue",
          "GM.deleteValue"
        ]
      },
      pretty: true
    })
  ]

As far as I understand that leads to an increasing buildNo and therefor version number everytime I save my script and webpack-dev-server watching it compiles it? At the beginning I installed the script from http://localhost:8080/Test.user.js but what if a new version has been compiled?
Do I need to instruct Tampermonkey to look for updates then? In my case Tampermonkey does not find a new version. Maybe I may not use downloadURL and updateURL headers because they point to different URLs?
Could you please share how exactly you are working or how this is intended to use?

For your question 1, actually I think the arguments of webpack-dev-server are all up to you.
The point here is that a HTTP server is required to host your userscript. Luckily webpack-dev-server can serve it for you with additional functionalities like rebuilding the userscript when source files are changed.
In the old workflow, when developing userscript, people usually just copy their source code and paste it to the TamperMonkey editor panel. The webpack-dev-server (or any other HTTP server) provides another way to transport the source code to TamperMonkey; that is, to install the userscript via URL.

For question 2,

As far as I understand that leads to an increasing buildNo and therefor version number everytime I save my script and webpack-dev-server watching it compiles it?

Yes, the build number increases when the userscript is compiled and the compilation is triggered by saving on source files.

Do I need to instruct Tampermonkey to look for updates then?

Yes, you need to click on the recent update field and TamperMonkey will do the update for you.

In my case Tampermonkey does not find a new version. Maybe I may not use downloadURL and updateURL headers because they point to different URLs?

It's actually the issue of how TamperMonkey detects version changes. If your https://mydomain/Test/Test.meta.js does not contain a version high than your installed one, it is expected to have no update.

Thanks for your answers. Regarding 2:
What would be needed then is a method to distinguish the dev mode so that the metadata "downloadURL" and "updateURL" are not written to the user script (or include the dev localhost url ) and a production mode, so when I do e.g. a npm run build the two metadata tags are actually written.

Usually, people use the environment variable NODE_ENV set to production or development to distinguish which mode should be used under current environment.
You can take a look at the example of Hot Development, especially how dev variable is determined and is used.
new WebpackUserscript({
      headers: {
        version: dev ? `[version]-build.[buildNo]` : `[version]`
      }
    })

OK, I got it to work.
But this code from your example does not work in webpack 4.32.2 which I was using:
const dev = process.env.NODE_ENV === 'development'
process.env.NODE_ENV is always undefined.
What works for me:
module.exports = (env, argv) => ({
  mode: argv.mode,
  ...
  plugins: [
    new WebpackUserscript({
      headers: {
        "name": argv.mode === "development" ? "Test-Debug" : "Test",
        "version": argv.mode === "development" ? "[version]-build.[buildNo]" : "[version]",
        "downloadURL": argv.mode === "development" ? "" : "https://mydomain/Test/Test.user.js",
        "updateURL": argv.mode === "development" ? "" : "https://mydomain/Test/Test.meta.js",
        "grant": [
          "GM.getValue",
          "GM.setValue",
          "GM.deleteValue"
        ]
      },
    })
  ]
});

Setting downloadURL and updateURL empty in dev mode leads to not outputting these meta tags in dev mode, so hot module updating works now.

argv is also a feasible way.

But!

process.env.NODE_ENV is always undefined.

That's just as expected since the NODE_ENV environment variable should never be set by any framework but yourself.
I assume that you are not familiar with some of the NodeJS conventions, I suggest you to do more search about NODE_ENV, or at least read this one.
If your system is Linux, you can export the NODE_ENV set to "development" or "production" via
export NODE_ENV=production

in your system, or you can set it per command, for example,
NODE_ENV=production webpack-dev-server

The NODE_ENV variable only exists during this runtime of webpack-dev-server, also you got to know that the syntax in this example only works on Linux, for Windows, try cross-env.

OK, I assumed that NODE_ENV is set to either development or production automatically if you call webpack-dev-server with --mode=development or --mode=production (just during runtime of webpack-dev-server of course).
Either way it works fine for me now. Thanks a lot for a great plugin!

Happy scripting!

Really love this project and sorry for resurrecting this post – with the config described here updating the build number works fine, my webpack-dev-server is always serving the latest version and the page is reloaded when I make a change to my script.
However, I have to manually instruct TamperMonkey to update the userscript by clicking on the "last update" timestamp for my userscript. Is there a way to get TamperMonkey to always fetch the latest version when reloading the page, without having to manually click on the update button ("last update" timestamp)?

A nasty work around I am using for now:
I have created another userscript with a @require directive that points to my userscript, eg. // @require https://localhost:8080/my-userscript.user.js. The trick is to make sure TamperMonkey does not cache externals, can be configured as described in https://github.com/Tampermonkey/tampermonkey/issues/475. However, I still need to reload the page twice to receive the update.

@georgms Sorry but I have not found any way to force Tampermonkey update the script on each page load either. BTW, why the page reload should be twice to receive the update?

@momocow

BTW, why the page reload should be twice to receive the update?

I really don't know but that's the way it is for me. Sometimes I have to reload three times so it's probably a matter of how long I wait rather than how often I reload.
Anyway, this method works sufficiently well for me. Maybe it's something to add to the docs?


Set default branch to main in manifest

Sets default branch in the manifest file to main so that you can run the pipeline without specifying the version using -r [TAG].
Without this fix, if you run like nextflow run phac-nml/mikrokondo ... you will get the following error:
Project `phac-nml/mikrokondo` is currently stickied on revision: main -- you need to explicitly specify a revision with the option `-r` in order to use it

This was a change that was already previously done with e.g., SNVPhyl:
https://github.com/phac-nml/snvphylnfc/blob/adafa83a72caf490963db588f334f36254af156a/nextflow.config#L213
This PR is going to main so that the current release of the pipeline will run after merging.

Thank you for this!


Bug in the PublisherTelemetryConfiguration.cs file - Deserializing Interface Properties

Hi Everybody,
I Discover a problem with the new issue of iot-edge-opc-publisher. The problem is related to the Deserializing interface properties in the "PublisherTelemetryConfiguration.cs" row 600:
_telemetryConfiguration = JsonConvert.DeserializeObject(await File.ReadAllTextAsync(PublisherTelemetryConfigurationFilename).ConfigureAwait(false));
Every time that I configure the telemetry file or use the default telemetry in the release, the program goes fail and the message is:
Loading of the telemetry configuration file failed. Does the file exist and has correct syntax? Exiting...
Newtonsoft.Json.JsonSerializationException: Could not create an instance of type OpcPublisher.ITelemetryConfigurationFileModel. Type is an interface or abstract class and cannot be instantiated. Path 'Defaults', line 23, position 15.
Of course the file exist and is configured correctly as per default.
Take note that with the previous version of OPC Publisher solution it work fine.
Some one know hot to fix or how to contact the support?

exactly the same problem here. no solution right now. :-/

Hi Eichi87,
thank you now I know that I'm not alone.
Do you have also problem regard the communication? I mean that after 1 hour (more or less) that the publisher is active, appear a message that say:
2019-02-11 09:49:25.572 +01:00 [ERR] Connection status changed to 'Disconnected_Retrying', reason 'Communication_Error'
2019-02-11 09:49:25.802 +01:00 [INF] Connection status changed to 'Connected', reason 'Connection_Ok'
and seems like that the publisher is freezed?

I will test that

Hi Eichi87,
I'm using the current version and that happen on it.
This is my personal mail in case we wont to have a private conversation about this<EMAIL_ADDRESS>Thank you

Any information on this? Just updated the images in our IoT clients and now it's broken...

Hi lucianoctorres,
nothing new. I think there is a bug in the software (the interface is wrong).

I close the issue, they fix the problem. Now it work