jupyter file cannot save data normal

Environment data

VS Code version: 版本: 1.51.1
OS and version: macOS 11.0.1
Python version (& distribution if applicable, e.g. Anaconda): 3.7.3.
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A

Expected behaviour
The data can be actually saved after clicking command + S
Actual behaviour
After click command + S, the * in the upper right corner of the file name disappears. It seems that the data has been saved, but in fact, it is not saved at all.
Steps to reproduce:
unknown. No pattern was found, but it does happen in some cases. Has happened many times.
f*!

Thanks for the bug. We haven't seen this issue internally. Any more information you can give about a repro would be great.
For example do you have a notebook that this seems to happen on? What steps were you performing when the problem occurred?
Can you also include the 'Jupyter' tab output?

Thanks for the bug. We haven't seen this issue internally. Any more information you can give about a repro would be great.
For example do you have a notebook that this seems to happen on? What steps were you performing when the problem occurred?
Can you also include the 'Jupyter' tab output?

@rchiodo
This bug happened again this morning.
I currently confirm that there will be at least two of the following situations:

Sometimes I modify the file at time a and save it (via command + S; confirm that the * sign in the upper right corner of the file name has disappeared), and then at some time b after time a, I found that the modified content of a cell is not displayed, and the display is still unmodified.

But after I closed the file, exited VSCode, and reopened the file, the modified content was displayed again. This situation is very strange, but fortunately.

The second situation is very bad. Similarly, I modify the file at a certain time and save it (via command + S; confirm that the * sign in the upper right corner of the file name has disappeared), and then I close and reopen the file, and I will find the latest modified content is not saved; only the earlier changes are saved.

Both yesterday and today resulted in the loss of my code, and I dare not use it anymore.
More information that may be useful:

My intuitive feeling is that the saved execution is like a queue. The problem is that although it shows that the latest modification has been saved, it is not actually saved;
There are about 300 cells in the jupyter file with these problems. Although there are occasional slight delays and freezes, I think this file is not very large. I am not sure if it is related to this.

Thank you.

Can you share the notebook that causes the problem?

The root cause of this problem is a mismatch between what the UI thinks and what the backend (VS code has an extension host process where our backend lives) thinks the contents of the notebook are .
We've heard this before, but it's always been really difficult to get a repro.

You might have better luck using our newer notebook preview that we've been working on.
https://devblogs.microsoft.com/python/notebooks-are-getting-revamped/
That doesn't have the separate processes trying to keep data in sync (VS code is handling updates).

Dupe of #427.

Can you share the notebook that causes the problem?

@rchiodo
Thanks for the reply and processing. I can send it to you before next Monday. How can I send it to you? Maybe you can give me an email?

You might have better luck using our newer notebook preview that we've been working on.
https://devblogs.microsoft.com/python/notebooks-are-getting-revamped/
That doesn't have the separate processes trying to keep data in sync (VS code is handling updates).

Thanks Reply. I am very happy to know this news. I will continue to follow this progress of the project and hope to support it in the official version as soon as possible.
thanks again!

Thanks for the reply and processing. I can send it to you before next Monday. How can I send it to you? Maybe you can give me an email?

If you don't want to upload it here, you can e-mail it to me. My same github id followed by at microsoft.com.


Add example to 'sources' section of the params

The formatting comment wasn't clear on specifying multiple repos on
upgrade. Added an example for the case with the most confusion.

Looks better.


Changing the whole site name to "Blog"

Firstly thanks for this awesome package! :heart_eyes:
When i open the blog the <title> of my site change to "Blog"no matter if i go to other routes later, thanks :+1: :+1:

You need to set it back using document.title = 'some title'. If using iron router you should be able to do it in your after hook like this.route('someRoute', {path: '/some-path', after: function() { document.title = 'Some path'; }})

Hey cool man that helps, thank you bud :+1:  @coopermaruyama

@javierocanas Yes, iron-router (and meteor in general as a single-page-app) requires special handling of the page title. Here's a recent thread on the matter:
https://groups.google.com/forum/#!topic/meteor-talk/earnDjNSHGw

@hellogerard  Thanks buddy i learned something new :+1:

@hellogerard While this can certainly be fixed on a case-by-case basis as described above, it seems pretty easy to let users prevent this override behavior, if they want to.
I've fixed this for my own use by adding a Blog.settings.retainPageTitle option that skips the two overwriting document.title assignments in the current code.
Would you be interested in a (tiny) pull request with this change?


[Backport release-22.05] matrix-synapse: 1.59.1 -> 1.60.0

Description of changes
cc #175607. nixosTests.matrix-synapse passes, but it'd be nice if someone running 22.05 could try this.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Merging since no opposition.


Handle route delegation properly

When a set of routes is delegated to another router, there is a missing "/" between the prefix of the delegated route and the last part of the route.
For instance:
->       /endpoint        myEndpoint.router.Routes

and in the myEndpoint.router.routes:
GET    /my/endpoint     ....

would end up with url /endpointmy/endpoint insead of /endpoint/my/endpoint

@larochef could you add some test cases?

@pjfanning i'll have a look at how the delegation can be done in tests

@pjfanning I've added some tests, they helped me see some forgotten cases.

@pjfanning  did you get a chance to have a look at it ?

@larochef your change looks ok to me but you'll need a Member to review and merge

I have the same problem and would like this fix. Is it going to be merged soon and released to maven ?


Unable to validate a  json that contains a arraylist with groovy contract

I have already reported same issue(#817) and since it is closed so raising new one. I  currently developing a spring cloud project where I need to validate a JSON request. My json includes an array list, here I am facing issues while validating a particular element of the array list. My groovy contract at producer end looks like as below:
`package contracts.beer.rest
import org.springframework.cloud.contract.spec.Contract
Contract.make{

description("""
Validating A CWP MO For New Customer
""")

request{

    method 'POST'
    url    '/ValidateMO'
    body(file("cwp_mo.json"))
    bodyMatchers {
        jsonPath('$.id',byRegex("[0-9]{2}"))
        jsonPath('$.name',byRegex('[A-Za-z]+'))
        jsonPath('$.age',byRegex('[2-9][0-9]'))
        jsonPath('$.salary',byRegex('[1-9][0-9]{5}'))
        jsonPath('$.designation',byRegex('[A-Za-z]+'))
        jsonPath("\$.['addressList'][*].['zipCode']", byRegex('[0-9]{5}'))

    }
    headers {
        contentType(applicationJson())
    }
}
response{
    status 200
    body("""
 {
 "status":"OK"
 }
 """)
    headers{
        contentType(applicationJson())
    }
}
}'

and corresponding stub generated for this contract is:
{
  "id" : "3592fe24-24d3-4040-a5b1-8287654348da",
  "request" : {
    "url" : "/ValidateMO",
    "method" : "POST",
    "headers" : {
      "Content-Type" : {
        "matches" : "application/json.*"
      }
    },
    "bodyPatterns" : [ {
      "matchesJsonPath"<EMAIL_ADDRESS>== 'Park Avny')]"
    }, {
      "matchesJsonPath" : "$.['phoneNumbers'][?(@ == '987654')]"
    }, {
      "matchesJsonPath"<EMAIL_ADDRESS>== 'West chester')]"
    }, {
      "matchesJsonPath" : "$.['phoneNumbers'][?(@ == '123456')]"
    }, {
      "matchesJsonPath"<EMAIL_ADDRESS>== 'Park Avn')]"
    }, {
      "matchesJsonPath"<EMAIL_ADDRESS>== 'Male')]"
    }, {
      "matchesJsonPath"<EMAIL_ADDRESS>== 'married')]"
    }, {
      "matchesJsonPath"<EMAIL_ADDRESS>=~ /([0-9]{2})/)]"
    }, {
      "matchesJsonPath"<EMAIL_ADDRESS>=~ /([A-Za-z]+)/)]"
    }, {
      "matchesJsonPath"<EMAIL_ADDRESS>=~ /([2-9][0-9])/)]"
    }, {
      "matchesJsonPath"<EMAIL_ADDRESS>=~ /([1-9][0-9]{5})/)]"
    }, {
      "matchesJsonPath"<EMAIL_ADDRESS>=~ /([A-Za-z]+)/)]"
    }, {
      "matchesJsonPath"<EMAIL_ADDRESS>=~ /([0-9]{5})/)]"
    } ]
  },
  "response" : {
    "status" : 200,
    "body" : "{\"status\":\"OK\"}",
    "headers" : {
      "Content-Type" : "application/json"
    },
    "transformers" : [ "response-template" ]
  },
  "uuid" : "3592fe24-24d3-4040-a5b1-8287654348da"
}

Here addressList can contain multiple addresses. so at consumer end suppose I have two different addresses, if pass wrong value of zipcode for both addresses then my testcase got failed, but if I pass one zipcode as wrong and another one correct in that scenario testcase got executed successfully which is wrong behavior.
In my previous case(Case no:817), you suggested to use bodyMatcher and define an entry with index [0].
But I want to make it generic... instead of using index[0] or index[1], I want to use *, so that it fails when any of the zipcode is wrong at any index. After writing the validation part in bodyMatcher, it is still failing.
Providing consumer end testcase also:
@Test
       public void validateMO() throws Exception {
        long[] phnNum={12345,56789};
        HashMap<String,String> personalInformation=new HashMap<String,String>();
        personalInformation.put("gender","Male");
        personalInformation.put("maritialstatus","Unmarried");
        BigDecimal sal=new BigDecimal(100000);
        List<Address> addressList =new ArrayList<>();
        addressList.add(new Address("Park Avn","West chester",123451));
        addressList.add(new Address("Park Avn","West chester2",56789));
        mockMvc.perform(MockMvcRequestBuilders.post("/validate")
                .contentType(MediaType.APPLICATION_JSON)
                .content(json.write(new Employee(12,"Aman",25,sal,"Programmer",addressList,phnNum,personalInformation)).getJson()))
                .andExpect(status().isOk())
                .andExpect(content().string("THERE YOU GO"));
        //remove::end[]
    }

Can you please tell now what is wrong. Thanks in Advance.

I don't think you can do it in JSON Path. Remember however that it's a contract test. You should verify if you can communicate properly. Just fix the size to be e.g. 2 and ensure via the index entries that both json paths are matched. Then on the client side, someone will need to assert against a list of 2 and 2 only. As a client, if those tests pass then it means that the rest of the tests doesn't need to be executed against the stubs.

Thanks for your feedback.


Trivial (ocd?) test refactor

One less assertion
Uses explaining variables to clarify intent of assertion

coverage: 94.401%. remained the same
when pulling e92dbd8cde7875acf57c508682b7c94f8767f97f on jdc-violation-test-refactor
into a39126f761f133df28af0421ff1767f45c69d71c on main.


Quic Flood Attack

I am trying to initiate for quic flood attack on a host server on a virtual machine and save the CPU statistics from the host machine. I do not see any instruction on how to successfully send a quick packet. Also, is it possible to send spoofed quic packets using the repository.

Closing as dupe of #1021


[Bug]: chips take whole line

Requirements

[X] I checked the troubleshooting section in the README to verify that I have the latest Mushroom version.
[X] I did a search to see if there is a similar issue or if a pull request is open.

Current Behavior
I need to display multiple chips and some of them only when some condition is true. I tried using entity-filter which works fine. But it displays the chips on two lines instead of one.

Expected Behavior
No response
Steps To Reproduce
No response
Context
YAML state
    badges:
      - type: custom:mushroom-chips-card
        chips:
          - type: entity
            entity: sensor.realtemperature_104_sitting_room_temperature_and_humidity_sensor
          - type: entity
            entity: sensor.humidity_104_sitting_room_temperature_and_humidity_sensor

      - type: entity-filter
        entities:
          - entity: binary_sensor.reedcontact_104_door
            type: custom:mushroom-chips-card
            chips:
              - type: entity
                entity: binary_sensor.reedcontact_104_door
                use_entity_picture: false
                content_info: name
        state_filter:
          - "on"

Environment
- Browser: https://whatismybrowser.com/w/ZZL5FMZ
- HA Version:
Home Assistant 2023.7.1
Supervisor 2023.07.1
Operating System 10.3
Frontend 20230705.1 - latest
- Mushroom: v3.0.0

Anything else?
No response

Each chip card use one line. You define 2 chips card so you get 2 lines.
To solve your issue, you can use conditional chip to add display condition for some chips within the same chip line.

Thanks. I got it. FOr next generation
    badges:
      - type: custom:mushroom-chips-card
        chips:
          - type: entity
            entity: sensor.realtemperature_104_sitting_room_temperature_and_humidity_sensor
          - type: entity
            entity: sensor.humidity_104_sitting_room_temperature_and_humidity_sensor
          - type: conditional
            conditions:
              - entity: binary_sensor.reedcontact_104_door
                state: "on"
            chip:
              entity: binary_sensor.reedcontact_104_door
              type: template
              content: Dveře
              icon: mdi:door-sliding-open
              tap_action:
                action: more-info
          - type: conditional
            conditions:
              - entity: binary_sensor.reedcontact_104_balcony_door
                state: "on"
            chip:
              entity: binary_sensor.reedcontact_104_balcony_door
              type: template
              content: Balkonové dveře
              icon: mdi:door
              tap_action:
                action: more-info
          - type: conditional
            conditions:
              - entity: binary_sensor.reedcontact_104_balcony_handle
                state: "on"
              - entity: binary_sensor.reedcontact_104_balcony_door
                state: "off"
            chip:
              entity: binary_sensor.reedcontact_104_balcony_handle
              type: template
              content: Klika
              icon: mdi:hand-back-left
              tap_action:
                action: more-info
          - type: conditional
            conditions:
              - entity: binary_sensor.reedcontact_104_hs_portal
                state: "on"
            chip:
              entity: binary_sensor.reedcontact_104_hs_portal
              type: template
              content: HS
              icon: mdi:application-outline
              tap_action:
                action: more-info
          - type: conditional
            conditions:
              - entity: binary_sensor.reedcontact_104_window
                state: "on"
            chip:
              entity: binary_sensor.reedcontact_104_window
              type: template
              content: Okno
              icon: mdi:window-open-variant
              tap_action:
                action: more-info
          - type: conditional
            conditions:
              - entity: binary_sensor.reedcontact_104_window_ventilation
                state: "on"
              - entity: binary_sensor.reedcontact_104_window
                state: "off"
            chip:
              entity: binary_sensor.reedcontact_104_window_ventilation
              type: template
              content: Okno ventilace
              icon: mdi:fireplace-off
              tap_action:
                action: more-info


report wrong

hi,i use this plugin write angular2,but find a problem.please see http://plnkr.co/edit/?p=preview. This code
getHeroes() {     this._heroService.getHeroes().then(heroes => this.heroes = heroes);   }
will report a wrong.Specific information is Type '{}' is not assignable to type 'Hero[]'.But in browser angular2 can run right.It is bug?

@SallyOne You didn't share a link that others can view. However, it's unlikely that this is an Atom TypeScript issue as it's just a TypeScript problem you're facing. I can't comment without seeing the code, but most likely the types aren't assignable.

sorry.this is link:  http://plnkr.co/edit/39QSqX24mV6gVqGxVUSq?p=preview

I can't replicate your issue, it's working fine for me. Please make sure the typings you use for Promise are correct.

@SallyOne , it's not atom-typescript issue. As i understand, problem with this._heroService.getHeroes()


No versions avaliable for tesla-screenplay-playwright🐞

What is the issue?
Al instalar la librería tesla segun el comando descrito en la documentación oficial me muestra un erro que no hay versiones disponibles
Log output
PS C:\Users\CCXC Administrador\Documents\FOLDER 2024\FOLDER 2023\PORVENIR\Automatización\Auth Tesla> npm install --save-dev testla-screenplay-playwright
npm ERR! code ENOVERSIONS
npm ERR! No versions available for testla-screenplay-playwright
npm ERR!      C:\Users\CCXC Administrador\AppData\Local\npm-cache_logs\2024-03-19T17_38_29_861Z-debug-0.log
Steps to reproduce
No response
Testla version
18.16.0
Browser
Chromium

Thanks for reporting the issue. Please keep in mind to report issues in english language.
Also we would need a little more details to investigate that.
The library is currently in version 1.5.0, a version 18.16.0 does not exist - we assume that your node version is 18.16.0?
Can you please forward us the the correct version information of @testla/screenplay, @testla/screenplay-playwright and node from your package.json.

I think here the documentation might be wrong. it should be  npm install --save-dev @testla/screenplay-playwright not npm install --save-dev testla-screenplay-playwright

@achiong22 you are absolutely right. That is definitely a documentation mistake. We'll fix that!

In the meantime, gicuadrado please be aware of you have to install testla like mentioned by achiong22.
npm install --save-dev @testla/screenplay-playwright


Persisting Dark Mode Preference Across Visits

Hi @PriyaGhosal ,
I noticed that when users reload the site, their last dark mode preference isn't saved, and the site defaults back to light mode. I'd like to fix this so that when users revisit the site, their last selected theme is saved and applied automatically, preventing the need to change it every time.
I plan to implement this using either localStorage or cookies. Kindly assign this issue to me.

Hey @PriyaGhosal , if the issue is resolved, please close it, mark it as done, and add the necessary labels so it can be included in my GSSoC contribution. Thanks!

Hey @PriyaGhosal , if the issue is resolved, please close it, mark it as done, and add the necessary labels so it can be included in my GSSoC contribution. Thanks!


Use implicit mode to add ngInject annotations

Create a mode for developers to "retrofit" an existing code-base to use explicit annotations.
This mode should use our "implicit" matching to find all of the locations where we think that annotations belong.  Instead of adding the $inject annotations in these places, we should then add the 'ngInject' directives that would be necessary to use this plugin in "explicit-only" mode.
In practice, this transformation would be run against an entire codebase (with no other transformations), manually reviewed by a human, and the results would then be checked into source control.
Ideally, this should also convert /* @ngInject */ to 'ngInject'; wherever possible.

As a follow-up, also add a "verify" mode that scans existing code for missing 'ngInject' directives, and prints a warning or error.
Along with strictDi, this brings us most of the benefits of disabling "implicit" matching, while also providing a layer of safety to warn developers when they forget to add 'ngInject' to their code.

What do you we think would be needed to make this happen? I'd like to see this land.

This is a really good proposal. I currently have a hybrid application with AngularJs + Angular7, and ngAnnotate is one of the things which block me to use AngularCLI.
I would be really happy if I can find all places where implicit annotation applied, and replace it to manual annotation, and remove ngAnnotate at all.
Your proposal can help with that because I can manually one by one check all places with 'ngInject', and fix them and then push to GIT.
By the way, if it's hard to implement, may be you can point me where I can put a console.log to see all files with implicit annotations?

The project isn't really set up to do codemods at the moment (and I'd kind of discourage folks from doing this, because it provides little safety in the event of a forgotten annotation), but it's certainly something I can look at once I have time to dust this off and get the other bugs fixed.

I'm actually want to run ngAnnotate in a kind of "dry run" mode, just to point me places where I don't have explicit annotations.
I have explicit markers via 'ngInject' (which I can find just by string in IDE) and also I have places where code annotated implictly by "euristic" detection. I want to find all this 'euristic' places, mark them explicitly and then one by one convert to manual annotations / decorators / port to new angular / whatever.
For me even list in console with filename and line numbers will be enough. So the question where can I hack this ngAnnotate and add console.log statement to display this files.

In ng-annotate-main.js, look for anything that calls insertBefore or insertAfter
I'm planning to eventually add something like a "dry-run" mode with the explicitOnly mode -- we'll only annotate functions that you've explicitly marked for injection, but warn or fail with an error if it looks like you forgot one.
(In my experience, it isn't really safe to run this once and forget it if you're actively developing a codebase -- the manual injection syntax is ridiculously error-prone when humans are writing it)

Finally I got that. In my case almost 99% occurrences was in block.unshiftContainer("body", [expr]); in addInjectArrayBeforePath() function.
I also had to add file name information in visitor to context, to be able to console.log it further.
      Program: {
        enter(path, file) {
          file.opts.explicitOnly = file.opts.explicitOnly || false;

          ctx.suspects = [];
          ctx.blocked = [];
          ctx.fragments = [];
+         ctx.filename = file.filename;
          ctx.srcForRange = function(node) {
            return file.file.code.slice(node.start, node.end);
          };
        },
        exit() {
          judgeSuspects(ctx);
        }
      }


Fix duplicate images bug

Modify 0006_review_test_data.py to prevent the creation
of a new image every time the function test_model_review_list test
in file tests.py is executed
Add new_test_image.jpg to prevent the creation of the image
on migrate 0006_review_test_data.py
fix: #92

@VeraTs @ellaml @LeonRabinovich As GitHub runs the tests only once, to be on the safe side, I'll appreciate it if one of you can double-check me on his local repo

The issue is mostly fixed, however on the first time you run pipenv run pytest it will create a new image which is not in our repository(and also not in this PR) -  "new_test_image.jpg"
Running the command again will not create more test images other than the first one.
Just to make myself extra clear:
Currently under ..\ClassRater\homepage\media\images we have 2 images :

histogram.png
test_image.jpg

When running pytest with the updated migration file from this PR, a 3rd image is created - new_test_image.jpg

@LeonRabinovich If you look closely, the new_test_image.jpg was already added to this PR

@LeonRabinovich If you look closely, the new_test_image.jpg was already added to this PR

My bad :)


[Bug]

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the issue
we are currently using this zendesk package to manipulate the data that lands in our warehouse through DBT. The issue is that some of the data are missing specifically all business_minutes calculations in ticket metrics
The int tables they create in the same dataset are also not created unlike most of the other fields.

Relevant error log or model output
No response
Expected behavior
All the fields as listed in the package should be created/computed in the zendesk support ticket metrics
dbt Project configurations
vars:
zendesk_schema: landing_zendesk
using_schedules:            False
using_domain_names:         False
using_organization_tags:    False
models:
zendesk:
+schema: staging_zendesk
zendesk_source:
+schema: staging_zendesk
Package versions
packages:

package: fivetran/zendesk
version: 0.11.0

What database are you using dbt with?
bigquery
dbt Version
version: '1.3.0'
Additional Context
No response
Are you willing to open a PR to help address this issue?

[ ] Yes.
[X] Yes, but I will need assistance and will schedule time during our office hours for guidance
[ ] No.

Hi @elijamuel! So to get the business_minutes fields, you'll need to set this variable to True. This is because business minutes relies on schedules being used.
using_schedules: False
It is outlined in the README, but I will make a note to make the connection between schedules and business minutes more clear in our next update to the package.

Hi @fivetran-reneeli, thank you for the prompt response.
We have turned it on as advised but the business times are still not computed.
We checked the Zendesk support platform and have seen that one schedule (Hongkong Hours) is enabled in it (since October 22, 2023) but still had no extracted data (schedules and scheduly_holiday, thru Fivetran) and also no computed business times in it.
What should we do to enable it?

Hi @fivetran-joemarkiewicz. repoening this because according to Fivetran Support, this issue I raised above falls under your scope. We have created the schedule_holiday and schedule tables but its still computing all business minutes as zero. Hope you can help us with this.

Although, right now its computing incorrectly but that will be resolved with Fivetran support.

cc. @IamElaya

Hi @elijamuel thanks for responding back to this issue. Would you be able to confirm if you have any data in the schedule_holiday or schedule tables? Similarly, do you have any data in the ticket_schedule table synced by the Fivetran connector?

Hi @fivetran-joemarkiewicz,
Here's the tables you requested:

schedule_holiday contains 2 test holiday values encompassing the start date of all the tickets we have (May 1, 2023) and a recent one on November 1-3, 2023
schedule contains below attached picture as values:

ticket_schedule - apparently it was not resynced along the schedule and schedule_holiday tables. Will update if this resolves the issue

@fivetran-joemarkiewicz, resync-ing the ticket_schedule created no table and most likely contains no data.
Saw in the documentation that setting a schedule for tickets is only applicable for enterprise plans. Does this mean that lower billing plans cant have this functionality computed by the package?

Hi @elijamuel thanks for sharing more details. I believe you should still be able to gain access to the schedule feature in Zendesk regardless of having the Enterprise plan. Albeit my understanding of the various plans in Zendesk is slim to none, but I imagine if you have been able to create schedules, then you should have access to use those. Before going too far, are you able to view your business metrics in the Zendesk UI reporting tool? I would be curious if for a given ticket are you able to see the comparison of calendar vs business minutes?
Additionally, from my understanding the ticket_schedule table's purpose is to associate your various tickets with the corresponding schedule. However, if you only have one schedule, then that can be assumed to be the default schedule and our downstream model should make this connection. Would you be able to query an given ticket_id from the int_zendesk__ticket_schedules table in your warehouse and let me know what the results are? I would imagine you should see the Hong Kong timezone being populated for all the tickets.

Yes, the business metrics are accessible in Zendesk UI reporting tool.
As for the int_zendesk__ticket_schedules, there are available data in each ticket shown below

Would this mean that the business metrics in ticket_metrics should also be calculated correctly?

Thanks for sharing @elijamuel. Given this, I would assume the business metrics should be calculating properly. Would you be able to share one ticket as an example of the Zendesk UI properly recording the business minutes metrics and then the result for that same ticket from the end model of this Zendesk package? This will help me get a better picture of what may be going on.
If I am not able to identify what is going on or a solution following that, I may request we meet in over a video call to investigate more thoroughly. Thanks!

@fivetran-joemarkiewicz
Right now, all business metrics computed are all zeros which means almost all tickets we have are computed incorrectly.
Here are sample ids and their corresponding business metrics computed by the UI and by the package:

@elijamuel thanks so much for sharing and I see the issue you are encountering. I also just noticed you are using the v0.11.0 version of the Zendesk package. We actually identified some issues with that version a while back and have released some updates that should address the business metrics issues. Would you be able to upgrade to the latest version (v0.12.0) and let me know if you still see this issue?
Please note that when you upgrade I recommend running a dbt clean && dbt deps before rerunning the package in order to capture all the latest code updates.
If that doesn't seem to do the trick, you can reach out to my team via<EMAIL_ADDRESS>to communicate with us and we can schedule some time with you to meet over a video call to troubleshoot further. Let me know if upgrading the package helps!

@fivetran-joemarkiewicz, just to verify, can you send the step by step to upgrade to the latest versions? Should I have it deployed to prod before it will compute the correct values or will running it on a branch show the correct ones?

Hi @elijamuel I am not entirely sure what your environment setup is, but you could create a branch of your own dbt project and update the packages.yml to reference the latest Zendesk version. You can then run this and inspect the results to see if this looks good before pushing your changes to prod. As long as the underlying data is the same between prod and dev, you should be able to determine if the correct results are showing.
Let me know if you have any other questions. Thanks!

Hi @fivetran-joemarkiewicz, I have updated the zendesk package on a separate branch to v0.12.0 but the issue with the business metrics persists. I have sent an email to<EMAIL_ADDRESS>as well to see if we can troubleshoot this over video.

Hi @elijamuel thanks again for meeting with myself and @fivetran-reneeli today to troubleshoot the bug in more detail.
From our investigation, we found that the root of the issue seems to be that the Hong Kong schedule was being corrupted within the int_zendesk__schedule_spine model since the Hong Kong daylight savings time ended in 1979, and the package does not take into consideration that a timezone would have daylight savings for a period, and then stop. Therefore, the schedule spine stops for Hong Kong in 1979, and therefore does not report properly for business metrics for this timezone.
This is something we should be able to recreate on our end with out test data and will plan to have a branch that you may test in more detail in the coming week. Thanks again for your collaboration and helping us understand this scenario and how we should address it in the package. Be sure to follow here for more updates as we share them.

Hi @fivetran-joemarkiewicz, thank you to you and @fivetran-reneeli for helping troubleshoot, and for the summary of the issue discovered yesterday.
Sounds good, looking forward to your next update!

Hi @IamElaya I believe I may have found a solution! Would you be able to test the following version of the package in your packages.yml and let us know if this addresses the issues you are seeing?
packages:
  - git: https://github.com/fivetran/dbt_zendesk.git
    revision: bugfix/daylight-end-catch
    warn-unpinned: false

Please note that once you update your packages.yml to reference the git branch as opposed to the dbt hub version of the Zendesk package you will need to also run dbt clean and then dbt deps to capture the updates.
Let us know if this resolves your issue. If so, we can work to incorporate these changes into the next release of the package. Thanks 😄

Hey Joe!
Thank you for this.
So I would add those lines of code in the root packages.yml like this?

@IamElaya that is very close to it! You would just want to remove (or comment out) the official zendesk version you have pinned. So it would be the following:
packages:
  - package: fivetrtan/quickbooks
    version: 0.11.0
  - package: fivetran/stripe
    version: 0.10.1
  - package: dbt-labs/dbt_utils
    version: 1.1.1
  # - package: fivetran/zendesk
  #   version: 0.12.0
  - git: https://github.com/fivetran/dbt_zendesk.git
    revision: bugfix/daylight-end-catch
    warn-unpinned: false

That shouldn't be an issue when you run dbt deps. That is just to ensure you don't get a warning when you run dbt deps. If you want, you can remove it but it shouldn't be an issue.

Hi @IamElaya I just wanted to check in and see if you were able to test the changes from within the branch?

Heya, I tried adding it today and the business minutes are still showing up as zero.

Hmm that is peculiar 🤔 I was able to recreate the issue locally and saw this branch addressed the issue. Did you run dbt deps after updating the packages.yml?
If you did, then I think it may make the most sense to meet again over a google hangout to discover why this adjustment hasn't solved the issue you are seeing.

Hmm yes let's do that over google hangout again! I may be missing some other step.

Thanks again for meeting with me today @IamElaya! Following our call from earlier it seems the fix proposed in the above branch does in fact correct the business metrics for your tickets in the Hong Kong timezone!
I will move forward with adding polishing the PR and work to include these changes in the next minor (v0.X.0) release of the dbt_zendesk package.
Thanks again for all your help!

No worries @fivetran-joemarkiewicz, and thank you for getting the fix for us. Appreciate it! Looking forward to the official fix!

@IamElaya thanks again for all the help with understanding and addressing this issue. The fix for this issue is now live within the v0.13.1 release of the dbt_zendesk package! Thanks again and please let us know if you see any other irregularities with the package.
As this issue has been addressed in the latest release, I will close out this ticket. Thanks again 😄


[BIOMAGE-1938] - Create trajectory analysis plot

Description

Create trajectory analysis plot

The plot will not yet be shown to the user because it does not have the capability to show the pseudotime yet. Therefore, before merging, the plot will be disabled/commented out.
https://ui-agi-ui7-api6-worker4.scp-staging.biomage.net/
Comments in previous PR: https://github.com/hms-dbmi-cellenics/ui/pull/777
Details
URL to issue
https://biomage.atlassian.net/browse/BIOMAGE-1938
Link to staging deployment URL (or set N/A)
N/A
Links to any PRs or resources related to this PR

UI - https://github.com/biomage-org/ui/pull/7
API - https://github.com/biomage-org/api/pull/6
Worker - https://github.com/biomage-org/worker/pull/4

Integration test branch
master
Merge checklist
Your changes will be ready for merging after all of the steps below have been completed.
Code updates
Have best practices and ongoing refactors being observed in this PR

[ ] Migrated any selector / reducer used to the new format.

Manual/unit testing

[ ] Tested changes using InfraMock locally or no tests required for change, e.g. Kubernetes chart updates.
[ ] Validated that current unit tests for code work as expected and are sufficient for code coverage or no unit tests required for change, e.g. documentation update.
[ ] Unit tests written or no unit tests required for change, e.g. documentation update.

Integration testing
You must check the box below to run integration tests on the latest commit on your PR branch.
Integration tests have to pass before the PR can be merged. Without checking the box, your PR
will not pass the required status checks for merging.

[ ] Started end-to-end tests on the latest commit.

Documentation updates

[ ] Relevant Github READMEs updated or no GitHub README updates required.
[ ] Relevant Wiki pages created/updated or no Wiki updates required.

Optional

[ ] Staging environment is unstaged before merging.
[ ] Photo of a cute animal attached to this PR.

As Sara mentioned, trajectory analysis is not available using tSNE, because of monocle3:order_cells. The trajectory plot should be disabled with a message that trajectory is only available for UMAP. This should also be included in the tooltip in Configure embedding.
Changing any of the embedding controls does not cause the trajectory plot to rerender: so the plot becomes a combination of old trajectory nodes and new embedding data.

This is the case for both metric and minimum distance. This doesn't happen if they are changed before the trajectory plot is first rendered, then the plot renders correctly.
The panning and zooming starts to stutter for larger data sets, which should probably be tested for big datasets to see if alternatives for dynamic panning and zooming (e.g. setting the axis ranges manually) is needed.

When I zoom in on the trajectory plot, the 'reset' button should become enabled so that I can easily reset the zoom level. Right now it's hard to get back to the original plot!
Changing any of the plot styling options when zoomed in resets the plot. This is a bit annoying.
Generally about the zoom - I don't think it's intuitive to zoom in/out in the Plots and Tables plots. This function is not available (to my knowledge) in any of the other plots in this module. For consistency, would it not be better to implement @jszpila314 's new way of controlling axes ranges?
I reproduced the issue Jakub already mentioned about selecting a t-SNE plot - the trajectory plot loads a weird plot with an error message.

@jszpila314 Fixed. The plot now takes into account the embedding settings before recalculating.

Replying to @vickymorrison

When I zoom in on the trajectory plot, the 'reset' button should become enabled so that I can easily reset the zoom level. Right now it's hard to get back to the original plot!

Fixed. Now reset button appears when plot is zoomed / panned. Clicking reset resets the zoom and pan.

Changing any of the plot styling options when zoomed in resets the plot. This is a bit annoying.

Fixed. Now changing plot styling does not reset the zoom.

Generally about the zoom - I don't think it's intuitive to zoom in/out in the Plots and Tables plots. This function is not available (to my knowledge) in any of the other plots in this module. For consistency, would it not be better to implement @jszpila314 's new way of controlling axes ranges?

Zooming and panning is not meant to prepare the plot for exact view. It is to help select nodes in the cases where there are many nodes placed close together. I think zooming and panning is much quicker/more intuitive rather than having to estimate the ranges via the controls.
That said, there still is a use case for the new controls Jakub is working on. I think the controls are very useful when you want to get the exact ranges when preparing the plot for download. Hence, that control will be implemented, in the ticket to set up the plot controls: https://biomage.atlassian.net/browse/BIOMAGE-1940

I reproduced the issue Jakub already mentioned about selecting a t-SNE plot - the trajectory plot loads a weird plot with an error message.

Fixed. Using tSNE in embedding now shows a UMAP embedding for the trajectory plot.


Border/Color on notification

Screenshots

There is no borders on each notifications which makes it hard understand the notification area (size), until we hover on each of them.
Solutions

Use different background color on each of them.

Well using different background on each notification seems an overkill. However a different background and a border would suffice.

Yeah, I agree with @lightify97 about having a border here. Possibly background but I try to keep from having too much going on unnecessarily. Thanks, I'll look into this!

Added a background and a border. Thanks! Please upgrade and restart!


Can someone help me?

I need to get the path of a.exe file and make all traffic generated by it sent to my VPN (proxy server), but I have no idea how to do that. Can anyone help me?
I don't know if this helps, but I have a c# application that connects via SSH similar to Putty on my server. I just need to monitor the files .exe that I added to a list so they send packages via my proxy server.
I can do it using a software called Proxifier.
Thanks alot.

I have no experience in doing it.. but, since it's networking-related, maybe you can try to hook the functions to Winsock (ws2_32.dll)?
This link may be helpful:
https://msdn.microsoft.com/en-us/library/windows/desktop/ms741394(v=vs.85).aspx

I'm not familiar with the easy hook nor with winsock (ws2_32.dll), I tried to find some article related to your tip, but did not have much success.


Update Travis secure variable for API key integration test

Merging since this change is trivial, autogenerated, similar to previously approved, and needed for test cycle.

Changes Unknown when pulling 12f98c492a91cef7d03c5d4ee1af1c4bb2564777 on tcoffee-google:api_key_update into ** on GoogleCloudPlatform:master**.


join fails, but pandas merge is successful

Hi,
I am having challenges joining data with blaze. Any tips would be greatly appreciated as I struggle to see from the traceback what is going wrong.
Here is the snippet.
from blaze import Data, compute, join, odo, by
import numpy as np
import pandas as pd

d = Data('entities.csv.gz', "var * {document_id: string, variable: string, value: string}")

# Filter data on documents for the respective mentions only
docs = odo(d[(d.variable=='tweet_type') & (d.value =='mention')]['document_id'].distinct(), list)
targets = d[(d.document_id.isin(docs)) & (d.variable == 'mention')][['document_id', 'value']].relabel(value='target')
sources = d[(d.document_id.isin(docs)) & (d.variable == 'username')][['document_id', 'value']].relabel(value='source')
m = join(targets, sources, 'document_id', how='inner')

compute(m)

NotImplementedError                       Traceback (most recent call last)
<ipython-input-39-f3368fbd0c0d> in <module>()
----> 1 compute(m)

..\site-packages\multipledispatch\dispatcher.pyc in __call__(self, *args, **kwargs)
    161             self._cache[types] = func
    162         try:
--> 163             return func(*args, **kwargs)
    164
    165         except MDNotImplementedError:

..\site-packages\blaze\interactive.pyc in compute(expr, **kwargs)
    176         raise ValueError("No data resources found")
    177     else:
--> 178         return compute(expr, resources, **kwargs)
    179
    180

..\site-packages\multipledispatch\dispatcher.pyc in __call__(self, *args, **kwargs)
    161             self._cache[types] = func
    162         try:
--> 163             return func(*args, **kwargs)
    164
    165         except MDNotImplementedError:

..\site-packages\blaze\compute\core.pyc in compute(expr, d, **kwargs)
    452         d3 = dict([(e, pre_compute_(expr2, dat, **kwargs))
    453                         for e, dat in d2.items()
--> 454                         if e in expr2])
    455     else:
    456         d3 = d2

..\site-packages\multipledispatch\dispatcher.pyc in __call__(self, *args, **kwargs)
    161             self._cache[types] = func
    162         try:
--> 163             return func(*args, **kwargs)
    164
    165         except MDNotImplementedError:

..\site-packages\blaze\compute\csv.pyc in pre_compute(expr, data, comfortable_memory, chunksize, **kwargs)
     40
     41     # Insert projection into read_csv
---> 42     oexpr = optimize(expr, data)
     43     leaf = oexpr._leaves()[0]
     44     pth = list(path(oexpr, leaf))

..\site-packages\multipledispatch\dispatcher.pyc in __call__(self, *args, **kwargs)
    161             self._cache[types] = func
    162         try:
--> 163             return func(*args, **kwargs)
    164
    165         except MDNotImplementedError:

..\site-packages\blaze\compute\csv.pyc in optimize(expr, _)
     24 @dispatch(Expr, CSV)
     25 def optimize(expr, _):
---> 26     return lean_projection(expr)  # This is handled in pre_compute
     27
     28

..\site-packages\blaze\expr\optimize.pyc in lean_projection(expr)
     22     """
     23     fields = expr.fields
---> 24     return _lean(expr, fields=fields)[0]
     25
     26

..\site-packages\multipledispatch\dispatcher.pyc in __call__(self, *args, **kwargs)
    161             self._cache[types] = func
    162         try:
--> 163             return func(*args, **kwargs)
    164
    165         except MDNotImplementedError:

..\site-packages\blaze\expr\optimize.pyc in _lean(expr, fields)
    229         The fields that this expression requires to execute
    230     """
--> 231     raise NotImplementedError()

NotImplementedError:

When I use pandas to merge it seems to work
t = compute(targets)
s = compute(sources)
pd.merge(t,s,on='document_id', how='inner')

document_id	target	source
0	id_1	user_12321	user_17
1	id_2	user_323	user_12325
2	id_3	user_5	user_122525

Great project so far guys, thanks for all the efforts.

can you post some sample data that causes this?

This should be fixed on master, please let me know if this is still broken after updating and I will look into this more.

@llllllllll Thank you very much for looking into this! I will give this a test this week and let you know whether I still encounter this problem.


[docs] Horizontal scrollbar visible on main content

Duplicates

[X] I have searched the existing issues

Related page
https://mui.com/material-ui/api/snackbar/
Kind of issue
Other
Issue description
The main content of the page is not fully visible, and needs to be horizontally scrolled (there's an horizontal scrollbar visible at the bottom of the first table).
Context 🔦
I'm using Chrome and Firefox on Ubuntu 20.04, and my resolution is 2560 x 1440.

Duplicate of #34085


Bug in description Product lifecycle dashboard

Tile Top five products nearing end-of-life shows products that are nearing end-of-life in next eighteen months not six as it is currently written

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

ID: 8e5b5231-0498-f10a-fcd7-58230e43cca5
Version Independent ID: aa3f037a-f646-e683-867e-b8e16f27966d
Content: Product lifecycle dashboard - Configuration Manager
Content Source: sccm/core/clients/manage/asset-intelligence/product-lifecycle-dashboard.md
Product: configuration-manager
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

Thanks @AdamZajacT for catching that! If you would like to submit an edit for this, I'll happily review it. For more information on how to contribute, see How to contribute.

CLosing this issue as it is resolved by the linked pull request.


Make it work with "dotnet new"

More info: https://blogs.msdn.microsoft.com/dotnet/2017/02/07/announcing-net-core-tools-updates-in-vs-2017-rc/
Tooling: https://www.microsoft.com/net/core#windowscmd
We'll be probably able to get rid of the https://github.com/Kentico/cloud-boilerplate-net/blob/master/RenameProject.ps1 file.
Wiki: https://github.com/dotnet/templating/wiki/"Runnable-Project"-Templates
Existing templates: https://github.com/dotnet/templating/wiki/Available-templates-for-dotnet-new
Example installation: https://blogs.msdn.microsoft.com/webdev/2017/02/14/building-single-page-applications-on-asp-net-core-with-javascriptservices/

we want to get to this list: https://github.com/dotnet/templating/wiki/Available-templates-for-dotnet-new


There are very very very lots of mistakes in your code, i'm very sorry about that

If you try to fix one bug, it appears another one. There are lots of mistakes in your project.

for example
File "build/bdist.linux-x86_64/egg/gluoncv/model_zoo/rpn/anchor.py", line 96, in hybrid_forward
AttributeError: 'module' object has no attribute 'slice_like'

@sheldon606  try to compile the latest mxnet 1.3.x version. Some of those features are not supported in the earlier mxnet version.   Lots of faults are because of the version mismatch.
When you raise a question, please mention what command you use.  It would be helpful to reproduce the error if it exists, and identify the bug on our own PC.


Add Mozilla-related branding to the site?

I was thinking that it might be nice to have something indicating that this was funded by and the GH repo is owned by Mozilla. e.g., something like the footer on https://www.html5rocks.com/en/ that indicates it was Google-sponsored/supported?
I'm not sure if this is in line with the direction you were going with this or not; feel free to clarify for me :-)

The intent was that WebVR Rocks would be a unbiased source for WebVR information.   Because of this, we wanted to keep Mozilla branding off of it.   But I agree that we should be transparent and honest that this is a Mozilla project.


Wallet unlock time

Close and No have been changed back to English, did you mean to?

I dont think so, I go with transifex , let me check transifex.


Support basic authentication

When running private docker registries, basic authentication is enabled quite a lot of the time. Currently only Bearer authentication is supported.

Follow up in https://github.com/jgsqware/clairctl/issues/15


Add support for building images from ostree containers and the Fedora CoreOS qcow2 image type

This PR adds support for the org.osbuild.ostree.deploy.container stage and a new image type that uses it called coreos-qcow2 for Fedora.
I tested the new image type locally with the container we added to the gitlab registry for testing the sample manifest in osbuild.

Some notes and open questions:

I use the same pipeline as the regular ostree deployment with a condition on the type of source.  I think it wasn't worth making a whole separate pipeline for this, but we can revisit this decision if the two start diverging.
The new image type is added to Fedora which means that if this is merged and updated in osbuild-composer it will be available to build on-prem.  If we want this, we'll have to figure out the command line options for specifying a container source and potentially a target-imgref.
When specifying a container source to deploy, you must specify the ref (otherwise the default will be assumed).  This requirement also exists for ostree commits from a repo, but I wonder: Is there a way to determine the container's ostree ref without downloading it?

You can generate the manifest for this image with:
go run ./cmd/gen-manifests --distros fedora-38 --arches x86_64 --images coreos-qcow2 --output .

or build it directly with
go build -o bin/build ./cmd/build
sudo ./bin/build --distro fedora-38 --image coreos-qcow2 --output builds --config ./test/configs/ostree-container-pull.json

You can also add a user to the config file which will create a user after the deployment before finalising the image:
{
  "name": "ostree-container-pull",
  "ostree": {
    "ref": "ostree/1/1/0",
    "container": "registry.gitlab.com/redhat/services/products/image-builder/ci/images/fedora-coreos:stable",
    "tls-verify": true
  },
  "blueprint": {
    "customizations": {
      "user": [
        {
          "name": "<login>",
          "password": "<password>",
          "key": "<pubkey>"
        }
      ]
    }
  }
}

Partially addressed comments (and rebased on main).  Will resume addressing comments later.

Moved most of the commits to #272.  I'll rebase this PR so that it only adds the coreos image once that's merged.

@achilleas-k should be able to rebase now as #272 has landed \o/

Will rebase and fix conflicts then start addressing the remaining comments.


Backport #41213: gitAndTools.svn-all-fast-export: 1.0.11 -> 1.0.12

patch for https://github.com/svn-all-fast-export/svn2git/pull/40 is
contained in this release, so we can drop it here.
Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Why does this need backporting?

I doesn't necessarily needs to.
I just thought it was cleaner to bump to a newer version.
But considering the other fixes regarding svn:global-ignores, ignoring of subpaths and timeouts, the bump might be a good idea anyways:
3ad67d2 src.pro: add target.path and INSTALLS
0a71ed4 src.pro: only use git to get version if VERSION is not set before
6d0b8a6 src/src.pro: allow overriding SVN_INCLUDE, APR_INCLUDE
024b6b5 src/repository: remove absolute path to cat binary
496fd52 Make git fast-import timeout configurable (#39)
f00d5a5 Do not output the generated .gitignore contents in dry-run mode or it ends up in the log
5e1038a Make sure the fast import process is started before writing to it, otherwise Qt 5 will warn about the not running process
51dab7f Fix inconsistent whitespacing
d9faf96 Do not set the working directory of the fast import processes in dry-run or create-dump mode, as the directory will not exist there and Qt 5 does not quite like that
f3cc079 Do not try to recursively dump a file
e910002 Also consider svn:global-ignores when generating .gitignore files and fix wrong patterns not correctly migrated from svn:ignore (#34)
61ebde7 Do not warn about svn:mergeinfo, it is usually taken care about already by creating merge commits or cherry-pick commits
7085d41 Fix a type in debug output
cf94ee3 Fixed ignoring of subpaths inside a repository (implemented rule matching for recursiveDumpDir) (#33)
efba961 Add LICENSE file containing GPLv3 to the source tree. (#32)
6b1ed63 Fixed package install command for Ubuntu into README file. (#31)
c7864da Always mark repositories case-sensitive (#21)
abc0951 Add package install command for Ubuntu to README.md (#25)

Fine.


tests: lwm2m_rd_client: Added more fff tests

If the rdclient is suspended or deregistered any network errors caused by lwm2m_engine and forwarded by socket_fault handler should not change the rd client state.

Rebased and fixed the tests submitted originally in https://github.com/zephyrproject-rtos/zephyr/pull/54134 by @chmielewskiandreas


Update README.md to include holiday footnote asterisk

See https://github.com/CivicActions/guidebook/issues/1353

📚 Documentation preview 📚: https://civicactions-handbook--1354.org.readthedocs.build/en/1354/

The 2 links found in the pipeline tests to reach a 403 is not correct. I tested manually and both those links work
FILE: ./common-practices-tools/security/README.md
[✖] http://askubuntu.com/questions/57572/how-to-delete-files-in-secure-manner → Status: 403

ERROR: 1 dead links found!
[✖] http://askubuntu.com/questions/57572/how-to-delete-files-in-secure-manner → Status: 403

Yup I fixed that in #1356


FireFox Plugin Failed Review

The plugin was disabled in the Firefox marketplace for the following reason:
This add-on didn't pass review because of the following problems:

1) This extension uses a registered trademark of Mozilla in its name or icon and is unallowable under Mozilla’s Trademark Policy (https://www.mozilla.org/foundation/trademarks/policy/).
For Mozilla trademarks, the usage of “Mozilla” and “Firefox” are generally not allowed in the add-on name, except in the following format: “ADD-ON NAME for Firefox.”
icon32.png
about-logo.png
firefox-wordmark.svg

Will close this issue once the plugin is available again in the markeplace.

I have submitted v14 to be reviewed, now we play the waiting game.

Well, that did not take very long, plugin is now available again in the plugin marketplace :tada:


ISPCTexComp

There is a Fedora specific bug when attempting to build this package from source on Fedora: https://github.com/GameTechDev/ISPCTextureCompressor/issues/37
After patching this, there is a linker error which I have not solved yet that only occurs when building with rpmbuild:
/usr/bin/ld: /usr/lib/gcc/x86_64-redhat-linux/13/../../../../lib64/Scrt1.o: in function `_start':
(.text+0x1b): undefined reference to `main'
collect2: error: ld returned 1 exit status


Proxy authentication with HTTP_PROXY_AUTH?

Hi, thanks for the great docker images. I am hoping to get some help with a proxy problem, or some direction. Here is the message I am getting.:
UPDATE/EDIT: It looks like this is reproducible only if I have a bang (!) in my password. I tried encoded (%21) and just the character (!) and both fail out with the same error below when in HTTP_PROXY.

fetch http://dl-cdn.alpinelinux.org/alpine/v3.4/main/x86_64/APKINDEX.tar.gz
WARNING: Ignoring http://dl-cdn.alpinelinux.org/alpine/v3.4/main/x86_64/APKINDEX.tar.gz: Permission denied
fetch http://dl-cdn.alpinelinux.org/alpine/v3.4/community/x86_64/APKINDEX.tar.gz
WARNING: Ignoring http://dl-cdn.alpinelinux.org/alpine/v3.4/community/x86_64/APKINDEX.tar.gz: Permission denied

This seems to have started this week on some docker containers we use for PHP development (based on alpine). We are behind a proxy that requires authentication. We have been using the http_proxy environment variables with our user name and password inline. This is set when we build the images (docker build --build-arg HTTPS_PROXY=$$HTTP_PROXY). i.e.:
HTTP_PROXY=http://user:pass@proxyserver:port
HTTPS_PROXY=http://user:pass@proxyserver:port
http_proxy=http://user:pass@proxyserver:port
https_proxy=http://user:pass@proxyserver:port
This has worked for the last couple months without issues. However, now it seems we need to use HTTP_PROXY_AUTH ? I noticed this in : https://github.com/gliderlabs/docker-alpine/issues/191#issuecomment-246944011
Here is an excerpt of our Dockerfile where it is failing (specifically on docker-php-ext-install xdebug as it needs to apk add).:

# Xdebug.
RUN git clone https://github.com/xdebug/xdebug.git /tmp/xdebug \
    && mkdir -p /usr/src/php/ext \
    && mv /tmp/xdebug /usr/src/php/ext/ \
    && docker-php-ext-install xdebug

To work around this I changed the code to:

# Xdebug.
RUN git clone https://github.com/xdebug/xdebug.git /tmp/xdebug
RUN mkdir -p /usr/src/php/ext
RUN mv /tmp/xdebug /usr/src/php/ext/
ENV HTTP_PROXY=http://proxy:port
ENV HTTP_PROXY_AUTH=basic:*:user:pass
RUN docker-php-ext-install xdebug

If I do this for https the clone fails with message:

Cloning into '/tmp/xdebug'...
fatal: unable to access 'https://github.com/xdebug/xdebug.git/': Received HTTP code 407 from proxy after CONNECT
ERROR: Service 'web' failed to build: The command '/bin/sh -c git clone https://github.com/xdebug/xdebug.git /tmp/xdebug     && mkdir -p /usr/src/php/ext     && mv /tmp/xdebug /usr/src/php/ext/     && docker-php-ext-install xdebug' returned a non-zero code: 128

In troubleshooting I was also able to use curl and wget without setting HTTP_PROXY_AUTH...
Looking at the man for libfetch I think this maybe an issue there? Maybe this was updated recently in alpine?

HTTP_PROXY_AUTH
    Specifies authorization parameters for the HTTP proxy in the sameformat as the HTTP_AUTHvariable.
This variable is used if and only if connected to an HTTP proxy, andis ignored if a user and/or a password were specified in the proxyURL.

http://rpm.pbone.net/index.php3/stat/45/idpl/33638817/numer/3/nazwa/fetch

Have you tried quoting the variables?
ENV HTTP_PROXY_AUTH="basic:*:user:pass"

@andyshinn , I tried quoting the value in http_proxy and HTTP_PROXY and got varied results. It looks like the problem is limited to a password that contains a bang (!) followed by one or more numbers (I don't have an account with alpha after the bang to verify this). I have another account that has a bang as the last character and don't see the problem, nor do I see the problem if the password is alpha numeric.
Here is the "modified" code I was running to troubleshoot.:

RUN export HTTP_PROXY=username:password@proxy:80 && export http_proxy=http://username:password@proxy:80 && \
git clone https://github.com/xdebug/xdebug.git /tmp/xdebug \
    && mkdir -p /usr/src/php/ext \
    && mv /tmp/xdebug /usr/src/php/ext/ \
    && docker-php-ext-install xdebug

These are the results with different ways of setting the proxy environment variables.:

Fails with encoded ! (%21) in password
i.e.: HTTP_PROXY=http://username:password%211234@proxy:80
Works with ! in password
i.e.: HTTP_PROXY=http://username:password!1234@proxy:80
Fails with single quoted encoded !(%21) in password
i.e.: HTTP_PROXY='http://username:password!1234@proxy:80'
Fails with different error (Couldn't resolve proxy '(nil)') with single quoted !
i.e.: HTTP_PROXY='http://username:password!1234@proxy:80'
Fails with double quoted encoded ! (%21)
i.e.: HTTP_PROXY="http://username:password!1234@proxy:80"
Works with double quoted !
i.e.: HTTP_PROXY="http://username:password!1234@proxy:80"
Works with ! as last character in password
i.e.: HTTP_PROXY=http://username:password!@proxy:80
Works with encoded ! (%21) as last character
i.e.: HTTP_PROXY=http://username:password%21@proxy:80
Works with alpha numeric password
i.e.: HTTP_PROXY=http://username:password123@proxy:80

In my test, I don't see git using those environment variables at all. Can you provide more of your Dockerfile so I can reproduce? I am assuming it is FROM php:alpine?

Those environment variables are needed because I have to go through a proxy to get internet access. Here is a Dockerfile, almost identical to what I am using.: https://github.com/drupalwxt/site-wxt/blob/8.x/docker/images/dev/Dockerfile
The way I understand the problem is that when going through a proxy (and configuring it with the environment variables) with a password that has a bang. Git and apk don't handle this the same as other linux commands (like wget or curl).

Any updates on this?

I'm experiencing this issue and its super frustrating; this is the first Linux distro where this has been a problem.  Our proxy requires authentication credentials, and our service account passwords often have @ signs in them, thus making it impossible to use if we cannot encode the special characters.  Every other tool we've used has had no problem with encoded characters in the proxy var.

Finally I solved this corporate proxy chore. I'm on windows 10.

Set windows environment var (this is optional, otherwise you need to type it out in docker build)

HTTP_PROXY_AUTH=basic:*:user:password
(password is without html encoding for special char)
HTTP_PROXY=http://user:password@proxy:port
HTTPS_PROXY=http://user:password@proxy:port
(password is with html encoding for special char)

You can check if these are set by typing $HTTP_PROXY, etc in shell.

Inside docker file:

ARG proxy
ARG auth
ENV http_proxy=$proxy https_proxy=$proxy
ENV HTTP_PROXY_AUTH=$auth

Add this to "docker build" command:
--build-arg proxy=$HTTP_PROXY --build-arg auth=$HTTP_PROXY_AUTH

Having the same issues as @brettjacobson with different apps handling proxy encoding differently, more specifically apk not supporting encoded characters.

Any update? Having the same problem here ... :/

I haven't seen this problem recently, I am assuming this got fixed (probably upstream) sometime over the last few years.

I faced this issue.
dockerfile:
FROM golang:alpine as builder
...
ENV http_proxy "$http_proxy"
ENV https_proxy "$https_proxy"
ENV HTTP_PROXY "$http_proxy"
ENV HTTPS_PROXY "$https_proxy"
RUN apk add --update --virtual build-dependencies build-base linux-headers
...
error log:
fetch https://dl-cdn.alpinelinux.org/alpine/v3.13/main/x86_64/APKINDEX.tar.gz
ERROR: https://dl-cdn.alpinelinux.org/alpine/v3.13/main: Permission denied
WARNING: Ignoring https://dl-cdn.alpinelinux.org/alpine/v3.13/main: No such file or directory
fetch https://dl-cdn.alpinelinux.org/alpine/v3.13/community/x86_64/APKINDEX.tar.gz
ERROR: https://dl-cdn.alpinelinux.org/alpine/v3.13/community: Permission denied
WARNING: Ignoring https://dl-cdn.alpinelinux.org/alpine/v3.13/community: No such file or directory

hi, to run the command make it work,
sed -i -e 's/https/http/' /etc/apk/repositories

I have the same problem and was not able to fix it.
This is not really a fix, but using GNU wget works as expected, so it seems to be an issue with busy box wget: https://github.com/mirror/busybox/issues/21


Remove extraneous code from CraneEngine

Turns out that we were making an extra call to io.Copy that is completely unnecessary. We can use the io.Reader directly from crane's mutate.Extract call.

coverage: 84.413% (-0.03%) from 84.446%
when pulling 160e2323b581154b0239d0ceb801a38d720071b6 on bcrochet:remove-extra-code
into 0184783e920af6c646444b76229f27bba189e024 on redhat-openshift-ecosystem:main.

I'm a bit confused why we're losing code but also losing coverage here.


Cargo.toml: Git Dependencies

When I try to build a package with this in the Cargo.toml:
[dependencies]
rocket = { git = "https://github.com/SergioBenitez/Rocket.git", rev = "1f1f44f336e5a172361fc1860461bb03667b1ed2", features = ["tls"] }

I get this error message:
fetching Git repository 'https://github.com/SergioBenitez/Rocket.git'fatal: couldn't find remote ref refs/heads/1f1f44f336e5a172361fc1860461bb03667b1ed2
error: --- Error ------------------------------------------------------------------------------------- nix
program 'git' failed with exit code 128
(use '--show-trace' to show detailed location information)

But the rev seems to exist.
If you want I can also attach the --show-trace.

I have a similiar issue. Those are the commands it tries to execute in this case:
nix-build        29705  16611    0 /scratch/joerg/.nix-profile/bin/nix-build --show-trace
git              30279  29705    0 /nix/store/cdmxpgjyx8nkxyf64rjvpazpi6p5dslb-git-2.30.0/bin/git -C /scratch/joerg/.cache/nix/gitv3/01lpsaj1cwwnf8a3nh0wglq0jpqc1dazg5864mvw47gdf3nvw8vl cat-file -e 5847f1286492b7191f1400e6647fb220f8941f89
git              30280  29705    0 /nix/store/cdmxpgjyx8nkxyf64rjvpazpi6p5dslb-git-2.30.0/bin/git -C /scratch/joerg/.cache/nix/gitv3/01lpsaj1cwwnf8a3nh0wglq0jpqc1dazg5864mvw47gdf3nvw8vl fetch --quiet --force -- https://github.com/rust-vmm/vm-device refs/heads/5847f1286492b
7191f1400e6647fb220f8941f89:refs/heads/5847f1286492b7191f1400e6647fb220f8941f89
git              30281  30280    0 /nix/store/cdmxpgjyx8nkxyf64rjvpazpi6p5dslb-git-2.30.0/libexec/git-core/git remote-https https://github.com/rust-vmm/vm-device https://github.com/rust-vmm/vm-device
git-remote-http  30282  30281    0 /nix/store/cdmxpgjyx8nkxyf64rjvpazpi6p5dslb-git-2.30.0/libexec/git-core/git-remote-https https://github.com/rust-vmm/vm-device https://github.com/rust-vmm/vm-device

The fetch command fails in a reproducible way:
 /nix/store/cdmxpgjyx8nkxyf64rjvpazpi6p5dslb-git-2.30.0/bin/git -C /scratch/joerg/.cache/nix/gitv3/01lpsaj1cwwnf8a3nh0wglq0jpqc1dazg5864mvw47gdf3nvw8vl fetch --quiet --force -- https://github.com/rust-vmm/vm-device refs/heads/5847f1286492b7191f1400e6647fb220f8941f89:refs/heads/5847f1286492b7191f1400e6647fb220f8941f89
fatal: couldn't find remote ref refs/heads/5847f1286492b7191f1400e6647fb220f8941f89

Thank you! This fixed the part of:
rocket = { git = "https://github.com/SergioBenitez/Rocket.git", rev = "1f1f44f336e5a172361fc1860461bb03667b1ed2", features = ["tls"] }

But sadly I seem to have a similar Issue with:
ruma = { git = "https://github.com/ruma/ruma", features = ["rand", "client-api", "federation-api", "unstable-pre-spec", "unstable-synapse-quirks", "unstable-exhaustive-types"], rev = "ee814aa84934530d76f5e4b275d739805b49bdef" }

If anyone has a similar issue. I am just patching the Cargo.toml file for now.
Ruma has main as a default branch, and if I specify it in the Cargo.toml everything works.

Yes. Different default branches are still a problem. The commit that was reverted tried to solve this, but broke the general case.

I'm also trying to build a project (conduit) which has a dependency on ruma using naersk.
It's failing for the same main-branch related reason. However, I'm unable to add branch = "main" to conduit's ruma entry in Cargo.toml as the latest commit on main is incompatible.
Do you have any ideas for how to address this?
(Sorry for resurrecting a closed thread and thanks for the awesome lib!)

@jcarrag,
I was also trying to build conduit. I changed the Cargo.toml in a very hacky way.
I used the branch of Ruma to point to a different repository that has the rev as the main branch.
I don't know of any other way to remedy this atm, and would also be happy if anyone has a more
elegant solution.


[next][List] Use flex layout instead of absolute positioning for ListItem

Problem
List item primary div may overflow the secondary action.
Solution
Respect the width of the secondary action and grow the primary to fill the space.
Details
Use flex layout instead of absolute positioning for automatic width growth of first item to meet the width of the secondary item.

[x] PR has tests / docs demo, and is linted.
[x] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[x] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

Review request on ListItem and ListItemSecondaryAction:
Now that flex layout is used, the usefulness of ListItemSecondaryAction is in question.  ListItem could be changed to see if there is more than one child, and if so, use the last one as the secondary action.  Caveat: This may be too magical/implicit.
The current use of ListItemSecondaryAction is explicit, so it may still be useful as-is.

Please don't merge - I merged this locally and some of my lists aren't displaying correctly.  I'll figure that out and add some cases for regression.

@oliviertassinari - this is good to merge/review.  I had a problem with my app code, added a regression baseline to verify.

@rosskevin That's your fifteenth great PR here ✨ . Thanks for your time!
Would you be interested in joining the core contributor group? You definitely deserve it.

@oliviertassinari - I would be happy to join the core contributor group!
I added the 8px, it is not an exact to the old baseline but I did some measuring in a variety of samples and it seems reasonable.  Please let me know if you think it needs further adjustment.

I would be happy to join the core contributor group!

I'm gonna ask @hai-cea 😄 .


Crop size is not calculated correctly

Describe the bug
Cropsize width is not calculated correctly

Cropper is inside of chakra-ui modal btw.
But when I resize chrome viewport, it works

Expected behavior
Cropsize width should take up all available space so that user can capture the whole width of the image

This is a known issue when displaying the Cropper inside a modal
Unfortunately, there isn't much we can do as part of the lib (unless someone proves me wrong).
You can have a look to similar issues such as #400 for workarounds.

Thank you, I'll take a look


Bug: Add buttons jumping around in QA module

Context
The add button jumps around in the QA module for appendix e heat from oil and appendix e heat input from gas. This issue was resolved for TEE data. Need to have further discussion with the team on the issue and how it can be fixed across all test types

Bug fixed


Hello

Is there any option inside bot to use Incense or lucky egg or troy disk automatically furthermore is there any option to make bot fight gems ?

for lucky egg there is. it just pops it as soon as the evolve script starts. not sure for incenses nor lure modules. but there must be. i've heard options to fight gyms automatically, but not sure if they have it here.

Official statement regarding the gyms.

@mms20102 you can alter the config.json to use Lucky eggs if available

thanks for information


Feature/company-employees/employees

Please update Describe label as we documented in ISSUE-TEMPLATE.md.


Use namespace to generate cf-operator.fullname

Motivation and Context
#175984792
Checklist:
Check item if necessary.

[ ] Review PRs of changed Quarks dependencies
[ ] Update this PR after the release of Quarks dependencies

This refers to https://github.com/cloudfoundry-incubator/quarks-operator/issues/1257


Add documentation about what the data are

Describe the solution you'd like
Add a description of the data housed in baytrendsmap
Additional context
Participants recommended at ad hoc modeling workgroup (7/14/2020) that description be added

https://github.com/tetratech/baytrendsmap/commit/60948d54f78e2b4a6f5d018f9f664eabc03b6f9d
Update to help and other description files for basic interface.

done


Not possible to verify unsigned token (signed with alg none)

Is it possible to verify (as in jwt.verify()) a token that was signed with algorithm none?
var payload =  {
    iss: 'https://example.com'
    aud: 'test.client.id'
};
var signedToken = jwt.sign(payload, null, { algorithm: 'none' });

// First try with empty key, causes 'secret or public key must be provided'
jwt.verify(signedToken, null, { audience: 'test.client.id', issuer: 'https://example.com' }, ...);

// Second try with a dummy key, causes 'jwt signature is required'
jwt.verify(signedToken, '<dummy key>', { audience: 'test.client.id', issuer: 'https://example.com' }, ...);

I expected the first version to do what I want, which is to realize that the token does not have a signature and therefore doesn't require a signature validation and hence not a key either.

The verify method options support algorithms which is an array of supported algs

Thanks for the suggestion. Unfortunately, using options.algorithms didn't help:
// Third attempt, also causes 'secret or public key must be provided'
jwt.verify(signedToken, null, { algorithms: ['none'] }, ...);

I had a look at the jwt.verify() implementation and it does not seem to handle unsigned JWTs correctly. This is the snippet of code where the problem occurs:
  if (parts[2].trim() === '' && secretOrPublicKey){
    return done(new JsonWebTokenError('jwt signature is required'));
  }

  if (!secretOrPublicKey) {
    return done(new JsonWebTokenError('secret or public key must be provided'));
  }

If the token does not have a signature, the method will always return with an error. One way to fix this would be to add an additional condition to the second if statement:
  if (parts[2].trim() !== '' && !secretOrPublicKey) {
    return done(new JsonWebTokenError('secret or public key must be provided'));
  }

This change allows jwt.verify() to successfully verify an unsigned token, assuming you pass { algorithms: ['none'] }. It does not break any of the existing test cases, but I am not sure it is correct.

Yes! Sounds like a good plan
El El jue, mar 17, 2016 a las 6:16 PM, Jon Ekdahl<EMAIL_ADDRESS>escribió:

I had a look at the jwt.verify() implementation and it does not seem to
handle unsigned JWTs correctly. This is the snippet of code where the
problem occurs:
if (parts[2].trim() === '' && secretOrPublicKey){
return done(new JsonWebTokenError('jwt signature is required'));
}
if (!secretOrPublicKey) {
return done(new JsonWebTokenError('secret or public key must be provided'));
}
If the token does not have a signature, the method will always return with
an error. One way to fix this would be to add an additional condition to
the second if statement:
if (parts[2].trim() !== '' && !secretOrPublicKey) {
return done(new JsonWebTokenError('secret or public key must be provided'));
}
This change allows jwt.verify() to successfully verify an unsigned token,
assuming you pass { algorithms: ['none'] }. It does not break any of the
existing test cases, but I am not sure it is correct.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/auth0/node-jsonwebtoken/issues/185#issuecomment-198085544

Okay, great. I updated the title to reflect that this is more of a defect than a question.


Animate when position changes

Hi, I'm new to flutter and would like to have an animation when withGridPlacement changes position.
Is this possible with this library?

Hey @geovannimp,
Sorry this took me so long. I've finally got time to work on this again.
The answer is no, that isn't currently possible...but from a technical standpoint it could be done.
Do you have an example of an effect similar to the one you're looking for, so I can get a feel for it?

Hey @geovannimp,
Sorry this took me so long. I've finally got time to work on this again.
The answer is no, that isn't currently possible...but from a technical standpoint it could be done.
Do you have an example of an effect similar to the one you're looking for, so I can get a feel for it?

I don't actually remember anymore what I was trying to do, I think was reorganize the grid with drag and drop with itens that could change size.
I ended using a fadein/fadeout animation and placing two grids over each other to simulate the transition animation.

I don't actually remember anymore what I was trying to do, I think was reorganize the grid with drag and drop with itens that could change size.
I ended using a fadein/fadeout animation and placing two grids over each other to simulate the transition animation.

Awesome. Looks really good.
I'm going to close this off for now, but I'll keep it in mind in case someone else has the same ask.

Awesome. Looks really good.
I'm going to close this off for now, but I'll keep it in mind in case someone else has the same ask.


Support custom type/struct IN parameters

Description:
ATM we only support OUT params for structs. Need to $subejct

This is already supported in Swan lake dependant implementation. We can pass any TypedValue which includes Struct too as IN param


hugo import jekyll fails for jekyll projects with nested _posts directories

With modern Jekyll, it's possible to nest _posts directories deeper in the hierarchy:
.
├── _config.yml
├── blog
|   ├── index.html
|   ├── _posts
|       └─── 2016-02-23-a-post-to-my-blog.md
├── photoblog
|   ├── _index.html
|   ├── _posts
|       └─── 2016-02-23-a-photo-post.md
└── index.html

Hugo's import jekyll doesn't appear to know about this, and fails to import these posts correctly.

Just to add a few notes:
Jekyll appears to look for _posts and _drafts in all directories it walks, so they can be nested arbitrarily deep. The superdirectory names above _posts appear to become part of of the post's categories. Other than that, a post's final resting location in the generated _site seems to depend only on the permalink setting, not its location in the source directory structure.
(There's also a curious comment in jekyll's categories_from_path implementation that seems to suggest there is some other meaning to subdirectories of a _posts directory. I've no idea what this is; might be of interest.)
Hugo's importer only looks for top-level _posts and _drafts directories. There's a split between that code (which imports posts) and the code that blindly copies what it believes to be static content; it seems like this needs to be restructured. Finally, I'm not sure I understand what the importer is doing with permalink but it might also need to be revisited.

I took a quick stab at this, here: https://github.com/davepeck/hugo/commit/d6e225af7f0143fc3e5a5778886a8c84eb3ab764
Not suitable for a pull request; just a quick hack that might help someone.

@coderzh there are couple of open issues/feature requests about the Jekyll importer now... Would this be something that you could have a look at?

@bep I'd be glad to do something about these issues.

@davepeck do you know any in-the-open (on GitHub) Jekyll sites with the "symptoms" of this and may the other issue(s) you have posted, that we could use to test this?

I humbly offer my own personal blog as an example: https://github.com/davepeck/davepeck.org
I wonder if a cleverly crafted github search could also get us there?
Sorry I haven't had time to contribute code myself on this but hope at least the issues/comments were helpful.

I humbly offer my own personal blog as an example

And it is > 1 GB ... Could you take the PR referenced here for a spin and tell us if that works for you?

This issue has been automatically marked as stale because it has not been commented on for at least four months.
The resources of the Hugo team are limited, and so we are asking for your help.
If this is a bug and you can still reproduce this error on the master branch, please reply with all of the information you have about it in order to keep the issue open.
If this is a feature request, and you feel that it is still valuable, please open a proposal at https://discuss.gohugo.io/.
This issue will automatically be closed in four months if no further activity occurs. Thank you for all your contributions.

Note/Update: This issue is marked as stale, and I may have said something earlier about "opening a thread on the discussion forum". Please don't.
If this is a bug and you can still reproduce this error on the latest release or the master branch, please reply with all of the information you have about it in order to keep the issue open.
If this is a feature request, and you feel that it is still relevant and valuable, please tell us why.


Index out of range Exception for less number of rows or columns

@kishikawakatsumi  Hi again, I have used Schedule app logic.  I have 4 columns and 3 rows. I am rendering the data array by transposing it. For eg. By your logic, [''1", "5", "9"] are placed in first row, but after transposing data array  ["1", "2", "3","4"] is placed in first row.
let data = [
        ["1", "2", "3","4"],
        ["5", "6", "7", "8"],
        ["9", "10", "11", "12"]
        ]

Logic for Transpose:
public func transpose<T>(input: [[T]]) -> [[T]] {
        if input.isEmpty { return [[T]]() }
        let count = input[0].count
        var out = [[T]](repeating: [T](), count: count)
        for outer in input {
            for (index, inner) in outer.enumerated() {
                out[index].append(inner)
            }
        }

        return out
    }

When I reduce number of rows or columns, then I get "fatal error: Index out of range" exception on this line   let text = result[indexPath.column-1][indexPath.row-1].
For eg.
["1", "2", "3"],
        ["5", "6", "7"],
        ["9", "10", "11"],
        ]

or
["1", "2", "3", "4"],
        ["5", "6", "7", "8"],
        ]

#Expected Behaviour
The remaining rows or columns should have "-" when no data is present. Please help.
I am using swift 3, Xcode 8.2.1

@ikyh Can you share your project? Your question is very unclear and you do not seem to understand the correct usage.

@ikyh SpraedsheetView's API is the same as UITableView/UICollectionView's manner. If there is any change in the data source, you must tell the view of the change using reloadData().

@kishikawakatsumi Hello..! It does not work. I have 5 columns but the grid data I am rendering to columns has 4 elements in the array. I am still getting Index out of bounds exception.


Don't use abstract go-perun types like wallet.Address in the API

This is an eth-specific wrapper, so it should hide abstractions. Client.OnChainBalance currently uses wallet.Address in its parameters. Didn't check if there are other abstract types visible that could be concretized.

I replaced it with a common.Address.
We also use wallet.Address and wire.Address in some cases. They could all be changed/should all be changed to common.Address. This would require some wrapper code.

To be honest, I don't immediately see the benefit of using wallet.Address over common.Address.
@sebastianst Can you give an example?

One of the purposes of this eth payment api is to hide abstract types like wallet.Address. It is just confusing for the user, particularly because most of the api already uses common.Address.

Right now, the API mixes up the use of concrete types like common.Address and the abstract wallet.Address, which is not clean. A new user would be confused how to even pass a common.Address for this wallet.Address, because they would need to find out how to convert it.
Moreover, a function should only ask in its parameters for what it can handle. And the functions in this package can actually only handle common.Addresses and not any other implementation of wallet.Address, so it provides more type safety to accept only the concrete type.


Accept: Assign DM Entries to Characters

Description
Acceptance Test for #111
Ability to read, and create DM entries belonging to a particular user
UAT Environment for executing the acceptance flow
See #315 for automation of this flow
Acceptance flow

User Logs in
User Navigates to the Character index
User clicks "DM Entries"
User chooses an existing DM entry (or opts to create a new one)
User assigns the entry to a chosen character and chooses a reward type
User clicks save
User observes the item / level up corresponding the the reward chosen on the character they assigned the entry too

Everything works perfectly! Approved!
-Kate Ruscito


[feature] add rcan model for remote sensing image super-resolution

使用RCAN模型，对遥感影像进行超分重建，x4

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[TimePicker] Use clock icon when editing in mobile mode

[x] I have followed (at least) the PR section of the contributing guide.

The time picker toolbar used the calendar icon instead of the clock icon for the button to switch away from the text input view, which was confusing. This replaces the icon on the time picker as well as the aria label to match the actual use of the component.
Steps to reproduce:
Insert a TimePicker
Open TimePicker in mobile mode
Click the pen icon to go to the keyboard edit view
The icon to return to the clock view is a calendar. This is confusing.

Could you include steps to reproduce? I'm not sure what you're fixing in this PR.

Steps to reproduce added

Will revisit the test when we add a more comprehensive visual regression test suite for all things picker related.

@alcwhite Thanks!


Additional cache: gocache

https://github.com/golang/groupcache

https://github.com/buraksezer/olric ?

https://github.com/XenitAB/azad-kube-proxy/issues/589


Debugging issue with graceful-fs (<4.1.0): "Code not running"

While debugging a program, that uses fs-extra, I stumbled upon the following message:

Instead of hitting the breakpoint, the above message is shown. After some investigation, it turned out that node-graceful-fs is the reason, which is used by fs-extra. Specifically, any breakpoint in the callback of the mkdir function of graceful-fs is not hit.
Example code (breakpoints on both console.log commands):
var fs = require("graceful-fs");

console.log('Hello world'); // breakpoint is hit

fs.mkdir("C:/Users/AIT/Desktop/t2", 511, function (err) {
    console.log("hello"); // debugging is paused, but "Code not running" message is shown
});

At that time, it used the graceful-fs version 3.0.8. Now they changed the implementation and with version 4.1.0, everything works fine. However, whatever they did there, prevented debugging with the Node Tools, so you might still want to look into that issue.
I suspect that it is related to what they did in fs.js at that time, see the file of version 3.0.8

Still a problem in NTVS 1.2 Alpha with that version of Graceful FS. Not a problem with 4.x.
Seems like this logic from fs.js could be a cause (4.x doesn't use vm in this file, but I only took a quick glance at the files):
var mod = require("module")
var pre = '(function (exports, require, module, __filename, __dirname) { '
var post = '});'
var src = pre + process.binding('natives').fs + post
var vm = require('vm')
var fn = vm.runInThisContext(src)
fn(exports, require, module, __filename, __dirname)

Closing, since this is no longer a problem with current versions of Node and Graceful FS


Feature Request: Customizable Folder Layout

Unifi Protect Backup version: docker:latest
Python version: docker
Operating System: docker

Description
Presently, the backup script syncs with a folder structure of /{camera_name}/{date}/video_file. While this is great from an archival purpose, it's less ideal for other usages (correlating events, etc).
I'd love an option to flatten the structure by date and potentially time. So something like: /{date}/{hour}/{YYYY-MM-DDTHH:mm:ss}_{camera_name}_{event_type}.mp4 or potentially without the hour-resolution sub-folder.
One alternative to this would be to templatize the naming format with the appropriate variables to allow users to customize the filesystem layout.

Hi, thanks for the feature request. I've taken a stab at allowing the file path to be templated and this is available on the dev branch/container depending on how you use the tool. I would appreciate if you could test it out and see if its suitable.
Docs can be found here: https://github.com/ep1cman/unifi-protect-backup/tree/dev#file-path-formatting

Successfully implemented and working well. Thanks for the quick implementation!
One note, the ENV var for the docker container isn't documented. Could be worth tossing FILE_STRUCTURE_FORMAT into the docs for it as well. Otherwise, working well!

It is listed in the README.md file, are you referring to the fact its not in the dockerfile?

Whoops, I had missed that. I looked for it and didn't notice, so went and looked at the commit to find it. It was there all along.
Thanks again for the support, and the rapid turn around here! I'll go ahead a close since this is implemented. Thanks!

I will create a new release with this and the fixes for your other issue shortly


Limited set of Actions and Bindings are hardcoded

This relates to https://github.com/GodotVR/godot_openvr/issues/71 and https://github.com/GodotVR/godot_openvr/pull/80
Currently godot_openxr hardcodes "trigger", "grab", "menu" and "handpose" actions. For godot 3.x we probably want to leave those actions hardcoded and create a unified UI for creating and binding actions with godot 4.x.
The plugin code also hardcodes bindings for the interaction profiles "/interaction_profiles/khr/simple_controller", "/interaction_profiles/valve/index_controller" and "/interaction_profiles/mnd/ball_on_stick_controller" which has essentially the inputs of a Playstation Move controller.

We should have bindings for at least Oculus Touch, WMR and HTC Vive Controller. Interaction Profiles with available inputs for each controller are documented in the spec https://www.khronos.org/registry/OpenXR/specs/1.0/html/xrspec.html#semantic-path-interaction-profiles
It would be nice to be able to add and change bindings without having to recompile the plugin. Perhaps put them in a text file like with OpenVR?

Unlike with OpenVR, there (currently) is no file format for actions and bindings that can be given to a runtime. For 2. we would have to define our own file format and have our own parser. We could also choose to write a parser for OpenVR manifests and reuse those, but they do not map perfectly to OpenXR.
For those unfamiliar with the OpenXR action system:
Like with OpenVR, an application creates "actions" which is an arbitrary name with a type, for example you might create a jump boolean action and then later querying the state of this action results in a true or false value.
Then the application can suggest bindings for those actions which are supposed to be sane defaults for a set of known controllers, for example you might say "If a Valve Index Controller is used, the a button should be bound to the jump action. If a HTC Vive Controller is used, the touchpad click should be bound to the jump action" and so on.
If the user has for example an Oculus Touch controller and the application didn't suggest actions for this controller, then it is not exactly specified what a runtime has to do. Maybe the runtime engages a compatibility mode where it knows how to map for example Index Controller bindings to an Oculus Touch Controller. Maybe the runtime pops up a binding dialog with all the actions that the application suggested and requires the user to map the inputs of the current controller to those actions. Etc.
Either way runtimes are free to allow remapping all of the suggested bindings as they like with UIs like the one found in SteamVR. For example a user might not like that the application developer suggested binding the jump action to the a button of their Valve Index Controller, and uses the runtime's binding UI to map the jump action to the b button instead, without the application knowing that the source of the jump action is not what they suggested (if the application really wants, it can find out with xrEnumerateBoundSourcesForAction()).

I'm going to be working on a lot of this soon and looking into how we can better integrate this with Godot itself.


Add The Graph subgraph-package tutorial

変更内容
背景
備考

@kmjones1979
Thank you for creating the tutorial !
I would like to inform you that the base branch will be changed from main to feature/add-thegraph. This is to review the content of the documents and add Japanese text in the feature/add-thegraph branch before finally merging it into main.

Thank you! I will be creating an additional one soon :) Happy to help!


Update default box shadows

This PR updates the default box shadows to new values carefully chosen by @sschoger and I:

It also adds two new shadow sizes, xl, and 2xl.
This is a breaking change for anyone depending on the default theme, but because we have encouraged people to publish the entire default config historically, it won't really be a big deal in practice.

Worked with Steve to improve the shadows a bit more, this time incorporating a progressive negative spread to better convey an increase in elevation:


Run SonarCloud only on pull requests to master

Run SonarCloud less often to reduce build times for normal CI builds.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: tencreek sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Not working with EnigmaticThunder API

It does, atleast it's working for me
Do you have MMhook as well?

Yes I do
On Tue., Mar. 30, 2021, 11:07 p.m. TheNoble, @.***>
wrote:

It does, atleast it's working for me
Do you have MMhook as well?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TheMysticSword/MysticsItems/issues/87#issuecomment-810723665,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOISKFSG4MQEPTRCRE7QPQDTGKGVNANCNFSM42DQQL2A
.

I have the same problem. Either with just MMHOOK or MMHOOK and EnigmaticThunder enabled. No other mods, and the items are simply not present in the game, or the logbook.

Can you provide an error log? Maybe I can get this fixed in today's patch.

I will be able to send a log later today
On Wed., Mar. 31, 2021, 12:35 p.m. David Galiev, @.***>
wrote:

Can you provide an error log? Maybe I can get this fixed in today's patch.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TheMysticSword/MysticsItems/issues/87#issuecomment-811222859,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOISKFSYDIOBZDFFNMDO4DLTGNFMFANCNFSM42DQQL2A
.

EnigmaticThunderAPI is no longer used. Closing.