SqlException (0x80131904): Principal 'xyz' could not be resolved: How can I add a Managed Identity to Azure Sql Server when running under a Service Principal?

I want to add a Managed Identity (coming from an App Service) to Azure Sql Server.
I created an AAD group where a group of my team and the Service Principal is part of.
AzureSqlAdminGroup = TeamGroup + Service Principal
This AAD group is added as an Azure Sql admin during the provisioning of the Azure Sql Server.
When I run CreateSqlUserFromManagedIdentity  under my personal account everything works fine. Whereas when I run the code under a service principal, Sql Server tells me that it can not resolve the Managed Identity of my app service and that the Service Principal doesn't have the permissions to do so.

System.Data.SqlClient.SqlException (0x80131904): Principal 'xyz' could not be resolved. Error message: ''
2020-06-10T16:34:12.6605990Z Cannot add the principal 'xyz', because it does not exist or you do not have permission.
2020-06-10T16:34:12.6606728Z Cannot add the principal 'xyz', because it does not exist or you do not have permission.
2020-06-10T16:34:12.6607420Z Cannot add the principal 'xyz', because it does not exist or you do not have permission.

Code:
public async Task CreateSqlUserFromManagedIdentity(string managedIdentityName, params string[] roles)
{
    var credential = new DefaultAzureCredential(new DefaultAzureCredentialOptions
    {
        ExcludeVisualStudioCredential = true,
        ExcludeVisualStudioCodeCredential = true
    });

       var accessToken = await credential.GetTokenAsync(new TokenRequestContext(new[] { "https://database.windows.net/.default" }));

    var sqlConnectionStringBuilder = new SqlConnectionStringBuilder(_connectionString);
    var stringBuilder = new StringBuilder();

    stringBuilder.AppendLine($"IF DATABASE_PRINCIPAL_ID('{managedIdentityName}') IS NULL");
    stringBuilder.AppendLine("BEGIN");
    stringBuilder.AppendLine($"\tCREATE USER [{managedIdentityName}] FROM EXTERNAL PROVIDER;");
    stringBuilder.AppendLine("END");
    Console.WriteLine($"Adding Managed Identity '{managedIdentityName}' to '{sqlConnectionStringBuilder.DataSource}\\{sqlConnectionStringBuilder.InitialCatalog}' with roles ...");

    foreach (var role in roles)
    {
        Console.WriteLine($"\t{role}");
        stringBuilder.AppendLine($"ALTER ROLE {role} ADD MEMBER [{managedIdentityName}];");
    }

    await using var sqlConnection = new SqlConnection(_connectionString) { AccessToken = accessToken.Token };
    await sqlConnection.OpenAsync();

    var sqlCommand = sqlConnection.CreateCommand();
    sqlCommand.CommandText = stringBuilder.ToString();
    await sqlCommand.ExecuteNonQueryAsync();
    ConsoleEx.WriteSuccessLine("successfully");
}

How can I add a Managed Identity to Azure Sql Server when running under a Service Principal?
See also: https://github.com/MicrosoftDocs/sql-docs/issues/2323

@Rookian -- Alex, thank you for your question.  Are you following a particular docs.microsoft.com article?  Meanwhile, please consider these resources:

DBA Stack Exchange
Stack Overflow

@Rookian -- Alex, thank you for your question. Are you following a particular docs.microsoft.com article? Meanwhile, please consider these resources:

DBA Stack Exchange
Stack Overflow

I follow this article: https://docs.microsoft.com/en-us/azure/app-service/app-service-web-tutorial-connect-msi
But I use Azure.Identity instead and use SqlConnection (see my code) instead of sqlcmd.

@Rookian -- Alex, thank you for clarifying.  This repo is the place to create a documentation issue, and you're welcome to add your product feedback here:

SQL Server Feedback

@cephalin -- Cephas, please look into this issue.

@Rookian - I think someone has already mentioned this however a a service principal cannot be added directly as a user of a Azure SQL Database.  You can however create a group in your Azure Active Directory and grant the group access to the SQL Database.  You can then add the service principal to the group.

@alexmbaker that workaround works fine thank you. But unfortunately this is not something that works within my organisation where we (service principal account) don't have the permission to create AAD security groups or assign members to such a group.

@alexmbaker, @cephalin and @WilliamAntonRohm  we have a found a working solution: https://github.com/MicrosoftDocs/sql-docs/issues/2323#issuecomment-652907219


Immer + withSlices discussion

Background
When using immer with zustand-slice's withSlices middleware an error arises:
Error: [Immer] An immer producer returned a new value *and* modified its draft. Either return a new value *or* modify the draft.

I cloned the zustand-slices library and changed some of the package's code locally to make Immer "work" using withSlices in a small repo. I put "work" in quotations because it's more of a workaround than an absolute fix.
Issue Explanation
The set function in with-slices.ts shows that nextState is always returned, which explains the issue because when passing Immer middleware the state is converted into a draft state for you to operate with mutable syntax on immutable data structures. However, Immer requires that you either modify the draft directly or return a new state, not both.
Proposal
To fix this, I suggest adding a withImmer (or immerEnabled) boolean flag to the slice configuration, which withSlices can use to decide how to handle state updates. Like so:
export type SliceConfig<
  Name extends string,
  Value,
  Actions extends SliceActions<Value>,
> = {
  name: Name;
  value: Value;
  actions: Actions;
  withImmer?: boolean;
};

Usage:
const userSlice = createSlice<'user', UserState, UserActions>({
  name: 'user',
  withImmer  : true,
  value: {
    name: 'John Doe',
  },
  actions: {
    setName: (newName) => (prev) => {
      prev.name = newName;
    }
  },
});

const useUserStore = create()(immer(withSlices(userSlice)))

This proposal ensures that withSlices can handle state updates correctly, whether or not Immer is used. One thing thing not mentioned is potential error handling, for example if withImmer property is used but immer middleware is not passed. This can be a potential drawback that may need to be addressed.
proof of concept:

Conclusion
This proposal is a hypothetical solution, and its feasibility would need further evaluation. I investigated this issue because the conflict between Zustand's Immer middleware and the zustand-slices library was a dealbreaker for me. I dislike using Immer's produce directly because the middleware's advantage is that it allows you to reap the benefits of Immer without having to explicitly manage it, similar to Redux Toolkit.
Immer's produce needs to be called in every single action, which is very tedious and gives me PTSD, lol. If I could just add a withImmer flag or some equivalent to apply Immer middleware to every single action, that would be awesome.

Thanks for opening up the discussion.

The set function in with-slices.ts shows that nextState is always returned

So, is this TypeScript only issue? Does it work just fine with JS without types?

I suggest adding a withImmer (or immerEnabled) boolean flag

That's not acceptable for some preferences (of mine 😄 ):

It should be used without immer and it shouldn't increase even a byte in the production code.
immer middleware has a "mutator" type, so technically we could support create()(...) syntax, but avoiding the syntax is one of my motivation of developing this library.
Basically, I'd prefer and recommend bare produce which is explicit and type safe.

That said, what would be acceptable is a type-only solution or even a separate util.

I dislike using Immer's produce directly because the middleware's advantage is that it allows you to reap the benefits of Immer without having to explicitly manage it, similar to Redux Toolkit.

That's a fair point and I understand such an opinion.
I think we should export a new util createSliceWithImmer or something like that.
There are two design choices: a) use immer from zustand/middleware/immer or b) add immer as an optional peer dependency in this lib.
My personal preference is b), because it allows to mix a slice with immer and one without it. I'm not sure what people feel such an experience though. (In general, my personal preference is to avoid Immer, so my preference on this point can be ignored.)

That's not acceptable for some preferences (of mine 😄 ):

I figured that was the case, but sometimes like in chess you have to make a move even if it's not the best one to move forward
Just to clarify the immer error of both returning a new value and modifying its draft is not a TypeScript issue, it's a common error that exists in Redux Toolkit as well. For example: https://stackoverflow.com/questions/60806105/error-an-immer-producer-returned-a-new-value-and-modified-its-draft-either-r
this happens when you have something like this example that both returns a new draft and modifies it:
// bad
immerAction: (state) => { return draftState.count = 10 }
as opposed to only modifying the draft:
// good
immerAction: (draftState) => { draftState.count = 10 }
which is common in libraries using Immer under the hood as shown in the RTK thread
In with-slices we can see:
    for (const [actionName, actionsBySlice] of sliceMapsByAction) {
      state[actionName] = (...args: unknown[]) => {
        set(((prevState: Record<string, unknown>) => {
          const nextState: Record<string, unknown> = {};
          for (const [sliceName, actionFn] of actionsBySlice) {
            const prevSlice = prevState[sliceName];
            const nextSlice = actionFn(...args)(prevSlice);
            nextState[sliceName] = nextSlice;
          }
          return nextState;
        }) as never);
      };
    }

nextState is always returned, in my modification with a withImmer flag I thought to handle cases of modifying the draft state like so:
  for (const [actionName, actionsBySlice] of sliceMapsByAction) {
        state[actionName] = (...args: unknown[]) => {

          const useImmer = Array.from(actionsBySlice.keys()).some(sliceName =>
            configs.find(config => config.name === sliceName && config.withImmer === true)
          );

          // handle case of immer
          if (useImmer) {
            set((prevState) => {
              for (const [sliceName, actionFn] of actionsBySlice) {
                actionFn(...args)(prevState[sliceName]);
              }
            });
            // doesn't return draft state because it modifies it directly
            return;
          } else {
            // doesn't use immer
            set(((prevState: Record<string, unknown>) => {
              const nextState: Record<string, unknown> = {};
              for (const [sliceName, actionFn] of actionsBySlice) {
                const prevSlice = prevState[sliceName];
                const nextSlice = actionFn(...args)(prevSlice);
                nextState[sliceName] = nextSlice;
              }
              return nextState;
            }) as never);
        }

and in this case I was able to handle stores using either immer or without, although requiring a withImmer flag, as shown:
const countSlice = createSlice({
  name: 'count',
  value: 0,
  actions: {
    inc: () => (prev) => prev + 1,
    reset: () => () => 0,
  },
});

const textSlice = createSlice({
  name: 'text',
  value: 'Hello',
  actions: {
    updateText: (newText: string) => () => newText,
    reset: () => () => 'Hello',
  },
});

const userSlice = createSlice<'user', UserState, UserActions>({
  name: 'user',
  withImmer  : true,
  value: {
    name: 'John Doe',
  },
  actions: {
    setName: (newName) => (prev) => {
      prev.name = newName;
    }
  },
});

const useUserStore = create()(immer(withSlices(userSlice)))
const useCountStore = create(withSlices(countSlice, textSlice));

I understood this is almost certainly not the best approach but it is theoretical

I think we should export a new util createSliceWithImmer or something like that.
There are two design choices: a) use immer from zustand/middleware/immer or b) add immer as an optional peer dependency in this lib.

I think this is a good idea, I want to research this when I get home later.

b) add immer as an optional peer dependency in this lib.

In that case it's probably with a different entry point:
import { createSlice } from 'zustand-slices/immer';

I'm not 100% sure yet.


collection.onSnapshot not retrieving all documents [Firestore]

[REQUIRED] Step 2: Describe your environment

Operating System version: Container using node:11.15-alpine
Firebase SDK version: 8.2.0
Firebase Product: Firestore
Node.js version: 11.15
NPM version: Npm default version for image node:11.15-alpine

[REQUIRED] Step 3: Describe the problem
Steps to reproduce:
I have a search in my app that get all users and then apply a filter. As I need it to be in real time, I'm using a snapshot to retrieve the documents. My code was working fine for a few days, but in september 6th morning the query.size property was around 1.5% of the real size (I readed using a android SDK and confirmed using the console). As a workaround, I restarted the server and it worked again, but that is not a real solution, since I had to restart the production server.
Until now I could not reproduce it in my test environment
Relevant Code:
The query:
admin
    .firestore()
    .collection(collectionName)
    .onSnapshot((data) => {
        console.log(data.size); // Was logging 3, but should log ~2500
        // apply filter
    });

Dockerfile:
FROM node:11.15-alpine

# install deps
ADD package.json /tmp/package.json
RUN apk update && apk add yarn python g++ make && rm -rf /var/cache/apk/*
RUN cd /tmp && npm install

# Copy deps
RUN mkdir -p /opt/app && cp -a /tmp/node_modules /opt/app

# Setup workdir
WORKDIR /opt/app
COPY . /opt/app

# run
EXPOSE 3000
CMD ["npm", "start"]

EDIT 1
From my android app I received a "UNAVAILABLE: The service is temporarily unavailable. Please retry with exponential backoff." error around the same time, not sure if it is correlated

@lucasSzavara Thanks for filing this report. This does sound a little scary - the Server SDKs should never raise a snapshot before we have received all documents. While the backend sends us documents one at a time, we buffer these changes until the backend tells us that it has delivered a consistent snapshot.
Since you are not able to reproduce it, I would suggest that you file a support ticket with our backend support team. They will likely be able to look at your logs (once the ticket has been filed). Include a project ID and the time at which you saw the error: https://cloud.google.com/support-hub/
The query handling on Android is very different and we would need to know whether the SnapshotMetadata.isFromCache() flag was set or not to be able to say whether this might constitute expected behavior.

@schmidt-sebastian I just created a case at https://firebase.google.com/support/ > Database issues > Cloud Firestore > Queries > Queries help > Different problem and selected admin SDK. I'll keep this thread updated. Thanks for the help!

This is being tracked via our support system.


type 'int' is not a subtype of type 'double' of 'function result'

Steps to Reproduce

Run flutter create bug.
Update the files as follows: ...
...

Expected results:
Actual results:

Logs

Slider(
value: height.toDouble(),
min: 120.0,
max: 300.0,
label: '$height',
activeColor: Color(0xFFEB1555),
inactiveColor: Color(0xFF8D8E98),
onChanged: (double newValue) {
setState(() {
height = newValue;
});
},
),
Output :-
type 'int' is not a subtype of type 'double' of 'function result'

Hi @softjectico
opening an issue please try to provide
all or some of the information indicated in the template

Issue Template
If you are looking for support, please check out our documentation
or consider asking a question on Stack Overflow:
- https://flutter.dev/
- https://api.flutter.dev/
- https://stackoverflow.com/questions/tagged/flutter?sort=frequent
If you have found a bug or if our documentation doesn't have an answer
to what you're looking for, then fill our the template below. Please read
our guide to filing a bug first: https://flutter.dev/docs/resources/bug-reports
Steps to Reproduce
Please tell us exactly how to reproduce the problem you are running into.
Please attach a small application (ideally just one main.dart file) that
reproduces the problem. You could use https://gist.github.com/ for this.
If the problem is with your application's rendering, then please attach
a screenshot and explain what the problem is.

1. ...
2. ...
3. ...

Please tell us which target platform(s) the problem occurs (Android / iOS / Web / macOS / Linux / Windows) Which target OS version, for Web, browser, is the test system running? Does the problem occur on emulator/simulator as well as on physical devices?
Target Platform: Target OS version/browser: Devices:
Run your application with `flutter run --verbose` and attach all the
log output below between the lines with the backticks. If there is an
exception, please see if the error message includes enough information
to explain how to solve the issue.
Run `flutter analyze` and attach any output of that command below.
If there are any analysis errors, try resolving them before filing this issue.
Finally, paste the output of running `flutter doctor -v` here.

Closing this issue, since no information has been provided.
Thank you.


Use CMake for build system

The Makefile is simple and works for now, but maybe a more flexible build system (CMake) should be used?

9e9ce016e9957ac349eb5a0b5d74aa253a36cde2


New version: BlenderFoundation.Blender version 4.0.1

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@Trenly Seem to be having some issues with this one.

It seem the server did not return file size.  This link https://www.blender.org/download/release/Blender4.0/blender-4.0.1-windows-x64.msi is a webpage.  The Sigcheck did not detect as publisher.

It seem the server did not return file size. This link https://www.blender.org/download/release/Blender4.0/blender-4.0.1-windows-x64.msi is a webpage. The Sigcheck did not detect as publisher.

I agree, I did initially sumbit with different URLs but it was blocked. Just wondering what we need to do here.

Is there is more than one mirror?

@PckgrBot where is your InstallerUrl

Is there is more than one mirror?

Most companies use a content distribution network (CDN) which is location based. Most users will only ever be aware of one mirror because their location will only allow them to connect to one. I get a different mirror when I connect.
In general, the direct URL is better, but that only really applies if there is only one download location and no CDN is being used.

The InstallerURL was missing on the manifest

@PckgrBot


SolutionContainerComponent should function like ContainerManagerComponent

Except in an ECS way. i.e.

Support for multiple solutions on 1 entity
Possible for components to reference solutions by name
Possible to get all solutions on an entity.
Other code shouldn't have to worry about the manager and should just worry about the solution itself.

To document things from the Discord:
In addition of SolutionContainerManager
There was an agreement to completely gut Capabilities and turn them into Systems/Components on their own.
So each Capability like for example Injectable -> InjectableComponent/InjectableSystem.
Vera also mentioned that relation between SolutionContainer and SolutionContainerManager should be the same as between  SlotContainer and ContainerManagerComponent


Access to seed CSR from VS/VU modes

I'm trying to pin down exactly the proposed behaviour for accessing the seed CSR from VU/VS modes, as there seems to be a conflict between the text in the specification and what was said in issue #134. The specification says this:
The seed CSR is never directly available from virtual (VS or VU) modes. A read-write instruction causes a virtual instruction Exception (while a read-only instruction always causes an illegal instruction Exception.)
A comment in #134 said this:

When a trap occurs due to an attempt to access seed from VS or
VU mode, the specific exception should depend on the type of access
and the value of the sseed bit of mseccfg.  If the attempted access
is read-only (never allowed), an illegal instruction exception must be
raised.  Else, following the rules for virtual instruction exceptions
laid down by the hypervisor extension, if mseccfg.sseed = 1, a virtual
instruction exception should be raised, and if mseccfg.sseed = 0, an
illegal instruction exception should be raised.

These don't seem to be saying the same thing. Which is correct? And does the value of mseccfg.useed affect anything?
Thanks.

@mjosaarinen - I think this might be one for you?

The difference seems to be what kind of an exception occurs in the case of a read-write in VS/VU when mseccfg.sseed = 0. The spec says that it always causes a Virtual Instruction Exception and John said that it causes an illegal instruction exception. From the viewpoint of writing virtualization systems, it would feel awkward if it depended on a global flag like that; I'd think things that can possibly be legal in some virtual system can be handled via the virtual instruction exception. If seccfg.sseed is a Kernel configuration flag, as it may well be, then the hypervisor would need to implement two handlers to manage the gues depending on the particular configuration. I'm going to ask John about this.

FYI, I think the relevant part of the Hypervisor Extension that discusses this might be 5.6 traps, which says among other things:

When V=1, a virtual instruction exception (code 22) is normally raised instead of an illegal instruction exception if the attempted instruction is HS-qualified but is prevented from executing when V=1 due to insufficient privilege or because the instruction is expressly disabled by a supervisor or hypervisor CSR such as scounteren or hcounteren. An instruction is HS-qualified if it would be valid to execute in HS-mode (for some values of the instruction’s register operands), assuming fields TSR and TVM of CSR mstatus are both zero.

In this case, when mseccfg.sseed=1, the instruction could be considered HS-qualified because it would execute in HS mode, but it has insufficient privilege for VS mode. When mseccfg.sseed=0, it would not be HS-qualified. It isn't clear to me what the rules would be for VU accesses with respect to mseccfg.useed though...

yeah it certainly seems easier to just always handle "potentially allowed" virtual things via the virtual instruction exception, rather than make it dependant on a global variable like mseccfg. the handling of illegal instructions and virtual exceptions are notably different: "It is not unusual that hypervisors must emulate the instructions that raise virtual instruction exceptions, to support nested hypervisors or for other reasons. Machine level is expected ordinarily to delegate virtual instruction traps directly to HS-level, whereas illegal instruction traps are likely to be processed first in M-mode before being conditionally delegated (by software) to HS-level. Consequently, virtual instruction traps are expected typically to be handled faster than illegal instruction traps."
This makes the virtualization of something simple like an entropy source a nightmare if the global setting happens to be mseccfg.sseed=0.  (Sect 5.6.1)

Anyway, here's a brief rationale for the spec being the way it is wrt VS/VU and mseccfg.sseed/useed. https://groups.google.com/a/groups.riscv.org/g/isa-dev/c/wHsZ986slaU/m/h98Sb8J-AAAJ

@JamesKenneyImperas - just checking this latest change is enough for you to go on? If not, let us know and we'll discuss it some more. Otherwise, feel free to close this issue.

Hello Ben,
Yes, this is good, already implemented and tested in riscvOVPsim.
Thanks,
James.


[Spotify Player] Spotify extension not working

Extension
https://www.raycast.com/mattisssa/spotify-player
Description
Extending all operation commands did not work.
View gif image:

Steps To Reproduce

There are no special steps, you can view the gif above

Current Behaviour
No response
Expected Behaviour
No response

Oh wow that is weird... some thoughts

are you logged in to the same account in the app and raycast?
can you try quitting the app and running again?
can you try restarting raycast?

need to iron out the above first :D

Yes, I've re-downloaded the extension several times, the spotify authorization account is correct, raycast and the computer have been restarted, yet he doesn't take effect!

Oh wow that is weird... some thoughts

are you logged in to the same account in the app and raycast?
can you try quitting the app and running again?
can you try restarting raycast?

need to iron out the above first :D

Yes, I've re-downloaded the extension several times, the spotify authorization account is correct, raycast and the computer have been restarted, yet he doesn't take effect!

same here


Betterment/update job

This PR solves #112.

Remove update_job_status, update_metadata, update_job_optimistically, post_job_update functions from Database implementation.
Reimplemented update_job to use JobItemUpdates.
Provide JobItemUpdates object with all the needed changes only, which get's reflected on the database.

WHY this approach?

Code had three functions update_job, update_job_status & update_metadata to update job values in database.
update_job was being passed with the whole jobItem to update, in it's usage it was only changing 2-3 values in the job and not the whole job object, resulting in other values unnecessarily being rewritten. (for eg, same version value is being sent within jobItem, resulting in a pointless rewrite 1 -> 1).
The above is also why the code had to update version and updated_at fields post these updates, since first the version get's rewritten (1->1) and then it's overwritten to an increased value in post_job_update (1->2).
update_job_status and update_metadata followed the same trend but only updated individual values.

WHAT now ?

Trimming the above problem, reinventing update_job seemed the best approach to move forward with.
Provide a JobItemUpdates object that consist of all the updates that needs to be done to the job in a single query.
Code doesn't allow updating id and created_at.
Code implemented only_status and only_metadata to allow for cleaner code for these updates.

Done with Self Review,
I think we are missing a test case for the new update_job.
WDYT ? @apoorvsadana

coverage: 65.404% (+0.6%) from 64.811%
when pulling b78dd9d9612873be106894323dc7cd95f448680b on betterment/update_job
into ab66b83a49f62f774b9de48d012feab3f4f124d3 on main.

The version shouldn't come from the user. It's a invisible implementation that you don't need to worry about when writing the code. All you should know is that if your job is stale the update won't work.

Updated the logic to don't let parent fn to decide whether to update version or not, it will now always be updated.
Additionally, the same logic is now followed for updated_at.


slowness when nmap is executed from pod

Expected Behavior
when we run nmap to find open ports from a node or within a pod it should  complete within same amount of time
current Behavior
nmap commands take  more time when we execute it within a pod.
no of packets sent within a pod is less when  compared with node
pods use calico interface.
nodes use eth0 interface.

I think this issue needs a bit more information in order to properly triage.
A few bits of information that might help:

Could you share a sample of what the nmap commands you're running look like (both from within the pod and from the host)?
Are you running nmap against DNS names or known IP addresses? Are they cluster locations or outside the cluster?
How are you measuring the time it takes to execute the command?
Is it only nmap that is affected, or are other connections from pods slow?
Do you have any network policies defined in the cluster?

FWIW, I would expect that to be slower; Calico is policing the traffic that your pod can send, that requires work and nmap generates lots of unusual packets that will all look like new flows.


sysid-test-state-Drive file appears in AdvantageScope but not in SysID

This could very well be an issue with the SysID program. I used URCL to log my four drive motors during the four sysid tests, and at the end I was able to download the log file and open it in AdvantageScope. At this point, the sysid-test-state-Drive file appeared with all of the other URCL stuff I needed, and the graphs looked pretty good. The next step in the docs was to open it in the sysid program, but when I opened the log file, everything BUT the UCRL and sysid-test-state-drive files were there for me to load into sysID.

Note how there is no UCRL file under NT, and no sysid-test-state-Drive file.

https://github.com/Mechanical-Advantage/AdvantageScope/blob/main/docs/REV-LOGGING.md
Here are the instructions I followed. I didn't do anything that said "advantagekit" users because I didn't know what that was. Do I need to do that?

Did you export the log file from AdvantageScope and open the exported (not the original) log in SysId? You'll also need to make sure to check "Include Generated" in the AdvantageScope export options.
If it still doesn't work after following those steps, it would be helpful if you could upload both the original log file and the exported version. You may need to put them in a zip archive to upload them to GitHub.

I reran the tests and exported them again and I was able to get the file open in SysID with all of the UCRL and sys-id-state stuff. However, when I hit load, SysID crashed. I probably just need to redo the test. Thanks for the help, Mr. jwbonner.

Glad you got it to open correctly. In regard to the crashing, make sure you're running the latest version of WPILib (2024.2.1) and AdvantageScope (v3.0.2). There was a bug in the kickoff versions that caused SysId to crash while opening logs from AdvantageScope.

Alrighty! Thanks again Mr. Bonner.


Accuracy in Degree to Radian conversion

Hello visose,
I was facing somme small (but not negligible) accuracy problems today. It seems to me that the Degree to Radian component does its conversion with not enough accuracy has it rounds the results to only 2 digits.
I would suggest to increase the rounding to 6/8 digits x => $"{x:0.00000000}") :
protected override void SolveInstance(IGH_DataAccess DA)
        {
            var degrees = new List<double>();
            GH_RobotSystem robotSystem = null;
            int group = 0;

            if (!DA.GetDataList(0, degrees)) { return; }
            if (!DA.GetData(1, ref robotSystem)) { return; }
            if (!DA.GetData(2, ref group)) { return; }

            var radians = degrees.Select((x, i) => (robotSystem.Value).DegreeToRadian(x, i, group));
            string radiansText = string.Join(",", radians.Select(x => $"{x:0.00000000}"));

            DA.SetData(0, radiansText);
        }

In my case, it solves the problem.
Lionel

Thanks, I merged your pull request.


Streamlit app is down.

Not entirely sure what happened, but it crashed in the last hour or so. Trying to use the gif creator in a sedimentology class for lab.
Thanks for building this!

It should be working now. The app keeps crashing when too many people visit the site. GEE doesn't allow too many concurrent sessions.


State management

(Note: haven't used the context yet but it currently works)

The initialization code I have with some dummy claims is giving me a lot of problems. When I had it in page.tsx I would get a bizarre runtime TypeError that referenced the compiled javascript code which I was briefly able to access. That might have been a problem with Home() being async which is needed for the API Call. The program worked when I moved the dummy claims to ClaimList except that the website would crash when I tried to drop one of the claims into its new position.
For now I'm going to get rid of the dummy initializations and work towards implementing the create claim buttons properly. Hopefully the issue doesn't come up again there.

Closing this pull request since I'm going to merge straight from state-management's successor to its predecessor.


Install on MacOS Ventura with Apple Silicon

Has anyone run the demo code on a Mac with Apple silicon?
Trying conda env create -f environment.yml on a MacBook Pro, MacOS 13.1, Apple M1 Max,
I see the error shown in the attached trace.txt, which some Googling suggests may be an Apple silicon problem.
error: legacy-install-failure
× Encountered error while trying to install package.
╰─> RPi.GPIO

Hi @ianfoster, really glad to see that you got the demo up and running! The problem is likely with one of the following two:
https://github.com/sparks-baird/self-driving-lab-demo/blob/50160d67742e13c052982e743a3a2bbeca011af3/environment.yml#L43-L44
Incidentally, both are no longer needed after the switch from the Raspberry Pi 4B single board computer to the Pico W microcontroller, so I'm removing them in https://github.com/sparks-baird/self-driving-lab-demo/pull/165. If there are any other dependency conflicts you run into, it might be worth also trying to run pip install -e . inside of a fresh conda environment and while inside the self-driving-lab-demo main directory in case one of the Anaconda packages is causing an issue. The conda environment file contains

That works, thanks! (And I have the Google Colab running now also, via the other issue.)

For sure! Glad it's working, and thanks for your patience. I appreciate the detailed issues.


Lock up Loop

When email fails (as it is tonight), the emails go in endless loops trying to send, save, send, save fail.

Fixed.


[FLINK-15648][k8s] Introduce CPU limit support for JM/TM Container

What is the purpose of the change
Currently, we use kubernetes.jobmanager(taskmanager).cpu to configure CPU request and CPU limit at the same time for the JM/TM Container; given that people are asking for different request and limit values for the sake of elasticity, this PR proposes to separate these two configurations, people can set the CPU limit separately via new config options kubernetes.jobmanager(taskmanager).cpu.limit.
Brief change log

Introduce new config option kubernetes.jobmanager.cpu.limit
Introduce new config option kubernetes.taskmanager.cpu.limit

Verifying this change
This change added tests and could be covered by existing tests.
Open questions

K8s allows us to set CPU request or limit to zero for a Container. After this PR,

JobManager CPU limit allows zero value
TaskManager CPU limit allows zero value
JobManager CPU request allows zero value
But TaskManager CPU request does not allow zero value, this is somewhat inconsistent with K8s or the JobManager side, at the moment I think it is a trivial problem, but we can support it if necessary; welcome any feedback on this issue.

For the existing config option of kubernetes.jobmanager.cpu and kubernetes.taskmanager.cpu, actually they are used for specifying hard CPU request, therefore, I tend to append a postfix .request to them. Looking forward to your thoughts.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (docs)

@TisonKun @tillrohrmann @wangyang0918 Could you help review this PR? I have left two open questions there and looking forward to your thoughts.

Hi @xintongsong. Could you help review this PR?

Thanks for inviting me, @zhengcanbin.
I'm not entirely sure about the proposed approach. Sorry for haven't bringing this up earlier. I'll comment my concerns on the jira ticket.

Shall we close this PR until we have reached agreement on the JIRA ticket @zhengcanbin?

Shall we close this PR until we have reached agreement on the JIRA ticket @zhengcanbin?

Sure!


result.

Hello, your job is very good. But when I re-implement the experimental results, I executed the command line as required, and only got the value of 73.2 f1 score. It does not match the paper, is there a problem?

Great question - the results in this repository are slightly higher than the results reported in the paper. This is because my dataset splits were not done with a random seed (due to my inexperience at the time), and thus when I went to make this reproducible (and gave it a random seed), the splits are slightly different than when I evaluated in on the paper.  Since the results are roughly the same, and even better on the new split, I have not updated the paper (the difference between reported and reproducible results is ~0.7%).  I'm sorry for any trouble that may cause!
If you're comparing against our baseline in an upcoming research paper, I think reporting our results as either of those values would probably be acceptable (though since the data split is easier, I would imagine that a good new model would improve upon the ~73.2 F1 score).
See the notes in https://github.com/orionw/RedditHumorDetection/blob/master/create_data.sh for more details and please let me know if you have any more questions.


awsbatch not perfoming the multinode job on the spot compute instances

Required Info:

AWS ParallelCluster version -2.9.1

Problem description and how to reproduce:
I have created the AWS Parallel cluster with AWS batch scheduler and completed the tutorial multi-node job tutorial successfully as given the user guide. But when I tried to run the same tutorial with cluster_type = spot and spot_bid_percentage = 85, the job stays in the RUNNABLE state for the last four hours.

I have tried to figure out the reason why it is not running and found out that this statement from aws batch user guide.

So, I just want to know is this the reason because of which my job is not running?
Please let me know how to form the spot cluster and perform successful jobs with the aws-batch scheduler.
Thanks & Regards,
Shadab

Hi @Shadabdyn
Yes that's the reason why your job remains in a RUNNABLE state and is never executed. AWS Batch Multinode jobs cannot be executed in a Spot cluster which means you cannot specify a number of nodes greater than 1 (with the -n option) when cluster_type = spot.
If you need to run multinode jobs please use cluster_type = ondemand. While if you don't need to run MPI jobs spanning across multiple jobs than you can use single node or array jobs in a spot cluster.

We also updated the docs to call this out explicitly: https://docs.aws.amazon.com/parallelcluster/latest/ug/awsbatchcli.awsbsub.html
I'm going to resolve this one since the limitation you are facing is expected and is not something controlled by ParallelCluster. Feel free to reopen this one or a new one if you need further support.


Rename or rephrase "CreateShortcut" while checking shortcut/weblink properties.

Summary of the new feature / enhancement
I'm writing a script for the students who leave the school I work for during the schoolyear.
One of these things in the script is placing a shortcut to the website of our supplier.
Now, I wrote in that script a check if nothing changed the URL and the icon. Yet, when you want to check the location of a weblink shortcut... You get this:
The variables of shortcutlocation and shortcuticon are filled earlier in the script.
      $ShellShortcut = New-Object -ComObject WScript.Shell
      $ShortcutTest = $ShellShortcut.CreateShortcut($ShortcutLocation)

      if (($ShortcutTest.IconLocation -eq "") -OR ($ShortcutTest.IconLocation -ne $ShortcutIcon)) {
          $ShortcutTest.IconLocation = $ShortcutIcon
          $ShortcutTest.Save()
      }

What is tripping me up every time is "CreateShortcut". I'm not a native English speaker (yet I speak it extremely fluently) ... and I always read it as it creates a new shortcut and skips the checking part.
What's even stranger is that if you do a "Get-Content $ShortcutLocation", you can see the URL in there just fine.
I honestly think that this function should get another name OR more information in "Get-Content". Since, it might help with confusion.
Proposed technical implementation details (optional)
No response

The COM object isn't something we can change, it's built into Windows. You'd need to use the Feedback Hub to request that (but changing the name would be a huge breaking change so it is not likely to be accepted)
Get-Content just reads the file as is, it doesn't do any processing of file formats. You would need a dedicated command to managed shortcuts. There might be one on the PowerShell gallery you could install


Narrator's focus gets stuck on activating Adding new tab/Close tab button in TabView.

Describe the bug

Steps to reproduce the bug

Prerequisite:

Turn on Narrator.
Keep narrator in item mode.

Repro steps :

Launch XAML Controls Gallery.
Activate TabView from home page.
Navigate to "Add new Tab button", activate it and observe the Narrator's behavior.
Similarly, activate "close" button and observe.

Expected behavior

Narrator's focus should move to the next UI element after activating Adding new tab/Close tab button in TabView.
Screenshots

Version Info

Xaml Controls Gallery version <IP_ADDRESS>
NuGet package version:

Additional context

Issue is with either tabview or list view.
Copied from internal bug

@YuliKl Just to clarify, when the close button or the add tab button get's invoked, we should move the focus next focusable UI element? So in the case of the add button, it most likely would move the focus inside the content of the currently selected TabViewItem.
@StephenLPeters I would like to look into this.


Add Bears v.s Babies

Add Bears v.s Babies to the list

Thanks. I'll review the PR shortly, but as this is an open contribution repo, no issue is needed.
Further discussion will be handled directly in the PR #57.


How to run prosit locally in high performance computing environment?

To whom it may concern,
Thank you all for writing this amazing software. And I am trying to run this locally.
I don't have a GPU locally, but our HPC has. Is there a way to dockerize everything into a single container?
Thanks.
Best,
Wenjin

Have a look if this helps: https://github.com/kusterlab/prosit/blob/master/Dockerfile


Trading cards for packs (in shops)

Hi, this is more of a question than a feature request, sorry! Great plugin by the way!
I see that rewards for collecting full sets of rarities aren't in yet, but that's not quite what I want anyway! Is there a way of doing this:
-Players can only get cards by buying booster packs which contain (for example) 3 common cards.
-Player can trade in any 3 different cards for 1 card of the next rarity up; e.g. 3 common for 1 uncommon.
-Ideally, being able to trade a number of cards for a booster pack; maybe 9 common cards for a pack of 3 uncommons, or a number of common cards to get a pack of random other cards of the same rarity (if they don't like the ones they have?)
Perhaps I could use a different simple shop plugin that can take in and give out items and take 3 common_card (is there an id for shops like that?) for 1 uncommon_card and make a shop for each rarity? (This would require players to go to a specific place on the server to trade them in, rather than type /card upgrade or whatever the command would be)
Seems to me that asking players to complete a full set of rarities to get a reward might be kind of difficult if I've added a few dozen more cards to the server, haha! A way of trading in just a few of them would be much more helpful, if there's some way I could add that to my server :)

For disabling drops, see https://github.com/sarhatabaot/TradingCards/blob/master/tradingcards-plugin/src/main/resources/settings/chances.yml
enter the chances for various rarities below, out of 100,000. 1 = 1 in 100,000 or 0.001%
10,000 = 10% chance, 1000 = 1% chance, 100 = 0.1% chance, 10 = 0.01% chance, 1 = 0.001% chance
chances are in descending order - make sure the chances for common are higher than uncommon etc.
if a type is omitted from this section, the rarity won't drop at all from that type.
and for the rewards, see https://github.com/sarhatabaot/TradingCards/blob/master/tradingcards-plugin/src/main/resources/settings/rarities.yml
replace "eco give %player% 100" with "cards give pack %player% name-of-pack"

Hi, thanks very much! This would be really helpful and exactly what I was searching for, but what threw me off when I searched previously was at the top of your second link: "rewards: [...] not implemented yet"?
Is this comment just a left-over, and rewards are implemented? :)

To be honest, I hadn't noticed that. I would have to defer to @sarhatabaot for that one.

Thanks very much! TCRecipes would be perfectly and exactly what I want, if only the parameters could be "general" (as you just said!):
From:
result: "uncommon,blaze,1,false"
recipe:
- "common,zombie,4,false"
To:
result: "uncommon,,1,false"
recipe:
- "common,,4,false"
Would be great 😄

Hi, bit of an update for this. I'm splitting this feature request into 2, I will reference them using other issues below.

The cards upgrade feature.
Trade cards for booster packs.

Implemented in #186 & #184 !


411 implementidentify solution for correctly mapping categories to multiple objects in the same array

Opened this by mistake. @branchedelac  should I leave it of close it?

Ah, this got weird @branchedelac. I commited to what I thought was a branch for #416.
I am closing this and reopening #411 once this is merged.


Double in-disk checkpoint/restart creating zero-byte size files

Original author: Esteban Meneses
Original issue: https://charm.cs.illinois.edu/redmine/issues/257

The double in-disk checkpoint restart is creating empty checkpoint files.
After the CMK_CHKP_ALL flag was introduced (now set by default), the in-disk version is no longer storing the checkpoint in the files.
In function:
void CkMemCheckPT::recvArrayCheckpoint(CkArrayCheckPTMessage *msg)
the checkpoint message msg is simply stored in a table, which is sufficient for double in-memory C/R. For the in-disk case, that checkpoint message should be stored in a file. The line:
chkpTable[idx] = msg;
has to be changed to store the message in-memory or in-disk accordingly.

Original date: 2013-08-04 05:28:34

Marking for 6.6. If the relevant change was made before 6.5, then this should be retargeted to 6.5.2.

Original author: Xiang Ni
Original date: 2013-08-28 22:17:33

Fixed the bug. Now when CMK_CHKP_ALL is turned on, checkpoints will be written to files for double in-disk checkpointing.


IndexError: too many indices for tensor of dimension 3

Firstly, thanks for your code sharing. I use your partly code to convert image from RGB to Lab colorspace. I didn't change your code, but I encountered an error.
File "/content/drive/My Drive/BICYCLEGAN/dataset.py", line 36, in __getitem__
    img_lab = rgb2lab(img)
  File "/content/drive/My Drive/BICYCLEGAN/colorspace.py", line 126, in rgb2lab
    lab = xyz2lab(rgb2xyz(rgb))
  File "/content/drive/My Drive/BICYCLEGAN/colorspace.py", line 45, in rgb2xyz
    x = .412453 * rgb[:, 0, :, :] + .357580 * rgb[:, 1, :, :] + .180423 * rgb[:, 2, :, :]
**IndexError: too many indices for tensor of dimension 3**

Could you give me some advice to solve this problem? Thanks.

I have solved this problem, because I did not keep the same dimensions with the functions input.


ImportError when using tensorflow 2.8.0 and python 3.9.16 undefined symbol: PyCMethod_New

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
2.8.0
Custom code
Yes
OS platform and distribution
Linux/amd64
Mobile device
EMR cluster
Python version
3.9.16
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
I am running a python script(udf from pyspark) from emr node that is launched using a custom docker image. I have python 3.9 installed and tensorflow 2.8.0 installed. When running this particular python script as a spark-submit job, I have an error with tenosrflow PyCMethod_New.

Standalone code to reproduce the issue
Below is the simple python code that I am testing as a spark-submit job that uses docker as runtime environment and error is thrown while importing this package itself.

from pyspark.sql import SparkSession
from pyspark.sql.functions import udf
from pyspark.sql.types import StringType
import sys
sys.path.append("/home/ec2-user/venv/lib/python3.9/site-packages")
import zooperetl
import tensorflow as tf

# Initialize Spark session
spark = SparkSession.builder.appName("MySparkJob").getOrCreate()

# Create a sample DataFrame
data = [("zooperetl",), ("numpy",), ("tensorflow",)]
columns = ["Package"]
df = spark.createDataFrame(data, columns)

# Define a custom UDF to import a library inside udf.
def get_tensorflow_version():
    return tf.__version__

# Register the UDF
get_tf_version_udf = udf(get_tensorflow_version, StringType())

# Apply the UDF to create a new column with TensorFlow version
df_with_tf_version = df.withColumn("tensorflow_version", get_tf_version_udf())

# Show the DataFrame with the added column
df_with_tf_version.show()

# Submit the Spark job to the EMR cluster
# Replace 's3://your-bucket/your-script.py' with your actual script location
# spark-submit --master yarn --deploy-mode client s3://your-bucket/your-script.py

Relevant log output
No response

@sunitha8687 You are using an older TF version 2.8 which is not actively supported so could you please try to upgrade to the latest and follow the tested build configurations here. Thank you!

Thanks for your response. I checked the tested build configurations for Linux from your link, looks like 2.8 and 2.10 supports python 3.7to 3.10.
Just to give you some background context here: I am building a docker image in local and then pushing it to the artifactory. This also uses amazoncoretto:17 as base image. So my local system is Mac os, but anyway i specify the platform to be linux/amd64 when i build docker image. The EMR cluster also uses amazon linux os.
In this case i look for linux right ?

@sunitha8687 Since your tested build configurations support Python 3.7 to 3.10, please update the base image in your Dockerfile . Kindly rebuild your Docker image locally with the updated base image and let us know?
Thank you!


<EMAIL_ADDRESS>hash check failed

Output:
Installing 'color-cop' (5.4.5) [64bit]
colorcop.zip (5.1 KB) [=======================================================================================] 100%
Checking hash of colorcop.zip ... ERROR Hash check failed!
App:         extras/color-cop
URL:         http://colorcop.net/tools/colorcop/colorcop.zip
First bytes: 3C 21 64 6F 63 74 79 70
Expected:    d8575e286565460c0fc312d5653c41f72cfe3068a7428b0c0ae5d99ed6bcd90d
Actual:      bb3b44653f7388c9b2282b5ecd7145204efec59e6ff12e59e4c2da6600a2c4ed

You're right. Malwarebytes was blocking it. Thanks!


[AMBARI-23056] [branch-2.6] Fix AMS dependencies that have security issues.

What changes were proposed in this pull request?
Fix AMS dependencies that have security issues.
How was this patch tested?
mvn clean test
Manually tested the newer versions of artifacts introduced.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/766/
Test PASSed.


Update qhimm.xml

We keep talking about getting rid of Remako, let's just finally do it.  Also remove Borderless since FFNx now handles that natively.

Also remove DrLilo FMV to avoid confusing users. With all due respect, now we have Cosmos FMV.

I didn't see DrLilo in there, unless it's being called something else?

From line 934.

I see it now...

Closed for deprecated solution


permission

Can you add permission? I dont want members to use commands.


Create a CKAN instance

[] Platform(AWS, GCP, Azure)
[] Install SW
[] Add campaign finance data
[] Add election results
[] Add campaign docs
[] Add voter demo and history

Duplicate


Animate Family Parameter

When in a family (e.g. conceptual mass) it would be nice to be able to animate the family parameters. Have to check if there's a node for manipulating family parameters, though.

hey @andydandy74. What exactly do you mean with this issue. I'm currently using Dynanimator only to animate family parameters (like in the screenshot).

Fabian

Dynanimate Numeric Parameter is for animating parameters of placed family instances (or family types) in a project or family file. I'd like to be able to animate the global parameters of a family file (e.g. a conceptual mass family) from within the family file itself. The Revit API uses a different method to get/set those kinds of parameters. That's what this is for. Once I have that I can use Dynanimator to rapidly generate different versions of a mass family.

ok, but I don't see the usecase here. If I want to generate different versions of my mass family, I place it in my project and run Dynanimator. That is what I'm doing right now. What is in your opinion the difference of modifying the familiy parameters while in family editing mode? Do you want to create different family instances that are then fixed (with the parameters remoced)?

Hi, I am having troubles creating an animation with family parameter change and I dont know where else to ask for advice. I have tried both tutorial and sample files but I always end up with no animation at all or error: Cant make type ...
Here are some screens.

There seems to be too many "nulls" to me but I am new to both dynamo and dynanimation.
I have tried revit 2015 and 2016 and dynamo 0.7 and 0.8 but nothing changed.
System is windows 10 64bit

Hi, @andydandy74,
hello to all that can help.
I have troubles to get this to run so that multiple images are saved.
If you compare the screenshot with the older version from May 2015, the node "Dynanimate Numeric Parameter" has changed: it has now an additional entry ("Iteration") and the entry "Iterations" changed to "Total Iterations".
With the current set up only 1 image is saved and then the process stops.
Any help would be greatly appreciated.
Many thanks in advance.

This package was never updated for any post-0.7 versions of Dynamo, so it's hard to give you any feedback. I'd like to update this once Dynamo 1.0 has been released but I'm not sure when I'll find the time. One thing that I can tell you from looking at your screenshot is that you should definitely get rid of the 0.7 version of Clockwork. Having two Clockwork versions installed side-by-side is a recipe for disaster...

@andydandy74 Thank you for your reply.
I have deinstalled the old version now.
However what bugs me is that the "DynanimatorFunction Numeric Parameter" node is different from the version you posted. So at least some changes have been made.
Do you get it to run like it used to?
And: to your knowledge, is there any other means available to change variables automatically and save an image with each iteration?


build: Error (Xcode): ../../.pub-cache/hosted/pub.dev/pdfx-2.5.0/lib/src/viewer/pdf_page_image_provider.dart:24:55: Error: Type 'DecoderCallback' not found.

Description
package giving error on just installing and running!
Describe what changes need to be done to the build system and why.
Requirements

[ ] The build system is passing

Hi @RoBiN-HoOd-1, thank you for bringing to my attention.
This package completely depending on pdfx to extract pages, and seems like pdfx has issues with the latest versions of Flutter at the moment.

We can follow up on here, I'll update when they update:
https://github.com/ScerIO/packages.flutter/issues/465

Same issue here

@BrianMwas what's your flutter and pdf_thumbnail version?


[FSDE-1085] Add E2E tests to chi-vue drawer component

[x] Added E2E tests to chi-vue drawer component

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-101/1/. :white_check_mark:

You can check this PRs instance in https://2e6d05d8299f3838acf9040e1a1ab6761f837aac-chi-pr.enterprise.ctl.io

The CI pipeline did not run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-101/2/. :x:

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-101/3/. :white_check_mark:

You can check this PRs instance in https://e93444541d56721bc939d1dad56a0e6566d3733d-chi-pr.enterprise.ctl.io

The CI pipeline did not run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-101/4/. :x:

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-101/5/. :white_check_mark:

You can check this PRs instance in https://7aeb337cdfd3a14d424079049a9e464fa03a1519-chi-pr.enterprise.ctl.io


Minor formatting

Add space between "against" and "target".

already fixed in dev branch, don't want to create a merge conflict.


Update DTO.md

Hi Vinnie, how's it going? Thinking about merging with you. LMK ASAP.
First Pull Request.

Wrong type of Pull Request .. in the future make sure you start with a draft request rather than a full PR unless you are already ready to go with your merge.

Converted to draft for now. Comment here when you need me to check it out.

Duplicate of #15


[Typechecker] Perform capture analysis for enums as well

We weren't calling checkPatternBindingCaptures() for enums, only for structs and classes. If you look at the crasher code, the PBD for the property foo is inside the enum and was never checked.
Resolves SR-11509.

@swift-ci please smoke test


chore(vers): upgrade prop-types

Do you want to request a feature or report a bug?
A bug
What's the current behavior?
The package.json currently specifies version 15.6.2 of prop-types.
The file propTypes.js refers to the function PropTypes.elementType, which was added in version 15.7.0 of prop-types. (See here.) This causes some type-checking to wrongly throw an NPE in development.
What's the expected behavior?
No NPE.
package.json should be updated to refer to version 15.7.2 of prop-types

PR welcome!

Ball is in your court, Bogdon!


exporting in publish mode fails if the existing image has bad metadata on the label "sh.packs.build"

The problem is due to
https://github.com/buildpack/lifecycle/blob/1fe614565217b026a0fb5158241bb299b0da1a5e/exporter.go#L143-L146
if the label is an empty string then it fails to parse as JSON
There are really to scenarios and two options.
Scenarios:

Label doesn't exist
Label exists but is not json

Options:

Return error (with helpful text)
Ignore the error and simply don't reuse any layers.

My personal opinion is option 2 for both scenarios.

pretty sure this has been fixed for awhile


Insiders API Change: no NotebookData class in NotebookSerializer.serializeNotebook handed over (only same Interface)

Does this issue occur when all extensions are disabled?: Yes

Version: 1.95.0-insider (user setup)
Commit: 38dc6ac5a771cc94bde1344722bb2d02c80096ea
Date: 2024-10-25T22:08:52.005Z
Electron: 32.2.1
ElectronBuildId: 10427718
Chromium: 128.0.6613.186
Node.js: 20.18.0
V8: 12.8.374.38-electron.0
OS: Windows_NT x64 10.0.22631
Steps to Reproduce:

implement NotebookSerializer interface in an extension
check first args of serializeNotebook with notebookData instanceOf vscode.NotebookData
with newest VSCode Insiders this checks returns false. with latestet VSCode this checks returns true

There were changes in NoteBookSerializer, maybe these produce this changed behaviour: https://github.com/microsoft/vscode/commit/c4645ea0ec0093893e7c16b1f628128d439da671 (worker uses structuredClone)
Code which produces this error: https://github.com/AnWeber/httpbook/blob/ef2aef382f6e906ca09bafede1bd7591f15125a0/src/extension/notebook/httpNotebookSerializer.ts#L190
Bug of User with Insiders: https://github.com/AnWeber/httpbook/issues/134
Workaround: I willl implement a typeguard to check if object matches API of NotebookData Class (https://github.com/AnWeber/httpbook/blob/526a9f4d15e99d239cd73c03e124b0bce94644bc/src/extension/notebook/httpNotebookSerializer.ts#L227)

@rebornix Would this have something to do with Remote Save.
@AnWeber Thank you for filing this issue, however the changes you highlighted are not related, as those changes are in a built in extension and not the API for other extensions (i.e. ipynb is an extension that is shipped with VS Code for serializing of Jupyter *.ipynb notebook files)
I'll have a look and see whats going on here.

. with latestet VSCode this checks returns true

@AnWeber Here I'm assuming you are referring to 1.94 of VS Code

@rebornix Confirmed, this is due to the change related to remote save.

@DonJayamanne yes, I meant 1.94. I think the ticket can also be closed. The code is already productive. In other words, the relevant areas should already be fixed in the extensions. I just wanted to point out the changed behavior in the Insiders


Navigation arrows not showing - what is disabling the arrows?

I have added in my style.css
.slick-prev:before, .slick-next:before {
color: #09529b !important;
}
also I have arrows:true in my js file where I call .slick.
When I Inspect the html in the browser console I have the following:
<button class="slick-prev slick-arrow slick-disabled" aria-label="Previous" type="button" aria-disabled="true" style="display: block;">Previous</button>

Does that mean the arrows are disabled? What setting is disabling the arrows?

the arrow were not disabled, it seems they had a minus padding which displayed them outside of the screen.


Split onadmin_proposal

Split onadmin_proposal into several standalone functions to allow deploying a single proposal by calling e.g. deploy_single_proposal keystone after sourcing the qa_crowbarsetup.
I extracted the code from onadmin_proposal that was not specific to a proposal into prepare_proposals that is about to be called only once before messing up with the barclamps' installations. Also added set_dashboard_alias and do_ipmi functions that were extracted from the onadmin_proposal and onadmin_allocate function.
Tested  successfully.
Note: these changes are refactoring of the existing code, no new functionality has been added.

The indention fix should be squashed into the main commit via git rebase --interactive master

Method renamed, commits squashed.

:+1: besides the wrong call of the function

Use correct function name eboot_nodes_via_ipmi

merged

Belated :+1: - great refactoring thanks!


Delete volumes entirely with Kubernetes

Hi,
I have launched Longhorn with my Kubernetes Setup, attaching volumes and detaching volumes works fine, but how can I enforce that a volume gets really deleted and not only detached? Adding persistentVolumeReclaimPolicy: Delete to the PersistentVolume seems to be the wrong option...

We haven't added the support for persistentVolumeReclaimPolicy: Delete, and currently the volume can only be deleted from Longhorn API/UI, not from K8S. It's in the roadmap.

Sad that it's not possible in an automated, integrated way. Is the roadmap somewhere public visible?

Regarding this feature, it should be implemented with our Dynamic Provisioner support, which should be there in about a month.

Hi @tboerger
We just completed the support for persistentVolumeReclaimPolicy, since we've added support Kubernetes Dynamic Provisioner now. You can give it a try now.
Notice the API changes, so you would need to clean up and redeploy.


Make current machine's arch as defaulted GOARCH

When building on arm64 machine, don't build amd64 as default GOARCH.
Signed-off-by: Changwei Ge<EMAIL_ADDRESS>

Codecov Report

Merging #141 (c1a8520) into main (56f943f) will increase coverage by 0.18%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #141      +/-   ##
==========================================
+ Coverage   29.26%   29.44%   +0.18%
==========================================
  Files          20       20
  Lines        1647     1647
==========================================
+ Hits          482      485       +3
+ Misses       1100     1097       -3
  Partials       65       65

Impacted Files
Coverage Δ

pkg/filesystem/fs/blob_manager.go
55.55% <0.00%> (+11.11%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Looks OK.


Allow latest version of fractal 0.13

Hi,
I am not able to use the latest version of fractal as this package is requiring 0.12.

Thanks for reporting. Fixed on v5.11.13.

Cheers!


SPI available blocked

Hi i am just trying to use your examples with nucleo-f767 and an ili9488 v1.0.
I tried to make first the version without the touchgfx.
i have set the pins but it seems that i get in infinite loop inside the first function
Displ_CLS(BLACK);
where it seems that inside it in the
void Displ_Transmit(GPIO_PinState DC_Status, uint8_t* data, uint16_t dataSize, uint8_t isTouchGFXBuffer ){

	while (!Displ_SpiAvailable) {};  // i am getting stack inside here

}

i will cross check again the setup of cs pins but i think there is no issue there. What  do you think could be the cause of not updating the variable Displ_SpiAvailable to Zero?
It is strange that previously on init there is no problem though probably you are not checking there the Displ_SpiAvailable.

Then checking one by one my spi setup in cube mx is the next.If you see something please tell me cause i am watching it 4 hours and i do not see the problem:
I am using the D0-D7 (not all ) that can be accessed by the pin headers. And the wiring is what you have in the png on your guide. But before discussing wiring issue the first is the setup in program.

I have changed the DMA to polling and then it was unlocked but the screen is still blank. The blacklight i not set on dimming just to on/off. The rest of my setup is:
 * which display are you using?
 *************************************************/
//#define ILI9341
#define ILI9488_V1
//#define ILI9488_V2

/******************    STEP 2    ******************
 **************** PORT PARAMETERS *****************
 ** properly set the below th 2 defines to address
 ********  the SPI port defined on CubeMX *********
 **************************************************/
#define DISPL_SPI_PORT 	hspi1
#define DISPL_SPI 		SPI1

/******************    STEP 3     ******************
 ***************** SPI PORT SPEED  *****************
 * define HERE the prescaler value to assign SPI port
 * when transferring data to/from DISPLAY or TOUCH
 * Keep in mind that Touch SPI Baudrate should be no more than 1 Mbps
 ***************************************************/
#define DISPL_PRESCALER SPI_BAUDRATEPRESCALER_2     //prescaler assigned to display SPI port
#define TOUCH_PRESCALER SPI_BAUDRATEPRESCALER_256	//prescaler assigned to touch device SPI port

/*****************     STEP 4      *****************
 ************* SPI COMMUNICATION MODE **************
 *** enable SPI mode want, uncommenting ONE row ****
 **** (Setup the same configuration on CubeMX) *****
 ***************************************************/
#define DISPLAY_SPI_POLLING_MODE
//#define DISPLAY_SPI_INTERRUPT_MODE
//#define DISPLAY_SPI_DMA_MODE // (mixed: polling/DMA, see below)

/*****************     STEP 5      *****************
 ***************** Backlight timer *****************
 * if you want dimming backlight UNCOMMENT the
 * DISPLAY_DIMMING_MODE below define and properly
 * set other defines.
 * Using backlight as a switch (only on/off) leave
 * DISPLAY_DIMMING_MODE commented
 * if DIMMING:
 * On CubeMX set DISPL_LED pin as a timer PWM pin.
 * Timer COUNTER PERIOD (ARR) defines dimming light steps:
 * keep it low value - e.g. 10 - if dimming with buttons,
 * use higher value - e.g. 100 - if dimming with encoder, ...
 * Avoiding display flickering timer PRESCALER should
 * let timer clock to be higher than COUNTER PERIOD * 100 Hz.
 * Set all other defines below
 ***************************************************/
//#define DISPLAY_DIMMING_MODE						// uncomment this define to enable dimming function otherwise there is an on/off switching function
#define BKLIT_TIMER 				TIM3			//timer used (PWMming DISPL_LED pin)
#define BKLIT_T 					htim3			//timer used
#define BKLIT_CHANNEL				TIM_CHANNEL_2	//channel used
#define BKLIT_CCR					CCR2			//Capture-compare register used
#define BKLIT_STBY_LEVEL 			1				//Display backlight level when in stand-by (levels are CNT values)
#define BKLIT_INIT_LEVEL 			100				//Display backlight level on startup

/*****************     STEP 6      *****************
 ************* frame buffer DEFINITION *************
 * IF NO TOUCHGFX:
 * BUFLEVEL defines size of the 2 SPI buffers:
 * buffer size is 2^BUFLEVEL so 2 means 4 bytes buffer
 * and 10 means 1 kbyte (each).
 * it must be not below 10!
 * If TOUCHGFX:
 * buffers are not used if displays handle RGB565, if
 * display uses RGB666 one buffer will be used for
 * color format translation. In this case set
 * BUFLEVEL following this table:
 * TouchGFX buffers>10KB need BUFLEVEL 15
 * TouchGFX buffers>5KB  need BUFLEVEL 14
 * TouchGFX buffers>2700bytes need BUFLEVEL 13
 * TouchGFX buffers>1300bytes need BUFLEVEL 12
***************************************************/
#define BUFLEVEL 12

After the setup  i do:
  Displ_Init(Displ_Orientat_0);		// initialize the display and set the initial display orientation (here is orientaton: 0°) - THIS FUNCTION MUST PRECEED ANY OTHER DISPLAY FUNCTION CALL.
  Displ_CLS(BLACK);			// after initialization (above) and before turning on backlight (below), you can draw the initial display appearance. (here I'm just clearing display with a black background)
  Displ_BackLight('1');

  while (1)
  {
	  // display testing
	  Displ_PerfTest();		// shows display graphics and performance
	  // touch testing
//	  Touch_ShowData();		// polls touch device and shows values returned, showing touch interrupt calls also
//	  Touch_TestDrawing();

And the screen is just white empty.

Also for the order of init i have seen these also before DMA is before SPI
  MX_GPIO_Init();
  MX_DMA_Init();
  MX_SPI1_Init();
  MX_USART3_UART_Init();

and even if i place usart between DMA and SPI it is still the same. Also in your other issue in both proposals DMA is first just SPI is not always the next function. So i do not see miss ordering except if i did not understand something well

About "blank" screen in polling mode, one possibility is that GPIO pins involved in display handling have a too low speed. I see that all of them are set to LOW ("maximum output speed" parameter). I don't know too much about F7 (and, please, refer to the datasheet) but setting SPI prescaler to 2 and pins speed to low, could be a mistake. Try to rise speed of DISPL_DC, DISPL_CS and TOUCH_CD at least to HIGH.

It seems that several test made problems in series. What do i mean. Initial i had wrongly set the backlight to I while i was not using the dimming of led. So probably DMA and high speed was working just the screen was down.
Second i changed the speed to low and probably broke the SPI that worked on polling only. Now after fixing the backlight, i had the issue with SPI that no data are sent.
I placed the high frequency in polling and it still did not work.
But DMA and high frequency with proper backlight works.
This brings me to last question touch seems to work but i do not know if it needs calibration the data are in hex. just transaltion to decimal will give the value from the resolution?

I did calibration and it seems that  even after this the Touch_TestDrawing(); it opens what you say text to place the touch and move and dark blue background but.
After moving across the screen it get confused several other colours our coming and at the end we end up with white page.

tried prescaler to 2 and 4 with 54mhz and 27mhx but the result is the same. this is only in touch_testdrawing() the Touch_ShowData dows not break.

in any case it seems that i am wrong and i apologize i rechecked the code and it seems that i am using all that time polling and not DMA, which means that DMA does not work for me. i have all pins in high speed the prescaller either at 4 (27Mhz) or 2 (54Mhz) and the stm at 214Mhz the dma interupt is enabled :( apologeis again i thought problem was solved but it is not true

It seems that this is more about discussion of issues not issue. If you request to split i will try to split or move the discussion in some other system. My final goal is to user TouchGFX .
I have done calibration to the Direct access and it seems that it works i have set the calibration variables. I have set orientation to 0 in main.c and in TOUCHGFX i have maid the screen to be landscape it seems that this way i get proper display though i do not know if this is the proper setup.
What i did not get is touch. I placed a button that on pressed it changes icon and in the simulation it works (so base on that i can then read touchgfx to understand complete controll) which is a very basic test for screen and touch. But i do not get any touch.
I need to recalibrate? or add something else?
  printf("Hello World\n\r");

  HAL_GPIO_WritePin(LD2_GPIO_Port, LD2_Pin,GPIO_PIN_SET);
  HAL_Delay(1000);

  HAL_GPIO_WritePin(LD2_GPIO_Port, LD2_Pin,GPIO_PIN_RESET);
  HAL_Delay(2000);
  HAL_GPIO_WritePin(LD2_GPIO_Port, LD2_Pin,GPIO_PIN_SET);
  Displ_Init(Displ_Orientat_0);		// initialize the display and set the initial display orientation (here is orientaton: 0°) - THIS FUNCTION MUST PRECEED ANY OTHER DISPLAY FUNCTION CALL.
  touchgfxSignalVSync();			// after initialization (above) and before turning on backlight (below), you can draw the initial display appearance. (here I'm just clearing display with a black background)
  Displ_BackLight('1');
  printf("Hello World\n\r");

  /////////////

  /* USER CODE END 2 */

  /* Infinite loop */
  /* USER CODE BEGIN WHILE */

  while (1)
  {

	  touchgfxSignalVSync();
    /* USER CODE END WHILE */

  MX_TouchGFX_Process();
    /* USER CODE BEGIN 3 */
  }

Hi kyrpav,
my advice is to not move to TouchGFX before having a good experience and a full control of your system (uC, DMAs, wiring, displays and so on).
Moving to TouchGFX not will just add more complexity (a lot of) to your work.
That's my two cents to your project: dont' move to TouchGFX before a good experience on direct control of the display.
By the way, I'm working on a video entirely dedicated to developing a TouchGFX project with these small displays.
Supposing that user have full control of the display, I'll show him (her) how to get (or better said: my experience in getting) a performing TouchGFX project.
I'm working to show how to setup and handle buttons, more complex widgets (like sliders or scroll wheels and scroll lists) or screen transitions.
But all of this is just my hobby (even if I'm an electronic engineer, I'm involved in a different field) and I need some more time.

thank you for the info. You propose to switch back to direct access to try to fix the DMA which i do not know at all how. Do you have any other idea about the dma?
My main current issue for touchgfx is how the touch is passed, do you know if this is problem of touchgfx setup or problem of the setup of driver?

I can not resist to explain the mistakes. First one is solved for the DMA by rechecking your video and my setup. I was using SPI_RX for the dma and not TX , so you can imagin what happens.
No the touch with touchgfx i have to check again since no other function is needed to be added

Good! Keep on working!

I saw the rest of the video and remembered the rest of functions for the touch. No touch is almost ok.
I say almost cause it seems that does something but it does not work as the simulator. Functionality was to have a button with one icon that changes when user presses and the icon is different while the button remains pressed.
In the screen i see flickering of trying to refresh the button (it seems that it is only the portion of the button) that it is pressed but it does not stay at pressed status it goes immediately back to not pressed. On top of this if you have this DELAY_KEY_REPEAT >0 you get the button that tries to be pressed and unpressed all the time like flickering.
Tried with value =0 but it is not fixed.
So the question is which part of the touch library handles that continues press of button that i missed? It seems in your example you do not have this issue or it was hidden on video below the finger.


Confirm Python 3.12 support

I followed the pattern set by the previous 3.11 MR: #447
The tox envlist change might be debatable, but my understanding is that Python 3.12 will only be supported on 4.2 and above.
As my colleague mentioned in #570, we've just started using v3 of this package, so happy to chip in with small things like this. I thought it might want adding for 3.0. We know that we shouldn't be using v3 yet, but the risk level is fine for what we're doing.
Thanks for your efforts!

Thanks @jgillard for this path 🙏
Based on Django supported Python versions, Django not support Python 3.12.
Django 5.0 supports Python 3.12 and will be released on December 2023.
So, let's keep 3.12 only for main branch for now.

You're correct about Django 4.2 not supporting Python 3.12 (although that may change at some point), so for now I've only applied it to django's main branch as you suggested.


add detect_region code for S3Builder

close #2632
I am newer for OpenDAL and this is my first PR for OpenDAL.
Now the test case for detect_region is not added.
I am confusing how to build a S3Builder with custom region, default is None, and use region(str) function needs the S3Builder has a custom region.
BTW, how check Error in test case, I try use assert_eq!() for check Error, it failed.
Thanks for helping.

Thanks for the PR! Let me make the feature request more clear.
As described in the issue, I want an API of S3Builder like async fn detect_region(&self) -> Result<Option<String>>, it's used to fetch region with input endpoint and bucket. (Maybe API should be async fn detect_region(&self, endpoint: &str, bucket: &str)?)
The API should use the client in S3Builder to detect the region, the behavior will be similiar to
https://github.com/apache/incubator-opendal/blob/44e93563227e58520f07e0ebcf3be14601b59d9a/src/services/s3/backend.rs#L504-L604
The caller can use this function to detect region when they needed in async context. (Please note that S3Builder::build is blocking API, so we can't do this.)


Android: unescape html entities when showing alt text

In the Android app, showing title text also show html entites; see for example this post in the android app:
https://www.newsblur.com/rss_feeds/original_story?story_hash=169:a6ca74 "Technically" (http://xkcd.com/1475/)
It shows like this:
"Technically that sentence started with 'well', so--" "Ooh, a rock with a fossil in it!"
My instinct is to convert html entities, but maybe you could show it in a webview. This needs to be examined.

Yes, this is definitely something we should be doing.


Error : Missing Parameters (classes.dart file)

Describe the bug
In the original code, the joinLink parameter was missing in the Classes
(Friday\lib\models\classes.dart)
Prerequisites

I am running the latest version
I checked the documentation and found no answer
I checked to make sure that this issue has not already been filed
I'm reporting the issue to the correct repository (for multi-repository projects)

To Reproduce
Steps to reproduce the behavior:

Go to Friday repo
Click on classes.dart file
See error

Expected behavior
code with

joinlink
no errors

Helloo
@avinashkranjan Please review my Pr #224
Thank you


bug: menu not visible

What version of daisyUI are you using?
v2.51.5
Describe your issue
I have a NavBar with a menu and a slider under it the slider hides a part of the menu
What browsers are you seeing the problem on?
Firefox
Reproduction URL (optional)
No response

Please provide a example code so I can see the issue.

`

daisyUI

Item 1

          Parent

Submenu 1
Submenu 2

Item 3

❮
❯

❮
❯

❮
❯

❮
❯

`
Sry im always caling the carousel slider

Item 1

            Parent

Submenu 1
Submenu 2

Item 3

daisyUI

Item 1

          Parent

Submenu 1
Submenu 2

Item 3

Get started

❮
❯

❮
❯

❮
❯

❮
❯

My problem was that if you use a Carousel below a navbar it automaticly places the menu (providerd by the navbar) under the Carousel at the time i made this Issue i didn't knew about negativ Z values the problem can easyly be fixed with a -z-2 class in the carousel

insted of

Looks like you found a fix.
Let me know if you had any questions.


Is there a way to make llmcord not log messages received?

Heyo, great project! I'm curious if there's a way to get rid of Message received messages that include user's discord ID and their prompt. Even if it's just in-memory, I'd rather make my users feel that their private DM conversations are that, private.
I looked around at the documentation and didn't see anything like this there so uh, if it's not a feature yet then would it be possible to implement such a thing? :3

Good point, I'll think about this. Since llmcord.py does not impose any user rate limits I figured every message should get logged so you can monitor for abuse, etc...
For now you can easily disable it by commenting out this 1 line in llmcord.py (by adding a # at the beginning):
https://github.com/jakobdylanc/llmcord.py/blob/68d1ed0254a70042ff660016b8010d3d50f1d7f5/llmcord.py#L177
Or you can keep the logging but without the message content, by changing that line to:
logging.info(f"Message received (user ID: {new_msg.author.id}, attachments: {len(new_msg.attachments)}, reply chain length: {len(reply_chain)})")

I'm curious to know more people's thoughts on this. Maybe I should just remove message content from the logging messages entirely? Maybe replace it with "character count" so you can still know the size of the message.

I'm curious to know more people's thoughts on this. Maybe I should just remove message content from the logging messages entirely? Maybe replace it with "character count" so you can still at least know the size of the message.

could just pipe the output to /dev/null.

What do you mean exactly? I wouldn't want something that only works on Linux.

I just meant you could pipe the output to /dev/null when running the script if you wanted to make sure the input wasn't being stored in memory.  But yes that would only work on linux.

Good point, that is the way. Closing :)


Gateway BRP069A78 not working

Addon not connecting to ip adapter

BRP069A78 does not have local API, so can't be done.


Content store writer checks expected

Fixes #1004

Codecov Report

Merging #1006 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1006   +/-   ##
=======================================
  Coverage   59.05%   59.05%
=======================================
  Files           5        5
  Lines         779      779
=======================================
  Hits          460      460
  Misses        205      205
  Partials      114      114

Continue to review full report at Codecov.

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

@dmcgowan If expected is provided, it has been to validated against the result at the end.

@stevvooe should we just hold that value in memory for now? When we switch to boltdb we can persist/load it

@dmcgowan Fair enough!
LGTM

LGTM


Prevent a click event on pan

I'm binding click events to polygons that display a tooltip. If the mouseup target is a polygon, the tooltip is shown. This becomes an issue when the user is panning because it will inadvertantly trigger the tooltip due to the mouseup being fired at the end of the pan. Is there a method I can call to check if a pan is active?
Edit:
I ended up finding a solution and figured I'd post it in case anyone else runs into this issue. Here's the snippet from stackoverflow that I used to check for pans:
// Check if the mouse is dragging

var element = document.getElementById('canvas');
var flag = 0;

element.addEventListener("mousedown", function() {
    flag = 0;
}, false);

element.addEventListener("mousemove", function() {
    flag = 1;
}, false);

element.addEventListener("mouseup", function() {
    if (flag === 0) {
        console.log( "Click" )
    } else if (flag === 1) {
        console.log( "Drag" );
    }
}, false);

I encountered the same problem. I solved it by using the this.state in svg-pan-zoom.js with a trick. The main idea is alike.

Thanks from 2019 @jonnymaceachern


doubly noncentral F

http://www.math.wm.edu/~leemis/chart/UDR/PDFs/Doublynoncentralf.pdf
The PDF of doubly noncentral F has a function in the denominator called B. I could not find the definition for it or an alternative version of the pdf since the resources on this distribution are very limited.

That would be the Beta function.  Represented as betaFunc in the external syntax.


Sigh downloads latest profile. Gym doesn't use it. (Xcode8)

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
Hi there. Prior to Xcode 8, the instructions in the code signing guide gave predictable results when using sigh to select the correct provisioning profile. However, after following the project setup instructions for Xcode 8 here, I have found the results to be unpredictable. sigh will always successfully downloads/creates/selects the desired profile, but gym doesn't appear to always use that profile. Sometimes, it uses an earlier version of the profile. Are there any additional steps to take with Xcode8 to get more reliable results? Thanks.
Complete output when running fastlane, including the stack trace and command used
n/a
Environment
Please run fastlane env and copy the output below. This will help us help you :+1:
Scotts-iMac:Test scottrhoyt$ fastlane env
[22:16:22]: -------------------------------------------------
[22:16:22]: --- Step: Verifying required fastlane version ---
[22:16:22]: -------------------------------------------------
[22:16:22]: Your fastlane version 1.105.3 matches the minimum requirement of 1.105.0  ✅
[22:16:22]: ------------------------------
[22:16:22]: --- Step: default_platform ---
[22:16:22]: ------------------------------
[22:16:22]: Variable Dump:
[22:16:22]: {:DEFAULT_PLATFORM=>:ios}
[22:16:22]: Could not find lane 'ios env'. Available lanes: ios test, ios beta, ios adhoc, ios release

could you please - upgrade fastlane and fastlane_core to its latest version, and re-run the fastlane env - as this will help to troubleshoot the issue. many thx

Thanks for the assistance, @hjanuschka. Unfortunately, fastlane env doesn't seem to be working for me:
Scotts-iMac:Test scottrhoyt$ gem update fastlane_core
Updating installed gems
Nothing to update
Scotts-iMac:Test scottrhoyt$ gem update fastlane
Updating installed gems
Nothing to update
Scotts-iMac:Test scottrhoyt$ fastlane env
[10:09:08]: -------------------------------------------------
[10:09:08]: --- Step: Verifying required fastlane version ---
[10:09:08]: -------------------------------------------------
[10:09:08]: Your fastlane version 1.105.3 matches the minimum requirement of 1.105.0  ✅
[10:09:08]: ------------------------------
[10:09:08]: --- Step: default_platform ---
[10:09:08]: ------------------------------
[10:09:08]: Variable Dump:
[10:09:08]: {:DEFAULT_PLATFORM=>:ios}
[10:09:08]: Could not find lane 'ios env'. Available lanes: ios test, ios beta, ios adhoc, ios release

Scotts-iMac:Test scottrhoyt$ fastlane -v
fastlane 1.105.3

The latest fastlane version is 1.106.1 (https://rubygems.org/gems/fastlane), seems like gem update didn't work. Could you try sudo gem update fastlane? How do you usually update fastlane?
Have you followed the Xcode 8 docs you linked? Can you share a screenshot of your Xcode project's build settings

I did follow the instructions on the Xcode 8 page, by disabling automatic management and selecting my profiles manually. Here's what the build settings look like (sorry I had to redact things for legal reasons):

Just did another gem update fastlane and it picked up 1.106.1 this time so fastlane env worked:
fastlane env output
🚫 fastlane environment 🚫
Stack

Key
Value

OS
10.12

Ruby
2.3.1

Bundler?
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
8.0

Git
git version 2.10.1

Installation Source
/usr/local/bin/fastlane

Host
Mac OS X 10.12 (16A319)

Ruby Lib Dir
/usr/local/Cellar/ruby/2.3.1_2/lib

OpenSSL Version
OpenSSL 1.0.2j  26 Sep 2016

fastlane files:
`./fastlane/Fastfile`
# Customise this file, documentation can be found here:
# https://github.com/fastlane/fastlane/tree/master/fastlane/docs
# All available actions: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Actions.md
# can also be listed using the `fastlane actions` command

# Change the syntax highlighting to Ruby
# All lines starting with a # are ignored when running `fastlane`

# If you want to automatically update fastlane if a new version is available:
# update_fastlane

# This is the minimum version number required.
# Update this, if you use features of a newer version
fastlane_version "1.105.0"

default_platform :ios

platform :ios do
  before_all do
    # ENV["SLACK_URL"] = "https://hooks.slack.com/services/..."

    #carthage
  end

  desc "Runs all the tests"
  lane :test do
    scan
  end

  desc "Submit a new Beta Build to Apple TestFlight"
  desc "This will also make sure the profile is up to date"
  lane :beta do
    # match(type: "appstore") # more information: https://codesigning.guide
    gym(scheme: "Test") # Build your app - more options available
    pilot

    # sh "your_script.sh"
    # You can also use other beta testing services here (run `fastlane actions`)
  end

  desc "Deploy a new version to Hockey"
  lane :adhoc do
    sigh(adhoc: true, force: true)
    cert
    gym(scheme: "Test")
    hockey(api_token: "************")
  end

  desc "Deploy a new version to the App Store"
  lane :release do
    # match(type: "appstore")
    # snapshot
    gym(scheme: "Test") # Build your app - more options available
    deliver(force: true)
    # frameit
  end

  # You can define as many lanes as you want

  after_all do |lane|
    # This block is called, only if the executed lane was successful

    # slack(
    #   message: "Successfully deployed new App Update."
    # )
  end

  error do |lane, exception|
    # slack(
    #   message: exception.message,
    #   success: false
    # )
  end
end

# More information about multiple platforms in fastlane: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Platforms.md
# All available actions: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Actions.md

# fastlane reports which actions are used
# No personal data is recorded. Learn more at https://github.com/fastlane/enhancer

`./fastlane/Appfile`
app_identifier "com.scotthoyt.Test" # The bundle identifier of your app
apple_id "***********" # Your Apple email address

team_id ********  # Developer Portal Team ID

# you can even provide different app identifiers, Apple IDs and team names per lane:
# More information: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Appfile.md

fastlane gems

Gem
Version
Update-Status

credentials_manager
0.16.2
✅ Up-To-Date

spaceship
0.36.1
✅ Up-To-Date

fastlane_core
0.53.0
✅ Up-To-Date

deliver
1.14.3
✅ Up-To-Date

snapshot
1.16.2
✅ Up-To-Date

frameit
2.8.0
🚫 Update availaible

pem
1.3.2
✅ Up-To-Date

cert
1.4.3
✅ Up-To-Date

sigh
1.11.2
✅ Up-To-Date

produce
1.2.1
✅ Up-To-Date

gym
1.11.3
✅ Up-To-Date

pilot
1.11.0
✅ Up-To-Date

scan
0.13.1
✅ Up-To-Date

supply
0.7.1
✅ Up-To-Date

match
0.8.1
🚫 Update availaible

screengrab
0.5.5
✅ Up-To-Date

fastlane
1.106.1
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded*
Loaded gems

Gem
Version

did_you_mean
1.0.2

highline
1.7.8

commander
4.4.0

terminal-table
1.4.5

json
1.8.3

multipart-post
2.0.0

faraday
0.9.2

faraday_middleware
0.10.0

dotenv
2.1.1

net-ssh
3.2.0

net-sftp
2.1.2

plist
3.1.0

security
0.1.3

krausefx-shenzhen
0.14.10

slack-notifier
1.5.1

rouge
1.11.1

xcpretty
0.2.4

terminal-notifier
1.7.1

claide
1.0.1

colored
1.2

credentials_manager
0.16.2

multi_xml
0.5.5

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20160826

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
1.6.8

xcode-install
2.0.7

word_wrap
1.0.0

multi_json
1.12.1

babosa
1.0.2

excon
0.54.0

rubyzip
1.1.7

gh_inspector
1.0.2

fastlane_core
0.53.0

bundler
1.13.6

mini_magick
4.5.1

xcpretty-travis-formatter
0.0.4

uber
0.0.15

representable
2.3.0

retriable
2.1.0

mime-types-data
3.2016.0521

mime-types
3.1

hurley
0.2

little-plugger
1.1.4

logging
2.1.0

jwt
1.5.6

memoist
0.15.0

os
0.9.6

signet
0.7.3

googleauth
0.5.1

httpclient
<IP_ADDRESS>

google-api-client
0.9.19

i18n
0.7.0

thread_safe
0.3.5

tzinfo
1.2.2

activesupport
<IP_ADDRESS>

io-console
0.4.6

xcodeproj
1.3.3

generated on: 2016-10-24

By the way, when this happens to me, my workaround is to just nuke my installed provisioning profiles on my machine. So, sigh is correctly downloading and installing the right profile. And gym uses it when it's the only option.

@scottrhoyt Thanks for reaching out about this. Are your provisioning profiles set manually in your build settings? Fastlane isn't compatible with automatic code signing introduced in Xcode 8.


[Core] Add anyio dependency for lock handling

This introduces anyio as a dependency to azure-core, and we leverage its Lock class in AsyncBearerTokenCredentialPolicy to allow it to work with both asyncio and trio concurrency mechanisms.
requests was also bumped to allow the mindependency check to pass with the minimum version of anyio.

Closes: https://github.com/Azure/azure-sdk-for-python/issues/32968

API change check
API changes are not detected in this pull request.

/check-enforcer override

😩
This forces installation of multiple additional transitive dependencies for any and all users of azure-core.
anyio-4.2.0 exceptiongroup-1.2.0 sniffio-1.3.0

It might be better to explore whether the same thing could be done without forcing the dependency; after all, there are plenty of other places in azure-core that only support asyncio still.

there are plenty of other places in azure-core that only support asyncio still.

Thanks for your feedback.
In terms of "there are plenty of other places in azure-core that only support asyncio still."
if you don't mind, could you share more details?


https://www.reuters.com/

Energized Protection - block ⚡

Let's make an annoyance free, better open internet, altogether!

Issue Submit Form

  Provide the following infos properly, which will help us to resolve your issue quickly.

Issue(s):
Type x in between [ ] and make sure there isn't any space between brackets. Example; for Your Selected Issue(s), type like this - [x]
You can select more than one category of issues if you need to!

[ ] Whitelist
[x] Blacklist
[ ] App Broken
[ ] Website Broken
[ ] Request
[ ] Other Issue

Pack(s):
Write the name of the pack(s) you are using.

Pack(s) Name: blu

Extension(s):
Also name the extension(s) if you are using any. If you don't then leave it blank.

Extension(s) Name:

Domain(s):
If you are submitting this issue for whitelist/blacklist issue, send us the domain(s) for whitelisting/blacklisting here. Kindly use the Code Tag to prevent tracking.

Domain(s):

https://www.reuters.com/article/us-usa-trump-mattis/trump-annoyed-by-resignation-letter-pushes-out-mattis-early-idUSKCN1OM0HO

Your Config:
Just to ensure there is no issues or conflicts with other app/software/magisk module/extension/source list.
Make sure you are running Energized Protection Service only.

Client: blokada

Version: 18.12.355

Details:
Write us a lil bit more about your issue or query. You can attach any screen shot or log of the issue or advert, this will help to highlight it.

Your Issue Detail:

There are pictures of sponsored ads. They're not clickable and tapping it just opens the picture. But idk if anything can be done about it.

Thank you for making Energized Protection great with your kind help!

A project by Ador with ❤

This ad images are from the same origin. But clicking on those images takes you to www.dianomi.com. Which is blocked. So no worry about that!


Fix dependency verification when first resolved configuration has dependency verification disabled

Previously if the dependency artifact was already resolved with an unverified configuration, then when resolving the second configuration resolveArtifact was not called. Since in resolveArtifact we register verification operation (and download signature file) verification was not done.
So now for DependencyVerifyingModuleComponentRepository when resolveArtifacts is called we also do a call to resolveArtifact in case it was not called before, so verification operation is registered.
Fixes #3497

@bot-gradle test QFL

OK, I've already triggered the following builds for you:

QuickFeedbackLinux build

@bot-gradle test this

OK, I've already triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test this

OK, I've already triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test and merge

OK, I've already triggered a build for you.


Windows Port

It'd be awesome if Windows users also could utilise this application. When this issue is solved, the first release of this port will be the latest one (Probably 1.3)

Tasks to be done:

[] Add cross-platform checking
[] Build script for generating an installer
[] Build Progress Windows build