How to make ApiVersionNeutral as default behavior ?

In other words, if I don't specify any ApiVersion attributes, it works as ApiVersionNeutral.

In short, out-of-the-box there is not a way because, honestly, it wouldn't make any sense. API version-neutral means that an API accepts any and all versions, including none at all. If this was a supported default behavior, then it would be like you didn't have API versioning at all.
I suspect you have a specific reason why you want to enable such a behavior. If you can articulate you scenario in more detail, I can help you come up with a configuration to reach your goals.

Thanks your answer.
Because I currently have a small asp.net core project, its backend is mixed with API for desktop application calls, website payment interface, some other functions of the website interface.
Only the "API for desktop application calls" needs versioning. Sometimes I forget to add an ApiVersionNeutral attribute to Controller, then causes the request to return an error, then I'll add ApiVersionNeutral.
Although this can solve the problem, it seems that every time a new controller is added, there are some obstacles.

I see... it sounds like you want to enable:
options.AssumeDefaultVersionWhenUnspecified = true;

This feature was meant to ease grandfathering in existing services as you have, although some people have tried to use it for other purposes.
The way to think about this is that all of your services have some existing, logical API version that you never assigned a value to. Maybe you all it "V1", "Initial Release", or something else, but it has some kind of version. As you're observing, if all APIs suddenly require an explicit API version for every call (the default behavior) that breaks existing clients. Enabling this option allows clients to call these existing services without explicitly specifying an API version. When that happens, the default behavior will be to match the options.DefaultApiVersion value. Controllers which do not have any attribution or conventions will implicitly have this API version. That means without any code changes, all of your existing controllers have this API version and clients can explicitly or implicitly match it with your existing route configuration.
The result of these behaviors is that your existing services do have an API version, but existing clients do not have to specific to continue using it. In this configuration, the behavior only applies to that API version. In order for clients to use new API versions, they will have to specify the API version explicitly. Existing clients are already bound to a specific API version, they just don't know (or care) what API version that is. If they want to use a newer API, then requiring an explicit API version to request it is part of that migration process.
The default value for options.DefaultApiVersion is 1.0. You can change it to whatever version you want, but once you set it, you shouldn't change it - ever. If you're using a numeric scheme, then you might choose a value 0.9. If you're using a date scheme, then you might use the (approximate) date you released your services.
Ultimately, this configuration allows you to transition your existing services over time. You almost certainly do not want existing APIs to be version-neutral because that would mean your existing services support all versions now and in the future. It is possible to configure large swaths of controllers as API version-neutral, but you should be careful in doing so. Once you configure that it may be difficult or impossible to configure explicit versioning against them without breaking existing clients. Only APIs that have zero visible contract changes in their wire protocols should be version-neutral. Possible examples include health monitoring APIs such as pings and heartbeats or DELETE APIs.
I hope that helps clear things up.

Thank you for your detailed answer. I will take some time to understand it.

Did this make sense? Were you able to get things working the way you wanted and expected?

Yes, AssumeDefaultVersionWhenUnspecified works well. :)


amp js install

Hi there,
we have a js project to be referenced on client's site. Is it possible to install js to page by another js?  I got "[Amplitude] Error: could not load SDK" if I inject install code by another js I have. Also, how to set namespace in our project if there are amp js already installed on their site?
Thanks.

Hi, could you give me more details how you install our js sdk?


Add mode 7 background floors and create a grass theme

Adds mode 7 background that use shaders to project a square texture onto a floor in the background. Using this I created a test grass theme using some sprites I borrowed from legacy 127. The background image still needs set up properly with the layers as it's just using the preview from 127. The recommended images from me are an image that is 512x512 at scale 2 in the shader params. I don't recommend touching the other parameters in the shader as those are fairly fine tuned to the perspective. I also fixed up the grass textures so they'll be easier to implement later on.

Oh one last thing the sky is still night in the grass theme but that should probably be changed

made the sky daytime now, fixed the background not looping correctly as well


SOCKS5 CONNECTION_NOT_ALLOWED  issue

Hi!
I am observing strange behavior when connecting with username/password via a SOCKS5 proxy.
The connection to some proxy servers is fine, but when I try to connect to other servers, I get an error CONNECTION_NOT_ALLOWED.
I am always using standard connection method:
private SocketFactory getSocksSocketFactory() {         final InetSocketAddress address = new InetSocketAddress(request.proxyIp, Integer.parseInt(request.proxyPort));         final AuthenticationCredentials credentials = getCredentials();         final BrokerConfiguration configuration = new StandardBrokerConfiguration(SOCKS5, address, credentials);         final SocketFactory defaultSocketFactory = SocketFactory.getDefault();         return new BrokeredSocketFactory(configuration, defaultSocketFactory);     }
Trace is:
Caused by: java.net.ConnectException: SOCKS Connect Request Failed: [CONNECTION_NOT_ALLOWED] Status [2]   at com.exceptionfactory.socketbroker.protocol.socks.Socks5SocketBroker.connect(Socks5SocketBroker.java:95)   at com.exceptionfactory.socketbroker.BrokeredSocket.connect(BrokeredSocket.java:103)
The same situation is with the browser settings. If I use Firefox and input the username and password separately, then the connection does not occur. But if I fill an automatic proxy configuration field like socks5://user1013:123456@<IP_ADDRESS>:10652 then everything works well.
Please tell me where is the problem?

It seems like has relations with using DNS thru the SOCKS. Like curl --socks5-hostname option.
I was noticed that
curl --socks5 username:userpassword@proxy_ip:proxy:port  http://example.com
get error (2) CONNECTION_NOT_ALLOWED
but:
curl --socks5-hostname username:userpassword@proxy_ip:proxy:port  http://example.com
works fine.

Fixing it!
Just in file: com.exceptionfactory.socketbroker.protocol.socks.SocksRequestEncoder
replace:
final InetSocketAddress socketAddress = request.getSocketAddress();         final InetAddress destinationAddress = socketAddress.getAddress();         if (socketAddress.isUnresolved()) {                    outputStream.write(SocksAddressType.DOMAIN_NAME.getCode());
to
final InetSocketAddress socketAddress = request.getSocketAddress();         final InetAddress destinationAddress = socketAddress.getAddress();         //if (socketAddress.isUnresolved()) {         if (socketAddress != null) {             outputStream.write(SocksAddressType.DOMAIN_NAME.getCode());
At date the hostname allways will sent to the server side, and then will be resolved to IP address.
It's works fine for Me.

Thanks for describing the issue and different behavior @r-voland.
When Socket Broker encodes the SOCKS5 proxy connection request, it checks whether the InetSocketAddress is resolved. If the address is unresolved, it creates a DNS connection request and uses InetSocketAddress.getHostString(). If the address is resolved, it uses InetSocketAddress.getAddress().
In this scenario, if you want to force the use of the DNS address, instead of the resolved IP address, use InetSocketAddress.createUnresolved() with the DNS name and port number.

I think it should be possible to force the hostname to be sent to the server so that it is resolved there and not locally. In my case, for example, the hostname was resolved locally to the wrong IP address (not to be unresolved). And on the server - was resolved in the correct one.  Therefore, it would be good to send hostname to the server without finding out whether the name was resolved locally or not. This is exactly what the --socks5-hostname parameter does in curl.

Thanks for the reply and suggestion @r-voland.
During the initial implementation, I considered something along the lines of the resolution policy described. Instead of introducing a configuration option, however, I opted to follow the pattern of the JDK SocksSocketImpl, which uses InetSocketAddress.isUnresolved() to determine whether to send the request as a Domain Name.
The downside is that this behavior is not necessarily intuitive, which gets back to some policy-based approach. This could be added to the BrokerConfiguration, retaining the existing behavior, but providing other options.
At the moment, however, it seems better to add some documentation around the current behavior, which still allows the caller to make hostname resolution decisions. If there are libraries that don't provide direct control of InetSocketAddress construction, that may be a reason to consider the policy option.
If you have a public example of where this new policy is necessary, versus changing the InetSocketAddress construction, that would be a helpful point of consideration.


YARN-11365. Fix NM class not found on Windows

Description of PR
The environment variable HADOOP_YARN_HOME wasn't enclosed properly. The leading % was missed out. This caused the classpath to become invalid. Ultimately, leading to the NM not being able to start -
PS D:\> cmd /c $Env:HADOOP_HOME\bin\yarn.cmd nodemanager
Error: Could not find or load main class org.apache.hadoop.yarn.server.nodemanager.NodeManager

This PR fixes the same.
How was this patch tested?
Tested locally.
For code changes:

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m  2s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
43m 14s

trunk passed

+1 :green_heart:
mvnsite
5m  6s

trunk passed

+1 :green_heart:
shadedclient
71m  4s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 51s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
4m 59s

the patch passed

+1 :green_heart:
shadedclient
27m 46s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
9m 21s

hadoop-yarn in the patch passed.

+1 :green_heart:
asflicense
1m  3s

The patch does not generate ASF License warnings.

117m  2s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5093/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5093

Optional Tests
dupname asflicense mvnsite unit codespell detsecrets

uname
Linux 4534ccd9f6c9 4.15.0-191-generic #202-Ubuntu SMP Thu Aug 4 01:49:29 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / a74fbf6dcf290ea54a2a2ba7cd548262d696bc60

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5093/1/testReport/

Max. process+thread count
533 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn U: hadoop-yarn-project/hadoop-yarn

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5093/1/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

Justification for not satisfying the test4tests criterion - there's currently no test that exercises this flow. Adding a new test for this isn't trivial. The change is simple and effective that it doesn't need to be tested.


log file

create a log file which shows any adjustments:

late day counts (and modifications)
explicitly waived assignments

could be stored in a new column per student to explain grade computation (clarifies late penalties are applied between "hw_mean" and "mean" etc)

might be easiest to just redirect:
python3 -m gradescope_mean scope.csv > log.txt

so no need to explicitly include this option (I suppose a quiet option might be relevant ... but seems more work than its worth).
Leaving this issue open as some more work is needed to ensure log gives user confidence that their inputs were applied properly.  Maybe provide a student level log string, which collects information so that its output to the per_student csv.


[Event Request] codeunit 99000833 "Req. Line-Reserve" - OnBeforeTransferReservationsReqLine

Could we please have an event OnBeforeTransferReservationsReqLine in Codeunit 99000833 "Req. Line-Reserve" in function TransferReqLineToReqLine?
Code Example:
NewReqLine.TestField("No.", OldReqLine."No.");
NewReqLine.TestField("Variant Code", OldReqLine."Variant Code");
NewReqLine.TestField("Location Code", OldReqLine."Location Code");
    // start kvspsa - Event Raiser
    OnBeforeTransferReservationsReqLine(OldReservEntry, OldReqLine, NewReqLine);
    // stop kvspsa

    if TransferAll then begin
        OldReservEntry.SetRange("Source Subtype", 0);

Event:
[IntegrationEvent(false, false)]
local procedure OnBeforeTransferReservationsReqLine(var OldReservEntry: Record "Reservation Entry"; var OldReqLine: Record "Requisition Line"; var NewReqLine: Record "Requisition Line")
begin
end;
Thank you and kind regards

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Rendering taxonomy term page

Hi, I'm trying to show a tag term page.
As example I created a url pattern in admin/config/search/path/patterns/add for taxonomy terms of tags vocabulary as tags/[term:name].
Then I added in the pages folder a file named term.tsx with the following code:
import { GetStaticPathsResult, GetStaticPropsResult } from "next"
import Head from "next/head"
import { DrupalTaxonomyTerm } from "next-drupal"

import { drupal } from "lib/drupal"
import { TaxonomyTermTags } from "components/taxonomy_term--tags"
import { Layout } from "components/layout"

const RESOURCE_TYPES = ["taxonomy_term--tags"]

interface TaxonomyTermPageProps {
  resource: DrupalTaxonomyTerm
}

export default function TaxonomyTermPage({ resource }: TaxonomyTermPageProps) {
  if (!resource) return null
  return (
    <Layout>
      <Head>
        <title>{resource.title}</title>
        <meta name="description" content="A Next.js site powered by Drupal." />
      </Head>
      {resource.type === "taxonomy_term--tags" && <TaxonomyTermTags term={resource} />}
    </Layout>
  )
}

export async function getStaticPaths(context): Promise<GetStaticPathsResult> {
  return {
    paths: await drupal.getStaticPathsFromContext(RESOURCE_TYPES, context),
    fallback: true,
  }
}

export async function getStaticProps(context): Promise<GetStaticPropsResult<TaxonomyTermPageProps>> {
  const path = await drupal.translatePathFromContext(context)

  if (!path) {
    return {
      notFound: true,
    }
  }

  const type = path.jsonapi.resourceName
  let params = {};
  if (type === "taxonomy_term--tags") {
    params = {
      include: "name,path",
    }
  }

  const resource = await drupal.getResourceFromContext<DrupalTaxonomyTerm>(path,
    context,
    {
      params,
    }
  )

  // At this point, we know the path exists and it points to a resource.
  // If we receive an error, it means something went wrong on Drupal.
  // We throw an error to tell revalidation to skip this for now.
  // Revalidation can try again on next request.
  if (!resource) {
    throw new Error(`Failed to fetch resource: ${path.jsonapi.individual}`)
  }

  // If we're not in preview mode and the resource is not published,
  // Return page not found.
  if (!context.preview && resource?.status === false) {
    return {
      notFound: true,
    }
  }

  return {
    props: {
      resource,
    },
  }
}

And also a new component called taxonomy_term--tags.tsx with the following code:
import { DrupalTaxonomyTerm } from "next-drupal"

interface TaxonomyTermTagsProps {
  term: DrupalTaxonomyTerm
}

export function TaxonomyTermTags({ term, ...props }: TaxonomyTermTagsProps) {
  return (
    <article {...props}>
      <h1>{term.name}</h1>
      {/* {console.log(term)} */}
    </article>
  )
}

I'd expect to visit http://localhost:3000/tags/xyz and find the title of the term, but it is not showing anything. Can you help me? I'm not finding any guide to how show different entities rather than nodes on the documentation.

This is correct. The following code is fetching the term entity only.
const resource = await drupal.getResourceFromContext<DrupalTaxonomyTerm>(path,
  context,
  {
    params,
  }
)

If you want to fetch articles tagged by this term you need to use something like this:
const nodes = await drupal.getResourceCollectionFromContext(
  "node--article",
  context,
  {
    params: {
      "filter[field_tags.id]": resource.id,
    },
  }
)

This will fetch all articles tagged with the term. Which you can then pass as props to render.

Closing this issue as a viable solution is provided.


icelandic update

I just wanted to merge, but you commit is from an author: Your Name<EMAIL_ADDRESS>Can you please fix the ownership?

@chrisbra

thanks!


Add pIC50 uncertainties in schema objects

Updated the CDD parsing functions to calculate and store the variance in the pIC50 measurement.

Also, I'm not sure if you're storing the semiquantitative data yet, but if you are, then you can set the standard error to a default value (e.g. 0.5 pIC50 units) if there is no confidence interval defined.

Also, I'm not sure if you're storing the semiquantitative data yet, but if you are, then you can set the standard error to a default value (e.g. 0.5 pIC50 units) if there is no confidence interval defined.

Should this be handled when parsing the data or should the value be left up to the user at train/test time? Currently I'm storing NaNs where there is no CI info, but I can change that to 0.5 or whatever value we want. I am storing semiquantitative data: https://github.com/choderalab/covid-moonshot-ml/blob/add-uncertainties/covid_moonshot_ml/datasets/utils.py#L216-L217
so this can be used as well

Codecov Report

Merging #11 (4dcb1a0) into master (a0d0c7e) will decrease coverage by 0.72%.
The diff coverage is 9.09%.

Should this be handled when parsing the data or should the value be left up to the user at train/test time?
Currently I'm storing NaNs where there is no CI info, but I can change that to 0.5 or whatever value we want.

Good question!
For preparing CSV files for redistribution, it would be best to pre-populate the uncertainty with a sensible value that reflects an appropriate uncertainty for the limits here, which may be tighter than 0.5 pIC50 units.
For use in your ML models, it may makes the most sense to enable you to interpret the NaNs and experiment with different uncertainties for the semiquantiative data.

@jchodera I've updated the code to now fill the standard error for any compounds outside the dynamic range of the fluorescence assay with the mean of the compounds that are within the dynamic range. The compounds still keep track of whether the pIC50 value was above, within, or below the dynamic range, so it's trivial to update the default value.

That sounds like a decent approach! It looks like you just need to rebase or merge in main and then we can merge!

Closing this PR in order to avoid unintended side effects when removing the data from git and github history.


Samsung Purifier compatibility

HI! Great card!
I have a samsung purifier that i have connected to HA via Smartthings...
This my code:
type: custom:purifier-card
entity: switch.depuratore_aria
aqi:
  - entity_id: sensor.depuratore_aria_air_quality
    unit: 'AQI'
stats:
  - entity_id: sensor.depuratore_aria_dust_level
    unit: 'μg/m^3'
    subtitle: PM10
  - entity_id: sensor.depuratore_aria_fine_dust_level
    unit: 'μg/m^3'
    subtitle: PM2.5
  - entity_id: sensor.depuratore_aria_odor_sensor
    unit: ' '
    subtitle: GAS`

The only issue is that i can display the AQI value on the animation... Any idea?
Thanks a lot!

You have a typo in your config. This part:
aqi:
  - entity_id: sensor.depuratore_aria_air_quality
    unit: 'AQI'

should look like this:
aqi:
  entity_id: sensor.depuratore_aria_air_quality
  unit: 'AQI'


Allow creation and verification for a specified time

Unit testing classes with expiring tokens with this library is difficult because it is tied to System.currentTimeMillis() in multiple places. I use JodaTime, which has a number of features which make unit testing easier, including freezing the joda clock at a specified time. I would like to inject the time for iat/exp on creation, and for verifying exp, such that it can be made to work with this Joda functionality.
I am already setting iat/exp manually to leverage this feature, but it doesn't seem possible to do so for verification at this time.

@chadjaros - do you have a PR?

I do not. I set up a workaround in my unit tests because that was the path of lesser resistance.

@chadjaros the path of lesser resistance is to make clock public and figure out how to inject it and avoid people messing with this since it's meant only for testing.  So if you are able please submit a PR for allowing to define a clock


IsValidPhone  number is incorrect return  for<PHONE_NUMBER>

Hi
i am windows phone developer when i am using this number with IN locale
but this api is giving wrong autntication please help this is my code
public static bool ValidatePhoneNumber(string strphonenumber, string locale)
{
bool isValid = false;
try
{
PhoneNumberUtil phoneUtil = PhoneNumberUtil.GetInstance();
PhoneNumber swissNumberProto = phoneUtil.Parse(strphonenumber, locale);
isValid = phoneUtil.IsValidNumber(swissNumberProto);
}
catch (Exception ex)
{
}
return isValid;
}
but is retun false

@ashishgupta-kiwi ,
Thank you for using libphonenumber!
Please upgrade to the latest release, this is not a live issue. To check the latest behavior for yourself, see the demo (instructions in our contribution guidelines).
Closing this issue as library is working as intended.
https://libphonenumber.appspot.com/phonenumberparser?number=7042094906&country=IN

Hi
I am using this library from nuget. I think this is latest library but this library function isValidNumber return false for locale "IN" and phonenumber<PHONE_NUMBER>.
Please help.
Thanks
Ashish Kumar Gupta

@ashishgupta-kiwi
We do not maintain the csharp port of libphonenumber.
You can contact actual owners of the library that you are using:
https://www.nuget.org/packages/libphonenumber-csharp.unofficial/


Safari 9 fails to load during Raven setup

Safari 9 fails to initialize (on http://openstreetmap.us/iD/master/) and throws the following console error:
TypeError: Attempting to change access mechanism for an unconfigurable property.
   defineProperty master:47
   (anonymous function) master:47

/cc @bhousel @1ec5

I don't think the canary ever helped. We can remove it.

removed!


missing GlobalFilter java to jni method

ios-gpuimage-plus is already has these method like follow
- (void)enableGlobalFilter :(const char*)config;
- (void)enableFaceBeautify :(BOOL)shouldDo; //FaceBeautify 与 GlobalFilter 属于同类模块， 不可同时开启
- (void)setGlobalFilterIntensity :(float)intensity;
- (float)globalFilterIntensity;
but android version has no methods (although JNI method existed)
I need these 'Global Filter' feature.
Because I want to exclusive affect over two adjusts.

Aha, thanks~


Loom support in crossbeam-deque

It would be very useful to test with Loom.

I would accept a PR that does this in a similar way as https://github.com/crossbeam-rs/crossbeam/pull/487.


Dependency updates

Another dep update rollup, also bumps our sub-dependencies in yarn.lock as that's not done automatically when individual packages are upgraded.

Coverage decreased (-2.3%) to 73.008% when pulling 19f19d334add45d52a12e85a05cdd051f4557f1e on kevinansfield:dep-updates into df3da411df4df2e9263f9981cdce64145843c291 on TryGhost:master.


Method resolveArguments requires parameter "name" that is never used

The method resolveArguments is defined as:
private function resolveArguments($name, array $argumentDefinitions)

However, in the class itself, the variable $name is never used?

Thank you, I'll have a look.

Fixed and merged #3

Awesome, thank you


Disable sampling for trace type in ApplicationInsights.config is not working

i disabled the sampling for certain types in the ApplicationInsights.config as following

i am not sure if this will work for sure after prod release.
so i released to QA enviroment and run the following query
union requests,dependencies,pageViews,browserTimings,exceptions,traces
| where timestamp > ago(21h)
| summarize RetainedPercentage = 100/avg(itemCount) by bin(timestamp, 1h), itemType
but i still seeing the following, request and trace types are still sampled.

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

ID: 2f4c906d-7abe-eab7-331a-120230344583
Version Independent ID: ab24ade5-5e4c-b430-c3f8-2a00073e9e81
Content: Telemetry sampling in Azure Application Insights - Azure Monitor
Content Source: articles/azure-monitor/app/sampling.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @AaronMaxwell
Microsoft Alias: aaronmax

@tlsun0921
It would be great if you could add a link to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!

yes thank you!
here is the link about the applicationInsights.config
https://learn.microsoft.com/en-us/azure/azure-monitor/app/sampling?tabs=net-core-new#adaptive-sampling
and following is where i got the query https://learn.microsoft.com/en-us/azure/azure-monitor/app/sampling?tabs=net-core-new#knowing-whether-sampling-is-in-operation
thanks~

@tlsun0921
Thanks for your feedback! We will investigate and update as appropriate.

@tlsun0921 Can you please follow the steps of turning off adaptive sampling and then customize it ? Here is the link for the same.

thank you @SwathiDhanwada-MSFT !
just quick question before i get into the code change, do you think the way that i just changed the ApplicationInsights.config is not wokring? thanks~

thank you @SwathiDhanwada-MSFT for the reply.
just confirm, i was following the highlighted config to exclude certain types from sampling

do you mean i should turn off the sampling first, by the following, then turn it back on to customuze?
"To switch off adaptive sampling, remove the AdaptiveSamplingTelemetryProcessor node(s) from ApplicationInsights.config."

i did turned the % as 100, then completely removed the applicationInsights.config file, but the sampling is still happening.


[BUILD] make-distribution should find JAVA_HOME for non-RHEL systems

What changes were proposed in this pull request?
make-distribution.sh should find JAVA_HOME for Ubuntu, Mac and other non-RHEL systems
How was this patch tested?
Manually

Test build #70437 has started for PR 16363 at commit b05d768.

Test build #70437 has finished for PR 16363 at commit b05d768.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70437/
Test PASSed.

Merged build finished. Test PASSed.

merged to master, thanks!


Static builds for frontend/

We want auth to work wel in this setup as well, so worth setting up now.

Done here
https://github.com/opensource-observer/oso/pull/772


Add a loading screen

Context and Problem Statement
Load times are too long.
Describe the proposed option and how it helps to overcome the problem or limitation
This is a proposal to mask the loading time, but it doesn't solve it.
Describe how your proposal will work, with code, pseudo-code, mock-ups, or diagrams

Join a local room on launch
Keep load times low
When loading a big server, load on the background until loaded
Suggestion: Use a tunnel visualization from common scifi

Positive Consequences

Able to mask loading times
Able to direct players to the positive parts of v-sekai

Negative Consequences

coordinating cats

Option graveyard

Do nothing
Currently bad because players join and are lost.

If this enhancement will not be used often, can it be worked around with a few lines of script?
Not script.
Is there a reason why this should be core and done by us?
We maintain the core.
References

V-Sekai

License of the contribution
Copyright (c) 2022 V-Sekai contributors.
Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions:
The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.
 IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

Kosyne — Today at 17:24
interesting. High fidelity/vircadia also does similar to that second method mentioned, where it loads on the fly
Malek
OP
— Today at 17:24
It makes for a far less frictionless experince
For style points we could enforce portals that seamlessly transport you between different worlds, at least, as the default option, you place a portal, it has a camera, and you can walk through it, and after a certain point it dissapeares
But again, resource intensive issues, so will have to be optional
iFire (he, him) — Today at 17:29
I like option 4: " stick around in the world until enough of you is loaded in, then you are transported and get to watch as everything else loads in"
Malek
OP
— Today at 17:29
I also like that option

Malek listed some options:

Leading peer style Loading Screen, you click go, you are put in front of a loading screen until you are loaded in completely.

When you click go, you are put in front of a loading screen until enough of the world you are going to is loaded in, and then you are put in that world, and experience the rest of the worlds assets and players loading in over time.

You click go and are left in your current world while loading in the other world, once the other world is completely loaded in you are transported to it instantaneously.

The drawback to the multiworld load is that it is more resource intensive, though it should only last a short while, so if we do go that route it should be an option to use a loading screen instead.

A loading screen for when the application first starts up does make sense. However, a loading screen for switching worlds is not technically necessary. While this is the approach taken by VRChat it is not the approach taken by NeosVR. In VRChat when you click to go to a new world, you are taken to a loading screen until ur world is downloaded and u are loaded into it. In NeosVR you are left in your original world while also loading into the second world, and once enough of the second world is loaded that you can be there, then it switches the camera and such over, and also does something interesting where it doesn't wait until it's completely loaded, but lets assets, such as models, textures, and players, load in continuously while you are there. So, by remixing here our what I think our options are.

When you click go, you are put in front of a loading screen until enough of the world you are going to is loaded in, and then you are put in that world, and experience the rest of the worlds assets and players loading in over time.
You click go and are left in your current world while loading in the other world, once the other world is completely loaded in you are transported to it instantaneously.

The drawback to the multiworld load is that it is more resource intensive, though it should only last a short while, so if we do go that route it should be an option to use a loading screen instead.

It makes for a far less frictionless experince
For style points we could enforce portals that seamlessly transport you between different worlds, at least, as the default option, you place a portal, it has a camera, and you can walk through it, and after a certain point it dissapeares
But again, resource intensive issues, so will have to be optional
For option Number 3, we could even add a little button you click once you are ready to be transported, so it doesn't happen against ur will if ur trying to say something last minute. I like the idea of using a toaster being done noise as a signal for "you're ready to go to a new world"

Updated the post with the discussion.

Added as https://github.com/V-Sekai/V-Sekai.github.io/commit/591c86d6d86bd539072eedc1c2edef01a48462bd


Could it be the numerical values for day of week are off by one?

Hi,
I use job_scheduler https://docs.rs/job_scheduler/latest/job_scheduler/ which apparently pulls cron 0.6.1 as dependency.
I noticed that this expression
0 0 17 * * Mon-Sat
is not
0 0 17 * * 1-6
Which according to crontab format however, it should be. The latter expression would fire on Sundays till Friday.
Are you one off in the numerical format for day-of-week?
Or were you one off back then with 0.6.1?
Should I ask job_scheduler to bump to newer version? I opened an issue with them https://github.com/lholden/job_scheduler/issues/29
Best Regards,
Thomas

are there plans to add some sort of chrontab support?

Please see this issue. This crate was coded against the Quartz scheduling engine's cron syntax specification. Unfortunately, there is no single definitive standard.
I'm open to supporting various standards via a builder/config API, but I don't have the bandwidth to work on this crate actively.


fix(index.d.ts): add enable and disable functions

:tada: This PR is included in version 1.106.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Check bucket series time range when decoding chunks from index

Signed-off-by: Ben Ye<EMAIL_ADDRESS>

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes
This idea behind this pr is to check the chunk time range when decoding the series from the index. This change can benefit a lot the skipChunk queries because it can immediately return if it finds a valid chunk instead of decoding all the chunks in this series.

Verification
Uses the benchmarks from https://github.com/thanos-io/thanos/pull/3366. Benchmark and compare between this pr and current master.

SkipChunk query.

It is expected that this type of query should be much faster than before.
➜  thanos git:(improve-index-valid) ✗ benchcmp old new9
benchcmp is deprecated in favor of benchstat: https://pkg.go.dev/golang.org/x/perf/cmd/benchstat
benchmark                                                                           old ns/op      new ns/op      delta
BenchmarkBucketSeriesSkipChunks/1000000SeriesWith1Samples/1of1000000-8              374105769      304828968      -18.52%
BenchmarkBucketSeriesSkipChunks/1000000SeriesWith1Samples/10of1000000-8             366452232      287316200      -21.60%
BenchmarkBucketSeriesSkipChunks/1000000SeriesWith1Samples/1000000of1000000-8<PHONE_NUMBER><PHONE_NUMBER>     -26.59%
BenchmarkBucketSeriesSkipChunks/100000SeriesWith100Samples/1of10000000-8            31472869       22883196       -27.29%
BenchmarkBucketSeriesSkipChunks/100000SeriesWith100Samples/100of10000000-8          31160010       23195988       -25.56%
BenchmarkBucketSeriesSkipChunks/100000SeriesWith100Samples/10000000of10000000-8     134128144      108503863      -19.10%
BenchmarkBucketSeriesSkipChunks/1SeriesWith10000000Samples/1of10000000-8            2564966        99784          -96.11%
BenchmarkBucketSeriesSkipChunks/1SeriesWith10000000Samples/100of10000000-8          2799881        125497         -95.52%
BenchmarkBucketSeriesSkipChunks/1SeriesWith10000000Samples/10000000of10000000-8     29997992       373188         -98.76%

Normal series query

The expected performance of this should be almost the same or a little bit slower than before. However, this pr is about 10% - 20% slower when there are a lot of samples in one series. Not sure how to improve this case. I think there is no extra overhead except we are using more if-else statements than before when decoding the index.
➜  thanos git:(improve-index-valid) ✗ benchcmp b-old c-13
benchcmp is deprecated in favor of benchstat: https://pkg.go.dev/golang.org/x/perf/cmd/benchstat
benchmark                                                                 old ns/op      new ns/op      delta
BenchmarkBucketSeries/1000000SeriesWith1Samples/1of1000000-8              317048216      274136169      -13.53%
BenchmarkBucketSeries/1000000SeriesWith1Samples/10of1000000-8             316798190      278900447      -11.96%
BenchmarkBucketSeries/1000000SeriesWith1Samples/1000000of1000000-8<PHONE_NUMBER><PHONE_NUMBER>     -1.01%
BenchmarkBucketSeries/100000SeriesWith100Samples/1of10000000-8            25975122       23109618       -11.03%
BenchmarkBucketSeries/100000SeriesWith100Samples/100of10000000-8          25753028       23400939       -9.13%
BenchmarkBucketSeries/100000SeriesWith100Samples/10000000of10000000-8     138522971      137316413      -0.87%
BenchmarkBucketSeries/1SeriesWith10000000Samples/1of10000000-8            2516395        129818         -94.84%
BenchmarkBucketSeries/1SeriesWith10000000Samples/100of10000000-8          2500083        152185         -93.91%
BenchmarkBucketSeries/1SeriesWith10000000Samples/10000000of10000000-8     33321908       36184470       +8.59%

Running benchmarks is tricky but there are ways to make it reliable and more trustworthy.

If you run Goland ID you have benchtime only 1s, so this one thing to improve is to increase this time. You can to that by edit config and setting extra program -test.benchtime=10s
The other must have flag is to either set benchmem or specify in the code b.ReportAllocs()
Useful thing is to limit to 1 or 2 CPUs, because that's the usual value we give for Store GW.

Anyway good work I think it makes sense, wonder if we can clean decodeWithReq a bit more :hugs:
Some reference for benchmarks: https://gist.github.com/bwplotka/3b853c31ed11e77c975b9df45d105d74

I finished the benchmarks and the new results between this pr and current master are:

SkipChunks

name                                                                  old time/op    new time/op    delta
BucketSeriesSkipChunks/1000000SeriesWith1Samples/1of1000000              423ms ± 0%     442ms ± 0%   +4.32%
BucketSeriesSkipChunks/1000000SeriesWith1Samples/10of1000000             496ms ± 0%     458ms ± 0%   -7.56%
BucketSeriesSkipChunks/1000000SeriesWith1Samples/1000000of1000000        3.39s ± 0%     3.31s ± 0%   -2.29%
BucketSeriesSkipChunks/100000SeriesWith100Samples/1of10000000           38.6ms ± 0%    36.6ms ± 0%   -5.18%
BucketSeriesSkipChunks/100000SeriesWith100Samples/100of10000000         36.1ms ± 0%    42.3ms ± 0%  +17.31%
BucketSeriesSkipChunks/100000SeriesWith100Samples/10000000of10000000     276ms ± 0%     278ms ± 0%   +0.76%
BucketSeriesSkipChunks/1SeriesWith10000000Samples/1of10000000           4.53ms ± 0%    0.13ms ± 0%  -97.06%
BucketSeriesSkipChunks/1SeriesWith10000000Samples/100of10000000         4.13ms ± 0%    0.13ms ± 0%  -96.79%
BucketSeriesSkipChunks/1SeriesWith10000000Samples/10000000of10000000    16.5ms ± 0%     0.4ms ± 0%  -97.48%

name                                                                  old alloc/op   new alloc/op   delta
BucketSeriesSkipChunks/1000000SeriesWith1Samples/1of1000000             98.2MB ± 0%    98.2MB ± 0%   -0.00%
BucketSeriesSkipChunks/1000000SeriesWith1Samples/10of1000000            98.2MB ± 0%    98.2MB ± 0%   -0.00%
BucketSeriesSkipChunks/1000000SeriesWith1Samples/1000000of1000000       1.00GB ± 0%    1.00GB ± 0%   -0.00%
BucketSeriesSkipChunks/100000SeriesWith100Samples/1of10000000           8.43MB ± 0%    8.43MB ± 0%   +0.00%
BucketSeriesSkipChunks/100000SeriesWith100Samples/100of10000000         8.43MB ± 0%    8.43MB ± 0%   -0.00%
BucketSeriesSkipChunks/100000SeriesWith100Samples/10000000of10000000    95.9MB ± 0%    95.9MB ± 0%   -0.00%
BucketSeriesSkipChunks/1SeriesWith10000000Samples/1of10000000           4.34MB ± 0%    0.17MB ± 0%  -96.00%
BucketSeriesSkipChunks/1SeriesWith10000000Samples/100of10000000         4.34MB ± 0%    0.17MB ± 0%  -96.00%
BucketSeriesSkipChunks/1SeriesWith10000000Samples/10000000of10000000    17.3MB ± 0%     0.7MB ± 0%  -96.06%

name                                                                  old allocs/op  new allocs/op  delta
BucketSeriesSkipChunks/1000000SeriesWith1Samples/1of1000000               760k ± 0%      760k ± 0%   -0.00%
BucketSeriesSkipChunks/1000000SeriesWith1Samples/10of1000000              760k ± 0%      760k ± 0%   -0.00%
BucketSeriesSkipChunks/1000000SeriesWith1Samples/1000000of1000000        8.04M ± 0%     8.04M ± 0%   -0.00%
BucketSeriesSkipChunks/100000SeriesWith100Samples/1of10000000            76.1k ± 0%     76.1k ± 0%    0.00%
BucketSeriesSkipChunks/100000SeriesWith100Samples/100of10000000          76.1k ± 0%     76.1k ± 0%   -0.00%
BucketSeriesSkipChunks/100000SeriesWith100Samples/10000000of10000000      804k ± 0%      804k ± 0%   -0.00%
BucketSeriesSkipChunks/1SeriesWith10000000Samples/1of10000000              195 ± 0%       172 ± 0%  -11.79%
BucketSeriesSkipChunks/1SeriesWith10000000Samples/100of10000000            195 ± 0%       172 ± 0%  -11.79%
BucketSeriesSkipChunks/1SeriesWith10000000Samples/10000000of10000000       543 ± 0%       445 ± 0%  -18.05%

Normal query

name                                                        old time/op    new time/op    delta
BucketSeries/1000000SeriesWith1Samples/1of1000000              451ms ± 0%     432ms ± 0%   -4.07%
BucketSeries/1000000SeriesWith1Samples/10of1000000             450ms ± 0%     450ms ± 0%   -0.09%
BucketSeries/1000000SeriesWith1Samples/1000000of1000000        3.82s ± 0%     4.27s ± 0%  +11.66%
BucketSeries/100000SeriesWith100Samples/1of10000000           36.7ms ± 0%    36.2ms ± 0%   -1.34%
BucketSeries/100000SeriesWith100Samples/100of10000000         36.1ms ± 0%    38.0ms ± 0%   +5.44%
BucketSeries/100000SeriesWith100Samples/10000000of10000000     321ms ± 0%     359ms ± 0%  +12.04%
BucketSeries/1SeriesWith10000000Samples/1of10000000           5.07ms ± 0%    0.17ms ± 0%  -96.56%
BucketSeries/1SeriesWith10000000Samples/100of10000000         4.80ms ± 0%    0.17ms ± 0%  -96.51%
BucketSeries/1SeriesWith10000000Samples/10000000of10000000    77.6ms ± 0%    81.6ms ± 0%   +5.08%

name                                                        old alloc/op   new alloc/op   delta
BucketSeries/1000000SeriesWith1Samples/1of1000000              117MB ± 0%      98MB ± 0%  -16.35%
BucketSeries/1000000SeriesWith1Samples/10of1000000             117MB ± 0%      98MB ± 0%  -16.35%
BucketSeries/1000000SeriesWith1Samples/1000000of1000000       1.31GB ± 0%    1.31GB ± 0%   +0.18%
BucketSeries/100000SeriesWith100Samples/1of10000000           10.4MB ± 0%     8.5MB ± 0%  -18.46%
BucketSeries/100000SeriesWith100Samples/100of10000000         10.4MB ± 0%     8.4MB ± 0%  -18.55%
BucketSeries/100000SeriesWith100Samples/10000000of10000000     126MB ± 0%     126MB ± 0%   +0.38%
BucketSeries/1SeriesWith10000000Samples/1of10000000           5.90MB ± 0%    0.22MB ± 0%  -96.19%
BucketSeries/1SeriesWith10000000Samples/100of10000000         5.90MB ± 0%    0.22MB ± 0%  -96.19%
BucketSeries/1SeriesWith10000000Samples/10000000of10000000    38.0MB ± 0%    38.0MB ± 0%   -0.00%

name                                                        old allocs/op  new allocs/op  delta
BucketSeries/1000000SeriesWith1Samples/1of1000000              1.26M ± 0%     0.76M ± 0%  -39.69%
BucketSeries/1000000SeriesWith1Samples/10of1000000             1.26M ± 0%     0.76M ± 0%  -39.69%
BucketSeries/1000000SeriesWith1Samples/1000000of1000000        12.1M ± 0%     12.1M ± 0%   -0.00%
BucketSeries/100000SeriesWith100Samples/1of10000000             126k ± 0%       76k ± 0%  -39.65%
BucketSeries/100000SeriesWith100Samples/100of10000000           126k ± 0%       76k ± 0%  -39.64%
BucketSeries/100000SeriesWith100Samples/10000000of10000000     1.21M ± 0%     1.21M ± 0%   -0.00%
BucketSeries/1SeriesWith10000000Samples/1of10000000              232 ± 0%       205 ± 0%  -11.64%
BucketSeries/1SeriesWith10000000Samples/100of10000000            231 ± 0%       205 ± 0%  -11.26%
BucketSeries/1SeriesWith10000000Samples/10000000of10000000      170k ± 0%      170k ± 0%   -0.00%

After one more optimization, I think the performance is good now.


It is better to use the repository field in Cargo.toml

By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
To allow Crates.io , lib.rs and the Rust Digger to link to it. See the manifest for the explanation.

Thank you for your pull request @szabgab


Page refuses to fire JQuery, if not reloaded completely

Hi all!
Ok, so I have a little issue here. I have one particular page in my entire setup with a set of buttons. I have .on('click', function(){}) method hooked to these buttons. When I load the page for the first time, everything works as expected, but when I leave the page while staying on my website, thus not reloading the HTML completely, and then I come back, the script does not get triggered. No errors, nothing in the console log. Is it anything to do with barba or is it just me being stupid?
$('.button').on('click', function () {
        var id = $(this).data('id');
        $('[data-content]').hide();
        $('[data-text]').hide();
        $('[data-content="' + id + '"]').show()
        $('[data-text="' + id + '"]').show()
        let workTL = new TimelineMax();
        workTL.fromTo($('[data-content="' + id + '"]'), 1, {
            autoAlpha: 0,
            scale: 0.8
        }, {
            autoAlpha: 1,
            scale: 1,
            ease: Power4.easeOut
        })
    });

<button class="button" data-id="1">WORK</button>

<button class="button" data-id="2">WORK</button>

<button class="button" data-id="3">WORK</button>

<button class="button" data-id="4">WORK</button>

<button class="button" data-id="5">WORK</button>

Here is the code for the sake of understanding.

Ok now I see that this issue has been answered many times, but Could I please ask for a complete code on how it works? I just cannot get it right :(

You must use event propagation and on event.
Example:
$(document).on('click', 'button:not(.initialized)', function () {
    $(this).addClass('initialized');
    // your code
})

The :not(.initialized) selector is to avoid creating multiple assigning same callback multiple times. This works perfectly.


Adding support for env variables.

Description
Please include a summary of the change and link to an issue this PR fixes. Please also include relevant context. List any downstream components that might be affected by this change.
Fixes # (issue)
Type of change
Please check the options that are relevant.

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] Proposes a change (non-breaking change that isn't necessarily a bug)
[ ] Refactor
[ ] New feature (non-breaking change that adds a new functionality)
[ ] Breaking change (fix or feature that would break some existing functionality downstream)
[ ] This is a unit test
[ ] Documentation only change
[ ] Datasets Release
[ ] Models Release

Type of requested review

[ ] I want a thorough review of the implementation.
[ ] I want a high level review.
[ ] I want a deep design review.

Before and After
If this PR introduces a change or fixes a bug, please add before and after screenshots (if this is causes a UX change) or before and after examples(this could be plain text, videos etc ) to demonstrate the change.
If this PR introduces a new feature, please also add examples or screenshots demonstrating the feature.
Testing
Please describe the tests that you ran to verify your changes and how we can reproduce.
Checklist:

[ ] I have performed manual end-to-end testing of the feature in my environment.
[ ] I have added Docstrings and comments to the code.
[ ] I have made changes to existing documentation where needed.
[ ] I have added tests that show that the PR is functional.
[ ] New and existing unit tests pass locally with my changes.
[ ] I have added relevant collaborators to review the PR before merge.
[ ] [Polymetis only] I have verified that all scripts in tests/scripts runs on hardware.

Codecov Report

Merging #808 (c38872b) into main (6a358a1) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #808      +/-   ##
==========================================
- Coverage   33.11%   33.03%   -0.08%
==========================================
  Files         337      336       -1
  Lines       33060    32825     -235
==========================================
- Hits        10949    10845     -104
+ Misses      22111    21980     -131

Impacted Files
Coverage Δ

interpreter/craftassist/tasks.py
74.45% <0.00%> (-0.19%)
:arrow_down:

agents/droidlet_agent.py

agents/craftassist/tests/fake_agent.py
73.67% <0.00%> (+0.69%)
:arrow_up:

interpreter/robot/tests/world.py
89.05% <0.00%> (+0.72%)
:arrow_up:

Starting a new one, based on discussions with Leo. env will be made an argument of Process


[Insets] Setup migration for accompanist/insets

An initial PR beginning the process of deprecating accompanist/insets in favor of built-in androidx.compose.foundation support.
Most of the accompanist/insets APIs have been deprecated, with ReplaceWiths where possible. There are cases where ReplaceWith doesn't exactly match the new behavior and where complete auto-migration isn't possible (parameters became specific types), so this was a best-effort to help along the way.
The insets doc has been updated with a migration guide, along with a more complete table of migrations and an explanation of behavior differences.
There are some things that are not yet supported in androidx:

the IME animation controller has been left un-deprecated.
the components in insets-ui still exist, and I've added some variations that will help those work with the androidx.compose.foundation insets, and I am tracking upstreaming those if possible.

TODO:

[x] Depend on non-snapshot version of Compose

@bentrengrove and @simona-anomis this is ready for another look!

Run


[1452] update traits_for_enum to accept _prefix and _suffix

in response to Issue.
goal is to allow _prefix and _suffix for enum trait definitions to stay in line with ruby implementations
also will not define the standard enum (without prefix/suffix) will only define the ones with prefix/suffix

Hey jasonkarns.
not wanting to replicate enum config in factories and models makes sense. This implementation was chosen because the flip side of explicitly setting these values would require hooking into active record which currently does not store or make available enough information to build this out.
my reasoning for thinking it wouldn’t be that bad to use this is:

the factory only needs to define the enum and prefix/suffix. If you are inheriting the enum list from the model then it will continue to inherit. Change in model enums are usually due to adding values which would automatically be added to the factory’s trait list, it’s not often that the enum attribute name or prefix/suffix change
this will only be needed for enums with a prefix/suffix. Most enums don’t have these so will continue to be automatically added as traits to factories


maven LWRP always reports updated_by_last_action(true)

At line 72 of the provider code, you can see that the provider always reports it was updated, which can result in undesirable behavior (e.g., restart a service via a subscribes on an instance of the LWRP, which will restart the service on every Chef run).
You can see a great example with the cron_d cookbook in the Chef docs.
Wondering how this might be modified to improve this behavior, and be more idempotent (i.e., not report updated if nothing happened).

This should be fixed on master now


Interact with notifications

If you go to your notifications, you can't reply or go to the post this is about or anything. There should at least be a go to topic, if possible combined with #22

This is pretty much the use-case #22 is aimed at. Viewing a full discussion is most likely not what you want to view a single notification.

Fixed in 5a8118e2e60d619005355836362d4913117f1ff5 and cf5f89ae0625acda3b16ee15a94de3273e0488a1.


SQL Server Native Client 13.0 / SNAC 13. Does it exist?

I can find a "SQL Server Native Client 13.0" mentioned here
https://docs.microsoft.com/en-us/sql/relational-databases/native-client/applications/support-policies-for-sql-server-native-client?view=sql-server-ver15

SQL Server Native Client 13.0 supports connections to, SQL Server 2008, SQL Server 2008 R2, SQL Server 2012, SQL Server 2014, SQL Server 2016, SQL Server 2017, SQL Server 2019, and Azure SQL Database.

And also a few mentions of a "SNAC 13" here
https://techcommunity.microsoft.com/t5/sql-server-blog/snac-lifecycle-explained/ba-p/385381

Looking at the last 2 rows of the table, I think you might have meant to write this instead

The SQL Server Native Client 2012 (11.0) is going out of support on July 2022 and I'm looking for an upgrade path.
Is moving to the SQL Server Native Client 13.0 a possibility?
Does such version really exist?
I can't find any download link.

@AgostinoSturaro -- Agostino, thank you for your feedback and questions.  Please consider these resources:

SQL Server on Microsoft Q&A
DBA Stack Exchange
Stack Overflow

@markingmyname -- Mark, please look into this potential documentation issue.

@WilliamAntonRohm @markingmyname Any news?
The End of Support for the SQL Server Native Client is this month and we need a clear upgrade path.

@AgostinoSturaro
You are correct that the docs article and blog post incorrectly refer to a non-existent "SNAC 13". I've updated the blog post to reflect the correct driver names and submitted a change to remove the reference in the doc article.
You upgrade path is to move to the Microsoft ODBC Driver for SQL Server for applications requiring ODBC connectivity and the Microsoft OLE DB Driver for SQL Server for applications requiring OLE DB connectivity.
https://docs.microsoft.com/en-us/sql/connect/sql-connection-libraries?view=sql-server-ver16
Regards,
David

Great. Thank you!


AR not working anymore after 25th April release on iOS

Sorry this was an issue with all my usdz models. Its working. Closing.


Listing restaurants

Pull Request Template
Description
Listing restaurants by asking crucial questions like name of restaurant, address, working hours and working weekdays, contact details for restaurant, add images of restaurants, etc.
Related Issue
Fixes #113
Screenshots
https://github.com/sourabhkumar47/ResQFood/assets/132752204/a9243abf-a75f-40c8-a8de-d6970aa92001

Awsome!


UKCensusAPI for Scotland seems to lose data

https://github.com/alan-turing-institute/UKCensusAPI/blob/86c45eb096cf38cdf413c913dc67e1e8e9642c4a/ukcensusapi/NRScotland.py#L236 This line used in Step 1 of the HPC script returns an empty dataframe causing errors but still seemingly allowing further analysis to occur.

I can't tell if this is intention based on the region code swapping that happens. If so, why does the function not return earlier? Is there expected data here?

The "workaround" statement just above this is concerning and the reason for this. As it leads to the georgraphy's being missing and not aligning

Oh wait... it might be a typing issue of geography code!

The parameters passed to nomi in nomisweb.py getdata function is ignoring places!

So the reason this doesn't work is because
self.data_api_sc.get_geog(self.region, "LSOA11") is called to get a list of geography codes. Calling self.data_api_sc._NRScotland__get_rawdata("QS103SC","LSOA11") to get a list of the data results in columns which cannot be matched up! Still scratching head on this.


allow constructors to be called as Foo.ctor<String>

I’ve been somewhat bothered by the fact you instantiate generic classes through constructors as Foo<String>.ctor, but you access static members as Foo.member<String>.
Something occurred to me a couple months ago: we can generalize the behavior we have for static access of instance members to static member access, but, instead of doing so with value argument lists, we do it for type argument lists.
That is, consider the following classes:
class Foo<Argument>
{
	shared static String member = `Argument`.string;

	shared new ctor()
	{
	}
}

class Bar(String argument)
{
	shared String member = argument;
}

Since we can do Bar("string").member, and it means the same as Bar.member("string"), then we could say one can do Foo<String>.ctor and it would mean the same as Foo.ctor<String>.
It’s more difficult to do the opposite and have Foo.member<String> mean the same as Foo<String>.member, as Argument need to be in scope for member, and its value might change depending on the argument provided. Of course, it’s not impossible (as the Foo<String> in Foo<String>.member is not an expression), but it’s more unintuitive.
Further, we could generalize this behavior to regular instance members too. That is, consider the following class:
class Baz<ClassArgument>(String classArgument)
{
	shared String member<MemberArgument>(Integer memberArgument) => "hello!";
}

We could say that one could write all of the following and they would all mean the same thing:

Baz<String>("string").member<Integer>(123)
Baz<String>.member("string")<Integer>(123)
Baz.member<String>("string")<Integer>(123)

Of course, the grammar doesn’t allow things such as foo("string")<String> today, but the grammar could easily change.
Also, it’s interesting to note that this makes the inference for the type arguments of a constructor’s class feel more at home.

So, I figured I’m dumb. One can already write Foo<String>.member. I was thinking you can’t because the following is already possible:
shared void run()
{
    value member = Foo.member;
    value res = member<String>;
    print(res);
}

However, I still hold to my proposed idea on constructors.


Add "module" support

Bundlers are now leveraging the module property to support es6 improts/exports. Would a PR for adding this property be welcome?

Implications for the codebase are that it will need to allow users to import an esnext build which uses es6 modules. Having users import moment via es6 modules rather than commonjs that will allow apps such as webpack, rollup, parcel, etc to treeshake modules, producing smaller and more optimized builds for the user

@amilajack you mean the es6 import/export? So there needs to be a separate codebase (they can not coexist as with UMD in the past).
Also treeshaking moment is non-sense -- its one object with everything attached to it. Anyway, I though there was an ES5 compatible way to export ES6 modules, I guess I was dreaming a bit too much.


Create additional variables from the raw data (or) the ones created have very limited added value for further data analysis.

Add control variables, or variables that are not necessarily in the dataset but could be relevant. (can be done when the basis is good).


OCPBUGS-34227: feat: chunk size policy

/test all

/test all

Codecov Report
Attention: Patch coverage is 86.36364% with 6 lines in your changes missing coverage. Please review.

Project coverage is 60.72%. Comparing base (a0a2217) to head (2dd9e9a).
Report is 12 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##             main     #633       +/-   ##
===========================================
- Coverage   77.97%   60.72%   -17.26%
===========================================
  Files          28       54       +26
  Lines        2016     4448     +2432
===========================================
+ Hits         1572     2701     +1129
- Misses        335     1497     +1162
- Partials      109      250      +141

Files
Coverage Δ

api/v1alpha1/lvmcluster_types.go
100.00% <ø> (ø)

internal/controllers/vgmanager/controller.go
78.35% <100.00%> (+5.48%)
:arrow_up:

internal/controllers/vgmanager/lvm/lvm.go
99.03% <100.00%> (ø)

api/v1alpha1/lvmcluster_webhook.go
88.16% <96.15%> (+4.44%)
:arrow_up:

...ternal/controllers/vgmanager/lvm/mocks/mock_lvm.go
35.86% <44.44%> (ø)

... and 27 files with indirect coverage changes

/lgtm

/cherry-pick release-4.16


Example of versions.json

It might be a simple question but I can't find a example of the versions.json file. Could you add it so we know how it should be build?

Great question.

This project is not ready for use yet. When it is, it will be bumped to version 1.0.0 and published on npm. But thanks for the early interest
The versionUrl part of the API can be set to any URL you like, the version.json that is shown there is just an example. It can be any endpoint, even one that just returns plain text of the latest version number.
Whatever content is returned will then be passed into the confirmNewVersion and downloadPath functions you provide.

confirmNewVersion - You define whatever function you want here, I will hand it the response from the network request (from versionUrl) as well as the latestLocal which will be a string like 1.2.5 (whatever the current latest version of your app is that was previously downloaded). If it is the first time it is ran, then latestLocal will be undefined. You can then use whatever data your network response returned to decide if a new version is available and should be downloaded. If a new version is available return a string of the newer version number so we can create folders where downloads will be stored. If your function determines that the latest remote version is already on the local machine then it can return false to skip downloading/extracting a new version, and instead just launch the current local copy of the app.
downloadPath - You define whatever function you want here. This function will only be called if the confirmNewVersion function returned a string, instead of false. I will hand it the response from the network request (from versionUrl). Your logic can then either find the url to the latest ZIP in that response, or compose a url (if it is a predictable pattern), or even make it's own async network calls to get the details. Whatever you want! But ultimately, it needs to return a string of a url to a zip file to be downloaded/extracted.

The idea here is that everyone will have a different system and way of keeping track of things, so we want to give you as much control as possible.
I'll update the README later to try to make it more clear that it's just an example.

Thanks for the reply, I will wait for the release. Already looking forward to is, if you could make a demo like you did for the nw-splash you would do me a big favor. Keep up the good work!


Updated the NuGet package dependencies

Many of the dependencies now reference .NET Core 1.1.2 and 1.1.3 instead of 1.0.x.

This was not assigned work but it gives us the latest and greatest and consolidates the dependencies of our dependencies.

Travis CI build is failing because .travis.yml is not present in this branch. It was added in another pull request.


[Gregorian Calendar] Implement weekend-related functions

Calendar's weekend support requires the following functions:

isDateInWeekend: answers whether a given date is in weekend
weekendRange: function that returns the days and times of the weekend

This implements the first. The second one, weekend range, is in fact related to calendar's locale rather than calendar itself. Move it over to _LocaleProtocol and have locale handle it.

@swift-ci please test


Add a Gitter chat badge to README.md

moip/moip-sdk-ruby now has a Chat Room on Gitter
@rodrigosaito has just created a chat room. You can visit it here: https://gitter.im/moip/moip-sdk-ruby.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Thanks!


chore: Remove unused errors

Description
Removal of errors that aren't used or referenced outside of their definition.
Contributor Checklist

[ ]  All status checks have passed prior to code review
[ ]  I have added unit tests to a reasonable level of coverage where suitable
[ ]  I have added UI tests to new user flows, if applicable
[ ]  I have manually tested newly added UX
[ ]  I have open a documentation PR, if applicable

Reviewer Checklist

[ ]  I have verified that a suitable set of automated tests has been added
[ ]  I have verified that the title prefix aligns to the code changes + whether a release is expected after merging the PR
[ ]  I have verified the documentation PR aligns with this PR, if applicable

Before Merging

[ ]  If introducing a breaking change, I have communicated it internally
[ ]  Any related documentation PRs are ready to merge

Other Stuff

You can find out more about our automation checks here
Find out more about conventional commits here

LGTM - has failedToImport3DS been replaced with the more generic missingDependency error somewhere along the way?


Fix interpreter tests

For https://github.com/microsoft/vscode-jupyter/issues/7403

Codecov Report

Merging #7410 (734da8f) into main (c7b94ff) will decrease coverage by 1%.
The diff coverage is n/a.

@@          Coverage Diff           @@
##            main   #7410    +/-   ##
======================================
- Coverage     64%     63%    -2%
======================================
  Files        361     361
  Lines      22563   22563
  Branches    3416    3416
======================================
- Hits       14499   14218   -281
- Misses      6866    7135   +269
- Partials    1198    1210    +12

Impacted Files
Coverage Δ

src/client/common/refBool.ts
66% <0%> (-34%)
:arrow_down:

...atascience/interactive-window/interactiveWindow.ts
18% <0%> (-29%)
:arrow_down:

src/client/datascience/cellMatcher.ts
72% <0%> (-19%)
:arrow_down:

.../datascience/editor-integration/codeLensFactory.ts
56% <0%> (-14%)
:arrow_down:

...nt/datascience/editor-integration/hoverProvider.ts
38% <0%> (-14%)
:arrow_down:

...ce/interactive-window/interactiveWindowProvider.ts
60% <0%> (-13%)
:arrow_down:

src/client/datascience/jupyter/kernels/kernel.ts
62% <0%> (-12%)
:arrow_down:

...nt/datascience/notebook/helpers/notebookUpdater.ts
88% <0%> (-8%)
:arrow_down:

...lient/datascience/variablesView/notebookWatcher.ts
61% <0%> (-8%)
:arrow_down:

...client/datascience/commands/activeEditorContext.ts
84% <0%> (-7%)
:arrow_down:

... and 45 more


Enhanced typings for pickAll

Adding a better typing option when all the keys are known on the object

Closing. Will re-apply changes in new types repo as described here: https://github.com/ramda/ramda/discussions/3329


Storefront uses http to request to api (api uri with https) to load images

What I'm trying to achieve
…
To fetch all the data of a product (object)
Steps to reproduce the problem

host the saleor api on some different host with https (nginx)
use that url in .env file of saleor-front as next_public_api_uri
start the app, open a category , it doesn't show the images of products, in inspect it can be seen that request is being made to http instead of https while this is not the case with saleor-dashboard.

What I expected to happen
…
it should make https requests to media content also as it does for other get and post requests.
Screenshots

System information
Operating system:
Browser:

your nginx should set a https header,
set_header X_FORWARDED_PROTO https

and the django shoud set SECURE_PROXY_SSL_HEADER = ('HTTP_X_FORWARDED_PROTO', 'https')


Set default number of namespaces to 1

What type of PR is this?

/kind bug
What this PR does / why we need it:
Details: https://github.com/kubernetes/perf-tests/issues/887
Which issue(s) this PR fixes:

Fixes https://github.com/kubernetes/perf-tests/issues/887
Special notes for your reviewer:
The type conversion was rounding fractions to 0. Hence when .Nodes passed in is less than NODES_PER_NAMESPACE (defaults to 100), namespaces = DivideInt .Nodes  NODES_PER_NAMESPACE = 0. thereafter namespace is used as the denominator for several computations eg. https://github.com/kubernetes/perf-tests/blob/master/clusterloader2/testing/load/config.yaml#L7
Does this PR introduce a user-facing change?:

NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

/cc @mm4tt

/cc @mm4tt

/ok-to-test

/ok-to-test

@mm4tt Noticed a few other places where this fix might not work for small clusters eg. https://github.com/kubernetes/perf-tests/blob/master/clusterloader2/testing/load/config.yaml#L67 and https://github.com/kubernetes/perf-tests/blob/master/clusterloader2/testing/density/config.yaml#L17. These may have to be scaled down also - the goal is so that anyone can run load / density tests on a 1 node cluster?

@mm4tt Noticed a few other places where this fix might not work for small clusters eg. https://github.com/kubernetes/perf-tests/blob/master/clusterloader2/testing/load/config.yaml#L67 and https://github.com/kubernetes/perf-tests/blob/master/clusterloader2/testing/density/config.yaml#L17. These may have to be scaled down also - the goal is so that anyone can run load / density tests on a 1 node cluster?

Instead of trying to patch the load and density tests so that they will work on small clusters, wdyt about defining a separate, minimal test config that will run on a 1 node cluster?

Instead of trying to patch the load and density tests so that they will work on small clusters, wdyt about defining a separate, minimal test config that will run on a 1 node cluster?

I'd really like to make the load test to work at any scale. All the hard-coded values we have there are actually a technical debt and we should aim to get rid of them. A few notes:

Please disregard the density test, we're aiming to deprecate it soon
Is there anything else other than MIN_PODS_IN_SMALL_CLUSTERS that's blocking us?
For MIN_PODS_IN_SMALL_CLUSTERS, I believe we have some options:

Ideally, we'd just fix the https://github.com/kubernetes/perf-tests/issues/1027, so we don't need this constant. Unfortunately, that's not that simple, so let's think of something else
Instead of having the MIN_PODS_IN_SMALL_CLUSTERS as a constant, hard-coded in the config.yaml, it could be a test param that is set to 500 in the test-infra presets only for our 100+ node tests
Given that this part of the test doesn't really work if we have less than 100 nodes (if we have less then we're not able to accurately measure the scheduling throughput), we should probably conditionally disable this part if # nodes < 100.

I believe 3 is the best option currently. Thoughts?

I'd really like to make the load test to work at any scale. All the hard-coded values we have there are actually a technical debt and we should aim to get rid of them. A few notes:

Please disregard the density test, we're aiming to deprecate it soon
Is there anything else other than MIN_PODS_IN_SMALL_CLUSTERS that's blocking us?
For MIN_PODS_IN_SMALL_CLUSTERS, I believe we have some options:

Ideally, we'd just fix the https://github.com/kubernetes/perf-tests/issues/1027, so we don't need this constant. Unfortunately, that's not that simple, so let's think of something else
Instead of having the MIN_PODS_IN_SMALL_CLUSTERS as a constant, hard-coded in the config.yaml, it could be a test param that is set to 500 in the test-infra presets only for our 100+ node tests
Given that this part of the test doesn't really work if we have less than 100 nodes (if we have less then we're not able to accurately measure the scheduling throughput), we should probably conditionally disable this part if # nodes < 100.

I believe 3 is the best option currently. Thoughts?

Yes I think approach 3 sounds good. Will also need to handle BIG_GROUP_SIZE and MEDIUM_GROUP_SIZE in the same manner I suppose?

Yes I think approach 3 sounds good. Will also need to handle BIG_GROUP_SIZE and MEDIUM_GROUP_SIZE in the same manner I suppose?

Will also need to handle BIG_GROUP_SIZE and MEDIUM_GROUP_SIZE in the same manner I suppose?

Hmm, wouldn't it work if we just round down everywhere? E.g. if we have just 10 nodes, the total number of pods will be 300 (30*10). Then we'll calculate how many small/medium/big deployments we can fit and e.g. for big we'll do: 300 / 4 / 250 = 0 (assuming we round down). For medium it'll be 300/4/30 = 2. For small it'll be 300/2/5 = 30. Does it make sense?

Will also need to handle BIG_GROUP_SIZE and MEDIUM_GROUP_SIZE in the same manner I suppose?

Hmm, wouldn't it work if we just round down everywhere? E.g. if we have just 10 nodes, the total number of pods will be 300 (30*10). Then we'll calculate how many small/medium/big deployments we can fit and e.g. for big we'll do: 300 / 4 / 250 = 0 (assuming we round down). For medium it'll be 300/4/30 = 2. For small it'll be 300/2/5 = 30. Does it make sense?

Will also need to handle BIG_GROUP_SIZE and MEDIUM_GROUP_SIZE in the same manner I suppose?

Hmm, wouldn't it work if we just round down everywhere? E.g. if we have just 10 nodes, the total number of pods will be 300 (30*10). Then we'll calculate how many small/medium/big deployments we can fit and e.g. for big we'll do: 300 / 4 / 250 = 0 (assuming we round down). For medium it'll be 300/4/30 = 2. For small it'll be 300/2/5 = 30. Does it make sense?

But it's also used in number of replicas https://github.com/kubernetes/perf-tests/blob/master/clusterloader2/testing/load/config.yaml#257. Seems like BIG_GROUP_SIZE, MEDIUM_GROUP_SIZE needs to be a factor of number of nodes?

Will also need to handle BIG_GROUP_SIZE and MEDIUM_GROUP_SIZE in the same manner I suppose?

Hmm, wouldn't it work if we just round down everywhere? E.g. if we have just 10 nodes, the total number of pods will be 300 (30*10). Then we'll calculate how many small/medium/big deployments we can fit and e.g. for big we'll do: 300 / 4 / 250 = 0 (assuming we round down). For medium it'll be 300/4/30 = 2. For small it'll be 300/2/5 = 30. Does it make sense?

But it's also used in number of replicas https://github.com/kubernetes/perf-tests/blob/master/clusterloader2/testing/load/config.yaml#257. Seems like BIG_GROUP_SIZE, MEDIUM_GROUP_SIZE needs to be a factor of number of nodes?

It will be fine if we have 250 there even if there is no place for so many pods. This is because this object will be never created. The replicasPerNamespace will be 0 - https://github.com/kubernetes/perf-tests/blob/master/clusterloader2/testing/load/config.yaml#L244

It will be fine if we have 250 there even if there is no place for so many pods. This is because this object will be never created. The replicasPerNamespace will be 0 - https://github.com/kubernetes/perf-tests/blob/master/clusterloader2/testing/load/config.yaml#L244

Ah yes. I'm curious if that will all fit on 1 node 😅

Ah yes. I'm curious if that will all fit on 1 node 😅

Doing a local run, noting down some issues:

[ ] Creates ~1000 namespaces due to https://github.com/kubernetes/perf-tests/blob/master/clusterloader2/testing/load/config.yaml#L94

Doing a local run, noting down some issues:

[ ] Creates ~1000 namespaces due to https://github.com/kubernetes/perf-tests/blob/master/clusterloader2/testing/load/config.yaml#L94

@mm4tt Added defaults and disabled heavier workloads when running on clusters < 100 nodes (I tested with a single node cluster)

@mm4tt Added defaults and disabled heavier workloads when running on clusters < 100 nodes (I tested with a single node cluster)

Other than the timeLimit one, I've addressed the other comments!

LGTM with one comment

Cool :) will squash commits

/lgtm
/approve


Fix Table of Contents (main.swift)

Current Table of Contents has broken links, namely Other and Other development.
Project URL
https://github.com/serhii-londar/open-source-mac-os-apps
Category
N/A
Description
ToC have two 'Other', and the second one is pointing at #other-1, which doesn't exist. Moreover, category Other development can't be found in ToC.
This change fixes both issues by adding Other development and modifying #other-1 to #other.
Why it should be included to Awesome macOS open source applications  (optional)
N/A
Checklist

[ ] Edit applications.json instead of README.md.
[x] Only one project/change is in this pull request
[ ] Screenshots(s) added if any
[x] Has a commit from less than 2 years ago
[ ] Has a clear README in English

@kcy1019 Thank you! 👍


Bug on displaying of layers and districts

Description
The layers and districts present are not displayed and even those newly created.
Steps to Reproduce

take a study whom have some layers and district.
Go to the map and go see the layers and districts

You can also follow this steps to reproduce

Create a new study
add few areas and links
create a layer with two areas
go back to the map
and try to see the layer create previously

Screenshoots

Additional Information
This issue is on all the versions of AntaresWeb (desktop, server).

Le bug est présent en recette et en prod.

C'est fix, mais le correctif n'a pas encore été déployer visiblement


EXCEPTION_ACCESS_VIOLATION

Hi bsavery, I have several bugs when I use your plugin, this is the first I've found, in debug mode.
Renderman 20.10
C:\Users\pafurijaz\Desktop\blender-2.77.0-git.f2fa737-AMD64\2.77\scripts\startup\bl_ui\space_node.py:427 bpy.ops.shading.add_renderman_nodetree(idtype="material") WM_operator_last_properties_store: storing properties for 'SHADING_OT_add_renderman_nodetree' A00003 {WARNING} Invalid initialization state. Error: EXCEPTION_ACCESS_VIOLATION

Please post a scene or steps to reproduce this?

This happens, as soon as I select the car, and then I add a node for the material.
http://www.pasteall.org/blend/42815

Hmmm for whatever reason that url doesn't work.    What are your addon pref settings?
Brian
On Thu, Jul 21, 2016 at 3:46 PM, pafurijaz<EMAIL_ADDRESS>wrote:

This happens, as soon as I select the car, and then I add a node for the
material.
http://www.pasteall.org/blend/42815 http://url
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bsavery/PRMan-for-Blender/issues/311#issuecomment-234406906,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABzffCWvx30taBraj7ahrGWfHK01Vf4rks5qX_a4gaJpZM4JSSR6
.

--
<EMAIL_ADDRESS>508-274-8700

OK I installed everything ProServer and also Renderman for Maya
This is my setting
http://www.pasteall.org/pic/show.php?id=105279

With the official version of Blender I get this in debug mode.
C:\Program Files\Blender Foundation\Blender\2.77\scripts\startup\bl_ui\space_node.py:427 DAG_id_tag_update_ex: id=NTGlass.001 flag=0 DAG_id_tag_update_ex: id=NTGlass.001 flag=0 DAG_id_tag_update_ex: id=NTGlass.001 flag=0 DAG_id_tag_update_ex: id=NTGlass.001 flag=0 bpy.ops.shading.add_renderman_nodetree(idtype="material") WM_operator_last_properties_store: storing properties for 'SHADING_OT_add_renderman_nodetree' A00003 {WARNING} Invalid initialization state. Error: EXCEPTION_ACCESS_VIOLATION

official version, debug mode.  same file other steps
info I have node mat for BI and for Cycles
C:\Program Files\Blender Foundation\Blender\2.77\scripts\startup\bl_ui\space_node.py:427 ERROR: Python context internal state bug. this should not happen! ERROR: Python context internal state bug. this should not happen! ERROR: Python context internal state bug. this should not happen! ERROR: Python context internal state bug. this should not happen! ERROR: Python context internal state bug. this should not happen! ERROR: Python context internal state bug. this should not happen! ERROR: Python context internal state bug. this should not happen! ERROR: Python context internal state bug. this should not happen! ERROR: Python context internal state bug. this should not happen! ERROR: Python context internal state bug. this should not happen! ERROR: Python context internal state bug. this should not happen! ERROR: Python context internal state bug. this should not happen! ERROR: Python context internal state bug. this should not happen! ERROR: Python context internal state bug. this should not happen! ERROR: Python context internal state bug. this should not happen! A00003 {WARNING} Invalid initialization state. Error: EXCEPTION_ACCESS_VIOLATION

Having nodes for all render engines in the same file can cause issues.  It's best to have one file per render engine.  Also, for what it's worth, I've had issues when I tried to run the Maya 2016.5 stuff.  I ended up using the plain vanilla Maya 2016.

Brian, I did a test with Ubuntu 16. 04 and I have no problems with any files, as happens with Windows Me 10, as soon as I can lead this operating system,  I need It for CAD application, but I'm changing my workflow

I don't think that any more information on this has been reported and it was from a 20.x version of RenderMan. I am going to close this unless there are any objections.


[TC-187] fix HTTPs bugs

use ds id as key to store ssl certificate in riak server;
update ssl certificate in keystore if delivery service modified xml-id or subdomain;
repose decoded certificate in restful api.

@zhilhuan can you please rebase this PR?
This PR also seems to fix an issue we'll have once per-DeliveryService routing names are implemented soon. Basically, if a DNS DS starts using the DNS routing name "my-edge" rather than the default "edge", the new hostname will mismatch the hostname used in the SSL cert. Your PR seems to address that issue, which is helpful in addition to the issue of changing the xml_id.

Can one of the admins verify this patch?

@rawlinp rebased.
Yes, we found several SSL cert issues related to update delivery service xml_id or sub-domain. And fixed those issue in this PR.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/26/


700 add to example apps how to apply custom styling using our design tokens

By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---
CONTRIBUTING.md also tells you what to expect in the PR process.
Description
Include tokens project usage in the example apps for showcasing

📦 Published PR as canary version: 20.35.0--canary.711.b63bf19046d977618b19e7b38b48fa5d863b07c7.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

I've reviewed it, and it's good for merging.
Just resolve the merge conflicts, and merge it.


Check for cuBLAS handle creation

Check for cuBLAS handle creation

We shouldn't have NPE thrown away: https://github.com/deeplearning4j/deeplearning4j/issues/1943


Fix test for intersections between coincident geodesics

The test for coincident geodesics in calculate_aux_intersection_distances does not find all coincident geodesics.
For example, if a long geodesic crossing the equator (e.g. LatLong(1.0, 0.0) to LatLong(-0.998286322222,<PHONE_NUMBER>91667)) is split in half, the two halves should be processed as coincident, but  calculate_aux_intersection_distances only returns the correct result when the geodesics start at the same point.

Added test test_intersection_same_geodesic_split as proposed above and implemented a version of C. F. F. Karney's solution from "Geodesic intersections section 7 Coincident Geodesics" by calculating a new geodesic between the geodesic start points and testing the cross products of the poles at the start and end points of the new geodesic.


Parameter & Pipeline variables naming rules

This document is missing parameter and pipeline variable naming rules.

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

ID: 65f81ce0-8915-2390-2c8e-53eb8a93180b
Version Independent ID: 49fb5956-019b-0f5a-ebff-1ae19fe87c80
Content: Rules for naming Azure Data Factory entities - Azure Data Factory
Content Source: articles/data-factory/naming-rules.md
Service: data-factory
GitHub Login: @dcstwh
Microsoft Alias: weetok

@kvijaianand - Thanks for your valuable feedback. I am moving it to content author to review.
@dcstwh - Can you please take a look?

@minhe-msft can you help to update the doc for this and include the naming rules for parameters and pipelines? Thank you.

Fixed by adding parameter and variable naming rules.
#please-close


Add pre/post transfer balance checks

Pre/post transfer balance checks.

Pull Request Test Coverage Report for Build 145

6 of 6 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.9%) to 65.591%

Totals

Change from base Build 140:
2.9%

Covered Lines:
42

Relevant Lines:
59

💛  - Coveralls


xr.decode_cf() fails to replace FillValue with np.nan with numpy >= 2.0 when data array dtype is np.float32

What happened?
Since numpy >= 2.0, xr.decode_cf() fails to replace FillValue with np.nan when:

the data array dtype is np.float32
the  FillValue attribute is of type float.

The issue does not occur for data arrays with dtype np.float64.
This bug should not affect decoding of dataset opened with open_dataset because since numpy >= 2.0, numeric values in the attributes are now directly stored as np.int<...> or np.float32 types instead of the previously used int or float types.
To keep my code compatible with all numpy versions I currently use:
if version.parse(np.__version__) >= version.parse("2.0.0"):
    vars_and_coords = list(ds.data_vars) + list(ds.coords)
    for var in vars_and_coords:
        if "_FillValue" in ds[var].attrs:
            ds[var].attrs["_FillValue"] = ds[var].data.dtype.type(ds[var].attrs["_FillValue"])

What did you expect to happen?
xr.decode_cf() to replace fillvalues with np.nan also when the FillValue attribute is of type float.
Minimal Complete Verifiable Example
import numpy as np
import xarray as xr

fill_value = -99.9
data = np.ones((2,3)) * fill_value
data = data.astype(np.float32) # The error does not occur with float64 !

ds = xr.DataArray(data, dims=["x", "y"]).to_dataset(name="var")
ds["var"].attrs["_FillValue"] = fill_value # float
xr.decode_cf(ds)["var"].data # do not replace -99.9 values with np.nan
ds["var"].attrs["_FillValue"] = np.float32(fill_value) # np.float
xr.decode_cf(ds)["var"].data # do replace -99.9 values with np.nan

MVCE confirmation

[X] Minimal example — the example is as focused as reasonably possible to demonstrate the underlying issue in xarray.
[X] Complete example — the example is self-contained, including all data and the text of any traceback.
[X] Verifiable example — the example copy & pastes into an IPython prompt or Binder notebook, returning the result.
[X] New issue — a search of GitHub Issues suggests this is not a duplicate.
[X] Recent environment — the issue occurs with the latest version of xarray and its dependencies.

Environment

INSTALLED VERSIONS
------------------
commit: None
python: 3.11.9 | packaged by conda-forge | (main, Apr 19 2024, 18:36:13) [GCC 12.3.0]
python-bits: 64
OS: Linux
OS-release: 6.2.0-33-generic
machine: x86_64
processor: x86_64
byteorder: little
LC_ALL: None
LANG: en_US.UTF-8
LOCALE: ('en_US', 'UTF-8')
libhdf5: 1.14.3
libnetcdf: 4.9.2
xarray: 2024.7.0
pandas: 2.2.2
numpy: 2.0.1
scipy: 1.14.0
netCDF4: 1.7.1
pydap: None
h5netcdf: None
h5py: 3.11.0
zarr: None
cftime: 1.6.4
nc_time_axis: None
iris: None
bottleneck: None
dask: 2024.8.1
distributed: 2024.8.1
matplotlib: 3.9.2
cartopy: 0.23.0
seaborn: None
numbagg: None
fsspec: 2024.6.1
cupy: None
pint: None
sparse: None
flox: None
numpy_groupies: None
setuptools: 72.1.0
pip: 24.2
conda: None
pytest: 8.3.2
mypy: None
IPython: 8.26.0
sphinx: 8.0.2

Thanks for the nice bug report, this seems to be broken with float16 too.
Are you able to investigate a fix ?

Would it help to cast fv to dtype here:
https://github.com/pydata/xarray/blob/main/xarray/coding/variables.py#L228-L232


Put ASP version number in log files

Is your feature request related to a problem? Please describe.
In the many .txt log files that ASP writes out, we promptly run uname and paste the output there, but there does not seem to be a record of what ASP version number is being used.
Describe the solution you'd like
Include the ASP version number in the logfile output.

Done. Now at the top of each log file there is the line:
ASP 2.7.0
with the version being pulled from the appropriate CMake file.


Quick start guide?

Is there a simple quick start guide?
The code for the website/online example is:
<!DOCTYPE html>

	<meta charset="utf-8">
	<meta content="width=device-width,initial-scale=1" name="viewport">
	<link href="/favicon-192.png" rel="icon" type="image/png">
	<link href="/favicon-192.png" rel="apple-touch-icon">
	<meta content="yes" name="apple-mobile-web-app-capable">
	<meta content="black" name="apple-mobile-web-app-status-bar-style">

	<title>jsconsole</title>
	<link href="/static/css/main.e714727a.css" rel="stylesheet">

Where can I find /static/css/main.e714727a.css and /static/js/main.6c2dc45f.js non-minified?

What do you want a quick start to do?
Using jsconsole, you treat it as a REPL…

Reproduce this. I cloned the repo. Now what? Is there a back end, or does this run completely client side? I'm looking for a client side JS repl for an app I want to make.

Hi there, sorry to jump in but since jsconsole.com gives 404 when I call the script I'm interested in running this nice tool localy.
@abalter You can start the project with a npm run start (have a look at the package.json for other commands).
@remy On which version of Node and stuff this is running? After the project start and I get a web interface I also get an Unexpected token < in JSON at position 0 error when I try the :listen command. Thanks for your time!


update rules_pkg to 0.2.1

Needed if we flip incompatible_no_target_output_group.

See https://github.com/bazelbuild/bazel/issues/7949


Intel and M1 homebrew location differences

My Intel MBP16 and M1 minis have different locations to store cmake modules， therefore I add
"/opt/homebrew/Cellar/cmake/3.20.3/share/cmake/Modules/FindSodium.cmake"
here to help M1 users. 2 of my M1 rigs have successfully  run with this directory.

How can i bring it to run?

I also had issues so I just changed the "wget -O" to my "~/chia-plotter/" directory and then added "set(CMAKE_MODULE_PATH ${CMAKE_CURRENT_SOURCE_DIR})" in the CMakeLists.txt file

Hoe can i change that?
Sorry for my new question
Von meinem iPhone gesendet

Am 12.06.2021 um 17:00 schrieb vvashkot @.***>:

I also had issues so I just changed the "wget -O" to my "~/chia-plotter/" directory and then added "set(CMAKE_MODULE_PATH ${CMAKE_CURRENT_SOURCE_DIR})" in the CMakeLists.txt file
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

same here wit Mac mini m1 this is the error
CMake Error at CMakeLists.txt:23 (find_package):
  By not providing "Findsodium.cmake" in CMAKE_MODULE_PATH this project has
  asked CMake to find a package configuration file provided by "sodium", but
  CMake did not find one.

  Could not find a package configuration file provided by "sodium" with any
  of the following names:

    sodiumConfig.cmake
    sodium-config.cmake

  Add the installation prefix of "sodium" to CMAKE_PREFIX_PATH or set
  "sodium_DIR" to a directory containing one of the above files.  If "sodium"
  provides a separate development package or SDK, be sure it has been
  installed.

My Intel MBP16 and M1 minis have different locations to store cmake modules， therefore I add
"/opt/homebrew/Cellar/cmake/3.20.3/share/cmake/Modules/FindSodium.cmake"
here to help M1 users. 2 of my M1 rigs have successfully run with this directory.

where do I update the above code so the plotter knows were to find the libs

set(CMAKE_MODULE_PATH ${/opt/homebrew/Cellar/cmake/3.20.3/share/cmake/Modules/})" in the CMakeLists.txt file right after “PROJECT”

On Jun 22, 2021, at 10:00 PM, Oscar Quinteros @.***> wrote:

My Intel MBP16 and M1 minis have different locations to store cmake modules， therefore I add
"/opt/homebrew/Cellar/cmake/3.20.3/share/cmake/Modules/FindSodium.cmake"
here to help M1 users. 2 of my M1 rigs have successfully run with this directory.
where do I update the above code so the plotter knows were to find the libs
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.


fix(chat): add { and } in not allowed symbols (Issue #928)

Description:
add { and } in not allowed symbols
Issues:

Issue #928

Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review


News Feed task doesn't work behind proxy, impacting performance

Please verify that this bug has NOT been raised before.

[X] I checked and didn't find a similar issue

Describe the bug*
The update_news_feed task attempts to fetch the RSS/Atom feed once daily. This, however, doesn't work behind a proxy server.
The result is that these tasks occupy workers all the time, and never complete.
Each worker is terminated roughly every 90 seconds due to this.
Steps to Reproduce

Put the InvenTree backend on a network unable to reach INVENTREE_NEWS_URL
Trigger the task
Task will lead to continuous timeout termination of workers

Expected behaviour
Task should finish with no new News entries added if URL is unreachable.
Deployment Method

[ ] Docker
[X] Bare metal

Version Information
0.12.10
Please verify if you can reproduce this bug on the demo site.

[ ] I can reproduce this bug on the demo site.

Relevant log output
No response

@LavissaWoW we have been discussing removing the news feed entirely - it is very rarely updated anyway. Happy to run with an interim fix until we make that change though.
In this case do you know what is blocking? Is it simply a connection timeout to the remote server?

@SchrodingersGat My guess is a network timeout. My prod instance sits behind a proxy, and from what I see, feedparser doesn't really account for timeouts when trying to use it for requests as well.
I can't really mess a whole lot around and experiment with my prod server, so not too much done on that front. Looking at the feedparser package though, it seems that the intended use of it is only parsing returned data from I.e the requests library. Not running the request itself. Saw some discussions on their GH page about removing the ability for it to do HTTP request by itself.
On my prod, I've deleted that task schedule from my admin view as a workaround. But I suppose that can pop back into existence after a reboot.

@LavissaWoW do you see the same problem in the check_for_updates task?

Only the news_feed is problematic here, And I think the issue lies in how it's implemented.
As I mentioned above, the maintainer for the feedparser package has stated that the ability to do requests with the library will be deprecated in due time.
Move the requests call out of feedparser, and the requests library should handle timeouts etc and cause the task to at least finish, even though it didn'treach the target URL.
So, in essence, something like this:
    # Fetch and parse feed
    try:
        req = requests.get(settings.INVENTREE_NEWS_URL)
        d = feedparser.parse(req)
    except Exception as entry:  # pragma: no cover
        logger.warning('update_news_feed: Error parsing the newsfeed', entry)
        return

This simple change will cause the task to properly catch requests.exceptions.RequestExceptions
PS: Deleting the scheduled task doesn't work. The task is simply re-added to "scheduled tasks" on a daily basis.

What about to also just ignoring the newsfeed feature if the url is empty, which can be used for such cases?

@LavissaWoW if you can submit a PR with your suggested change, that would be great :)