Decimal_128 type checks

This updates the type checks to be able to support 128-bit decimal. It mostly updates makes a distinction between decimal 64 support and decimal 128 support.  It also updates how we report both decimal and timestamp incompatibility in the docs and decimal incompatibility in the query explain.  The former is probably the biggest change and I would really like feedback on how it looks. This does cause a lot of merge conflicts so it might be best to hold off on this until 21.08 is really done.

build

build

build

build

Hopefully this is the last time I am going to need to upmerge. If the build passes before something else gets merged in that causes a conflict I will merge this.


Filtering malicious URLs

We're currently filtering malicious filetypes in our filetype filter - to be specific, we're filtering all filetypes we haven't specifically whitelisted. This does not, however, apply to URLs. That means it's easy enough to get around our filter by posting the file in a DM or on a different server, and then just linking the content instead.
This also allows users to post URLs that literally point at viruses, which isn't something we should permit.
Let's write a second filter that uses the same whitelist and same logics to filter URLs as well.
Let's keep this one simple - no need to download the file and inspect it to determine exactly what its filetype is. Just filter by extension, the way we're doing now.

Paste services sometimes include the extension in the URL which presents a problem. Either paste services would have to be whitelisted (there are so many out on the internet) or a separate whitelist would be needed for file extensions we don't want shared at all.

hmmm. that's a good point, @MarkKoz. Damn. I didn't think about that.
I'm not sure what the best approach here would be, but I'm open to suggestions.

I think regex will be good to use in this case

Not sure if this is doable without a whitelist of some kind; without keeping some kind of whitelist/blacklist for urls we could keep a list of extensions we don't want through (commonly malicious stuff like rar, exe, etc.) and if the url path ends with one of them, block the message. This shouldn't interfere with paste services since they'll end with extensions that won't be there, but some sites may use the "extension" for something else.

Since there’s no way to filter links based on extensions without having a lot of false positives, let’s close this. Feel free to add a comment if you can think about a better way of doing this.


How to map leader+key to pending motion operations

One command I used all the time was change inside something, usually inside quotes or brackets. So often that I had that mapped to  leader + c.
Is there any way I can achieve something similar in Atom's Vim mode?
Cheers.

+1, I'd like to create a shortcut with leader,b to switch "buffers" and leader,f to find files (Atom's command+T). Any hints?

@fullofcaffeine Leader is just a convention.  If you want space to be the mapleader, then adding this to keymap.cson should work:
'atom-text-editor.vim-mode.command-mode':
  'space b': 'fuzzy-finder:toggle-buffer-finder'
  'space f': 'fuzzy-finder:toggle-file-finder'

@lmartins I don't understand...  It sounds like you want to map leader+c to be the same as c+i... but that doesn't save any keystrokes!  What is the exact vim mapping you want to emulate?

@bronson Scott, thank you for your feedback on this.
In Vim I had the following mappings:
" ChangeInside
nnoremap <silent> <Leader>c :ChangeInsideSurrounding<CR>
nnoremap <silent> <Leader>C :ChangeAroundSurrounding<CR>

This is calling a function that I assume will work for any surroundings (single quote, quotes, brackets, etc).
I was looking for a easier way to trigger the change inside quotes and/or single quotes with the same shortcut. Is this possible?
The other use case was what you already described to @fullofcaffeine , thanks :)

:ChangeInsideSurrounding doesn't exist in vim...  I guess you're using this plugin?
https://github.com/briandoll/change-inside-surroundings.vim
If so then yes, that will require writing an Atom plugin.  Which, I'm happy to say, is actually not too hard.

Thank for your feedback on this. Closing now.

atom calls "command" mode  "normal" mode, so custom bindings will looks like this:
'atom-text-editor.vim-mode.normal-mode':
  'space b': 'fuzzy-finder:toggle-buffer-finder'
  'space f': 'fuzzy-finder:toggle-file-finder'
  'space p': 'core:save'

Thanks @willopez that helped.

Hey @lmartins vim comes out of the box with the ability to change inside!
given this line of javascript:
let example = 'change-me';

you can do the following
f'ci'changed!

^^ I usually misunderstand what this actually means so I will break it down for ya
f finds the next character on the current line, in this case it is ' (shift f, F searches backwards)
ci' is the vim function that means change inside the quote tag (you can do most things that wrap)
changed! is the string i chose to replace it with as you are now in insert mode
and the fun doesn't stop there! Check this one out!
  {
    form: 'observation',
    fields: ['name', 'number', 'description', 'id'],
    asyncValidate,
  },

let's say you have this structure ^^
As long as your cursor is anywhere inside of the braces{} you can just do ci{ and you will see this:
{}

with your cursor loaded and ready to fire in insert mode
Because of how awesome ViM is i bet you can guess what di{ does!
And since I am just going full blog post on this comment, here is a great stack overflow post with some great basic vim commands!
http://stackoverflow.com/questions/1276403/simple-vim-commands-you-wish-youd-known-earlier
Cheers! :)

Hey Michael, I've spent a long time trying to move from Atom to Vim just for things like this. Im glad Atom Vim support is improving as I believe that merging Vim features with a modern UI like Atom is the way forward.
Many thanks for your comments and useful resources. Cheers ;)

course! i completely agree, vim-mode is fantastic for pairing with my co-workers that don't know vim, which is what i think is the one downfall of only doing vim :(

Thanks :+1:


Apollo client with id default not found

Describe the bug
When I try to use the apollo client that vue uses outside of a component and also use provideApolloClient I get the error Apollo client with id default not found. use ProvideApolloClient() if you are outside of a compnent setup
To Reproduce
const client = provideApolloClient(apolloClient);
function something() {
  const { result } = client(() => useGraphQLGeneratorQuery());
  ....

Expected behavior
if provideApolloClient is called with the actual client, it should be found.
Versions
vue: 3.3.4
@vue/apollo-composable: ^4.0.0-beta.8
@apollo/client: 3.7.17
Additional context
Add any other context about the problem here.

I have next composable and many others here working example
import { provideApolloClient } from '@vue/apollo-composable'
import { apolloDefaultClient } from '@/modules/shared/plugins/apollo/vue-apollo'
import NotificationRead from '../graphql/mutations/notificationRead.gql'
import { toRawRecursive } from '@/modules/home/utils'
import { Notifications } from '@/modules/shared/plugins/apollo/schemaTypesGenerated'
import { DocumentNode } from 'graphql/index'
import { TypedDocumentNode } from '@graphql-typed-document-node/core'
import GetMyNotificationsTotalUnread from<EMAIL_ADDRESS>
provideApolloClient(apolloDefaultClient)
const defaultClient = useApolloClient('default').client

// clientId: 'public'
const { mutate, loading, onError, onDone } = useMutation(NotificationRead)

export default function useMutationNotificationRead() {
  const notificationRead = (uuid: string, query: DocumentNode | TypedDocumentNode, variables: object) => {
    mutate(
      { uuid },
      {
        update: () => {
          const rawVariables = toRawRecursive(variables)
          const apolloCachedData = defaultClient.cache.readQuery({ query, variables: rawVariables })
          if (!apolloCachedData) return

          const data = JSON.parse(JSON.stringify(apolloCachedData.myNotifications.data))
          const newData = {
            myNotifications: {
              ...apolloCachedData.myNotifications,
              data: data.map((myNotification: Notifications) => {
                return myNotification.uuid === uuid ? { ...myNotification, read_at: dayjs().toISOString() } : myNotification
              }),
            },
          }
          defaultClient.cache.writeQuery({ query: query, variables: rawVariables, data: newData })

          // reset counter for query myUnreadTotalNotifications
          const apolloCachedUnreadTotal = defaultClient.cache.readQuery({ query: GetMyNotificationsTotalUnread })
          if (!apolloCachedUnreadTotal) return

          const newUnreadNotification = {
            myUnreadTotalNotifications: {
              // @ts-expect-error property exist
              ...apolloCachedUnreadTotal.myUnreadTotalNotifications,
              // @ts-expect-error property exist
              total: apolloCachedUnreadTotal.myUnreadTotalNotifications.total - 1,
            },
          }
          defaultClient.cache.writeQuery({ query: GetMyNotificationsTotalUnread, data: newUnreadNotification })
        },
      },
    )
  }

  return { notificationRead, loading, onError, onDone }
}

Hi, I'm in the process of migrating to Vue Apollo 4 and, it's proving to be an insurmountable ordeal.
Are there any actual simple working examples out there?


Import address feature for dependency manager

Issue To Be Solved
(Optional): Suggest A Solution
It would be nice to have an option to import an address with all contracts and dependencies from network.

I like this idea! To clarify: the idea is to add every contract and dependencies found from an address, correct? For example, if no contract name was provided flow dependencies add testnet://7e60df042a9c0868. Then everything from 7e60df042a9c0868 would be installed and setup. This would setup contract a project quickly for an address.

On Thu May 23, 2024 at 5:25 PM UTC, Chase Fleming wrote:

I like this idea! To clarify: the idea is to add every contract and dependencies found from an address, correct? For example, if no contract name was provided flow dependencies add testnet://7e60df042a9c0868. Then everything from 7e60df042a9c0868 would be installed and setup. This would setup contract a project quickly for an address.

Yes, that's correct. The idea is to make it easy to add all the contracts
and their dependencies from a given address. This will make a lot easier
for developers to clone a project / or setup their flow.json file from active
configuration on mainnet/testnet quickly.


fix build error

Fix #573
This is a first PR, so if there are any problems, please tell me 🙇

Thanks!


add job to setup grafana-dev via gitops

Signed-off-by: Song Song Li<EMAIL_ADDRESS>

/retest

/test test-e2e

/test test-e2e

/hold

/test test-e2e

trace in https://github.com/stolostron/acm-aap-aas-operations/pull/4 let me close this one.


Sample Label tool is not recognizing tables in the invoice PDF(Azure Form Recognizer)

Here is my sample invoice pdf, I need to tag the table and all its rows and columns. The Sample tool is not recognizing the table. How would I train my model with the table data?

Hi @GowthamDisys, can you share with azure documentation you're following?

Hi @Grace-MacJones-MSFT ,
I am following the below MSFT docs for training form recognizer with sample labeling tool.
https://docs.microsoft.com/en-us/azure/cognitive-services/form-recognizer/quickstarts/client-library?pivots=programming-language-csharp&tabs=windows
https://docs.microsoft.com/en-us/azure/cognitive-services/form-recognizer/quickstarts/label-tool?tabs=v2-0

@GowthamDisys The table data should be automatically recognized by the tool as mentioned here. I think the issue here might be with the format of the image where visually it looks like table data but clear boundaries might be missing. @NHaiby Could you please confirm if the table format in the image could be the issue here?

@RohitMungi-MSFT The tool is unable to recognize the table is my query. There are other services in the market, which can recognize these types of tables. Most of the invoice documents won't have clear boundaries in it. It would be useful, only if the tool is able to recognize these types of tables or if the user has a choice to tag these types of tables as "table", which is not implemented right now in form recognizer.

New and enhanced table extraction will be available shortly.
#please-close


online argument is not working

Check the _check_params: It is not catching the indices dict when using online=True

Done!


New activity future messages

We need some new messages to support running activities in parallel with other workflow operations.  The current activity execute messages execute and then wait for the result,  We need to be able break this up into operations, one that starts the activity and returns a reference to the activity promise and a second operation that waits for the promise to complete.
We'll need messages for normal and local activities:

[x] ActivityStartRequest/Reply

ActivityStartRequest should start a normal activity but not wait for the future to return the result.  Instead, the cadence-proxy should persist the future in a local workflow map keyed by the ActivityStartRequest.ActivityId property (a long).  The reply simply indicates that the activity has been started.

[x] ActivityGetResultRequest/Reply

ActivityGetResultRequest should call Get() on the local activity promise keyed by ActivityId and then send a ActivityGetResultReply with the result.

[x] ActivityStartLocalRequest/Reply

ActivityStartLocalRequest should start a localactivity but not wait for the future to return the result.  Instead, the cadence-proxy should persist the future in a local workflow map keyed by the ActivityStartLocalRequest.ActivityId property (a long).  The reply simply indicates that the activity has been started.

[x] ActivityGetLocalResultRequest/Reply

ActivityGetLocalResultRequest should call Get() on the local activity promise keyed by ActivityId and then send a ``ActivityGetLocalResultReply` with the result.

@johncburns1: All of the messages above have been changed from inheriting from ActivityRequest/ActivityReply to inheriting from WorkflowRequest/WorkflowReply.
I realized that I need the current workflow replay status from you for these operations.
This change is pushed to cadence.

DONE.


Ability to sort bookmarks by website into a specific folder

I always want pages I bookmark on onlyinyourstate.com to always be in my "onlyinyourstate.com" folder in Other Bookmarks on Firefox for Windows, macOS, iOS, and Android. I think a great new version 3.0 feature would be to allow me to specify this in the add-on. I will contribute $25 if the developers will consider this!

Similarly, I always want everything I bookmark on washingtonian.com to be in my "washingtonian.com" folder in Other Bookmarks. This'll save me the trouble of hunting and fishing for the folder each time in Firefox's bookmarks editor! I hope to see a version 3.0 sometime in the next 365 days, by Oct. 28, 2019!

I can definitely see how having a system matching a specific URL (or pattern) to a predetermined folder would be useful! That is good idea. 🙂 We could even envision the ability to apply this system to the bookmarks created via the built-in system and/or the quick bookmarking system (depending on the user's preferences).
I have added this feature request to the 3.0 milestone. Hopefully I will be able to release it before a full year has passed. 😉

Bookmark Flypaper, an add-on that no longer works in Quantum and doesn't appear to be developed anymore, was able to automatically save any opened tab bookmark to a specific folder if it matched a certain regex expression. An options page allowed you to see a tree of your bookmark folders where you could enter the regex expressions.
https://addons.mozilla.org/en-US/firefox/addon/bookmark-flypaper/
So same idea, but with automatic bookmarking.


Can't open multiple instances of Museeks

I think it's due to a database lock issue. Do you think it's fine if we can't open multiple instances of Museeks at a time ?

Not sure. Think about concurrent database access.

I am fine with it. There is almost no point opening  Museeks more than once.

What is the interest of opening multiple instances of Museeks ?
I like the way it behave now.

ref: http://electron.atom.io/docs/api/app/#appmakesingleinstancecallback


Cancelling? Using with Hammer.js

Using vue-pull-refresh (VPR) with hammer.js works well, but...
When panning left/right on a page with a VPR component, it would be nice to be able to tell VPR to cancel (resetPullDown) and have VPR stop processing events, but this is private to the mounted/$nextTick handler.
Exposing resetPullDown (I know it's not that simple) would allow both panning and VPR functionality to co-exist.  Currently both work, but from a user perspective, once panning begins the VPR functionality should cease.

I've implemented a patch that seems to work for me.
1 - At the beginning of "touchmove":
if (this.cancelPull) { this.pullDown.height = 0; } else
2 - At beginning of "touchstart":
this.cancelPull = false;
3 - Then, from the wrapped component, find VPR ($children[0]) and simply set vpr.cancelPull = true; -- the pull will be cancelled and the puller reset.  From what I can tell, I don't think it matters how and when cancelPull gets set.
Works like a charm!

@garyskiba Maybe you can make a pull request ?

OK.  Now I have to create a project, etc.  :-)
That's ok.  I like your component and will try to get this done in the next
few days.
BTW, a return is required, not else (I had done it differently than I
showed).
-Gary
On Thu, Nov 30, 2017 at 8:07 PM, LinBin<EMAIL_ADDRESS>wrote:

@garyskiba https://github.com/garyskiba Maybe you can make a pull
request ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lakb248/vue-pull-refresh/issues/14#issuecomment-348380781,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABWGM7JywlLZhlmPakNo8AbJDgZGaAK9ks5s719kgaJpZM4QvxxX
.


Create a gallery page

Edit gallery.html
Add a breadcrumb and heading like in administration page.
Add an image grid, So each grid items contain an image and a heading.

@abhijithvijayan Do you have any suggestions?

@ashikmeerankutty
Adding a hover story is also a good choice

#207
Further Modifications Needed

Here is your screenshot.


Intinite loop on deploy when adding @unique to column

Current behavior
upon updating a schema in a specific way, deploy goes into infinite loop with a step
Reproduction
deploy this schema
type ContentProvider {
  id: ID! @unique
  type: ContentProviderType!
}

enum ContentProviderType {
  Twitter
}

insert 2 records with the Twitter type:
mutation {
  t1: createContentProvider( data: {
    type: Twitter
  }) {
    id
  }

  t2: createContentProvider( data: {
    type: Twitter
  }) {
    id
  }
}

update the schema by adding @unique to type field. This should cause SQL exception since the schema violates the existing data in the database.
type ContentProvider {
  id: ID! @unique
  type: ContentProviderType! @unique
}

enum ContentProviderType {
  Twitter
}

what I'm getting on console is repeated entries of these lines:
client Sending query to cluster local +505ms
  client http://localhost:4466/cluster +1ms
  client query ($name: String! $stage: String!) {
  client           migrationStatus(name: $name stage: $stage) {
  client             projectId
  client             revision
  client             status
  client             applied
  client             rolledBack
  client             errors
  client           }
  client         }
  client          +1ms
  client { stage: 'dev', name: 'foo' } +2ms
  client { migrationStatus:
  client    { rolledBack: 0,
  client      applied: 6,
  client      projectId: 'foo@dev',
  client      errors: [],
  client      revision: 2,
  client      status: 'SUCCESS' } } +20ms

Expected behavior?
Meaningful error message "translated" from the SQL error that this schema modification is not possible.

Setting a  unique constraint will error if there are already duplicates present in the case of a field update or if a new field is required and unique since SQL will set default values for the newly required fields. I'm closing this issue since it seems to show symptoms of this problem. With the deploy changes from 1.6.1 and this PR the issue should be fixed https://github.com/graphcool/prisma/pull/2371
If anyone is still experiencing problems please open a new issue with a reproduction.

I am still able to make an infinite loop. This is a database configuration issue on my end, but errors show up for the Migration and requests are continually sent. An exit is expected rather than an infinite loop.

Hi @gregwebs, please create a new issue in which you share a "before" and "after" version of your datamodel that causes the infinite loop. There are other known issues that cause this symptom collected here: #2018.

My point is not that I have a particular case that causes an infinite loop, but that an infinite loop should not be possible. prisma appears to have an unbounded error retry in some error situations during deploy.


Fence Gates?

I mean, there are Trapdoors and Doors, why not fence gates?

Its on the todo for when I have time to make gate models


model editor

𝗤𝘂𝗲𝘀𝘁𝗶𝗼𝗻
A clear and concise description
𝗛𝗼𝘄 𝘄𝗲 𝗽𝗿𝗶𝗼𝗿𝗶𝘁𝗶𝘇𝗲 𝗶𝘀𝘀𝘂𝗲𝘀
We will try to answer all questions that do not require research within 24hr.
To prioritize cases that require research we use the following labels in order.

❇ Exclusive-Services
❤ Bronze Sponsor
❤ Sponsor
❤ Backer
Installation
ShowStopper
Nuget
Contribution
BreakingChange
ReproSample
Deployment
Must-Have

For all other issues the posting time is respected.

I created the ticket by accident.


icon not show for Hugeicons

Describe the bug

Reproduction
Development
System Info
System:
    OS: macOS 13.6.7
  Binaries:
    Node: 20.15.0
    bun: 1.1.17
  Browsers:
    Edge: 126.0.2592.81

Used Package Manager
bun
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

Contributions

[ ] I am willing to submit a PR to fix this issue
[ ] I am willing to submit a PR with failing tests (actually just go ahead and do it, thanks!)

v0.9 should fix it


ICRC-3 timestamp nanoseconds?

The specification doesn't define if the ts timestamp field should be in seconds, milliseconds or nanoseconds.
Assuming it's required to be in nanoseconds.
https://github.com/dfinity/ICRC-1/blob/main/standards/ICRC-3/README.md

Actually ts isn't part of the ICRC3 Generic Block Schema, just ICRC1 and ICRC2


Apollo5.0: How to Run Perception Module on Your Local Computer can't launch camera

System information
-*ubuntu14.04

*binary
**Apollo version 5.0

Steps to reproduce the issue:
cyber_launch start /apollo/modules/perception/production/launch/perception_camera.launch
can't work.
Supporting materials (screenshots, command lines, code/script snippets):
[mainboard_default_6443]  E1024 13:11:50.780318  6451 class_loader_utility.cc:220] [mainboard] poco LibraryLoadException: libcuda.so.1: cannot open shared object file: No such file or directory
[mainboard_default_6443]  E1024 13:11:50.780418  6451 class_loader_utility.cc:236] [mainboard] poco shared library failed: /apollo/bazel-bin/modules/perception/onboard/component/libperception_component_camera.so
[mainboard_default_6443]  E1024 13:11:50.780437  6451 class_loader_manager.h:70] [mainboard] Invalid class name: FusionCameraDetectionComponent
[mainboard_default_6443]  E1024 13:11:50.780453  6451 module_controller.cc:59] [mainboard] Failed to load module: /apollo/modules/perception/production/dag/dag_streaming_perception_camera.dag
[mainboard_default_6443]  E1024 13:11:50.780463  6451 class_loader_utility.cc:258] [mainboard] Attempt to UnloadLibrary lib, but can't find lib: /apollo/bazel-bin/modules/perception/onboard/component/libperception_component_camera.so
[mainboard_default_6443]  E1024 13:11:50.780469  6451 mainboard.cc:43] [mainboard] module start error.
[mainboard_default_6443]
[cyber_launch_6443] ERROR Process [mainboard_default_6443] has finished. [pid 6451, cmd mainboard -d /apollo/modules/perception/production/dag/dag_streaming_perception_camera.dag /apollo/modules/perception/production/dag/dag_motion_service.dag -p mainboard_default_6443 -s CYBER_DEFAULT].
[cyber_launch_6443] INFO All processes has died.
[cyber_launch_6443] INFO Cyber exit.
[cyber_launch_6443] INFO All processes have been stopped.

It is complaining about not being able to find cuda. Can you verify that the library is available inside your docker? Running ldconfig -p | grep libcuda.so should return a line like libcuda.so.1 (libc6,x86-64) => /usr/lib/x86_64-linux-gnu/libcuda.so.1. If the command does not return anything, can you verify if nvidia-smi returns the correct output in your docker?

Thank you very much for your answer. You are right. I have solved this problem, but there are other problems here.
[mainboard_default_15781]  [NVBLAS] NVBLAS_CONFIG_FILE environment variable is NOT set : relying on default config filename 'nvblas.conf'
[mainboard_default_15781]  [NVBLAS] Cannot open default config file 'nvblas.conf'
[mainboard_default_15781]  [NVBLAS] Config parsed
[mainboard_default_15781]  [NVBLAS] CPU Blas library need to be provided
[cyber_launch_15781] ERROR Process [mainboard_default_15781] has finished. [pid 15800, cmd mainboard -d /apollo/modules/perception/production/dag/dag_streaming_perception_camera.dag /apollo/modules/perception/production/dag/dag_motion_service.dag -p mainboard_default_15781 -s CYBER_DEFAULT].
[cyber_launch_15781] INFO All processes has died.
[cyber_launch_15781] INFO Cyber exit.
[cyber_launch_15781] INFO All processes have been stopped.
it can't open nvblas.conf

Normally, this file nvblas.conf is not needed in order to run the perception. You can get rid of this error message, by downloading a default nvblas file from nvidia, but this is not related to the crash you are experiencing: https://docs.nvidia.com/cuda/nvblas/index.html#configuration_example
You can run the command: gdb --args mainboard -d /apollo/modules/perception/production/dag/dag_streaming_perception_camera.dag to attach a debugger to the program and then have a look at the stack trace when your process crashes. Maybe this helps you identify possible issues with your setup.

Normally, this file nvblas.conf is not needed in order to run the perception. You can get rid of this error message, by downloading a default nvblas file from nvidia, but this is not related to the crash you are experiencing: https://docs.nvidia.com/cuda/nvblas/index.html#configuration_example
You can run the command: gdb --args mainboard -d /apollo/modules/perception/production/dag/dag_streaming_perception_camera.dag to attach a debugger to the program and then have a look at the stack trace when your process crashes. Maybe this helps you identify possible issues with your setup.

Thank you. Now I can see the specific error report. When I turn on the camera, my collection screen will appear on the screen, and then turn on the perception module, and this will be displayed.
[mainboard_default_6060]  E1029 15:18:06.564347  6092 transform_wrapper.cc:224] [mainboard] Can not find transform.<PHONE_NUMBER>.486972 frame_id: novatel child_frame_id: front_6mm Error info: canTransform: target_frame novatel does not exist. canTransform: source_frame front_6mm does not exist.canTransform: target_frame novatel does not exist. canTransform: source_frame front_6mm does not exist.canTransform: target_frame novatel does not exist. canTransform: source_frame front_6mm does not exist.canTransform: target_frame novatel does not exist. canTransform: source_frame front_6mm does not exist.canTransform: target_frame novatel does not exist. canTransform: source_frame front_6mm does not exist.
[mainboard_default_6060]  E1029 15:18:06.564409  6092 fusion_camera_detection_component.cc:684] [perception] failed to get camera to world pose, ts:<PHONE_NUMBER>.486972 camera_name: front_6mm
[mainboard_default_6060]  E1029 15:18:06.564422  6092 fusion_camera_detection_component.cc:299] [perception] InternalProc failed, error_code: 4001

@cxr1996
If you have your own training data or data from nuscenes-devkit, argoverse or waymo-open-dataset, you could config the transform based on the extrinsic parameters the specific dataset provided.
Besides that, you may also disable traffic light detection, there is some HDMap definition mismatch when I test it in Aug. It must be a developing feature. I'm not sure it has been updated or not now.

The error you see is because no static transform topic is launched: You have to publish a topic to /static_transform with the correct transforms. If you use the same setup (or don't care about the error introduced) you can just launch it with the extrinsics of the reference vehicle. In that case, just run cyber_launch start modules/transform/launch/static_transform.launch. Otherwise, create a new dag file in modules/transform/dag with a path to a new config, which you have to create yourself as well. There you can set up the extrinsics files, which contain the actual parameters, look at /apollo/modules/transform/conf/static_transform_conf.pb.txt for an example for that file.
That being said, normally, this error is just getting spammed into the log, it does not make the program crash. So, for debugging, it might be good enough to just publish the default transforms and verify if that actually fixes the problem.

The error you see is because no static transform topic is launched: You have to publish a topic to /static_transform with the correct transforms. If you use the same setup (or don't care about the error introduced) you can just launch it with the extrinsics of the reference vehicle. In that case, just run cyber_launch start modules/transform/launch/static_transform.launch. Otherwise, create a new dag file in modules/transform/dag with a path to a new config, which you have to create yourself as well. There you can set up the extrinsics files, which contain the actual parameters, look at /apollo/modules/transform/conf/static_transform_conf.pb.txt for an example for that file.
That being said, normally, this error is just getting spammed into the log, it does not make the program crash. So, for debugging, it might be good enough to just publish the default transforms and verify if that actually fixes the problem.

Ok, thank you. I also found that this log does not affect the process to continue to run, the camera-based perception module is actually running. Another question, if I want to see the visual object detection box, what should I do? My record is visualized lane results, "If you want to visualize lane results overlaid on the captured image and in bird view, mark enable_visualization: True in modules/perception/production/conf/perception/camera/lane_detection_component config before executing the above command, It will pop up when you play the recorded data in point 9, "according to the set already.

If you have your own training data or data from nuscenes-devkit, argoverse or waymo-open-dataset, you could config the transform based on the extrinsic parameters the specific dataset provided.
Besides that, you may also disable traffic light detection, there is some HDMap definition mismatch when I test it in Aug. It must be a developing feature. I'm not sure it has been updated or not now.

@cxr1996
If you have your own training data or data from nuscenes-devkit, argoverse or waymo-open-dataset, you could config the transform based on the extrinsic parameters the specific dataset provided.
Besides that, you may also disable traffic light detection, there is some HDMap definition mismatch when I test it in Aug. It must be a developing feature. I'm not sure it has been updated or not now.

Ok, thank you very much for your answer.

@cxr1996  hi, i have the same issue when compile pperception in my local computer.
E0212 09:17:52.781256 26092 class_loader_utility.cc:220] [mainboard] poco LibraryLoadException: libcuda.so.1: cannot open shared object file: No such file or directory E0212 09:17:52.781301 26092 class_loader_utility.cc:236] [mainboard] poco shared library failed: /apollo/bazel-bin/modules/perception/onboard/component/libperception_component_camera.so E0212 09:17:52.781332 26092 class_loader_manager.h:70] [mainboard] Invalid class name: LaneDetectionComponent E0212 09:17:52.781340 26092 module_controller.cc:59] [mainboard] Failed to load module: /apollo/./modules/perception/production/dag/dag_streaming_perception_lane.dag E0212 09:17:52.781347 26092 class_loader_utility.cc:258] [mainboard] Attempt to UnloadLibrary lib, but can't find lib: /apollo/bazel-bin/modules/perception/onboard/component/libperception_component_camera.so E0212 09:17:52.781353 26092 mainboard.cc:43] [mainboard] module start error.
can you share how you solve the problem?

@xmyqsh Hi, Can you use the external dataset that you mentioned before including nuscenes-devkit, argoverse or waymo-open-dataset to test apollo system such as perception, prediction, planing module? If so, Can you tell me how to convert the external data into the format understood by Apollo? I don’t know how to get started. Thank you in advance.

Hello @christian-lanius
I have a similar problem by launching v2x. I receive the error below:
[/apollo/bazel-bin/modules/v2x/v2x --flagfile=/apollo/modules/v2x/conf/v2x.conf]  /apollo/bazel-bin/modules/v2x/v2x: error while loading shared libraries: libfastcdr.so.1: cannot open shared object file: No such file or directory
However, I run "ldconfig -p | grep libcuda.so" and the terminal shows me:
libcuda.so.1 (libc6,x86-64) => /usr/lib/x86_64-linux-gnu/libcuda.so.1
libcuda.so (libc6,x86-64) => /usr/lib/x86_64-linux-gnu/libcuda.so
Do you have any suggestions?

@cxr1996 hi, i have the same issue when compile pperception in my local computer.
E0212 09:17:52.781256 26092 class_loader_utility.cc:220] [mainboard] poco LibraryLoadException: libcuda.so.1: cannot open shared object file: No such file or directory E0212 09:17:52.781301 26092 class_loader_utility.cc:236] [mainboard] poco shared library failed: /apollo/bazel-bin/modules/perception/onboard/component/libperception_component_camera.so E0212 09:17:52.781332 26092 class_loader_manager.h:70] [mainboard] Invalid class name: LaneDetectionComponent E0212 09:17:52.781340 26092 module_controller.cc:59] [mainboard] Failed to load module: /apollo/./modules/perception/production/dag/dag_streaming_perception_lane.dag E0212 09:17:52.781347 26092 class_loader_utility.cc:258] [mainboard] Attempt to UnloadLibrary lib, but can't find lib: /apollo/bazel-bin/modules/perception/onboard/component/libperception_component_camera.so E0212 09:17:52.781353 26092 mainboard.cc:43] [mainboard] module start error.
can you share how you solve the problem?

Have you solved this problem? I'm facing the same problem and don't know how to solve it.


mixpanel reserved properties

From @yields on January 13, 2014 23:1
https://mixpanel.com/docs/properties-or-segments/special-or-reserved-properties
Copied from original issue: segmentio/analytics.js-integrations#35

From @DirtyAnalytics on January 13, 2014 23:3
Some of the properties already get taken care of, we're just missing a few.
$last_login and $ignore are requested by the customer who asked for this.

From @calvinfo on January 13, 2014 23:16
yeah, .active() is what you want

From @ianstormtaylor on January 13, 2014 23:12
https://mixpanel.com/docs/people-analytics/people-http-specification-insert-data
cc @calvinfo i believe we support the $ignore_time through another property but cant remember. calv can you point amir in the right direction?
@DirtyAnalytics i dont see a straight up $ignore did you mean ignore time?
for $last_login i'd want to normalize that across services. we usually refer to it as lastSeen unless login is actually the act of logging in, but that doesnt seem as useful. dunno if mixpanel distinguishes.

From @DirtyAnalytics on January 13, 2014 23:24
I might have messed up. Let me check with the user who asked about this.

From @DirtyAnalytics on February 10, 2014 15:14
Just had a request to add $signup which is an event name, but Mixpanel does use it for some things in their UI. We would want to change "Signed up" and "Signup" to  $signup when we send that event to Mixpanel.

From @ianstormtaylor on February 10, 2014 19:39
mm yeah that makes sense. luckily the way amir implemented it with regexps it should be a pretty easy check. we might want to start doing /view(ed)? product/ for them all @yields

From @robvolk on January 14, 2014 23:2
Hey guys, I'm the customer.  Thanks for jumping on this so quickly.  Once it's in place, we can migrate our Mixpanel scripts to Segment.io.
It looks like Mixpanel's docs aren't complete:
$ignore is described here: https://mixpanel.com/help/questions/articles/can-i-exclude-myself-from-mixpanel-data
I'm not sure where I found $last_login, but it looks like you guys may support it already with lastSeen:
http://www.tuicool.com/articles/ZBFv6j

From @seth-admittedly on February 14, 2014 14:57
Supporting mixpanel's $ignore would be a big help to us, as it's the easiest way of excluding our team from our stats.

this sounds more of a platform spec question. One could argue that segment should spec something out for completing omitting sending data for certain users based on an optional flag, similar to how we treat .active().
active is to denote whether that .identify() call should be passive (just update traits but not last_seen). We don't have a spec for something like $ignore.
The mixpanel docs linked says ideally you want to add custom logic to catch internal employees and add the $ignore flag but why not just not call .identify() at all? Assuming you wouldn't want your internal employee data going to any tool.
Will close this issue for inactivity as well but definitely something we should discuss internally regarding if we want to spec something like this.


add operator-class to only reconcile resources with mongodb.com/atlas-operator-class= annotation

Related ticket: CLOUDP-154088

Closing as namespaced secret ref should fix it for me.


Server crash on 1.16.3

Issue
Server may crash when attacking other entities.
Environment
Forge 1.16.3 server with MITE Reborn R04.1 only.
Forge 1.16.3 client with MITE Reborn R04.1 only.
Error message

Updating MITE:Reborn to Fabric 1.17, which should fix many of the old issues with the 1.16 version


[Tech] Use the new Coil 2.3.0 FakeImageLoaderEngine api for testing

Introduce a new coil-test artifact, which includes FakeImageLoaderEngine. This class is useful for hardcoding image loader responses to ensure consistent and synchronous (from the main thread) responses in tests. See here for more info.

We're blocked since we have to wait for https://github.com/adevinta/spark-android/pull/281 to be merged first.
We can use FakeImageLoaderEngine for our previews currently and maybe our demo app as well?

We are currently using version 2.5.0 and the test artifact is available.
Although, I'm not sure why we would use it in this project, since we can already fake the input in screenshot tests.
@soulcramer should we close this, or did you have something specific in mind?


Create a calibration code for IMU

Create a calibration method based on Application Notes.

Version 0.0.4  implemented it! (Commit 087dc8436b44231fd30c230e1572f3e7a24e2149)


Many problem after updating to v3.6.7, spam console every minutes

Greeting,
You can find the console error related :
https://pastebin.com/abv8QkM9
Regards

Dear Noiknez,
Can you try a clean install, if that is not working can you give some more information - the error you have shared gives 0,0 information sadley.
Kind regards,
Tycho
MinecraftMarket

Hello,
I have a more precise error that appear on server starts if this can help : https://pastebin.com/mzUpKbf8
Regards

Dear Noiknez,
Your guiLayoutConfig.yml shoud not be okay - specific your GUIRows need to be higher or equal to 1.
Kind regards,
Tycho
MinecraftMarket


Update munit, munit-scalacheck to 0.7.29

Updates

org.scalameta:munit
org.scalameta:munit-scalacheck

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

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta" } ]

labels: test-library-update, semver-patch

Codecov Report

Merging #177 (f90c2db) into master (e9fa163) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #177   +/-   ##
=======================================
  Coverage   84.04%   84.04%
=======================================
  Files          19       19
  Lines         282      282
  Branches        6        6
=======================================
  Hits          237      237
  Misses         45       45

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 e9fa163...f90c2db. Read the comment docs.


Integration Test - Release - 2019/02/07 22:09:29

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

@PRMergerTest2 : Thanks for your contribution! The author, @v-caxian, has been notified to review your proposed change.


What is the theme used in that screenshot?

It looks like a very nice light theme.

Looks like Atom One Light to me.


I am a greener. Just ask a simple question about the toolchain in the container.

// For format details, see https://aka.ms/devcontainer.json. For config options, see the
// README at: https://github.com/devcontainers/templates/tree/main/src/rust
{
	"name": "Rust",
	// Or use a Dockerfile or Docker Compose file. More info: https://containers.dev/guide/dockerfile
	"image": "mcr.microsoft.com/devcontainers/rust:1-1-bullseye"

	// Use 'mounts' to make the cargo cache persistent in a Docker Volume.
	// "mounts": [
	// 	{
	// 		"source": "devcontainer-cargo-cache-${devcontainerId}",
	// 		"target": "/usr/local/cargo",
	// 		"type": "volume"
	// 	}
	// ]

	// Features to add to the dev container. More info: https://containers.dev/features.
	// "features": {},

	// Use 'forwardPorts' to make a list of ports inside the container available locally.
	// "forwardPorts": [],

	// Use 'postCreateCommand' to run commands after the container is created.
	// "postCreateCommand": "rustc --version",

	// Configure tool-specific properties.
	// "customizations": {},

	// Uncomment to connect as root instead. More info: https://aka.ms/dev-containers-non-root.
	// "remoteUser": "root"
}

I had added the rust dev container. But I stiil need a cmake. How can I add the cmake?
And if I want to upgrade the rust version, What can I do ?

Hello 👋

And if I want to upgrade the rust version, What can I do ?

We try to keep the mcr.microsoft.com/devcontainers/rust:1-1-bullseye image up-to-date. However, you can install another version of rust with the Rust Feature.
Feel free to look at https://containers.dev/ for more information.

But I stiil need a cmake. How can I add the cmake?

You can use a Dockerfile instead with the following contents and point the devcontainer.json to use it (ie In your devcontainer.json file, make sure to set "dockerfile": "Dockerfile" to use the Dockerfile you just created). Read more at https://containers.dev/guide/dockerfile
FROM mcr.microsoft.com/devcontainers/rust:1-1-bullseye

# Install CMake
RUN apt-get update && \
    apt-get -y install cmake


Sometimes the wrong number of total available song files in the status bar is not correct.

Can't reproduce.

I'll try to reproduce it, but can't remember when/how it happened.


[Request] Retrieving blob list based on tags filter using procedure ListBlobs from ABS Blob Client codeunit

Hello,
We are running into the following issue with codeunit 9053 ABS Blob Client.
We need to retrieve a list of blobs from Azure Blob Storage, but in the result set we only want to see blobs where one of the tags has a specific value. E.g. I only want to see the blobs in a specific container where tag X is set to Y, and not A B C etc.
Currently using procedure ListBlobs does not seem to allow for any filtering using ABS Optional Parameters. We need to be able to set this where clause that is necessary for filtering according to documentation here using ABS Blob Client and ABS Optional Parameters codeunits:

https://learn.microsoft.com/en-us/rest/api/storageservices/find-blobs-by-tags?tabs=azure-ad#remarks
Thank you in advance for your time.

@EvaCegeka the system application is open for contribution.
If you really need this then you should create a pull request to set an addtional optional paramter for filtering.

Hi @pri-kise, another way to solve the problem is to create a new procedure in codeunit 9053 ABS Blob Client, called: Find Blobs by Tags in Container
https://learn.microsoft.com/en-us/rest/api/storageservices/find-blobs-by-tags-container?tabs=azure-ad
It will require more code, but it is a more direct solution.
Unless you think there is an issue with this, we will create a PR for this in the coming weeks.

@GTimmers fell free to create a PR for this addition.

 We decided to retrieve the blobs from a certain specified folder and now also include the tags in the parameters.
We can do some filtering on them and thats a good solution for us.

If any work remains in this issue, feel free to create further pull requests addressing them.


[FLINK-18621][sql-client] Simplify the methods of Executor interface in sql client

What is the purpose of the change
After TableEnvironment#executeSql is introduced, many methods of Executor interface in sql client can be replaced with TableEnvironment#executeSql. Those methods include:
listCatalogs, listDatabases, createTable, dropTable, listTables, listFunctions, useCatalog, useDatabase, getTableSchema (use DESCRIBE xx)
Brief change log

remove the methods which can be replaced with executeSql method in Executor interface, and update the related logic

Verifying this change
Update the existing tests, such as LocalExecutorITCase
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

cc @JingsongLi @fsk119

Can rebase master for triggering testing?

Can rebase master for triggering testing?

sure


fix conversion of anonymous volumes in compose-file

the convertVolumeToMount() function did not take
anonymous volumes into account when converting
volume specifications to bind-mounts.
this resulted in the conversion to try to
look up an empty "source" volume, which
lead to an error;
undefined volume:

this patch distinguishes "anonymous"
volumes from bind-mounts and named-volumes,
and skips further processing if no source
is defined (i.e. the volume is "anonymous").
fixes https://github.com/docker/docker/issues/29512
- What I did
Updated `convertVolumeToMount() to take anonymous volumes into account
- How I did it
Behind my computer, in pyjamas
- How to verify it
Deploy a docker-compose file that uses an anonymous volume, and verify that no error is produced, and the service is running afterwards;
$ cat > docker-compose.yml <<EOF
version: "3.0"
services:
  main:
    image: nginx:alpine
    volumes:
      - "/foo/bar"
EOF

$ docker stack deploy --compose-file=docker-compose.yml foo
Creating network foo_default
Creating service foo_main

$  docker service ps foo_main
ID            NAME        IMAGE         NODE          DESIRED STATE  CURRENT STATE           ERROR  PORTS
sr0u5f6vqjhr  foo_main.1  nginx:alpine  5945fab4e8ad  Running        Running 54 seconds ago

- Description for the changelog

Not needed; this functionality is new in 1.13
- A picture of a cute animal (not mandatory but encouraged)

Added a second commit with simple validation and a unit test for that

If we cherry-pick this into 1.13 I suggest we take #29003 along with it, or it's not going to merge.

Good point; if not, we should have a or against the 1.13 branch i guess

"cherry picked" through https://github.com/docker/docker/pull/29859


Update section.go

Add a check for false on private indicator:
private_indicator – This is a 1-bit flag that shall be set to 0.

helloooooo

Agree let's do it as warning

Are you going to give me an update with the warnings?
You already did all the work, update it and I'll merge it.
Jjust change log.Fatal to something like log.Print...... they need a log.Warn


Issue with latest version of .NET core

Hello,
Is it possible that the library don't work with the latest version of ASP.NET CORE /IDENTITY 2.2?
I can't reach my AD domain....
Thanks.

@jusbuc2k  Any updates ?

@joel-ld sorry, I missed this issue. I've not compiled this for .NET 2.1 or 2.2 or used it with Identity 2.2, but I wouldn't expect not being able to reach the domain would be related to that. Can you post sample code with an error message or screenshots showing the problem?

@joel-ld I just compiled and ran an example MVC app against netcoreapp 2.2 with identity 2.2, and it all seems to be working. See my branch and example MVC app here:
https://github.com/jusbuc2k/Justin.AspNetCore.LdapAuthentication/tree/feature/aspnetidentity22


[New Process Improvement Need]: Best Practices on Training Material Curation & Sharing

Checked for duplicates
Yes - I've already checked
Category
Information Sharing - documentation design, templates, communication, etc.
Describe the need
We have a need for:

Best practices for organizing and making accessible training material for evolving technological trends
Best practices for organizing and accessing training materials on containerizing and developing PGEs (+1'd by @LucaCinquini)
Process for linking to curated training guides, e.g. cloud training (+1'd by @LucaCinquini @nttoole)
"Get started" pages for different kinds of app development (archetypes) (+1'd by @mcduffie @nttoole)

Some earlier thoughts from @hookhua: "may need tools etc. but at the least, if there was a more consistent process to share in training to keep up with the technology, that is more of the point"
Some thoughts from @riverma: "focus on education platforms, curated training guides / resources, etc."

+1'd by @drewm-jpl (Unity)


Better intellisense for Lavalink objects

Updates Lavalink objects to reflect documentation, and adds documentation comments that show up on intellisense.

I will run a second pass through all the existing code comments, some of them are just plain wrong or worded badly on first glance.

The git history is pretty messed up on this one, when merging squashing is better for our collective sanity.

Superseded by #202


Update quick-start.md

Added a note to provide more instructions on using Azure OpenAI Service.
Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
All Promptflow Contribution checklist:

[x] The pull request does not introduce [breaking changes].
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.
[x] Create an issue and link to the pull request to get dedicated review from promptflow team. Learn more: suggested workflow.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

Made a small change to the QuickStart instructions


v79 bug "Dep (go.googlesource.com/tools) restored, but was unable to load it with error"

Note that the following does not occur with v78, and the cause appears to be https://github.com/tools/godep/commit/f15f6db5da33a4ac48be83e20b9dd838e14f117b.
Expected behavior
godep restore should succeed.
Actual behavior
godep restore fails.
Steps to reproduce behavior
Run the homebrew test block:
  test do
    ENV["GOPATH"] = testpath.realpath
    (testpath/"Godeps/Godeps.json").write <<-EOS.undent
      {
        "ImportPath": "github.com/tools/godep",
        "GoVersion": "go1.7",
        "Deps": [
          {
            "ImportPath": "go.googlesource.com/tools",
            "Rev": "3fe2afc9e626f32e91aff6eddb78b14743446865"
          }
        ]
      }
    EOS
    system bin/"godep", "restore"
    assert File.exist?("src/go.googlesource.com/tools/README")
  end

Error is
==> /usr/local/Cellar/godep/HEAD-f15f6db_1/bin/godep restore
godep: [WARNING]: godep should only be used inside a valid go package directory and
godep: [WARNING]: may not function correctly. You are probably outside of your $GOPATH.
godep: [WARNING]:	Current Directory: /private/tmp/godep-test-20170202-85545-1uu8n5t
godep: [WARNING]:	$GOPATH: /private/tmp/godep-test-20170202-85545-1uu8n5t
godep: Dep (go.googlesource.com/tools) restored, but was unable to load it with error:
	no buildable Go source files in /private/tmp/godep-test-20170202-85545-1uu8n5t/src/go.googlesource.com/tools
godep: Error checking some deps.
/usr/local/Homebrew/Library/Homebrew/debrew.rb:11:in `raise'

godep version output
bash-3.2$ godep version
godep v79 (darwin/amd64/go1.7.5

go version output
bash-3.2$ go version
go version go1.7.5 darwin/amd64

Contents of Godeps.json file
{
  "ImportPath": "github.com/tools/godep",
  "GoVersion": "go1.7",
  "Deps": [
    {
      "ImportPath": "go.googlesource.com/tools",
      "Rev": "3fe2afc9e626f32e91aff6eddb78b14743446865"
    }
  ]
}

That's not a valid Godeps.json file because the package specified is not a valid go package declaration because it (a) contains no valid *.go files and (b) the packages contained there are actually golang.org/x/tools. So if the file is changed to:
{
  "ImportPath": "github.com/tools/godep",
  "GoVersion": "go1.7",
  "Deps": [
    {
      "ImportPath": "golang.org/x/tools/cover",
      "Rev": "3fe2afc9e626f32e91aff6eddb78b14743446865"
    }
  ]
}

godep restore works properly because the correct namespace for the code is golang.org/x/tools and golang.org/x/tools/cover is an actual go package.
PS: This may have worked in previous versions of godep, but that was purely by accident.

Thanks for the help!


Performance issue fix

Found out that the performance issue was caused by looping through files in the bash scripts. Instead, the files from the provided path will be looped inside js.
Also, changed watch script to compile the changed file directly.

Thanks for the PR. Will take a look in the evening.
In the meanwhile, could you remove commit adding addon-viewport from this branch for now?
I would recommend making granular PRs as that would be easier to review.

Could we also use /src/styles as an output path instead of /src/components/styles ? This would mean changing currently imported css.js's path one folder back but would make it more acceptable for different types of packages which don't necessarily have components.

@chanar we could change the output path, if you think it makes sense e.g. to allow utility files like abu-theme.css.js to be used on their own, which someone would potentially want.

@chanar squash / amend top commits, don't add reverts etc noise.

@lkraav squashing is for merging two commits. Amending is for changing the commit message.

@web-padawan I think it makes sense for encouraging adoption and using your framework packages in parallel with someone's own app. But I would wait until if you decide to merge this pull request and then make a new PR for the path changes.

@chanar can you please comment on why eslint seems to be forcing formatting changes into the commit? This is making the PR very hard to review. @web-padawan is this configuration intentional, or?

https://github.com/web-padawan/aybolit/blob/889df602dd7b057abf43f6621aadf593ae6eb371/package.json#L87
Not sure why it didn't do changes for padawan in master, to begin with.

@lkraav it looks like I haven't run some scripts through prettier yet. The formatting changes are ok.

$ [git:scripts-edits?{1}] yarn watch
yarn run v1.15.2
$ node scripts/watcher.js
watcher started!
saw change to packages/white-label/scss/abw-button.scss
building styles

watcher build complete!

@chanar there seems to be no output of what was built, for clarity. Easy or not to add?

@chanar legacy for loop was excluding ^_ glob pattern to avoid building partials.
watcher build complete!
saw change to packages/bootstrap/scss/_variables.scss
building styles

watcher build complete!

Your JS globbing seems to have missed it.


Deprecating modules

We need to deprecate the primer-cards module as it's no longer needed. The styles have been replaced by Box and box-shadow.
I also never properly deprecated the primer-states module on npm which I feel bad about so we should do something for that too.
We should have a clear process for this so that we can deprecate modules safely. As a rough outline I believe would should give a warning running up to deprecating the module which I think we should do as part of a major release, but shouldn't be the sole reason we do a major release. I.e. right now our next major version will be 10, the reason for a next major release will hopefully be due to a major addition or update to primer modules, rather than cutting v 10 because we're removing something.
When we do publish the major release removing the module, we should have a process for what we do with the repo and the npm module. We also need to make this clear in the docs too, we already have a process for status, but I think we should have a notice that provides people helpful info such as what styles they can use in replacement etc. I'm happy to take that chunk on as I've already done some thinking on this.
Finally, we should have docs for how to deprecate a module once we've figured this out.
@shawnbot @jonrohan If you have ideas or feel confident about what this process should be, please share your thoughts here, and if you like, grab some to-do items below. I'm happy to work on this more than the docs too but wanted to hear from you both first.
To do:

[ ] process for notifying of upcoming deprecation
[ ] process for deprecating the module in the GitHub repo
[ ] process for deprecating on npm
[ ] process for updating docs - @broccolini

(FYI I found a great article on the process a lil while back, I'll try find it post here!)

This is one of those situations, though, where having at least major versions in our website-starter (see github/website-starter#9) would at least protect users from "accidentally" upgrading to a version that breaks them.

Not 100% sure what you mean so just going to clarify a few things... What the website starter is on, and what the other sites are on, are not connected once the site has been built. Marketing sites are usually "intentionally" updated by the web design team and often they stay on the same version they're built for a while—i.e. whatever version website starter points to at the time, if that's what they build it with (some sites are rails apps).
We still need all the things I listed above though to make the decision to update primer-css or one of the meta packages easier though. If they're pointing to a specific module separately, like primer-cards, and we're deprecating it, they can stay on the old version forever, even if we delete the repo, since npm packages are immutable.
I assume we'll use this https://docs.npmjs.com/cli/deprecate - and just realized you can specify the range which is handy, which is something we might want to do for the primer-css v7 & 8 since they don't have normalize so we can warn people 😬.

I think this was done in https://github.com/primer/css/pull/671.


Helm reponds with 401 error on private repo

Hello, I have an issue with helm authentication in private repo.
Step to reproduce:

have a private repo
Authentication in repo with command: helm repo add align <https-repo-url> --username=<user> --password=<password>
make helm repo update (doesn't emit any error)
make helm pull align/ccbackend-custom //let me spoil chart name for ya

Expected: A chart file downloaded in current directory.
Actual: Error: failed to fetch <specific repo-url>/ccbackend-custom/ccbackend-custom-1.2.2.tgz : 401 Unauthorized
More info:

I encode in base64 string <user>:<password> (same as before)
run command curl -H"Authorization: Basic <my-base64-string>" <same url where helm reports 401> -o ccbackend-custom
I get helm-chart.

Output of helm version:
3.6.3
My current platform:
Mac OS X Big Sur M1 (arm64)
Output of kubectl version:
unrelated
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
unrelated

I have an idea what might be going on. When a username and password are set for a repo they are only set for that repo location. If a chart archive is stored in another location (e.g., on a different domain) the credentials are not passed on by default. This is a security measure in order to not leak credentials. If you want to pass the credentials on to other domains you can use the --pass-credentials flag when you use helm repo add.

That was my first idea.
I made several changes to my helm config.
And in the end, there are facts:

My helms are in the same domain with repo.
I actually use --pass-credentials (this option also exists in helm pull)
I installed version 3.5.3 and it works with same config.

@stavenko I can't really tell from your last comment.. Were you able to resolve this issue using the --pass-credentials flag or not?
Your steps to reproduce the issue does not pass the --pass-credentials flag to helm repo add.
If you add that flag, what happens?

Ah... This might actually be a duplicate feature request for https://github.com/helm/helm/issues/9599, as you're asking that helm pull should use the same credentials provided with helm repo add. Is that correct?

Please also take a look at https://github.com/helm/helm/pull/9760 and see if that resolves this issue for you

@bacongobbler I am just telling, that I already tried what you proposing and this is not helping.
I have added this flag in helm config (Library/Preferences/helm/repositories.yaml) - result negative.
I have added this flag to helm pull command - same result.
I have added credentials to helm pull command - negative.
My chart's url is also has the same host with repo - so it doesn't even look like an issue here.
This issue reproduces only in 3.6.x helm.
Helm 3.5 works as expected.

Well, I checked this carefully and have updates:
When I set pass_credentials_all: true in ~/Library/Preferences/helm/repositories.yaml It actually works. I have pulled chart from repository.
Success.
But!
My url in repo is https://repo.aligntech.com/helm-bld/
and url of downloaded chart is  https://repo.aligntech.com:443/api/helm/helm-bld/ccbackend-custom/ccbackend-custom-1.2.2.tgz
How is that supposed to be different domains?

I think I see what's going on here. The URL host portion is different. In one case it's repo.aligntech.com and in another case it's repo.aligntech.com:443. This is something, if I remember right, that artifactory does. Others may do it, too. #9886 was created to address this issue.


Introducing AbstractPoolStore and allow pool users to plug in their own PoolStore implementations

The previous store has been renamed to SqlitePoolStore and is the default one.  As suggested in https://github.com/Chia-Network/pool-reference/issues/91 users can now write their own store implementations and use them with the pool. For this reason, an abstract class has been introduced.
To use a custom store, a users needs to do the following:
from pool.pool_server import start_pool_server
try:
    asyncio.run(start_pool_server(MyOwnPoolStoreClass()))
except KeyboardInterrupt:
    asyncio.run(stop())

Therefore this repository can now function as a framework, not just an end application.
For this reason, I changed all the imports to be relative to pool package, so they can function properly in any environment. I've also introduced a very simple __main__.py file, that allows to run the pool with python -m pool.

LGTM! @xdustinface do you have any thoughts?

rebased

I had to make another force push - due to mandatory signatures, that I have overlooked before.


setSrc() not working on Internet Explorer

The Script works fine in all Browsers except on Internet Explorer (but it's working in Internet Explorer
on Windows 10 and also in Edge)
Internet Explorer says setSrc is not supported by myaudio. Any ideas how to fix this?
<script>
  window.onload = function() {
  myaudio = document.getElementById('audio-player');
  button = $('.mejs-playpause-button');
  i = Math.floor( ( Math.random() * 1000000 ) + 1);
  source = 'http://localhost:8000/;';
  playpause = document.querySelector('.mejs-playpause-button');
  playpause.addEventListener('click', function() {
	if (myaudio.paused) {
		myaudio.setSrc('');
		myaudio.load();
		myaudio.pause();
		button.removeClass('mejs-pause').addClass('mejs-play');
	} else {
		i++;
		myaudio.setSrc(source + '?nocache=' + i);
		myaudio.load();
		myaudio.play();
		button.removeClass('mejs-play').addClass('mejs-pause');
	}
  });
 };
</script>

@NitroxydeX Is this still an issue?

@ron666 can't check this because all my systems are on Windows 10.

@NitroxydeX Thanks for your response. We'll close this issue and we can always reopen it if it appears in the future. Thanks!


Nanosecond timestamps

On the client side, generate timestamps with nanosecond precision.
On the protocol side, Metric.ts is now in nanoseconds instead of
seconds.
On the server side, send timestamps to influxdb in nanoseconds. For
graphite I'm keeping things as seconds.

+2 ...


When importing a PDF, arrow heads might randomly be wrong.

I'm not sure what caused this. The output PDF by MakieTeX+CairoMakie sometimes is not an exact copy of a input PDF.

Do you have a visual example handy?  Not sure what you mean by "wrong" here - it could be a rasterization error, or something in Poppler.
Also, is this also the case in GLMakie?

Please see this output, viewed in Adobe Reader.
The source PDF from GraphViz is correct.


refactor: PyTorchInference initialization

In this version, I have modified the code by combining two loops into a single loop within the same method. This enhancement has resulted in improved efficiency.

Source : It was revealed to me in a dream
Jokes aside, list comprehensions are faster in python, unless the iterator costs a lot, I doubt it will be faster

Hello @ExtReMLapin,
I appreciate your question. While list comprehensions are faster in python, I consolidated two loops into one to improve time complexity from O(2) to O(1). This should notably reduce overhead and improve performance, especially with larger inputs.
Thanks :)

@arjun-234 your code lacks another thing that may make it slower than just using list comprehension.
Basically this:
arr = [0] * length
for i in range(length):
    arr[i] = input[i]

Is faster than this:
arr = []
for i in range(length):
    arr.append(input[i])

But still, list comprehension is the fastest of them all. And not only that, it also looks cleaner 👌.

Greetings @threeal,
Yeah, you're correct that list comprehensions are always faster, but especially with larger inputs, having fewer iterations is always good, which improves performance in terms of time complexity.
Thanks :)

Hello @ExtReMLapin, I appreciate your question. While list comprehensions are faster in python, I consolidated two loops into one to improve time complexity from O(2) to O(1). This should notably reduce overhead and improve performance, especially with larger inputs. Thanks :)

O(2) == O(1) :thinking:


TypeError: 'NoneType' object is not subscriptable

When trying to run report after executing collect.
Does it make sense that I'm getting this error if I'm getting accesses denied on a small subset of resources.
image

I'm closing this as I believe the situation was explained in #420.  You appear not to have privileges, either due to your principal's own IAM, an SCP applied to them, or the policy on the bucket itself.  The paths forward I see with CloudMapper for this issue for you would be:

Get the privileges needed
Add code to bypass this issue or remove references to this S3 bucket from your s3-list-buckets.json file

@0xdabbad00 same issue here. We unfortunately have about 30 s3 buckets with source IP restriction. As I can't run this report from that IP, we get access denied, thus reporting fails with "TypeError: 'NoneType' object is not subscriptable".
Is your recommendation to comment or delete each of these buckets from all of the s3-list-buckets.json?
Seems like a common use case btw.

@ulte I don't have a good answer.  My default answer has thus far been that any problems encountered running CloudMapper without the necessary privileges is outside of the scope of the project.
One option, as you mentioned is to manually modify your data to workaround these limitations.  No one wants to do that unless they are really in a pinch.
Another option is to give CloudMapper the access it expects, which is not really possible or desirable for your situation, but can be in some others.
Another option is to add code to ignore these failures.  The concerns there are:

It creates a cat and mouse game of continually having to add workaround code for different environments that are blocking different calls.
The potential for bugs in CloudMapper increases dramatically in these situations as there are lots of assumptions in the code that it has access.
It potentially hides problems. For example, if someone has an SCP that denies them the ability to look at an S3 bucket in an account, but that S3 bucket is publicly readable, then CloudMapper will not be able to identify that the bucket is otherwise public.

I think we do need to add workaround code to ignore these problems, and somehow bubble up to the user that they didn't have expected access.  I just haven't faced the situation yet to take the time to resolve it.

I'm encountering this issue.
Is it possible to just have cloudmapper just ignore the buckets if the account doesnt have access?

Just keep a file of ignored buckets you have no permission for and throw a warning, don't totally fail to run.

yes, that ^
it seems a bit overkill to completely prevent the entire tool from running on the rest of an environment because there is literally one s3 bucket that the account doesn't have access to.

what he said


unable to load app in simulator

simulator related
I'm submitting a ...  (check one with "x")

[ ] bug report
[ x] feature request

Issue description
Current behavior:

Expected behavior:

Steps to reproduce:

Related code:

/bin/sh: react-native-scripts: command not found
error Command failed with exit code 127.

Other information:
**OS, device, application version **
Mac mini(A1347),Xcode

Hi,issue resolved, having some problem due to yarn ,switched to npm.
it's working fine now!


Fixes: #15 add a logo

Fixes issue #15
Changes: Added the new logo to readme.md file
Screenshots for the change:

Looks great ! Thank you @isuruAb :tada:


structure: yoda conditions

enforces Use Yoda conditions when checking a variable against an expression to avoid an accidental assignment inside the condition statement (this applies to ==, !=, ===, and !==).
recorded in #66

Thanks for your contribution @wickedOne!


Accuracy

What is your question?
How do I get the accuracy 0 to 100 with only 2 decimal places (100.00)?

don't post duplicates


What is soft margin? If we use hard margin the distance difference should be less than hard margin for the same objects?

If we use hard margin the distance difference should be less than hard margin for the same objects? Is it correct?
I see that after retraining the code with the provided configs and hard-margin =0.5 but when infer the model with 2 images of the same object, the distance is still very large compared to hard-magin value
Thanks

As I already mentioned in #51, the distance between images of the same object are not directly constrained at all by the triplet loss. The margin is just added on top of the distance between positives samples and that is used to determine the minimal distance to the negative samples.
I'm quite sure that after training this margin constraint will not be violated by most of the training set, but as also mentioned before, the embeddings are more noisy on the test set and will thus likely result in both negatives and positives being quite close to some anchors.
You could add another loss term that tries to push together positives without any limit, essentially trying to collapse them to a single point. This might potentially work better, but we never experimented with this much. It reminds me of the contrastive loss by Hadsell et al., in which case the margin likely has a more important and definite meaning, and of this paper, which you should take with a big spoon of salt though, since they do not achieve anywhere near state-of-the-art results.

Thanks for your info, but it seems for me that the trained model produces large distance (> 7*margin), but not smaller than margin even on the training set


Revert "chore(storage): bump object_store_read_attempt_timeout_ms from 8s to 16s"

Reverts risingwavelabs/risingwave#16719

Should merge https://github.com/risingwavelabs/risingwave/pull/16782 instead.


Rework shared examples

Fixes on top of #534:

workaround to prevent 1 spec from crashing JRuby (JRuby lists internal Fiber thread in Ruby's Thread.list)
Ruby < 2.x fixes for getting Task class from ENV
reorganize shared examples, so that at least actor specs aren't in shared examples (makes dev harder)

Celluloid::IO specs will need to also updated to work with these changes (comming up...).

@e2 thanks for pushing to this branch. I'll keep snowballing #535 with all your contributions, then we can revisit celluloid/celluloid-io#134 once that lands.

At some point I'd like to get most of the RuboCop violations --auto-corrected and have a sane RuboCop policy too.

@tarcieri - yes, that's probably best done after most PRs have been pulled in and there aren't many feature branches (otherwise it can result in a ton of unnecessary merges/conflicts).

I like how you can immediately see that the hanging tests are due to failing TaskThread tests (JRuby): https://travis-ci.org/celluloid/celluloid/builds/56668865

@e2 that is a substantial improvement.


Deploy container with rclone in test cluster

From https://github.com/QCDIS/NaaVRE/tree/datamesh build and deploy the container with rclone

No luck. When I launch a terminal it can't find rclone.
Utrecht university has it: https://utrechtuniversity.github.io/research-engineering/manuals/install-rclone/
I get the following errors when I try that (something with privileges?):
jovyan@jupyter-justinmingo:~$ rclone config
bash: rclone: command not found
jovyan@jupyter-justinmingo:~$ curl https://rclone.org/install.sh | sudo bash
sudo: effective uid is not 0, is /usr/bin/sudo on a file system with the 'nosuid' option set or an NFS file system without root privileges?
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100  4669  100  4669    0     0  27427      0 --:--:-- --:--:-- --:--:-- 27627
curl: (23) Failure writing output to destination
jovyan@jupyter-justinmingo:~$ rclone version
bash: rclone: command not found

Will try to install with apt and rebuild the image

We can also use : conda install -c conda-forge rclone

Rclone works, S3 I think also:
Current remotes:
Name                 Type
====                 ====
test                 s3

[ ] Check if we can move files from and to S3.

rclone from apt installs 1.50.2
rclone from conda installs 1.60.1


[SPARK-34558][SQL] warehouse path should be qualified ahead of populating and use

What changes were proposed in this pull request?

Currently, the warehouse path gets fully qualified in the caller side for creating a database, table, partition, etc. An unqualified path is populated into Spark and Hadoop confs, which leads to inconsistent API behaviors.  We should make it qualified ahead.
When the value is a relative path spark.sql.warehouse.dir=lakehouse, some behaviors become inconsistent, for example.
If the default database is absent at runtime, the app fails with
Caused by: java.lang.IllegalArgumentException: java.net.URISyntaxException: Relative path in absolute URI: file:./lakehouse
	at org.apache.hadoop.fs.Path.initialize(Path.java:263)
	at org.apache.hadoop.fs.Path.<init>(Path.java:254)
	at org.apache.hadoop.hive.metastore.Warehouse.getDnsPath(Warehouse.java:133)
	at org.apache.hadoop.hive.metastore.Warehouse.getDnsPath(Warehouse.java:137)
	at org.apache.hadoop.hive.metastore.Warehouse.getWhRoot(Warehouse.java:150)
	at org.apache.hadoop.hive.metastore.Warehouse.getDefaultDatabasePath(Warehouse.java:163)
	at org.apache.hadoop.hive.metastore.HiveMetaStore$HMSHandler.createDefaultDB_core(HiveMetaStore.java:636)
	at org.apache.hadoop.hive.metastore.HiveMetaStore$HMSHandler.createDefaultDB(HiveMetaStore.java:655)
	at org.apache.hadoop.hive.metastore.HiveMetaStore$HMSHandler.init(HiveMetaStore.java:431)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.apache.hadoop.hive.metastore.RetryingHMSHandler.invokeInternal(RetryingHMSHandler.java:148)
	at org.apache.hadoop.hive.metastore.RetryingHMSHandler.invoke(RetryingHMSHandler.java:107)
	at org.apache.hadoop.hive.metastore.RetryingHMSHandler.<init>(RetryingHMSHandler.java:79)
	... 73 more

If the default database is present at runtime, the app can work with it, and if we create a database, it gets fully qualified, for example
spark-sql> create database test;
Time taken: 0.052 seconds
spark-sql> desc database test;
Database Name	test
Comment
Location	file:/Users/kentyao/Downloads/spark/spark-3.2.0-SNAPSHOT-bin-20210226/lakehouse/test.db
Owner	kentyao
Time taken: 0.023 seconds, Fetched 4 row(s)

Another thing is that the log becomes nubilous, for example.
21/02/27 13:54:17 INFO SharedState: Setting hive.metastore.warehouse.dir ('null') to the value of spark.sql.warehouse.dir ('datalake').
21/02/27 13:54:17 INFO SharedState: Warehouse path is 'lakehouse'.

Why are the changes needed?

fix bug and ambiguity
Does this PR introduce any user-facing change?

yes, the path now resolved with proper order - warehouse->database->table->partition
How was this patch tested?

w/ ut added

Test build #135533 has started for PR 31671 at commit bd26d8f.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40114/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40114/

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

Test build #135533 has finished for PR 31671 at commit bd26d8f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Test build #135543 has started for PR 31671 at commit 8650c11.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40124/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40124/

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

Test build #135543 has finished for PR 31671 at commit 8650c11.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Test build #135562 has started for PR 31671 at commit 897a55c.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40143/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40143/

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

Test build #135562 has finished for PR 31671 at commit 897a55c.

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

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

cc @cloud-fan @maropu @HyukjinKwon thanks for reviewing

Test build #135571 has started for PR 31671 at commit d1f32ce.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40152/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40152/

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

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

What's concrete steps to reproduce this error by users?
Caused by: java.lang.IllegalArgumentException: java.net.URISyntaxException: Relative path in absolute URI: file:./lakehouse
	at org.apache.hadoop.fs.Path.initialize(Path.java:263)
	at org.apache.hadoop.fs.Path.<init>(Path.java:254)

What's concrete steps to reproduce this error by users?
Caused by: java.lang.IllegalArgumentException: java.net.URISyntaxException: Relative path in absolute URI: file:./lakehouse
	at org.apache.hadoop.fs.Path.initialize(Path.java:263)
	at org.apache.hadoop.fs.Path.<init>(Path.java:254)

with a fresh built or downloaded spark dist package, which means there is no default DB yet, then run a simple case like,
bin/spark-submit --master local -c spark.sql.warehouse.dir=lakehouse --class org.apache.yao.spark.sql.ShowHiveTables ~/sugar/target/sugar-1.0-SNAPSHOT.jar

and the source code can be found https://github.com/yaooqinn/sugar/blob/master/src/main/scala/org/apache/yao/spark/sql/ShowHiveTables.scala, which is a quite simple one w/ show databases

Test build #135588 has started for PR 31671 at commit e949845.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40169/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40169/

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

Test build #135588 has finished for PR 31671 at commit e949845.

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

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

Test build #135621 has started for PR 31671 at commit c4b4cda.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40200/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40200/

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

Test build #135621 has finished for PR 31671 at commit c4b4cda.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

retest this please

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

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

retest this please

Test build #135652 has started for PR 31671 at commit c4b4cda.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40232/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40232/

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

Test build #135652 has finished for PR 31671 at commit c4b4cda.

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

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

thanks, merging to master!


hybrid search with two vector fields

Hi
I would like to add additional vector fields to my documents for hybrid search.
Is updating the index in azure AI search service enough, or do I need to change something in the code as well?
Thanks in advance.
Regards
agnosus

Yes, you'd need to change some code. Currently the approach files like chatreadretrieveread.py call approach.py:compute_text_embedding to return a VectorizedQuery for the embedding field, and they pass that to the approach.py:search method. You would want to pass in a VectorizedQuery for each of the fields, though I think you could reuse the same vector for each, assuming you're searching with user query vector.

Thanks a lot Pamela. 😊 This did the trick:
VectorizedQuery(vector=query_vector, k_nearest_neighbors=50, fields="embedding1, embedding2")
I have a related question on search algorithm:
I am trying to use exhaustive KNN search instead of the default HNSW.
In the searchmanager.py, I modifiyed the code to :
vector_search=VectorSearch(
algorithms=[
ExhaustiveKnnAlgorithmConfiguration(
name="vector-config-1711876367278",
parameters=ExhaustiveKnnParameters(metric="cosine"),
)
],
profiles=[
VectorSearchProfile(
name="fullknn",
algorithm_configuration_name="vector-config-1711876367278",
vectorizer=(
f"{self.search_info.index_name}-vectorizer" if self.use_int_vectorization else None
),
),
],
vectorizers=vectorizers,
)
I also manually added  'fullknn' to my index but CAN NOT delete the existing HNSW profile. So I ended up with two vector search profiles. Now I am not sure which profile is being used for the vector search??
Appreciate your help.
Thanks.

If you're trying to use exhaustive KNN, I don't think you need to add a new profile - you can just send exhaustive=True in VectorizedQuery constructor.
As to what profile is being used, if you don't specify exhaustive, it should be whatever profile is associated with the field (embedding1, embedding2). More details in https://learn.microsoft.com/en-us/azure/search/vector-search-how-to-create-index?tabs=config-2023-11-01%2Crest-2023-11-01%2Cpush%2Cportal-check-index


Rollup calculates only when Full Calculates, no when I update a record

I have a rollup that only works when I run a full calculate job. It doesn't get update when I update a child record.
It sums the amounts on won opportunities closed  during the Last fiscal year for non anonymous accounts and  with  empty lookup field on the opportunity. I haven't use the NPSP Customizable Rollups as the rollup as additional filters I cannot configure.
Calculation mode: Real Time
Calculation Sharing Mode: System
Relationship Criteria:
Customfield__c = null  and CloseDate = LAST_FISCAL_YEAR AND (not(Account.Name LIKE 'Anonymous%')).
I run the full calculate and the outcome is correct.
I update the Opp amount or the close date and the Aggregate Result Field value returns always 0.

@isita can you confirm the Trigger is deployed for the Opportunity?
That is the only reason i can think of why realtime configured job wouldn't work correctly the way you described.

I am seeing the same issue after upgrading to the latest package version (from a very old version!), it is as if the Relationship Criteria Fields are being ignored.  FYI rollup is aggregating a formula field, with the dependent fields added to 'Relationship Criteria Fields'.  Rollups are being triggered fine where they are directly aggregated (i.e. a directly aggregated field triggers a rollup).
I'll work my way back through versions to find one that works.

@isita or @grigorian do either of you still have examples of this?
Would you be able to share your configuration records and version?

@aheber I think I've now worked around all the issues I had.   I'm not sure if this is still an issue or not.

@grigorian thanks!

I'm going to mark this as closed. If there are additional problems or questions please direct them to the Trailblazer Community Group for better support.


Identify CSS to reload from the href

Identify the CSS to reload from the href, which is already sent to the browser in the WebSocket message, instead of using the CSS id.
This opens the possibility to have more than one CSS file that is hot-reloaded by cargo-leptos. I plan on using that for the Tailwind implementation.
This is backwards compatible with previous cargo-leptos versions.
@benwis Ok for you? I have to admit that I didn't test it with an Axum setup as I don't have one easily available. Would appreciate it if you could test it!
@gbj I would be really good to include this in release 0.2.0.

@akesson Is the actix one supposed to be matching "fail"?

🤦  Thanks @benwis for spotting that. It was for testing the error message.

Other than that, It looks fine to me. I see no reason Axum would work any differently than Actix here. I tried it and it seems sane

Yep looks good to me. I'm doing a little refactoring into a leptos_integration_utils package as part of the in-order streaming/async stuff, by the way, so this code will live in one place in the future. Will merge right now.


Don't allow creating infinite loops

Right now it's possible to wire graphs so that infinite loops are created

this is not a backend issue, removing the label

please provide example steps to reproduce


Skip waypoints if occupied

I plan on using opennav_coverage in the future but haven't yet so I can't answer this question myself: Is there any functionality to skip waypoints if they are occupied (for too long). If not then this is an issue to implement that (opt-in).

No, but there are options to specify rows to use - so you could replan excluding it and any completed rows -- so in spirit but not as a "here's a parameter to do it"

Excluding a complete row is a bit excessive and impacts the coverage quite a lot. Alright, already good to know that it's not possible. Will come back with a design proposal when I come back to this

This is mostly a F2C thing: its really more meant for static fields that you know about the obstacles in to begin with. However, adding another module that uses some F2C and some custom (like the Row Coverage server) that does more active choices would be welcome here :-)


blade.php is not supported

In Laravel projects blade.php files is not supported. i.e there is no autocomplete list of CSS classes.
Version: 1.9.0, VScode 1.15.1, Ubuntu Linux 16.04, Laravel 5.4 Project.

Just tried it on a PHP file and it worked. How does this blade.php file differs from regular PHP files? Sorry, but I'm not versed in PHP.

There is no single file named blade.php in Laravel. The templating engine used for Laravel views is Blade so all Blade view files use the .blade.php file extension.
@saidbakr Are you using SCSS to generate CSS?

I have the same problem. The extension can pick up all of my CSS classes in any .php or .html file but it won't work inside of blade templates. I'm also using SCSS btw.

Same for me.


minDate params not working

I used this minDate={moment(new Date())} but it's not working

is not working mate, u should check

Not all these are working, maxDate, minDate, initialDate

tried: maxDate="2018/08/28", maxDate="2018-08-28", maxDate="08-28-2018"

<DateInput
dateFormat="YYYY/MM/DD"
maxDate="2019/07/07"
Not making any effect at all ...

@chenmargalit I tried this and seems like it works.
<DateInput
          placeholder='Date'
          popupPosition='bottom right'
          className='example-calendar-input'
          name='date'
          closable
          clearIcon={(<Icon name='remove' color='red' />)}
          clearable={clearable}
          animation='scale'
          duration={200}
          hideMobileKeyboard
          dateFormat='YYYY/MM/DD'
          maxDate='2019/07/07'
          value={this.state.date}
          iconPosition='left'
          preserveViewMode={false}
          autoComplete='off'
          onChange={this.handleChange}
        />

@arfedulov  thanks. What versions are u using ? of the react/semantic ui react/semantic ui calender/ react? cause for some reason the same code doesnt work on my machine

@arfedulov most of it is working, from what I checked and not working are the min/max date and the disable/enable

I don't think it's somehow related to react or semantic-ui-react. I tested this on<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Try using default date format. Does minDate/maxDate work after that?

@arfedulov  what do u mean with default ?
<DateInput
maxDate="2019/07/07"
pickerStyle={{
backgroundColor: "#ff725d",
color: "white"
}}
name="date"
placeholder="Date"
value={this.state.date}
onChange={this.handleChange}
inline
/>
this doesnt work

Default format is DD-MM-YYYY. If you omit dateFormat prop the default is used and you have to provide maxDate or minDate in that format too.
Your last example should not work. It should work if you add this:
<DateInput
    maxDate="2019/07/07"
+   dateFormat="YYYY/MM/DD"
    pickerStyle={{
    backgroundColor: "#ff725d",
    color: "white"
    }}
    name="date"
    placeholder="Date"
    value={this.state.date}
    onChange={this.handleChange}
    inline
/>

or change this:
<DateInput
-   maxDate="2019/07/07"
+   maxDate="07-07-2019"
    pickerStyle={{
    backgroundColor: "#ff725d",
    color: "white"
    }}
    name="date"
    placeholder="Date"
    value={this.state.date}
    onChange={this.handleChange}
    inline
/>

You also can provide Date or moment to maxDate and minDate. maxDate={ moment('2019-07-07') } or maxDate={ new Date('2019-07-07') }.

Yea I tried those before as well .. I dont know why but nothing has any kind of effect .. no matter what I tried
      <DateInput
                        dateFormat="YYYY-MM-DD"
                        minDate={moment("2019-07-03")}
                        pickerStyle={{
                            backgroundColor: "#ff725d",
                            color: "white"
                        }}
                        name="date"
                        placeholder="Date"
                        value={this.state.date}
                        onChange={this.handleChange}
                        inline
                    />

I didnt check yet, but maybe the effect is invisible because of overwriten styles? Try without pickerStyles.

not working as well ...

Can you create interactive example with this bug on codepen or any site like that?

a bit packed with work (as I have to write the calendar my self now :)) I'll try to do it when things will cool


Add a module in project/tests to exemplify how to structure the imports

Completed in version 1.0.3


Gitment and Gitalk | TaoAlpha's Blog

https://taoalpha.github.io/blog/2020/06/30/tech-gitment-and-gittalk/

测试一下 :)


Upgrading from 5.7.5 and 8.5 fails

Affected Version of Concrete CMS
9.x
Description
I enabled these two checks and it seems that Concrete can't upgrade versions 5.7.5 and 8.5.
See these sample test run:

Update from <IP_ADDRESS>: https://github.com/mlocati/concrete5/runs/5541188672
Update from 8.5.2: https://github.com/mlocati/concrete5/runs/5541188740

How to reproduce
Install an old version of Concrete, and try to upgrade it (at least, that's what the test script does :wink:)
Possible Solution
No response
Additional Context
No response

That's correct, we do ask that people update to the latest version of 8.5.x before they update to version 9.
We used to have this clearly marked on our download page but I do see that it has gone away unfortunately.