Dev Test: Capacitor-Lab-Basics 1.7.0-dev.7

@arouinfar , @ariel-phet , capacitor-lab-basics/1.7.0-dev.7 is ready for dev testing. Document issues in https://github.com/phetsims/capacitor-lab-basics/issues and link to this issue.
Assigning @ariel-phet for prioritization.

General Dev Test

What to Test

Click every single button.
If there is sound, make sure it works.
Make sure you can't lose anything.
Play with the sim normally.
Try to break the sim.

General Dev Test Platforms

[ ] Latest macOS, Chrome and Safari
[ ] Latest iOS, Safari
[ ] Windows 10, all browsers
[ ] Latest Chrome OS, Chrome

Issues to Verify

[ ] Inconsistent Screen Icons
Check screen icons look good on tested platforms
[ ] User suggestion: timer and play pause
Check the behavior of the timer along with play/pause button
[ ] Firefox/Edge Performance Issues
Note performance concerns if nessecary
[ ] [iPad]Can drag two items at the same time
Check if this is still present

These issues should have the "status:fixed-pending-testing" label. Check these issues off and close them if they are fixed. Otherwise, post a comment in the issue saying that it wasn't fixed.

Link(s)

capacitor-lab-basics

FAQs for QA Members

There are multiple tests in this issue... Which test should I do first?
Test in order! Test the first thing first, the second thing second, and so on.

How should I format my issue?

We typically assign the developer who opened the issue in the QA repository.

My question isn't in here... What should I do?
You should:

Consult the QA Book.
Google it.
Ask Katie.
Ask a developer.
Google it again.
Cry.

@KatieWoe putting this one at medium priority for the moment. But we would like to finish this sim off in the near'ish future as it has some enhancements we have been waiting to deploy.

QA is done

All issues have been addressed or closed. Closing this issue. Thanks, QA.


Allow marshaling of invalid enum values

Before this PR, attempting to marshal an invalid enum value would return an error.
This PR allows invalid enum values to be marshaled and unmarshaled as ints. In addition, empty bitmasks are now marshaled as "0" instead of an empty string.

Hello, seems all good to me. This has been merged, thanks.


[webpack 5] moduleIds hashed shouldn't be deprecated for deterministic

Feature request

What is the expected behavior?
Webpack 5 presents a warning that moduleIds: hashed is deprecated in favor of deterministic.  These aren't interchangeable when modules are referenced across more than 1 build.
What is motivation or use case for adding/changing the behavior?
In a multi-compiler build (e.g. SSR server & client) we collect which modules initiate dynamic imports() in the client bundle and pass that information to the server bundle.  When a request runs on the server bundle, if any of those modules get required(), we know to include that dynamic import's script tag in the server response html.  The module ids have to be hashed in this case so they match across builds.
How should this be implemented in your opinion?
Remove the build warning.
Are you willing to work on this yourself?
yes

Use should not use hashed or deterministic in this case

@evilebottnawi can a function currently be used? Documentation doesn't say that it can. You're right, it would be probably faster to only uniquely identify the modules we need and do the fastest/simplest thing for the rest.

@gpoitch because it is very non standard, I thing you need to implement own plugin, you can look at code of hashed plugin

@evilebottnawi ok, I think that could be doable.  What's the best way to determine if a module is a dynamic import trigger now?  I was checking module.reasons but it was removed in webpack 5.  My workaround was calling compilation.getStats() which still contains the reasons but I think that wouldn't work during the custom module id'ing phase.

You need graph for this

ModuleGraph getIncomingConnections
Yep, best don't use ids for this task. Even with hashes ids there is no guarantee that ids are identical. Use the default ids and store mappings for id to the information you need.

Thanks for the tips.  I'm realizing the modules ids aren't matching up anymore in webpack 5 prod mode even if I set hashed.
I ended up leaving moduleIds to its default (unset).
Then in my plugin, used the moduleGraph to find the module for each chunk in the client bundle that triggers a dynamic import and record its dependency request.  Then find each module in the server bundle who has matching requests and record their ids.


settings: Fix "message content in message notification emails".

In user settings, if "message content in message notification emails" is disabled at an Organisation level, then the "Allow
message content in message notification emails" should be shown both disabled and greyed out. At present a user can still
enable this box, and messages notification emails still do not contain message content

This PR fixes this issue.
Fixes: #27262

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[] Calls out remaining decisions and concerns.
[] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Thanks for working on this,  @kartikbhtt7 !
Please fix the failing tests and post a comment requesting a review.

You will also need to include a screenshot showing the tooltip.

Fixed the failing tests and added the screenshot of tooltip.
I think it is ready for review @alya

Thanks! The tooltip should be on the disabled setting itself. We don't want a separate i icon.

Thanks! The tooltip should be on the disabled setting itself. We don't want a separate i icon.

@alya
Fixed that in recent commit.

Thanks! The screenshot looks good to me now.
@sahil839 could you review this one when you have time (not an 8.0 goal)?

@kartikbhtt7 are you planning to address the feedback on this? I've tagged it as something other developers can pick up since it's been quite a while.

sure I'll address the feedbacks.

addressed the feedback

There are some unrelated changes which should be fixed.

ig these are some changes that got introduced by the ./tools/lint command
will implement rest of the changes as mentioned.

ig these are some changes that got introduced by the ./tools/lint command

I don't think that would be the case. Will most probably be fixed by rebasing onto main.

A few points -

There are some unrelated changes which should be fixed.
Can you also implement the changes discussed in https://github.com/zulip/zulip/pull/27543/files#r1438156860. Would be good if there is a separate commit for renaming the object in settings_config and then the commit which disables the setting.
Please update the commit message to mention the issue number there is no need to mention the PR number. And it should be written as "Fixes part of Changes to "Allow message content in message notification emails" #27262.", since the issue should still remain open.
Hey @sahil839, I've filed a new PR that builds upon this one and addresses all the points you mentioned. I'd appreciate it if you could take a look at it: #32067.

Thank you!

Closing in favor of #32067, thanks for the work on this @kartikbhtt7!


References in referenced files not resolved

suppose I have two files
# api.yaml
openapi: 3.0.3

info:
  title: chicken
  version: 0.0.0

paths:
  /get:
    post:
      operationId: getOneThing
      requestBody:
        $ref: '#/components/requestBodies/GetOneThingRequest'
      responses:
        200:
          $ref: '#/components/responses/GetOneThingResponse'

components:
  requestBodies:
    GetOneThingRequest:
      content:
        application/json:
          schema:
            $ref: 'schemas.yaml#/components/schemas/ThingKey'

  responses:
    GetOneThingResponse:
      description: one thing
      content:
        application/json:
          schema:
            $ref: 'schemas.yaml#/components/schemas/Thing'
# schemas.yaml
components:
  schemas:
    ThingKey:
      type: object
      properties:
        userId:
          type: integer
        thingId:
          type: integer
      required:
        - userId
        - thingId

    ThingData:
      type: object
      properties:
        lastSeen:
          type: string
          format: date-time
        value:
          type: integer

    Thing:
      type: object
      allOf:
        - $ref: '#/components/schemas/ThingKey'
        - $ref: '#/components/schemas/ThingData'

This is valid according to swagger-cli. However, if I run oapi2ts I get
stack: 'SyntaxError: Error resolving $ref pointer "#/components/schemas/ThingKey". \n' +
    'Token "schemas" does not exist.\n' +

However, if I change the definition of Thing to
    Thing:
      type: object
      allOf:
        - $ref: 'schemas.yaml#/components/schemas/ThingKey'
        - $ref: 'schemas.yaml#/components/schemas/ThingData'

It works correctly.
It almost seems as if the references are being resolved with respect to the original file they are given.
I can see that it might be very difficult to fix this issue with your current setup (which might require a stack of contexts or something equally awful). One option might be to look at using SwaggerParser.bundle at the beginning of execution to prevent this issue from arising.

@mtrimolet So I just checked into this again after having worked around it in the past. It turns out that the fix was not actually successful. I added a broken test in the new PR to demonstrate.

@mtrimolet So I just checked into this again after having worked around it in the past. It turns out that the fix was not actually successful. I added a broken test in the new PR to demonstrate.

@mtrimolet FWIW, I may have made an error of communication in the original report here. I might have given you the impression that the above example, and more or less exactly the above example, was my actual use case. Therefore, if your fix caused the above example to work, the issue was solved. If so, this was a mistake on my part. The above example was just a simple way to demonstrate the references were not being resolved correctly.
This is why I suggested solving the problem with SwaggerParser.bundle. For just the above case, maybe it would be overkill. But for a fully-general solution to the issue of correctly resolving references in valid specs, it's probably much simpler than expanding your custom fix from #17.

@mtrimolet FWIW, I may have made an error of communication in the original report here. I might have given you the impression that the above example, and more or less exactly the above example, was my actual use case. Therefore, if your fix caused the above example to work, the issue was solved. If so, this was a mistake on my part. The above example was just a simple way to demonstrate the references were not being resolved correctly.
This is why I suggested solving the problem with SwaggerParser.bundle. For just the above case, maybe it would be overkill. But for a fully-general solution to the issue of correctly resolving references in valid specs, it's probably much simpler than expanding your custom fix from #17.


Can you all more models?

The same endpoint has supported all new models like gpt4, all can you make it customized to we can pass a string name for lastest model.

Hey, @wyk111wyk.
Model name is actually a string, so you can use it like this:
openAI.chats(query: .init(model: "gpt-4", messages: messages)) { }

I will add constants "gpt-4*" constants shortly.

I will add constants "gpt-4*" constants shortly.

Thanks for your quick response. And sorry for my title typo.


add processing code to calculate method mix from PMA

Brief description
Adds code and data files for Kenya method mix to use in calibration. Addresses issue #54
Type(s) of change

[ ] Bugfix
[ ] Refactor
[ ] New feature
[x] Other

Checklist

My code follows the style guide

[x] Yes
[ ] N/A

I've commented my code

[x] Yes
[ ] N/A

I've incremented the version number

[ ] Yes
[x] N/A

I've updated the changelog

[ ] Yes
[x] N/A

I've added tests and checked code coverage

[ ] Yes
[x] N/A

This PR adds processing code and the data that result from it to examine method mix in the 2022 PMA out of Kenya.
Here's the mix of users vs. non-users, which you can see is very different from Senegal. This is by design, of course, because we wanted our second calibrated location to be a high-prevalence setting.

Amongst users, here is the mix of methods:

I'll note here that we had to write new script to recode the method variables in PMA, which allows for multiple methods, to only select the most effective method. This means we can match the PMA data as cleaned by IPUMS, but it's also more vulnerable to user error in the coding.

Looks good! Did you find a reference for how they code the ipums? I'd like to look at it.
If you're interested in different ways to code things (and ignore this if not because your ways works too), I would have created the method_recode variable using case_when(). Then you wouldn't have to worry about the order.

I didn't find a reference to how they coded it in ipums, which... ugh. We can shoot them an email and see, though.
How does case_when() handle conflicting 1s? E.g. if a woman is using withdrawal and condoms, she'd have a 1 with withdrawal and a 1 with condoms, and in the method recode, we'd want her to have 'condoms' to match the DHS 'most effective' recoding.


Add TRAFFICE_ROUTING as stage in k8s bluegreen example

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

Thank you.
/approve


Drop C++14 Support

IMO, there is no need to provide further support for C++14 like cpr has already mandated the use of C++17 for its building requirements.
https://github.com/libcpr/cpr/releases/tag/1.9.0
Expected Outcome

Some header files can be eliminated since they are the same as the standard library headers.

lib/inc/drogon/utils/optional.h
lib/src/filesystem.h
lib/inc/drogon/utils/string_view.h
lib/inc/drogon/utils/any.h

I can see that we should be able to remove "boost" from our dependencies entirely.

https://github.com/search?q=repo%3Adrogonframework%2Fdrogon boost&type=code

It's possible to simplify certain codes that involve macros for C++17.

https://github.com/drogonframework/drogon/blob/8b250f8638d526f4d766c1ae710808acb93044f7/lib/inc/drogon/utils/apply.h#L16
https://github.com/drogonframework/drogon/blob/8b250f8638d526f4d766c1ae710808acb93044f7/lib/inc/drogon/PubSubService.h#L39

Expected Drawbacks

Drogon will not be able to build with C++14 compatible compilers.

Tasks

[ ] Update CMakeLists.txt to use only C++17 or later: https://github.com/drogonframework/drogon/pull/1712
[ ]  Delete lib/inc/drogon/utils/optional.h and use the standard library instead
[ ] Delete lib/src/filesystem.h and use the standard library instead
[ ] Delete lib/inc/drogon/utils/string_view.h and use the standard library instead
[ ] Delete lib/inc/drogon/utils/any.h and use the standard library instead
[ ] Remove Boost from our depenedencies
[ ] Simplify some codes with macros for C++17.

https://github.com/drogonframework/drogon/blob/8b250f8638d526f4d766c1ae710808acb93044f7/lib/inc/drogon/utils/apply.h#L16
https://github.com/drogonframework/drogon/blob/8b250f8638d526f4d766c1ae710808acb93044f7/lib/inc/drogon/PubSubService.h#L39

Related: https://github.com/drogonframework/drogon/pull/1331#issuecomment-1199819795

please

https://github.com/drogonframework/drogon/pull/1740


security-proxy-setup: refactor to remove LoggingClient global variable

Fixes: https://github.com/edgexfoundry/edgex-go/issues/2120
Dependency on (includes work from) https://github.com/edgexfoundry/edgex-go/pull/2117.  That PR must be reviewed and merged prior to this one.

Codecov Report

Merging #2123 into master will decrease coverage by 0.02%.
The diff coverage is 37.5%.

@@            Coverage Diff             @@
##           master    #2123      +/-   ##
==========================================
- Coverage   41.76%   41.74%   -0.03%
==========================================
  Files         184      183       -1
  Lines       12151    12156       +5
==========================================
- Hits         5075     5074       -1
- Misses       6779     6785       +6
  Partials      297      297

Impacted Files
Coverage Δ

internal/security/proxy/init.go
0% <0%> (ø)
:arrow_up:

internal/security/proxy/resource.go
88% <100%> (+1.63%)
:arrow_up:

internal/security/proxy/consumer.go
27.38% <25%> (+0.43%)
:arrow_up:

internal/security/proxy/requestor.go
30% <33.33%> (ø)
:arrow_up:

internal/security/proxy/service.go
72.59% <65.78%> (+0.09%)
:arrow_up:

internal/security/proxy/certs.go
84.9% <88.88%> (+1.23%)
:arrow_up:

Continue to review full report at Codecov.

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

Rebased.


Map not working on drag with state region and onChangeRegion.

Bug report

Summary
When using a MapView while controlling the region as state, the map is janking and doesn't respond.
Environment info
Target Plarform

ios 11, 12
real and simulator

Map Provider

Both

react-native info output:
System:
    OS: macOS 11.0.1
    CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 1.81 GB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 12.16.2 - ~/.nvm/versions/node/v12.16.2/bin/node
    Yarn: 1.22.10 - ~/.../node_modules/.bin/yarn
    npm: 6.14.5 - ~/.nvm/versions/node/v12.16.2/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.9.3 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: iOS 14.2, DriverKit 20.0, macOS 11.0, tvOS 14.2, watchOS 7.1
    Android SDK: Not Found
  IDEs:
    Android Studio: Not Found
    Xcode: 12.2/12B45b - /usr/bin/xcodebuild
  Languages:
    Java: Not Found
    Python: 2.7.16 - /usr/bin/python
  npmPackages:
    @react-native-community/cli: Not Found
    react: 16.13.1 => 16.13.1
    react-native: https://github.com/expo/react-native/archive/sdk-39.0.4.tar.gz => 0.63.2
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

{
    "react": "17.0.1",
    "react-native": "https://github.com/expo/react-native/archive/sdk-39.0.4.tar.gz",
    "react-native-maps": "0.27.1",
}

Steps to reproduce

expo init
yarn add react-native-maps
use it on a screen with region and onChangeRegion or onChangeRegionComplete
use the following snippet

import * as React from "react";
import { Dimensions, StyleSheet } from "react-native";
import MapView from "react-native-maps";

const initialRegion = {
  latitude: 51.1261218,
  longitude: 3.347519199999965,
  latitudeDelta: 0.02,
  longitudeDelta: 0.02,
};

export default function TabOneScreen() {
  const [region, setRegion] = React.useState(initialRegion);

  const mapRegionChangehandle = (region: any) => {
    setRegion(region);
  };

  return (
    <MapView
      region={region}
      onRegionChange={mapRegionChangehandle}
      style={styles.mapStyle}
    />
  );
}

const styles = StyleSheet.create({
  mapStyle: {
    height: Dimensions.get("window").height,
  },
});

Describe what you expected to happen:

The map moves smoothly like ice cream

Describe what you actually happens:

The map barely moves, it's janking. Doesn't
Starts moving and stops

To be honest I suspect react-navigation to have a role to play.
Reproducible sample code

I tried this as well but it won't work. This is because MapView is listening to region value and it also is trying to update region via the onRegionChange={mapRegionChangehandle} at the same time.
A way to go about this will be to simple use initialRegion instead of region and use AnimateToRegion like so;
import React, { useState }  from 'react'
import { StyleSheet, View, StatusBar } from 'react-native'
import MapView from "react-native-maps";
import * as Location from "expo-location";

export default function MapComponent() {
  const [errorMsg, setErrorMsg] = useState(null);
  const [location, setLocation] = useState(null);
  const [map, setMap] = useState(null); // I'm using state to save a reference to the map because useRef() won't work and I don't know why

  // your button or whatever will invoke this function
  const centerMap = () => {
    map.animateToRegion(location);
  };

  useEffect(() => {
    (async () => {
      try {
        let { status } = await Location.requestPermissionsAsync();
        if (status !== "granted") {
          setErrorMsg("Permission to access location was denied");
        }

        const location = await Location.getCurrentPositionAsync({});
        setLocation({
          latitude: location.coords.latitude,
          longitude: location.coords.longitude,
          latitudeDelta: 0.0922,
          longitudeDelta: 0.0421,
          // latitudeDelta: 0.45,
          // longitudeDelta: 0.45,
        });
      } catch (err) {
        console.log({ err });
      }
    })();
  }, []);

  return (
    <View>
      <MapView
        ref={(map) => setMap(map)}
        initialRegion={location}
        showsCompass={true}
        rotateEnabled={false}
        style={styles.mapStyle}
        showsUserLocation={true}
        showsMyLocationButton={false}
      />
    </View>
  )
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    backgroundColor: "#fff",
    paddingTop: StatusBar.currentHeight,
  },
  mapStyle: {
    flex: 1,
  },
})

I hope this solves your problem. Let me know if you still need help

Perfect @jalasem ! It'll do for now! Thanks a lot!
The ref worked tho.
const mapRef = useRef(null);

const goToLocation = () => {
    mapRef.current.animateToRegion({...})
}

<MapView ref={mapRef}/>

I will leave the issue open since region and onChangeRegion are still an issue.

It's my pleasure. I'll try useRef as well and see if it'll work that way

I have been bugged by the weird behavior of region + onRegionChangeComplete combo on iOS. Manually animateToRegion avoids all the problems and allows fine control of when and where map view changes. This is the way to go.

Another approach could be the following:
export default function App() {
  const initialRegion: Region = {
    latitude: 52.092876,
    longitude: 5.104480,
    latitudeDelta: 0.23,
    longitudeDelta: 0.23,
  }

  const [region, setRegion] = React.useState<Region>(initialRegion);

  const onRegionChange = (region: Region, details: Details) => {
    setRegion(region);
  }

  return (
    <View style={styles.container}>
      <StatusBar style="auto" />
      <MapView
        region={initialRegion}
        onRegionChange={onRegionChange}
        style={styles.map}
      >
      </MapView>
    </View>
  );
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    backgroundColor: '#fff',
    alignItems: 'center',
    justifyContent: 'center',
  },
  map: {
    width: '100%',
    height: '100%',
  },
});

const mapRef = useRef(null);

hi, there seems to be a lag when i drag the map for some reason. Why is that?


remove unused postcss-loader as dependency

- Summary
In Victor-Hugo's package.json there is a declared dependency for postcss-loader. However, it is not being used anywhere, therefore it seems unnecessary to have it declared.
- Test plan
I removed the postcss-loader dependency from package.json with npm uninstall --save postcss-loader and ran npm start to verify there were not breaking changes in UI and in console output.
- Description for the changelog
remove unused postcss-loader as dependency
- A picture of a cute animal (not mandatory but encouraged)

LGTM


ETA for new release?

Hi Mathias,
I am wondering if you think about creating a new release in the near future. The version tagged with 0.1.0 is 3 years old and seems to be pretty different to master. Also would be nice if you could upload the new release to npm.
Thanks!
Ronny

v1.0.0 has now been published.


Return detailed error (incl. stack trace) instead of err.Error() string

In one of our projects, we are using the pkg/errors library. Errors created with this library can contain a stack trace. Unfortunately, when returning such errors from our build functions, extra information like stack traces are currently not being printed out.
With this change, errors created by pkg/errors are printed out in extended format (including stack trace).
"Normal" errors created with errros.New() or fmt.Errorf(...) are not affected, the output for them stays the same.

Coverage remained the same at 99.677% when pulling ba520805fd0312ec36080885d6be091fdd0f8161 on markusdosch:errors into 9157bf81bf3dafde4b12d6e889648de73ea5c2c6 on sarulabs:master.

Hey @sarulabs , thanks for the fast reply & review! 😃 I just updated my PR with the changes you requested.

I created a new release v2.4.1 with your changes.
Thanks again for you contribution.


Fine-tuning documentation is incorrect

Describe the bug
In fine-tuning documentation, the openai.Model.delete method does not receive the same input as the other listed methods. While, for example, openai.FineTuningJob.retrieve receives the fine-tune job id, openai.Model.delete should receive the name of the fine-tuned model. The documentation induces us to think both methods have the same parameters.
To Reproduce
N/A
Code snippets
No response
OS
N/A
Python version
N/A
Library version
openai-python v0.27.10

Will be fixed soon! Thanks for flagging.


__gpiomux, __gpiofunc size

The const array:
static uint8 __gpiomux[16] = {
should be uint32 type?
static uint32 __gpiomux[16] = {
regards,

You are correct that all the values are of type uint32, but when they are assigned to the uint8 array they are truncated to the one least significant byte, which is the GPIO mux index. Then PERIPHS_IO_MUX_GPIO macro adds the PERIPHS_IO_MUX address back to it and returns it to the original value, meaning the code should work just fine.
I guess I could change it for clarity, but then again the "if it ain't broken, don't fix it" principle would suggest we can keep it as it is, unless you are having any issues with the code?

The compiler gives me error and could not go further even it works with uint8.
(with treat warning as error.)

I guess that's a good enough reason, I just pushed commit ab77ad54fef363a090e88681233b4b565734abea please check and confirm that it fixes the problem.

yes. it solved the issue. I tried already.
Thanks.


Rename function must also handle first argument of Annotated if it is string

After rename OldName -> NewName, pylance does update Annotated.
from typing import Annotated
from typing import ClassVar

class NewName:
    pass

class C:
    x: ClassVar[NewName]
    y: Annotated["OldName", int]

Pylance language server 2024.2.105 (pyright version 1.1.351).

@minmax not fully following. why did you give "OldName" to Annotated rather than actually type OldName?
basically, why Annotated["OldName", ...] rather than Annotated[OldName, ...] ?

It would be required in a forward-ref scenario, and rename doesn't properly handle it there:
class C:
    y: Annotated["OldName", int]

class OldName:
    pass

I see so, the asking is make first argument of Annotated as one of place where we recognize string as type like a: "int" or ClassVar["str"] or # type: "myType"?
sounds reasonable.

make first argument of Annotated as one of place where we recognize string as type

yes, that's what I was trying to say


Parser

Currently struggling a bit with the state machine needed to parse this in a streaming manner; there are a lot of rather weird and hard to explain states needed to handle if statements, etc.

Implementation status:

Statement
Status

line
❌

lineWithEmote
❌

lineWithText
❌

lineWithEmoteAndText
❌

emote
❌

leave
❌

set
❌

if
❌

elseIf
❌

else
❌

menu
❌

label
❌

goTo
❌

background
❌

include
❌

Duplicate of #2


Update s3, sdk-core, sqs to 2.20.25

About this PR
📦 Updates

software.amazon.awssdk:s3
software.amazon.awssdk:sdk-core
software.amazon.awssdk:sqs

from 2.17.295 to 2.20.25
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "software.amazon.awssdk" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #585.


Ability to push server into destination site

Not sure if this is an issue rather than a question, but we're looking at consolidating JSS's (we have three now, and two of those will be merging into the 3rd), and curious if there's a known pathway to use JamfMigrating to basically push "Full JSS" into a specific site on the destination. I know the API can be kind of wonky if it doesn't have full access to the whole JSS (and is instead limited to just a site).
So for example, department X is taking their JSS and moving it to companyjss site X. Everything that can be set site specifically would then be pushed into the site X, and everything that's Full JSS specific would be merged in manually (probably).
Thanks!

@staze - that's one that's been on my list for a while, migrate/clone objects from one server (same server) into a site.  Some day I hope to have time to get it done but not likely before you could manually complete your tasks.


TSC Meeting notes for Feb 20.

Meeting notes.

Gah!  I should have waited to see if that went through!  Thank you for fix... My fault for trying to manually line wrap.  I try to remember to do a / $ to find any but forgot this time.


Added Camel Case Problem

🛠 Issue #315
✔ Check List

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
Program to convert text sentences in CamelCase format.
Examples:
1] INPUT: "this is camel case"
OUTPUT "thisIsCamelCase"
2] INPUT: "Hi Atul here"
OUTPUT: "HiAtulHere"
📷 Screenshots

@atul1510 Could you implement this algorithm in all four languages

Yes, Sure I Can implement this algorithm in all four languages.
@Kumar-laxmi  sir , You can assign me in C/C++/Java/Python languages.

After implementation in all four languages the PR will be reviewed, till then it will be in standby

After implementation in all four languages the PR will be reviewed, till then it will be in standby

Ok Got it, sir

@072arushi @Kumar-laxmi @zugzwang03 Please review this.

Done with all changes requested.

@Kumar-laxmi ?? still in review ??


useField isChecked value

Trying to figure out what type the isChecked field in the return value of useField actually is. However, it's computed as:
		isChecked: isCheckboxElement ? Boolean(value) : value === inputValue ? value : false,

Problem is that it doesn't guarantee booleans (which is a bit counter-intuitive), but only in the case the value hasn't changed from its initial value? Can you please explain the reasoning here and if it actually serves any purpose to not have it only be boolean?

isChecked should be only either true or false by convention, but I might've named it wrong.
I shouldn't have a nested ternary there. Meant to clean it up and forgot.
Checkboxes don't guarantee booleans in HTML. There's an 'indeterminate' state as well:
<input
  checked={/* either `'true'` or `null` */}
  indeterminate={/* either `'true'` or `null` */}
  value={/* either `'true'`, `null`, or some string */}`
/>

If we have no value defined on the checkbox, it's a boolean.
When interacting with HTML, we need to convert it to a string 'true' to denote it has a value. Another common practice is setting the prop to its own name: checked="checked".
If value is defined, we need to store it (inputValue) instead of the boolean state.

From what I'm seeing, isChecked will always be a boolean. The first check forces it to a boolean value. The second check, value === inputValue, can never be true for a checkbox as it's checking boolean === string.
If the input isn't a checkbox type, then value is null. If null === '', we get false`.

If the input isn't a checkbox type, then you're right, we might end up setting it to a non-boolean value.

Shouldn't we try to keep it a boolean no matter what? It's just easier to reason about for users, and this way they get no surprise values for it.

Correct. I'll have to look at Field to make sure, but this might be a place where the entire output value could be wrapped in Boolean. We might need to look for 0 first, but I don't think so.
If you're caring about isChecked, your value will be a boolean or string. It's DOM-specific meaning, we can assume no numbers are passed. But... Someone could pass a number as a default value and that would break the whole thing. We should probably adjust for 0.

Actually, with component libraries and more complex components, strings and numbers are definitely not the only possible value types people will try to control with OneForm.
For example, the component library I'm using for my current project is primereact, and quite a few select-type inputs (listbox, multi-select, even select-button) actually expect object values. It's the reason I left the value typing as any in the typings file, and didn't force it to string or string | number. Have to remember that forms in react rarely deal with actual html inputs directly nowadays, it's usually a component library of some sort.

Fixed in: bb1976da11a44ad7e57c0b0af7c0cd36f5af8ae1.
We're gonna have to figure out a better way of doing this nested ternary, but the solution today works.


update uploadFile method to accept buffer

Description of the feature request:
Please make uploadFile able to accept buffer or url, uploading from a local machine is very limiting.
What problem are you trying to solve with this feature?
This will help loading files in Express JS or Spring Boot
Any other information you'd like to share?
No response

How about this script? This Node.js script demonstrates how to implement resumable uploads using streams with various APIs, such as Gemini, Drive, YouTube, and more. It supports uploading files from both local paths and remote URLs. I originally created this for use with Gemini API and remote file URLs. I hope you find it helpful.
https://github.com/tanaikech/resumableUploadForGoogleAPIs_nodejs

Even better if you the gemini endpoint can accept a url to upload the file, that way we don't even have to keep the buffer in memory, we can just upload something like an s3 presigned url.

Agreed, imagine a environment where the isn't any fs, I'd just like to pass in a publicly accessible URI!


feat(en): Make state keeper work with pruned data

What ❔
Modifies state keeper so that it works with pruned node data during snapshot recovery.
Why ❔
Part of preparations of EN code to support snapshot recovery.
Checklist

[x] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[x] Tests for the changes have been added / updated.
[x] Documentation comments have been added / updated.
[x] Code has been formatted via zk fmt and zk lint.
[x] Spellcheck has been run via zk spellcheck.

Separately, would Main Node work with pruned data? I don't see why not.

Right now it won't because some components only run on the main node would need to be adapted to support snapshot recovery. Such adaptation seems straightforward though.


Startpage - Before you begin out of sight

Not sure what can be done about this, but I totally missed the 'Before you begin' text. Not sure if this is dependent on your screen or something, but I think one should at least see that there is information to be read 'before you begin'.
I now see this only applies to the Dutch text....

Duplicate of #812


Custom portal shape

I have a non-rectangular portal in my hub, and I tried to set it up as a portal to Survival, but it wasn't being detected. For now, I changed the shape of the portal, but I hope to see support for non-rectangular portals in the future.

Nether portal right? Or water portal? For nether portal I dont think it actually checks shape - unless a config.yml option is set to check for obsidian frames. Which can be turned off if it is on.

I tried it with an end portal in the ground, maybe it doesn't support end portals. I haven't checked that.

Apparently, end portals can't be used but end gateways can. I used the command to get a custom portal block.

But if I used the command to get a custom portal item then why wouldn't it work?

Maybe the issue is different, in that vanilla doesnt fire the events for the portal anymore or something. Ill have to look into that then, there's been several weird issues with end portals.
Of course do check its not fixed with https://ci.mg-dev.eu/job/MyWorlds/321/ just in case, as there was some nonsense with the respawn event not firing

The issue has not been resolved in the latest build. Thank you for your help, though.

I'll try to get back to you about this, a lot of end portal related things broke with 1.21.2+, I'll just have to dive in and rewrite some stuff.
Reason it's complicated is that the server does this weird player respawning logic for end portals, and the bukkit events for it are kind of bad. With 1.21.3 it also got very buggy.

I tried this on the latest Paper 1.21.4 and Spigot 1.21.4 builds, but it works fine there. Whats the exact server version youre testing with? And what type of world is the end portal on?
Also, what does /mw endportal say in the world the end portal is at?
You can also try to see if a [portal] sign near the end portal makes it do something.
I tried a flat end portal built using /mw giveportal end item in the overworld, a flat world and end world, and they work in all. I did have to adjust /mw endportal mode default in some worlds though.


Explore swapping build table for left outer joins

Is your feature request related to a problem? Please describe.
When performing a left outer join on a small left input and a large right input, we're going to use the large input to build the hash table and the small input to probe it.  This could lead to poor performance on the GPU because there's potentially many collisions during the build and potentially lower parallelism on the probe side after the build.
Describe the solution you'd like
In this scenario, we could use the much smaller left input as the hash table, but then we need to solve the problem of identifying rows in the left input that were not "hit' during the probe from the right input.  In this case we could take a similar approach that we do for full outer joins, which is use the left input gather map as a scatter map for false to build a filter mask identifying which rows were hit during the probe, and the remaining rows after the filter are the ones not matched to any row in the right input.

First step is to verify that this approach is viable when the left table is "small enough" and the right table is "big enough" where swapping the build side and performing the extra scatter is worth it and identifying heuristics to detect when to apply this algorithm.  I suggest trying two extreme cases, one where the left and right tables have no duplicate keys within each table, and another where both tables consist of a single key (i.e.: one join scenario does not explode while the other maximally explodes).


Can't get Radarr/Sonarr to connect.

Hey man, awesome little app.
So long story short, I get a 500 error when trying to connect prowlarr to this.
Here's my info.

Error

prowlarr-proxy config

suffix: !
paths:
  - label: radarr
    path: /radarr/*
    baseUrl: http://<IP_ADDRESS>:7878
  - label: sonarr
    path: /sonarr/*
    baseUrl: http://<IP_ADDRESS>:8989

Radarr config

prowlarr-proxy log

Unraid Docker container config

Seems like it's communicating with the app, just... not working?
Thank you for any help!

I have the same problem.


Cannot connect to pad-local with WLAN

I cannot connect to pad-local when I'm connected to WiFi in my home. When I'm using the hotspot on my iPhone, it suddenly works. Obviously, I cannot use my hotspot all the time, so I would appreciate it if you can help with the debugging.

This is the error message that I've got.
I pinged gateway.pad-local.com, and it fails. It works when I'm using my phone. I'm in the US, using AT&T as my ISP, if that matters.
Mirror issue
node: 14.17.0
npm: 6.14.13
wechaty: 0.62.3
wechaty-puppet-padlocal: 0.4.2
OS: Windows 10

It was caused by network issue.


require deme.start_time and epoch.end_time in input files

This change is to follow the reference implementation. See https://github.com/popsim-consortium/demes-spec/issues/32

My understanding was that the deme does not require a start time, except in scenarios where it has multiple ancestors. Otherwise it inherits the start time from the ancestor's end time, or the start time is set to inf if it has no ancestors. That was what I thought the takeaway of that linked issue was, and how it's implemented in the spec now. Am I wrong about that?

My understanding was that the deme does not require a start time, except in scenarios where it has multiple ancestors. Otherwise it inherits the start time from the ancestor's end time, or the start time is set to inf if it has no ancestors. That was what I thought the takeaway of that linked issue was, and how it's implemented in the spec now. Am I wrong about that?

Yeah, you're right. I opened this issue before looking into the gory details. The essential changes we need to make are to (1) disallow start_time in an epoch stanza, (2) allow the omission of end_time in the last epoch if the end time is 0.
I made a start on this in #188. But I won't get back to it for a couple of days because I'll be away from the keyboard (again). So, you're welcome to jump on it and resolve anything that doesn't seem to match the behaviour of the reference_implementation.

Yeah, you're right. I opened this issue before looking into the gory details. The essential changes we need to make are to (1) disallow start_time in an epoch stanza, (2) allow the omission of end_time in the last epoch if the end time is 0.
I made a start on this in #188. But I won't get back to it for a couple of days because I'll be away from the keyboard (again). So, you're welcome to jump on it and resolve anything that doesn't seem to match the behaviour of the reference_implementation.


chore: removing node7 job from CircleCI

We don't need to test on Node 7 anymore. Note: no action is required for this PR for now.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #28   +/-   ##
=======================================
  Coverage   82.08%   82.08%
=======================================
  Files           3        3
  Lines          67       67
=======================================
  Hits           55       55
  Misses         12       12

Continue to review full report at Codecov.

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


Add Travis CI badge [ci skip]

Thanks for the PR, but I don't see the point in adding Travis CI badges to READMEs. I will always run tests on all Ruby versions before I push to master, and if it is somehow happens that I don't, I will be notified via email.


Fix 'Trailing spaces' issue in .circleci\config.yml

CodeFactor found an issue: Trailing spaces
It's currently on:
.circleci\config.yml:7

fixed in #10


Initializing chatbot with a different database name for JsonFileStorage

Hi,
I would like to initialize a chatbot with a different db name.. Is it possible to do it?
I can do it for the JsonFileStorageAdapter class by
j1 = JsonFileStorageAdapter(database = "sampledb.db")
However, I am not sure how to create a chatbot that references a db other than database.db

As of now I am doing the following.. please let me know if this is correct:
c1 = ChatBot("wisebot", storage_adapter = "chatterbot.storage.JsonFileAdapter")
c1.storage.init(database = "sampledb.db")
I can see a separate db called sampledb being created if this is being done...But im not sure if this is the apt way to do it :)

Did you tried like this
# -*- coding: utf-8 -*-

from chatterbot import ChatBot
# Create a new instance of a ChatBot
bot = ChatBot(
    'Exact Response Example Bot',
    storage_adapter='chatterbot.storage.JsonFileStorageAdapter',
    database='myJson.db',
)

# Get a response given the specific input
response = bot.get_response('Help me!')
print(response)

Yes.  It works... silly me.
Thanks.

Response while using the storage adapter is too slow


Section 4.2 error: character dimensions

From section 4.2 of the book:

Each character is made up of eight rows of five pixels, with each row using a byte of data, meaning that each letter altogether takes up five bytes of data.

This should either be "five rows of eight pixels" or "eight columns of five pixels".

Thanks for catching that! Fixed in e0b3888bed29d5e7201e26c8a8a7cbc3d6becc8f


Some questions about interruptions

Thank you very much for your answer. I would like some system descriptions of interrupts, and I would also hope you to provide additional information about interrupts if possible

I'll update the repo as soon as possible to provide more information about the interrupt.

Well, thank you very much for your help with our work.

Regarding the three examples mentioned in your paper.For the igbvf driver, no syscalls are found in enable_syscalls.json. For the nosy driver, the error is lock,what kind of question is that.


Export Option in Menus

New Feature
Priority: High
Description:
Add an export option in the hamburger menus for specific collections, groups, and test cases, enhancing data portability.

TESTED OK:
Tested version - Testman 3.2.5.dmg
https://jam.dev/c/087cfa3c-89e6-4a95-a516-9053c0faa016


SPA Lazy Imports

What this PR does / why we need it:
Applies react lazy imports for page modules. Each page within the SPA build will be a separate module with its corresponding javascript and css. This will reduce the initial load time, loading only the assets needed for the SPA requesting path.
Which issue(s) this PR closes:

Closes #381

Special notes for your reviewer:
I made changes to the i18n configuration.
Several locale files were deleted and their content was moved to a shared file -- shared.json.
Suggestions on how to test this:
Code visual inspection and re-run of tests actions.
Does this PR introduce a user interface change? If mockups are available, please link/include them here: No.
Is there a release notes update needed for this change?: No.
Additional documentation:

coverage: 97.411% (+0.007%) from 97.404%
when pulling bee9a0d8c56691b963d11a5ac0c1d7679e721cd4 on feature/381-implement-lazy-loading-2
into 99a04c29fbc54a1fcc7ac5c97af71d76840c4e1d on develop.

@ekraffmiller I added a component test for the AppLoader component and solve merge conflicts 👍🏼

Ran tests in local, no issues discovered. This will be tested more over time.


Not working at all under Windows 10

Did some debugging and found out that applying the following fix makes it working:
let l:root = fnamemodify(b:NERDTree.root.path.str(), ':p:gs?\\?/?:S').
The modifier :gs?\\?/? replaces all backward slashes with forward slashes.
Not sure about any side effects though.

Broken for me too, the above fix works for me too.
For anyone stumbling across this trying to fix their nerdtree-git-plugin, modify line 81 in git_status.vim to what @jadero posted.


Minor schema fixes

Related issues
Refs: None
Describe the changes made in this pull request

Recreates the minor schema fixes originally made in #192

Instructions to review the pull request
git clone this repo
git checkout this branch
python3 -m venv env
source env/bin/activate
pip install --upgrade pip setuptools wheel
pip install -r requirements.txt
python3 schema_poc.py -d data.yml -s schema.json  # <- should be quiet

Superseded by #205. Closing.


Create elic4vet.json

Description
Hello 👋 i am Elisabeth a Junior Frontend Developer. I wanted to contribute to your DevDisplay project, as part of Hacktoberfest 2024.
Changes Proposed

[x] I added my profile details to a new json file

Checklist

[x] I have read and followed the Contribution Guidelines.
[x] All new and existing tests passed.
[x] I have updated the documentation to reflect the changes I've made.
[x] My code follows the code style of this project.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers
Thank you for giving me the opportunity to submit this pull request.

add level labels @codeaashu
Regards ,
Community Manager gssoc-ext


feat: to design the detail of insert operation

What would you like to be added:
Why is this needed:

关于insert，可以谈论关于主键等细节的设计方案，有兴趣的一起搞

done


[HybridApp] In development, opening the new experience shows the boot splash and doesn't load

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: N/A
Reproducible in staging?: N/A
Reproducible in production?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers): N/A
Logs: N/A
Expensify/Expensify Issue URL: N/A
Issue reported by: @arosiclair
Slack conversation: https://expensify.slack.com/archives/C036QM0SLJK/p1726157922153189
Action Performed:

Cloned Mobile-Expensify repo
Follow the README to setup the HybridApp
Enroll into the new experience by running the following snippet in the OldDot JS console:

NVP.set('tryNewDot', {classicRedirect: {dismissed: false}});

Run the app on an iOS/Android device
Log into the enrolled user

Expected Result:
The app loads and shows the new experience LHN
Actual Result:
The boot splash shows and never disappears
Workaround:
None
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

View all open jobs on GitHub

We did some troubleshooting in slack here, but no solutions found yet. cc @Julesssss
At a minimum, we're missing instructions on adding Mobile-Expensify/react-native/.env. But even adding that doesn't fix the issue.
@staszekscp @war-in @AndrewGable it sounds like you guys are the only ones successfully running the HybridApp. Can any of you take a look? Ideally, we should be able to just clone Mobile-Expensify, follow the instructions and be good to go for HybridApp dev.

@mateuuszzzzz noticed today that the react-native submode commit was outdated. This should be the breakthrough we were hoping for 🎉

Just noting there is an outstanding issue with live-markdown, so engineers need to checkout the prior commit within react-native: d5cfecfc53a

Bump to @Julesssss comment
Developers should use ND (submodule) with 16415328f9462eb4e9a8cd0099833d29948f1544 commit

https://github.com/Expensify/Mobile-Expensify/pull/13122 Should fix the issue

I don't think this is done. Has anyone completed the repro steps and confirmed the new experience loads? It doesn't look like it from the PR we just merged or the slack conversation. We also haven't updated the HybridApp instructions to include steps for adding a .env.

@arosiclair nope, thanks for calling this out.
This morning I switched my local builds to the latest commit and will report back.

Hi 👋 @arosiclair I went through the reproduction steps and I can confirm the issue with infinite bootsplash is fixed (it works fine for me). That's why I suggested to close the issue
However, I noticed a few inconveniences while setting up the repo. This is the list:

the main.jsbundle.map file is not created automatically for the Debug build which results in a failing build

workaround: run the Release build first

I was not able to correctly build iOS with the npm run ios command

workaround: build with Xcode

Thoughts:

the build commands should be updated/repaired
the main.jsbundle.map issue should be fixed
README should contain more details on troubleshooting a .env configuration

Please share your thoughts too and let me know if the infinite bootsplash issue is gone 🙏
cc @arosiclair @Julesssss

Just tested with npm run ios and it worked with the latest changes. I did have to kill and then relaunch the app to get it to connect to Metro. That has also been an issue with NewDot standalone so I think we can do nothing for that issue.

Thoughts:

the build commands should be updated/repaired
the main.jsbundle.map issue should be fixed
README should contain more details on troubleshooting a .env configuration

Agreed 👍. I'll post a PR to add instructions on the .env

Agreed 👍. I'll post a PR to add instructions on the .env

Amazing, thank you!
You can also be interested in this PR where @mateuuszzzzz is adding .env configuration into HybridApp

PR is code complete but held atm

Still held

README updates are in this is all done.


Consider estimating overscan with item size and viewport size

Overscan should be lower if item size is large and should be higher if item size is small. Current default value is not suitable for some case.

Closing in favor of #378.


French translation

Additional strings from devel branch are translated in french

I already merged #110, this PR is no longer needed.


[Bug] should dataReadAheadEnable default be true?

Before Creating the Bug Report

[X] I found a bug, not just asking a question, which should be created in GitHub Discussions.

[X] I have searched the GitHub Issues and GitHub Discussions  of this repository and believe that this is not a duplicate.

[X] I have confirmed that this bug belongs to the current repository, not other repositories of RocketMQ.

Runtime platform environment
Ubuntu 20.04
RocketMQ version
5.1.3
JDK Version
1.8
Describe the Bug
Before #6336 [RIP-62] Cold Read Control, the default value of madvice is MADV_NORMAL. after RIP-62, the default value changes to MADV_RANDOM. so I think the default value of dataReadAheadEnable should be true, to keep the same with old RocketMQ versions.
Steps to Reproduce
start new version of RocketMQ.
What Did You Expect to See?
madvice MADV_RANDOM is not called.
What Did You See Instead?
madvice MADV_RANDOM is called.
Additional Context
https://github.com/apache/rocketmq/blob/develop/store/src/main/java/org/apache/rocketmq/store/CommitLog.java#L149

+1，OS read ahead should be turned on by default.

不开readAhead会严重影响slave同步冷数据速度


Add icon for Marko

For the Marko templating language - https://github.com/marko-js/marko

Excellent PR! :D Bravo!

Awesome. Thanks guys!

Our pleasure! :D


Update readme

changed name, included link to edit files.

Ravi, you are trying to modify our example page. You probably meant to fork this project first and then modify your fork, right?


Create high fidelity design screens

Created all tablet hi-fi
https://www.figma.com/file/pWQbwawuUezdlGFhu0SCcy/Team-6-Auth_wiki-(Copy)?node-id=21%3A8097

High fidelity Error message sign in for desktop
https://www.figma.com/file/pWQbwawuUezdlGFhu0SCcy/Team-6-Auth_wiki-(Copy)?node-id=2%3A15385
High fidelity Error message for sign up desktop
https://www.figma.com/file/pWQbwawuUezdlGFhu0SCcy/Team-6-Auth_wiki-(Copy)?node-id=2%3A15279
High fidelity error message for sign in mobile
https://www.figma.com/file/pWQbwawuUezdlGFhu0SCcy/Team-6-Auth_wiki-(Copy)?node-id=2%3A17063

Created landing page for mobile
https://www.figma.com/file/pWQbwawuUezdlGFhu0SCcy/Team-6-Auth_wiki-(Copy)?node-id=21%3A4924
Created about us
https://www.figma.com/file/pWQbwawuUezdlGFhu0SCcy/Team-6-Auth_wiki-(Copy)?node-id=21%3A4840
Created main dashboard for mobile
https://www.figma.com/file/pWQbwawuUezdlGFhu0SCcy/Team-6-Auth_wiki-(Copy)?node-id=29%3A6852

Created high fidelity for Python dashboard desktop screen
https://www.figma.com/file/pWQbwawuUezdlGFhu0SCcy/Team-6-Auth_wiki-(Copy)?node-id=21%3A5507
Created high fidelity for Javascript dashboard desktop screen
https://www.figma.com/file/pWQbwawuUezdlGFhu0SCcy/Team-6-Auth_wiki-(Copy)?node-id=21%3A5735
Created high fidelity for PHP dashboard desktop screen
https://www.figma.com/file/pWQbwawuUezdlGFhu0SCcy/Team-6-Auth_wiki-(Copy)?node-id=21%3A5963

Created Python dashboard screen for tablet
https://www.figma.com/file/3NkDH8KNO8CluU3F3iv1yI/TEAM-6-AUTH-WIKI?node-id=1091%3A6156
Created PHP dashboard screen for tablet
https://www.figma.com/file/3NkDH8KNO8CluU3F3iv1yI/TEAM-6-AUTH-WIKI?node-id=1091%3A6422
Created Javascript dashboard screen for tablet
https://www.figma.com/file/3NkDH8KNO8CluU3F3iv1yI/TEAM-6-AUTH-WIKI?node-id=1091%3A6289

Sign in HIFI
https://www.figma.com/file/3NkDH8KNO8CluU3F3iv1yI/TEAM-6-AUTH-WIKI?node-id=685%3A3247

Created Hi-fi for Python, JavaScript and PHP language dashboards for mobile screen
https://www.figma.com/file/nwzkIJMYBP6YlhUqoQh6pH/Team-6-Auth_wiki-(Copy)?node-id=29%3A6984
https://www.figma.com/file/nwzkIJMYBP6YlhUqoQh6pH/Team-6-Auth_wiki-(Copy)?node-id=29%3A6899
https://www.figma.com/file/nwzkIJMYBP6YlhUqoQh6pH/Team-6-Auth_wiki-(Copy)?node-id=29%3A7069
Created Hi-fi for create account page for mobile screen
https://www.figma.com/file/nwzkIJMYBP6YlhUqoQh6pH/Team-6-Auth_wiki-(Copy)?node-id=21%3A5077
Created Hi-fi for Get started dashboard for mobile screen
https://www.figma.com/file/nwzkIJMYBP6YlhUqoQh6pH/Team-6-Auth_wiki-(Copy)?node-id=29%3A6081

Created Hi-fi landing page for desktop
https://www.figma.com/file/pWQbwawuUezdlGFhu0SCcy/Team-6-Auth_wiki-(Copy)?node-id=21%3A6706

Created sign-up hi-fi for desktop
https://www.figma.com/file/pWQbwawuUezdlGFhu0SCcy/Team-6-Auth_wiki-(Copy)?node-id=21%3A3695
Created sign-in hi-fi for desktop
https://www.figma.com/file/pWQbwawuUezdlGFhu0SCcy/Team-6-Auth_wiki-(Copy)?node-id=21%3A3898
Created about us for desktop
https://www.figma.com/file/pWQbwawuUezdlGFhu0SCcy/Team-6-Auth_wiki-(Copy)?node-id=21%3A7073


Optimize and, modify REP clocks and SLOWMODE FREQ.

pri timebalance(HOST CPU:Core i5 12600, in TMENU) value increased from 240000 to 290000 from previous version, and slow mode speed should also be close the first generation FM TOWNS(but not perfect. For example Lemmings need FREQ 8 more).
Reference REP instruction Clocks
https://www2.math.uni-wuppertal.de/~fpf/Uebungen/GdR-SS02/opcode_i.html

Thanks!  Looks good to me!


Dynamic container construction to mitigate InferenceService Python dependency issues

/kind feature
Describe the solution you'd like
Currently there are issues where if a model was not built with the same versions of dependencies baked into the InferenceService model server container images, then the InferenceService will fail post-deploy during inference execution.
A typical error code:
kfserving-container:
    Container ID:  docker://e32e4a84a9de06f12af7c05bcc469958de43dbb60c95651f64ee0e407e157f42
    Image:         gcr.io/kfserving/sklearnserver@sha256:6702dddff99de5b8945f35d913a423e406b6cd039a14cefdbd1886078d393ce5
    Image ID:      docker-pullable://gcr.io/kfserving/sklearnserver@sha256:6702dddff99de5b8945f35d913a423e406b6cd039a14cefdbd1886078d393ce5
    Port:          8080/TCP
    Host Port:     0/TCP
    Args:
      --model_name=aws-sklearn-model
      --model_dir=/mnt/models
      --http_port=8080
    State:       Waiting
      Reason:    CrashLoopBackOff
    Last State:  Terminated
      Reason:    Error
      Message:   [I 200516 20:20:16 storage:35] Copying contents of /mnt/models to local
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/local/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/sklearnserver/sklearnserver/__main__.py", line 33, in <module>
    model.load()
  File "/sklearnserver/sklearnserver/model.py", line 32, in load
    self._joblib = joblib.load(model_file) #pylint:disable=attribute-defined-outside-init
  File "/usr/local/lib/python3.7/site-packages/joblib/numpy_pickle.py", line 605, in load
    obj = _unpickle(fobj, filename, mmap_mode)
  File "/usr/local/lib/python3.7/site-packages/joblib/numpy_pickle.py", line 529, in _unpickle
    obj = unpickler.load()
  File "/usr/local/lib/python3.7/pickle.py", line 1088, in load
    dispatch[key[0]](self)
  File "/usr/local/lib/python3.7/pickle.py", line 1376, in load_global
    klass = self.find_class(module, name)
  File "/usr/local/lib/python3.7/pickle.py", line 1426, in find_class
    __import__(module, level=0)
ModuleNotFoundError: No module named 'sklearn.svm._classes'

To get around this, currently we have to either:
(1) manually check:
https://github.com/kubeflow/kfserving/blob/master/config/default/configmap/inferenceservice.yaml#L9
and then make sure our model was built with the same dependency versions
(2) build a custom inference service image manually
This feature enhancement request proposes that KFServing adopt a dynamic image building execution path such that the system allows use to specify dependency versions in the resource yaml. An example of this yaml is shown below:
apiVersion: "serving.kubeflow.org/v1alpha2"
kind: "InferenceService"
metadata:
  name: "sklearn-iris-gs"
spec:
  default:
    predictor:
      minReplicas: 1
      dependencies:
      	python: 3.7
      	pandas: 1.0.3
      	joblib: 0.4.1
      sklearn:
        storageUri: "gs://pct-tmp"

This [dependencies] would create functionality similar to Anaconda's environments, and would potentially make KFServing's DevOpsUX easier.
Ideally the kfserving execution system would examine the yaml and determine if the stock container for sklearn (a supported built-in model server) was compatible with the requested dependencies. If kfserving determined this was not true, then it would attempt to build a new docker image with the requested dependencies.
On the first container storage cold-cache run, then image would not exist so there would be a performance hit; The system would be building the image in the background, and we might reflect this with a STATUS column:
kubectl get inferenceservices
NAME                URL   READY   DEFAULT TRAFFIC   CANARY TRAFFIC   AGE    STATUS
sklearn-iris-gs        False                                                                             2m.   BUILDING

For every deploy after that using the same dependency requests, the pre-flight check would pick up the cached container (from gcr.io, artifactory, etc) and there would be no performance hit.
Obviously this adds complexity to the deployment logic, but would be easier for the average devops team than doing a custom inference service each time
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]

This is something we would also really like and I wonder if we can model it off of how MLFlow does it: https://www.mlflow.org/docs/latest/projects.html#project-environments (though potentially without using conda)

Re-creating the environment from scratch every time adds some overhead. For MLflow models, Seldon supports creating up the environment dynamically from a conda.yaml file. However, what we found out is that once you add a couple dependencies the time it takes to start up the pod just sky rockets.
I'm not sure what could be a good way to cache these environments so that they can be re-used across runs.
Another option worth looking into is compiling the models to an intermediate format first. I haven't explored these much, but one could use something like ONNX or microsoft/hummingbird so that the server only needs to deal with those. This would solve the problem of dealing with multiple framework versions and potentially even improve the server's performance.


Typing concrete default parameters in generic declarations

Not sure what's going wrong with examples/checker_tests/good/default-package-argument.p4 but it should typecheck and currently doesn't.

Took another look... here's the problem.
 control nothing(inout empty_t hdr, inout empty_t meta, in intrinsic_metadata_t imeta) {
    apply {}
 }

 control C<H, M>(
     inout H hdr,
     inout M meta,
     in intrinsic_metadata_t intr_md);

 package P<H, M>(C<H, M> c = nothing());

In the last line, the typechecker can't figure out how to show that nothing() has the type C<H,M> since that requires also supplying "default arguments" for the type variables.


Missing module-info.java in okhttp-urlconnection

error: module okhttp3.urlconnection reads package okhttp3 from both okhttp3 and okhttp3.urlconnection
https://stackoverflow.com/questions/62758635/split-package-okhttp3-errors-out-build-if-okhttp-urlconnection-dependency-adde

Not sure we have an easy fix for this.  If you are using Authenticator you can use okhttp.Authenticator.JAVA_NET_AUTHENTICATOR directly.
Not sure about JavaNetCookieJar.

OK, thank you. With Java 17 LTS released, I think it is time to add module support.

@igouss What's your request here?  Something more than Automatic-Module-Name? Or that we fix okhttp-urlconnection to support JavaNetCookieJar?

Well, I've tried to use latest Java, but it seems that they broke everything with the strict module support. I'm still on JVM 11, so I don't 100% know why my app does not work, but with JVM 16/17 I get a lot of errors related to modules. Such as when two jars have files with the same namespace or when jar has no module-info.java.
I'm not 100% sure what I'm need from you ;-) (that's why i've closed the ticket), JDK 17 compatibility of some sort i guess.

The most effective workaround now is probably to copy JavaNetCookieJar into your repository and drop the dependency on okhttp-urlconnection

Tracking here.
https://github.com/square/okhttp/issues/4184


docs: Slider

:tada: This issue has been resolved in version 7.26.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Subtitles in HLS are shown too early

Have you read the FAQ and checked for duplicate open issues?
Yes.
If the problem is related to FairPlay, have you read the tutorial?

N/A
What version of Shaka Player are you using?

4.11.8 / latest
Can you reproduce the issue with our latest release version?
Yes.
Can you reproduce the issue with the latest code from main?
Yes.
Are you using the demo app or your own custom app?
Both.
If custom app, can you reproduce the issue using our demo app?
Yes.
What browser and OS are you using?
Windows 11, Chrome stable and beta.
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
N/A
What are the manifest and license server URIs?

https://cdn.contentflow.net/corporations/cSR2UwvAfK8yti4pErXo/workspaces/HavgCUYVXa6kAea0IkTK/media/44672e62-8555-4728-b131-794c00b2a6a6/index.m3u8
What configuration are you using?  What is the output of player.getNonDefaultConfiguration()?

Plain default of the Demo Player.
What did you do?

Enable the subtitles and press play.
What did you expect to happen?
Show subtitles at the correct timestamp that is defined in the webvtt file.
What actually happened?

There's a subtitle starting within the first second of the video, but is not shown. The second subtitle, starting with second 00:02.880 is shown at the beginning of the 2nd second. The result is that subtitles are asynchronous to the spoken text and will be shown before the speaker speaks the first word of the current subtitle line.
It works fine with HLS.js.
Are you planning send a PR to fix it?
No.

With manifest.hls.sequenceMode = false your stream plays fine

Sure, thank you for the workaround. :)


Feature request: Packet API that supports writing to a mutable Vec

Currently if I want to prepend any bytes to the Vec that is generated by to_vec, I need to create a new Vec, write my desired bytes, then copy the Vec contents generated by to_vec.
However, it would be nice if a Packet had a function such as append(&mut Vec).
This would eliminate the need to copy all the packet bytes, and write directly into the Vec that I have already initialized to my liking.

I have planned changes for the Packet trait, let me know if that's good for your use case.
/// Trait implemented for all toplevel packet declarations.
pub trait Packet: Sized {
    type Parent;

    /// Try parsing an instance of Packet from the its parent representation.
    /// On success, returns the parsed object and the remaining unparsed slice.
    /// On failure, returns an error with the reason for the parsing failure.
    /// If Packet::Self is a root packet (i.e. Packet::Parent is [u8]) then the
    /// behavior of this method is the same as Packet::decode.
    fn decode_partial(buf: &Self::Parent) -> Result<Self, DecodeError>;

    /// Try parsing an instance of Packet from the input slice.
    /// On success, returns the parsed object and the remaining unparsed slice.
    /// On failure, returns an error with the reason for the parsing failure.
    fn decode_full(buf: &[u8]) -> Result<(Self, &[u8]), DecodeError>;

    /// Try parsing an instance of Packet from the input slice.
    /// Returns an error if unparsed bytes remain at the end of the input slice.
    fn decode(buf: &[u8]) -> Result<Self, DecodeError>;

    /// Return the length of the encoded packet.
    fn encoded_len(&self) -> usize;

    /// Write the packet to an output buffer.
    fn encode<T: BufMut>(&self, buf: &mut T) -> Result<(), EncodeError>;

    /// Encode the packet to a byte vector.
    fn encode_to_vec(&self) -> Result<Vec<u8>, EncodeError>;

    /// Encode the packet to a Bytes object.
    fn encode_to_bytes(&self) -> Result<Bytes, EncodeError>;
}

Thanks!
Looks like fn encode<T: BufMut> fits my use case perfectly.
I personally prefer Write over BufMut to save myself an import statement, but that's just me being picky!

BufMut is different in that it implies the write cannot fail, where as Write is for fallible I/O operations. So, I think BufMut is the right choice here since it's targeting a buffer rather than a socket, etc.

Ah! I have overlooked the fallibility of Write.
With that context, I definitely prefer the infallible BufMut.


How to use your display packages in spacemacs

Hi,
I see your blog, and the displays are really amazing.
I want to use it as my private layer. But I see there are some variables depend on other layers in your spacemacs.d, is that possible that I only use your display package, while keep my settings for other things?
BTW, I use macOS.
Thanks a lot!
Songpeng

Hey did you figure this out? I'm a trying to figure out how to use this but it proving to be quiet a challenge. Would you mind pointing me in the right direction?

Each package in the display layer (its local dir) is setup by display/packages.el's display/init-pkg-name functions. They are are all self-contained and separate*.
So installing a pkg is as simple as appending that function into the pkg.el, putting it on your load/path or in your init.el, and running it.
It's ordered this way so my personal settings for each pkg aren't hardcoded into the pkgs themselves. So feel free to mess with any of the contents in the display/init-pkg-name funcs.

Except for the char <-> unicode points set by pretty-fonts which other packages assume are set (otherwise you will see weird chars all-over).

Each package in the display layer (its local dir) is setup by display/packages.el's display/init-pkg-name functions. They are are all self-contained and separate*.
So installing a pkg is as simple as appending that function into the pkg.el, putting it on your load/path or in your init.el, and running it.
It's ordered this way so my personal settings for each pkg aren't hardcoded into the pkgs themselves. So feel free to mess with any of the contents in the display/init-pkg-name funcs.

Except for the char <-> unicode points set by pretty-fonts which other packages assume are set (otherwise you will see weird chars all-over).

Each package in the display layer (its local dir) is setup by display/packages.el's display/init-pkg-name functions. They are are all self-contained and separate*.
So installing a pkg is as simple as appending that function into the pkg.el, putting it on your load/path or in your init.el, and running it.
It's ordered this way so my personal settings for each pkg aren't hardcoded into the pkgs themselves. So feel free to mess with any of the contents in the display/init-pkg-name funcs.

Except for the char <-> unicode points set by pretty-fonts which other packages assume are set (otherwise you will see weird chars all-over).

After hours of trying, I figured it out, copy all the folders from this repo within layers folder to .emacs.d/private/ then add display into your .spacemacs config file in this section dotspacemacs-configuration-layers. @songpeng @ekaschalk Is it possible to just get the mathematics notation only without the other changes? I will appreciate this.:grinning:


rocket.chat icon invisible in menu bar on mac when using dark mode

My Setup

Operating System: MacOS 10.15.2
App Version: 2.16.0
Installation type: AppStore

[x] I have tested with the latest version
[x] I can simulate the issue easily

Description
When using dark mode on mac the rocket.chat menubar item is invisible when there are no unread messages.
Current Behavior
When using dark mode the menubar is black this makes the rocket.chat icon invisible due to it being black. When their unread messages there is a white variant of the icon that is displayed.
Dark Mode Icon invisible screenshot (no unread)

Dark Mode Icon visible screenshot (only visible with unread messages)

Icon visible in light

Expected Behavior
The menubar icon should have a white variant that is used when the system is in dark mode so that the menubar icon is visible at all times, even with unread messages.

Dup of #1386


Add storage with custom url

In this issue it was requested to have the custom url getters for storage.

Looks good, @nbransby - shall we get this in before the next release?


Define default paginate_by

I want to merge this change because having multiple const PAGINATE_BY = 20 annoyed me.
Pull Request Checklist

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] Database migration files are up to date.
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[ ] GraphQL schema and type definitions are up to date.
[ ] Changes are mentioned in the changelog.

@dominik-zeglen changelog pls


SAK-48248 Rubrics: Allow removal of item associations by parsing blank rubric IDs as 0

This PR addresses SAK-48248:
https://sakaiproject.atlassian.net/browse/SAK-48248
RubricsServiceImpl.saveRubricAssociation required a rubric ID in order to save associations, but that value is blank when a user attempts to remove an association. Earle showed me that using NumberUtils.toLong on the blank value would return 0 in this case, which we can then process much more naturally than the try/catch statement that createLong required, as it would throw an error trying to parse the same blank. This code interprets a rubric ID of 0 as an indication that the existing association for the tool and item should be removed ["deactivated"].
I added a couple tests based on Earle's advice, and added another workflow to handle the case of an association that has been removed and needs to be reactivated.

Fixed up the comments as requested, and put some on the API method. Thanks Adrian!


Is onStop hook supposed to work if you close your tab?

If not, how would you detect that?

That would be so great. It would allow easy page transitions before routing.


The computer directly crashes when training on the 4090.

I have been using the current computer with a 4090 graphics card to run code based on the IsaacGym framework without any issues. Recently, I migrated the humanoid task to Genesis, and it trained normally on a 4060 Ti. Today, I uploaded the code to the 4090, but every time I attempt to train, the system freezes, and the mouse and keyboard become unresponsive. Initially, I suspected that the number of environments was too high, so I switched to debug mode, but it still froze.

Sorry to hear that and apologize
Are u rendering anything? Could you provide more information for us to reproduce?

Sorry, I tested some other code frameworks. When using the IsaacGym framework, segmentation faults frequently occurred, and there was one instance where the system froze. It should be a hardware issue. Thank you very much for your response.

I tried again with a computer equipped with an RTX 4090 graphics card. After installing Genesis following the tutorial and attempting to run the example code go2_train.py from the locomotion module, I encountered a "Floating point exception (core dumped)" error. Could you help me resolve this issue?
python version: 3.9
Cuda version: 12.4
Driver version:550.142


Added UserCallbacks, this allows to execute code from the renderthread.

In case you need to execute custom code from the render thread, we can now queue
a command with a callback and a pointer to be executed by the render thread.

Why do you need this?

Hi!
I have a bit of a problem with the library on Android and iOS, I need to capture video properly and the current implementation of readPixels blocks up to a point that turns to be not usable. The problem accentuates the bigger the phone, actually an iphone 6+ gives the worst performance so far due to the large density and size of the display.
I need then to test alternatives to capture video, readback textures are not supported as far as I can tell, so my only hope will be native capturing the video on each platform (that would be the last resort) or trying a bunch of PBO's and reading pixels from them.
In order to test this It will be nice to have some way to push commands inside the command buffer, as a way to hack solutions from the render thread. I could launch my own render thread, and do some stuff after the renderframe (but not all renderframes are actually render frames, like in the initialization or at shutdown where several render frames are issued but are not meant to be captured).
I see two advantages of having the possibility to add custom commands, test features, hack things, and so on, and maybe it can be used to sync application and render frame (I don' need that but maybe it could be used for that).

If it's of any help, thanks to be able to inject commands to the render thread I've successfully captured video on iOS with sustained framerates. It would be very helpful to have a feature like this when the engine can not perform well across all platforms and particular solutions have to be made by the user on very rare occasions.
If there is another way to do this I'm glad to help implement the feature as requested.

Actually it would be good to implement capture that buffers up to 3 frames before start reading those frames.

I'm using glReadPixels but after the complete frame is rendered (I capture even the debug text) this blocks the renderthread, but the main thread keeps pushing the next frame and the cost of capturing the frame is acceptable enough.
I'm not sure how to support OpenGL ES 2.0 where there is no option to read pixels other than glReadPixels as far as I know, on OpenGL ES 3.0 hardware we could use PBO and buffer several frames before reading. How do you propose to use solve the issue on OpenGL ES 2.0?


Define metrics to be collected from Git Commit History

If a company has influence in a project, the majority of authors associated with that company will identify the association in their commit activity.
It's not clear how accurate the company affiliation in the Github profile is.

Name Normalization: This is a free-form text field and the names need to be normalized.
No History: There is no public history of Github profile changes. If a contributor changes companies, all of their activity will be associated with the new company.
Authors could be contributing to the project DESPITE their company affiliation, not because of it.

If an author's contributions are condoned by their company, they should be using company resources to make those contributions. Committers who use their work email address likely have a strong company affiliation or else they wouldn't be using company resources in such a publicly visible space. Committers who do not use their work email likely want to obscure their employer affiliation.
Extracting company affiliation through commit email domain name is sufficient to identify company affiliation both historically and presently.
Commit months is a better indicator of company influence than number of commits.
Because it is not possible to accurately identify company affiliation for every single committer, rankings depending on the number of commits made by affiliated authors are insufficient for determining a company's involvement with a project.

A prolific committer can easily skew total commits
Committers identified through Github profiles have no historical context (see above)

Commit months per company
On the other hand, measuring whether a company had at least one author committing to the repository using their company email address will indicate whether or not the company had any involvement for a given time period. This also provides a clear historical pattern of activity, particularly how long the company has been represented in the project.
Commit months per author per company
Evaluating whether an affiliated author had at least one commit using their company email address indicates how much time a company is allowing its affilated authors to work upstream. If the company allows a significant proportion of their time to be spent upstream, the authors will have more consistent commit months. This also is not susceptible to potential skewing in the same way as numbers of commits.
If the majority of authors has only one commit month, it suggests that while a company may have a large number of commits and a large number of authors, they may not be consistently engaging with the community.
Project Participation
Commits percent change: how proportion of commits changes over time (what proportion is a min/max)
Commit months per author is an interesting metric for estimating project popularity.
Additionally, non-human users in projects with a high proportion of drive-by authors can be identified by a significantly higher proportion of commits than the other authors. The activity in this case is masked by an internal Google process so it's not clear what analysis would be beneficial without something else to compare to. However, projects with open governance that has clear "bot users" could be analyzed and used as a foundation for comparison.

Next steps, fix up this analysis and run it on other repos in the list.


figure out how to produce replacement tags

It may be possible to send National Band and Tag a list of numbers to make. Jess sent them a message to check.
Jess : "I know someone tried engraving once, but it was very hard to be steady and get deep enough. This may be a better tool and worth a try though!" . Refers to this tool:  https://www.amazon.com/FOLAI-Electric-Engraving-Engraver-Jewellery/dp/B01M6X5KWV?th=1

For stems ≥10cm needing re-tags in the 2022 mortality census, we can order pre-printed tags (#7).
For others, we will need a way to engrave.

We are ordering the tags that need replacement, and will try out an engraver.

We tested this tool and decided it will work. It doesn't engrave as deeply or cleanly as the original tags, but is overall a better option than stamping (also not as deep as original tags, hard to do in the field).


Proxy VM is not sending heartbeats in certain circumstances

Information

Onefuzz version: 2.1.0
OS: Windows (CLI)

This happens in our production instance which you may be familiar with c***rod05, but it does not seem to happen on our less busy or newer instances.
Provide detailed reproduction steps (if any)

Run the 'debug' command
$ onefuzz.exe debug node rdp

Connect as you would if you wanted to inspect some onefuzz logs on the node

Browse the web, watch TV, or just leave the RDP session open

After 10 to 15 minutes you will be disconnected.

You can recreate the debug session using the same command in Step 1, and will see the node in the same state it was before you disconnected.

Upon further investigation, the instance activity log shows the App Function (System-Assigned Managed Identity) has deleted the Proxy VM and that App Insights did not log a heartbeat for 10 minutes before deleting the Proxy VM. So partly, the behavior was expected. That is why we think this issue has something to do with the heartbeats.
Expected result
We expect the Proxy VM to send heartbeats and then for the App Function to process them so it does not delete the Proxy VM.

I can reproduce this.  Investigating.

This apparently isn't fixed.  #520 will help debug this further.

No repro

no repro in latest version I meant


Container name update issue fixes when migration from one CDN to another

 SPYD - Investigate and fix issues of the file-uploader plugin for migrating files from one CDN to another and update the README


Replace class with interface

@kwonoj ruh oh. Halp?

@kwonoj what failed?

I think is that TravisCI is "connected" to the repo but not yet configured. And it fails by default :(
I will try and create a first configuratuon by tomorrow

@ladyleet Yes, I enabled it but haven't created configs for if anyone interested in contribution. https://github.com/ReactiveX/rxjs-docs/issues/18 . I can do if anyone's not interested in.

@kwonoj can we merge this now?

@ladyleet :ship: I'll enable protected branch shortly too.


in_app_purchase: ^0.3.4+10 crashes when fetching product's detail from the Google Play Store

I have already add billing library "com.android.billingclient:billing-ktx:3.0.0" to my gradle file , uploaded app to google play as production and defined some of my in app products in my play console. After then I start implementing the flutter code, fetching product's detail from the google play console, my app crash and I got those error messages in my log cat.
java.lang.NoSuchMethodError: No virtual method getIntroductoryPriceCycles()Ljava/lang/String; in class Lcom/android/billingclient/api/SkuDetails; or its super classes (declaration of 'com.android.billingclient.api.SkuDetails' appears in /data/app/com.test.inapppurchase/base.apk)
E/AndroidRuntime(20258): 	at io.flutter.plugins.inapppurchase.Translator.fromSkuDetail(Translator.java:27)
E/AndroidRuntime(20258): 	at io.flutter.plugins.inapppurchase.Translator.fromSkuDetailsList(Translator.java:49)
E/AndroidRuntime(20258): 	at io.flutter.plugins.inapppurchase.MethodCallHandlerImpl$1.onSkuDetailsResponse(MethodCallHandlerImpl.java:185)
E/AndroidRuntime(20258): 	at com.android.billingclient.api.zzj.run(com.android.billingclient:billing@@3.0.0:8)
how can I solve that?
please

Hi @yetunnaing18121996
I see there's an open issue (https://github.com/flutter/flutter/issues/59400) addressing the case you described.
Please follow up on that issue, I'm closing the current one as a duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you


support labels in compose up --build

Thanks
Can we support labels in compose up --build too ?
<EMAIL_ADDRESS>
Originally posted by @AkihiroSuda in https://github.com/containerd/nerdctl/pull/653#pullrequestreview-840205643

/assignme


set secure flag on cookies when using https

I haven't been able to test this but it looks like what we need

Shouldn't this be on at all times, not just when HTTPS is set?

Only works on https

Yeah, the application won't send the cookies if you don't have HTTPS on otherwise, right?


feat: Add Redis Chat Memory

This PR is about including the Redis component as a cache for chat memory.

Thanks @MarceloNunesAlves ! Sorry for the slow response. LGTM

No problem! Thank you so much.


Scoll of the main page gets lost after Drawer closes.

So, I am using this, the problem the scroll of the page over which drawer draws gets lost after drawer is closed.

Do you mean that it jumps to the top?


Interfaces authentication and multi-user

All the interfaces have been protected by JWT based authentication system, with only logged-in users can access the configs and make modifications to announcements and configs of his own. All the authentications are based on user, and the sign-up and log-in interface have been added, which enables multiple users to share the same backend while still supporting a single user running uvicorn locally.

/browser should open the link in frontend instead of backend, since backend can be put on a server instead of running locally

Generally it should open in the frontend but in wallpaper engine it bans user from opening web browser so maybe we can set this as an option in config file.

You need to add new dependencies like passlib to requirements.txt. Test it in a new environment (clone in another folder, create venv).
I have error when starting:
  File "auth.py", line 51, in gen_key
    with open(secret_file, 'w') as f:
         ^^^^^^^^^^^^^^^^^^^^^^
FileNotFoundError: [Errno 2] No such file or directory: './canvas/.secret'

You need to add new dependencies like passlib to requirements.txt. Test it in a new environment (clone in another folder, create venv).
I have error when starting:
  File "auth.py", line 51, in gen_key
    with open(secret_file, 'w') as f:
         ^^^^^^^^^^^^^^^^^^^^^^
FileNotFoundError: [Errno 2] No such file or directory: './canvas/.secret'

If you encounter this, please add a folder called canvas, where there are two empty folders, one called user_configs and the other called user_caches. The .secret file will be automatically written into the canvas directory. This FileNotFoundError seems to be because the canvas folder is ignored by .gitignore (of course files in it MUST be ignored, but the empty folder should be there.). I will fix it.

Hi, could you push code to a branch in this repo rather than in this PR?

ok. I forgot to close this PR. I am closing it now. I will open a new branch as soon as I fix the bugs.


[NoTargets] Projects referencing a NoTargets project do not see ResolvedProjectReference items

In VS, the dependencies tree relies on the design-time build producing resolved items for a projects references in order to display them as resolved in the tree.
When a project references a NoTargets project, it does not see such a resolved item and hence the yellow triangle in the dependencies tree in solution explorer remains visible:

The dependencies tree obtains these items from this target:
https://github.com/dotnet/project-system/blob/eb4e01a2d65fae7b0d466718356e7b9c5a175dcd/src/Microsoft.VisualStudio.ProjectSystem.Managed/ProjectSystem/DesignTimeTargets/Microsoft.Managed.DesignTime.targets#L389-L408
Something about the NoTargets project means that this reference is not handled by this target.

Thanks for reporting @drewnoakes.  Do you see anything obvious in the NoTargets logic that would be causing this?
https://github.com/microsoft/MSBuildSdks/tree/main/src/NoTargets/Sdk

@jeffkl nothing jumps out at me. Diffing binlogs of a bare project with a single project reference to a NoTargets vs. a regular SDK project should reveal the issue. You might like to copy this target into the referencing project and ensure it's executed, then work back from there.


Fix/3.0/local shuffle disable

What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes:

Fixes #
Problem Summary(Required):

This is cherry-picked from #23027.
Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 3.0
[x] 2.5
[x] 2.4
[x] 2.3

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.23 out of 24 committers have signed the CLA.:white_check_mark: Smith-Cruise:white_check_mark: evelynzhaojie:white_check_mark: trueeyu:white_check_mark: EsoragotoSpirit:white_check_mark: gengjun-git:white_check_mark: amber-create:white_check_mark: zombee0:white_check_mark: stephen-shelby:white_check_mark: GavinMar:white_check_mark: liuyehcf:white_check_mark: packy92:white_check_mark: stdpain:white_check_mark: QingdongZeng3:white_check_mark: Seaven:white_check_mark: hellolilyliuyi:white_check_mark: decster:white_check_mark: alberttwong:white_check_mark: rickif:white_check_mark: blanklin030:white_check_mark: ZiheLiu:white_check_mark: silverbullet233:white_check_mark: Youngwb:white_check_mark: wyb:x: mergify[bot]You have signed the CLA already but the status is still pending? Let us recheck it.


Improving multi distro support.

This improves deployment on multiple distributions.  Slave deployment on ubuntu systems using newer bits is now supported.

:-1: for this PR in its present form.  The history needs to be squashed and the merge commits removed.  The
dependency being added on  openstack-hyper-v/windows_common is for a module not published on the forge -- which would make this module uninstallable via PMT.  I object to the inclusion of the jenkins-slave.exe binary, even though it is small.  I am concerned about the way jenkins.model.JenkinsLocationConfiguration.xml is being managed since replace is not false and the template can not be overridden.  It also goes against the pattern this module has of managing global configuration via the cli jar.  Debian/RHEL init scripts are already present.

@ppouliot I'm going to close this PR due to the issues I noted in my previous comment.  I would be happy to work with you in the future on adding windows support.


OpenSSL pared the issuer and subject to hexadecimal while GnuTLS and ZCertificate parsed them to local characters

OpenSSL 1.1.1
Ubuntu 18.04 x64
openssl x509 -in path/to/cert_file -noout -text
OpenSSL pared the issuer and subject to hexadecimal while GnuTLS and ZCertificate parsed them to local characters.
cert-example-ie-cer08.zip

It's a feature request, not a buhg.

Marking as inactive, to be closed at the end of 3.4  dev, barring further input


Docker "latest" and default pull not working

This is related to the docker build, so hopefully this is still the right place to post this.
🐛 Bug Report
When attempting to pull either ducktors/turborepo-remote-cache or ducktors/turborepo-remote-cache:latest you get Error response from daemon: manifest for ducktors/turborepo-remote-cache:latest not found: manifest unknown: manifest unknown
If you try to pull a specific version like 2.1.1, then it works correctly. Up until recently the above commands used to work and they are the ones listed as examples on the docker hub page.
To Reproduce
Steps to reproduce the behavior:
docker pull ducktors/turborepo-remote-cache

Expected behavior
It should succeed and not error
Your Environment

os: Linux, but i don't believe this is OS specific

@smmccabe, thanks for reporting this. @matteovivona any clues?

@smmccabe did you try with ducktors/turborepo-remote-cache:2.2.1? unfortunately, the official docker action cannot push the latest on multi arch