Is the omission of RFC8813 intentional?

Looking at UnmarshalJSON and FromString in file lint/source.go, there is no processing for RFC8813.
So when verifying using zlint, if we add RFC8813 to the options, the following error occurs.

5, unable to configure included/exclude lints: unknown lint source in list: "RFC8813".

We solved the problem by modifying the code as follows. Could you please apply it to the distribution version?
func (s *LintSource) UnmarshalJSON(data []byte) error {
        ...
	switch LintSource(throwAway) {
	case RFC8813, RFC5280, RFC5480, RFC5891, RFC8813, CABFBaselineRequirements, CABFEVGuidelines, CABFSMIMEBaselineRequirements, MozillaRootStorePolicy, AppleRootStorePolicy, Community, EtsiEsi:
		*s = LintSource(throwAway)
		return nil
        ...
	}
}

func (s *LintSource) FromString(src string) {
        ...
	switch LintSource(src) {
	case RFC5280:
		*s = RFC5280
	case RFC5480:
		*s = RFC5480
	case RFC5891:
		*s = RFC5891
	case RFC8813:
		*s = RFC8813
	...
	}
}

It's a handling issue with the options, but the function is working fine.
Thank you.

Fixed in #879


about category and interval

In nets/utils.py function two_point,
The interval is 10 but category is 12,
If interval is 10，bins should be [0,10,20,30,40,50,60,70,80,90,100] length is 11, according to my understanding,
But   embed's length is 12,
I don't know why.

you'd better print the categories,which may help you to understand it; only 10 bins in paper: why we got 12 category: we can split it as "[0, 10, ... 110 ]" by two points!

Thanks for your reply, and I understand that.


Few Issues related to login

After installation could not login, tried different with different browsers but not able to login.
The I tried node server.js sync force, after this I try to login but I am getting error message saying that "Company is blocked due to Payment issues"...
After that I tried to duplicate the admin user and change the name and group policy to superadmin, now i am able to login but i could not do other than monitor the users, usergroups and company details.

Use Opera browser

Any one could set tba show property in backoffice Im never get done this área. In Last update TV show póster can view in User side but never season or episode Please any advice thanks in asvance

Will look into it.. Question? Did you have issue linking your server and connecting apk to panel?

Tried to login with admin/admin and getting "Company is blocked due to Payment issues"...
Please someone advise.

Will look into it.. Question? Did you have issue linking your server and connecting apk to panel?

No i did not try that, will check and revert back.

Hello @fabgsouza76
To resolve the issue you are having, please update the value on table settings billing_date
After updating the value, restart the service and you should be able to login

Hello @alonzo313
What issue are you having on connecting the application to your local server?

There is no field billing_date on table settings.
Please advise again.

Good morning Iris, I have input url and attempted to link server several
times to STB apk. Each time it was unresponsive
On Tue, Sep 10, 2019 at 4:32 AM irisstaro<EMAIL_ADDRESS>wrote:

Hello @alonzo313 https://github.com/alonzo313
What issue are you having on connecting the application to your local
server?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MAGOWARE/backoffice-administration/issues/58?email_source=notifications&email_token=AMTR273PGGRF6VCGDB3FPE3QI5LRXA5CNFSM4IUHJVCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6KJTCA#issuecomment-529832328,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMTR2723VQYIDQF3IYRJIK3QI5LRXANCNFSM4IUHJVCA
.

Try to log from stb app and mobile and say User not found or 0 not found

no value on table settings billing_date

What Stb apk is functional. I am having difficulty linking panel to working
apk
On Fri, Sep 6, 2019, 2:08 PM yamcell<EMAIL_ADDRESS>wrote:

Any one could set tba show property in backoffice Im never get done this
área. In Last update TV show póster can view in User side but never season
or episode Please any advice thanks in asvance
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MAGOWARE/backoffice-administration/issues/58?email_source=notifications&email_token=AMTR27YKWRFC3V24O7SWEEDQIKMDPA5CNFSM4IUHJVCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6DUAGA#issuecomment-528957464,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMTR27Y4NNESYFQUKBCMIVDQIKMDPANCNFSM4IUHJVCA
.

16 days and billing date not found

Hello!
You can receive the new update for the billing date where we have set a default value
This will resolve the issue you are having with login and be able to login


DP-1136 Fixed uncommenting a comment with a nested comment

Changes in TextEditingTrait caused by possible processed token removal (resulting in cell detach). Earlier current token also had a chance to be removed, but this was happening in other paths which took this into a consideration and no error was encountered.
Changes in TextEditingTrait are covered by new and some old tests.

I noticed that the pasteBeforeComment test is passing without the changes in the commit. Is it expected?

@timzam The test proves changes in TextEditingTrait.onPaste() needed for new version on uncomment processing in TokenCellTraits. Applying only latter will fail the test.

I see.
Looks good to me.


Don't require Action to return a Result

Some actions can't error, so it's annoying to be forced to do so. I think you could just remove that and have the output associated type be a result if needed.

The containers needs to check if the action was successful before proceeding, e.g. in Record::apply. Not sure how this would be handled without returning Result. Maybe when specialization is stable?
I think most actions will fail when undo and redo is involved, as long as it is not short example snippets.


[$1000] Workspace - User is allowed to create 2 workspaces in the same name

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

Action Performed:

Launch app
Tap profile icon
Tap Workspaces
Tap new workspace
Tap on workspace name
Change name to "v"
Tap save
Follow step2-step7 and create another workspace in the same name "v"

Expected Result:
User should not be allowed to create 2 workspaces in the same name
Actual Result:
User is allowed to create 2 workspaces in the same name "v"
Workaround:
Unknown
Platforms:

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

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/c0e90030-d9a0-44be-b6d4-466b21be0843
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~010a918b4f91b57aa4
Upwork Job ID:<PHONE_NUMBER>973748736
Last Price Increase: 2023-08-28

Proposal
Please re-state the problem that we are trying to solve in this issue.
The user is allowed to create 2 workspaces with the same name.
What is the root cause of that problem?
Currently, we do not validate if the workspace name or policy name in all policies list already exists or not. We are not showing an error if the workspace already exists.
What changes do you think we should make in order to solve the problem?

Write a function in Policy.js to check if workspace name or policy name already exists or not.

/**
 * Check if the workspace already exists or not. Return true if workspace name or policy.name already exists.
 * @param {String} worksapceName
 * @returns {Boolean}
 */
function isExistingWorkspace(worksapceName) {
    return _.some(allPolicies, (policy) => policy.name === worksapceName)
}

Add an if condition in WorkspaceSettingsPage.js to validate new workspace name and show an appropriate error if workspace name is not valid.

else if (Policy.isExistingWorkspace(name)) {
            errors.name = 'workspace.editor.nameAlreadyExistsError';
}

Add nameAlreadyExistsError in languages in workspace.editor object to define an error message.
en.js

nameAlreadyExistsError: 'Workspace with that name already exists.',

es.js
nameAlreadyExistsError: 'El espacio de trabajo con ese nombre ya existe.',

if user is at workspace settings page and decided to not to change the name, he can simply go back without changing the name.

Here is screencast for this solution.
Screencast from 2023-08-15 11-58-57.webm
What alternative solutions did you explore? (Optional)
There is also another situation. For example, the user is at the workspace setting page and
try to save name without changing it, we ask the user to 'Please change name or Go back to workspaces to keep it unchanged'. it is more user friendly.
For this we can add another logic in isExistingWorkspace function to check if user is trying to save name without changing it. I have already code for this in my mind.
I can submit PR with all above code within just 1 hour.

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~016bb3597ed6938fbc

@parasharrajat Can you please review proposals on this?

This is intentional
https://github.com/Expensify/App/issues/23448#issuecomment-1650133087

@shubham1206agra Two workspace with same name does not make any sense. If users are allowed to make workspace with same name, then what is the purpose of workspace. In any application, workspace concept exits, so user can categorise things according to their needs. Any user identify its workspace by name. If user can make multiple workspace with same name, how will he know, what thing he put in which workspace. Tell me how workspaces with same name make sense????

I'm still working on finding the conversation in which we decided the behavior was intentional. Once I locate it, I'll report back!

This was raised earlier, and was closed saying intentional
https://github.com/Expensify/App/issues/23448

@parasharrajat thanks for the bump! No, I haven't been able to find the conversation yet. Let me bump some folks and see what I can find!

I bumped the linked issue above to find the expected behavior. The BZ team member is currently OOO, but it looks like they'll be back online tomorrow, so I should have a response soon!

I'm still chasing down the expected behavior.

As of now, it's my understanding that this behavior is expected.

From Ben:

From memory, it was a two stage slack process - in https://expensify.slack.com/archives/C049HHMV9SM/p1682075335194889, the same issue had been raised, and that itself had been back-linked to https://expensify.slack.com/archives/C049HHMV9SM/p1682086817167539?thread_ts=1682075335.194889&cid=C049HHMV9SM, where Georgia/Shawn confirmed things were as expected

Thanks, @parasharrajat! I'm going to close the issue since the behavior is expected.


Setting "viewportSize" is not work.

Are you using Hugo extended version? You will find them towards the end on the release page https://github.com/gohugoio/hugo/releases/tag/v0.72.0.
I can see that it is working just fine :)

Yes, it is working well


instagram scraping sometimes missing replies

instagram_live_test.py started noticing this yesterday: https://circleci.com/gh/snarfed/granary/1498
seems flaky. when i run it locally, it still passes sometimes, but not always. haven't dug in yet. sigh.

looks like an A/B test where sometimes comments are in edge_media_to_parent_comment instead of  edge_media_to_comment. see https://github.com/timgrossmann/InstaPy/issues/4236, https://github.com/timgrossmann/InstaPy/issues/3712


Minor lints

use more specific asserts like assert_ne!
ignore intellij IDE files
remove a few un-needed namespace prefixes
link fixes

This is a followup from https://github.com/djc/bb8/pull/125

@djc audit & coverage CI targets don't seem to work. Coverage seem to only work if PR is from the same repo. No idea about audit.

Thanks!


Automator: update<EMAIL_ADDRESS>in<EMAIL_ADDRESS>

Generated by Automator - 2023-10-11T00:58:09+00:00

/test release-test

/test unit-tests


To-Do app pull request for partner-review

To-Do list: clean_code_partner_review
In this exercise I implemented the following as per the requirements:

Cloned To-Do app for clean code partner review.

General requirements checklist.

[X] No linters error.
[X] Followed the GitHub flow.
[X] documented work professionally.
[X] Followed the best practices of HTML, CSS, and JavaScript.

STATUS: APPROVED 🥇 🏆 🍹
Good job!!! 🎉 🎉🎉

Highlights 💯

No Linter errors ✔
Modules used professionally ✔
function for "Clear all completed" button is implemented ✔
Has event listener on the checkbox (change) ✔
Updated array of items is stored in local storage ✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


JSCS Demo Link

Added a link to the JSCS Demo Page.

Coverage remained the same when pulling a444937b5b5ffadb945384f136f8fb067da46632 on mdevils.demo-link into 4d90c261c9323293e8098d01209cecb26ab4fa20 on master.

+1. The demo looks really cool. Nice work!


IHttpOperationResponse should inherit from IDisposable

The interface defines two properties of types HttpRequestMessage and HttpResponseMessage (which are both IDisposable, but it doesn't define a way to dispose of them. The class HttpOperationResponse does.
We should move the IDisposable bit up to the interface so that those using the interface have access to the full set of functionality required to correctly manage the life cycle of one of these responses.

@fearthecowboy @markcowl -- Moved this here from the AutoRest repo, since the C# client runtime is apparently moving to here.

Note: HttpOperationResponse implements IDisposable, so ensure that we won't need to generate new code to support this.

I recommend we punt until we have a release with required breaking changes.  Any public interface changes require a breaking change notification, and this fix is not worth pegging the major version number.
Recommend punt to next major version.

@matthchr is this still an issue? I am cleaning up old issues. Will close this if I don't hear back in a week.

@shahabhijeet - yes, I don't see this fixed in the latest drops of the SDKs, so it's still a problem.

@matthchr ok I looked at this and what I am not very clear is the following:

are you suggesting that the base interface should have IDisposable (from a good design point of view)
or
You have a scenario where you will be using the base Interface IHttpOperationResponse and not the concrete class HttpOperationResponse? If yes can you provide me a scenario in your SDK where you are using the Interface and not the Concrete class.
I am actually more curious to know which API/Type you have in your SDK that you work with using the IHttpOperationResponse and not the base concrete class.

@shahabhijeet - I guess most just 1?
Since IHttpOperationResponse has two properties whose types implement disposable, it (from a good design point of view) should probably implement IDisposable itself.
We do use the interfaces to set up generic constraints, for example:
    public abstract class BatchRequestBase<TOptions, TResponse> : IBatchRequest<TResponse>
        where TOptions : IOptions, new()
        where TResponse : IAzureOperationResponse

Right now it doesn't really matter that IAzureOperationResponse doesn't implement IDisposable because of the way that the concrete class actually implements Dispose and the way we invoke things we never actually manually call Dispose on an IAzureOperationResponse -- we just rely on the default ref-counting implementation to do it for us. But we cant call dispose right now even if we wanted to because IAzureOperationResponse doesnt implement dispose.
As per here: https://github.com/aspnet/Security/issues/886 it's probably not that big of a deal the way we're doing it now but still think we should just fix the interfaces anyway.

@matthchr thank you for pointing this out.
At this point, we are not thinking of changing anything in our base interfaces. We will revisit this when we will be redesigning our base class probably after Titanium.
Plus the referenced issue is good confirmation that this will be good to have and not a must.
Closing.


Where do tests go for loaders?

I considered extending some of the loaders, but no tests exist. Since there are so many loaders and no (loader) tests it seems like a pattern or policy. Can I add tests to the main test folder? Should they go into the loader's folder? Is it considered a bad style to add tests for some reason?
(I am new to the repo; apologies if this question has an answer elsewhere already. Please point to it in that case.)

There's a tests folder
https://github.com/emptycrown/llama-hub/tree/main/tests


Configuration Conflict in containerd When Deploying Dragonfly

Description
When using Helm to deploy a mirror acceleration service in a Kubernetes cluster, the containerd service fails to start and logs the following warning:
failed to load plugin io.containerd.grpc.v1.cri" error="invalid plugin config: `mirrors` cannot be set when `config_path` is provided

Even though this is a warning log, it prevents the containerd service from starting.
Environment

Kubernetes version: v1.24
Dragonfly Chart version: v1.1.65
Containerd version: v1.6.20

Expected Behavior
The containerd service should start and run normally.
Actual Behavior
The containerd service fails to start and logs the following warning:
failed to load plugin io.containerd.grpc.v1.cri" error="invalid plugin config: `mirrors` cannot be set when `config_path` is provided

Containerd Error Log Screenshot

Containerd Configuration
My containerd configuration before installing Dragonfly was as follows:
[plugins.'io.containerd.grpc.v1.cri'.registry]
  [plugins.'io.containerd.grpc.v1.cri'.registry.mirrors]
    [plugins.'io.containerd.grpc.v1.cri'.registry.mirrors.'docker.io']
      endpoint = ['https://mirror.baidubce.com']

After installing Dragonfly, the following configuration was injected, causing containerd to fail and the node to show as NotReady:
[plugins.'io.containerd.grpc.v1.cri'.registry]
  config_path = "/etc/containerd/certs.d"
  [plugins.'io.containerd.grpc.v1.cri'.registry.mirrors]
    [plugins.'io.containerd.grpc.v1.cri'.registry.mirrors.'docker.io']
      endpoint = ['https://mirror.baidubce.com']

Removing the following configuration allows containerd to start normally:
[plugins.'io.containerd.grpc.v1.cri'.registry.mirrors]
  [plugins.'io.containerd.grpc.v1.cri'.registry.mirrors.'docker.io']
    endpoint = ['https://mirror.baidubce.com']

Question

Why is the configuration injected into containerd so abruptly?
Can I control the configuration injected into containerd? I could not find any configurable options in the values.yaml file.

@liuyuxuan0723
Can you provide your entire containerd config before installing, dfinit entire config and  dfinit verison?

After deploying Dragonfly, the complete containerd configuration is as follows:
version = 2
disabled_plugins = []
imports = []
oom_score = -999
required_plugins = []
root = '/cce/containerd'
state = '/run/containerd'
[debug]
  address = '/run/containerd/debug.sock'
  level = 'info'
[plugins]
  [plugins."io.containerd.nri.v1.nri"]
    # Enable NRI support in containerd.
    disable = false
    # Allow connections from externally launched NRI plugins.
    disable_connections = false
    # plugin_config_path is the directory to search for plugin-specific configuration.
    plugin_config_path = "/etc/nri/conf.d"
    # plugin_path is the directory to search for plugins to launch on startup.
    plugin_path = "/opt/nri/plugins"
    # plugin_registration_timeout is the timeout for a plugin to register after connection.
    plugin_registration_timeout = "5s"
    # plugin_request_timeout is the timeout for a plugin to handle an event/request.
    plugin_request_timeout = "2s"
    # socket_path is the path of the NRI socket to create for plugins to connect to.
    socket_path = "/var/run/nri/nri.sock"
  [plugins.'io.containerd.grpc.v1.cri']
    enable_selinux = false
    enable_tls_streaming = false
    max_concurrent_downloads = 10
    sandbox_image = 'registry.baidubce.com/cce-public/pause:3.1'
    stream_server_address = '<IP_ADDRESS>'
    stream_server_port = '0'
    [plugins.'io.containerd.grpc.v1.cri'.cni]
      bin_dir = '/opt/cni/bin'
      conf_dir = '/etc/cni/net.d'
      conf_template = ''
    [plugins.'io.containerd.grpc.v1.cri'.containerd]
      default_runtime_name = 'runc'
      [plugins.'io.containerd.grpc.v1.cri'.containerd.runtimes]
        [plugins.'io.containerd.grpc.v1.cri'.containerd.runtimes.runc]
          container_annotations = []
          pod_annotations = []
          privileged_without_host_devices = false
          runtime_type = 'io.containerd.runc.v2'
    [plugins.'io.containerd.grpc.v1.cri'.registry]
      config_path = "/etc/containerd/certs.d"
      [plugins.'io.containerd.grpc.v1.cri'.registry.mirrors]
        [plugins.'io.containerd.grpc.v1.cri'.registry.mirrors.'docker.io']
          endpoint = ['https://mirror.baidubce.com']

The Helm chart configuration is similar to the following:
containerRuntime:
  containerd:
    enable: true
    injectConfigPath: true
    registries:
      - "docker.io"

  initContainer:
    image:
      repository: dragonflyoss/openssl
      tag: latest

scheduler:
  image:
    repository: dragonflyoss/scheduler
    tag: latest
  replicas: 1
  metrics:
    enable: true
  config:
    verbose: true
    pprofPort: 18066
    console: true
  initContainer:
    image:
      repository: dragonflyoss/busybox
      tag: latest

seedPeer:
  image:
    repository: dragonflyoss/dfdaemon
    tag: latest
  replicas: 3
  metrics:
    enable: true
  config:
    verbose: true
    pprofPort: 18066
    console: true
  initContainer:
    image:
      repository: dragonflyoss/busybox
      tag: latest

dfdaemon:
  image:
    repository: dragonflyoss/dfdaemon
    tag: latest
  metrics:
    enable: true
  config:
    verbose: true
    pprofPort: 18066
    console: true
    keepStorage: true
    scheduler:
      manager:
        seedPeer:
          enable: true
  initContainer:
    image:
      repository: dragonflyoss/busybox
      tag: latest

manager:
  enable: true
  image:
    repository: dragonflyoss/manager
    tag: latest
  replicas: 1
  metrics:
    enable: true
  config:
    verbose: true
    pprofPort: 18066
    console: true
  initContainer:
    image:
      repository: dragonflyoss/busybox
      tag: latest

I did not enable the client and seed-client in my configuration, but when I deployed using the v1.1.72 version of the chart, dfinit was used for initialization, and I encountered similar logs (not the exact logs, but the general idea):
key [plugins.'io.containerd.grpc.v1.cri'.registry] already exists, containerd configuration failed

After manually removing this configuration from containerd, the client started normally.

@liuyuxuan0723 I think you need use the rust client, refer to https://d7y.io/docs/next/getting-started/installation/helm-charts/.

I tried deploying using the Rust client and enabled dfinit initialization. When using mirror mode to inject the configuration, the containerd configuration is as follows:
[plugins."io.containerd.grpc.v1.cri".registry]
    [plugins."io.containerd.grpc.v1.cri".registry.mirrors.'docker.io']
      endpoint = ['https://mirror.baidubce.com']

[plugins."io.containerd.grpc.v1.cri".registry.mirrors."artifactory-registry.com"]
endpoint = ["http://<IP_ADDRESS>:4001", "https://artifactory-registry.com"]

However, when pulling images on the host, it seems that the client is not acting as a proxy. The logs of this task can be viewed in the client, but no cached pieces are found in /var/lib/dragonfly/content.
When dfinit injects the configuration using the config_path method, it correctly proxies the download requests. Cached pieces can be found in the /var/lib/dragonfly/content directory. The certs.d configuration is as follows:
server = "https://artifactory-registry.com"

[host."http://<IP_ADDRESS>:4001"]
capabilities = ["pull", "resolve"]

[host."http://<IP_ADDRESS>:4001".header]
X-Dragonfly-Registry = "https://artifactory-registry.com"

I have one more question. Does the certs.d configuration need to include the following entry to ensure that images can still be pulled if the client goes down?
[host."https://artifactory-registry.com"]
capabilities = ["pull", "resolve"]


Crash on 32-bit XP when using --single-process flag

At first, that command flag greatly improved performance of Supermium, but on the second launch, it crashes on start

As I know from very old Chrome versions (may be v.35), the "--single-process" flag was for running all Chrome.exe in single process, not just GPU-related. At some time this was retired by chromium developers, And I dont know how much changes in architecture must be to return this feature. May be few, May be no.
Now I use "--process-per-site". But this in not sufficient, of couse.

How about with --disable-gpu switch as well? I wonder if it's trying to use half-baked GPU acceleration.

--disable-gpu switch doesn't help me to avoid crash

As I mentioned above, the "--single-process" flag is just "unsupported, deprecated, and retired" flag by upstream Chromium very long time. To just avoid crash it can be disabled at all.... But to restore feature, its need to rework. May be not so simple. You can try by youself, if you are developer.
https://superuser.com/questions/918831/chrome-exe-single-process-not-working (from 2015 year)

It actually worked in some newer builds from 2019 but extensions either didn't work at all or worked rarely.

The --single-process flag places the renderer processes in the main process (as threads). This means that any fault in the renderer processes crashes the main process as well, instead of having only the renderer process crash (and a new one created by the renderer watchdog in the main process).
I have noticed several bugs specific to the --single-process switch over time, which is why I focus on making it usable without any these special flags.

Some tests (all tests were performed with Supermium 122.0.6261.85 Hotfix (R2) 32bit):
1.) starting it without any flags on my RealHW XPSP3 host Pentium E5200 2cores, does succeed
2.) starting it with the flag --single-process on my RealHW XPSP3 host PentiumE5200 2cores does succeed, no crash with the single-process-flag in this scenario
3.) starting it without any flags from within a Win2k3sp2 guest VM, emulating a single core-CPU via VBox4.3.12, no VT-x, starts up the browser, but maxes out the CPU with 100% load for really long time. Not possible to work with it like that. At least it did start up now and didn't crash which itself is an improvement already, (startup didn't work in Supermium121 yet). As an attempt to mitigate the slowness a bit, I tested scenario 4:
4.) starting it with the flag --single-process within a Win2k3sp2 guest VM, emulating a single core-CPU via VBox4.3.12, no VT-x, makes the browser crash within 2 seconds: 2 chrome.exe processes do get created, then it crashes before the GUI is shown. Here are 3 crashdumps of 3 consecutive attempts of scenario4:
8e8ee8db-b276-45a0-aee5-9d2cbac44da0.dmp
016b5078-fb29-4453-9c19-6fa3346c80a7.dmp
b6a312e3-71ec-4349-8573-d1a2ea0c009b.dmp


Robo3t is not available, can you update it?

Robo3t is not available, can you update it?

Upstream issue. Any windows download do not work.

Thank you, has resumed downloading~

Thank you, it has been restored


WIP: introducing concept of bill.Receipt

Refining concepts around bill.Receipt to use for payment and remittence documents.
Replaces #299

Replaced with #379


add email for Erasmus University Rotterdam

add email for Erasmus University

school name: Erasmus University Rotterdam
web site: https://www.eur.nl/en
address: Burgemeester Oudlaan 50, 3062 PA Rotterdam, Netherlands

@pudongkai I am sorry, the domain eur.nl is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


RangeSync should handle PARENT_UNKNOWN errors

Describe the bug
From this comment in the block processor, if the sync ignores WOULD_REVERT_FINALIZED_SLOT the sync must also be able to recover.
https://github.com/ChainSafe/lodestar/blob/4e8e969fa4223749ce760a02563913f3dbb44a94/packages/lodestar/src/chain/blocks/index.ts#L104-L117

the issue could be reproduced with "--weakSubjectivitySyncLatest` flag
Sep-24 13:38:15.294 [CHAIN]           error: Block error slot=1329602, errCode=BLOCK_ERROR_PARENT_UNKNOWN code=BLOCK_ERROR_PARENT_UNKNOWN, parentRoot=0xb58db597b73b564299825326035c05706203e15755e887a9450898de92358a30
Error: BLOCK_ERROR_PARENT_UNKNOWN
    at verifyBlockSanityChecks (/Users/tuyennguyen/Projects/workshop/lodestar/packages/lodestar/src/chain/blocks/verifyBlock.ts:58:11)
    at verifyBlock (/Users/tuyennguyen/Projects/workshop/lodestar/packages/lodestar/src/chain/blocks/verifyBlock.ts:32:3)
    at processChainSegment (/Users/tuyennguyen/Projects/workshop/lodestar/packages/lodestar/src/chain/blocks/index.ts:101:51)
    at JobItemQueue.BlockProcessor.jobQueue.queue_1.JobItemQueue.maxLength [as itemProcessor] (/Users/tuyennguyen/Projects/workshop/lodestar/packages/lodestar/src/chain/blocks/index.ts:31:18)
    at Timeout.JobItemQueue.runJob [as _onTimeout] (/Users/tuyennguyen/Projects/workshop/lodestar/packages/lodestar/src/util/queue/itemQueue.ts:85:33)
    at listOnTimeout (node:internal/timers:557:17)
    at processTimers (node:internal/timers:500:7)


Add GraphQL query validators

https://github.com/keystonejs/keystone-5/issues/1469

Love it! But I also want @timleslie or @jesstelford to look at it


Migrate to pnpm

Migrate the package manager from yarn to pnpm.

coverage: 100.0%. remained the same
when pulling 950a8895795f0cdc280e8030fc6ea83959e310f2 on migrate_to_pnpm
into 2b1fbab23cd6a0f1622585cf28d8256e05f7dd7d on master.


Set outfit is broken

Description

Only shows clothing not sets
Reproduction

Screenshots

Additional context

dupe #27276

thanks for the report and yeah its a duplicate!


cloud/capabilities endpoint is not implemented

Sending GET request to cloud/capabilities endpoint returns 404 status
ocis-ocs on  capabilities-test [!] via 🐹 v1.13.3
❯ curl http://localhost:9110/ocs/v2.php/cloud/capabilities | xmllint --format -
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   141  100   141    0     0   137k      0 --:--:-- --:--:-- --:--:--  137k
<?xml version="1.0" encoding="UTF-8"?>
<ocs>
  <meta>
    <status>error</status>
    <statuscode>998</statuscode>
    <message>not found</message>
  </meta>
</ocs>

@dpakach this is an old issue that was transferred from the archived ocis-ocs repo a few weeks ago.
Please close if no longer relevant.

this has been fixed


No imaging component suitable to complete this operation was found

When loading .webp images, I get this error and no thumbnail for

🤔

Which version are you using?

Latest (v1.1)

Are you sure the image is valid? (Non-empty image, opens in other image viewers?)

Yes, I am able to preview them in my browser (webp files)

Could you post a few samples of the affected images here so I can try on my end?

Sure, here's a zip with the iamges I tried:
images.zip

It seems to work fine for me.

Can you try right-click > Remove Entry then Scan folders for new images?

Can you try right-click > Remove Entry then Scan folders for new images?

Still receiving the same error.
Here's a screen-recording displaying the error. I would assume this is an error related to caching, because the folder I had it set to was non-existing and it still loaded the same entries you see in the video. 🤷🏼
output.webm


Implement email notifications

[ ] Specify what email notifications will be sent
[ ] Design the notifications (especially if they are reports)
[ ] Implement the notifications

Email notifications were not included in MVP.  Much of this will be easier once the WDP integration and authentication aspects have been finalised, and the system's relationship to user accounts is better understood.


Assignee selector

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/21716
$ https://github.com/Expensify/App/issues/21716#issuecomment-1627954990
Tests

Open Expensify as User A.
Go to any room.
Click on 'Assign Task,' fill in the task details, and share it in the room.
Open Expensify as User B, who is not the assignee, assignor, or PolicyAdmin.
Go to the same room and open the task report.
Verify that User B cannot edit the title, description, assignee, or mark the task as completed

Perform same steps with user C who is policy admin and verify that he can perform all actions.
Perform same steps with user D who is Task assignee and verify that he can perform all actions.

Open Expensify as User A.
Go to any group.
Click on 'Assign Task,' fill in the task details, and share it in the room.
Open Expensify as User B, who is not the assignee, or assignor.
Go to the same group and open the task report.
Verify that User B cannot edit the title, description, assignee, or mark the task as completed.

[x] Verify that no errors appear in the JS console

Offline tests

Same as above.
QA Steps

Open Expensify as User A.
Go to any room.
Click on 'Assign Task,' fill in the task details, and share it in the room.
Open Expensify as User B, who is not the assignee, assignor, or PolicyAdmin.
Go to the same room and open the task report.
Verify that User B cannot edit the title, description, assignee, or mark the task as completed

Perform same steps with user C who is policy admin and verify that he can perform all actions.
Perform same steps with user D who is Task assignee and verify that he can perform all actions.

Open Expensify as User A.
Go to any group.
Click on 'Assign Task,' fill in the task details, and share it in the room.
Open Expensify as User B, who is not the assignee, or assignor.
Go to the same group and open the task report.
Verify that User B cannot edit the title, description, assignee, or mark the task as completed.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/93134676/4959530e-eb41-4dc3-b795-4265a1276699

Mobile Web - Chrome
https://github.com/Expensify/App/assets/93134676/9780d4cd-a462-4199-b5d2-b476702b0e0d

Mobile Web - Safari
https://github.com/Expensify/App/assets/93134676/82235cf2-717d-46b9-822e-56715b6523f3

Desktop
https://github.com/Expensify/App/assets/93134676/f1027ede-df16-48d5-9f14-c074e7dbd5ba

iOS
https://github.com/Expensify/App/assets/93134676/51f475dc-4d36-42f9-bb03-20ef20797e33

Android
https://github.com/Expensify/App/assets/93134676/0b2d07fe-6e04-42f0-90f9-85b94545bb5d

@Nodebrute Can you resolve the conflicts please?

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/3069065/67909ba5-2184-4302-897d-85d3cdd7bb64

Mobile Web - Chrome
https://github.com/Expensify/App/assets/3069065/7012084c-e245-4434-9765-fbb357be9fa7

Mobile Web - Safari
https://github.com/Expensify/App/assets/3069065/68604c33-897f-4509-85f2-2bd4c2b6167a

Desktop
https://github.com/Expensify/App/assets/3069065/6d8a6811-6999-421d-a5a7-cf37d0f0a2b7

iOS
https://github.com/Expensify/App/assets/3069065/4c7db2ae-1e15-4945-916d-36ff147e4ffc

Android

@Nodebrute We made a mistake in the merge conflict. Can you please correct it?

@Nodebrute I see you're marking my comments resolved but when I review the code I don't see any changes. Can you confirm if you've pushed the change? and can you wait for the reviewer to review and mark them resolved? In a large PR, it would be difficult for review to verify the changes.

Thanks for the update @Nodebrute. I've completed my checklist. Waiting to get another engineer reassigned to review the PR.

All covered, thanks for the patience here!

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/Li357 in version: 1.3.44-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 1.3.45-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Read-Only public dashboard list

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
UI Feature
🔖 Feature description
Making the panel public but not editable by users not logged in.
I think the panel should have a option to allow users not logged in to be able to view but not edit or change anything.
✔ Solution
Add the option for view only.
❓ Alternatives
No response
📝 Additional Context
No response

Agreed - this would be a crucial item to publishing a public-facing dashboard for "customers" (look, but don't touch). ;)

@ComputerTech312
I think this feature is better covered by using status pages.
Could you go into details why this would be not an option for you?


Issue with load_image not accessing the correct index in an array

I was running into an issue withload_image line 394, where
z = deltas == np.min(np.abs(deltas))
Is supposed to return an index value for the deltas array where it is equal to the minimum absolute value. However, this was returning an empty array. I fixed it using the following code
z = np.where(np.abs(deltas) == np.min(np.abs(deltas)))[0]
Not sure if this is something specific to me, but I ran into it when I was trying to do the extract function.

I fixed this


Support Binary --> Utf8View casting

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
While working on https://github.com/apache/datafusion/issues/12788 with  StringView upstream in DataFusion, @goldmedal  found that casting from BinaryArray --> Utf8View is not supported
Describe the solution you'd like
Support casting BinaryArray --> Utf8View
Describe alternatives you've considered
Here is a modified test_binary_to_view test that should pass

    #[test]
    fn test_binary_to_view() {
        _test_binary_to_view::<i32>();
        _test_binary_to_view::<i64>();
    }

    fn _test_binary_to_view<O>()
    where
        O: OffsetSizeTrait,
    {
        let binary_array = GenericBinaryArray::<O>::from_iter(VIEW_TEST_DATA);

        assert!(can_cast_types(
            binary_array.data_type(),
            &DataType::Utf8View
        ));

        assert!(can_cast_types(
            binary_array.data_type(),
            &DataType::BinaryView
        ));

        let string_view_array = cast(&binary_array, &DataType::Utf8View).unwrap();
        assert_eq!(string_view_array.data_type(), &DataType::Utf8View);

        let binary_view_array = cast(&binary_array, &DataType::BinaryView).unwrap();
        assert_eq!(binary_view_array.data_type(), &DataType::BinaryView);

        let expect_string_view_array = StringViewArray::from_iter(VIEW_TEST_DATA);
        assert_eq!(string_view_array.as_ref(), &expect_string_view_array);

        let expect_binary_view_array = BinaryViewArray::from_iter(VIEW_TEST_DATA);
        assert_eq!(binary_view_array.as_ref(), &expect_binary_view_array);
    }

Full Diff

diff --git a/arrow-cast/src/cast/mod.rs b/arrow-cast/src/cast/mod.rs
index e3fad3da19..f147a9c3f6 100644
--- a/arrow-cast/src/cast/mod.rs
+++ b/arrow-cast/src/cast/mod.rs
@@ -5523,7 +5523,7 @@ mod tests {
     }

     #[test]
-    fn test_bianry_to_view() {
+    fn test_binary_to_view() {
         _test_binary_to_view::<i32>();
         _test_binary_to_view::<i64>();
     }
@@ -5534,14 +5534,25 @@ mod tests {
     {
         let binary_array = GenericBinaryArray::<O>::from_iter(VIEW_TEST_DATA);

+        assert!(can_cast_types(
+            binary_array.data_type(),
+            &DataType::Utf8View
+        ));
+
         assert!(can_cast_types(
             binary_array.data_type(),
             &DataType::BinaryView
         ));

+        let string_view_array = cast(&binary_array, &DataType::Utf8View).unwrap();
+        assert_eq!(string_view_array.data_type(), &DataType::Utf8View);
+
         let binary_view_array = cast(&binary_array, &DataType::BinaryView).unwrap();
         assert_eq!(binary_view_array.data_type(), &DataType::BinaryView);

+        let expect_string_view_array = StringViewArray::from_iter(VIEW_TEST_DATA);
+        assert_eq!(string_view_array.as_ref(), &expect_string_view_array);
+
         let expect_binary_view_array = BinaryViewArray::from_iter(VIEW_TEST_DATA);
         assert_eq!(binary_view_array.as_ref(), &expect_binary_view_array);
     }

The test currently fails with
assertion failed: can_cast_types(binary_array.data_type(), &DataType::Utf8View)
thread 'cast::tests::test_binary_to_view' panicked at arrow-cast/src/cast/mod.rs:5537:9:
assertion failed: can_cast_types(binary_array.data_type(), &DataType::Utf8View)
stack backtrace:
   0: rust_begin_unwind
             at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/std/src/panicking.rs:665:5
   1: core::panicking::panic_fmt
             at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/core/src/panicking.rs:74:14
   2: core::panicking::panic
             at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/core/src/panicking.rs:148:5
   3: arrow_cast::cast::tests::_test_binary_to_view
             at ./src/cast/mod.rs:5537:9
   4: arrow_cast::cast::tests::test_binary_to_view
             at ./src/cast/mod.rs:5527:9
   5: arrow_cast::cast::tests::test_binary_to_view::{{closure}}
             at ./src/cast/mod.rs:5526:29
   6: core::ops::function::FnOnce::call_once
             at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/core/src/ops/function.rs:250:5
   7: core::ops::function::FnOnce::call_once
             at /rustc/eeb90cda1969383f56a2637cbd3037bdf598841c/library/core/src/ops/function.rs:250:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Additional context

It should be relatively straightforward to make the cast kernel do this by first casting to StringArray

Hi, I saw this issue looked doable, created a PR to add this enhancement. Thanks tustvoid for the hint, helped me figure out the direction in this large file!


Normalize relationship ids when normalizeIds is set to true

This is a work in progress change that needs some feedback.
Fixes #972 mainly with code I stole from @jelhan
Changes the normalize method in ActiveModelSerializer to respect the serializeIds property and convert REST type id arrays into JSONApi objects.
Some specific issues:

Should this transform be applied to ActiveModel type APIs. I think they are similar to REST in this way, but need to confirm with someone who knows.
Is piggybacking on serializeIds the right path here or should I add a new normalizeIds property to the serializer?
This line is problematic:
let { belongsToAssociations, hasManyAssociations } = this.schema._registry[type].class.prototype;
It depends on the private _registry and also breaks several tests because they will need mock schema objects injected into the serializer.

@samselikoff et al is there a better way to determine what the types of associations are for a given model?

Thanks for pushing this forward.

Is piggybacking on serializeIds the right path here or should I add a new normalizeIds property to the serializer?
I'm quite sure if we really need this configuration at all. There are just a very few use cases in which a server ignores relationships presented in POST or PATCH request. It might be okay to not support these uses cases out of the box.

I tend to agree that (in REST type at least) the serializeIds should always be set to 'always'. However, since the option does exist there isn't a lot of reason not to make this configurable as well - given how simple the separation is to accomplish.

Please mind the difference between serialisation of a record for response and normalization of request payload. serializeIds option controls if relationship ids should be included in response. It's another question if relationships presented in payload should be ignored.

@jelhan yes, that was my issue with using this configuration option again. I think I will add a new option to control the normalization behavior. Thanks.

@jherdman I've fixed tests and found a way to use public APIs to get to the relationship data. What do you think?

@jherdman / @samselikoff I believe this is ready. We've been using this code in our application serializer for some time successfully. Are there further changes I should make? Should I abandon this PR and just continue to rely on application level serializers for REST type mocking?

Just checking in here. I'm happy to rebase this again, but don't want to keep spending the time if it wont be merged. I'm not really sure how anyone could be using the RestSerializer without this code though.

I apologize Jonathan - I should have said either yes or no, rather than leave you hanging.
Taking a look now!

@jrjohnson I am also happy to make the changes I mentioned in my review and add as a commit on top of yours, if you'd like (understandably) move on from this pr! :)

Thanks @samselikoff I will make these changes (you have enough to do) if I get stuck I will ask for some help.

🎉  Thank you @jrjohnson !


[Storage] Increased timeout for small chunks tests to reduce flakeyness

I noticed some flakeyness that was still occurring. Increased the timeout like Jacob did in his previous PR that relieved some of the flakeyness.

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


Coordinate Conversion Add-in generates bad grid when tool settings are set to "Default Map Unit" as Display Coordinate

From @migueldls on December 13, 2016 22:32

Grids are not accurate using the Map Point Tool when the Coordinate Conversion tool settings are set as "Default Map Unit" in the Display Coordinate settings. When the Map or Data Frame is set to any projection other than WGS84 Web Mercator coordinates not accurate or correct. Problem persists in both ArcMap and ArcPro. If is set the map or data frame to eg. WGS84 UTM 13N projection,  I get 31RE0000000000 in an area that should be 13RCR0000000000.

Expected Behavior

Should get accurate grids using the Map Point Tool in any projection or at least at a minimum get accurate grids when map or data-frame is set to any WGS84 projection.
Current Behavior

I get 31RE0000000000 in an area that should be 13RCR0000000000. Using the Map Point Tool in both ArcMap 10.4.1 and ArcPro 1.4
Possible Solution

Steps to Reproduce (for bugs)

In ArcPro add map set to Web Mercator prj. Add map set to WGS84 UTM zone 13N.
Pan to El Paso, TX.
Open CC Add-in, set display coordinate to "Default Map Unit"
Check Grids using Map Point Tool
Same process for ArcMap, change dataframe coordinate system to WGS84 UTM zone 13N. Verify problem persits.

Context

Your Environment

Version used: ArcMap 10.4.1, ArcPro 10.4 (7160)
Environment name and version (e.g. Chrome 39, node.js 5.4):
Operating System and version (desktop or mobile): Windows 7 En.
Link to your project:

Copied from original issue: Esri/military-tools-for-arcgis#85

@migueldls as I was looking at another issue I noticed that the commit for this was not quite correct. You should see the correct values reported but the wkid is incorrectly being included in the display string shown in the Input box. Will need to do another commit.

@jmhauck Yeah I noticed the wkid, and verified the fix for both ArcMap, ArcPro just now. I re-check after the commit for the wkid but this issue seems good now. ?..Does this tool only accept a Projected spatial reference with uniits in (meters)?

@migueldls it should accept projected coordinate values regardless of the linear unit.

@jmhauck, I verified the fix in ArcMap 10.4.1 and Pro 1.4 this issue now works as expected in both. Tool outputs correct coordinates using "default map" setting and I don't see the wkid in the display.


Feature/search functionality

Description
Created search functionality using axios call with the endpoint provided by DS and handled state using Redux. This allows users to search the document database using keywords/terms and populates the results. You can then click on a document from the list of search results which redirects user to the box account to view the desired document. Used thumbnail endpoint provided by DS to provide thumbnail view of documents in search results view.
Type of change

[X] New feature (non-breaking change which adds functionality)

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have removed unnecessary comments/console logs from my code
[X] My changes generate no new warnings
[X] I have checked my code and corrected any misspellings
[X] No duplicate code left within changed files
[X] Size of pull request kept to a minimum
[X] Pull request description clearly describes changes made & motivations for said changes

Loom Video:
https://www.loom.com/share/b97e8a34d41c4327b7536ba796542729

This looks great!  Awesome job and clean code.  Really like the link to the box PDF when you click the card.

 FE can send a search query to the DS search endpoint and get a response


fix(openapi): replace swagger-cli with redocly

its not working right now.
see: https://www.npmjs.com/package/@apidevtools/swagger-cli
and: https://redocly.com/docs/cli/guides/migrate-from-swagger-cli/#validate-openapiswagger-files

The issue was that @redocly/cli was not version pinned and yesterday's release is breaking it. I added the package in package.json and now when the release job for ci-standard-checks is run<EMAIL_ADDRESS>should be packed and npx should run that version instead of running @latest.

:tada: This PR is included in version 1.27.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


WPCS flagging an echoed $_GET variable as not-sanitised, while it shouldn't be sanitised, rather escaped

Bug Description
I have this code:
<input type="text" placeholder="Hit Enter To Search..." name="s" value="<?php echo isset( $_GET['s'] ) ? esc_html( $_GET['s'] ) : ''; ?>">
Running WPCS complains that I have an unsanitized $_GET variable.
I do not understand why this would be flagged. It is echoed, not stored, so we shouldn't sanitize, we should escape.
Either I have to take a break, or something is wrong here.
I am sorry if I don't see the tree in the forest...

Did you check the wiki  that explains the handling of the input data?
You could see the solution there 😉

That wiki shows how to sanitize
As far I know when we echo we do not sanitize, we escape.
Yet.... that wiki does not touch this aspect. Sanitising is the wrong thing to echo something.
In fact, if I split the $_GET into key/value and then echo the value... it asks me to escape it, not sanitize it.
I am lost 🤣

Running your code against the WordPress codebase (which you shouldn't do really in your project) gives me the following security warnings and errors:
 1 | WARNING | Processing form data without nonce verification. (WordPress.Security.NonceVerification.Recommended)
 1 | WARNING | Processing form data without nonce verification. (WordPress.Security.NonceVerification.Recommended)
 1 | ERROR   | $_GET data not unslashed before sanitization. Use wp_unslash() or similar
   |         | (WordPress.Security.ValidatedSanitizedInput.MissingUnslash)
 1 | ERROR   | Detected usage of a non-sanitized input variable: $_GET['s'] (WordPress.Security.ValidatedSanitizedInput.InputNotSanitized)

Nonce verification is necessary to guard against CSRF attacks. You need to sanitize and unslash your $_GET variable because you are using it in the input element. Which is a search field, and that means you are executing a DB query.
So the correct way to write the above code would be:
<?php

if (
	isset( $_GET['s'] )
	&& wp_verify_nonce( sanitize_text_field( wp_unslash( $_GET['s'] ) ), 'search_action' ) ) {
		$search_query = sanitize_text_field( wp_unslash( $_GET['s'] ) );
	?>
		<input type="text" placeholder="Hit Enter To Search..." name="s" value="<?php echo esc_html( $search_query ); ?>">

	<?php
}

You might need to check on what nonce you'll use, you can generate one inside your search form using wp_create_nonce() function.

It is just a snip from larger code (which doesn’t query the database) but I get it now;
It flash it because it’s added to an Input which might get resubmitted and then becomes … an input not an output.
got it.
thanks!


error: command 'gcc' failed with exit status 1

Hello!
I have in use:
Raspberry Pi B+ V1.2
USB Soundcard
Raspbian GNU/Linux 7 (wheezy)
#cat /proc/version
Linux version 4.1.13+ (dc4@dc4-XPS13-9333) (gcc version 4.8.3 20140303 (prerelease) (crosstool-NG linaro-1.13.1+bzr2650 - Linaro GCC 2014.03) ) #826 PREEMPT Fri Nov 13 20:13:22 GMT 2015
Here is the ERROR message:
pi@raspberrypi ~/rpi-audio-levels $ python setup.py build_ext --inplace
/usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown distribution option: 'install_requires'
  warnings.warn(msg)
running build_ext
cythoning src/rpi_audio_levels.pyx to src/rpi_audio_levels.c

Error compiling Cython file:
------------------------------------------------------------
...
                list bands_indexes not None):
        cdef:
            unsigned int i
            int **c_bands_indexes
            unsigned int bands_count = len(bands_indexes)
            float[:] data_memview = data
                 ^
------------------------------------------------------------

src/rpi_audio_levels.pyx:46:18: Expected an identifier or literal
building 'rpi_audio_levels' extension
creating build
creating build/temp.linux-armv6l-2.7
creating build/temp.linux-armv6l-2.7/src
creating build/temp.linux-armv6l-2.7/opt
creating build/temp.linux-armv6l-2.7/opt/vc
creating build/temp.linux-armv6l-2.7/opt/vc/src
creating build/temp.linux-armv6l-2.7/opt/vc/src/hello_pi
creating build/temp.linux-armv6l-2.7/opt/vc/src/hello_pi/hello_fft
gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv -O2 -Wall -Wstrict-prototypes -fPIC -I/usr/include/python2.7 -c src/rpi_audio_levels.c -o build/temp.linux-armv6l-2.7/src/rpi_audio_levels.o -I/opt/vc/src/hello_pi/hello_fft/
src/rpi_audio_levels.c:1:2: error: #error Do not use this file, it is the result of a failed Cython compilation.

Best regards!

What version of Cython do you have installed?
cython -V
I just ran into the same problem installing it on Wheezy.  The version installed with Wheezy is 0.15.1
You can upgrade to this version manually
https://packages.debian.org/wheezy-backports/armhf/cython/download
or install with pip
pip install cython --upgrade


redmine 5.x

to work with version 5. seems like this plugin might need change to init.rb:
#require 'redmine_magic_link/hooks'
require File.expand_path('lib/redmine_magic_link/hooks', __dir__)

There might be more.

in plugins/redmine_datetime_custom_field/init.rb
this should do it:
# Custom patches
#require_dependency 'redmine_datetime_custom_field/hooks'
require_dependency File.expand_path('lib/redmine_datetime_custom_field/hooks', __dir__)

There may be other 5.x compatibility issues, but this fixes the issue with the hooks.

Thank you for contributing.
The plugin has been updated and is now fully compatible with Redmine 5.


Support submodules in sub-directories, and non-master branches of submodules

Submodules in sub-directories weren't supported due to the '/' character being the field separator in sed; switched to ':' as being unlikely to appear in paths.
I needed to merge non-master branches too - added an optional positional argument for a branch name to override master.
Let me know if you want this repackaged as 2 PRs.

Hold off on merging - just realised the defaulting to master is now broken :-(

Ok, should be fine now.

Thanks for this!


command not available in pipeline

ProcessBuilder does not provide a shell for the shell "builtin" command

I've run everything through sh, so commands remain the same and we don't need a special case for command to be run as a parameter to sh


[TS migration] Migrate 'CommunicationsLink.js' component to TypeScript

Details

Fixed Issues

$https://github.com/Expensify/App/issues/25031
PROPOSAL:
Tests

[x] Open any profile and verify that contact details displayed as expected
[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

[x] Open any profile and verify that contact details displayed as expected
[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

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

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
android-native-converted.webm

Android: mWeb Chrome
android-web-converted.webm

iOS: Native
https://github.com/Expensify/App/assets/23215729/07d9bfab-df6e-4591-8e44-eeb1bcae0628

iOS: mWeb Safari
Uploading ios-web-converted.mp4…

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/23215729/7b713bdc-2bc2-4f99-9453-d33adcd182e2

MacOS: Desktop
https://github.com/Expensify/App/assets/23215729/00eba271-dada-4953-8079-c31704e1d921

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/jasperhuangg in version: 1.4.40-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
failure ❌


Get tag vertices or edgetype's edges

This pr is based on pr #228
Need to merge pr #228 first

Reimplement, close this pr.


Setup Staging Environment for testing on Firebase

[x] create firebase project

[x] setup GitHub actions for CI (https://github.com/marketplace/actions/github-action-for-firebase)

[x] setup subdomain and add TXT/A Records in DNS zone settings (optional)

@yudilevi what sub domain do you want to use for beta?

staging.swap.bancor.network?

closed.
CI is setup using GitHub Actions. Whenever you push to master it will be auto deployed to firebase production instance. Same goes for staging.
@yudilevi @Velua we can also talk about rules to those branches to protect them from accidental deployments.
Subdomain is also set up and will be available shortly.


Facing issue for redirecting to domain name

I have deployed the application using ng-toolkit/universal. facing an issue with redirecting to the domain name from hhtps://example.com to https://www.example.com

@darteam43 If you are using AWS, I do recommend reading Setup www domain redirect section at _https://serverless-stack.com/_.
Cheers!

@KingDarBoja  Thank you for the reply,
we are using google cloud for our application and we have configured load balancing problem is we tried to redirect all the request using htaccess but the htaccess file is not working on the angular server

@darteam43 I really wanna help you but that is beyond my current knowledge on Google Cloud :(
Maybe this StackOverflow post could help you but not sure to be honest.
Also this kind of question should be better posted on StackOverflow because it is related to the server configuration and more people can help you with it 🤖

@KingDarBoja
Thank you for your help will post a question in StackOverflow
Have a nice day!


Workflow assigned with TrustSitelists=True can fail at WorkQueueManager if files have not replicated

Impact of the bug
WMAgent
Describe the bug
When a workflow is assigned with TrustSitelists=True, it means that data can be read from any Disk storage and jobs don't necessarily need to run at the site that is associated with the RSE.
There is one requirement enforced in MSTransferor though, which is, every single primary (and parent) input block needs to be available somewhere in Disk, otherwise primary input data placement is required.
Here is a workflow/WQE that is failing to be acquired in vocms0255:
haozturk_Run2022C_ParkingDoubleElectronLowMass3_10Dec2022_230221_140043_3850
precisely this WQE (which will be removed once this workflow completes):
https://cmsweb.cern.ch/couchdb/workqueue/_design/WorkQueue/_rewrite/element/098bafb31af9d37863bad1286d43012b
and here is the error thrown in the WorkQueueManager component log [1].
Here is the current status of the replication rule for the relevant block [2].
How to reproduce it
Unsure, it could be that a rule got deleted and recreated after the workflow went from assigned to staging status transition.
Expected behavior
I don't think we can avoid this situation, given that data transfer problems can happen at any time.
However, we need to at least:

ensure that MSTransferor enforces input data to be FULLY available in at least one Disk endpoint (keep in mind that only T1 and T2 is used as valid input location)
and we should implement an alert in WMAgent such that people get notified when this unusual situation happens.

Additional context and error message
[1]
2023-03-01 17:07:16,373:140522083251968:INFO:WMBSHelper:"haozturk_Run2022C_ParkingDoubleElectronLowMass3_10Dec2022_230221_140043_3850" Injecting block /ParkingDoubleElectronLowMass3/Run2022C-v1/RAW#fb8c28
eb-9c46-439c-853a-21b00fd4309f (253 files) into wmbs.
2023-03-01 17:07:16,373:140522083251968:INFO:WMBSHelper:Adding files into WMBS for haozturk_Run2022C_ParkingDoubleElectronLowMass3_10Dec2022_230221_140043_3850 with PNNs: []
2023-03-01 17:07:16,455:140522083251968:INFO:WMBSHelper:Inserting 253 files in bulk into WMBS for haozturk_Run2022C_ParkingDoubleElectronLowMass3_10Dec2022_230221_140043_3850
2023-03-01 17:07:16,455:140522083251968:ERROR:File:File created in WMBS without locations!
File lfn: /store/data/Run2022C/ParkingDoubleElectronLowMass3/RAW/v1/000/357/440/00000/eb6367dc-cde6-4964-8ab3-2af9679e2557.root

2023-03-01 17:07:16,465:140522083251968:ERROR:WorkQueue:Failed to create subscription for haozturk_Run2022C_ParkingDoubleElectronLowMass3_10Dec2022_230221_140043_3850 with block name /ParkingDoubleElectro
nLowMass3/Run2022C-v1/RAW#fb8c28eb-9c46-439c-853a-21b00fd4309f
Error: File created in WMBS without locations!
File lfn: /store/data/Run2022C/ParkingDoubleElectronLowMass3/RAW/v1/000/357/440/00000/eb6367dc-cde6-4964-8ab3-2af9679e2557.root
Traceback (most recent call last):
  File "/data/srv/wmagent/v<IP_ADDRESS>/sw/slc7_amd64_gcc630/cms/wmagentpy3/<IP_ADDRESS>/lib/python3.8/site-packages/WMCore/WorkQueue/WorkQueue.py", line 357, in getWork
    match['Subscription'] = self._wmbsPreparation(match,
  File "/data/srv/wmagent/v<IP_ADDRESS>/sw/slc7_amd64_gcc630/cms/wmagentpy3/<IP_ADDRESS>/lib/python3.8/site-packages/WMCore/WorkQueue/WorkQueue.py", line 458, in _wmbsPreparation
    sub, match['NumOfFilesAdded'] = wmbsHelper.createSubscriptionAndAddFiles(block=dbsBlock)
  File "/data/srv/wmagent/v<IP_ADDRESS>/sw/slc7_amd64_gcc630/cms/wmagentpy3/<IP_ADDRESS>/lib/python3.8/site-packages/WMCore/WorkQueue/WMBSHelper.py", line 423, in createSubscriptionAndAddFiles
    addedFiles = self.addFiles(block)
  File "/data/srv/wmagent/v<IP_ADDRESS>/sw/slc7_amd64_gcc630/cms/wmagentpy3/<IP_ADDRESS>/lib/python3.8/site-packages/WMCore/WorkQueue/WMBSHelper.py", line 463, in addFiles
    totalFiles = self.topLevelFileset.addFilesToWMBSInBulk(self.wmbsFilesToCreate,
  File "/data/srv/wmagent/v<IP_ADDRESS>/sw/slc7_amd64_gcc630/cms/wmagentpy3/<IP_ADDRESS>/lib/python3.8/site-packages/WMCore/WMBS/Fileset.py", line 246, in addFilesToWMBSInBulk
    files = addFilesToWMBSInBulk(self.id, workflowName, files,
  File "/data/srv/wmagent/v<IP_ADDRESS>/sw/slc7_amd64_gcc630/cms/wmagentpy3/<IP_ADDRESS>/lib/python3.8/site-packages/WMCore/WMBS/File.py", line 558, in addFilesToWMBSInBulk
    raise RuntimeError(msg)
RuntimeError: File created in WMBS without locations!
File lfn: /store/data/Run2022C/ParkingDoubleElectronLowMass3/RAW/v1/000/357/440/00000/eb6367dc-cde6-4964-8ab3-2af9679e2557.root

[2]
amaltaro@lxplus8s11:~ $ rucio list-rules "cms:/ParkingDoubleElectronLowMass3/Run2022C-v1/RAW#fb8c28eb-9c46-439c-853a-21b00fd4309f"
ID                                ACCOUNT            SCOPE:NAME                                                                               STATE[OK/REPL/STUCK]    RSE_EXPRESSION                                                                                COPIES    EXPIRES (UTC)    CREATED (UTC)
--------------------------------  -----------------  ---------------------------------------------------------------------------------------  ----------------------  -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------  --------  ---------------  -------------------
a1d876b4be1542f2ba72699ecd004431  wmcore_transferor  cms:/ParkingDoubleElectronLowMass3/Run2022C-v1/RAW#fb8c28eb-9c46-439c-853a-21b00fd4309f  REPLICATING[245/8/0]    T1_UK_RAL_Disk|T2_UK_SGrid_Bristol|T1_IT_CNAF_Disk|T2_HU_Budapest|T2_PL_Swierk|T2_CH_CSCS|T2_ES_CIEMAT|T2_BE_IIHE|T2_TW_NCHC|T2_IT_Bari|T2_BE_UCL|T2_DE_DESY|T2_IT_Legnaro|T1_DE_KIT_Disk|T2_RU_ITEP|T2_CH_CERN|T2_IT_Pisa|T2_UK_London_IC|T2_IT_Rome|T2_ES_IFCA|T2_EE_Estonia                     1                          2023-02-21 14:04:15
61fc37c07af248feb006b8e713fc6048  wmcore_transferor  cms:/ParkingDoubleElectronLowMass3/Run2022C-v1/RAW#fb8c28eb-9c46-439c-853a-21b00fd4309f  SUSPENDED[207/46/0]     T2_IT_Bari|T1_RU_JINR_Disk|T2_US_Nebraska|T2_HU_Budapest|T2_BE_UCL|T2_BE_IIHE|T2_US_MIT|T2_BR_UERJ|T2_IT_Legnaro|T2_US_Vanderbilt|T2_FR_GRIF_IRFU|T2_ES_CIEMAT|T1_UK_RAL_Disk|T2_UK_London_Brunel|T2_IT_Pisa|T2_US_Purdue|T1_DE_KIT_Disk|T2_EE_Estonia|T1_IT_CNAF_Disk|T2_US_Wisconsin|T2_ES_IFCA  1                          2023-01-21 08:58:33

@jhonatanamado Jhonatan, once you have some spare cycles, would you mind looking into the rules above and try to get those last files finally replicated?
@haozturk FYI

Hi Alan, I updated the rules with some priority. Lets see if that helps. But Im puzzle about those rules. Why the rules (replicating the same data) were created with a 1 month of difference? Could be the aborted vs the new campaign ?

Thanks @jhonatanamado , yes, there were two workflows processing this data. The first one was aborted beginning of Feb, second one came in later in Feb, thus triggering the rule creation.

Hi @amaltaro thanks for spotting this. In general, we will start the processing of many workflows with partial input in production since the staging is the bottleneck. In this regard, we should start focusing on how we can achieve this in the optimal way.
Concerning this issue; I didn't know this bug. As an exception this year, we kept all the 2022 data at CERN and ran jobs in Eurasia by counting on AAA [1] It's more than 40 workflows.
Our expectation is to fix this bug and don't allow jobs to run unless the corresponding input is on disk. I know that this works when AAA is off. We should make it work when AAA is ON, too. Can you please explain why it doesn't work when AAA is ON? How hard is it to fix this bug, roughly?
Note that, fixing this bug is key as we'll start using partial pileup soon w/ secondary AAA ON. There, we'll have thousands of failures unless we fix this, as I understand.
How to handle these workflows when they are stuck in running-open due to non-existing files because of the problems in staging is another story. We're developing tools to handle that in P&R.
Hope it was clear. Please let me know what you think.
[1] https://its.cern.ch/jira/browse/CMSTRANSF-477

Btw, Alan can you please explain impact of this failure? As I understand it doesn't lead to job failures. What harm does it really cause?

As I was about to report another workflow having this issue, I noticed I've never replied to your question, sorry.
This should be a fairly container issue, which will cause LQ -> WMBS work acquisition to fail for that specific workflow, but it should not be a blocker for the rest of the work in the agent. Relevant code is around this try/except block:
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/WorkQueue/WorkQueue.py#L356-L369
Nonetheless, I do see a behavior where the WorkQueueManager thread seems to stop processing the remaining "matches" in the queue, from submit10 logs...
For the record, here is further log from WMAgent <IP_ADDRESS>:
2024-12-20 19:08:48,587:140064165144320:INFO:WorkQueueBackend:Accepting workflow: cmsunified_task_GEN-Run3Summer23BPixNanoAODv13-00007__v1_T_241206_133910_3564, with prio: 110110, element id: 2445cac635e54454648b1c1d93bd7813, for site:
 T1_US_FNAL
2024-12-20 19:08:48,587:140064165144320:INFO:WorkQueueBackend:And 10 elements passed location and siteJobCounts restrictions for: http://cmsgwms-submit10.fnal.gov:5984
2024-12-20 19:08:48,588:140064165144320:INFO:WorkQueueBackend:Total of 10 elements passed location and siteJobCounts restrictions for: http://cmsgwms-submit10.fnal.gov:5984
2024-12-20 19:08:48,588:140064165144320:INFO:WorkQueue:Got 10 elements matching the constraints
2024-12-20 19:14:06,871:140064165144320:INFO:WorkQueue:Running WMBS preparation for cmsunified_task_GEN-Run3Summer23BPixNanoAODv13-00004__v1_T_241205_130555_814 with ParentQueueId 13871493ab56a6949b2ff7afe3ef88c0,
  with common location ['T2_CH_CERN', 'T1_US_FNAL_Disk', 'T3_CH_CERNBOX']
2024-12-20 19:14:06,875:140064165144320:INFO:Fileset:Fileset created: cmsunified_task_GEN-Run3Summer23BPixNanoAODv13-00004__v1_T_241205_130555_814-GEN-Run3Summer23BPixNanoAODv13-00004_0-/TTto2L2Nu-2Jets_TuneCP5_13p6TeV_amcatnloFXFX-pythia8/Run3Summer23BPixMiniAODv4-130X_mcRun3_2023_realistic_postBPix_v2-v2/MINIAODSIM
2024-12-20 19:14:06,878:140064165144320:INFO:Workflow:Workflow id 48875 created for cmsunified_task_GEN-Run3Summer23BPixNanoAODv13-00004__v1_T_241205_130555_814
2024-12-20 19:14:06,909:140064165144320:INFO:WMBSHelper:"cmsunified_task_GEN-Run3Summer23BPixNanoAODv13-00004__v1_T_241205_130555_814" Injecting block /TTto2L2Nu-2Jets_TuneCP5_13p6TeV_amcatnloFXFX-pythia8/Run3Summer23BPixMiniAODv4-130X_mcRun3_2023_realistic_postBPix_v2-v2/MINIAODSIM (1798 files) into wmbs.
2024-12-20 19:14:06,910:140064165144320:INFO:WMBSHelper:Adding files into WMBS for cmsunified_task_GEN-Run3Summer23BPixNanoAODv13-00004__v1_T_241205_130555_814 with PNNs: []
2024-12-20 19:14:07,169:140064165144320:INFO:WMBSHelper:Inserting 1798 files in bulk into WMBS for cmsunified_task_GEN-Run3Summer23BPixNanoAODv13-00004__v1_T_241205_130555_814
2024-12-20 19:14:07,169:140064165144320:ERROR:File:File created in WMBS without locations!
File lfn: /store/mc/Run3Summer23BPixMiniAODv4/TTto2L2Nu-2Jets_TuneCP5_13p6TeV_amcatnloFXFX-pythia8/MINIAODSIM/130X_mcRun3_2023_realistic_postBPix_v2-v2/70000/e11cfa18-634b-44c2-98a2-2074550075ca.root
2024-12-20 19:14:07,170:140064165144320:ERROR:WorkQueue:Failed to create subscription for cmsunified_task_GEN-Run3Summer23BPixNanoAODv13-00004__v1_T_241205_130555_814 with block name /TTto2L2Nu-2Jets_TuneCP5_13p6TeV_amcatnloFXFX-pythia8/Run3Summer23BPixMiniAODv4-130X_mcRun3_2023_realistic_postBPix_v2-v2/MINIAODSIM
Error: File created in WMBS without locations!
File lfn: /store/mc/Run3Summer23BPixMiniAODv4/TTto2L2Nu-2Jets_TuneCP5_13p6TeV_amcatnloFXFX-pythia8/MINIAODSIM/130X_mcRun3_2023_realistic_postBPix_v2-v2/70000/e11cfa18-634b-44c2-98a2-2074550075ca.root
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/WMCore/WorkQueue/WorkQueue.py", line 357, in getWork
    match['Subscription'] = self._wmbsPreparation(match,
  File "/usr/local/lib/python3.8/site-packages/WMCore/WorkQueue/WorkQueue.py", line 463, in _wmbsPreparation
    sub, match['NumOfFilesAdded'] = wmbsHelper.createSubscriptionAndAddFiles(block=dbsBlock)
  File "/usr/local/lib/python3.8/site-packages/WMCore/WorkQueue/WMBSHelper.py", line 422, in createSubscriptionAndAddFiles
    addedFiles = self.addFiles(block)
  File "/usr/local/lib/python3.8/site-packages/WMCore/WorkQueue/WMBSHelper.py", line 462, in addFiles
    totalFiles = self.topLevelFileset.addFilesToWMBSInBulk(self.wmbsFilesToCreate,
  File "/usr/local/lib/python3.8/site-packages/WMCore/WMBS/Fileset.py", line 246, in addFilesToWMBSInBulk
    files = addFilesToWMBSInBulk(self.id, workflowName, files,
  File "/usr/local/lib/python3.8/site-packages/WMCore/WMBS/File.py", line 558, in addFilesToWMBSInBulk
    raise RuntimeError(msg)
RuntimeError: File created in WMBS without locations!
File lfn: /store/mc/Run3Summer23BPixMiniAODv4/TTto2L2Nu-2Jets_TuneCP5_13p6TeV_amcatnloFXFX-pythia8/MINIAODSIM/130X_mcRun3_2023_realistic_postBPix_v2-v2/70000/e11cfa18-634b-44c2-98a2-2074550075ca.root


Use pathlib.Path instead of os.path.*

[x] Tests with a significant number of years to be tested for your calendar.
[x] Changelog amended with a mention describing your changes. Note Please do NOT change the version number here. It's the project maintainers' duty.

Thanks a lot for this contribution!


How to add file proxy

Hi! How to add file proxy when the script asks for a proxy file name or a proxy API

Put the proxy file into the YouTube-Viewer folder..when script asks for a proxy file name or a proxy API type your file name.. say, your proxy file name is proxy so type proxy.txt
Or you can drag and drop the proxy file when script asks for a proxy file name or a proxy API..you will see the path on the cmd like this "E:\YouTube-Viewer\proxy.txt" but you need to remove these " " from the path

Thank you . it worked. but why i don't see views appear in realtime

Check YouTube studio


keg_relocate: only replace matches at the start of a path

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

See #11302
This PR modifies the matching used in keg_relocate so that prefix/cellar/library/repository paths are only replaced with a placeholder if they appear at the start of the path. For example, this:
cat /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/objc-run.rb
cat /foo/bar/usr/local/baz.txt

Becomes this:
cat @@HOMEBREW_LIBRARY@@/Taps/homebrew/homebrew-core/Formula/objc-run.rb
cat /foo/bar/usr/local/baz.txt

I've tested this locally with the objc-run formula (see https://github.com/Homebrew/homebrew-core/pull/76394) and it appears to work:
$ diffoscope existing-objc-run--1.4.big_sure.bottle.1.tar.gz new-objc-run--1.4.big_sur.bottle.2.tar.gz
--- existing-objc-run--1.4.big_sure.bottle.1.tar.gz
+++ new-objc-run--1.4.big_sur.bottle.2.tar.gz
│   --- existing-objc-run--1.4.big_sure.bottle.1.tar
├── +++ new-objc-run--1.4.big_sur.bottle.2.tar
│ ├── objc-run/1.4/bin/objc-run
│ │ @@ -103,15 +103,15 @@
│ │
│ │     )
│ │     xcodebuild -workspace "$projectDirPath/clitest.xcworkspace/" -scheme clitest -configuration Release DEPLOYMENT_LOCATION=YES DSTROOT=build DWARF_DSYM_FOLDER_PATH=build
│ │     xcodebuildExitCode=$?
│ │
│ │     # on clang success, run compiled application and remove it
│ │     if [[ $xcodebuildExitCode -eq 0 ]]; then
│ │ -           cp "$projectDirPath/build@@HOMEBREW_PREFIX@@/bin/clitest" $target
│ │ +           cp "$projectDirPath/build/usr/local/bin/clitest" $target
│ │             trap '{ rm -f "$target" ; exit 1; }' INT
│ │             "$target" "$@"
│ │             trap '' INT
│ │             rm -f "$target"
│ │     else
│ │             echo "objc-run: xcodebuild returned with error" 1>&2
│ │             exit 1

The @@HOMEBREW_PREFIX@@ placeholder is no longer used in the new bottle.

Some other notes:
Right now I used [a-zA-Z0-9] as the pattern for characters not to match before the given path. This means that anything like a/usr/local, A/usr/local, 0/usr/local will not cause the placeholders to be used. If there's a better pattern to use I'm open to it. I don't think these should cause any issues, though, as I can't think of a situation in which the text would match that pattern but still need a placeholder.
Also, I haven't really tested this througoughly and I'm not totally sure what the best way to do so it. I've tested on objc-run, but that's it. Is there anywhere in brew tests that tests the placeholder substitution? I wasn't able to find one but if there is, I'd like to add a test to ensure that these patterns aren't matched.

Review period will end on 2021-05-07 at 05:32:56 UTC.

Review period ended.

Unless there are more comments, I'll plan on merging a little bit later today. I'll try to be around after to make sure there are no major problems.

Okay, this last commit is pretty minimal. Pretty much just style changes from the review. I've retested, though, so I feel good that the rpath relocation still works as expected.


[ContentAssist] Variables declared below the usage should not appear as suggestions

 Issue by dilinisg
Tuesday Aug 22, 2017 at 13:33 GMT
Originally opened as https://github.com/ballerina-lang/composer/issues/2837

Release 0.92
Steps

Declare a variable as int i;
Add a while loop above this line and write the condition as "i < 2"

Issue
"i" appears as suggestions at step 2

In order to implement this capability, we need to have the position information of the BSymol nodes. Since the current core implementation does not support this we will be adding this as an improvement. We need to implement this behavior as a feature

Above issue is no longer valid in VSCode plugin. Hence, closing the issue.

Above issue is no longer valid in VSCode plugin. Hence, closing the issue.