fix issue #70 in token.py

Fix issue #73.
It makes the u8 function accept negative value as for u16, u32 and u64.

Codecov Report

Merging #76 into master will decrease coverage by 0.00%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master      #76      +/-   ##
==========================================
- Coverage   78.48%   78.47%   -0.01%
==========================================
  Files         343      343
  Lines       44943    44943
==========================================
- Hits        35272    35271       -1
- Misses       9671     9672       +1

Impacted Files
Coverage Δ

ppci/arch/token.py
95.03% <66.66%> (-0.57%)
:arrow_down:

ppci/cli/base.py
83.76% <0.00%> (-0.14%)
:arrow_down:

ppci/lang/tools/grammar.py
100.00% <0.00%> (ø)

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 881cbd1...0bb845a. Read the comment docs.


AuthenticationApi.verify2FA not working due to incorrect data format

AuthenticationApi.verify2FA fails returning a HTTP status code 400 due to incorrect data format in the request, in the code the format is the following:
vrchat/dist/api.js - line 627
localVarRequestOptions.data = (0, common_1.serializeDataIfNeeded)(twoFactorAuthCode, localVarRequestOptions, configuration);
api.ts - line 7116
localVarRequestOptions.data = serializeDataIfNeeded(twoFactorAuthCode, localVarRequestOptions, configuration)
According to the VRChat API Docs (https://vrchatapi.github.io/docs/api/#post-/auth/twofactorauth/totp/verify) the code should be nested:
vrchat/dist/api.js - line 627
localVarRequestOptions.data = {
  code: (0, common_1.serializeDataIfNeeded)(twoFactorAuthCode, localVarRequestOptions, configuration)
}

api.ts - line 7116
localVarRequestOptions.data = {
  code: serializeDataIfNeeded(twoFactorAuthCode, localVarRequestOptions, configuration)
};

Making the above change fixes the issue,  it looks like the same issue may effect verify2FAEmailCode and verifyRecoveryCode looking at the code but I haven't tested and confirmed this.

The path of the URL is invalid. It should be /auth/twofactorauth/totp/verify. In the codebase, it is /auth/twofactorauth/otp/verify. Also, you are right.

The path of the URL is invalid. It should be /auth/twofactorauth/totp/verify. In the codebase, it is /auth/twofactorauth/otp/verify. Also, you are right.

mb i was looking at verifyRecoveryCode instad of verify2FA...


Auto tag docker images

closes https://github.com/woodpecker-ci/woodpecker/issues/3131

That's great overall, I forgot about the issue.
I would need to lookup the exact issues, but already the -alpine suffix might create problems.
As this runs only on tag, it is kinda dangerous to use it without proper testing. Personally I moved all my repos away from auto_tag some time ago. Imo it needs a proper refactor in the plugin.


Revert urllib3 version pin

Hi. We are trying to integrate with OIP model servers over at feast feature store and need to add mlserver and tritonclient as optional dependencies. The problem is that we also already depend on snowflake-connector-python which still has a strict urllib3<2.0.0 requirement for python 3.9. I saw that urllib3 version pin here was only to avoid vulnerability reports. #457  Not sure which vulnerability that was referring to, but seems like urlib3 plan to ship security fixes for v1 still. Is is possible to revert the version pin? Or maybe allow something like (>=1.26.18<2 or >=2.0.7). Thanks

@nvda-mesharma @mc-nv in this PR: https://github.com/triton-inference-server/client/pull/457, we updated the urllib3 requirement.
Is this something we can investigate?

CVE-2023-45803 prevent locking version as urllib3<2.0.0.

@mc-nv CVE says that This issue has been addressed in versions 1.26.18 and 2.0.7. I know it's not pretty, but are you opposed to pinning like this urllib3>=1.26.18,!=2.0.2,!=2.0.3,!=2.0.4,!=2.0.5,!=2.0.6? (Or maybe there's some other way to write this.. but you get the idea)

Sorry to bother you guys once again. Have you had an opportunity to look into this?

Its not bothering us. We just have to be careful with the CVE issue. Let me ping one more person.
@nvda-mesharma is this something we can accommodate?

Dear customer we understand your request and have to clarify following for you.
According to Python Packaging User Guide#Version Specifier documentation both version are valid as you mentioned above.
Question that raise for us, is why https://pypi.org/project/urllib3 project carry two different semantic versions (see https://semver.org/ )?
Suppose it cause because these two version can carry different set of API with a similar functionality.
We may not be impacted, but it will require rebuild entire product and make sure that none of the other customers is impacted.

@mc-nv Hey, thanks for the clarification. Regarding semver, I'll note here a quote from urllib3 v2 migration guide:

The primary goal for migrating to urllib3 v2.x should be to ensure your package supports both urllib3 v1.26.x and v2.0 for some time

And their recommended version specifier during the migration is "urllib3>=1.26,<3" (which will have to be modified a bit because of the CVE, of course).

Hey guys. Just pinging if there's any update on this?

@nvda-mesharma any idea what is happening on this? I am out of the loop on this now

We are also running into this in an application that uses boto3. Boto requires urllib3<2.0.0.

Any progress on this issue?

Following up again. This issue is blocking our ability to update the client package.


fix install.sh when sudo is required to delete /usr/local/bin/kubescape

@matthyx just confirming - did we test the fix on different OS?

@matthyx just confirming - did we test the fix on different OS?

MacOS (10.13 and 10.14) and Linux - I don't have a Windows machine to test on MinGW.

There is another issue (saw it on Windows) the cleanup of kubescape deletes the location where the file is downloaded (probably a path detection issue).


JSON API missing release

Using the warehouse API https://warehouse.python.org/pypi/pytest/json the response has no info on latest py.test release. The end of the response (as of now) looks like:
   "2.7.1": []}, "urls": []}

The response from https://pypi.python.org/pypi/pytest/json is ok.
I would guess this is a cache issue, so might not be easily reproducible in the future!

This is because warehouse.python.org was broken and a new release that has the fixed code hasn't yet been deployed. Going to go ahead and close this since the problem doesn't exist in the current master branch.


Support for b2c User/Local Account in GraphRbacManagementClient.users.create

Package Name: azure-graphrbac

Is your feature request related to a problem? Please describe.
I would like to be-able to create a Local/Azure b2c user, i don't think we can currently do this.
Describe the solution you'd like
Not sure if this is the right place or terminology but is it possible this can support UserCreateParameters not requiring user_principal_name and rather taking the creationType/signInNames properties so we can make a Local User or Azure b2c User.
Describe alternatives you've considered
Possibly manually doing it via REST request or extracting the auth from this lib at-least.
Additional context
Relevant documentation of API:
https://docs.microsoft.com/en-us/previous-versions/azure/ad/graph/api/users-operations#CreateLocalAccountUser
Similar issue here:
https://stackoverflow.com/questions/55617626/creating-new-ad-b2c-user-via-microsoft-graph-api

Need help from graph team.


Missing dependency (line_profiler, ali-dev branch)

line_profiler should be added to the dependencies.
For reproduction: Follow the install instructions on the ali-dev branch. Run the streamlined_detections.py script.

ModuleNotFoundError: No module named 'line_profiler'

Ah sorry about that. This should actually not be in the requirements because thats for benchmarking, and isnt required to use the system.
The line should be commented out (ideally removed at some point during a cleanup). Where (which file and line) was the error coming from? Either way just comment out any line profiler related things.

closing due to inactivity. if anyone else has that problem or you find the problematic line in the code lmk.


Delete obsolete code

Since we started using Larray there is still a lot of unnecesery offheap memory.

Most of old code has been removed. There is still some in terra-memory but it should be deleted as well,


 System Linux 4.4.21-v7+
gyp ERR! command "/usr/bin/nodejs" "/usr/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /usr/lib/node_modules/homebridge/node_modules/mdns
gyp ERR! node -v v7.10.1
gyp ERR! node-gyp -v v3.5.0
gyp ERR! not ok
/usr/lib
└── (empty)
npm ERR! Linux 4.4.21-v7+
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "install" "-g" "--unsafe-perm" "homebridge"
npm ERR! node v7.10.1
npm ERR! npm  v4.2.0
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the mdns package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs mdns
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls mdns
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /root/.npm/_logs/2016-09-23T05_42_51_541Z-debug.log

I got this error when my system time was wrong. Your log date is 2016, check it. :)


Drop dependency on git.exe

Use LibGit2Sharp instead

This tool doesn't make a lot of sense if git.exe isn't available. I've been looking into this, and it complicates the code for little benefit. I'm removing this from v0.4.0 and may eventually mark this as won't fix.


More fixes for the host report

Added the ability to add erratum message and number via env variables:

With the number (only affects title):

I'm also proposing the following message as Erratum for the latest host report issue:

We're currently changing how we record transactions to make everything easier to understand, more standard, and scalable. As part of these changes, we missed an update in the previous "2021/6 Monthly Host Report" email that translated into showing wrong numbers in the breakdown below for Host Fees and Host Fee Shares.
This email includes the fixed numbers. Feel free to let us know if you see anything wrong or if you have any questions!


test/integration: Add Ruby Integration tests

closed with https://github.com/parca-dev/parca-agent/pull/2501


fix: open URL in browser on WSL

Here's a more elegant solution for opening URLs from WSL in your favorite browser. It is based on sensible-browser which is included in the default distribution for WSL. This avoids issues with the WSL environment, cmd.exe. quoting parameters, etc.
In WSL, set the default browser using the BROWSER variable, for example,
export BROWSER="/mnt/c/Program Files (x86)/Google/Chrome/Application/chrome.exe"
or
export BROWSER="/mnt/c/Program Files (x86)/Microsoft/Edge/Application/msedge.exe"

Note: To permanently set the default browser, add the appropriate entry to your shell's RC file, e.g. .bashrc or .zshrc.
To launch a URL from the WSL command line:
sensible-browser https://google.com

To launch a URL using promise-spawn:
const promiseSpawn = require('@npmcli/promise-spawn')
promiseSpawn.open('https://google.com')

Replaces #118
Closes #62
Test
os: <IP_ADDRESS>-microsoft-standard-WSL2
node: 20.18.0
npm: 10.8.2

If BROWSER is not set, it now rejects the promise with a message similar to what you get in WSL cli:

I adjusted the tests accordingly


B 21365 int

Agility ticket
Summary
Is there anything you would like reviewers to give additional scrutiny?
this article explains more about the approach used.
Verification Steps for the Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Have the Agility acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

[ ] Has the branch been pulled in and checked out?
[ ] Have the BL acceptance criteria been met for this change?
[ ] Was the CircleCI build successful?
[ ] Has the code been reviewed from a standards and best practices point of view?

Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test

Access the
Login as a

Frontend

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, Edge).
[ ] There are no new console errors in the browser devtools.
[ ] There are no new console errors in the test output.
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.
[ ] This change meets the standards for Section 508 compliance.

Backend

[ ] Code follows the guidelines for Logging.
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Database
Any new migrations/schema changes:

[ ] Follows our guidelines for Zero-Downtime Deploys.
[ ] Have been communicated to #g-database.
[ ] Secure migrations have been tested following the instructions in our docs.

Screenshots

Warnings

:warning:

Files located in legacy directories (src/shared or src/scenes) have
been edited. Are you sure you don’t want to also relocate them to the new file structure?
View the frontend file org ADR for more information

  Generated by :no_entry_sign: dangerJS against db8f79b2c357cb6dc28a5cceaf9258467c736afd

Reviewed code and test case passed, except for deleting the UB Shipment. Clicked the delete button but nothing happened. ACs have been met.

@TevinAdams  Looks like you tried to delete as the office user - this is only for the customer. I just tested again and delete works as the customer


Bulk Import bug

Hi,
type of import function should be defaults to "auto". However, if I do not give opts argument, and then import an array of document, I will become such error:
[ArangoError: no JSON string array found in first line]

I believe it's still a bug of request parameter. Please fix.
Bg,
Kevin

Does it work if you specify type: "auto" explicitly? I think the default value was accidentally dropped when I added support for JSON-LD (JSON "streams").

Yes, it work, if specify type: "auto"explicitly.

Fixed in 3.3.1.


webapi batch requests process, such as asp.net framework for webapi

This feature is not currently supported.
I wonder if such a feature might be partially or fully replaced by HTTP/2 multiplexing.

ASP.NET Core 2.2 is to be released later this year.

Is HTTP/2 multiplexing feature in preview yet in ASP.NET CORE 2.2?
@Eilon

@Microshaoft yes, it has released.


Visitor for authorization logic AST

This is exactly the same as  #643 which I did not bring into google3 correctly.

Nope there is a better way!


how can i integrate umijs/antd to my map built with es modules

how can I integrate umijs/antd to my map built with es modules

Failed to compile
1.Errors, such as
Module not found: Error: Can't resolve 'C:/simplegis/jyjjport/node_modules/@umijs/babel-preset-umi/node_modules/@babel/runtime/helpers/esm/objectSpread2' in 'C:\simplegis\jyjjport\node_modules@arcgis\core'
Did you mean 'objectSpread2.js'?
BREAKING CHANGE: The request 'C:/simplegis/jyjjport/node_modules/@umijs/babel-preset-umi/node_modules/@babel/runtime/helpers/esm/objectSpread2' failed to resolve only because it was resolved as fully specified
(probably because the origin is a '.mjs' file or a '.js' file where the package.json contains '"type": "module"').
The extension in the request is mandatory for it to be fully specified.
Add the extension to the request.
2.Compile info
[BABEL] Note: The code generator has deoptimised the styling of<EMAIL_ADDRESS>as it exceeds the max of 500KB.
[BABEL] Note: The code generator has deoptimised the styling of<EMAIL_ADDRESS>as it exceeds the max of 500KB.
[BABEL] Note: The code generator has deoptimised the styling of<EMAIL_ADDRESS>as it exceeds the max of 500KB.
ERROR  Failed to compile with 5000 errors (refer above)

Hi @songfaqi My suggestion is to tackle those errors one at a time. The errors seems to be configuration issues, for example:

Module not found:  this looks like a babel configuration issue or a bug in babel
BREAKING CHANGE: this looks like a possible webpack configuration issue, most likely need to specify a module.rules.test  object, for example:

  module: {
    rules: [
           {
              test: /\.m?js/,
              resolve: {
                  fullySpecified: false
              }
            },

Babel calcite-component deoptimization - that's just a warning as far as I'm aware. Check the docs for babel compact https://babeljs.io/docs/en/options#compact.
geometryEngineBase.js size - same check the babel doc for compact
@arcgis\core\chunks\index.js size - same, check the babel for compact

@odoe do you have any additional thoughts?

I'd also suggest not processing the modules with Babel unless you have to.

@dasa suggestion best, something like the object spread error might be fixable via browserlist using latest configuration.

Closing.


Added sort parameter to feature_importance

Pull Request Test Coverage Report for Build 75

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.485%

Totals

Change from base Build 66:
0.0%

Covered Lines:
427

Relevant Lines:
429

💛  - Coveralls


[Question] Using ResumableZoom inside a ScrollView, how to disable scroll when pinching?

Hello Again! As I comment on another issue I'm using react-native-snap-carousel to develop a photo gallery with zoom behavior.
SnapCarousel uses ScrollView with horizontal scroll and snap enabled. Each item of the carousel is a ResumableZoom with pinch and double tap enabled with 4x zoom scale.
On Android, when I start a pinch gesture, moving two fingers together, works properly. But if I move only 1 finger even if I have 2 touch points, the pinch is not detected and ScrollView starts scrolling to the next or previous photo depending on the direction of the moving finger.
I try to convert the SnapCarousel to an animated component but can't find a way to make it work.
Is there any advice that you can give me in this situation?

Let's start for most basic question, why are using ResumableZoom instead of the already built-in Gallery?

It's an ongoing project that already has a carousel gallery. I tried to negotiate to change the carousel lib but didn't have success. Because of this, I'm trying to make things work together.
But simply, I have the following structure (I'm going to create a snack to better tests):

Gallery Component
function Gallery({ index, photos, onLoadEnd, gallerySync }) {
  const [currentIndex, setCurrentIndex] = useState(index);
  const [isZoomEnabled, setZoomEnabled] = useState(false);
  const resumableZoomRef = useRef<ResumableZoomType[]>([]);
  const timeoutID = useRef<NodeJS.Timeout>();

  const navigation = useNavigation<StackNavigationProp<ParamListBase>>();

  const { width } = useWindowDimensions();

  const { isLandscapeOrientation } = useDeviceOrientation();

  function setGalleryIndex(newIndex: number) {
    if (currentIndex !== newIndex) {
      resumableZoomRef.current[currentIndex].reset(true);
    }
    gallerySync?.(newIndex);
    setCurrentIndex(newIndex);
  }

  function onResizeForIndex(itemIndex: number) {
    return function onResize(scale: number) {
      if (itemIndex !== currentIndex) return;

      if (timeoutID.current) {
        clearTimeout(timeoutID.current);
        timeoutID.current = undefined;
      }

      timeoutID.current = setTimeout(() => {
        setZoomEnabled(scale !== 1);
      }, 500);
    };
  }

  const onPanEnd = useCallback((item: PhotoItem) => {
    tracker.trackEvent('partner_gallery_photo_scrolled', {
      photo_position: index,
      photo_id: item.id,
    });
    // eslint-disable-next-line react-hooks/exhaustive-deps
  }, []);

  const onPinchEnd = useCallback((scale: number, item: PhotoItem) => {
    const event = scale > 1 ? 'zoom_in' : 'zoom_out';
    tracker.trackEvent(`partner_gallery_photo_${event}`, {
      photo_position: index,
      photo_id: item.id,
    });
    // eslint-disable-next-line react-hooks/exhaustive-deps
  }, []);

  // This is the component that uses ResumableZoom
  function renderItem({ item, index: itemIndex }) {
    return (
      <Photo
        item={item}
        panEnabled={isZoomEnabled}
        pinchEnabled={enablePartnerGalleryZoom}
        tapsEnabled={enablePartnerGalleryZoom}
        onLoadEnd={onLoadEnd}
        onPanEnd={() => onPanEnd(item)}
        onPinchEnd={e => onPinchEnd(e.scale, item)}
        onZoomUpdate={onResizeForIndex(itemIndex)}
        ref={el => {
          if (el) resumableZoomRef.current[itemIndex] = el;
        }}
      />
    );
  }

  const style = StyleSheet.absoluteFillObject;

  return (
    <Box alignItems="center" justifyContent="center" style={style}>
      <SnapCarousel
        enableSnap
        enableMomentum
        horizontal
        useScrollView
        scrollEnabled={!isZoomEnabled}
        initialNumToRender={1}
        maxToRenderPerBatch={2}
        sliderWidth={width}
        itemWidth={width}
        data={photos}
        firstItem={currentIndex}
        keyExtractor={(item: PhotoItem) => item.id}
        renderItem={renderItem}
        onBeforeSnapToItem={setGalleryIndex}
        onSnapToItem={setGalleryIndex}
      />
      {photos.length > 1 && (
        <Dots length={photos.length} current={currentIndex} />
      )}
    </Box>
  );
}

Photo Component
export const Photo = forwardRef<ResumableZoomType, PhotoProps>(
  (
    { item, panEnabled = false, onLoadEnd, onZoomUpdate, ...zoomProps },
    ref,
  ) => {
    const resumableZoomRef = useRef<ResumableZoomType>(null);

    useImperativeHandle<Partial<ResumableZoomType>, Partial<ResumableZoomType>>(
      ref,
      () => ({
        reset: (...args) => resumableZoomRef.current?.reset(...args),
      }),
      [],
    );

    const { isLandscapeOrientation } = useDeviceOrientation();

    const { height, width } = useWindowDimensions();
    const { resolution, isFetching } = useImageResolution({ uri: item.uri });

    function onUpdateHandler({ scale }: CommonZoomState<number>) {
      'worklet';

      runOnJS(onZoomUpdate)(scale); // Used to listen to zoom changes
    }

    if (isFetching || resolution === undefined) {
      return (
        <Loading
          testID="gallery-photo-loading"
          hardwareAccelerationAndroid
          autoPlay
          loop
        />
      );
    }

    const imageStyle = getAspectRatioSize({
      aspectRatio: resolution.width / resolution.height,
      width: isLandscapeOrientation() ? undefined : width, // Set width and height based on device orientation
      height: isLandscapeOrientation() ? height : undefined,
    });

    return (
      <Box flex={1} testID="gallery-image-container">
        <ResumableZoom
          extendGestures
          panEnabled={panEnabled}
          maxScale={3}
          scaleMode="clamp"
          onUpdate={onUpdateHandler}
          ref={resumableZoomRef}
          {...zoomProps}>
          <Image
            accessibilityRole="image"
            resizeMethod="scale"
            onLoadEnd={onLoadEnd}
            source={{ uri: item.uri }}
            style={imageStyle}
          />
        </ResumableZoom>
      </Box>
    );
  },
);

SnapCarousel uses ScrollView with horizontal scroll and snap enabled, but I can't pass an Animated.ScrollView.
I can't pinch in any direction in Android because some of those enable swiping on ScrollView.
https://github.com/user-attachments/assets/30e40002-ad61-40fd-aa9c-a0c041b5e6b9
I tried wrapping SnapCarousel with a GestureDetector and configuring a way to disable pinch only on ScrollView. I also tried negating ScrollView to be the move responder (probably incorrectly). I tried adding disableScrollViewPanResponder in SnapCarousel and tried to detect more than two touch points to disable the scroll...
None of this works. I don't have ideas to fix this issue.

The lib you guys are using does not let any room for a work around, gestures and scroll are by nature conflicting behaviors and whichever triggers first negates the other.
Talking of ResumableZoom, I designed it with scroll-able usage in mind (Flatlist) however it requires the developer to create it's own scroll logic which in itself is not only painful but a time consuming task, if you're brave enough you could attempt such task by using onOverPanning property, this of course breaks the convenience of having a scroll library if you have to write scroll logic yourself.
My best advise is to negotiate again and let know however takes the final decision that you can have scroll or zoom but not both as the zoom capabilities need to be tightly integrated with the scroll-able component itself in such a way they do not collide with each other.

Thanks. That was my suspicion and this library is no longer maintained but has been used in the app for a while. I found another one that uses gesture-handler and reanimated instead of ScrollView.
What would be the best approach in your opinion, switching to one like the one I mentioned above or using your library's gallery together with ResumableZoom?
Basically I just need to have the 4 gestures working together: double tap, pinch both to zoom, pan when zoom applied, and swipe to change image when zoom is not applied. Also, working fine in all orientations.

If you're speaking of react-native-reanimated-carousel I've never tested myself, all I know is the most nested gestures have the bigger priority, I think you would not be able to scroll because of the pan gesture attached to ResumableZoom, this is only a guess based of experience working with the lib, however you're free to give a try.
Talking of the gallery shipped with the library this one does not require ResumableZoom, all gestures are already part of the gallery so you just need to pass your images (Not wrapped by any zoom component), all your items share a single instance of ResumableZoom this made for performance reasons, when I mean zoom and scroll need to be tightly integrated this is what I meant because the zoom component needs to drive the scrolling.
The only problem I think you may encounter is the orientation thing, so try it out and let me know.


Missing file

Hello，I want to realization this project，but lack genNN.h and kernel_buffer.h and tinyengine_function.h
How to get this files？

Hi, we are still cleaning the TinyEngine part codebase. Stay tuned!

@tonylins Hello, this is an amazing project, very appreciate for the work you've done. I'm also trying to realize mcunet, but got failed. So could you provide a tutorial on it? thank you.

Hi, we are currently cleaning the code for TinyEngine which will be released under the refactored repo: https://github.com/mit-han-lab/mcunet. Sorry for the waiting!


[FIX]: To handle seccomp correctly in profile command

Signed-off-by: Mritunjay Sharma<EMAIL_ADDRESS>Fixes-#308
What
docker-slim profile was creating buggy seccomp profile and was not working as expected while docker-slim build works fine.
This PR fixes #308
Why
This might be because some parts of the handler in profile and build command were not aligned
How Tested
TBD

thanks @mritunjaysharma394 !


Don't work at all

birkhoff at Birkhoff-MBPR in ~/dev/tesasd
$ bash-obfuscate
env: node\r: No such file or directory

$ bash-obfuscate 123.sh -o 321.sh
env: node\r: No such file or directory

This was caused by Windows-style CRLF line endings in bin/cli.js. Funnily enough, I only tested this on a Windows computer (and ran the output of the script through a bash shell).
I fixed this in 83822be (v1.1.1) by normalizing line endings of .js files.

Thanks for the fast reply. I'll test later today and see if it works.
On Jun 15, 2017, 7:34 AM +0800, William Shiao<EMAIL_ADDRESS>wrote:

Closed #1.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Prepare for 0.6.0

:+1:
Already shipped the gem, but can't push to master ;)

1 Warning

:warning:
Tests were not updated

  Generated by :no_entry_sign: danger


Problem with auto-scrolling the last active element on the list

There is problem with auto-scrolling highlighted element for list with infinite scroll (select2-more-results). The problem occurs when I try to select last element which is not disabled, but below there are disabled items.
Registered behavior:

This is directly related to https://github.com/select2/select2/issues/1879 and isn't something I have plans to fix in the 3.5.x branch.
The highlight handling is pretty nasty there and it's handled in a different way in 4.0.0.


Alias path is reflected when installing shadcn-ui component

Description:
When installing a component of shadcn-ui, the path set in alias's components is reflected as it is. Therefore, the import path of cn in the generated component does not work properly.
Steps to reproduce:
Install shadcn-ui
Install any component
Import and use the component

Error message:
Module not found: Can't resolve '/path/to/component'
Cause:
The path set in alias's components is reflected as it is when installing a component of shadcn-ui, so the import path of 「cn」 in the generated component does not work properly.
Solution:
Change the path set in alias's components to match the import path of cn in the generated component.
Additional information:
This issue occurs in shadcn-ui v1.0.0-beta.7.
following information to the issue:
OS macOS:14.2（23C64）
tailwindcss: "^3.4.1",
"vite": "^4.4.5"

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


K8SPXC-654 use admin port for liveness/readiness probs

 ❨?❩

start using admin port for liveness/readiness probs
to be able to connect to DB in case of 'Too many connections'
issue

Test name
Status

haproxy
passed

upgrade-haproxy
passed

big-data
failed

recreate
passed

scaling
passed

storage
passed

init-deploy
passed

self-healing
passed

scaling-proxysql
passed

restore-to-encrypted-cluster
passed

security-context
passed

upgrade-proxysql
passed

limits
passed

self-healing-advanced
passed

monitoring-2-0
passed

smart-update
passed

demand-backup
passed

affinity
passed

operator-self-healing
passed

one-pod
passed

auto-tuning
passed

proxysql-sidecar-res-limits
passed

pitr
passed

users
passed

upgrade-consistency
passed

demand-backup-encrypted-with-tls
passed

tls-issue-self
passed

tls-issue-cert-manager
passed

tls-issue-cert-manager-ref
passed

validation-hook
passed

scheduled-backup
failed

in what case cluster check password still used?

That is why it is 'Draft' :) After these changes we can remove clustercheck  user at all.

Test name
Status

scaling
passed

scaling-proxysql
passed

security-context
passed

big-data
failed

storage
passed

self-healing
passed

recreate
passed

self-healing-advanced
passed

restore-to-encrypted-cluster
passed

operator-self-healing
passed

demand-backup
passed

pitr
passed

demand-backup-encrypted-with-tls
passed

haproxy
passed

upgrade-haproxy
passed

scheduled-backup
passed

init-deploy
passed

limits
passed

monitoring-2-0
passed

affinity
passed

one-pod
passed

auto-tuning
passed

proxysql-sidecar-res-limits
passed

users
passed

tls-issue-self
passed

tls-issue-cert-manager
passed

tls-issue-cert-manager-ref
passed

validation-hook
passed

upgrade-proxysql
passed

smart-update
passed

upgrade-consistency
passed

Test name
Status

upgrade-haproxy
passed

scaling
passed

storage
passed

haproxy
passed

big-data
passed

recreate
passed

self-healing
passed

init-deploy
passed

scaling-proxysql
passed

restore-to-encrypted-cluster
passed

security-context
passed

upgrade-proxysql
passed

self-healing-advanced
passed

limits
passed

demand-backup
passed

monitoring-2-0
passed

smart-update
passed

operator-self-healing
passed

affinity
passed

one-pod
passed

auto-tuning
passed

pitr
passed

proxysql-sidecar-res-limits
passed

demand-backup-encrypted-with-tls
passed

users
passed

upgrade-consistency
passed

tls-issue-self
passed

scheduled-backup
failed

tls-issue-cert-manager
passed

tls-issue-cert-manager-ref
passed

validation-hook
passed

Test name
Status

upgrade-haproxy
passed

storage
passed

big-data
passed

scaling
passed

recreate
passed

haproxy
passed

self-healing
passed

scaling-proxysql
passed

init-deploy
passed

upgrade-proxysql
passed

restore-to-encrypted-cluster
passed

security-context
passed

self-healing-advanced
passed

smart-update
passed

limits
passed

demand-backup
passed

operator-self-healing
passed

monitoring-2-0
passed

affinity
passed

one-pod
passed

auto-tuning
passed

proxysql-sidecar-res-limits
passed

upgrade-consistency
passed

pitr
passed

users
passed

demand-backup-encrypted-with-tls
failed

tls-issue-self
passed

tls-issue-cert-manager
failed

Test name
Status

PXC operator docker - perconalab/percona-xtradb-cluster-operator:PR-778-8e3da020

Test name
Status

upgrade-haproxy
passed

big-data
passed

scaling
passed

scaling-proxysql
passed

security-context
passed

recreate
passed

restore-to-encrypted-cluster
passed

demand-backup
passed

pitr
passed

demand-backup-encrypted-with-tls
passed

scheduled-backup
passed

storage
passed

self-healing
passed

self-healing-advanced
passed

operator-self-healing
passed

haproxy
passed

init-deploy
passed

limits
passed

monitoring-2-0
passed

affinity
passed

one-pod
passed

auto-tuning
passed

proxysql-sidecar-res-limits
passed

users
passed

tls-issue-self
passed

tls-issue-cert-manager
passed

tls-issue-cert-manager-ref
passed

validation-hook
passed

upgrade-proxysql
passed

smart-update
passed

upgrade-consistency
passed

@hors it does look like this change breaks the PXC with MySQL 5.7 since that mysql admin used to check the status of the cluster is a MySQL 8 thing.
In my case, the galera-pxc-0 Pod is "Running" but not ready since the readiness probe fails with the error "Can't connect to MySQL".
One way I see to solve this is to make that user, password and port configurable or detecting the target version and conditionally using one or the other.


not loading articles in global feed.

there is an successful request in network tab, but page says "Articles not yet loaded"

Is this supposed to be for halogen-realworld? This is just a form library, which doesn't sound relevant to your question.


Create index.html

The single html file riot tag compiler. The dependencies are the javascript file riot+compiler.js. It are using the riot.compile() function to compile riot tags.

Here is examples at plunker of the html riot online tag compiler:
Version with html and JavaScript
https://plnkr.co/edit/1ukKJeYxfcmeXf1VLY1f?p=preview

Version with html, JavaScript and Riot tags
https://plnkr.co/edit/mqQBuqoVNfitDqSxi1Ch?p=preview

thank you!

Thank you : ).

Here is a plunker version that are using the Ace editor and when you type on the left, it compiles automatically on the right for each change on the left. If you like this version, you can just use it.  ( it is w3c compliant with 0 errors and warnings).
Nice improvements on the index file btw : ).
Here is the link:
http://plnkr.co/edit/IkhpFo9tE3toAhG2vDkZ?p=preview


Icon list

I like to know what is the icon list you use in the app. I need to add a new section and I want maintain the same UI.

Hi @shuffledex, I think all the icons we use are custom, maybe you'll find this library useful.


Unique Migration Waves

The initial design has migration waves' name optional.
When waves don't have name, the user can distinct them from their start and end dates meanwhile to help avoid creating identical wave we cannot do that if the name is optional.
One solution is to have a composite unique key for each migration wave record using name, startDate and endDate fields, where name field is not NULL.

I don't think there is a need for this. I guess it's just a matter of having a logic that implements the following:

If the name is null, there can't be two waves with the same startDate and endDate.
If the name is not null, there can't be two waves with the same name, startDate and endDate.

So instead of a key, this would be handled by validation logic.

@rromannissen,  that makes sense.
My concern is to have the same business rules validation on both UI and API to not leave cracks open.
@jortel, @mansam, what do you think ?

@jortel, could the name be null as well ?


feat() Add Hadoop example config for google exporter

Added Hadoop example config for Google Cloud Monitoring.
Checklist

[x] Changes are tested
[ ] CI has passed

Seems like we're actually keeping these configs with the current exporter that's released.


[Unexpected Behavior] Song Rate

What you're using
Rate and Sound Effect don't seem to be operating correctly.
Expected behavior
I expect one of the options in Sound Effect to proportionally change the pitch with the speed, and the other to try to keep the pitch the same. I'll call the former "LP" (because it is reminiscent of a vinyl record on the wrong RPM), and the latter "Stretch" (because FL Studio calls it that).
I propose for the third option, Off, to select between these two depending on the speed chosen. For smaller differences, LP sounds better; at the expense of different pitch, the waveform is unmodified, making the sound crisp. For extreme rates, Stretch sounds better; LP would make the song sound too muddy or tinny if the pitch were changed too. I propose using LP if 0.7 ≤ Rate ≤ 1.5 (since Pump It Up used to have LP and a rate option between 0.8 and 1.5, and I'd like LP to also be used if Rate is 0.75).
Also, Stepmania 5's time stretching function is grating to the ears, compared to, for example, YouTube's play speed options.

This is confirmed from our voice conversation.
The code is written so that pitch only works, or speed without pitch only works, but both sound like a wasp in a tin can.
I'll factor in a rewrite of this code to support both, though we will need to add a 'compatibility' layer for themes which do not support the new selections.
I was able to get the desired pitch scales working fine as you saw, but in game, it was ignored, due to it dropping one or the other back to 1.0 rate.
It's gonna take a while, but it is a bug, and does need fixing.
Thanks for the report!

Tested Alpha 4.8.5.
How “Speed” and “Pitch” behave: Both behave as Stretch. Preview does not update upon changing the options.
How “Off” behaves: Stretch.
Sound quality of Stretch compared to 5.1: This may just be placebo, but I feel like the PaulStretch is better in 5.3 now.


Current page highlight in sidebar menu works only on localhost

The automatic menu highlighting / expanding in the sidebar seems to work only on localhost.
When I run the site on localhost, selecting a page makes its selection in the side menu be highlighted and visible, and it is "sticky" even if I refresh the page. However, after I publish, this does not work, and the menu remains collapsed.
Here are two screenshots showing localhost and the published version. Both were tested on latest Edge and latest MacOS.
https://help.esolia.pro/en/security/deep-packet-inspection/

After inspecting your site, looks like the issue is for different cases.
The url in the menu is https://help.esolia.pro/en/Security/deep-packet-inspection/ (Note the upper case S in Security) but the real url is https://help.esolia.pro/en/security/deep-packet-inspection/.
I need more investigation but generally it's good idea to have all urls in lowercase for cross compatibility

oh, wow, thanks. I'll fix them all to lowercase. I think this was when I was trying to make the sidebar menu look "normal" from a language perspective by capitalizing the words...


DSND-1682: Remove error consumer from exemptions delta consumer

All error consumer related classes removed.
All references to the error consumer removed from application.yml config
Tested locally. Seems to consume and route messages to error and retry fine still.

DSND-1682

Looks good, just a few unused imports

addressed in most recent commit


Birdsong without Nodejs dependency

I really love this plugin, It worked well on my development machine, but when I deployed it on my cpanel shared hosting it failed because their Python apps don't have access to Nodejs.
Is it possible to use some other implementation of mjml like this pure python one? Or maybe use a binary executable of the mjml package (I don't know it exists, I can't find one) instead of using Nodejs.
Thanks.

Have you looked into running an MJML http server? You could set it up on a different server (or something like aws lambda or fly.io) and call out to it from your wagtail server
There's a bit about it in the django-mjml readme - https://github.com/liminspace/django-mjml#httpserver-mode
Adding a pure python mjml backend might be better addressed in django-mjml, looks like there's an issue about custom backends in that repo - https://github.com/liminspace/django-mjml/issues/122

@seb-b Thanks, for now, I am implementing a simple newsletter using Django forms and stuff, while researching the links you provided.
It would be nice to have a system where you design your email in an external software, like Figma or MJML designer and upload the email files as zip and use that.

Example of how to use your own mjml http server can also be found in Birdsong's own tests:

settings.py
docker-compose.yml

Birdsong uses django-mjml and django-mjml has a GitHub Issue in that they discuss about using mjml-python which has no Nodejs dependency.
https://github.com/liminspace/django-mjml/issues/184
Thus, one option is contribute that issue or just wait until it is done.


Fixes for API 43.0 breaking changes

Coverage increased (+0.05%) to 71.157% when pulling bd155a19a134bdac723829561c084feeea72af6d on eatonphil:breaking-changes into 6e06458ab989d683cff0663a03eba451ff322c01 on linode:develop.

Coverage increased (+0.07%) to 71.244% when pulling 13e41998e888d3a18d17c524107b836c45401465 on eatonphil:breaking-changes into 2c8d00d4063db45554868c9308eb670805105850 on linode:develop.

Coverage increased (+0.03%) to 71.205% when pulling b818cdc86e6308dc75d5b8048c0fd25748cd3066 on eatonphil:breaking-changes into 2c8d00d4063db45554868c9308eb670805105850 on linode:develop.

Coverage increased (+0.03%) to 71.205% when pulling b818cdc86e6308dc75d5b8048c0fd25748cd3066 on eatonphil:breaking-changes into 2c8d00d4063db45554868c9308eb670805105850 on linode:develop.


chore: clean up span naming

Clean up span naming to OpenTelemetry specifications (existing names are discouraged as having high cardinality)
Centralize logic for what event types to allow, and allow dblclick as well as click
(Bad form, but blergh): forgot to unpatch removeEventListener from yesterday's work.

@johnbley Is there a plan to do the same for plugin-document-load and plugin-xml-http-request (in both cases URLs are used as span operation names)?

@johnbley Is there a plan to do the same for plugin-document-load and plugin-xml-http-request (in both cases URLs are used as span operation names)?

Yes, working my way through a number of smaller issues like this.  Doing one area at a time to not overwhelm the maintainers. 😄

@obecny particularly looking for your feedback on this one

We can either add such possibility to a plugin config. This can be a 2 callbacks
Sorry, not understanding this.  You would like _updateInteractionName to be protected as well?

By default I would support click event only
Will do.

We can either add such possibility to a plugin config. This can be a 2 callbacks

Sorry, not understanding this. You would like _updateInteractionName to be protected as well?

By default I would support click event only

Will do.

I meant when you instantiate plugin
const providerWithZone = new WebTracerProvider({
  plugins: [
    new UserInteractionPlugin({
      allowEventType(eventType: string) {
        if (eventType === ....) { // write your own logic
          return true;
        }
      },
      updateSpanName(target: HTMLElement) {
       // write your own logic
       return 'spanName';
      }
    }),
  ]
});

and 2nd possibility
class MyUserInteractionPlugin extends UserInteractionPlugin {
  allowEventType(eventType: string) {
    if (eventType === ....) { // write your own logic
      return true;
    }
  },
  updateSpanName(target: HTMLElement) {
   // write your own logic
   return 'spanName';
  }
}

class MyUserInteractionPlugin extends UserInteractionPlugin {

This (strongly typed) is my preference - easier for IDEs, static tools, etc.  to analyze/understand the specialization.  That's what the current patch uses.


UnboundLocalError from variable 'position'

Expected Behavior
position should be set before calling array
Actual Behavior
Arrived at Pokestop
Now at Pokestop: Riksdagslejontv├Ñ - Spinning...

Loot:
50 xp
1x Pokeball
1x Pokeball
1x Razz Berry
PokeStop on cooldown. Time left:<PHONE_NUMBER>41 seconds.
Found fort 30e2a68d345441f2abfa141a964deabd.16 at distance 4.87814204604m
Traceback (most recent call last):
File "D:\Pokemon stuff\PokemonGo-Bot\pokecli.py", line 120, in
main()
File "D:\Pokemon stuff\PokemonGo-Bot\pokecli.py", line 117, in main
bot.take_step()
File "D:\Pokemon stuff\PokemonGo-Bot\bot.py", line 30, in take_step
self.stepper.take_step()
File "D:\Pokemon stuff\PokemonGo-Bot\stepper.py", line 49, in take_step
self.bot.work_on_cell(cell,position)
File "D:\Pokemon stuff\PokemonGo-Bot\bot.py", line 59, in work_on_cell
hack_chain = worker.work()
File "D:\Pokemon stuff\PokemonGo-Bot\cell_workers\seen_fort_worker.py", line 36, in work
self.api.fort_details(fort_id=self.fort['id'], latitude=position[0], longitude=position[1])
UnboundLocalError: local variable 'position' referenced before assignment

Steps to Reproduce
Always after two pokestops using location starting location 59.328569,18.064374 this error occure
Other Information
OS: Win 10 pro x64
Git Commit: ebb8dba06e94fb0addb15b634124edc2079ec87a

Also exists on dev branch, c80ab0d70475f2faa8407d1318152f9cd8f73f2f

Fixed in dev already. Thanks!

Problem persists in dev:
[#] Found fort 30e2a68d345441f2abfa141a964deabd.16 at distance 4.87814204604m
Traceback (most recent call last):
File "D:\Pokemon stuff\PokemonGo-Bot\pokecli.py", line 107, in
main()
File "D:\Pokemon stuff\PokemonGo-Bot\pokecli.py", line 104, in main
bot.take_step()
File "D:\Pokemon stuff\PokemonGo-Bot\pokemongo_bot_init_.py", line 31, in take_step
self.stepper.take_step()
File "D:\Pokemon stuff\PokemonGo-Bot\pokemongo_bot\stepper.py", line 58, in take_step
self._work_at_position(position[0], position[1], position[2], True)
File "D:\Pokemon stuff\PokemonGo-Bot\pokemongo_bot\stepper.py", line 103, in work_at_position
self.bot.work_on_cell(cell, position, pokemon_only)
File "D:\Pokemon stuff\PokemonGo-Bot\pokemongo_bot_init.py", line 54, in work_on_cell
hack_chain = worker.work()
File "D:\Pokemon stuff\PokemonGo-Bot\pokemongo_bot\cell_workers\seen_fort_worker.py", line 43, in work
self.api.fort_details(fort_id=self.fort['id'], latitude=position[0], longitude=position[1])
UnboundLocalError: local variable 'position' referenced before assignment
git commit: 6199d496d37b40a5ec9bfda0d4fb69c93b44a6cd


feat: Implement overriding API methods (prefer #47)

This is a successor to https://github.com/allan2/dotenvy/pull/17
Update: To address and resolve the discussion points below, I am offering another, preferred PR at https://github.com/allan2/dotenvy/pull/47.
This PR

Adds overriding variants to all relevant public API methods.
Fixes documentation.
Adds missing test for from_read().
Tests non-overriding behavior with greater precision.
Tests overriding behavior correctly.
Fixes some inconsistent test formatting.

Discussion
Update: To address and resolve these points, I am offering another, preferred PR at https://github.com/allan2/dotenvy/pull/47.
Note that the API is still inconsistent, due to #41:
overload() vs. the more consistent dotenv_overload()
Following the naming schema the public fn overload() should be called fn dotenv_overload() -- in line with the crate-wide [base]_[variant]() pub fn-naming scheme, e.g. dotenv() & dotenv_iter().
See also the sorting of public methods in the documentation, where the broken naming scheme in overload() makes it be placed out of context:

Proper naming: Prefer "override" over "overload"
Quoted from https://github.com/allan2/dotenvy/pull/17#issuecomment-1339589211:

Remove your dotenv_override function.
Change "override" to "overload" in your function naming.

I disagree. "override" describes the functionality much better than "overload".
I would strongly suggest going forward with "override" rather than "overload".
There is no "overloading" happening as is understood in IT. Instead, existing values are overridden by new values.
Why mention loading in overload() when the dotenv() function is not named load()?
The following naming makes the most sense:
pub fn dotenv()
pub fn dotenv_iter()
pub fn dotenv_override()

pub fn from_filename()
pub fn from_filename_iter()
pub fn from_filename_override()

pub fn from_path()
pub fn from_path_iter()
pub fn from_path_override()

pub fn from_read()
pub fn from_read_iter()
pub fn from_read_override()

iter::load()
iter::load_override()

Optionally, _with_override() rather than _override().

Good naming is of course difficult.

Indeed, and thank you for taking the time to consider it.

Why mention loading in overload() when the dotenv() function is not named load()?

As per #39 it looks like we are moving towards renaming the function load. In which case load and overload are quite natural from an aesthetic standpoint.
You are right that load_override is more explicit, but overload is consistant with other dotenv libraries. E.g. bkeepers/dotenv.


Caliscope calibration support in Pose2Sim

Not an issue, just keeping you informed that Caliscope calibration files are now natively supported in Pose2Sim and in the Pose2Sim_Blender addon :)
https://github.com/davidpagnon/Pose2Sim_Blender/assets/54667644/a2cfb75d-a2d4-471a-b6f8-8f1ee999a619

WOW! That is awesome, @davidpagnon!
Your blender add-on is exactly the kind of thing that I'd thought would provide an excellent way to examine the output of a motion trial (projecting the image onto the camera and being able to snap the view to the camera). But you actually IMPLEMENTED it. Fantastic to see that in action.
Thank you for keeping me in the loop. I'm about to go into a black hole as I prepare for my dissertation proposal next month, but I hope we can keep in touch on these projects for more bandwidth becomes available.

You're welcome, thanks for the appreciation! On the other hand, with rigmarole you rigged the whole thing, and I haven't found time to look into it so that could be a good source of inspiration.
I really need to stop working on the nitty-gritty of everything as well, because I've got other urgent projects; so there is no rush at all. And good luck (or "bon courage" as we say in French) for your dissertation! 💪✍
And it would be a pleasure to work together in the future if we have the occasion :)


Merge pull request #483 from codinasion/master

Why:
Closes #512
What's being changed: Create file
Check off the following:

[x] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

class Solution:
def init(self):
self.diagonal1 = 0
self.diagonal2 = 0
def get_input(self):
    self.diagonal1 = float(input("Diagonal 1: "))
    self.diagonal2 = float(input("Diagonal 2: "))

def calculate_rhombus_area(self):
    ans = (self.diagonal1 * self.diagonal2) / 2
    return ans

sol = Solution()
while 1:
sol.get_input()
print(sol.calculate_rhombus_area())


Alex's Caves Compatability

The huts to find scroll works fine, but structures like the volcano does not spawn with OG Stratospherical

Ok


[rtorrent] Adding a torrent can sometimes take a while, so verification fails

Motivation for changes:
Loading torrents into rtorrent can take a while for me, so when adding multiple torrents it cannot verify the subsequent additions, but it's never happened for me that it wasn't added.
Detailed changes:

Changed entry.fail to log.warning

If there are no comments I will merge this later today


Add Channel.extend

@thejameskyle, after speaking about this offline, do you think we're okay to close this w.o. merge?

yup


Type inheritance in 0.4.3

Is Type inheritance supported in 0.4.3?
For example, I have these two classes:
class NbnOrderFeasibilityRequest(types.Type):
    sqId = validators.String()
    nbnLocationId = validators.String()
    tc4DownloadSpeed = validators.Integer()
    tc4UploadSpeed = validators.Integer()
    cpiId = validators.String(default=None)
    ntdId = validators.String(default=None)
    potsInterconnectId = validators.String(default=None)
    transferType = NbnOrderTransferType(default=None)
    localNumberPorting = validators.String(enum=['Yes', 'No'], default=None)
    customerAuthorityDate = validators.Date(default=None),
    appointmentId = validators.String(default=None)
    installationWorkforce = NbnOrderInstallationWorkforceType(default=None)  # TODO conditions
    installationCentralSplitter = validators.String(enum=['Yes', 'No'], default=None)
    exchangePairConnection = validators.String(enum=['No'], default=None)
    ntdPortId = validators.Integer(default=None)
    batteryBackup = validators.String(enum=['Yes', 'No'], default=None)

class NbnOrderRequest(NbnOrderFeasibilityRequest):
    accessSeekerRef = AccessSeekerRef(default=None)

When I instantiate an NbnOrderRequest, only accessSeekerRef seems to be available.
data = {
    'sqId': 'SQ00001',
    'nbnLocationId': 'LOC00001',
    'tc4DownloadSpeed': 10,
    'tc4UploadSpeed': 10
}
request = NbnOrderRequest(data)
assert {'accessSeekerRef': None) == dict(data) # Works, and it shouldn't

I'm guessing there's something that needs to happen in the new operation of TypeMetaclass to get fields/attributes from the parent class, but I just wanted to check if this is supposed to be possible.

It’s not supported yet, would need a little work on the metaclass. Yes I’d like to see it added. If anyone wants to take it on it’d be worth looking at the metaclass implementation of Django REST framework ModelSerializer

Also worth noting. In JSON schema we’ll want to map these to allOf cases, so we can included the child and parent definitions. (Tho only works in the case of extending fields, not overriding them)

I'm having a crack at this now.
I've duplicated most tests in test_types.py to make sure the expected operations work with an extended Type.
The bit of code added was very similar to the SerializerMetaclass, except that we're plucking the properties out of the validator on the base class as follows:
    for base in reversed(bases):
        if hasattr(base, 'validator'):
            properties = [
                (key, base.validator.properties[key]) for key
                in base.validator.properties
                if key not in attrs
            ] + properties

The last, and probably, trickiest bit is how this should be presented in the JSON Schema. It's tricky for me because I'm not all that familiar with the JSON Schema, though I'd like to write the test first, and then modify the code to meet the requirement.
Given the existing Product type used in the testing, I've extended that type to create a ReviewedProduct, which added a reviewer property as follows:
class ReviewedProduct(Product):
    reviewer = validators.String(max_length=20)

My current test for an extended type passes, though the generated structure is flat:
def test_extended_as_jsonschema():
    struct = encode_jsonschema(ReviewedProduct, to_data_structure=True)
    assert struct == {
        "$ref": "#/definitions/ReviewedProduct",
        "definitions": {
            "ReviewedProduct": {
                "type": "object",
                "properties": {
                    "name": {
                        "type": "string",
                        "maxLength": 10
                    },
                    "rating": {
                        "type": "integer",
                        "minimum": 0,
                        "maximum": 100,
                        "default": None,
                        "nullable": True
                    },
                    "created": {
                        "type": "string",
                        "format": "datetime"
                    },
                    "reviewer": {
                        "type": "string",
                        "maxLength": 20
                    }
                },
                "required": [
                    "name",
                    "created",
                    "reviewer"
                ]
            }
        }
    }

At a guess, it should be popping out something like the following:
def test_extended_as_jsonschema():
    struct = encode_jsonschema(ReviewedProduct, to_data_structure=True)
    assert struct == {
        "$ref": "#/definitions/ReviewedProduct",
        "definitions": {
            "Product": {
                "type": "object",
                "properties": {
                    "name": {
                        "type": "string",
                        "maxLength": 10
                    },
                    "rating": {
                        "type": "integer",
                        "minimum": 0,
                        "maximum": 100,
                        "default": None,
                        "nullable": True
                    },
                    "created": {
                        "type": "string",
                        "format": "datetime"
                    }
                },
                "required": [
                    "name",
                    "created"
                ]
            },
            "ReviewedProduct": {
                "allOf": [
                    {
                        "$ref": "#/definitions/Product"
                    },
                    {
                        "properties": {
                            "reviewer": {
                                "type": "string",
                                "maxLength": 20
                            }
                        },
                        "required": [
                            "reviewer"
                        ]
                    }
                ]
            }
        }
    }

I've been looking at the examples from https://spacetelescope.github.io/understanding-json-schema/structuring.html and https://spacetelescope.github.io/understanding-json-schema/reference/combining.html#combining.
I'm also keeping a suspicious eye on Object(Validator).additional_properties. Apparently, if the JSON Schema equivalent is False, then you can't extend that type, and by default, the Object validator used on a Type will set this to None. Though I haven't see the class property pass through to the JSON Schema yet.
If anyone has a bit more experience with JSON Schema than me and could tell me if the presented test is on track or not, that would be great.

I've just been having a look at the allOf tests in allOf.json, particularly the "allOf with base schema" entry and the following might also fit the bill.
def test_extended_as_jsonschema():
    struct = encode_jsonschema(ReviewedProduct, to_data_structure=True)
    assert struct == {
        "$ref": "#/definitions/ReviewedProduct",
        "definitions": {
            "Product": {
                "type": "object",
                "properties": {
                    "name": {
                        "type": "string",
                        "maxLength": 10
                    },
                    "rating": {
                        "type": "integer",
                        "minimum": 0,
                        "maximum": 100,
                        "default": None,
                        "nullable": True
                    },
                    "created": {
                        "type": "string",
                        "format": "datetime"
                    }
                },
                "required": [
                    "name",
                    "created"
                ]
            },
            "ReviewedProduct": {
                "properties": {
                    "reviewer": {
                        "type": "string",
                        "maxLength": 20
                    }
                },
                "required": [
                    "reviewer"
                ],
                "allOf": [
                    {
                        "$ref": "#/definitions/Product"
                    }
                ]
            }
        }
    }

Looking at the code a bit more, it would appear that there's no catering for any of the combining types (allOf, anyOf, oneOf).
Introducing allOf is probably a bit beyond my expertise at this point, although I'm hoping it would be something along the lines of adding a defintion to JSON_SCHEMA to cater for the allOf structure, and then modifying JSONSchemaCodec.encode_to_data_structure to recognise that the item is not a direct subclass of types.Type, and to add a definition of the base type to defs[], determining the new properties introduced by item, and add an allOf entry to value that references the parent. (* and breathe *)
At this point, I'm going to do a pull request that will observe type inheritance in the classes, but JSON Schema will present the extended class as a flat class, but no knowledge of the parent type.
When allOf is introduced into apistar, then the combination relationship can also be implemented for extended Types.

At this point, I'm going to do a pull request that will observe type inheritance in the classes, but JSON Schema will present the extended class as a flat class, but no knowledge of the parent type.

Yup. Approaching this incrementally makes sense.

Closed via #444. Thanks!


Link to instructions for Cloudant setup

Describe the bug
Deploying the Inventory Services app from the pre-built solution (https://ibm-gsi-ecosystem.github.io/ibm-gsi-cloudnative-journey/developer-intermediate/inventory-app) fails because it relies on Cloudant credentials. This is actually mentioned in the Advanced section (https://ibm-gsi-ecosystem.github.io/ibm-gsi-cloudnative-journey/developer-advanced/inventory-cloudant).
Expected behavior
The instructions to deploy the pre-built solution for the Inventory app, https://ibm-gsi-ecosystem.github.io/ibm-gsi-cloudnative-journey/developer-intermediate/inventory-app/, should link to how to setup Cloudant credentials.

Hi Jean,
We have seperated the cloudant code from Deploying the Inventory Services App from the pre-built solution(https://ibm-gsi-ecosystem.github.io/ibm-gsi-cloudnative-journey/developer-intermediate/inventory-app/#deploy-the-inventory-app-solution). The code (https://github.com/ibm-gsi-ecosystem/inventory-management-svc-solution/tree/master/src/main/java/com/ibm/inventory_management) can be run without cloudant credentials now.
Thanks & Regards,
Ramya

Pre-built solution deploys successfully.


PowerToys Run show different results after update

I have some issues in the newest version of PowerToys
I updated PowerToys from 0.18 (I don't remember last digit but it was 0.18.x) to 0.19
Environment
Windows build number: Microsoft Windows [Version 10.0.19041.388]
PowerToys version: 0.19
PowerToy module for which you are reporting the bug (if applicable): PowerToys Run

App that I want to launch I have installed on my PC
PHPStorm is possible to run from Windows
Steps to reproduce
I'm typing "Paint" or "PHPStorm" to launch app.
Expected behavior
In previous veriosn for "Paint" on first row I had Paint app (not 3D)
For PHPStorm on first row I had PHPStorm
Actual behavior
For Paint I have now Paint3D on first row. I had to move down in list to find Paint
For PHPStorm I cannot find even link for app. It lost from list
Screenshots

Here I can launch PHPSTorm from Start menu:

with 0.19, we had to delete the cache, 0.20 should drastically improve this which will be out this week.

I'v updated PowerToys to 0.20 and Run now works much better
Thanks! 👍
I found one more issue now
But now with notepad
I used to work with launchy.
There, when I wrote "not" and few times selected notepad in further times it showed at first row
In PowerToys Run when I write "not" there is no Notepad
I have to write whole word "Notepad"

I don't know why, but now it works. When I typing "not" notepad is first in results.
So I think all for me is solved ;)


New package: FinanceTools v0.1.0

Registering package: FinanceTools
Repository: https://github.com/joshniemela/FinanceTools.jl
Created by: @joshniemela
Version: v0.1.0
Commit: c7a0c4412ba6f6591f29a2894012340a8f2e24e8
Reviewed by: @joshniemela
Reference: https://github.com/joshniemela/FinanceTools.jl/commit/c7a0c4412ba6f6591f29a2894012340a8f2e24e8#commitcomment-74690587
Description: Various tools to process financial time series

@JuliaRegistrator register

Error while trying to register: Register Failed
@joshniemela, it looks like you are not a publicly listed member/owner in the parent organization (JuliaRegistries).
If you are a member/owner, you will need to change your membership to public. See GitHub Help

Presumably superseded by#61191


fofa errmsg

  __  ______  _________ _   _____  _____
 / / / / __ \/ ___/ __ \ | / / _ \/ ___/
/ /_/ / / / / /__/ /_/ / |/ /  __/ /
\__,_/_/ /_/\___/\____/|___/\___/_/

                projectdiscovery.io

[INF] Current uncover version v1.0.3 (latest)
[fofa] [820031] F点余额不足

@ShubhamRasal, @tarunKoyalwar The task is in draft because first we need to merge the uncover refactoring https://github.com/projectdiscovery/uncover/pull/182


Extension Manager no longer sorts recommended extensions first

The Extension Manager now sorts the extensions alphabetically, not placing the recommended extensions first.  This looks like one of our customizations got lost in the previous vscode source refresh.

This isn't actually an issue.  The confusion was in the dev build the recommended extensions are preinstalled and don't show the star indicator in Extension Manager.


Update to use Azure SDK modules

Update the Azure SDK modules as Autorest is legacy, see https://learn.microsoft.com/en-us/azure/developer/go/azure-sdk-authorization, for more information.
The result is that the custom credentials functions is no longer needed, simplifying the code.
A downside is that this custom prefixes are no easily supported, but Microsoft have standardized everything be prefixed with AZURE_, and the possibility to changed the prefix was not documented, so it could be considered okay to remove this functionality, please let me know what you think.
Also fixes #35

@kichristensen was this merged elsewhere ?

@Doommius I honestly cannot remember why I closed this one. But it is not merged somewhere else as far as I can see

I noticed that the doc mentions it being able to handle az cli Auth. I assume that means it's possible to pass managed identity via azure resources as well? At least that's the use case I'm looking at to remove credentials that can be exposed/leaked.

@Doommius If you can get the managed identity token to be resolved inside the container, it should be possible, at least according to the code, but haven't tested it.
It will most likely involve running the container with host network enabled


gh-258: Added logging config for debugging to FAQ

Per the conversation in Gitter today, a PR with a FAQ update.  I don't know how to test the generated documentation outside of just looking at the adoc rendering in Intellij, so just a heads up incase I've done something that formats terribly ;)

Codecov Report

Merging #259 into 1.0.x will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              1.0.x     #259   +/-   ##
=========================================
  Coverage     59.44%   59.44%
  Complexity     1371     1371
=========================================
  Files           201      201
  Lines          5686     5686
  Branches        715      715
=========================================
  Hits           3380     3380
  Misses         1911     1911
  Partials        395      395

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 c1e285e...bc5d8ad. Read the comment docs.


Add SMPModel in /templates/meshmodel

Description
This PR related to https://github.com/meshery/meshery-nighthawk/issues/27#issuecomment-1956627115
Notes for Reviewers

Generated SMP spec json file manually from https://github.com/service-mesh-performance/service-mesh-performance.git, and copied to /templates/meshmodel.
Created the manifests/charts/crds.yaml to define performance profile in the future.

Signed commits

[ ] Yes, I signed my commits.

@MUzairS15 @leecalcote  This is the initial PR based on my understanding, I don't know if it is correct, please forgive me I haven't figured out the Meshery Model very well, please correct me if I'm wrong. Thanks!

I'm very excited about this, @gyohuangxin. 😀

We're in need of having RegisterMeshModelComponents changed to RegisterMeshModelComponents inside of the Meshery Adapter Library. Let's do what we can to get away from MeshModel over to Model.

I will update the components so that we can move forward.

@MUzairS15 Thanks for your help!

Oh yes, I missed that.


Fixes an issue when running on python 3

In python3 the map function returns a generator rather than a list.
This generator has no len() function, causing an error. By
explicitly casting it into a list we use a bit more ram, but it
then has a known length
Fixes #61

Thank you :)


refactor(linter): rename eslint file

Rename eslint file extension to json

Codecov Report

Merging #30 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           develop     #30   +/-   ##
=======================================
  Coverage     8.33%   8.33%
=======================================
  Files            3       3
  Lines           24      24
  Branches         9       9
=======================================
  Hits             2       2
  Misses          14      14
  Partials         8       8

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 5815703...8a567d6. Read the comment docs.


AAF adapter

Added an adapter for reading Advanced Authoring Format (AAF) compositions into OTIO.
This contrib adapter requires PyAAF 1.0.0 to work: https://github.com/markreidvfx/pyaaf/releases

Addresses #1


refactor(flux): convert the allocator into an interface

This follows the changes from influxdata/flux#4539.

I wonder why fluxtest is failing here? Doesn't seem like your changes would have affected it, or at least not in the way it seems to be failing.


Restyled by prettier-markdown

Browser metadata
Path:      /
Browser:   Chrome <IP_ADDRESS> on Windows 10
Viewport:  2374 x 1149 @0.75x
Language:  en-US
Cookies:   Enabled

Open in BrowserStack

Open Deploy Preview · Mark as Resolved


Clean trailing spaces and lower case strings in Updog Harmonization

Trailing spaces and arbitrary capitalization carry no information on certain input values and can often prevent data from being inline causing extra cleaning work.
ACCEPTANCE CRITERIA

[ ] values that need cleaning are agreed upon
[ ] Trimming is implemented
[ ] lower casing is implemented

Won't do, UPDOG is retired.


zend-view-(extra-)helpers performance overhead

I just wanted to report that upgrading components in a small zend-expressive testing blog-like app caused a penalty in performance. I tried do downgrade and/or disable extra components (zend-i18n, zend-form) and I came to the conclusion that loading the view helpers caused a standard 10 blog post listing to more than double the processing time from 13~20 ms to 36~45 ms. In both cases disabling all the (extra) view helpers the app runs in about 6~12 ms. This in php-fpm 7.0.x. In php-fpm 5.5.x the timing is of course doubled, so i get about 70~80 ms which is comparable to a full zf(2.5) processing time.
The only way i could mitigate the problem  was to add to the view_helpers config section only the plugins and aliases I needed instead of the whole zend-i18/zend-form configuration.
btw, I tested the plugin helpers using both v2 and v3 of plugin manager construct signature.
(the small test app uses a nzz setup: nikic-fastroute+zend-servicemanager+zend-view)
upgraded packages
zend-expressive-viewrenderer 1.0.1 => 1.1.0
zend-form 2.5.3 => 2.7.0
zend-stdlib 2.7.4 => 3.0.1
zend-servicemanager 2.7.4 => 3.0.1
zend-eventmanager 2.6.2 => 3.0.1
zend-form 2.6.0 => 2.8.3
zend-i18n 2.5.? => 2.7.2
... and other packages but they don'r seem related to the change in performance
kind regards.
maks

fixed by:
https://github.com/zendframework/zend-form/pull/77
https://github.com/zendframework/zend-i18n/pull/47


Anonymous usage stat collection

It would be nice to compile how people are using TW.  What options are set, how many tabs closed, etc.

Going to close this as I don't plan to address it. Usage data might be useful for a project with more active development, but the tradeoff of asking people to volunteer data when I don't think the project can make good use of it is difficult.
I'm also a privacy and security stickler, and going from collecting no data to collecting even a little adds a lot of responsibility and vulnerability to this extension. I'm happy maintaining that this extension collects zero information from its users (aside from the install, uninstall, and active user counts that the Chrome Store collects for all extensions).


Enabling Disucssions Feature for the Flutter repository in GitHub

I suggest trying the discussion new feature in GitHub so people can have conversation without opening issues. It might look like Stack Overflow but in the same repo but I think we should give it a try. Because then engineers could take a look and answer some questions there or discuss other stuff.
It will make personal issues disappear, because people usually look for experts in the field to help them that is why they come to here and open an issue but if we could enable discussion in the repo which is a new feature in GitHub it will really lift some load off the issues tab 🚀
Here is an example in my repo:

Cc @Hixie to react on this proposal.

@pedromassangocode duplicate of https://github.com/flutter/flutter/issues/60696

Before taking action let us wait for the reply of Hixie

This is indeed a duplicate of #60696.

(commented in #60696, please don't hesitate to follow up there)

Sure 🚀


Top Bar user menu

Dear friend,
First, thanks for the great update, with the new features. Great package and great implementation.
Sorry this is silly, but I would like to ask about the user menu above. How can I set the lablel/icon for the anchor link? it appears empty, but when you hover with the mouse it is clickable, and it shows the menu. but the span inside has empty string.
Is there a way to set this from the config?

thanks,
Ahmed

Could you share the part for the user menu of the configuration file ?
Also What type of layout are you using?

The usermenu label should be your username, it’s not definable by config.

Could you share the part for the user menu of the configuration file ?
Also What type of layout are you using?

Thanks for replying, these are my configs:
`
'usermenu_enabled' => true,
'usermenu_header' => false,
'usermenu_header_class' => 'bg-primary',
'usermenu_image' => false,
'usermenu_desc' => false,
'layout_topnav' => null,
'layout_boxed' => null,
'layout_fixed_sidebar' => null,
'layout_fixed_navbar' => null,
'layout_fixed_footer' => null,

`
Menu:
`
'menu' => [
    ['header' => 'Main Menu'],
    [
        'text'        => ' Restaurants',
        'url'         => '#',
        'icon'        => 'fas fa-utensils',
        'label_color' => 'success',
        'roles' => [
            'admin',
        ],
        'submenu' => [
            [
                'text' => 'Add Restaurant',
                'route' => 'restaurants.create',
                'icon'  => '',
            ],
            [
                'text' => 'View Restaurants',
                'route' => 'restaurants.index',
                'icon'  => '',
            ],
        ],
    ],
    [
        'text'        => ' Bookings',
        'url'         => 'admin/pages',
        'icon'        => 'far fa-calendar-alt',
        'label_color' => 'success',
        'submenu' => [
            [
                'text' => 'Add Booking',
                'route' => 'tablebookings.create',
                'icon'  => '',
            ],
            [
                'text' => 'View Bookings',
                'route' => 'tablebookings.index',
                'icon'  => '',
            ],
        ],
    ],
    [
        'text'        => ' Users',
        'url'         => 'admin/pages',
        'icon'        => 'fas fa-users',
        'label_color' => 'success',
        'submenu' => [
            [
                'text' => 'Add User',
                'route' => 'users.create',
                'icon'  => '',
            ],
            [
                'text' => 'View Users',
                'route' => 'users.index',
                'icon'  => '',
            ],
        ],
    ],
    [
        'text' => 'My Profile',
        'route' => 'users.profile',
        'icon' => 'fas fa-fw fa-user',
    ],

],

`

The usermenu label should be your username, it’s not definable by config.

This makes sense, but I am using the default Auth for Laravel, with no username. How can I customize what is displayed?

Which Laravel Version do you use? V6 oder V7? Laravel's User Class comes with the "name" field and Laravel-AdminLTE use this field.

Hey, I am using Laravel 6.

Which Laravel Version do you use? V6 oder V7? Laravel's User Class comes with the "name" field and Laravel-AdminLTE use this field.

Ok. thanks for the hint, I have updated the package with my custom method, as I don't have the 'name' field, and reinstalled the views again :
 public function getFullName(){

        return $this->first_name." ".$this->last_name;
 }

Thanks for the hint, if there is a better way to do it please let me know.

No, this seems like a good way, I don't see any problems here.

Thanks for your reply, case is solved.


Office.ribbon.requestUpdate is throwing exception : The API you are trying to use is not available. It may be available in a different scenario.

URL of sample
https://github.com/OfficeDev/PnP-OfficeAddins/tree/main/Samples/office-contextual-tabs
Describe the bug
Followed instructions mentioned in https://github.com/OfficeDev/PnP-OfficeAddins/tree/main/Samples/office-contextual-tabs
Loaded the manifest, clicked on "Import Data" and the table showed up. Clicked on the table and the context tab did not show up. Instead there was an error in console saying
"Uncaught (in promise) RichApi.Error: The API you are trying to use is not available. It may be available in a different scenario.
at new n (excel-web-16.00.js:formatted:12049)
at r.i.processRequestExecutorResponseMessage (excel-web-16.00.js:formatted:14829)
at excel-web-16.00.js:formatted:14754"
This was working earlier and seems to be broken now.
To Reproduce
Steps to reproduce the behavior:
As mentioned above
Expected behavior
Context tab should show up.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment

Platform [PC desktop, Mac, iOS, Office Online]: Office Online
Host [Excel, Word, PowerPoint, etc.]: Excel
Office version number: ______
Operating System: Windows 10
Browser (if using Office Online): Chrome

Additional context
This was working till 10th June 2021.

Hi @nvazhayil,
Thanks for reporting this. I could not reproduce this issue. It doesn't look like anything in the PNP changed in the last week. Let me assign this to @sohailzafar to investigate from the product side.

I can reproduce with both the manifest files.

I'm not able to reproduce this. The sample works as described for me.

It's working for me as well now. Did anything change? what could potentially throw the exception "The API you are trying to use is not available. It may be available in a different scenario."? I used to get this every time in the past few days.

I can only speculate that some Office online servers had not been updated with the version of Office that recognizes the API, and you were hitting one of them. Otherwise, @sohailzafar may know.
In the meantime, I'll close this issue.


login issue - "ERROR: Please login before running this command"

I believe I've successfully installed the Force CLI and can run the HELP command everything else just gives me ERROR: Please login before running this command. If I try Force Login I get the same error. I know that may not be much to go on but am hoping someone here can help.
Thank you!

Can you run force version and report back the version shown on the screen?

Hi @dcarroll , thanks for the reply. I have version v0.22.59.

One step closer. I updated and am now pushed to the login screen in my browser. After logging in I see the message in terminal Logged in as<EMAIL_ADDRESS>(API v37.0) I thought I was good to go at that point however any command thereafter gives me the same error message I saw previously.

Ok, that means the CLI cannot find the config file that is created that
saves account data.  if should be in the root of your file system and is
named .force.
If you can't access that folder then you will need to configure your perms
so that you can.
On Fri, Oct 28, 2016 at 5:25 AM, normcopeland<EMAIL_ADDRESS>wrote:

One step closer. I updated and am now pushed to the login screen in my
browser. After logging in I see the message in terminal Logged in as
<EMAIL_ADDRESS>(API v37.0) I thought I was good to go at that point
however any command thereafter gives me the same error message I saw
previously.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/heroku/force/issues/333#issuecomment-256907421, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAHGHn979hWwjft4SLBThUz8hMTcs1Erks5q4empgaJpZM4KhnD-
.

--
Thanks,
Dave

That did the trick! Thank you @dcarroll , I'm an aspiring Salesforce Dev and am excited to learn Force CLI!


[BUG] Repeated Calendar Entries not showing on Month View

I'm seeing the same behaviour as this closed issue.
Describe the bug
Events that repeat daily only appear on the first day of the calendar.
To Reproduce
Add some events that repeat daily.
Expected behavior
I expect to see the events listed on every day of the month.
In the screenshot below are both calendars are showing the same google calendar. The calendar only contains 5 events which repeat every day. However there is only an icon on the first day of the month. Clicking the first day (27 April) shows the day's events. Clicking any of the other days does nothing.
Screenshots

Setup Information (please complete the following information):
Plugin Version
1.4.2
Browser & Version
Chrome 81.0.4044.138
Opera 68.0.3618.104
Additional context
This also affects the List view, where you only see the repeating events for 1 day, no matter what maxDaysToShow or maxEventCount are set to.

@honkerst I think I am now seeing this and just want to check do you have multiple calendars?
Would it be possible to post a copy of your card config as this helps me set my tests up the same to make sure I see the same issues.

No problem, thanks for looking at it.
I am only using one calendar in both of these calendars.
My config looks like this:
` - type: horizontal-stack
cards:
- type: "custom:atomic-calendar-revive"
name: Calendar
enableModeChange: true
defaultMode: 2
dateFormat: L
maxDaysToShow: 30
sortByStartTime: true
showDescription: true
showNoEventsForToday: true
entities:
- entity: calendar.automator
type: icon3

type: horizontal-stack
cards:

type: "custom:atomic-calendar-revive"
name: "Scheduled events"
entities:

entity: calendar.automator
fullDayEventText: 'All day'
untilText: 'Until'
enableModeChange: true
showColors: true
maxDaysToShow: 30
maxEventCount: 0
showLocation: true
showMonth: true
showWeekDay: true
disableEventLink: true
disableLocationLink: true
showCurrentEventLine: false
dateColor: '#DDDDDD'
dateSize: 90
timeColor: '#cccccc'
timeSize: 80
titleColor: '#ffffff'
titleSize: 100
locationIconColor: 'rgb(230, 124, 115)'
locationLinkColor: '#BBBBBB'
locationTextSize: 90
dimFinishedEvents: true
finishedEventOpacity: 0.3
finishedEventFilter: 'grayscale(100%)'
dayWrapperLineColor: '#999999'
eventBarColor: '#aaaaaa'
showProgressBar: true
progressBarColor: '#aaaaaa'`

Thanks, next time I'm working on the card I will take a look at this and try figure out what's happening.
I'm currently working on the next feature release so will try get this fixed as part of that. If not I'll prioritise it for after.

@honkerst ok im not sure this is a bug as my test calendar with the above config appears to work at first glance.
have you followed the setup here specifically showing more than 5 events

You can see here all the daily scheduled events from  the 18th onwards.

No, dammit, I didn't see that bit about showing more than 5 events! Thanks a bunch! That completely solves my problem.
One last question, on some days I have more than 10 events. To get icons on every day of the monthly calendar I'd have to set max_results to 400 or 500.

I've tried out 500, and it seems to be fine. Loads quickly, and moving from month to month loads all events for all days.
I did notice that if I am in list mode and choose maxEventCount: 20 it correctly displays 20 events, but if I enableModeChange is set to true and I click the header to change modes there are 0 icons on the calendar, not even on the first two days.

Unfortunately i have never set it to something so high so wouldn't know.
As that is part of the HA integration for Google Calendar it may be worth asking on the HA forums :)

I did notice that if I am in list mode and choose maxEventCount: 20 it correctly displays 20 events, but if I enableModeChange is set to true and I click the header to change modes there are 0 icons on the calendar, not even on the first two days.

I will take a look at this :)

I take it all back. I didn't have 'type: icon3' on that calendar. It's working properly (in fact it's even better than expected, changing modes displays all 500 events, and list mode only shows the next 20, this is even better than expected).
You can close this ticket, thanks for all your help :-)

Glad all is working for you :)


Journal-week-3

Nice reflection on the struggles of week 3. :+1:

Good reflection and tutorial, keep on chugging along hermana! :+1:


Subscription.GetHashCode() throw exception when uuid is null.

Hi,
When I tried to retrieve a subscription with pending subscription, the pending subscription will have uuid null, which will cause exception when you call GetHashCode(); actually is called by AutoMapper.
Can someone fix that?
Thanks,

@nguyenphu1012 Thanks a lot for reporting this. I will try to reproduce it first thing in the morning. If it is a bug, it should be a simple fix.

I believe I fixed this same issue on a different object a while ago. GetHashCode() was overridden to get the hash code of the uuid and was sometimes null, causing an issue. I'll see if I can find it.

On Jun 16, 2016, at 9:47 PM, Benjamin Eckel<EMAIL_ADDRESS>wrote:
@nguyenphu1012 Thanks a lot for reporting this. I will try to reproduce it first thing in the morning. If it is a bug, it should be a simple fix.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@nguyenphu1012 Is this still a problem? I was never able to reproduce

@bhelx Pretty much any code that is overriding GetHashCode() should be removed. There's no reason to override it and only serves to create hard to reproduce issues. As stated by @nguyenphu1012 you should be able to reproduce this with Automapper when mapping from one type to another. I'd be happy to change this if you are okay with it.

It looks like the creator of the library intended for two recurly objects to be equal if their Id's were the same, in this case Uuid. I believe you could essentially take two completely different subscription objects with the same id, compare them, and determine that they are the same. So I believe the solution in this case is to check if the object is null first so that all null objects would get the same hashcode. I can start on this.


Have a single module with synced classes from keycloak/keycloak

Closes #34

Not 100% sure if the name of the maven module name can be only keycloak-client-common as the fact that sources are synced might be an "implementation detail" from the point of view of the consumer of that maven artifact? But does not matter much to me, so approving.

I just wanted to stress that this module is purely synced and should not be used directly by anybody.

Not 100% sure if the name of the maven module name can be only keycloak-client-common as the fact that sources are synced might be an "implementation detail" from the point of view of the consumer of that maven artifact? But does not matter much to me, so approving.

I just wanted to stress that this module is purely synced and should not be used directly by anybody.

That works for me. Thanks for the clarification!


The keyboard related version is too confusing

There are currently three known versions.
First, I think the most useful version is on the web page (https://google.github.io/blockly-samples/plugins/keyboard-navigation/test/index). This version is 0.6.10 according to the display. In this version, line_cursor can be used to select text in the block, which is very important to me.
In the future, I may use this function to realize the center focus of the main workspace block to solve the problem that too many blocks exceed the field of view and there is no edge scrolling function.
Then there is the keyboard-navigation plug-in installed through npm. The current version is 0.6.9, and the version in the repository is 0.6.11, but this has little impact.
Finally, the line_cursor in blockly-keyboard-experimentation needs to import {FieldColour} from '@blockly/field-colour', but I did not find @blockly/field-colour in the dependencies, so I need to install it myself using npm.After configuring const keyboardNav = new KeyboardNavigation(workspace), field-colour will overwrite the original style.

Hi @HenTaiCJN.
So, for the @blockly/keyboard-navigation plugin—like any of our other plugins:

The version on npmjs.com should match the version number in the google/blockly-samples repository—so npmjs.com being at 0.6.9 and the repo being at 0.6.11 is bad, and we are investigating this problem now as google/blockly-samples#2469.  (It looks like none of the plugins have actually been published for three months, even though the publish workflow runs every week and has not reported any failures.)
The version on github.io is updated by another workflow that runs each week, so it is also surprising that it was behind, but we see that it is now at v0.6.11 so it looks like everything is working fine.  Maybe you just happened to look between when the (failed) npm publish workflow updated the version number in the repo, and when the workflow that updates github pages ran.
The repo might have additional commits made since the last weekly publish run, so the actual code you get in the repo might be somewhere between v0.6.11 and whatever number the next published version gets (probably v0.6.12 but maybe v0.7.0 or v1.0.0).

The @blockly/keyboard-experimentation plugin is special.  It is based on work being done in this repository, where we're making a lot of experimental changes quickly, with less review and without worrying about breaking changes.  At some point when we have decided how we want keyboard navigation to work, we will replace the existing @blockly/keyboard-navigation plugin with a new version based on the work done here.
Neither of these are ready for production use.  The @blockly/keyboard-experiment plugin should be considered a very early beta for a new version of @blockly/keyboard-navigation, and we would welcome any feedback you have about how to make it better.